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

Keep track of cluster groups for evacuations #696

Merged
merged 6 commits into from
Mar 29, 2024
Merged

Conversation

stgraber
Copy link
Member

No description provided.

@github-actions github-actions bot added the Documentation Documentation needs updating label Mar 29, 2024
@stgraber
Copy link
Member Author

The fact that we weren't running cluster requests in notify mode is a bit odd considering the number of places in this code path which was checking whether we were handling a notification (to avoid running config checks twice).

In my local tests, the logic seems to be behaving just fine when in notify mode, so hopefully it was in fact a long standing mistake and not something that we had to do on purpose and failed to document.

@hallyn hallyn merged commit 8c6d7d6 into lxc:main Mar 29, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation needs updating
Development

Successfully merging this pull request may close these issues.

2 participants