canonical / data-platform-libs

A collection of charm libraries curated by the Data Platform Team
https://charmhub.io/data-platform-libs
Apache License 2.0
10 stars 9 forks source link

Add `version` to DatabaseRequires interface #60

Open carlcsaposs-canonical opened 1 year ago

carlcsaposs-canonical commented 1 year ago

Database requirer should specify which version of the database it needs (e.g. mysql version, not charm revision) so that the provider knows if it's safe to upgrade the database version

Requires changes to https://github.com/canonical/charm-relation-interfaces

github-actions[bot] commented 1 year ago

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

taurus-forever commented 1 year ago

My $0.05: mysql charm need to know mysql-router version to block "juju refresh mysql" if router is not-yet upgraded, etc. So, database requirer (mysql-router) should specify his (mysql-router) version when relating to database provider (mysql-server).

Most probably, there are more use-cases for mysql to know router version (charm revision, router snap/deb version, interface version we are related thorugh, profile version used, ..., ....).