slurmctld: fix influxdb-info action - Githubissues

omnivector-solutions / slurm-charms

Repository containing the slurm operator charms

https://omnivector-solutions.github.io/osd-documentation/master/

Apache License 2.0

13 stars 7 forks source link

slurmctld: fix influxdb-info action #142

Closed heitorPB closed 2 years ago

heitorPB commented 2 years ago

Please provide description of the purpose of this pull request, as well as its motivation and context.

Fix the influxdb-info slurmctld action. Running the function was crashing the code due to a change in how Juju/ops expects the output values.

New tests added:

test the action
test the relation to influxdb
test removing the relation

Adds ~50 seconds to the total time to run all tests.

How was the code tested?

Local test suite pass.

Final checklist

[x] I am the author of these changes, or I have the rights to submit them.
[x] I added the relevant changed to the README and/or documentation.
[x] I self reviewed my own code.
[x] I updated the CHANGELOG according to the contributing guide