PR name collisions cancel prod jobs
Closed this issue · 4 comments
What happened?
If the PR names (and branch names) used by the tool were unique, we wouldn't have this problem. As things stand, test jobs have cancelled out prod jobs for bridge update.
Example
N/A
Output of pulumi about
N/A
Additional context
N/A
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).
It looks like the root cause is that --target-pulumi-version=08c4f69f96fcaedc4018bc516f72f0b4ec504736 doesn't make it into the name. I see two possible paths forward here:
- Ensure that SHA parameters are always wind up in the name.
- Ensure that CI driven builds always generate unique names by appending some random bit to them.
My instinct is that (2) will be the simplest and least breakable. We can add a --unique flag set by CI that appends, say, the last commit SHA to the branch name. This will preserve the local development experience, but improve the CI experience.
Or (3), we can punt to the user by adding a --branch-name flag and letting the running job choose.
I like (2) will be the simplest and least breakable.
we can punt to the user by adding a --branch-name flag and letting the running job choose
We + platform team are the user. This requires further small but hard to test change through GHA wrappers. Do we have time for that?
We + platform team are the user. This requires further small but hard to test change through GHA wrappers. Do we have time for that?
I think so, but I still think that (2) will be best overall.