Tracking bug for Configuration of Alarms

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

Bug Description

Tracking bug to implement configuration of alarms.

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

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

commit 547d4e66bc448dc1223c4174c4b5dd8e7eb6f800
Author: Sundaresan Rajangam <email address hidden>
Date: Thu Apr 7 19:43:54 2016 -0700

Schema changes for alarm configuration

Change-Id: Ifb66ab7cab13b52de9f741fe2aac73e702ca3e0b
Partial-Bug: #1561596

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

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

commit 7101b6058755ec499700aaaffc0a33bb85eab452
Author: Sundaresan Rajangam <email address hidden>
Date: Mon Apr 18 12:06:08 2016 -0700

Changes to sandesh Alarm structure to adhere to the Alarm schema

Change-Id: I799c7bc44113bab00a247cb5f3640a29fddbbc9b
Partial-Bug: #1561596

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

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

commit b03f85fc76ef80ac634310713841bcdba47ed9cd
Author: Sundaresan Rajangam <email address hidden>
Date: Fri May 6 11:13:13 2016 -0700

Update alarm plugins to use modified Alarm structure

Change-Id: I00002e455d9a567e0295ed2c1367063ddda311b7
Partial-Bug: #1561596

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

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

commit 3a549e40108b4104f83ae7bb93c483bbfe026215
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jun 1 07:59:17 2016 -0700

Alarm schema update

schema for alarm_rules has been updated to the following format:

"alarm_rules": {
"or_list": [
{
"and_list": [
{AlarmExpression1},
{AlarmExpression2},
...
]
},
{
"and_list": [
{AlarmExpression3},
{AlarmExpression4},
...
]
},
...
]
}

Change-Id: I4235d0b55853932028ac3b38551e1575351238ac
Partial-Bug: #1561596

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

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

commit 4899232d0c685fa0d3a86c7f23e092104b19cffa
Author: Sundaresan Rajangam <email address hidden>
Date: Sun Jun 5 08:23:12 2016 -0700

Alarm configuration update

This patch implements the following:
- Creation of inbuilt-alarms in contrail-api from contrail-alarm-gen
- alarmg-gen that owns partition 0 creates the inbuilt-alarms
- Handling config notification from rabbitmq
- Generic alarm processing code
- Removed __call__ method from the alarm plugins. __call__ method may be
implemented to override the generic alarm processing code

Change-Id: Ibad594b9eb4f89a14551026d142fcc0dcff5a33b
Partial-Bug: #1561596

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

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

commit d45ae347b57b541e361921b35aa033811ca600b7
Author: Sundaresan Rajangam <email address hidden>
Date: Mon Jun 6 08:28:40 2016 -0700

Changes to sandesh Alarm structure to adhere to the Alarm schema

Change-Id: I9f137cffd215296105321c33d1178852eb7020b3
Partial-Bug: #1561596

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

Review in progress for https://review.opencontrail.org/21014
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/20967
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/21020
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/21022
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/20967
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/21014
Committed: http://github.org/Juniper/contrail-controller/commit/56af1b414aa8431b9d71e41ef3897af3e934f861
Submitter: Zuul
Branch: master

commit 56af1b414aa8431b9d71e41ef3897af3e934f861
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jun 8 00:27:48 2016 -0700

Update alarm-gen to use modified sandesh Alarm structure

- Sandesh Alarm structure has been updated to be in sync with the alarm
schema. Update alarm-gen to send alarm rules in the new format.
- Bug fix: q_name passed in VncKombuClient should be unique per client.
Presently, its just module_id+':'+instance_id, which is not unique
across alarm-gen services. Hence, changed q_name to
host_name:module_id:instance_id

Change-Id: I62a5a5445c9a92830264babc47204dfdeaf41eb3
Partial-Bug: #1561596

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

Reviewed: https://review.opencontrail.org/20967
Committed: http://github.org/Juniper/contrail-provisioning/commit/2fc53be8245e25de6676226fc646739ffa0bb67b
Submitter: Zuul
Branch: master

commit 2fc53be8245e25de6676226fc646739ffa0bb67b
Author: Sundaresan Rajangam <email address hidden>
Date: Mon Jun 6 12:27:33 2016 -0700

Provisioning changes for contrail-alarm-gen

Provision rabbitmq server list and keystone parameters for
contrail-alarm-gen

Change-Id: I5de61d2bf9b7610111ffa4ce7e3de6ede0fdf25d
Partial-Bug: #1561596

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

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

commit 6a3c4ae00cd4013ce071864df9fdd9e629b10ba8
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jun 8 11:28:16 2016 -0700

Change format of command-line argment rabbitmq_server_list for alarm-gen

Specify rabbitmq_server_list in format ip1:port,ip2:port to be in sync
with rabbitmq server format in other modules

