Skip to content
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

Recipe led tests are picking up release candidate branch runs #120135

Closed
godofredoc opened this issue Feb 6, 2023 · 11 comments
Closed

Recipe led tests are picking up release candidate branch runs #120135

godofredoc opened this issue Feb 6, 2023 · 11 comments
Assignees
Labels
P1 High-priority issues at the top of the work list team-infra Owned by Infrastructure team

Comments

@godofredoc godofredoc added the team-infra Owned by Infrastructure team label Feb 6, 2023
@godofredoc godofredoc added this to New in Infra Ticket Queue via automation Feb 6, 2023
@drewroengoogle
Copy link
Contributor

@godofredoc What command(s) did you use to run this? Anything ususual outside of the regular led get-builder?

@drewroengoogle drewroengoogle added the P2 Important issues not at the top of the work list label Feb 7, 2023
@godofredoc
Copy link
Contributor Author

@drewroengoogle
Copy link
Contributor

Ah, so this is run on any recipes CL?

@godofredoc
Copy link
Contributor Author

Ah, so this is run on any recipes CL?

Yes, that is correct.

@drewroengoogle drewroengoogle moved this from New to Triaged in Infra Ticket Queue Feb 7, 2023
@drewroengoogle
Copy link
Contributor

drewroengoogle commented Feb 7, 2023

To summarize the problem - When opening a CL for recipes, the presubmit will look for any modified recipes, take the last green build, and replicate it using the modified recipe to make sure it is still passing. However, in some cases, the last green build is not on the same branch as the recipes CL is merging into, causing problems. In the case above, the last green build was on flutter-3.7-candidate.1, not main/master.

We need to change the testing to search for the last green build that is on the same branch as the CL is merging into.

EDIT: Related build (so I don't have to search for it again): https://ci.chromium.org/p/flutter/builders/try/recipes-with-led/b8789786742166096689

@drewroengoogle drewroengoogle added P1 High-priority issues at the top of the work list and removed P2 Important issues not at the top of the work list labels Feb 7, 2023
@ricardoamador
Copy link
Contributor

Related: #120287

@drewroengoogle drewroengoogle self-assigned this Feb 8, 2023
@drewroengoogle drewroengoogle moved this from Triaged to In progress in Infra Ticket Queue Feb 8, 2023
@ricardoamador ricardoamador moved this from In progress to Blocked in Infra Ticket Queue Feb 9, 2023
@ricardoamador ricardoamador moved this from Blocked to In progress in Infra Ticket Queue Feb 9, 2023
@drewroengoogle
Copy link
Contributor

Finishing up a CL for this: need to write tests

@drewroengoogle
Copy link
Contributor

drewroengoogle commented Feb 13, 2023

@drewroengoogle
Copy link
Contributor

^ This has been merged!

@drewroengoogle
Copy link
Contributor

Closing, feel free to reopen

Infra Ticket Queue automation moved this from In progress to Done Feb 21, 2023
@github-actions
Copy link

github-actions bot commented Mar 7, 2023

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P1 High-priority issues at the top of the work list team-infra Owned by Infrastructure team
Projects
No open projects
Development

No branches or pull requests

3 participants