Add support for multiple API VIPs

Bug #1826892 reported by Edward Hope-Morley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
vault-charm
Fix Released
Wishlist
Edward Hope-Morley

Bug Description

The vault charm currently supports configuring a single vip on the api network which can be the "default" or "access" spaces. There are some use cases that might benefit from having the api accessible from more than one network e.g. where a use has deployed a vault consumer e.g. kubernetes, on a different network to that used by the primary network on which the vault service is listening. To achieve this we could allow the charm to be configured with multiple VIPs on different networks (much like charms that relate to hacluster can do) and to enable this we could add an extra space binding e.g. "public".

Tags: sts
description: updated
Revision history for this message
Edward Hope-Morley (hopem) wrote :

I've had a look at the charms' code to see what this might entail and one potential way to implement a second binding could be to add a new publish_url method [1] which would provide the url for a second space/binding should it be configured. That way the vault charm can optionally be configured on more than one space such that other charms can consume it without needing to be configured on its primary network.

[1] https://github.com/openstack-charmers/charm-interface-vault-kv/blob/master/provides.py#L45

Changed in vault-charm:
status: New → Triaged
Changed in vault-charm:
assignee: nobody → Edward Hope-Morley (hopem)
Revision history for this message
Edward Hope-Morley (hopem) wrote :
Changed in vault-charm:
status: Triaged → In Progress
Revision history for this message
Edward Hope-Morley (hopem) wrote :
David Ames (thedac)
Changed in vault-charm:
milestone: 19.07 → 19.10
Revision history for this message
Dmitrii Shcherbakov (dmitriis) wrote :

It seems to me that Juju could be extended to better support things like this:

https://discourse.jujucharms.com/t/multiple-space-bindings-per-endpoint/1999

Revision history for this message
David Ames (thedac) wrote :

Please see https://bugs.launchpad.net/vault-charm/+bug/1843809 which seems to have been caused by the above change.

David Ames (thedac)
Changed in vault-charm:
milestone: 19.10 → 20.01
Revision history for this message
Edward Hope-Morley (hopem) wrote :

This is actually fixed and released since the patch in comment #5 landed and is synced into the charm as part of 19.10.

Changed in vault-charm:
status: In Progress → Fix Released
milestone: 20.01 → 19.10
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.