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

fix: avoid race condition in max running processes #675

Merged
merged 7 commits into from
Feb 21, 2024

Conversation

Alex-Izquierdo
Copy link
Collaborator

Having set the max_running_activations to 1 in the environment, I create several activations. As expected, as I create the activations, the first one starts running an the rest remain in pending state. However, if after that I delete the one that is running, then the rest of the pending rulebook activations all start running at the same time (ignoring the max_running_activations parameter)
Jira: https://issues.redhat.com/browse/AAP-20719

@Alex-Izquierdo Alex-Izquierdo requested a review from a team as a code owner February 13, 2024 21:19
@Alex-Izquierdo Alex-Izquierdo requested review from bzwei and a team February 14, 2024 12:44
bzwei
bzwei previously approved these changes Feb 14, 2024
mkanoor
mkanoor previously approved these changes Feb 15, 2024
@Alex-Izquierdo Alex-Izquierdo dismissed stale reviews from bzwei and mkanoor via 88735c9 February 20, 2024 17:27
@Alex-Izquierdo Alex-Izquierdo force-pushed the fix-max-activation-race branch from 841c5fd to 88735c9 Compare February 20, 2024 17:27
@Alex-Izquierdo
Copy link
Collaborator Author

Needed rebase. @mkanoor @bzwei please, review again.

@Alex-Izquierdo Alex-Izquierdo merged commit b37fa13 into ansible:main Feb 21, 2024
3 checks passed
@Alex-Izquierdo Alex-Izquierdo deleted the fix-max-activation-race branch February 21, 2024 14:38
Alex-Izquierdo added a commit to Alex-Izquierdo/eda-server that referenced this pull request Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants