Juniper Openstack

Remove agent's thrift interface for port add and deletes

  1. Series trunk
  2. Bug #1623372
Bug #1623372 reported by Ashok Singh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Won't Fix
Low
Ashok Singh
R3.0.2.x
Won't Fix
Low
Ashok Singh
R3.0.3.x
Won't Fix
Low
Ashok Singh
R3.1
Won't Fix
Low
Ashok Singh
Trunk
Fix Committed
Low
Ashok Singh
Juniper Openstack r3.2.0.0-fcs "r3.2.0.0"

Bug Description

Agent thrift interface for port add/delete is replaced by REST interface. Thrift is not used by any clients and needs to be removed. This will ensure that agent does not open port 9090.

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

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

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

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

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

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

commit 779b8f8967b1b290c481ead3bf99483a1ac9cea5
Author: ashoksingh <email address hidden>
Date: Tue Sep 20 14:20:45 2016 +0530

Replace Thrift calls for Port add/deletes with REST

Update vrouter_api.py script to use REST APIs instead of thrift.
Remove thrift based test-cases

Change-Id: I776308da4ac710fa01fe55da1b3059745214eea5
Partial-Bug: #1623372

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

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

commit 459d86aba8e19f0fbc6d24575adf831c477c9479
Author: ashoksingh <email address hidden>
Date: Wed Sep 21 18:43:23 2016 +0530

Provide REST for Virtual Gateway adds and deletes

Agent already has thrift interface virtual gateway add/deletes. Providing REST API for these, so that thrift interface can be removed.

Change-Id: Ia3ad6fe67139d3dd91c867210ff9f23176940e6a
Partial-Bug: #1623372

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

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

commit 5b05b0ed7d576b7b367d7e05ddeb956785b129ee
Author: ashoksingh <email address hidden>
Date: Thu Sep 22 18:25:02 2016 +0530

Use REST API for Virtual Gateway adds and deletes

The thrift interface of agent for VGW adds/deletes will be removed.
Update the VGW provision script to use REST API instead of thrift.

Change-Id: I2dd45e10a49619b103d01dab996f2e9cd770bc3f
Partial-Bug: #1623372

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

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

Hari Prasad Killi (haripk)
tags: added: releasenote
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

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

commit cdf54088dc75a63aee64417a177b3c8591fdbe9c
Author: ashoksingh <email address hidden>
Date: Thu Sep 29 12:03:12 2016 +0530

Remove Agent’s thrift interface.

Agent thrift interface for port add/delete is replaced by REST interface.
Removing thrift interface as it is not required any more.

Change-Id: I71195d5d7a6c1feb134b63455b13dad2e17edbd2
Closes-Bug: #1623372

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

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

commit b6c8a707410b9365293798186787654348879d14
Author: ashoksingh <email address hidden>
Date: Thu Oct 6 10:23:27 2016 +0530

Remove references to agent/openstack directory.

As part of removal of Agent’s thrift interface, agent/openstack directory
was removed, but some references to that folder were left. Removing
those now.

Change-Id: Ia8bd59804520d3e9725ae76ab122a9043ab26a40
Closes-Bug: #1623372

Hari Prasad Killi (haripk)
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/27080
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/27081
Submitter: Ashok Singh (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/27082
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/27083
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/27082
Committed: http://github.org/Juniper/contrail-controller/commit/bd890daadb06bb8341bfdf217ab2c0c53988311d
Submitter: Zuul (<email address hidden>)
Branch: R3.0.2.x

commit bd890daadb06bb8341bfdf217ab2c0c53988311d
Author: ashoksingh <email address hidden>
Date: Tue Sep 20 14:20:45 2016 +0530

Replace Thrift calls for Port add/deletes with REST

Update vrouter_api.py script to use REST APIs instead of thrift.
Remove thrift based test-cases

Partial-Bug: #1623372
(cherry picked from commit 779b8f8967b1b290c481ead3bf99483a1ac9cea5)

Change-Id: Iccaac60466f3dc7a3c94eed0eba7731e4da60699

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

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

commit 8c1eacfb85bb878f695c67bc6ef3ca6ed9e8f127
Author: ashoksingh <email address hidden>
Date: Tue Sep 20 14:20:45 2016 +0530

Replace Thrift calls for Port add/deletes with REST

Update vrouter_api.py script to use REST APIs instead of thrift.
Remove thrift based test-cases

Partial-Bug: #1623372
(cherry picked from commit 779b8f8967b1b290c481ead3bf99483a1ac9cea5)

Change-Id: Iddfe52537e752bb47d81cf97a07b1c9fea2aa81c

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

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

commit 6a8e4b32a4179fb097ff714af5b0a087ba08cce5
Author: ashoksingh <email address hidden>
Date: Tue Sep 20 14:20:45 2016 +0530

Replace Thrift calls for Port add/deletes with REST

Update vrouter_api.py script to use REST APIs instead of thrift.
Remove thrift based test-cases

Partial-Bug: #1623372
(cherry picked from commit 779b8f8967b1b290c481ead3bf99483a1ac9cea5)

Change-Id: Ib4b16171bbaaac2d5eaa2fb666d4685aa33643dd

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

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

commit a1458bc3441791fa878b493cc3e086b83c674ffa
Author: ashoksingh <email address hidden>
Date: Tue Sep 20 14:20:45 2016 +0530

Replace Thrift calls for Port add/deletes with REST

Update vrouter_api.py script to use REST APIs instead of thrift.
Remove thrift based test-cases

Partial-Bug: #1623372
(cherry picked from commit 779b8f8967b1b290c481ead3bf99483a1ac9cea5)

Change-Id: I9c56c4fee984cb139b644ff25776dd04fdd66127

Revision history for this message
Ashok Singh (ashoksr) wrote :

In R3.0, R3.0.2.x, R3.0.3.x and R3.1 branches vrouter_api.py script will use Thrift interface itself. The changes made as part of this bug to make vrouter_api.py use REST API were reverted via https://bugs.launchpad.net/juniperopenstack/+bug/1665309

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.