vnc_openstack/neutron plugin v3 SG optimisations

Bug #1480901 reported by Sylvain Afchain on 2015-08-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R1.1
Won't Fix
Medium
Sylvain Afchain
Trunk
In Progress
Medium
Sylvain Afchain
OpenContrail
New
Medium
Sylvain Afchain

Bug Description

Opened this bug to track performance related patches around the security groups in vnc_openstack and the new V3 plugin

Changed in opencontrail:
assignee: nobody → Sylvain Afchain (sylvain-afchain)

Review in progress for https://review.opencontrail.org/12806
Submitter: Sylvain Afchain (<email address hidden>)

OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12807
Submitter: Sylvain Afchain (<email address hidden>)

OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12809
Submitter: Sylvain Afchain (<email address hidden>)

OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12810
Submitter: Sylvain Afchain (<email address hidden>)

OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12809
Submitter: Sylvain Afchain (<email address hidden>)

OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12810
Submitter: Sylvain Afchain (<email address hidden>)

Changed in opencontrail:
importance: Undecided → Medium
tags: added: config neutron
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12809
Submitter: Sylvain Afchain (<email address hidden>)

OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12810
Submitter: Sylvain Afchain (<email address hidden>)

OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/12809
Submitter: Sylvain Afchain (<email address hidden>)

Review in progress for https://review.opencontrail.org/12806
Submitter: Sylvain Afchain (<email address hidden>)

Reviewed: https://review.opencontrail.org/12807
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/e7573d1f38df4a6ba4f1a8e9e87303ae80877777
Submitter: Zuul
Branch: master

commit e7573d1f38df4a6ba4f1a8e9e87303ae80877777
Author: Sylvain Afchain <email address hidden>
Date: Mon Aug 3 12:36:05 2015 +0000

Avoid reading twice the same project

This patch avoid an API call by adding
the security_groups field.

Change-Id: I49933c97f4724362878fb8bd8f800e65eb82f4c6
Partial-Bug: #1480901

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

commit d71fe4c662f87abd08cafc8457d453320e20f3c6
Author: Sylvain Afchain <email address hidden>
Date: Mon Aug 3 12:27:01 2015 +0000

Avoid reading twice the same project

This patch avoid an API call by adding
the security_groups field.

Change-Id: I9571788a5f875dddf7ea495d4837496399371da3
Partial-Bug: #1480901

Reviewed: https://review.opencontrail.org/12810
Committed: http://github.org/Juniper/contrail-controller/commit/2bbf5478b5faf378c50bf43f782a5cca16cf7566
Submitter: Zuul
Branch: master

commit 2bbf5478b5faf378c50bf43f782a5cca16cf7566
Author: Sylvain Afchain <email address hidden>
Date: Mon Aug 3 14:31:41 2015 +0000

Use obj_uuids to filter SGs instead of prune phase

Change-Id: I9ce41d519795ded195e4773d84dbd09b5cf40541
Partial-Bug: #1480901

Reviewed: https://review.opencontrail.org/12809
Committed: http://github.org/Juniper/contrail-controller/commit/529c74346235eccc0be51ae58f8c8a5421a51964
Submitter: Zuul
Branch: master

commit 529c74346235eccc0be51ae58f8c8a5421a51964
Author: Sylvain Afchain <email address hidden>
Date: Wed Aug 5 13:43:04 2015 +0000

Avoid trace when listing SG with non-existing tenant_id

Currently when listing SGs with tenant_id that doesn't exist
we get a stacktrace. The behaviour of neutron/ML2 is to create
a default SG for the tenant_id given even if the tenant
doesn't exist. For contrail is seems to be better to return
an empty answer is this case.

This patch remove the traceback and return nothing
when listing SGs with a non existing tenant id.

Change-Id: I5521c7630036ed28f6c87ed542204800086e50bc
Partial-Bug: #1480901

Review in progress for https://review.opencontrail.org/13287
Submitter: Numan Siddique (<email address hidden>)

Reviewed: https://review.opencontrail.org/13287
Committed: http://github.org/Juniper/contrail-neutron-plugin/commit/f92b685ae8dd1d40f041f9c75753f9115000e344
Submitter: Zuul
Branch: master

commit f92b685ae8dd1d40f041f9c75753f9115000e344
Author: Numan Siddique <email address hidden>
Date: Tue Aug 25 15:55:44 2015 +0530

Fix few issues related to SG and SG rules.

1. Use obj_uuids to filter SGs instead of prune phase.
security-group-rule-list was applying the 'id' filter for
the sg instead of sg rule.
This patch also fixes this bug.
(partially cherry picked from I9ce41d519795ded195e4773d84dbd09b5cf40541
of vnc_openstack)
2. Avoid trace when listing SG with non-existing tenant_id
(cherry-picked from I5521c7630036ed28f6c87ed542204800086e50bc of
vnc_openstack)

Change-Id: I493bd2a9e3820318747e5c4f351181d2b7e41047
Partial-Bug: #1480901

Review in progress for https://review.opencontrail.org/15455
Submitter: Jean-Philippe Braun (<email address hidden>)

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers