canonical / istio-operators

Charmed Istio
2 stars 17 forks source link

fix: call self._cni_config_changed and handle exception #396

Closed DnPlas closed 6 months ago

DnPlas commented 6 months ago

The _cni_config_changed() method was not called correctly because of a typo, which led to the condition evaluating it to always be True, and running upgrades when they are not required. Calling correctly the method ensures the charm executes correct pieces of code. By doing this, the GenericCharmRuntimeError exception that gets raised on an error with CNI configuration must be re-raised when handling the reconcile.

Fixes #395