Skip to content

Conversation

@martindemko
Copy link
Collaborator

No description provided.

@martindemko martindemko marked this pull request as draft July 14, 2025 09:25
@martindemko martindemko self-assigned this Jul 14, 2025
@martindemko
Copy link
Collaborator Author

galaxy-re VM didn't have a privileged user (e.g., debian) so we need to create one manually

LC_ALL: C
# The cache directory holds the docker containers that get converted
SINGULARITY_CACHEDIR: "/cvmfs/singularity.galaxyproject.org/all/"
SINGULARITY_CACHEDIR: "{{ galaxy_mutable_data_dir }}/singularity/cache"
Copy link
Member

Choose a reason for hiding this comment

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

opts: rw,_netdev
state: mounted
- name: Create galaxy mutable data directory
- name: Create RBD data directory
Copy link
Member

Choose a reason for hiding this comment

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

A note about what is RBD used for and how we treat it (maybe in a readme?) would be welcome

galaxy_commit_id: release_25.0
galaxy_force_checkout: true
galaxy_job_config_file: "{{ galaxy_config_dir }}/galaxy.yml"
galaxy_data_dir: "{{ galaxy_mutable_data_dir }}"
Copy link
Member

Choose a reason for hiding this comment

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

I am not sure about this var, if this is truly just some local helper it would be better to not have the galaxy_ prefix, because it is not related to the galaxy role.

@martindemko
Copy link
Collaborator Author

I just ran it on REgalaxy, and it works. Only GCZ remains. I think we are ready to merge

@martindemko martindemko marked this pull request as ready for review January 19, 2026 10:17
@martindemko martindemko merged commit 1530c57 into main Jan 19, 2026
@martenson martenson deleted the clean_RE branch January 19, 2026 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants