Skip to content

Conversation

@tonnenpinguin
Copy link
Contributor

Fixes #194

This ensures the Supervisor doesn't restart Shards that we're trying to shut down using Phoenix.Tracker.graceful_permdown/1

@josevalim
Copy link
Member

@kianmeng would you be so kind to help us update this GitHub Action? :)

@josevalim josevalim merged commit 148ae10 into phoenixframework:main Mar 5, 2025
0 of 2 checks passed
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

I have ran the tests locally.

@SteffenDE
Copy link
Contributor

I updated CI

@tonnenpinguin tonnenpinguin deleted the fix/shard-restart branch March 5, 2025 15:25
@kianmeng
Copy link
Contributor

kianmeng commented Mar 5, 2025

@josevalim Sorry, just saw your message. I believe SteffenDE already patched it.

@tonnenpinguin
Copy link
Contributor Author

tonnenpinguin commented Mar 18, 2025

Thanks for looking into this and merging! :)
Is there anything else you guys need me to do before this can be released to hex? 🙂

@josevalim
Copy link
Member

Please give it a try via the github repo to ensure no other pitfalls and we can ship a new version later :)

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 this pull request may close these issues.

Shards are being restarted while calling Phoenix.Tracker.graceful_permdown/1

4 participants