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

[Bug/_MungeManager]: incorrect service name for MUNGE when using _AptManager #47

Closed
NucciTheBoss opened this issue Nov 12, 2024 · 0 comments · Fixed by #53
Closed

[Bug/_MungeManager]: incorrect service name for MUNGE when using _AptManager #47

NucciTheBoss opened this issue Nov 12, 2024 · 0 comments · Fixed by #53
Labels
bug Something isn't working

Comments

@NucciTheBoss
Copy link
Member

As part of integration testing for charmed-hpc/slurm-charms#35, turns out we have the incorrect service name for MUNGE if using the _AptManager implementation. The service name is munge, not munged:

unit-slurmctld-0: 09:27:19 ERROR unit.slurmctld/0.juju-log command ['systemctl', 'reload-or-restart', 'munged'] failed with message Failed to reload-or-restart munged.service: Unit munged.service not found.

unit-slurmctld-0: 09:27:19 ERROR unit.slurmctld/0.juju-log command ['systemctl', 'reload-or-restart', 'munged'] failed. stderr:
Failed to reload-or-restart munged.service: Unit munged.service not found.

This issue wasn't caught because the integration tests don't try to start or stop the MUNGE service, but the fix is relatively easy as the _ServiceType enum will need to have the service name changed to munge:

class _ServiceType(Enum):
"""Type of Slurm service to manage."""
MUNGED = "munged"

The additional work that will be needed here is that we'll need to change the service name on the Slurm snap to munge instead of munged. We'll just need to make sure that the service name munge doesn't clash with the /snap/bin/munge binary on the snap package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant