Fix Slurm accounting password being overridden at queue update

[jdeamicis]

Description of changes

• Fix a bug that caused the slurmdbd database password to be overridden during a cluster update where the queues were being updated and the Slurm accounting configuration was not being modified.
• Relates to (3.3.0-3.5.1) Cluster updates can break Slurm accounting functionality aws-parallelcluster#5151

Tests

• manually run a modified version of test_slurm_accounting (see Modify test_slurm_accounting test to account for issue #5151 aws-parallelcluster#5197).

References

• Add CHANGELOG entry for fixing Slurm accounting password bug aws-parallelcluster#5196

Checklist

• Make sure you are pointing to the right branch and add a label in the PR title (i.e. 2.x vs 3.x)
• Check all commits' messages are clear, describing what and why vs how.
• Make sure to have added unit tests or integration tests to cover the new/modified code.
• Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[codecov]

Codecov Report

Merging #2006 (c0acd9c) into develop (709565b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2006   +/-   ##
========================================
  Coverage    69.42%   69.42%           
========================================
  Files           12       12           
  Lines         1766     1766           
========================================
  Hits          1226     1226           
  Misses         540      540           

Flag	Coverage Δ
unittests	69.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.