search

omnivector-solutions / osd-documentation

Documentation for the Omnivector Slurm Distribution
https://omnivector-solutions.github.io/osd-documentation/master/
MIT License
3 stars 0 forks source link

Warn users that `scontrol update` is not permanent #42

Closed heitorPB closed 2 years ago

erik78se commented 2 years ago

Why?

heitorPB commented 2 years ago

Because scontrol update nodename=foo state=drain modifies the slurm.conf file without informing the charms. When the charms do something that needs to update the file, that change is discarded.

erik78se commented 2 years ago

Being able to use slurm userspace tools is of course important. How that distinction is communicated and implemented needs some thought. After all, slurm needs to be slurm.

Den ons 13 okt. 2021 17:39Heitor @.***> skrev:

Because scontrol update nodename=foo state=drain modifies the slurm.conf file without informing the charms. When the charms do something that needs to update the file, that change is discarded.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/omnivector-solutions/osd-documentation/pull/42#issuecomment-942430760, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGPVK2UCBLXND2BDHUPYFDUGWR3FANCNFSM5F5QA3WA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

heitorPB commented 2 years ago

Totally.

scontrol still works and is very handy. But changing the state of nodes/partitions should be done via Juju Actions.

On Wed, 13 Oct 2021 08:47:47 -0700 Erik Lönroth @.***> wrote:

Being able to use slurm userspace tools is of course important. How that distinction is communicated and implemented needs some thought. After all, slurm needs to be slurm.

Den ons 13 okt. 2021 17:39Heitor @.***> skrev:

Because scontrol update nodename=foo state=drain modifies the slurm.conf file without informing the charms. When the charms do something that needs to update the file, that change is discarded.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/omnivector-solutions/osd-documentation/pull/42#issuecomment-942430760, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGPVK2UCBLXND2BDHUPYFDUGWR3FANCNFSM5F5QA3WA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.