Skip to content

Automated rollback of commit f9863dfd3c98278e703be0e3c19ded2d09e60fde. #21355

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

Merged
merged 1 commit into from
Apr 18, 2025

Conversation

zhangskz
Copy link
Member

PiperOrigin-RevId: 749053436

PiperOrigin-RevId: 749053436
@zhangskz zhangskz requested a review from a team as a code owner April 18, 2025 16:47
@zhangskz zhangskz requested review from bshaffer and removed request for a team April 18, 2025 16:47
@bshaffer
Copy link
Contributor

bshaffer commented Apr 18, 2025

How was this missed when the PR was approved?

I approved the PR so I want to make sure this doesn't happen again!

@zhangskz
Copy link
Member Author

The test that caught this was a continuous-only test, so didn't run on presubmit. Adding #test-continuous to the PR description would force continuous-only tests to run on presubmit anyways.

On our side, we noticed this was failing a bit late, but these run hourly so we def could've caught this earlier.

@zhangskz zhangskz requested a review from haberman April 18, 2025 18:16
@zhangskz zhangskz enabled auto-merge (squash) April 18, 2025 18:16
@zhangskz zhangskz merged commit a59988c into 31.x Apr 18, 2025
210 checks passed
@zhangskz zhangskz deleted the cp-php-rollback branch April 18, 2025 18:41
@bshaffer
Copy link
Contributor

@zhangskz so would the next steps be for me to try and debug the failing continuous tests and then resubmit the PR?

@zhangskz
Copy link
Member Author

@zhangskz so would the next steps be for me to try and debug the failing continuous tests and then resubmit the PR?

Yes that would be great -- I also updated and reopened #19978 to let the original PR author know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants