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 stopping of gx-it-proxy #91

Merged
merged 1 commit into from
Dec 5, 2022
Merged

Conversation

abretaud
Copy link
Contributor

@abretaud abretaud commented Dec 5, 2022

A fix for #90, tested locally on 22.05

Copy link
Member

@mvdbeek mvdbeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thank you @abretaud!

@dannon
Copy link
Member

dannon commented Dec 5, 2022

Thank you! This has been an annoyance of mine, too -- I thought I even tried stopasgroup, too, but I might have changed too many variables at once :)

@abretaud
Copy link
Contributor Author

abretaud commented Dec 5, 2022

Cool :) Now I'm wondering how to apply this fix on my instance 🤔 is there a gravity release planned anytime soon?

@mvdbeek mvdbeek merged commit 0d069b2 into galaxyproject:main Dec 5, 2022
@mvdbeek
Copy link
Member

mvdbeek commented Dec 5, 2022

We can make a release and update the dependency in the 22.05 branch, no problem.

@natefoo
Copy link
Member

natefoo commented Dec 5, 2022

There's only one supervisord template in Gravity 1.x, so in merging this forward, we could set this conditionally per service, but I suspect there'd be no harm in setting it on all the managed services.

@natefoo
Copy link
Member

natefoo commented Dec 6, 2022

Merged forward in 3e9913d.

@natefoo
Copy link
Member

natefoo commented Dec 6, 2022

Released in Gravity 0.13.6 and the update to Galaxy is in galaxyproject/galaxy#15122.

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.

4 participants