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

24.2 is coming #614

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

24.2 is coming #614

wants to merge 12 commits into from

Conversation

bgruening
Copy link
Owner

No description provided.

@bgruening bgruening closed this Jan 10, 2025
@bgruening bgruening reopened this Jan 10, 2025
@kysrpex
Copy link

kysrpex commented Jan 14, 2025

Regarding your comment on usegalaxy-eu/issues#422; the old configuration setting still works and when provisioning the image gx_it_proxy_version is set to '>=0.0.6', so the new version should be automatically installed and it should just work.

However, is it possible to run multiple instances of this Docker image that share a common database? Then I think I'd have to make changes.

@bgruening
Copy link
Owner Author

No this is not possible. It is strictly not needed to use PG for the proxy for this project. I just thought its a good thing to do. Isn't the new way the way we would like to advertise/recommend?

@kysrpex
Copy link

kysrpex commented Jan 14, 2025

Isn't the new way the way we would like to advertise/recommend?

Maybe only if a high-availability setup is needed, right? It requires extra setup (creating a dedicated database and enabling PostgreSQL asynchronous notifications for the gxitproxy table), yet if there is only a single instance of Galaxy running it provides no benefit back in exchange.

Given that it is not a high-availability setup, using PostgresSQL for the proxy within this Docker image does not provide an advantage in itself, but still means more lines of code within the repository to write and maintain.

The Galaxy Docker Image is an easy distributable full-fledged Galaxy installation, that can be used for testing, teaching and presenting new tools and features.

If it is because of demonstrating and presenting the new feature: the GTN training was updated; the release notes for 24.2 also mention the feature; and the README for the proxy shows how to use it. Is it really still worth it to promote it by making the setup part of this Docker image? If you insist, it can still be done ofc

@bgruening bgruening closed this Feb 2, 2025
@bgruening bgruening reopened this Feb 2, 2025
bgruening and others added 3 commits February 2, 2025 11:33
        systemctl daemon-reload
        systemctl start cvmfs-reload

instead of:

        if [ -d /var/run/cvmfs ]; then
          cvmfs_config reload
        fi

in the "postinst" scripts. In the container we do not have systemctl installed.
@bgruening bgruening force-pushed the 24.2 branch 2 times, most recently from 55876cb to c6646eb Compare February 2, 2025 13:28
@bgruening
Copy link
Owner Author

cvmfs_2.12.4~1+ubuntu22.04_amd64.deb is broken for non systemd systems. The problem seems to be known a new package will be released soon.

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.

2 participants