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

[bk/flaky] fix env var for reusing artifacts #127503

Merged
merged 1 commit into from
Mar 10, 2022

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Mar 10, 2022

I'm guessing that BUILD_ID_FOR_ARTIFACTS is an old env var that we don't use anymore, but KIBANA_BUILD_ID is the env var expected by .buildkite/scripts/download_build_artifacts.sh so we should also gate the build step on this environment var.

@spalger spalger added Team:Operations Team label for Operations Team release_note:skip Skip the PR/issue when compiling release notes auto-backport Deprecated - use backport:version if exact versions are needed v8.1.0 v8.2.0 v7.16.3 v8.0.2 labels Mar 10, 2022
@spalger spalger requested a review from a team as a code owner March 10, 2022 19:16
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@spalger spalger requested a review from brianseeders March 10, 2022 19:16
@spalger spalger enabled auto-merge (squash) March 10, 2022 19:41
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@spalger spalger merged commit 62f8e58 into elastic:main Mar 10, 2022
@spalger spalger deleted the fix/flaky-build-artifacts-env branch March 10, 2022 20:39
kibanamachine pushed a commit that referenced this pull request Mar 10, 2022
kibanamachine pushed a commit that referenced this pull request Mar 10, 2022
kibanamachine pushed a commit that referenced this pull request Mar 10, 2022
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.1
7.16
8.0

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Mar 10, 2022
(cherry picked from commit 62f8e58)

Co-authored-by: Spencer <spencer@elastic.co>
spalger pushed a commit that referenced this pull request Mar 11, 2022
(cherry picked from commit 62f8e58)

Co-authored-by: Spencer <spencer@elastic.co>
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Mar 14, 2022
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@spalger spalger added v7.17.2 and removed v7.16.3 auto-backport Deprecated - use backport:version if exact versions are needed labels Mar 14, 2022
@spalger spalger added the auto-backport Deprecated - use backport:version if exact versions are needed label Mar 14, 2022
kibanamachine pushed a commit that referenced this pull request Mar 14, 2022
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
7.17

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Mar 14, 2022
(cherry picked from commit 62f8e58)

Co-authored-by: Spencer <spencer@elastic.co>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.17.2 v8.0.2 v8.1.0 v8.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants