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

Reports webapp with gunicorn #195

Open
microfuge opened this issue Oct 19, 2023 · 5 comments
Open

Reports webapp with gunicorn #195

microfuge opened this issue Oct 19, 2023 · 5 comments

Comments

@microfuge
Copy link

Hi,

Please let me know if I am wrong, but the "Galaxy reports" app will not run with uwsgi as now Galaxy uses Gunicorn. The templates still mentions uwsgi https://github.com/galaxyproject/ansible-galaxy/blob/main/templates/systemd/galaxy-reports.service.j2

And update to the role for this would be great. Or any advice on running the Reports app with gunicorn would also be very helpful.

Thanks!

@adRn-s
Copy link

adRn-s commented Oct 23, 2023

I think {{ galaxy_venv_dir }}/bin/uwsgi is still a valid path to an executable, and it's in fact used in many places. Did you get an error message? It's easier to debug GH issues with factual information.

disclaimer: not a dev or maintainer, just a watcher on the repo.

@hexylena
Copy link
Member

@adRn-s sure it's valid, but it's not used by Galaxy any more. Configuration for reports is now solely through gunicorn + gravity.

@microfuge we should definitely support this going forward, but I'm not sure when someone will update this issue.
@natefoo has the most experience with gravity which is now a necessary part of galaxy configuration.

@mvdbeek
Copy link
Member

mvdbeek commented Oct 23, 2023

@natefoo has the most experience with gravity which is now a necessary part of galaxy configuration.

No, you can use whatever you want, but it's certainly a lot easier to use gravity. I don't use gravity for development, that would be overkill.

@hexylena
Copy link
Member

@mvdbeek if it were added to this role, we would want to use gravity, wouldn't want to special case reports.

@mvdbeek
Copy link
Member

mvdbeek commented Oct 24, 2023

I do think that is exactly what you want to do. No need to litter gravity with special cases that have nothing to do with actually running Galaxy.

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

No branches or pull requests

4 participants