-
Notifications
You must be signed in to change notification settings - Fork 2.1k
buildkite: fix CPU template pipeline issues #4808
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
Conversation
13977c0
to
cee778c
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4808 +/- ##
=======================================
Coverage 84.34% 84.35%
=======================================
Files 249 249
Lines 27504 27504
=======================================
+ Hits 23199 23200 +1
+ Misses 4305 4304 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
187b7ca
to
8f13ad2
Compare
7bf3443
to
b9c4484
Compare
ccd7817
to
0a8423b
Compare
19e9051 assumed the `agents` is a dict, which was true in all cases except in one pipeline. Fixes: 19e9051 Signed-off-by: Pablo Barbáchano <[email protected]>
If we have more than one build step in a pipeline, Buildkite complains that they are not unique. fatal: Failed to upload and process pipeline: Pipeline upload rejected: The key "build_x86_64" has already been used by another step in this build We had a similar issue with shared build tarballs, so just reuse the tarball name as key. Fixes: 19e9051 Signed-off-by: Pablo Barbáchano <[email protected]>
0a8423b
to
84b2c8d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry!
19e9051 assumed the
agents
is a dict, which was true in all cases except in one pipeline.Fixes: 19e9051
Changes
...
Reason
...
License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license. For more information on following Developer
Certificate of Origin and signing off your commits, please check
CONTRIBUTING.md
.PR Checklist
PR.
CHANGELOG.md
.TODO
s link to an issue.contribution quality standards.
rust-vmm
.