Adding same security rules should raise Conflict (Neutron plugin V3)

Bug #1485026 reported by eon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Undecided
Unassigned
R2.21.x
In Progress
Undecided
Unassigned
Trunk
Fix Committed
Undecided
Unassigned
OpenContrail
Fix Committed
Undecided
Unassigned

Bug Description

Tempest tests expect a Conflict error when the same security rule is added multiple times

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

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

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

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

commit 14bbce77a91e666f779ae9feb98ea30aa6249564
Author: Jean-Philippe Braun <email address hidden>
Date: Tue Aug 18 11:49:02 2015 +0200

Return RefsExistError when secrule already exists

Check duplicate rules API side and catch RefsExistError plugin side.

Change-Id: Ie4c9cb99ec08dd74b4896c67f13d016a81441353
Partial-Bug: #1485026

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

commit e7d885f077951ba8ed6c488a83709e4451c87113
Author: Jean-Philippe Braun <email address hidden>
Date: Fri Aug 14 17:33:01 2015 +0200

Raise SecurityGroupRuleExists when the secrule already exists

Change-Id: Id63ec1f10a65677251e4920bffc92028f74c68f3
Depends-On: Ie4c9cb99ec08dd74b4896c67f13d016a81441353
Partial-Bug: #1485026

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

Review in progress for https://review.opencontrail.org/13984
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/13984
Committed: http://github.org/Juniper/contrail-controller/commit/863a418a0f2b3b66da2a82240c2fd1371a19d4a1
Submitter: Zuul
Branch: master

commit 863a418a0f2b3b66da2a82240c2fd1371a19d4a1
Author: Hampapur Ajay <email address hidden>
Date: Tue Sep 22 10:04:00 2015 -0700

Avoid dict comprehension for python 2.6 in acl rule eexist check.

Use gevent based retry for connect and command retries

Change-Id: I962a10d5f9c804ebf26cca2c6133bab45a04f5ee
Partial-Bug: #1485026

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

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

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

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

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

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

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

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

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

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

Reviewed: https://review.opencontrail.org/18662
Committed: http://github.org/Juniper/contrail-test/commit/cbb4481c88c975df77a6612f6f6d6a14852d99e4
Submitter: Zuul
Branch: R2.20

commit cbb4481c88c975df77a6612f6f6d6a14852d99e4
Author: Ritam Gangopadhyay <email address hidden>
Date: Tue Sep 15 14:15:43 2015 +0530

Removing duplicate rules from policy.

Removing duplicate rules from policy policy02 in test_policy_modify_vn_policy
because neutron doesn't accept duplicate rules in a policy anymore.

Adding the test test_policy_modify_vn_policy back to mainline ci sanity.

Partial-Bug: #1485026
Change-Id: Idc24567318b5168541f3b1ad0660d6cffb81760e
(cherry picked from commit 4f2f62eefa4c46ccd23cd34a69071935f5ea714d)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/18661
Committed: http://github.org/Juniper/contrail-test/commit/0644b5ebfcc57207a4c217462e0fec0642bd7a9a
Submitter: Zuul
Branch: R2.21.x

commit 0644b5ebfcc57207a4c217462e0fec0642bd7a9a
Author: Ritam Gangopadhyay <email address hidden>
Date: Tue Sep 15 14:15:43 2015 +0530

Removing duplicate rules from policy.

Removing duplicate rules from policy policy02 in test_policy_modify_vn_policy
because neutron doesn't accept duplicate rules in a policy anymore.

Adding the test test_policy_modify_vn_policy back to mainline ci sanity.

Partial-Bug: #1485026
Change-Id: Idc24567318b5168541f3b1ad0660d6cffb81760e
(cherry picked from commit 4f2f62eefa4c46ccd23cd34a69071935f5ea714d)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

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

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

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

Sachin Bansal (sbansal)
Changed in opencontrail:
status: New → Fix Committed
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.