Juniper Openstack

Vm project UUID should be optional for Port Add requests

  1. Series r3.2
  2. Bug #1664819
Bug #1664819 reported by Ashok Singh
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
Medium
Ashok Singh
Juniper Openstack r3.0.4.0
R3.0.3.x
Fix Committed
Medium
Ashok Singh
Juniper Openstack r3.0.3.3
R3.1
Fix Committed
Medium
Ashok Singh
Juniper Openstack r3.1.3.0
R3.2
Fix Committed
Medium
Ashok Singh
Juniper Openstack r3.2.1.0
Trunk
Fix Committed
Medium
Ashok Singh
Juniper Openstack r4.0.0.0-fcs "r4.0.0.0"

Bug Description

Currently the REST interface of contrail-vrouter-agent for Port Adds, makes it mandatory for vm-project-uuid to be specified when the port type is NovaVMPort. In case of thrift interface this was not mandatory. To ensure backward compatibility, vm-project-uuid should be optional even in REST interface

Tags: vrouter
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.2

Review in progress for https://review.opencontrail.org/28777
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/28833
Submitter: Ashok Singh (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/28777
Committed: http://github.org/Juniper/contrail-controller/commit/4de1b8771413f9d321cfc748964f661bd45d9b0a
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit 4de1b8771413f9d321cfc748964f661bd45d9b0a
Author: ashoksingh <email address hidden>
Date: Wed Feb 15 09:28:28 2017 +0530

Vm project UUID should be optional for Port Add requests.

Currently the REST interface of contrail-vrouter-agent for Port Adds, makes it mandatory for vm-project-uuid to be specified when the port type is NovaVMPort. In case of thrift interface this was not mandatory. To ensure backward compatibility, vm-project-uuid should be optional even in REST interface.
Also change the default value for port-type in vrouter_api.py to NameSpacePort to ensure backward compatibility.

Change-Id: I04620fe9b2997c39e46d3069cef8fb82d9f22270
Closes-Bug: #1664819

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

Reviewed: https://review.opencontrail.org/28833
Committed: http://github.org/Juniper/contrail-controller/commit/41a3809085fb940e578922fd9fb466bd3dd7e21d
Submitter: Zuul (<email address hidden>)
Branch: master

commit 41a3809085fb940e578922fd9fb466bd3dd7e21d
Author: ashoksingh <email address hidden>
Date: Thu Feb 16 08:49:07 2017 +0530

Vm project UUID is made optional for Port Add requests.

Currently the REST interface of contrail-vrouter-agent for Port Adds, makes it mandatory for vm-project-uuid to be specified when the port type is NovaVMPort. In case of thrift interface this was not mandatory. To ensure backward compatibility, vm-project-uuid should be optional even in REST interface. Also change the default value for port-type in vrouter_api.py to NameSpacePort to ensure backward compatibility.

Change-Id: I670229824923f5f282f3f7e98a85c85dcb09f2b2
Closes-Bug: #1664819

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

Review in progress for https://review.opencontrail.org/28976
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/28977
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/28978
Submitter: Ashok Singh (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/28976
Committed: http://github.org/Juniper/contrail-controller/commit/de72bafd363f96048ece41785b0b6f55e64f3a0d
Submitter: Zuul (<email address hidden>)
Branch: R3.0.3.x

commit de72bafd363f96048ece41785b0b6f55e64f3a0d
Author: ashoksingh <email address hidden>
Date: Tue Feb 21 17:04:57 2017 +0530

Vm project UUID should be optional for Port Add requests.

Currently the REST interface of contrail-vrouter-agent for Port Adds, makes it mandatory for vm-project-uuid to be specified when the port type is NovaVMPort. In case of thrift interface this was not mandatory. To ensure backward compatibility, vm-project-uuid should be optional even in REST interface.
Also change the default value for port-type in vrouter_api.py to NameSpacePort to ensure backward compatibility.

Change-Id: Ifc434992f87d8d8cff2da169d6bfeb308b9903bb
Closes-Bug: #1664819

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

Reviewed: https://review.opencontrail.org/28977
Committed: http://github.org/Juniper/contrail-controller/commit/bddcd93f1fc8eb00607d62c7f688f6d000f24419
Submitter: Zuul (<email address hidden>)
Branch: R3.0

commit bddcd93f1fc8eb00607d62c7f688f6d000f24419
Author: ashoksingh <email address hidden>
Date: Tue Feb 21 17:04:57 2017 +0530

Vm project UUID should be optional for Port Add requests.

Currently the REST interface of contrail-vrouter-agent for Port Adds, makes it mandatory for vm-project-uuid to be specified when the port type is NovaVMPort. In case of thrift interface this was not mandatory. To ensure backward compatibility, vm-project-uuid should be optional even in REST interface.
Also change the default value for port-type in vrouter_api.py to NameSpacePort to ensure backward compatibility.

Closes-Bug: #1664819
(cherry picked from commit de72bafd363f96048ece41785b0b6f55e64f3a0d)

Change-Id: Iacad9fc68acd6affcf9f8c3908644924407e8077

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

Review in progress for https://review.opencontrail.org/29029
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/29030
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/29031
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/29032
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/29033
Submitter: Ashok Singh (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/28978
Committed: http://github.org/Juniper/contrail-controller/commit/d50cc11f5ac266e3073d5a225393e8efc9fb075a
Submitter: Zuul (<email address hidden>)
Branch: R3.1

commit d50cc11f5ac266e3073d5a225393e8efc9fb075a
Author: ashoksingh <email address hidden>
Date: Thu Feb 16 08:49:07 2017 +0530

Vm project UUID is made optional for Port Add requests.

Currently the REST interface of contrail-vrouter-agent for Port Adds, makes it mandatory for vm-project-uuid to be specified when the port type is NovaVMPort. In case of thrift interface this was not mandatory. To ensure backward compatibility, vm-project-uuid should be optional even in REST interface. Also change the default value for port-type in vrouter_api.py to NameSpacePort to ensure backward compatibility.

Change-Id: I235bc3beb9adb9a75f74fe1db67d4f689ef92f2f
Closes-Bug: #1664819

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

Reviewed: https://review.opencontrail.org/29029
Committed: http://github.org/Juniper/contrail-controller/commit/392d0bee744cf066d6a1b901d294d9ea62bfeb57
Submitter: Zuul (<email address hidden>)
Branch: R3.0

commit 392d0bee744cf066d6a1b901d294d9ea62bfeb57
Author: ashoksingh <email address hidden>
Date: Wed Feb 22 16:19:45 2017 +0530

Add missing return statement

The return statement was missed when vrouter_api.py was modified to use REST interface instead of Thrift.

Change-Id: I7ee34a5cf8dd623edc0598039174b4343e810d96
Closes-Bug: #1664819

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

Reviewed: https://review.opencontrail.org/29032
Committed: http://github.org/Juniper/contrail-controller/commit/b5c60656988a3ab1914d958dfe04b1ab8393bf93
Submitter: Zuul (<email address hidden>)
Branch: R3.1

commit b5c60656988a3ab1914d958dfe04b1ab8393bf93
Author: ashoksingh <email address hidden>
Date: Wed Feb 22 16:19:45 2017 +0530

Add missing return statement

The return statement was missed when vrouter_api.py was modified to use REST interface instead of Thrift.

Closes-Bug: #1664819
(cherry picked from commit 392d0bee744cf066d6a1b901d294d9ea62bfeb57)

Change-Id: I4c1adb4eafc09eb410f3713e2957c27553b015f9

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

Reviewed: https://review.opencontrail.org/29030
Committed: http://github.org/Juniper/contrail-controller/commit/d1304ee42321e76b5c63a751387782c46992d3ff
Submitter: Zuul (<email address hidden>)
Branch: master

commit d1304ee42321e76b5c63a751387782c46992d3ff
Author: ashoksingh <email address hidden>
Date: Wed Feb 22 16:19:45 2017 +0530

Add missing return statement

The return statement was missed when vrouter_api.py was modified to use REST interface instead of Thrift.

Closes-Bug: #1664819
(cherry picked from commit 392d0bee744cf066d6a1b901d294d9ea62bfeb57)

Change-Id: Ida098e9f317aedc0e042b90a57553e93a18fdfc0

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

Reviewed: https://review.opencontrail.org/29033
Committed: http://github.org/Juniper/contrail-controller/commit/6e3d4a8ac336f78baaa46877ad833dc9f5752707
Submitter: Zuul (<email address hidden>)
Branch: R3.0.3.x

commit 6e3d4a8ac336f78baaa46877ad833dc9f5752707
Author: ashoksingh <email address hidden>
Date: Wed Feb 22 16:19:45 2017 +0530

Add missing return statement

The return statement was missed when vrouter_api.py was modified to use REST interface instead of Thrift.

Closes-Bug: #1664819
(cherry picked from commit 392d0bee744cf066d6a1b901d294d9ea62bfeb57)

Change-Id: Ie0529cf9e2318259598bd337a38cb22c8ffe5d4b

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

Reviewed: https://review.opencontrail.org/29031
Committed: http://github.org/Juniper/contrail-controller/commit/ea177450189fb65998517aa743e9cf1171777140
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit ea177450189fb65998517aa743e9cf1171777140
Author: ashoksingh <email address hidden>
Date: Wed Feb 22 16:19:45 2017 +0530

Add missing return statement

The return statement was missed when vrouter_api.py was modified to use REST interface instead of Thrift.

Closes-Bug: #1664819
(cherry picked from commit 392d0bee744cf066d6a1b901d294d9ea62bfeb57)

Change-Id: Ie588903284821a45246db20b583c37ae43c90b54

Ashish Ranjan (aranjan-n)
information type: Proprietary → Public
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.