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][Cypress][Bug] use default SOURCE inputs for checkout #5203

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport dc6a7ec from #5197.

#5134

Incorrectly uses the source of the code to pull down to BASE REF,
which would likely be `main` or `2.x`, etc.

It should be pulling down the PR branch. Cypress tests at the time
of merging were failing due to unrelated issue of disk allocation.

Not setting the env variables causes the workflow to rely on the default
values which is a return back to the original implementation and if the
env is set then it will be not empty.

Signed-off-by: Kawika Avilla <[email protected]>
(cherry picked from commit dc6a7ec)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot added autocut Skip the changelog verification check on backports Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry labels Oct 4, 2023
@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #5203 (9514ff2) into 2.x (31e91ef) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.x    #5203   +/-   ##
=======================================
  Coverage   66.80%   66.80%           
=======================================
  Files        3278     3278           
  Lines       63042    63042           
  Branches    10028    10028           
=======================================
+ Hits        42116    42118    +2     
+ Misses      18461    18437   -24     
- Partials     2465     2487   +22     
Flag Coverage Δ
Linux_1 35.32% <ø> (ø)
Linux_2 55.26% <ø> (ø)
Linux_3 43.76% <ø> (+<0.01%) ⬆️
Linux_4 35.55% <ø> (ø)
Windows_1 35.33% <ø> (ø)
Windows_2 55.23% <ø> (ø)
Windows_3 43.76% <ø> (ø)
Windows_4 35.55% <ø> (ø)

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

see 17 files with indirect coverage changes

@kavilla kavilla merged commit 9b1d128 into 2.x Oct 4, 2023
75 of 76 checks passed
@github-actions github-actions bot deleted the backport/backport-5197-to-2.x branch October 4, 2023 09:53
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 Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry v2.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants