1251335 - replace cef logging with mozlog

Status: RESOLVED FIXED

(Release Engineering Graveyard :: Applications: Balrog (backend), defect)

Description

[[e-mail] (:bhearsum)]

In speaking with Julien about Balrog migration to CloudOps today, CEF logging came up and he told me that we can kill it and log mozlog formatted lines instead. We should land this before we switch over to CloudOps.

Comment 1

[[e-mail] (:bhearsum)]

Attachment: Implement Mozlog and remove CEF logging

Details in the PR.

Comment 2

[[e-mail] (:bhearsum)]

Comment on attachment 8731723 [details] [review]
Implement Mozlog and remove CEF logging

11:33 <~bhearsum> ulfr: is there anything you want me to address on https://github.com/mozilla/balrog/pull/59 before merging it?
11:39 < ulfr> bhearsum: nope, it's good to go from my end
11:40 <~bhearsum> ulfr: \o/ thanks

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/8ba338d464286f8c5228834f36e79d38828e6605
Merge pull request #59 from bhearsum/mozlog

bug 1251335: Implement Mozlog and remove CEF logging. r=nthomas,ulfr

Comment 4

[[e-mail] (:bhearsum)]

Attachment: add test for logging configuration

Comment 5

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 8733406 [details] [review]
add test for logging configuration

r+ with the comment on the PR fixed/defeated in a logic cage match.

Comment 6

[[e-mail] (:bhearsum)]

This landed in production today.