969185 - Update syncstorage to use heka instead of metlog

Status: VERIFIED FIXED

(Cloud Services Graveyard :: Server: Sync, defect, P1)

Description

[Ryan Kelly [:rfkelly]]

Metlog out, heka in.  :RaFromBRC says this should be a pretty straightforward search-and-replace.

Comment 1

[James Bonacci [:jbonacci]]

yea, I think there are some puppet-config issues open about this...

Comment 2

[James Bonacci [:jbonacci]]

See also: 
https://github.com/mozilla-services/puppet-config/issues/120
https://github.com/mozilla-services/puppet-config/issues/211
https://github.com/mozilla-services/puppet-config/issues/251
https://github.com/mozilla-services/puppet-config/issues/278
https://github.com/mozilla-services/puppet-config/issues/287
https://github.com/mozilla-services/puppet-config/pull/305

Comment 3

[Ryan Kelly [:rfkelly]]

Attachment: syncstorage-simplify-metrics.diff

This patch updates server-syncstorage for the new simplified metrics infra proposed in Bug 1012509.  It's almost all just replacing metlog with stdlib logging, plus adding a couple of timers.

The missing half of this is a deploy config change to make it use the JSON logger in production, and then trying it out in stage to see whether heka can slurp the logs in properly.

Comment 4

[Ryan Kelly [:rfkelly]]

https://github.com/mozilla-services/server-syncstorage/commit/f7a2e4fb16986c7697804fe06a676761e08bd70a

The rest of the necessary changes will appear as config/deployment bugs as this moves forward.

Comment 5

[James Bonacci [:jbonacci]]

Verified in code and with unit tests.