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

Race condition between TACOs apply and PR merge. Cannot recover. #228

Open
fpacifici opened this issue Jun 24, 2024 · 2 comments
Open

Race condition between TACOs apply and PR merge. Cannot recover. #228

fpacifici opened this issue Jun 24, 2024 · 2 comments

Comments

@fpacifici
Copy link
Contributor

Environment

https://github.com/getsentry/tacos-gha/tree/stable on the ops repo
This is the PR where the issue happened: https://github.com/getsentry/ops/pull/11131

Steps to Reproduce

On any slice:

  • make a change that triggers a successful plan
  • trigger tacos apply
  • while apply is running merge the PR

Expected Result

One of the following:

  • merge would be rejected.
  • the apply operation would still be executed
  • I could run apply after the PR was merged if merge did not happen
  • Nothing of the above should be relevant as apply should automatically happen on the merge queue

Actual Result

@mwarkentin
Copy link
Member

Is there a way for us to configure GHA to not cancel on merge? Feels like an optimization for unit test runtime or something..

@mwarkentin
Copy link
Member

Re: apply on merge, I think we need to consider it very carefully as it can introduce some catastrophic failures.

An example:

A GCP disk is scaled up out of band. There is no way to shrink the disk, so the way it would apply will be to destroy the disk and recreate it. Hope your backups are working and tested. 😅

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

No branches or pull requests

2 participants