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

chore: bump systemd notices #42

Merged

Conversation

jedel1043
Copy link
Contributor

Description

Bumps the juju_systemd_notices library to its latest version, removing the custom version
we had before.

How was the code tested?

Unit tests locally on an Ubuntu Jammy development machine.
Integration tests cannot be run until we integrate the snap onto slurmdbd.

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.

@jedel1043 jedel1043 force-pushed the bump-systemd-notices branch from 1498459 to 3ebee76 Compare July 26, 2024 19:31
@jedel1043 jedel1043 force-pushed the bump-systemd-notices branch from 3ebee76 to 8fe01e4 Compare July 26, 2024 19:47
@NucciTheBoss NucciTheBoss self-requested a review July 26, 2024 19:51
Copy link
Member

@NucciTheBoss NucciTheBoss left a comment

Choose a reason for hiding this comment

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

Tests completed successfully the first time, we need to just rebase since we merged an earlier PR that changed the commit SHA.

@NucciTheBoss NucciTheBoss merged commit 331ef28 into charmed-hpc:experimental Jul 26, 2024
5 checks passed
@jedel1043 jedel1043 deleted the bump-systemd-notices branch July 26, 2024 20:47
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