Multiple calls to unitdata.kv().flush()

charms.layer.basic.clear_config_states() calls unitdata.kv().flush(), which will be the second time the storage is flushed in the hook. In the perverse case where a failure happens between these two calls, we can end up with invalid state.

I think this is a use case for an actual `@cleanup` phase in charms.reactive, per https://github.com/juju-solutions/charms.reactive/pull/20

Commencez par charms.layer.basic.clear_config_states() et suivez son appel à unitdata.kv().flush(), puis lisez la proposition référencée de charms.reactive concernant la cleanup-phase. C’est terminé lorsque le cycle de vie du vidage de l’état du hook ne laisse plus d’état invalide si une défaillance survient entre deux vidages ; le payload n’indique aucun test ni fichier à exécuter.

Rédigé par le modèle d'indexation à partir du texte de l'issue.