Treat warning as errors on Windows
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Juniper Openstack | Status tracked in Trunk | |||||
Trunk |
In Progress
|
Undecided
|
Michał Cłapiński | |||
OpenContrail |
In Progress
|
Undecided
|
Michał Cłapiński |
Changed in opencontrail: | |
assignee: | nobody → Michał Cłapiński (michal.clapinski) |
status: | New → In Progress |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #1 |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #2 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #3 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #4 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #7 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit e40146ee7322cc9
Author: Michal Clapinski <email address hidden>
Date: Wed Oct 31 19:36:38 2018 +0100
Remove unused code
Change-Id: I034ef3b128f3a5
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #8 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit f084fef6a738f0b
Author: Michal Clapinski <email address hidden>
Date: Wed Oct 31 19:11:01 2018 +0100
Remove old leftover pragmas
Change-Id: Icadb1935596cf7
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #9 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 70383222ea79372
Author: Michal Clapinski <email address hidden>
Date: Wed Oct 31 19:14:10 2018 +0100
Fix warnings and treat warning as errors
Change-Id: I4ad042b1e82c50
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #10 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #11 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d1ef3dc9fecfb81
Author: Michał Cłapiński <email address hidden>
Date: Tue Nov 6 10:40:24 2018 +0100
Fix undefined reference
Because of lack of a gate job, two changes merged that broke the master.
This change should fix it.
Change-Id: I6aadac70bc0c2f
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #12 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #14 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #15 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #17 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 220c131c1fe48f9
Author: Mateusz Jablonski <email address hidden>
Date: Fri Dec 7 10:46:40 2018 +0100
Fix compilation warnings for 'io' and 'sandesh'
- Add explicit casting where needed
- Change types
- Change friend function to templated function
- Fix whitespaces at end of lines
Change-Id: I3cb98e20e0d0bb
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #18 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit a61610d3055fd14
Author: Mateusz Jablonski <email address hidden>
Date: Mon Dec 10 10:19:47 2018 +0100
Fix compilation warnings for 'base' directory
Change-Id: Id12cabee2bf52f
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #19 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #20 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #21 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #22 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #23 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 4c1c07f1ac62916
Author: Mateusz Jablonski <email address hidden>
Date: Thu Dec 13 14:01:24 2018 +0100
Split flow management to multiple files
Change-Id: I7b9591584f9592
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #24 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #25 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #26 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 5e87ecf2135a79e
Author: Mateusz Jablonski <email address hidden>
Date: Fri Dec 21 14:40:25 2018 +0100
Fix most of level 1 warnings on Windows
- Invalid escape sequence
- Invalid unused field and getter
- Invalid preprocessor usage
- Wrong type of field
- Missing return in non-void returning functions
Change-Id: Id54b649f328451
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #27 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #28 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #29 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #30 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #31 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #32 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #33 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #34 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #35 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #36 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #37 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 68241e3400d6092
Author: Mateusz Jablonski <email address hidden>
Date: Fri Dec 21 14:52:56 2018 +0100
Fix warning related to not called destructor
Change-Id: Ic30b29e43ec7d1
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #38 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #39 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #40 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #41 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #42 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #43 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #44 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #45 |
Review in progress for https:/
Submitter: Arun RS (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #46 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #47 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #48 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #49 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #50 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 659044fc0f63585
Author: Michal Clapinski <email address hidden>
Date: Thu Jan 3 15:40:52 2019 +0000
Fix warning related to missing consts
Change-Id: I618e4c0d517aa4
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #51 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 870474906715e14
Author: Mateusz Jablonski <email address hidden>
Date: Tue Jan 8 17:06:16 2019 +0100
Fix warnings: comparison between signed and unsigned integer
Change-Id: I3758f57ac0fd60
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #52 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #53 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #54 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #55 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0dc3ef2f499d7d5
Author: Michal Clapinski <email address hidden>
Date: Mon Dec 24 21:52:09 2018 +0000
Remove unused variables
Change-Id: I49c5e50032038a
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #56 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 467acae273394e9
Author: Mateusz Jablonski <email address hidden>
Date: Fri Dec 21 13:31:08 2018 +0100
Fix warnings related to type conversion
Change-Id: I76babc14248087
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #57 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit f5905df29183fe8
Author: Mateusz Jablonski <email address hidden>
Date: Thu Jan 3 13:04:50 2019 +0100
Use semicolons properly with 'while(false)' in macros
Change-Id: I0578486b681c28
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #58 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0abd60e850cfeb6
Author: Mateusz Jablonski <email address hidden>
Date: Thu Jan 3 13:21:53 2019 +0100
Use semicolons properly with 'while (false)' in macros
Change-Id: I7c51d830f210f1
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #59 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #61 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 4a75e5f8bd746f6
Author: Mateusz Jablonski <email address hidden>
Date: Thu Jan 3 17:49:07 2019 +0100
Refactor base/timer*
- Remove support for old boost (<1.49)
- Remove usage of auto_ptr
- Remove friend declarations
Change-Id: Ibb1089d108be3e
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #62 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit e922fdd98b877ee
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jan 4 10:34:10 2019 +0100
Fix type conversion related warnings in base/timer*
Change-Id: Id64e160aa72897
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #63 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #64 |
Review in progress for https:/
Submitter: Arun RS (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #65 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #66 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #67 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 4d9961e43caa89e
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jan 4 11:32:11 2019 +0100
Fix type conversion related warnings in base/task*
Change-Id: I2b4ba7fbeccd41
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #68 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 553828ebafc447d
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jan 4 12:30:59 2019 +0100
Fix type conversion related warnings for common/base/task*
Change-Id: I5d05594649a714
Depends-On: I2b4ba7fbeccd41
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #69 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 9893a40550fe794
Author: Mateusz Jablonski <email address hidden>
Date: Tue Jan 22 12:48:08 2019 +0100
Add comment about incorrect definition of __cplusplus in MSVC
Change-Id: I79420b57880b7a
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #70 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #71 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #72 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #73 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit bafda324ce7fd1c
Author: Mateusz Jablonski <email address hidden>
Date: Wed Jan 30 12:19:05 2019 +0100
Partially fix formatting in sandesh/
Change-Id: Icc2edf2aa709e4
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #74 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 97a14125a5d4046
Author: Mateusz Jablonski <email address hidden>
Date: Mon Jan 7 10:20:01 2019 +0100
Partially fix formatting in sandesh/
Change-Id: Ibc90e7f360152c
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #75 |
Reviewed: https:/
Committed: http://
Submitter: Vinay Vithal Mahuli (<email address hidden>)
Branch: R6.0-WIP
commit fe6ebdaec8e7c90
Author: mkheni <email address hidden>
Date: Thu Nov 8 13:23:42 2018 -0800
Apply commits from master onto R6.0-WIP
Initliaze Sandesh Collector on given IP.
if not ip is provided, listen on 0.0.0.0
Partial-bug: #1802399
Contrail Stats to external Stats Collector
User can provide external collector (ip:port or local domain socket),
If the external collector is provided, contrail generators will
send the UVEs converted into JSON format to the external collector
in addition to the Contrail Collector.
Closes-bug: #1793204
Check build dependencies for Sandesh
Partial-Bug: #1806719
Fix compilation warnings for 'io' and 'sandesh'
- Add explicit casting where needed
- Change types
- Change friend function to templated function
- Fix whitespaces at end of lines
Partial-Bug: #1800894
Fix compilation warnings for 'base' directory
Partial-Bug: #1800894
ParseInt function should check for Uint also
Rapidjson sets the flag of type UnsingedInt if the MSB
is set. While retrieving such values, we need to check
for Uint values also.
Closes-Bug: #218680
Make sandesh and ssl talk TLSv1.2
Use SSLv23 client and server for SSL connections and disable the more vulnerable
versions thereby ensuring the communication happens only on TLSv1.2
This ensures introspect and sandesh services now use TLS v1.2
Partial-Bug: #1807296
JSON encoding of the sandesh UVE messages
This fix adds the file wihch provides implementation of
transport API's exposed. The transport API's will be called
by the generated sandesh code.
UT added for the JSON generator. Currently the validity of
the JSON was checked with online tools. Holding the
verification part until we export a JSON library to sandesh.
Code Change:
JSON encoding unlike the XML encoding uses comma seperation
for members, string values for keys and {} enclosure for
complex types. To accomodate this following data structure
were introduced:
A vector to keep track of the element we are currently encoding
and whether its a first member of the familt or not:
Here is an example:
If we are encoding a list within sandesh structure,
current_
list viz., sandesh-
is_first_
list element or we not.If we were in the middle of the list,
the structure is_first_
true->false->false
Here the first false, indicates that list is not the first
member of the struct and the second false indicates that we
are done encoding the first element of the list.
Partial-Bug: 1789312
Do not resolve 127.*.*.* ip to names
DNS reverse resolution some times can take seconds.. Most of BGP and XMPP UTs
run over 127...* addresses and run timers in milliseconds. If asynchronous IO
operations do not complete very fast, then they are teared down and new ones
are created. Hence, hard-code 127.* addresses to localhost.
Also mark an arg in se...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #77 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R6.0-WIP
commit 77df3b58265b3fa
Author: Shivayogi Ugaji <email address hidden>
Date: Mon Nov 5 22:07:18 2018 -0800
Apply commits from master onto R6.0-WIP
db_resync_done lock is used to indicate the amqp thread to wait for resync to
complete. In this case, when we call SchemaTransform
due to Casandra connection failure, this lock remains locked blocking
destroy_instance. destroy_instance calls _vnc_subscribe_
amqp queue which waits infinitely for db_resync_done lock to be released.
This fix releases db_resync_done lock so that destroy_instance doesnt get
blocked.
Closes-Bug: #1801474
[DM] Hitless image upgrade implementation
Closes-Bug: #1799322
Provisioner for the devicemanager node.
usage:
from /opt/contrail/utils
python provision_
--oper add --admin_user admin --admin_password contrail123 --admin_tenant_name
admin --openstack_ip 10.87.82.2 --api_server_ip 10.87.82.2
Closes-Bug: #1805303
CFM: Changes for onboarding L3PNF
- Add new platform SRX240
- Add L3PNF subnet is schema
- Add new namespace, VN and IPAM for L3PNF during brownfield onboarding
Closes-Bug: 1800701
Add entrypoint to vrouter-agent service on Windows
Introduce entrypoint for agent similar in design to that from
microservice deployment. For now it will only start agent,
actual features will be added in following changes.
Partial-Bug: #1806677
Check build dependencies for tbb, SimpleAmqpClient and rabbitmq
Closes-Bug: #1806719
Make agent's entrypoint update agent's config on Windows
In future we will generate the whole config from scratch
as on Linux, but for now we only update the vhost's ifname.
It's the only field that can change upon restart.
Partial-Bug: #1806677
bgp-peer selection support for bgpaas
1. Listener BgpRouterConfig is added for BgpRouter and ControlNodeZone
2. BgpRouterConfig builds BgpRouterTree and ControlNodeZoneTree
from IFMapNode
3. BGPaaS gets BgpRouter for configured ControlNodeZone from
BgpRouterConfig and Updates bgp-peer-ip and bgp-peer-port in
the flow.
4. Step 3 is followed for xmpp based peer-selection also.
5. BGPaaS sandesh is updated with primary_
secondary_
Partial-bug: #1775872
[DM] Inside-outside workflow - lag/mH
1. Change the exisiting business logic to adhere to the new data model for lag/mH workflow
2. Multi-vlan support
Partial-Bug: #1799329
Rework nodemgr before fixing ntp issue
- move windows/linux code to separate classes instead of same condition through the code
- simplify main.py
- remove copy duplication
Closes-Bug: 1800704
[fabric] Added playbook retry support to job manager
1) When playbook return retry_devices in the output, job manager will retry the playbooks against those devices
2) remove obsolete playbooks from 5.0
3) remove obsolete ansible roles from 5.0
4) added a warning log on missing loopback interface when ...
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #79 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #80 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #81 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #82 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #83 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #84 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #85 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #86 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #87 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #88 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #89 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #90 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #91 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #92 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #93 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit cd660b087f559ac
Author: Mateusz Jablonski <email address hidden>
Date: Mon Jan 7 10:32:16 2019 +0100
Fix type conversion related warnings in TCP server related code
Change-Id: I8a88446435531e
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #94 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 32aed6df42a552d
Author: Mateusz Jablonski <email address hidden>
Date: Wed Feb 20 16:13:09 2019 +0100
Change tabs to spaces in 'base'
Change-Id: I73d39575fd26a9
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #95 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit b12f5d7ecf2595a
Author: Mateusz Jablonski <email address hidden>
Date: Thu Feb 21 12:44:36 2019 +0100
Change tabs to spaces in 'sandesh/compiler'
Change-Id: I7bc8111bdb32c8
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #96 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0d8d68b870e2e75
Author: Mateusz Jablonski <email address hidden>
Date: Fri Feb 22 15:25:07 2019 +0100
Use 'base/address_
Change-Id: Iaba70c9f745908
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #97 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit e1d45895760627c
Author: Mateusz Jablonski <email address hidden>
Date: Fri Feb 22 16:14:07 2019 +0100
Use 'base/address.h' instead copy of the file from 'net' directory
Change-Id: Ib1631e77595f1d
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #98 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0f27ae0f4ce9a48
Author: Mateusz Jablonski <email address hidden>
Date: Fri Feb 22 16:36:02 2019 +0100
Use 'base/address_
Change-Id: Ia3cc332eba4f5c
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #99 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 7a170d410f37d55
Author: Mateusz Jablonski <email address hidden>
Date: Fri Feb 22 16:44:50 2019 +0100
Use 'base/address.h' instead copy of the file from 'net' directory
Change-Id: I8c2e223f4ddc10
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #100 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8cccaac2c19e054
Author: Mateusz Jablonski <email address hidden>
Date: Fri Feb 22 15:30:28 2019 +0100
Use 'base/address_
Change-Id: I8cc285d7dcb68f
Depends-On: Iaba70c9f745908
Depends-On: Ia3cc332eba4f5c
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #101 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit a478a8668d35192
Author: Mateusz Jablonski <email address hidden>
Date: Fri Feb 22 16:19:27 2019 +0100
Use 'base/address.h' instead copy of the file from 'net' directory
Change-Id: I68864d1e5be668
Depends-On: Ib1631e77595f1d
Depends-On: I8c2e223f4ddc10
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #102 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #103 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #104 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #105 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #106 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #107 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #108 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit ee5fd0c13eea7c5
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 4 16:58:34 2019 +0200
Do not use '-fPIC' option on Windows
Fix 'Ignoring unknown option -fPIC' warning on Windows.
Change-Id: I0b6ee97eeb311b
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #109 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #110 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #111 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d4cd7538334a092
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 4 16:22:52 2019 +0200
Fix warning C4456 in io/udp_server.cc on Windows
Fix "declaration of 'lock' hides previous local declaration"
Change-Id: I7b34d9870278c5
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #112 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 7ce8700fae10911
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 4 16:32:44 2019 +0200
Enable level 2 warnings and treat them as errors for contrail-common on Windows
Change-Id: Ifc0f6fc6c3d3e9
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #113 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit eea376ea95b3007
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 4 16:53:51 2019 +0200
Do not use '-fPIC' option on Windows
Fix 'Ignoring unknown option -fPIC' warning on Windows.
Change-Id: I2097904d2ed108
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #114 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #115 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #116 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #117 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #118 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d1979a315b14213
Author: Mateusz Jablonski <email address hidden>
Date: Fri Apr 5 11:42:50 2019 +0200
Partially fix formatting in t_cpp_generator.cc
Change-Id: Icd06c8f65ceb15
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #119 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit ad74e8ad6752f20
Author: Mateusz Jablonski <email address hidden>
Date: Fri Apr 5 12:23:47 2019 +0200
Partially fix formatting in t_cpp_generator.cc
Change-Id: I7d574c23c701f1
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #120 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #121 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #122 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #123 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #124 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #125 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #126 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #127 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8a888b58785bf52
Author: Mateusz Jablonski <email address hidden>
Date: Mon Apr 8 11:48:48 2019 +0200
Partially fix formatting in t_cpp_generator.cc
Change-Id: I06c242b64398ad
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #128 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8ffa4b85289f4f3
Author: Mateusz Jablonski <email address hidden>
Date: Mon Apr 8 11:54:04 2019 +0200
Partially fix formatting in t_cpp_generator.cc
Change-Id: I6e425981e956ca
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #129 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 5fab9bbfd3257a5
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 9 13:13:14 2019 +0200
Remove duplicated set_name method from sandesh types
The set_name method is already defined in t_type base class.
Change-Id: I062868a7d00406
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #130 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #131 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #132 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit b80ed1faeea1727
Author: Mateusz Jablonski <email address hidden>
Date: Fri Apr 12 11:51:43 2019 +0200
Allow using of deprecated POSIX functions on Windows
Change-Id: Idc09dfa9a85f29
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #133 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #134 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0e13ed93e28cda6
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 9 14:55:31 2019 +0200
Remove code duplication from t_sandesh and t_struct
Introduce t_struct_common, a base class for
t_sandesh and t_struct and move duplicated code there.
Change-Id: I7407415a288232
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #135 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 10526d37328dcd8
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 9 15:51:02 2019 +0200
Remove code duplication in generate_
Introduce generate_
move code responsible for method body generation there.
Partial-Bug: #1800894
generate_
Change-Id: I8b4c211559c43a
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #136 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #137 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #138 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #139 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #140 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit f3c67e95d0008bf
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 9 16:47:56 2019 +0200
Add missing is_sandesh definition in t_sandesh
Change-Id: I0f3f30acd53954
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #141 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #142 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #144 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #145 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #146 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #147 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #148 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #149 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 657ae7db3fd48ac
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 18 15:30:04 2019 +0200
Fix formatting in dp-core
Change-Id: I821e2974c4bf64
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #150 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #151 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #152 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #153 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #154 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #155 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 1b448d4c471b7be
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 9 17:39:19 2019 +0200
Remove code duplication from generate_
Introduce generate_
move code responsible for method body generation there.
Change-Id: I38c076320afc4c
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #156 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 1abaa4ef883c877
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 18 12:06:12 2019 +0200
Fix warning about returning int when bool is expected
Change-Id: I8173095c1855c8
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #157 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit ff0d194b186d0b7
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 23 16:38:26 2019 +0200
Fix formatting in all '.sandesh' files in contrail-analytics
Change-Id: I59e008119f0ca2
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #158 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit e817052afaa728a
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 11 14:54:47 2019 +0200
Fix warning C4065 in t_cpp_generator for Windows
Fix `Switch statement contains 'default' but
no 'case' labels` by generating just the contents
of 'default' case when there is no other 'case' labels
instead of 'switch' with just 'default' label.
Change-Id: Ief8f75b144b8f6
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #159 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #160 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #161 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #162 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #163 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #164 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #165 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #166 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 85826146e9a964a
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 16 16:19:19 2019 +0200
Fix warnings C4311 and C4312 on Windows
Fix warnings about pointer <-> type conversions
which may cause truncation. Use boost::any instead.
Change-Id: I056fc9c9d74784
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #167 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #168 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit b108a458016fb1d
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 23 16:27:05 2019 +0200
Fix 'struct' <-> 'class' mismatch
Declare ConfigClientOptions as a struct because it
is defined as a struct in other part of the code.
Change-Id: I9f8ffba7331faa
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #169 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit f6d48acaec30823
Author: Mateusz Jablonski <email address hidden>
Date: Mon Apr 15 11:31:56 2019 +0200
Disable incorrectly generated warning C4724: Potential mod by 0
Change-Id: Iff8940e3c80932
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #170 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 348303c74d9956c
Author: Mateusz Jablonski <email address hidden>
Date: Wed Apr 17 16:51:05 2019 +0200
Fix warning C4099 in contrail-controller
Fix 'An object declared as a structure is defined as a class,
or an object declared as a class is defined as a structure'.
Change-Id: I1bbbf041f65306
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #171 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #172 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 4ece1c4d9ddbf3d
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 23 16:41:11 2019 +0200
Fix formatting in all '.sandesh' files in contrail-controller
Change-Id: Id294563e83b044
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #173 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit de1940505d030a5
Author: Mateusz Jablonski <email address hidden>
Date: Tue Apr 23 16:25:07 2019 +0200
Fix formatting in all '.sandesh' files in contrail-common
Change-Id: Ic3658cca85cb78
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #174 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit f9f45667d75051f
Author: Mateusz Jablonski <email address hidden>
Date: Thu Apr 11 16:16:51 2019 +0200
Fix warning C4065 in process_signal for Windows
1. Fix `Switch statement contains 'default' but no 'case' labels`.
2. Move all OS-specific code to platform-specific files.
Change-Id: I653dffc1fd01ce
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #175 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit ae99fe09db37028
Author: Mateusz Jablonski <email address hidden>
Date: Fri Apr 12 15:28:19 2019 +0200
Fix formatting in sandesh/library/cpp
Change-Id: I7188a787db5593
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #176 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #177 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #178 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0f3307f595ff1e0
Author: Mateusz Jablonski <email address hidden>
Date: Wed Apr 17 11:08:37 2019 +0200
Fix warning about returning no value from function that should return
Change-Id: Icf1912b61d3b95
Partial-Bug: #1800894
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #179 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #180 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #181 |
Review in progress for https:/
Submitter: Przemys?aw Marcinkiewicz (<email address hidden>)
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
OpenContrail Admin (ci-admin-f) wrote : | #182 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
Review in progress for https:/ /review. opencontrail. org/47400
Submitter: Micha? C?api?ski (<email address hidden>)