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

Update deployment instructions #447

Closed
wants to merge 4 commits into from
Closed

Conversation

seallard
Copy link
Contributor

Update deployment instructions to reflect how to use the CLI images on Hasta.

@seallard seallard requested a review from a team as a code owner May 16, 2024 12:35
Copy link

@ChrOertlin ChrOertlin left a comment

Choose a reason for hiding this comment

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

Looks good, instructions on how to perform alembic migrations from a local machine would be great

DEPLOYMENT.md Outdated Show resolved Hide resolved
Copy link
Contributor

@diitaz93 diitaz93 left a comment

Choose a reason for hiding this comment

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

👍

DEPLOYMENT.md Show resolved Hide resolved
DEPLOYMENT.md Show resolved Hide resolved
seallard and others added 2 commits May 16, 2024 14:44
Co-authored-by: Sebastian Diaz <juan.sebastian.diaz.boada@scilifelab.se>
Co-authored-by: Sebastian Diaz <juan.sebastian.diaz.boada@scilifelab.se>
Copy link
Contributor

@Vince-janv Vince-janv left a comment

Choose a reason for hiding this comment

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

Nice 👍

DEPLOYMENT.md Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented May 17, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@seallard
Copy link
Contributor Author

seallard commented May 23, 2024

Had to give up on this. Basically, when running the image you need to mount slurm from the host (to be able to use squeue). However, to authenticate you need access to the munge config on the host as well, specifically the socket and key. You need sudo for that, they cannot be mounted. Which means we cannot retrieve info about the slurm jobs from inside the container.

If we had somewhat more recent version of SLURM, we could retrieve the info via the slurm rest API instead.

This is the error

[sebastian.allard@hasta:~] [S_base] $ singularity run --bind /usr/bin/squeue \
    --bind /etc/slurm \
    --bind /usr/lib64/slurm \
    --bind /usr/bin/munge \
    --bind /var/run/munge \
    --bind /usr/lib64/libmunge.so.2 \
    /home/proj/stage/singularity_containers/trailblazer_latest.sif squeue --all
squeue: error: If munged is up, restart with --num-threads=10
squeue: error: Munge encode failed: Failed to access "/var/run/munge/munge.socket.2": No such file or directory
squeue: error: authentication: Socket communication error
slurm_load_jobs error: Protocol authentication error

@seallard seallard closed this May 23, 2024
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