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

Auto-generate version file when slurmdbd operator is packed #16

Merged
merged 2 commits into from
Aug 4, 2023

Conversation

NucciTheBoss
Copy link
Member

@NucciTheBoss NucciTheBoss commented Aug 3, 2023

Description

This pull request modifies charmcraft.yaml so that the version file is automatically generated when packing the slurmdbd operator with charmcraft pack ...

How was the code tested?

Tested locally on my Ubuntu 22.04 LTS workstation by packing the slurmdbd charm and then examining the archive to ensure that the version file was successfully generated. The integration tests will likely fail for the time being due to issues identified in #15

Related issues and/or tasks

See Ubuntu HPC Matrix charming discussion

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.

This commit removes the version file generation
job step from the release workflow, removes
the version file generation utilities from the
integration tests, and modifies charmcraft.yaml
so that the version file will be automatically
generated when the slurmdbd charm is packed.

Signed-off-by: Jason C. Nucciarone <[email protected]>
Unused imports left in helpers.py in integration tests.

Signed-off-by: NucciTheBoss <[email protected]>
Copy link
Contributor

@jaimesouza jaimesouza left a comment

Choose a reason for hiding this comment

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

It looks good! Thank you @NucciTheBoss

@jaimesouza jaimesouza merged commit 7f92190 into charmed-hpc:main Aug 4, 2023
3 of 5 checks passed
@NucciTheBoss NucciTheBoss deleted the autogen-version branch August 23, 2023 17:22
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.

2 participants