ansible-galaxy icon indicating copy to clipboard operation
ansible-galaxy copied to clipboard

Reports webapp with gunicorn

Open microfuge opened this issue 1 year ago • 5 comments

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!

microfuge avatar Oct 19 '23 15:10 microfuge

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.

adRn-s avatar Oct 23 '23 11:10 adRn-s

@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.

hexylena avatar Oct 23 '23 11:10 hexylena

@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.

mvdbeek avatar Oct 23 '23 12:10 mvdbeek

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

hexylena avatar Oct 23 '23 13:10 hexylena

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.

mvdbeek avatar Oct 24 '23 07:10 mvdbeek