config: Treat ref-update ADD to existing ref as change

Bug #1634301 reported by Hampapur Ajay
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Undecided
Hampapur Ajay

Bug Description

If a ref-update API is done on an object to an existing ref, the existing ref was not treated as such and type-specific handlers see it as a list instead of set.

So e.g if VN-1 refers to IPAM-1 with [1.1.1.0/28] subnet, on update VN-1 to IPAM-1 with [1.1.1.0/28, 2.2.2.0/28] subnets, the type-specific handlers saw VN-1 having 3 subnets [1.1.1.0/28, 1.1.1.0/28, 2.2.2.0/28] and complains of overlap

cfgm_common.exceptions.BadRequest: Overlapping addresses: [IPNetwork('1.1.1.0/28'), IPNetwork('1.1.1.0/28')]

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/24959
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/24959
Committed: http://github.org/Juniper/contrail-controller/commit/648923cb6c0abc4aaeabe1d3a4021b900e918577
Submitter: Zuul
Branch: master

commit 648923cb6c0abc4aaeabe1d3a4021b900e918577
Author: Hampapur Ajay <email address hidden>
Date: Tue Oct 11 15:54:55 2016 -0700

For a ref-update ADD to an existing ref, treat it as change
possibly with attr.

Closes-Bug: 1634301
Change-Id: If55ec910fb942c8c39db9deec41980caf347d6a0

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.