Skip to content

Use canonical pollen snap in stable channel - #98

Merged
javierdelapuente merged 5 commits into
mainfrom
use-canonical-pollen
Jan 14, 2026
Merged

Use canonical pollen snap in stable channel#98
javierdelapuente merged 5 commits into
mainfrom
use-canonical-pollen

Conversation

@javierdelapuente

@javierdelapuente javierdelapuente commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

Overview

Use Canonical's pollen snap in the stable channel.

Rationale

Juju events changes

Module changes

Library changes

Checklist

@javierdelapuente
javierdelapuente requested a review from a team as a code owner January 13, 2026 11:53
@javierdelapuente
javierdelapuente requested review from arturo-seijas and swetha1654 and removed request for a team January 13, 2026 11:53

@weiiwang01 weiiwang01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you also add code to detect if the old snap is installed on the system and remove it before installing the new snap? This will allow the user to upgrade the charm in place.

@javierdelapuente

Copy link
Copy Markdown
Contributor Author

Can you also add code to detect if the old snap is installed on the system and remove it before installing the new snap? This will allow the user to upgrade the charm in place.

ouch, thanks for the heads up

@javierdelapuente
javierdelapuente marked this pull request as draft January 13, 2026 13:36
auto-merge was automatically disabled January 13, 2026 13:36

Pull request was converted to draft

@javierdelapuente
javierdelapuente marked this pull request as ready for review January 13, 2026 14:03
@javierdelapuente
javierdelapuente enabled auto-merge (squash) January 13, 2026 15:32
@javierdelapuente
javierdelapuente merged commit 36f341f into main Jan 14, 2026
18 checks passed
@javierdelapuente
javierdelapuente deleted the use-canonical-pollen branch January 14, 2026 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants