severity of alarm entities in templates are case sensitive

Bug #1583157 reported by Elisha Rosensweig
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Vitrage
New
Low
Sharat Sharma

Bug Description

When creating a vitrage template, with an alarm as an entity in the "definitions" section (not in the "scenarios" section), if the alarm severity is listed - it only works if it's in upper case (e.g., "CRITICAL", not "critical"). Should be case-insensitive.

Ifat Afek (ifat-afek)
tags: added: low-hanging-fruit
description: updated
description: updated
description: updated
Ifat Afek (ifat-afek)
tags: removed: evaluator template
tags: added: evaluator template
Changed in vitrage:
assignee: nobody → Sharat Sharma (sharat-sharma)
Revision history for this message
Abhinav Gupta (abhinavgupta) wrote :

Is anyone working on this bug ?
I would like to work on it.

Revision history for this message
Elisha Rosensweig (elisha-rosensweig) wrote :

you can take it - but note that it's not as simple as it looks. I'll give a short overview:

In the templates you can use any key-value pair in the entity properties to condition on. The condition is tested against the values in the entity graph.

Now, some fields (e.g., resource name) should be case-sensitive, while others (severity) should not. Thus, the name of this bug (and marking it as low-hanging) is confusing. It will require some design to support knowing which fields can be case-insensitive.

Revision history for this message
Ifat Afek (ifat-afek) wrote :

Elisha - in that case, I'll remove the low-hanging-fruit tag. I agree that it is confusing.

tags: removed: low-hanging-fruit
Ifat Afek (ifat-afek)
Changed in vitrage:
importance: Undecided → Low
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.