Skip to content

Commit

Permalink
fix: use snap_daemon instead of _daemon_
Browse files Browse the repository at this point in the history
Apparently the store doesn't support `_daemon_` yet, even if the documentation recommends using it as a replacement for `snap_daemon`.
  • Loading branch information
jedel1043 authored and NucciTheBoss committed Jul 25, 2024
1 parent f4567b4 commit f680e23
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions overlays/sbin/slurmrestd.wrapper
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ fi
# Export invalid Slurm JWT token to activate JWT authentication in slurmrestd.
# See for more details: https://slurm.schedmd.com/rest.html#jwt
export SLURM_JWT=
# Drop to _daemon_ because slurmrestd cannot run as either root or SlurmUser.
"${SNAP}"/usr/bin/setpriv --clear-groups --reuid _daemon_ --regid _daemon_ -- \
# Drop to snap_daemon because slurmrestd cannot run as either root or SlurmUser.
"${SNAP}"/usr/bin/setpriv --clear-groups --reuid snap_daemon --regid snap_daemon -- \
"${SNAP}"/sbin/slurmrestd \
-f "${SNAP_COMMON}/etc/slurm/slurm.conf" \
--max-connections "${SLURMRESTD_MAX_CONNECTIONS}" \
Expand Down
2 changes: 1 addition & 1 deletion snap/snapcraft.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ environment:
PATH: $SNAP/usr/sbin:$SNAP/usr/bin:$SNAP/sbin:$SNAP/bin:$SNAP/usr/local/bin:$SNAP/usr/local/sbin:$PATH

system-usernames:
_daemon_: shared
snap_daemon: shared

apps:
logrotate:
Expand Down

0 comments on commit f680e23

Please sign in to comment.