Skip to content

patch: update rock in metadata#49

Merged
skourta merged 2 commits into9/edgefrom
patch-update-rock-in-metadata
Apr 21, 2026
Merged

patch: update rock in metadata#49
skourta merged 2 commits into9/edgefrom
patch-update-rock-in-metadata

Conversation

@skourta
Copy link
Copy Markdown
Contributor

@skourta skourta commented Apr 21, 2026

This pull request updates the upstream source for the valkey-image resource in the metadata.yaml file. The new source points to a different repository and uses an underscore instead of a hyphen in the tag.

  • Resource configuration update:
    • Changed the valkey-image resource's upstream-source from ghcr.io/canonical/valkey:9.0.1-26.04-edge to ghcr.io/canonical/charmed-valkey:9.0.1-26.04_edge in metadata.yaml, switching to the charmed-valkey repository and updating the tag format.

Signed-off-by: Smail KOURTA <smail.kourta@canonical.com>
Copilot AI review requested due to automatic review settings April 21, 2026 14:10
Mehdi-Bendriss
Mehdi-Bendriss previously approved these changes Apr 21, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Juju resource image reference for the valkey-image OCI resource to pull from the canonical/charmed-valkey repository and adjusts the tag format accordingly.

Changes:

  • Switch valkey-image upstream-source from ghcr.io/canonical/valkey:9.0.1-26.04-edge to ghcr.io/canonical/charmed-valkey:9.0.1-26.04_edge.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread metadata.yaml
reneradoi
reneradoi previously approved these changes Apr 21, 2026
@skourta skourta dismissed stale reviews from reneradoi and Mehdi-Bendriss via aaade4d April 21, 2026 14:17
@skourta skourta merged commit 670b9da into 9/edge Apr 21, 2026
22 of 33 checks passed
@skourta skourta deleted the patch-update-rock-in-metadata branch April 21, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants