search

canonical / k8s-operator

Machine charm for K8s following the operator framework
Apache License 2.0
4 stars 3 forks source link

Safely configure datastore request in the same method #82

Closed addyess closed 6 months ago

addyess commented 6 months ago

Use the same _configure_datastore(...) method to configure datastore properties

addyess commented 6 months ago

Nice refactor. LGTM

Without this, i've seen the tests occasionally fail in CI like this: https://github.com/canonical/k8s-operator/actions/runs/8834631143/job/24269710706#step:21:1625