Skip to content
This repository has been archived by the owner on Aug 9, 2024. It is now read-only.

Add support for using local charms in integration tests #18

Merged
merged 2 commits into from
Aug 24, 2023

Conversation

NucciTheBoss
Copy link
Member

Description

Similar to charmed-hpc/slurmctld-operator#17, but slurmdbd focused 😃

How was the code tested?

Integration tests were run on my Ubuntu 23.04 laptop.

Related issues and/or tasks

Closes #17

Checklist

  • I am the author of these changes, or I have the rights to submit them.
  • I have added the relevant changes to the README and/or documentation.
  • I have self reviewed my own code.
  • All requested changes and/or review comments have been resolved.

* Removed old references to ETCD.
* Deleted helpers module because it was no longer necessary.
* Add `--use-local` option to direct integration tests to use local slurmctld operator.

Signed-off-by: Jason C. Nucciarone <jason.nucciarone@canonical.com>
* Needed to replace type(...) == type calls with isinstance(...)
    Using type(...) == type instead of isinstance violates lint rule E721.

Signed-off-by: Jason C. Nucciarone <jason.nucciarone@canonical.com>
Copy link
Contributor

@jamesbeedy jamesbeedy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 Nice!

@jamesbeedy jamesbeedy merged commit ec92a18 into charmed-hpc:main Aug 24, 2023
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable using local SLURM charms with integration tests
2 participants