Open Stack

+1 to overriding VIPs

On Thu, Jan 28, 2016 at 1:43 PM, Roman Prykhodchenko <me at romcheg.me> wrote:

> Folks,
>
> currently merge policy for network roles only allows to add new VIPs to
> already existing roles [1]. If a plugin supplies a VIP with a name that
> already exists but with different parameters, Nailgun will not allow to do
> that. We faced a need to override configuration of several VIPs by
> completely removing them from network roles by supplying something like
> [2]. To enable that I’ve made a temporary workaround [3] to the merging
> policy that only handles one cornerstone.
>
> I’ve talked to ikalnitsky and we realized that this functionality of
> merging new VIPs from plugins to an existing network role is actually wrong
> and should be replaced by complete overriding. Let’s discuss this
> possibility here.
>
>
> References:
>
> 1.
> https://github.com/openstack/fuel-web/blob/master/nailgun/nailgun/policy/merge.py#L55
> 2. http://xsnippet.org/361361/
> 3. https://review.openstack.org/#/c/273169/1
>
>
> - romcheg
>
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>


-- 
Yours Faithfully,
Vladimir Kuklin,
Fuel Library Tech Lead,
Mirantis, Inc.
+7 (495) 640-49-04
+7 (926) 702-39-68
Skype kuklinvv
35bk3, Vorontsovskaya Str.
Moscow, Russia,
www.mirantis.com <http://www.mirantis.ru/>
www.mirantis.ru
vkuklin at mirantis.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20160128/fc5f305e/attachment.html>