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

Unrecognized labels cancel entire workflow #133

Closed
kichik opened this issue Oct 24, 2022 · 6 comments · Fixed by #134
Closed

Unrecognized labels cancel entire workflow #133

kichik opened this issue Oct 24, 2022 · 6 comments · Fixed by #134

Comments

@kichik
Copy link
Member

kichik commented Oct 24, 2022

From #72 (comment)

In your case it sounds like it's because we expect every job with the self-hosted label to also have a label we recognize. It sounds like you have multiple installations of cdk-github-runners in different accounts, each implementing different labels. That means some of the installations will not recognize the label and cancel the job. We can probably add a flag to disable this behavior. Let's do this in a separate ticket.

@laxgoalie392

@laxgoalie392
Copy link

thanks for opening this up!

you're assumptions are correct. I actually have multiple installations in multiple aws accounts (multiple github orgs needing to deploy to multiple aws accounts). i thought the label would appropriately isolate them, but it seems to have caused more of a mess.

kichik added a commit that referenced this issue Oct 24, 2022
We assume our runners are the only self-hosted runners on the system, but that might not be the case. If we don't recognize the label because it's intended for another self-hosted runner, we shouldn't cancel the entire workflow.

Fixes #133
@mergify mergify bot closed this as completed in #134 Oct 24, 2022
mergify bot pushed a commit that referenced this issue Oct 24, 2022
We assume our runners are the only self-hosted runners on the system, but that might not be the case. If we don't recognize the label because it's intended for another self-hosted runner, we shouldn't cancel the entire workflow.

Fixes #133
@laxgoalie392
Copy link

i've validated that this fix works for me! you are lightning fast, once again!

@laxgoalie392
Copy link

@kichik when do you think you will cut another release?

@kichik
Copy link
Member Author

kichik commented Oct 25, 2022

I just cut a release for you. I was under the impression you can use my changes without a release so I was waiting for a few more things to make it bigger. I'm curious, how do you test it usually without a release?

@laxgoalie392
Copy link

thanks! i usually just look at the changes in the PR and apply them manually. not a perfect verification method but quickly does a good enough job i think

@pharindoko
Copy link
Contributor

:O thanks @laxgoalie392 - had the same issue - we had pretty weird issues because it even cancelled the jobs for other runners that were not created via this solution ...

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 a pull request may close this issue.

3 participants