Skip to content

Conversation

@carlcsaposs-canonical
Copy link
Contributor

Will update ops further in a separate PR (to make diff easier to review, since changes to unit tests will be needed with ops-scenario update)

@carlcsaposs-canonical carlcsaposs-canonical marked this pull request as draft October 27, 2025 10:32
@github-actions github-actions bot added the Libraries: Out of sync The charm libs used are out-of-sync label Oct 27, 2025
@carlcsaposs-canonical
Copy link
Contributor Author

carlcsaposs-canonical commented Oct 27, 2025

Think it might be worth waiting until we have a new stable release before merging

This is a pretty significant change & would like to avoid multiple significant changes (refresh v3) in a single stable release in case there are any issues & we want to isolate part of the changes

@carlcsaposs-canonical carlcsaposs-canonical added the not bug or enhancement PR is not 'bug' or 'enhancement'. For release notes label Oct 27, 2025
@carlcsaposs-canonical
Copy link
Contributor Author

Looks like ops-scenario has issues with relation.active (returning True when it should be returning False); investigating further

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Libraries: Out of sync The charm libs used are out-of-sync not bug or enhancement PR is not 'bug' or 'enhancement'. For release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants