change in schema for operand2 in alarm config

Bug #1607030 reported by Sundaresan Rajangam
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Invalid
High
Sundaresan Rajangam
R3.1
Fix Committed
High
Sundaresan Rajangam
Trunk
Fix Committed
High
Sundaresan Rajangam

Bug Description

operand2 in AlarmExpression can be either a uve attribute or a json value. Presently, it is defined as a string. Schema of operand2 should be changed to make it explicit that operand2 can take either uve attribute or json value.

Raj Reddy (rajreddy)
tags: added: blocker
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/22786
Submitter: Sundaresan Rajangam (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/22787
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/22787
Committed: http://github.org/Juniper/contrail-controller/commit/9677cd53b9d597e4bafd47a1aa290921ea5d4c5a
Submitter: Zuul
Branch: R3.1

commit 9677cd53b9d597e4bafd47a1aa290921ea5d4c5a
Author: Sundaresan Rajangam <email address hidden>
Date: Tue Aug 2 22:33:44 2016 -0700

Change in schema for operand2 in alarm config

- operand2 in AlarmExpression can be either a uve attribute or a json
value. Presently, it is defined as a string. Schema of operand2 has been
changed to make it explicit that operand2 can take either uve
attribute or json value.

Closes-Bug: #1607030

Presently, severity level for alarm is 0-7. Restricted the
severity level to 0-2 (0- critical, 1 - major, 2 - minor)

Change-Id: I3cfe2d987da023976f674a2c8c4c21e216a7d994
Closes-Bug: #1606676

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

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

commit 9b1adabefb2393ac2c061ef5831bb6c51a7b545c
Author: Sundaresan Rajangam <email address hidden>
Date: Tue Aug 2 22:21:48 2016 -0700

Change in alarm sandesh structure

Change in operand2 type in alarm sandesh structure to be in sync with the
alarm schema changes

Change-Id: I4a9df87aff03b6718e3bbf5c3828a69078842d41
Closes-Bug: #1607030
Closes-Bug: #1605395

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

Review in progress for https://review.opencontrail.org/23012
Submitter: Ankit Jain (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/23012
Committed: http://github.org/Juniper/contrail-test-ci/commit/37327c73c7daacd550ace7607a3070a486902145
Submitter: Zuul
Branch: R3.1

commit 37327c73c7daacd550ace7607a3070a486902145
Author: Ankit Jain <email address hidden>
Date: Sun Aug 7 03:49:22 2016 -0700

Alarm schema related script changes

Related-Bug: #1607030
changing json_vars to json_variables
Soaking enabled

Change-Id: I30ead466b694acaba49bcb7eb5e98194da7cd003

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

Review in progress for https://review.opencontrail.org/23039
Submitter: Ankit Jain (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/23124
Submitter: Sundaresan Rajangam (<email address hidden>)

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

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

commit 7234770011f57b56632a1252c0bf08ae1f8ff51c
Author: Sundaresan Rajangam <email address hidden>
Date: Tue Aug 2 22:33:44 2016 -0700

Change in schema for operand2 in alarm config

- operand2 in AlarmExpression can be either a uve attribute or a json
value. Presently, it is defined as a string. Schema of operand2 has been
changed to make it explicit that operand2 can take either uve
attribute or json value.

Closes-Bug: #1607030

Presently, severity level for alarm is 0-7. Restricted the
severity level to 0-2 (0- critical, 1 - major, 2 - minor)

Change-Id: I3cfe2d987da023976f674a2c8c4c21e216a7d994
Closes-Bug: #1606676
(cherry picked from commit 9677cd53b9d597e4bafd47a1aa290921ea5d4c5a)

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

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

commit d73f5fc7f0dbe10362262083d70ae4a3b978a9c6
Author: Sundaresan Rajangam <email address hidden>
Date: Tue Aug 2 22:21:48 2016 -0700

Change in alarm sandesh structure

Change in operand2 type in alarm sandesh structure to be in sync with the
alarm schema changes

Change-Id: I4a9df87aff03b6718e3bbf5c3828a69078842d41
Closes-Bug: #1607030
Closes-Bug: #1605395
(cherry picked from commit 9b1adabefb2393ac2c061ef5831bb6c51a7b545c)

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

Reviewed: https://review.opencontrail.org/23039
Committed: http://github.org/Juniper/contrail-test-ci/commit/445142adf4682b60c7fb7b2d1981df7bdd4e2c4b
Submitter: Zuul
Branch: master

commit 445142adf4682b60c7fb7b2d1981df7bdd4e2c4b
Author: Ankit Jain <email address hidden>
Date: Sun Aug 7 03:49:22 2016 -0700

Alarm schema related script changes - Cherry-pick

Related-Bug: #1607030
changing json_vars to json_variables
Soaking enabled

Conflicts:
 tcutils/collector/analytics_tests.py

Change-Id: Id5e9cf3858fc131b5f357ed1ca6597cb06d18a07

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

Review in progress for https://review.opencontrail.org/24171
Submitter: Ankit Jain (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/24171
Committed: http://github.org/Juniper/contrail-test-ci/commit/3a0a1c91916bf858e16cd3875e1f96b3fe1a92dd
Submitter: Zuul
Branch: R3.0

commit 3a0a1c91916bf858e16cd3875e1f96b3fe1a92dd
Author: Ankit Jain <email address hidden>
Date: Sun Aug 7 03:49:22 2016 -0700

Alarm schema related script changes <-Cherrypick from R3.1

Related-Bug: #1607030
changing json_vars to json_variables
Soaking enabled

Conflicts:
 tcutils/collector/analytics_tests.py

Change-Id: Ia21ca4cb62cba1250bad9161dc679f8b41438094

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

Review in progress for https://review.opencontrail.org/40095
Submitter: Ankit Jain (<email address hidden>)

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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