Page MenuHomePhabricator

phabricator metrics script should use slave, not master
Closed, ResolvedPublic

Description

11:02 < jynus> it would be nice if queries and stats were run on the m3-slave
11:02 < jynus> not on the master
11:02 < mutante> jynus: ok, the existing script uses m3-master
11:03 < mutante> so i used the credentials from that
11:03 < jynus> you know to not make T109279 worse :-P
11:03 < jynus> let's file a bug, then

Details

Event Timeline

Dzahn raised the priority of this task from to Needs Triage.
Dzahn updated the task description. (Show Details)
Dzahn added projects: Phabricator, acl*sre-team.
Dzahn subscribed.

@Aklapper we should adjust your script to use a slave instead of the master, per above

Reasons: 1) It will not affect production traffic, and 2) It will run faster!

The phabricator puppet role class sets $mysql_host = 'm3-master.eqiad.wmnet' to be used for phab in general and then these metrics scripts just do sql_host='<%= @mysql_host %>'. So it's not hardcoded in the scripts themselves.

Dzahn set Security to None.

Low until it affects production.

Change 236944 had a related patch set uploaded (by Dzahn):
phab: use mysql slave not master for scripts

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

Change 236944 merged by Dzahn:
phab: use mysql slave not master for scripts

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

Dzahn claimed this task.

applied on iridium now. scripts use slave. i ran one of them and works normal.

resolving

Thank you, @Dzahn, didn't had the time! Thanks a lot!