Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Requests for certificates hang when using vault charm in Juju Cross Model Relation #14

Closed
ChrisMacNaughton opened this issue Feb 5, 2019 · 0 comments · Fixed by #16
Closed
Assignees

Comments

@ChrisMacNaughton
Copy link

ChrisMacNaughton commented Feb 5, 2019

https://bugs.launchpad.net/vault-charm/+bug/1813605

As a result of using unit_name in the responses to key conversations to the units, the tls-certificates interface does not successfully handle Cross Model Relations.

@johnsca johnsca self-assigned this Feb 11, 2019
johnsca added a commit that referenced this issue Apr 18, 2019
With cross-model relations (CMR), the "unit name" visible on the
offering side of the relation is a UUID which doesn't match with the
unit's own view of its unit name.  Thus, the unit cannot find the
responses to its cert requests, as they are keyed by the UUID rather
than the unit name.  By explicitly publishing the unit name over the
relation, it ensures that the provider and requirer will use the same
key.

We use the unit name rather than a UUID or nonce to ensure that non-CMR
deployments are not broken upon upgrade.

Fixes #14
Cynerva pushed a commit that referenced this issue Apr 30, 2019
With cross-model relations (CMR), the "unit name" visible on the
offering side of the relation is a UUID which doesn't match with the
unit's own view of its unit name.  Thus, the unit cannot find the
responses to its cert requests, as they are keyed by the UUID rather
than the unit name.  By explicitly publishing the unit name over the
relation, it ensures that the provider and requirer will use the same
key.

We use the unit name rather than a UUID or nonce to ensure that non-CMR
deployments are not broken upon upgrade.

Fixes #14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants