alarm schema change for uve_keys field to circumvent the issue in c++ code generation

Bug #1604539 reported by Sundaresan Rajangam
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.1
Fix Committed
High
Sundaresan Rajangam
Trunk
Fix Committed
High
Sundaresan Rajangam

Bug Description

Presently, there is a limitation with the c++ code generation if the top level field in the schema is a list. To circumvent this limitation, need to wrap the uve_keys field (presently a list) in the alarm schema.

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

Review in progress for https://review.opencontrail.org/22244
Submitter: Sundaresan Rajangam (<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/22245
Submitter: Sundaresan Rajangam (<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/22295
Submitter: Sundaresan Rajangam (<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/22299
Submitter: Sundaresan Rajangam (<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/22244
Submitter: Sundaresan Rajangam (<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/22245
Submitter: Sundaresan Rajangam (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/22245
Committed: http://github.org/Juniper/contrail-controller/commit/e6b39d87bbe7f18beec8050e32339082c27eda12
Submitter: Zuul
Branch: R3.1

commit e6b39d87bbe7f18beec8050e32339082c27eda12
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jul 20 18:12:29 2016 -0700

alarm schema change for uve_keys field

- Presently, there is a limitation with the c++ code generation if the top
level field in the schema is a list. To circumvent this limitation, need
to wrap the uve_keys field (presently a list) in the alarm schema.
- Renamed 'vars' to 'variables' in AlarmExpression

Change-Id: I5159405bab2c746b907bbb04b20bc38bede9a04c
Partial-Bug: #1604539
(cherry picked from commit 7c97ec8c59b2516b2b1af4c4cb613f76495236c1)

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

Reviewed: https://review.opencontrail.org/22295
Committed: http://github.org/Juniper/contrail-sandesh/commit/dba559487f1536e24cbf79cb5e543f88c89c0e93
Submitter: Zuul
Branch: master

commit dba559487f1536e24cbf79cb5e543f88c89c0e93
Author: Sundaresan Rajangam <email address hidden>
Date: Thu Jul 21 13:55:29 2016 -0700

Renamed fields in sandesh alarm structure

- Renamed "vars" to "variables" and "json_vars" to "json_variables"

Change-Id: I5f0a3f8dda8787a36ae5100839aa58f08a4a127a
Partial-Bug: #1604539

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

Reviewed: https://review.opencontrail.org/22299
Committed: http://github.org/Juniper/contrail-sandesh/commit/a036a9e4fd434e0d16e9d47b65af9e4eeae4a008
Submitter: Zuul
Branch: R3.1

commit a036a9e4fd434e0d16e9d47b65af9e4eeae4a008
Author: Sundaresan Rajangam <email address hidden>
Date: Thu Jul 21 13:55:29 2016 -0700

Renamed fields in sandesh alarm structure

- Renamed "vars" to "variables" and "json_vars" to "json_variables"

Change-Id: I5f0a3f8dda8787a36ae5100839aa58f08a4a127a
Partial-Bug: #1604539
(cherry picked from commit dba559487f1536e24cbf79cb5e543f88c89c0e93)

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

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

commit 7c97ec8c59b2516b2b1af4c4cb613f76495236c1
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jul 20 18:12:29 2016 -0700

alarm schema change for uve_keys field

- Presently, there is a limitation with the c++ code generation if the top
level field in the schema is a list. To circumvent this limitation, need
to wrap the uve_keys field (presently a list) in the alarm schema.
- Renamed 'vars' to 'variables' in AlarmExpression

Change-Id: I5159405bab2c746b907bbb04b20bc38bede9a04c
Partial-Bug: #1604539

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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