Gehel added a comment.

Some discussion with Stas to clarify the need:

  • We do not declare a service in puppet, so puppet will not restart that service. This is on purpose, as we don't want updater to start before the initial import is done. This might be improved at some point.
  • As in this case, there is nothing that will restart that service except a server reboot, mask/unmask does not bring anything more than enable/disable.
  • The wdqs puppet module also supports upstart, but this is used only in labs, so can be ignored for the purpose of this discussion.

I'd be inclined to grant both mask/unmask and enable/disable as I don't think it adds much possibility for abuse and will let us see what the actual useage is.


TASK DETAIL
https://phabricator.wikimedia.org/T138627

EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Gehel
Cc: akosiaris, gerritbot, Zppix, JanZerebecki, Aklapper, Gehel, Smalyshev, Avner, Lewizho99, Maathavan, debt, D3r1ck01, Jonas, FloNight, Xmlizer, Izno, jkroll, Wikidata-bugs, Jdouglas, aude, Deskana, Manybubbles, mark, Mbch331, Jay8g, Krenair
_______________________________________________
Wikidata-bugs mailing list
Wikidata-bugs@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs

Reply via email to