Comment 2 for bug 2024583

relation-broken should have the JUJU_REMOTE_APP value set in my opinion. This hook runs when all remote related (counterpart) units have left the relation and this unit on which the hook runs is ready to leave scope.

In my view, at that point, there can be no expectation that the unit relation data bags have any data; the relation is broken after all and the counterpart units are no longer known to the local unit. This might differ from what John says on the linked issue so we should discuss it. In my view, any cleanup that requires data should have been done in the departed hook.

Having said that, the current implementation might still report that the relation application data bag has data. Since this is app level, perhaps there's an argument that data there could be needed to do any final app level work.