Re: [CCAMP] Warren Kumari's No Objection on draft-ietf-ccamp-alarm-module-08: (with COMMENT)

[stefan vallin <[e-mail]>]

Hi Warren!
Thanks for your review
See inline

br Stefan and Martin


> 
> I support Roman's DISCUSS, especially #2.
See our response to Roman
> 
> Also, please see the OpsDir review here -- I believe that you have already
> discussed it with the reviewer (Thank you, Joe Clarke!), but wanted to make
> sure that you also catch the nits.
Yes, these are cateched
> 
> Like Joe, and others, I find the term "Alarm Shelving" to be confusing -  it
> may be a formal term of art, but I've working in / with many NOCs, and have
> never heard the term used; if anything the terminology I'm familiar with "Shut
> up stupid alarm! I'll suppress it for now....". Shelving evokes memories of
> https://pics.me.me/10-ill-just-put-this-over-here-with-the-rest-32772452.png I
> would suggest introducing the term earlier, and more fully describing it as a
> courtesy to other readers.
This has been a topic in the CCAMP group as well, but we stick to it.
It is an established term in alarm standards and alarm systems in the process industry.

See for example the ISA 18.2 standard. Agree that it is not that well-known within IP and Telco
but there is a point in bringing in this concept since it is not just filtering.
As a reviewer said, aha it is "stashing"

> 
> "X.733 and especially 3GPP were not really clear on this point." - this sounds
> (unncessarily) rude - I would suggest perhaps "The X.733 and 3GPP Alarm IRP
> documents are not really clear …"

Good comment. Changed
> 
> Nits:
> "For example, a system with digital inputs that allows users to connects
> detectors" s/connects/connect/ I would also think that a better term is
> “sensors"
Corrected s/connects/connect/
I can see your point on sensors, but will keep detector….

> 
> "A potential drawback of this is that there is a big risk that alarm operators
> will receive alarm types as a surprise, ..." "big" reads oddly - I would
> suggest "significant" instead.

Good point, changed
> 
> "Alarm deletion (using the action "purge-alarms"), can use this state as a
> criterion." - superfluous comma.
> 
Fixed

>