Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@chinmaygarde
Copy link
Contributor

Reverts #17332. The presubmit added here is failing for all commits and blocking progress of all rollers. An example of a failure is here. I believe the issue everyone is tripping on was introduced in a commit a few hours earlier in f0caace. I think (I am not sure though) the presubmits for this commit passed because it was not rebased on ToT.

@auto-assign auto-assign bot requested a review from cbracken March 27, 2020 09:25
@chinmaygarde
Copy link
Contributor Author

TBR @yjbanov

@chinmaygarde chinmaygarde requested a review from yjbanov March 27, 2020 09:25
@chinmaygarde chinmaygarde added waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. and removed waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. labels Mar 27, 2020
@chinmaygarde chinmaygarde merged commit 6d33ee1 into master Mar 27, 2020
@chinmaygarde chinmaygarde deleted the revert-17332-analysis branch March 27, 2020 09:38
@chinmaygarde
Copy link
Contributor Author

The rollers should be back on their feet once they abandon their current runs. There was a post-submit test failure as well that has been disabled.

@yjbanov
Copy link
Contributor

yjbanov commented Mar 27, 2020

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants