Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Specify required juju version in metadata.yaml #517

Closed
mthaddon opened this issue Jul 2, 2024 · 3 comments · Fixed by #518
Closed

Specify required juju version in metadata.yaml #517

mthaddon opened this issue Jul 2, 2024 · 3 comments · Fixed by #518
Labels
bug Something isn't working

Comments

@mthaddon
Copy link

mthaddon commented Jul 2, 2024

Currently the charm includes the following in metadata.yaml:

assumes:
  - juju

However, it seems the current version of the charm requires juju 3.4.3 or greater. This can be managed by setting the assumes option per https://juju.is/docs/sdk/charmcraft-yaml#heading--assumes. Note that there's also an any-of sub-option if you want to allow 2.9 or >= 3.4.3.
@mthaddon mthaddon added the bug Something isn't working label Jul 2, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jul 2, 2024

https://warthogs.atlassian.net/browse/DPE-4809

@marceloneppel marceloneppel mentioned this issue Jul 2, 2024
Merged
@marceloneppel
Copy link
Member

Hi, @mthaddon! Thanks for the report.

It should be improved by #518.

@marceloneppel
Copy link
Member

Hi, Tom! Revisions 439 and 440 contain the required minimum Juju versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[DPE-4809] Enforce Juju versions canonical/postgresql-operator
2 participants
@mthaddon @marceloneppel