CVM needs global VLAN ID Pool

Bug #1779100 reported by Adrian Szczepański on 2018-06-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
High
Adrian Szczepański
Trunk
Fix Committed
High
Adrian Szczepański
OpenContrail
Fix Committed
Undecided
Adrian Szczepański

Bug Description

There should be only one pool of VLAN IDs per ESXi host.
Now, each portgroup has its own pool.

Changed in opencontrail:
assignee: nobody → Adrian Szczepański (adrian.szczepanski)

Review in progress for https://review.opencontrail.org/44191
Submitter: Adrian Szczepanski (<email address hidden>)

Review in progress for https://review.opencontrail.org/44193
Submitter: Adrian Szczepanski (<email address hidden>)

Review in progress for https://review.opencontrail.org/44191
Submitter: Adrian Szczepanski (<email address hidden>)

summary: - CVM need global VLAN ID Pool
+ CVM needs global VLAN ID Pool

Review in progress for https://review.opencontrail.org/44193
Submitter: Adrian Szczepanski (<email address hidden>)

Review in progress for https://review.opencontrail.org/44191
Submitter: Adrian Szczepanski (<email address hidden>)

Review in progress for https://review.opencontrail.org/44193
Submitter: Adrian Szczepanski (<email address hidden>)

Reviewed: https://review.opencontrail.org/44191
Committed: http://github.com/Juniper/contrail-vcenter-manager/commit/2d44756d1e5f5d4cef461a19a7ea0e016a1750ec
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 2d44756d1e5f5d4cef461a19a7ea0e016a1750ec
Author: Adrian Szczepański <email address hidden>
Date: Wed Jun 27 14:55:34 2018 +0200

Use one global VLAN ID pool

Store DVSwitch as an attribute of VCenterAPIClient
Add VlanIdPool to VMI Service
Modify assign_vlan_id method to use global id pool
Modify restore_vlan_id method to use global id pool
Remove unused methods and tests
Add DVSwitch name as a config parameter

Depends-On: Idfd94ecc640dc06a1c4269b4a891d240bbb62a4d
Closes-Bug: #1779100
Change-Id: I4ba047e230730a7bd3ff52cbd528823508510dcd

Review in progress for https://review.opencontrail.org/44363
Submitter: Krzysztof Gromadzki (<email address hidden>)

Reviewed: https://review.opencontrail.org/44363
Committed: http://github.com/Juniper/contrail-vcenter-manager/commit/8721db41884e2d6e681e3bdb530c77b29ca5f1a7
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 8721db41884e2d6e681e3bdb530c77b29ca5f1a7
Author: Adrian Szczepański <email address hidden>
Date: Fri Jun 29 15:21:41 2018 +0200

Fix VMI delete

Remove VMIs from to_delete queue

Closes-Bug: #1779321

Use one global VLAN ID pool

Store DVSwitch as an attribute of VCenterAPIClient
Add VlanIdPool to VMI Service
Modify assign_vlan_id method to use global id pool
Modify restore_vlan_id method to use global id pool
Remove unused methods and tests
Add DVSwitch name as a config parameter

Closes-Bug: #1779100

More informative logs for API Clients
Changed logging level for database queries
Logging for CVM Models

Partial-Bug: #1779323

Controller refactor

Create Update Handler for storing all event handlers
Move all update event handling to a new handler
Add AbstractChangeHandler
Add GuestNetHandler for guest.net updates
Add VmwareToolsStatusHandler

Partial-Bug: #1777404

CVM checks ownership of VNC entities before modification

Closes-Bug: #1779885

Handle External IPAM cases

Create Instance IP for external IPAM
Remove unused functions, tests and imports

Closes-Bug: #1779896

React to powerState property changes

Add PowerStateHandler
Add runtime.powerState to observed properties
Add method for updating powerState in VM Service
Update update_vmware_tools_status method
Update update_nic method
Remove excessive logs

Change-Id: I16210e5c7ae53a111e284ea1329945d245c7b96f
Closes-Bug: #1778758

Changed in opencontrail:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers