-
Notifications
You must be signed in to change notification settings - Fork 13.4k
ci: move dist-x86_64-linux job to codebuild #141501
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
ci: move dist-x86_64-linux job to codebuild #141501
Conversation
@bors try |
…r=<try> ci: move dist-x86_64-linux job to codebuild
💔 Test failed - checks-actions |
@bors try |
…r=<try> ci: move dist-x86_64-linux job to codebuild
Did you configure the environment for both try and auto branches on Codebuild? |
☀️ Try build successful - checks-actions |
I didn't have to configure branches as far as I remember |
@rustbot ready |
Ok, let's try. @bors r+ rollup=never |
…r=Kobzol ci: move dist-x86_64-linux job to codebuild
💔 Test failed - checks-actions |
maybe this was due to the github incident. 🤔 |
@bors p=6 (ahead of rollups) |
…r=Kobzol ci: move dist-x86_64-linux job to codebuild
The dist-x86_64-linux job doesn't appear to be starting |
yes, you are right, I canceled the workflow. I'll have a look tomorrow |
💔 Test failed - checks-actions |
rust-lang/simpleinfra#720 probably fixed the issue. |
Sure, feel free to r=me. |
I haven't seen that error anymore. Let's try again @bors r+ |
💡 This pull request was already approved, no need to approve it again.
|
@bors r=Kobzol |
💡 This pull request was already approved, no need to approve it again.
|
I forgot to r=Kobzol initially. |
☀️ Test successful - checks-actions |
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing cb678b9 (parent) -> 77101fe (this PR) Test differencesNo test diffs found Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 77101febcc9662a076ff43887497cd5b30674d93 --output-dir test-dashboard And then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
Finished benchmarking commit (77101fe): comparison URL. Overall result: ✅ improvements - no action needed@rustbot label: -perf-regression Instruction countThis is the most reliable metric that we have; it was used to determine the overall result at the top of this comment. However, even this metric can sometimes exhibit noise.
Max RSS (memory usage)Results (primary 1.9%, secondary 3.2%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResults (secondary -2.0%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 778.889s -> 777.676s (-0.16%) |
Related to rust-lang/infra-team#188
This job was already in codebuild, but in #141388 we moved it back to GitHub Actions to do the rust-lang-ci migration easily.
This PR moves it back to codebuild. It also moves the try job.
r? @Kobzol