1037494 - Add author field to Histograms.json

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect)

Description

[Roberto Agostino Vitillo (:rvitillo)]

The author field is needed to allow Telemetry's alerting system to send an e-mail to the the interested party when an histogram changes.

Comment 1

[Vladan Djeric (:vladan)]

I'd suggest calling it "alert_emails"  or something like that and having it support multiple comma-separated email addresses

Comment 2

[Mark Reid [:mreid]]

Please ensure that "histogram_tools.py" still parses the Histograms.json file after this change.

Comment 3

[Roberto Agostino Vitillo (:rvitillo)]

Attachment: Bug 1037494 - Add author field to Histograms.json, v1

Comment 4

[Roberto Agostino Vitillo (:rvitillo)]

Attachment: Bug 1037494 - Add author field to Histograms.json, v2

Comment 5

[Roberto Agostino Vitillo (:rvitillo)]

Attachment: Bug 1037494 - Add author field to Histograms.json, v3

Unbitrotted.

Comment 6

[Vladan Djeric (:vladan)]

Comment on attachment 8459590 [details]
Bug 1037494 - Add author field to Histograms.json, v3

I like the alert field's format, but I'm not sure making this change in bulk is the right approach. It's not always correct to tie the alerts to the developer's email address. For example, I added a bunch of LocalStorage histograms a long time ago but that feature has since been completely re-implemented by Necko team. Also if a dev leaves a company, we don't want the alerts getting ignored.

Let's start off with a discussion on the newsgroups about how people want to be notified. Then we can auto-generate emails to the histogram authors asking them to add the right notification addresses to the alert_emails field in Histograms.json, and also point these devs to the ServiceDesk link for creating new email aliases. We should implement and test the notification functionality before sending out this email request.

Comment 7

[Roberto Agostino Vitillo (:rvitillo)]

(In reply to Vladan Djeric (:vladan) from comment #6)
> Comment on attachment 8459590 [details]
> Bug 1037494 - Add author field to Histograms.json, v3
> 
> I like the alert field's format, but I'm not sure making this change in bulk
> is the right approach. It's not always correct to tie the alerts to the
> developer's email address. For example, I added a bunch of LocalStorage
> histograms a long time ago but that feature has since been completely
> re-implemented by Necko team. 
Right, my thought was that even if the developer that gets the alert is not right one, he probably knows who to get in touch with to correctly re-assign it.

> Also if a dev leaves a company, we don't want the alerts getting ignored.
Yes, in fact there are some developers that receive all alerts (like myself). 

> Let's start off with a discussion on the newsgroups about how people want to
> be notified. Then we can auto-generate emails to the histogram authors
> asking them to add the right notification addresses to the alert_emails
> field in Histograms.json, and also point these devs to the ServiceDesk link
> for creating new email aliases. We should implement and test the
> notification functionality before sending out this email request.

OK

Comment 8

[Roberto Agostino Vitillo (:rvitillo)]

Attachment: Bug 1037494 - Add author field to Histograms.json, v4

Comment 9

[Roberto Agostino Vitillo (:rvitillo)]

Attachment: Bug 1037494 - Add author field to Histograms.json, v4

Comment 10

[Vladan Djeric (:vladan)]

I'm assuming you've talked to Mark that this won't break anything server-side such as server-side parsing of Histograms.json

Comment 11

[Mark Reid [:mreid]]

(In reply to Vladan Djeric (:vladan) from comment #10)
> I'm assuming you've talked to Mark that this won't break anything
> server-side such as server-side parsing of Histograms.json

Yep, we just need to make sure histogram_tools.py is able to handle Histograms.json versions with and without the new field. Then when this lands I'll update the server(s) with the latest histogram_tools.py from mercurial.

Comment 12

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b39daaf8394b

Comment 13

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/b39daaf8394b