[ci] Fixes Bazel cache writes#114915
Merged
tylersmalley merged 3 commits intoelastic:masterfrom Oct 14, 2021
Merged
Conversation
tylersmalley
force-pushed
the
bazel-write-on-merge
branch
6 times, most recently
from
2021789 to
af3e058
Compare
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
tylersmalley
force-pushed
the
bazel-write-on-merge
branch
from
af3e058 to
3056372
Compare
tylersmalley
changed the title
tylersmalley
added
Team:Operations
Contributor
|
Pinging @elastic/kibana-operations (Team:Operations) |
spalger
reviewed
Oct 13, 2021
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
spalger
reviewed
Oct 13, 2021
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Member
Author
|
Pushed to upstream to kick off a test here: https://buildkite.com/elastic/kibana-on-merge/builds/5487#f85278dc-065e-419c-9848-19a8b1cd0965 and validate the build invocation on Buildbuddy. Included the screenshot in the description showing it's now uploading. |
Contributor
|
Where is the public key getting added? Maybe I'm just not seeing it because I'm on my phone, but I don't see it |
Contributor
|
Never mind, I see, my bad! |
brianseeders
approved these changes
Oct 13, 2021
Contributor
💚 Build Succeeded
Metrics [docs]
To update your PR or re-run it, just comment with: |
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Oct 14, 2021
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Contributor
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
kibanamachine
added a commit
that referenced
this pull request
Oct 14, 2021
jloleysens
added a commit
to jloleysens/kibana
that referenced
this pull request
Oct 14, 2021
…mple/introduce-baseline-tests * 'master' of github.com:elastic/kibana: (55 commits) [Fleet] Improve Functionality around Managed Package Policies (elastic#114526) cleanup (elastic#114902) remove stray semicolon (elastic#114969) [Security Solution] Edit host isolation exception IP UI (elastic#114279) [ML] APM Correlations: Round duration values to be used in range aggregations. (elastic#114833) [Index Management] Added `data-test-subj` values to the index context menu buttons (elastic#114900) [Stack monitoring] Fix logstash functional tests for react (elastic#114819) Implement hybrid approach to writing rule execution event logs (elastic#114852) [Detection Rules] Add 7.16 rules (elastic#114939) Fixing exceptions export format (elastic#114920) Clean up inaccurate comments (elastic#114935) chore(NA): fixes a typo on persist_bazel_cache.sh comment (elastic#114943) [ci] Fixes Bazel cache writes (elastic#114915) fix package.json: (elastic#114936) [Controls] Redux Toolkit and Embeddable Redux Wrapper (elastic#114371) [APM] Fixes incorrect index config names (elastic#114901) (elastic#114904) [Workplace Search] Fix button order and remove extra source name label (elastic#114899) [Actions] Fixed actions telemetry for multiple namespaces usage (elastic#114748) docs: fix config names (elastic#114903) Update kibana to EMS 7.16 (elastic#114865) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
After #114035, we are no longer writing Bazel cache to Buildkite.
We were specifying an API key in the general
.bazelrcfile, and setup_bazel.sh is attempting to overwrite that value. However, that doesn't work like other options and we end up with a read-only API key.Writing the
.bazelrcto the home directory was confusing. Instead, we're just overwriting the file on CI when we intend to write to the cache.