Change-Id: I33dfafa3437a9578e96093367048fe864c992be5
Partial-Bug: #1561596

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

Review in progress for https://review.opencontrail.org/21020
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/21185
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/21280
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/21020
Committed: http://github.org/Juniper/contrail-fabric-utils/commit/f8d2a7f1afba0826ca25243e237aba24dec04130
Submitter: Zuul
Branch: master

commit f8d2a7f1afba0826ca25243e237aba24dec04130
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jun 8 08:15:32 2016 -0700

Add rabbitmq server list and port to the analytics setup command-line
arguments

Change-Id: I9f3c6c3a8a7759f005612105b9c3733f0153204c
Partial-Bug: #1561596

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

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

commit 129eb9225c6ee5c6b0e8c189fb9e18b954231c36
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jun 22 23:04:14 2016 -0700

contrail-alarm-gen to accept multiple conf files

- contrail-alarm-gen to accept multiple conf files -
contrail-keystone-auth.conf in addition to contrail-alarm-gen.conf
- Change rabbitmq_server_list to accept list of ip addresses instead of
ip1:port,ip2:port format. Presently, puppet provisions only the ip
addresses separated by comma for rabbitmq_server_list for all services.
Added a new command line argument to get rabbitmq_port.

Change-Id: I9aff864c61aed6eed015a39d00458de23fbc9e10
Partial-Bug: #1561596

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

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

commit ed4cd591b3743eb280b4147cecd976f3f3d3de65
Author: Sundaresan Rajangam <email address hidden>
Date: Fri Jun 17 20:54:35 2016 -0700

Handle alarm config UPDATE/DELETE in alarm-gen

- Moved the alarm processing code from handle_uve_notif() to a new method
examine_uve_for_alarms() so that it may be invoked on alarm config changes.
- Upon receiving config notification on rabbitmq bus, AlarmGenConfigHandler
would call alarm_config_change_callback() method with a map containing table
name and the alarm that is created/updated/deleted.
Example => If a new alarm 'alarm1' is created with uve_keys
[analytics-node, config-node], then alarm_config_change_callback()
would be called with {'ObjectCollectorInfo': {'alarm1': 'CREATE'},
'ObjectConfigNode': {'alarm1': 'CREATE'}}
- Introduced alarm_config_change_worker() which runs in a separate gevent for
each partition. This gevent would be instantiated in run_uve_processing.
- Handled enable/disable of alarm config
- Presently, examine_uve_for_alarms() is invoked on alarm config change
which executes all the alarms configured for the table. However, for
alarm config changes, only the alarms in the alarm_config_change_map need to
be executed. This optimization will be done in the next commit.

Change-Id: I8ba594ac36b543a21ff06b6b615c60f87c048631
Partial-Bug: #1561596

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

Reviewed: https://review.opencontrail.org/21444
Committed: http://github.org/Juniper/contrail-provisioning/commit/a90790619665e8452d4f5acf798249f2af4b84a6
Submitter: Zuul
Branch: master

commit a90790619665e8452d4f5acf798249f2af4b84a6
Author: Sundaresan Rajangam <email address hidden>
Date: Fri Jun 24 22:59:49 2016 -0700

Provisioning changes to pass contrail-keystone-auth.conf to alarm-gen

- Removed KEYSTONE section from contrail-alarm-gen.conf; instead
contrail-alarm-gen should take KEYSTONE parameters from
contrail-keystone-auth.conf
- On upgrade to 3.10, contrail-alarm-gen need to be provisioned with
rabbitmq_server_list and rabbitmq_port and contrail-keystone-auth.conf
should be added to contrail-alarm-gen.ini

Change-Id: Ie109bca4dc97384ac5406b7b21c34f6b7f34d638
Partial-Bug: #1561596

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

Reviewed: https://review.opencontrail.org/21185
Committed: http://github.org/Juniper/contrail-puppet/commit/a5c1aaa3a6bde1f2c29dbca3a5ff69e7dadefa25
Submitter: Zuul
Branch: master

commit a5c1aaa3a6bde1f2c29dbca3a5ff69e7dadefa25
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Jun 15 08:45:17 2016 -0700

changes to provision rabbitmq server list and keystone parameters in alarm-gen

Change-Id: Id875a13b8d018e6103503c080d12177866268432
Partial-Bug: #1561596

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

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

commit e38a2b9b2c7daa77bccbb852d5cec9878a30d13f
Author: Sundaresan Rajangam <email address hidden>
Date: Thu Jul 7 12:20:34 2016 -0700

Add "description" field to the UVEAlarmInfo structure

Change-Id: Iac013dd35d477b03a13bddb91c3445363e029b33
Partial-Bug: #1561596

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.