Ticket 5294 - Slurmdbd storagepass character limitation
Summary: Slurmdbd storagepass character limitation
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: slurmdbd (show other tickets)
Version: 17.11.7
Hardware: Linux Linux
: 4 - Minor Issue
Assignee: Marshall Garey
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2018-06-12 01:38 MDT by CSC sysadmins
Modified: 2018-07-02 13:01 MDT (History)
1 user (show)

See Also:
Site: CSC - IT Center for Science
Slinky Site: ---
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: ---
DS9 clusters: ---
Google sites: ---
HPCnow Sites: ---
HPE Sites: ---
IBM Sites: ---
NOAA SIte: ---
NoveTech Sites: ---
Nvidia HWinf-CS Sites: ---
OCF Sites: ---
Recursion Pharma Sites: ---
SFW Sites: ---
SNIC sites: ---
Tzag Elita Sites: ---
Linux Distro: ---
Machine Name:
CLE Version:
Version Fixed: 17.11.8
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this ticket.
Description CSC sysadmins 2018-06-12 01:38:47 MDT 
Hi,

I decided to change mysql password when upgraded from mysql to mariadb. But seems that #-mark in slurmdbd.conf StoragePass=sec#ret does not work. I tried also to add quotes around "sec#ret" but no luck.
Comment 1 Marshall Garey 2018-06-13 15:25:23 MDT 
The last thing you could try is escaping the '#' character:

StoragePass=sec\#ret

but I doubt it will work - I don't believe we support the '#' character in a password (the '#' character denotes a comment), and that's not something we're likely to change.
Comment 2 CSC sysadmins 2018-06-14 00:19:52 MDT 
Well, maybe just add note about limitation to man page and slurmdbd documentation page?
Comment 4 Marshall Garey 2018-06-14 09:59:43 MDT 
(In reply to Tommi Tervo from comment #2)
> Well, maybe just add note about limitation to man page and slurmdbd
> documentation page?

Sure thing, I've uploaded a doc patch for review. I'll let you know when it's committed.
Comment 8 Marshall Garey 2018-07-02 13:01:26 MDT 
Updated documentation is in commit 0c606741f025417a7b051e58a0f272b81a2c6908.

Closing as resolved/fixed.