control-node functionality to retrieve config via cassandra
- Series trunk
- Bug #1632470
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Juniper Openstack | Status tracked in Trunk | |||||
R4.0 |
In Progress
|
Wishlist
|
Prakash Bailkeri | |||
R4.1 |
In Progress
|
Wishlist
|
Prakash Bailkeri | |||
Trunk |
In Progress
|
Wishlist
|
Prakash Bailkeri |
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #1 |
2 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #4 |
Reviewed: https:/
Committed: http://
Submitter: Zuul
Branch: master
commit a559aba3a53b841
Author: Tapan Karwa <email address hidden>
Date: Mon Oct 10 16:30:52 2016 -0700
Adding a few classes for control-node retrieving config via cassandra.
Partial-Bug: #1632470
Change-Id: I42c50e29885897
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #5 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #6 |
Reviewed: https:/
Committed: http://
Submitter: Zuul
Branch: master
commit 2a4eeab6eb44073
Author: Tapan Karwa <email address hidden>
Date: Tue Oct 25 13:54:53 2016 -0700
Code for control-node retrieving config via cassandra.
Add code to process config options for cassandra and start using
Db_UseColumnfamily with string.
Change-Id: Ibdfbac48609553
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #7 |
Review in progress for https:/
Submitter: Hampapur Ajay (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #8 |
Reviewed: https:/
Committed: http://
Submitter: Zuul
Branch: master
commit db8c947ea94365d
Author: Hampapur Ajay <email address hidden>
Date: Mon Nov 14 16:23:13 2016 -0800
Add specification for receiving configuration data from cassandra
to control-node. Submitting on behalf of tkarwa@
Change-Id: I2148f61007e185
Partial-Bug: 1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #9 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
1 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #11 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 63d6a8dca6a6faf
Author: Tapan Karwa <email address hidden>
Date: Wed Nov 16 15:20:56 2016 -0800
Code for control-node retrieving config via cassandra.
Add code to parse json messages. Add tests and json test data.
Change-Id: I1cb667a37dd402
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #12 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #13 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
1 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : | #15 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
1 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : | #17 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
1 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #19 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 90867da664858e6
Author: Tapan Karwa <email address hidden>
Date: Fri Dec 9 14:31:13 2016 -0800
Code for control-node retrieving config via cassandra.
Add code to read one row from cassandra and process the read columns. Also, add
code to convert the incoming column data to a json document that will finally
be fed to the json parser. The conversion is done by the ConfigCass2Json
class and handles the quirks in the data stored in cassandra.
Also, add code to read all-rows on startup (bulk sync).
Change-Id: I0ea706554024e3
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #20 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
1 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : | #22 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #23 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit daa49914ad6ba15
Author: Tapan Karwa <email address hidden>
Date: Mon Dec 5 14:29:28 2016 -0800
Code for control-node retrieving config via cassandra.
Add code to add links to parent.
Also, add code to perform a diff between previous and current json messages to
decide if properties are getting deleted. Add test for this.
Change-Id: I516a170aa97efe
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : | #24 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 95a47374423462d
Author: Tapan Karwa <email address hidden>
Date: Thu Dec 1 11:30:39 2016 -0800
Code to generate C++ code to handle input config data in json format.
Change-Id: I9787f1c318e964
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #25 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #26 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #27 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #28 |
Review in progress for https:/
Submitter: Tapan Karwa (<email address hidden>)
3 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : | #32 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #33 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 3a80c41f9d6230c
Author: Tapan Karwa <email address hidden>
Date: Fri Dec 30 09:30:11 2016 -0800
Code for control-node retrieving config via cassandra.
Change-Id: Iff4aa698ee794b
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : | #34 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 2f9757f1888e676
Author: Prakash M Bailkeri <email address hidden>
Date: Mon Jan 2 12:22:07 2017 +0000
Add rabbitmq-c and SimpleAmqpClient for c++
Add third party package detail for reading rabbitmq messages
Change-Id: I2162d82b4650a9
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #35 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #36 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 6fc054616355584
Author: Prakash Bailkeri <email address hidden>
Date: Mon Jan 2 23:02:59 2017 +0530
generateDS changes for getting wrapped fields
Auto generate a map of field to wrapped field for map and list property
Change-Id: I85f67652a1335a
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #37 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #38 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit e3bd9a324db8cee
Author: Prakash M Bailkeri <email address hidden>
Date: Wed Jan 4 18:19:38 2017 +0000
Config read from cassandra
Changes in this commit:
1. Json encoding for refs, list property, map property and parent
2. Bulk sync code. [Reading from fqname table, Starting multiple task to read in parallel etc]
3. Thirdparty packages for AMQP client and SConscript for same
4. Class for reading Rabbit messages
Other dependent review in progress
https:/
https:/
Change-Id: Idce47fd59fcdfd
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #39 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #40 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
216 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #257 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit af9d2265afa87ec
Author: Ananth Suryanarayana <email address hidden>
Date: Sun Mar 19 20:19:42 2017 -0700
Remove an unncessary call to dirty cache
Also clear input request_list uuid list in
ConfigCassandra
review)
Change-Id: I12e0764712e78f
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #258 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #259 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #260 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #261 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit aa800ba6d7e5004
Author: Ananth Suryanarayana <email address hidden>
Date: Wed Mar 15 12:47:09 2017 -0700
Add config_
Add specific tests to cover fq-name and uuid-table from cassandra db.
Simulate many possible scenarios using parametertized gtests. code coverage
reports 94% for this file when all ifmap UTs are run including this new test
Fix request list corruption issue there by found while running this test.
In the current code, assumption is that the entry being deleted in
RemoveObjReqEntry() is always at the front of the uuid_read_list_. This is not
the case when (after bunching a few entries), we encounter delete/eor. In that
case, delete/eor could be any where in the middle/end of the list (as its
previous entries are still not processed). But by removing wrong head of the
list, we not only take wrong entry from the list we also end up deleted entry
to remain in the list. Code can crash later on, when this freed entry is
accessed.
This is fixed by storing list of iterators to the entries in the list, instead
of entries themselves. By this, we can still delete the entry in constant time
irrespective of where it is present in the list. (IOW, we still get pop()
equivalent semantics like how it is the current code)
Change-Id: I66f6adb07eaf18
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #262 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #263 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #264 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
2 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #267 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 8cd721b7df8db04
Author: Ananth Suryanarayana <email address hidden>
Date: Fri Mar 17 11:50:32 2017 -0700
Add config_
Create ConfigAmqpChannel class to invoke AmqpClient::.*() and virtualize them
to override them in config_
for config_
Change-Id: Ie9003714d84cfb
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : | #268 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 300d42efc3b1037
Author: Ananth Suryanarayana <email address hidden>
Date: Mon Mar 20 15:00:27 2017 -0700
Remove some of spurious includes
Partial-Bug: #1632470
Change-Id: I47fd227c37f0ef
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #269 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #270 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #271 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #272 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #273 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 172d76517805fbd
Author: Ananth Suryanarayana <email address hidden>
Date: Wed Mar 29 12:17:11 2017 -0700
Initialize bool terminate_ in ConfigAmqpClien
This is used only in tests though, to terminate rabbit-mq msg receive loop
Change-Id: Ia7738b23afaf32
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #274 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #275 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 256d0a19a0947c4
Author: Ananth Suryanarayana <email address hidden>
Date: Mon Mar 27 15:34:17 2017 -0700
Add config_
Enqueue various combinations of updates and deletes for a set of uuids.
Pause work-queues and then resume to make sure that uuids work items get
queued up before they get processed. Add duplicate uuid operations and ensure
that they get processed correctly
Change-Id: I2fd2c36012fd4b
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #276 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #277 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 0cd7be4765fa170
Author: Prakash Bailkeri <email address hidden>
Date: Mon Apr 3 15:01:47 2017 +0530
Remove the UUID proces list
In the current code, UUID request-list or process-list is stored in two container.
1. List - Ensure the order of read/update
2. SET - To supress duplicate insertion of UUID to UUID process list.
Currently, since the UUID reading is done in multiple partition, the ordering of
read is not ensured. Code does handle the out of order reading of UUID in the
config reader task.
Remove the need for list(time ordered list).
Change-Id: Ia4734b3da90d58
Related-bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #278 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
2 comments hidden Loading more comments | view all 296 comments |
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #281 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 09e794df47482c0
Author: Prakash Bailkeri <email address hidden>
Date: Thu Apr 6 15:55:38 2017 +0530
Introduce partition concept for config workers
1. Code re-org in ConfigCassandra
2. Introduce Partition for ConfigCassandra
related to workers to Partition. e.g. obj cache, obj_req_set, task trigger and
work queue.
3. Remove iterator list RemoveObjReqEnt
4. Fix memory leak in config_
Change-Id: Id4eb741956e53e
Related-bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #282 |
Review in progress for https:/
Submitter: Nischal Sheth (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #283 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #284 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit a634ce74595c83f
Author: Prakash Bailkeri <email address hidden>
Date: Sat Apr 8 10:32:46 2017 +0530
Fix compilation error in bgp_stress_test
1. Error got introduced due to code re-org of ConfigCassandra
2. Incorporate review comments
Change-Id: I254e834309d8e0
Related-bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #285 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #286 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 129ba5c178f7969
Author: Prakash Bailkeri <email address hidden>
Date: Wed Apr 5 10:29:12 2017 +0530
Introspect for config client
Introduce introspect pages for config cassandra client for following data
1. FQName Cache
2. Object Cache
Added test cases for introspect validation
TODO: Basic test for ReqIterate is added for both the request. Need to add test
for bulk data to validate batching.
Related-bug: #1632470
Change-Id: I4992047fc34af2
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #287 |
Review in progress for https:/
Submitter: Prakash Bailkeri (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #288 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 15abf924b5a5437
Author: Prakash Bailkeri <email address hidden>
Date: Thu Apr 20 11:48:45 2017 +0530
Config cassandra client: Fix compile error in optimization build
Compile Error while making production build
In file included from /usr/include/
from /usr/include/
from /usr/include/
from build/include/
from controller/
/usr/include/
/usr/include/
}
^
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:/
Fix:
Remove the boost bind and pass function pointer to Sort method.
Change-Id: Ia513b3fc85ccea
Related-bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #289 |
Review in progress for https:/
Submitter: Vinay Vithal Mahuli (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #290 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit c1cedeaf68bdfd4
Author: Ananth Suryanarayana <email address hidden>
Date: Mon Jan 23 16:22:18 2017 -0800
Add rapidjson v1.1.0 code
https:/
commit f54b0e47a08782a
Merge: ab791ae 17254e0
Author: Milo Yip <email address hidden>
Date: Thu Aug 25 15:59:05 2016 +0800
Merge pull request #714 from miloyip/
Version 1.1.0
Change-Id: Icdffc6381bce20
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.0 | #291 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #292 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1 | #293 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #294 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: R4.1
commit 7d92395488f804f
Author: Ananth Suryanarayana <email address hidden>
Date: Tue Oct 17 16:54:18 2017 -0700
Change default project to default-
Change-Id: I70ffdf6e132bce
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : | #295 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: R4.0
commit 0c2b6fd7589e200
Author: Ananth Suryanarayana <email address hidden>
Date: Tue Oct 17 16:54:18 2017 -0700
Change default project to default-
Change-Id: I70ffdf6e132bce
Partial-Bug: #1632470
OpenContrail Admin (ci-admin-f) wrote : | #296 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit d7264b5827b9a62
Author: Ananth Suryanarayana <email address hidden>
Date: Tue Oct 17 16:54:18 2017 -0700
Change default project to default-
Change-Id: I70ffdf6e132bce
Partial-Bug: #1632470
Review in progress for https:/ /review. opencontrail. org/24799
Submitter: Tapan Karwa (<email address hidden>)