Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] [CI] allow for manual triggered cypress test jobs to accept inputs #5140

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 34a8594 from #5134.

Utilizing the GitHub `workflow_dispatch`, we can trigger
a job with a specific inputs.

By default, we run from this repo:
https://github.com/opensearch-project/opensearch-dashboards-functional-test

This workflow uses the target branch of the PR to pull down
from the FTRepo and run the tests from that specific branch.

For example, PRs against `main` will pull down `main` from
the FTRepo.

The problem occurs when new functionality is opened or a bug is
fixed and, per industry standard, we want to see tests added
to ensure functionality, stability, and raise the bar.

However, the cypress tests PR into the FTRepo depends on the new code
to be merged for the CI within the FTRepo to work. So we have a
stalemate that usually slows down PR review time.

Here, a manual run can be triggered by a maintainer. It will utilize
the source branch from the PR if provided.

Related to:
#4019

Signed-off-by: Kawika Avilla <[email protected]>
(cherry picked from commit 34a8594)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

Merging #5140 (bd00dba) into 2.x (7ae32d6) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x    #5140      +/-   ##
==========================================
- Coverage   66.48%   66.47%   -0.01%     
==========================================
  Files        3402     3402              
  Lines       64982    64982              
  Branches    10374    10374              
==========================================
- Hits        43201    43199       -2     
- Misses      19218    19219       +1     
- Partials     2563     2564       +1     
Flag Coverage Δ
Linux_1 34.87% <ø> (ø)
Linux_2 55.19% <ø> (ø)
Linux_3 44.38% <ø> (-0.01%) ⬇️
Linux_4 35.05% <ø> (ø)
Windows_1 34.88% <ø> (ø)
Windows_2 55.15% <ø> (ø)
Windows_3 44.38% <ø> (ø)
Windows_4 35.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

@abbyhu2000 abbyhu2000 merged commit 8a3e2ad into 2.x Oct 2, 2023
70 of 72 checks passed
@github-actions github-actions bot deleted the backport/backport-5134-to-2.x branch October 2, 2023 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants