Page MenuHomePhabricator

Add growthexperiments to allowed allowed_logtypes
Closed, ResolvedPublic

Description

GrowthExperiments extension introduced the growthexperiments log type, which can be seen at https://sr.wikipedia.org/wiki/%D0%9F%D0%BE%D1%81%D0%B5%D0%B1%D0%BD%D0%BE:%D0%94%D0%BD%D0%B5%D0%B2%D0%BD%D0%B8%D0%BA/growthexperiments for instance. This log is public for everyone to use, and there is no reason for it to not be in cloud replicas as well.

Patch incoming.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 636436 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[operations/puppet@production] Add growthexperiments to allowed logtypes

https://gerrit.wikimedia.org/r/636436

bd808 moved this task from Backlog to Wiki replicas on the Data-Services board.

Adding Security-Team for sign off.

This appears to be low risk as the log consists of publicly-available user data and doesn't appear to disclose any sensitive relationships. @JFishback_WMF might want to weigh in as well (here's an english-friendly version of the log) but IMO this should be fine.

JFishback_WMF moved this task from Incoming to Backlog on the Privacy Engineering board.

Will the patch be merged soon? :)

Will the patch be merged soon? :)

I just chatted with @JFishback_WMF - he was also going to have a quick look, though I did +1 the patch as it looks fine to me, as I mentioned in my previous comment. So we'd just need someone from Cloud-Services to +2 and handle any relevant deploys.

Change 636436 merged by Bstorm:

[operations/puppet@production] Add growthexperiments to allowed logtypes

https://gerrit.wikimedia.org/r/636436

Bstorm claimed this task.
Bstorm subscribed.

This is now fully deployed across the replica servers.

This is now fully deployed across the replica servers.

Thanks! Works correctly on Toolforge. :)