Small updates on Slurm config and some linter small rollbacks

[viniciusdc]

Reference Issues or PRs

closes #145

What does this implement/fix?

I had some errors with Ansible not addressing the restart tasks correctly due to Case-matching issues (this only affects redeployments, so was a bit tricky to find on first glance). Also, we made the base directory of the Slurm installation into a variable, as our test's default path may differ from user installation into pre-existing machines.

I also included some comments from the slurm-related package man docs on file permissions for future reference.

Put a x in the boxes that apply

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds a feature)
• Breaking change (fix or feature that would cause existing features not to work as expected)
• Documentation Update
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Build related changes
• Other (please describe):

Testing

• Did you test the pull request locally?
• Did you add new tests?

Documentation

Access-centered content checklist

Text styling

• The content is written with plain language (where relevant).
• If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
• All links describe where they link to (for example, check the Nebari website).
• This content adheres to the Nebari style guides.

Non-text content

• All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
• If there are emojis, there are not more than three in a row.
• Don't use flashing GIFs or videos.
• If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

[aktech]

Slurm config changes and linting changes are very orthogonal pieces and I would recommend to keep them in separate PRs. I also see some CI changes in here.

[aktech]

Looks good to me now.