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

[3006.x] Fix minion config option startup_states #67759

Open
wants to merge 5 commits into
base: 3006.x
Choose a base branch
from

Conversation

dwoz
Copy link
Contributor

@dwoz dwoz commented Feb 21, 2025

Make sure startup state coroutine is run as coroutine.

#66592

Thanks @transistortim

transistortim and others added 4 commits February 20, 2025 18:43
I am totally on board with making tune_in asynchronous but we'll have to
address it in syndic and proxy minions too. As well as the corresponding
managers. This change is a bit less intrusive but achieves the same
end result.
@dwoz dwoz requested a review from a team as a code owner February 21, 2025 02:04
@dwoz dwoz changed the title [wip] [3006.x] Fix minion config option startup_states [3006.x] Fix minion config option startup_states Feb 21, 2025
@dwoz dwoz added the test:full Run the full test suite label Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test:full Run the full test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants