Uh oh!
There was an error while loading. Please reload this page.
[CI] Rollup of 5 pull requests - #62147
Merged
Merged
Conversation
This commit turns off PR builds happening on Travis, instead entirely relying on Azure for PR builds to succeed.
The PR builder on Azure currently takes 2.5h which is a bit long, so this commit disables debug assertions and llvm assertions in an attempt to speed up that builder and have PR builds come back a bit more quickly. Other builders continue to enable debug assertions and test the compiler there.
This commit switches the `try` builers to officially happen on Azure Pipelines instead of Travis where they're currently run. This also cuts back the number of builders to just the two we run on Travis, leaving expansion as a possible future extension.
This commit moves toolstate publishing from Travis to Azure. We've been testing on azure for some time now and this works by deleting the Travis config and updating the credentials used on Azure.
Manually make sure that we do the same thing across all the services, uncovering one spot where we needed to pass one more configure flag on Azure but otherwise we're good to go!
…roalbini ci: Turn off PR builds on Travis This commit turns off PR builds happening on Travis, instead entirely relying on Azure for PR builds to succeed.
…troalbini ci: Disable assertions in PR builds The PR builder on Azure currently takes 2.5h which is a bit long, so this commit disables debug assertions and llvm assertions in an attempt to speed up that builder and have PR builds come back a bit more quickly. Other builders continue to enable debug assertions and test the compiler there.
ci: Switch official `try` builds to happen on Azure This commit switches the `try` builers to officially happen on Azure Pipelines instead of Travis where they're currently run. This also cuts back the number of builders to just the two we run on Travis, leaving expansion as a possible future extension.
ci: Publish toolstate changes from Azure This commit moves toolstate publishing from Travis to Azure. We've been testing on azure for some time now and this works by deleting the Travis config and updating the credentials used on Azure.
…bini ci: Sync AppVeyor/Travis with Azure configuration Manually make sure that we do the same thing across all the services, uncovering one spot where we needed to pass one more configure flag on Azure but otherwise we're good to go!
Centril
commented
Jun 26, 2019
ContributorAuthor
@bors r+ p=1000 rollup=never This is a CI only r0llup |
bors
commented
Jun 26, 2019
Collaborator
📌 Commit cd93888 has been approved by |
Centril
commented
Jun 26, 2019
ContributorAuthor
@bors treeclosed=1001 |
emilyalbini
commented
Jun 26, 2019
Member
@bors treeclosed- |
emilyalbini
commented
Jun 26, 2019
Member
@bors treeclosed- please |
Centril
commented
Jun 26, 2019
ContributorAuthor
@bors r+ p=1000 rollup=never |
bors
commented
Jun 26, 2019
Collaborator
💡 This pull request was already approved, no need to approve it again. |
bors
commented
Jun 26, 2019
Collaborator
📌 Commit cd93888 has been approved by |
Centril
commented
Jun 26, 2019
ContributorAuthor
@bors retry |
bors
commented
Jun 26, 2019
Collaborator
⌛ Testing commit cd93888 with merge dc395cf3965c56ceac508ff86bb7ba8f56b6413e... |
Centril
commented
Jun 26, 2019
ContributorAuthor
@bors treeclosed=999 |
bors
commented
Jun 26, 2019
Collaborator
💔 Test failed - checks-azure |
Centril
commented
Jun 26, 2019
ContributorAuthor
@bors retry Seems spurious. |
bors
commented
Jun 26, 2019
Collaborator
📌 Commit 6772ebc has been approved by |
bors
commented
Jun 26, 2019
Collaborator
bors added a commit
that referenced
this pull request
Jun 26, 2019
[CI] Rollup of 5 pull requests Successful merges: - #62140 (ci: Turn off PR builds on Travis) - #62141 (ci: Disable assertions in PR builds) - #62142 (ci: Switch official `try` builds to happen on Azure) - #62143 (ci: Publish toolstate changes from Azure) - #62145 (ci: Sync AppVeyor/Travis with Azure configuration) Failed merges: r? @ghost
bors
commented
Jun 26, 2019
Collaborator
💔 Test failed - checks-travis |
rust-highfive
commented
Jun 26, 2019
Contributor
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
alexcrichton
commented
Jun 27, 2019
Member
@bors: retry |
bors
commented
Jun 27, 2019
Collaborator
⌛ Testing commit 6772ebc with merge 6cc1835b38c77f6fd1e05a7d1bc69e6dfc624475... |
rust-highfive
commented
Jun 27, 2019
Contributor
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Centril
commented
Jun 27, 2019
ContributorAuthor
@bors retry linking with |
bors
commented
Jun 27, 2019
Collaborator
⌛ Testing commit 6772ebc with merge 14a25d10f53a0772571ccde07159dac7110678d8... |
alexcrichton
commented
Jun 27, 2019
Member
@bors: retry |
alexcrichton
commented
Jun 27, 2019
Member
er oops that was 30s too late, sorry! |
bors
commented
Jun 27, 2019
Collaborator
bors added a commit
that referenced
this pull request
Jun 27, 2019
[CI] Rollup of 5 pull requests Successful merges: - #62140 (ci: Turn off PR builds on Travis) - #62141 (ci: Disable assertions in PR builds) - #62142 (ci: Switch official `try` builds to happen on Azure) - #62143 (ci: Publish toolstate changes from Azure) - #62145 (ci: Sync AppVeyor/Travis with Azure configuration) Failed merges: r? @ghost
rust-highfive
commented
Jun 27, 2019
Contributor
Your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem. Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
alexcrichton
commented
Jun 27, 2019
Member
@bors: treeclosed- |
alexcrichton
commented
Jun 27, 2019
Member
@bors treeclosed=999 |
alexcrichton
commented
Jun 27, 2019
Member
@bors: treeclosed- I'm merging this manually. One builder is still running, |
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 freeto 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.
Successful merges:
trybuilds to happen on Azure #62142 (ci: Switch officialtrybuilds to happen on Azure)Failed merges:
r? @ghost