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

Comienza con charms.layer.basic.clear_config_states() y sigue su llamada a unitdata.kv().flush(); después, lee la propuesta referenciada de charms.reactive sobre la cleanup-phase. Se considera terminado cuando el ciclo de vida del vaciado de estado del hook ya no deja un estado no válido si ocurre un fallo entre vaciados; el payload no indica ningún test ni archivo que ejecutar.

Escrito por el modelo de indexación a partir del texto del issue.