[00:12:00] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717780 (10Luke081515) [00:13:10] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717783 (10JustBerry) >>! In T115196#1717764, @Krenair wrote: > Ah: 12:27, 27 April 2015 Jianhui67 (talk | contribs) renamed user Steinsplitter (98 edits) to S Thanks for explaining, wasn't quite sure... [00:17:23] 10Beta-Cluster-Infrastructure, 6WMF-Legal: Users with stew, os, shell access on betacluster should sign the Access to nonpublic information policy - https://phabricator.wikimedia.org/T115192#1717784 (10Luke081515) [00:19:12] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717787 (10JustBerry) >>! In T115196#1717775, @Luke081515 wrote: > I think @Steinsplitter raised the main concern against local steard rights (?) > There were given to that user by @Reedy >> (del/undel... [00:23:09] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717788 (10JustBerry) Steinsplitter appears to be most recently active at Wikimedia Commons. I have notified them of this ticket here: https://commons.wikimedia.org/w/index.php?title=User_talk%3ASteins... [00:55:08] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717792 (10JustBerry) Also, out of curiosity, why did @Reedy do this: "Reedy edited subscribers, added: Spage; removed: Steinsplitter"? [00:55:15] Hello James_F, welcome back. [00:55:39] PROBLEM - Puppet staleness on integration-dev is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [43200.0] [01:15:01] PROBLEM - Puppet staleness on deployment-conf03 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [43200.0] [03:08:10] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717799 (10zhuyifei1999) >>! In T115196#1717792, @JustBerry wrote: > Also, out of curiosity, why did @Reedy do this: "Reedy edited subscribers, added: Spage; removed: Steinsplitter"? Edit conflict? I'... [03:33:02] PROBLEM - Puppet failure on deployment-tmh01 is CRITICAL: CRITICAL: 20.00% of data above the critical threshold [0.0] [03:33:32] PROBLEM - Puppet failure on integration-slave-trusty-1014 is CRITICAL: CRITICAL: 30.00% of data above the critical threshold [0.0] [04:13:09] RECOVERY - Puppet failure on deployment-tmh01 is OK: OK: Less than 1.00% above the threshold [0.0] [04:13:32] RECOVERY - Puppet failure on integration-slave-trusty-1014 is OK: OK: Less than 1.00% above the threshold [0.0] [06:36:27] RECOVERY - Free space - all mounts on deployment-bastion is OK: OK: All targets OK [07:05:15] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717842 (10Steinsplitter) >>! In T115196#1717777, @Krenair wrote: > @Steinsplitter: I haven't seen any discussion about this on IRC. You need to explain yourself. There was a discussion on chanops int... [07:24:37] PROBLEM - English Wikipedia Mobile Main page on beta-cluster is CRITICAL: HTTP CRITICAL: HTTP/1.1 500 Internal Server Error - 50791 bytes in 0.013 second response time [07:24:55] PROBLEM - App Server Main HTTP Response on deployment-mediawiki01 is CRITICAL: HTTP CRITICAL: HTTP/1.1 500 Internal Server Error - 50511 bytes in 0.007 second response time [07:25:04] PROBLEM - App Server Main HTTP Response on deployment-mediawiki02 is CRITICAL: HTTP CRITICAL: HTTP/1.1 500 Internal Server Error - 50512 bytes in 0.006 second response time [07:29:38] RECOVERY - English Wikipedia Mobile Main page on beta-cluster is OK: HTTP OK: HTTP/1.1 200 OK - 34881 bytes in 0.437 second response time [07:29:54] RECOVERY - App Server Main HTTP Response on deployment-mediawiki01 is OK: HTTP OK: HTTP/1.1 200 OK - 44707 bytes in 0.446 second response time [07:30:06] RECOVERY - App Server Main HTTP Response on deployment-mediawiki02 is OK: HTTP OK: HTTP/1.1 200 OK - 44726 bytes in 0.681 second response time [08:46:50] (03Abandoned) 10Paladox: Remove extension example from mwext-example-testextension [integration/config] - 10https://gerrit.wikimedia.org/r/230125 (owner: 10Paladox) [08:49:46] (03Abandoned) 10Paladox: Update Slate skin test [integration/config] - 10https://gerrit.wikimedia.org/r/229176 (owner: 10Paladox) [09:19:02] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717872 (10Revi) Well, CUOS flag is very sensitive flag, and I wouldn't expect someone to give CUOS to who hasn't signed [[https://meta.wikimedia.org/wiki/Access_to_nonpublic_information_policy/Noticeb... [12:13:54] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717989 (10JohnLewis) Just to clarify - this is beta not production. CheckUser is *not* enabled on beta for more or less this reason and oversight is neither (revdel'd thing also should not be sensitiv... [12:15:07] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717993 (10Reedy) I'm somewhat confused. CheckUser isn't enabled on beta for precisely this reason, so there's no private information being leaked. Although not impossible, it's presumably very unlike... [12:16:22] Reedy: you and me both :) [12:16:46] RESOLVED STOPBEINGSOFSCKINGSTUPID [12:17:29] greg-g: ^ approve that status in phab? ;) [12:17:59] I mean, srsly [12:37:11] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718018 (10JustBerry) >>! In T115196#1717842, @Steinsplitter wrote: >>>! In T115196#1717777, @Krenair wrote: >> @Steinsplitter: I haven't seen any discussion about this on IRC. You need to explain your... [12:48:38] Is that even within the scope of phab? [12:50:26] Reedy: Beta Cluster is subject to the same TOU as production, I believe. Including access to private data. [12:53:03] James_F: Reedy may be AFK. [12:53:18] JustBerry: I have faith he'll see it. [12:54:08] James_F: It looks like he was last active about 30 minutes ago. [12:55:41] James_F: but what private data? :) [12:55:58] JohnFLewis: Oh wait, you're talking about my ticket? [12:55:59] JohnFLewis: Suppression counts. [12:56:36] JohnFLewis: "It's a test wiki" doesn't work when the product TOU is clearly linked on every page. [12:57:00] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718025 (10Luke081515) p:5Triage>3Normal [12:57:04] James_F: there are likely loads of violations then on beta [12:57:14] JohnFLewis: Indeed. [12:57:26] * James_F ponders deleting everyone's access and starting again. [12:57:38] Standard working practice and so on, this was just a normal case [12:58:03] May be best just revoke OS-level access [12:58:04] James_F: Well, to let you know, your colleague from WMF has given steward rights to users to conditions which you seem to object to - just putting that out there. [12:58:42] James_F: Not saying that your statement isn't valid. [12:58:53] JohnFLewis: And +sysop and +bureaucrat given the lack of checks which are the only reason those flags don't come under the policy in production. Oy. [12:59:31] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718030 (10zhuyifei1999) >>! In T115196#1718018, @JustBerry wrote: > @Steinsplitter I understand the privacy of other users at chanops internal is important; however, your point seems quite ambiguous.... [13:00:28] I'm also pondering a clarification from legal too honestly [13:01:36] JohnFLewis: Quite honestly, I talked to Barras, who said Beta-Cluster doesn't follow Meta-Wikimedia at all really. [13:01:52] JohnFLewis: Hence why https://meta.wikimedia.org/wiki/Admin_activity_review isn't followed either. [13:02:04] JustBerry: What some people say, and what is reality, don't match. [13:02:22] Well beta!=production [13:02:41] But... In a legal context, it may be wrong. [13:02:46] James_F: That's true. I had brought up https://meta.wikimedia.org/wiki/Admin_activity_review to greg, who said it's not necessary since it's not production. [13:03:09] JohnFLewis: I feel like there's a broader level of inconsistency that needs to be addressed prior to doing anything else. [13:04:11] JohnFLewis: The Meta-Wikimedia stewards don't really associate themselves with beta at all, though. They'll most likely respond telling you to ask deployment or beta meta-wiki [13:04:15] No. If there's something wrong, you fix the thing that's wrong. Waiting for the perfect solution is how things don't get done. [13:04:57] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718034 (10Luke081515) a:3Luke081515 I can reinstate them, if nobody have sth against that action, so I will wait a while, if somebody got objection, please tell me. [13:06:43] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718037 (10Jdforrester-WMF) >>! In T115196#1718034, @Luke081515 wrote: > I can reinstate them, if nobody have sth against that action, so I will wait a while, if somebody got objection, please tell me.... [13:08:24] James_F: Just letting you know, you realize Luke081515 has steward rights at beta meta-wiki w/o https://meta.wikimedia.org/wiki/Access_to_nonpublic_information_policy/Noticeboard? [13:08:50] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718038 (10Hydriz) I am okay with the proposed solution after following the discussion closely. Ultimately this is just a test wiki and there are no issues with the rights being granted and how JustBer... [13:08:58] Could have even identified under old policy still, who knows [13:10:34] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718039 (10Reedy) >>! In T115196#1718030, @zhuyifei1999 wrote: >>>! In T115196#1718018, @JustBerry wrote: >> @Steinsplitter I understand the privacy of other users at chanops internal is important; how... [13:12:45] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718040 (10JustBerry) >>! In T115196#1718037, @Jdforrester-WMF wrote: >>>! In T115196#1718034, @Luke081515 wrote: >> I can reinstate them, if nobody have sth against that action, so I will wait a while... [13:12:56] James_F: Hm, the privacy policy doesn't apply to labs and has no specific 'this applies to beta cluster' clause... [13:13:04] @ James_F|Away ^^ [13:15:56] Yeah, definitely need legal clarification. Beta technically is under no policy but Labs which has no clause for who can access private information. [13:16:28] JohnFLewis: See PM. [13:31:49] Why is beta different to the other wikis in labs? [13:32:28] Krenair: Are you Alex? [13:34:01] Nvm. @ Krenair Didn't you just close the task as invalid? Or was that a mistake? [13:34:22] Not that task. [13:36:38] I don't know, where the problem is. At beta enwiki suppress is used for test, but there is no private information, that would be suppressed at producation cluster [13:37:25] James_F had mentioned that it's a problem with the way user rights are being handled across beta. [13:39:14] hm, ok. So the only controversal things seems oversight, right? checkuser groups exists just on wikis, where checkusers got more rights than "checkuser" and "checkuser-log" [13:39:38] Luke081515: Yeah, looks like. [13:41:03] the checkuser extension is disabled, so noone can give others these rights without changing the config [13:41:20] Luke081515: Who has the ability to change the config? [13:41:35] Luke081515: I can't seem to remember or recall. [13:41:46] a lot of users, but there is no way to change without a review, so this is not a problem [13:41:52] Shell users. [13:41:59] hm, ok [13:42:02] Luke081515: Okay, seems fair then. [13:42:04] Krenair: ^^ [13:42:24] You can live hack stuff on deployment-bastion without it going through the production config repository, but.. [13:42:43] you need to be a project admin for that? [13:42:53] or just a member? [13:43:14] Not sure. IIRC to mess with those files in deployment-bastion you need to be able to sudo as jenkins-deploy, so probably project admin. [13:48:19] so the other thing is OS, but the oversightet things would not be oversightet at production cluster, so I think this is not a problem [13:49:40] Luke081515: Btw, I don't think Steinsplitter is a WMF volunteer though. [13:50:27] no, he is sysop at commons and mediawiki, and a OTRS member, but not a WMF volunteer [13:50:43] I imagine he would object to being called that. What do you think he is? [13:52:06] Krenair: What do you mean? [13:52:16] Krenair: WMF Staff*? [13:52:49] No he's not staff. [13:54:04] he removed the most of his permissions at beta today [13:55:15] I trust: legoktm!.*@wikipedia/Legoktm (2admin), hashar!.*@mediawiki/hashar (2admin), .*@wikipedia/.* (2trusted), .*@wikimedia/.* (2trusted), .*@mediawiki/.* (2trusted), .* (2trusted), .*@wikimedia/zeljko-filipin-wmf (2admin), [13:55:15] @trusted [13:56:35] that's something other [13:57:00] Luke081515: I know, I was just looking. [13:57:08] ah, ok [14:38:08] 10Beta-Cluster-Infrastructure, 6WMF-Legal: Users with stew, os, shell access on betacluster should sign the Access to nonpublic information policy - https://phabricator.wikimedia.org/T115192#1718056 (10yuvipanda) Just wanted to drop by and say I totally agree with the closing. Beta access shouldn't be restrict... [14:48:13] Exception encountered, of type "InvalidArgumentException" [14:48:13] [8b70b9ce] /wiki/Main_Page?uselang=qqx InvalidArgumentException from line 71 of /srv/mediawiki/php-master/extensions/QuickSurveys/includes/SurveyFactory.php: The "external example survey" external survey link requires https. [14:48:14] :| [14:49:22] 10Beta-Cluster-Infrastructure: Make Privacy Policy/ToS on Beta Cluster link to the labs version (not production version) - https://phabricator.wikimedia.org/T77858#1718068 (10Luke081515) a:3Luke081515 [14:49:51] Luke081515: Can you fix it? [14:50:39] no, not really [14:50:47] Luke081515, I was going to do it... [14:51:07] I got an idea to fix it [14:51:43] yes hello [14:51:49] I was pinged. [14:54:54] Krenair: Changed the text under every page, but I find the link for privacy policy at the footer not yet [14:55:01] Please don't do that. [14:55:09] ok [14:55:23] (03PS3) 10Legoktm: Remove wmf/ branch exclusion for mediawiki-core-phpcs job [integration/config] - 10https://gerrit.wikimedia.org/r/244192 [14:55:26] The proper way to fix this is in WikimediaMessages [14:55:33] (03CR) 10Legoktm: [C: 032] Remove wmf/ branch exclusion for mediawiki-core-phpcs job [integration/config] - 10https://gerrit.wikimedia.org/r/244192 (owner: 10Legoktm) [14:56:01] 10Beta-Cluster-Infrastructure: Make Privacy Policy/ToS on Beta Cluster link to the labs version (not production version) - https://phabricator.wikimedia.org/T77858#1718073 (10Luke081515) a:5Luke081515>3None [14:56:28] (03Merged) 10jenkins-bot: Remove wmf/ branch exclusion for mediawiki-core-phpcs job [integration/config] - 10https://gerrit.wikimedia.org/r/244192 (owner: 10Legoktm) [14:56:54] legoktm: By who? [14:57:08] wm-bot. [14:57:26] !log deploying https://gerrit.wikimedia.org/r/244192 [14:57:33] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL, Master [14:57:41] (03PS3) 10JanZerebecki: tests: restore failures report at end of run [integration/config] - 10https://gerrit.wikimedia.org/r/244507 (https://phabricator.wikimedia.org/T107683) (owner: 10Hashar) [14:57:51] (03CR) 10JanZerebecki: [C: 032] tests: restore failures report at end of run [integration/config] - 10https://gerrit.wikimedia.org/r/244507 (https://phabricator.wikimedia.org/T107683) (owner: 10Hashar) [14:58:55] (03Merged) 10jenkins-bot: tests: restore failures report at end of run [integration/config] - 10https://gerrit.wikimedia.org/r/244507 (https://phabricator.wikimedia.org/T107683) (owner: 10Hashar) [15:01:18] 10Continuous-Integration-Config, 5Patch-For-Review: Reduce nosetests output in integration/config - https://phabricator.wikimedia.org/T107683#1718076 (10JanZerebecki) 5Open>3Resolved [15:16:19] 10Beta-Cluster-Infrastructure, 5Patch-For-Review: Make Privacy Policy/ToS on Beta Cluster link to the labs version (not production version) - https://phabricator.wikimedia.org/T77858#1718085 (10Krenair) a:3Krenair [15:20:49] Luke081515: You claimed it again? :o [15:21:24] Has this task not been resolved since the end of last year? [15:21:48] JustBerry: Which task? [15:21:55] Luke081515: https://phabricator.wikimedia.org/T77858 [15:22:06] Luke081515: Krenair had mentioned he was going to fix it. [15:30:24] JustBerry: I'm not hte current assignee [15:31:07] Luke081515: "Krenair claimed this task."? [15:31:26] yes, and he sets up this patch: https://gerrit.wikimedia.org/r/#/c/244968/ [15:31:32] so he will fix this [15:34:08] another topic: Can someone take a look at this configuration change for itwikiversity? https://gerrit.wikimedia.org/r/#/c/244896/ [15:34:48] It was my first patch at gerrit, so don't wonder that the first things failed, but now it should be correct [15:36:33] Luke081515, patroller has autopatrol and rollback? [15:36:51] see here: https://phabricator.wikimedia.org/T114930 [15:37:24] It was request, so I used this, and the user writes that it is simal to itwikibooks, and there patroller got this rights too [15:37:54] *requested [15:38:10] ok [15:38:20] yep I think that's fine, thanks Luke081515 [15:39:04] 10Continuous-Integration-Config: gate-and-submit may run `composer validate` twice, while `composer test` doesn't - https://phabricator.wikimedia.org/T114451#1718092 (10JanZerebecki) I fixed some time ago that a job running composer test didn't validate the composer.json before. > Developers should be validatin... [15:39:08] Tahnks for review [15:39:15] 10Continuous-Integration-Config: gate-and-submit may run `composer validate` twice - https://phabricator.wikimedia.org/T114451#1718093 (10JanZerebecki) [15:39:28] https://gerrit.wikimedia.org/r/#/q/project:operations/mediawiki-config+label:Code-Review%252B1%252Ckrenair+status:open,n,z [15:39:46] My list has got too big to deal with in one deployment window :/ [15:40:30] Krenair: when I try to open this link I get: https://gerrit.wikimedia.org/r/#/q/project:operations/mediawiki-config+label:Code-Review%252B1%252Ckrenair+status:open,n,z [15:40:33] oh, soory [15:40:43] line 1:53 no viable alternative at character '%' [15:40:44] this [15:42:05] 10Continuous-Integration-Infrastructure, 5Patch-For-Review: use one job for all CI entry points - https://phabricator.wikimedia.org/T111181#1718094 (10JanZerebecki) A problem with my suggestion to use jenkins to trigger other jobs as part a bigger job is that those sub-jobs also need slave slots, but a nodepoo... [16:03:14] PROBLEM - Puppet failure on integration-slave-jessie-1001 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [16:03:14] PROBLEM - Puppet failure on deployment-parsoidcache02 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [16:03:14] PROBLEM - Puppet failure on deployment-mx is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [16:06:50] huh, I can't log into deployment-mx? [16:11:55] ?[1;31mError: Could not retrieve catalog from remote server: Error 400 on SERVER: secret(): invalid secret dkim/wikimedia.org-wiki-mail.key at /etc/puppet/manifests/role/mail.pp:84 on node deployment-mx.deployment-prep.eqiad.wmflabs?[0m [16:11:55] ?[1;31mWarning: Not using cache on failed catalog?[0m [16:11:55] ?[1;31mError: Could not retrieve catalog; skipping run?[0m [16:16:34] Well, you can log in as root if you use salt to add your key... [16:29:57] (03PS1) 10Paladox: [intersection] Change test extension-unittests to extension-unittests-generic [integration/config] - 10https://gerrit.wikimedia.org/r/245131 [16:33:39] Got puppet working, I can login properly too. [16:36:11] It made an interesting change to /etc/resolv.conf, wondering if it was related to the login issues [16:39:28] 10Continuous-Integration-Config: free some repositories from their unintended chain to mediawiki/core - https://phabricator.wikimedia.org/T107529#1718143 (10JanZerebecki) Patch for zuul that would help with that: https://gerrit.wikimedia.org/r/#/c/203290/ [16:41:31] 10Continuous-Integration-Config: free some repositories from their unintended chain to mediawiki/core - https://phabricator.wikimedia.org/T107529#1718144 (10JanZerebecki) Patch to integration/config that instead would duplicate and prefix jobs with the queue name: https://gerrit.wikimedia.org/r/#/c/243164/ [16:45:54] RECOVERY - Puppet failure on deployment-mx is OK: OK: Less than 1.00% above the threshold [0.0] [16:53:07] 10Continuous-Integration-Config, 10MediaWiki-extensions-TimedMediaHandler: Disable php-composer-test for TimedMediaHandler in REL1_25 and older - https://phabricator.wikimedia.org/T115211#1718145 (10Umherirrender) 3NEW [16:54:36] 10Continuous-Integration-Config, 10MediaWiki-extensions-TimedMediaHandler: Disable npm for TimedMediaHandler in REL1_25 and older - https://phabricator.wikimedia.org/T115212#1718152 (10Umherirrender) 3NEW [16:56:57] PROBLEM - Puppet failure on deployment-mx is CRITICAL: CRITICAL: 55.56% of data above the critical threshold [0.0] [17:02:34] 10Continuous-Integration-Infrastructure, 10MediaWiki-extensions-AbuseFilter: Check failure of php-composer-test in AbuseFilter - https://phabricator.wikimedia.org/T115213#1718161 (10Umherirrender) 3NEW [17:11:53] RECOVERY - Puppet failure on deployment-mx is OK: OK: Less than 1.00% above the threshold [0.0] [17:49:38] 10Deployment-Systems, 6Release-Engineering-Team: Move the train deployment from Thursday to Wednesday for some Wikipedia sites - https://phabricator.wikimedia.org/T115002#1718229 (10Amire80) [19:34:37] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718397 (10JustBerry) It looks like once the ToU link has been addressed via the patch @Krenair created (T77858), @Jdforrester-WMF's concern regarding the ToU linkage back to Meta Wikimedia should be r... [19:49:31] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718406 (10Steinsplitter) It is a TESTWIKI. Sorry, but i don't have time for this kind of drama. Your behavior here is explaining itself perfectly. The right was granted and i removed it because users... [20:04:14] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718422 (10JustBerry) To provide clarification to the comment above, leaving it to other users is fine, but stopping by at #wikimedia-releng would have been more effective in addressing the issue. Also... [20:07:41] (03PS1) 10Paladox: [AJAXPoll] Update jenkins tests [integration/config] - 10https://gerrit.wikimedia.org/r/245186 [20:08:56] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718427 (10JustBerry) >>! In T115196#1718406, @Steinsplitter wrote: > There is no regular process or policy for granting and removing right on beta, and this is a big issue. Any Beta-Cluster issues c... [20:11:09] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718432 (10Luke081515) At beta is no "real community", but right changes at beta should discussed here, if needed, like http://deployment.wikimedia.beta.wmflabs.org/wiki/Global_requests says. [20:20:44] (03PS2) 10Paladox: [AJAXPoll] Update jenkins tests [integration/config] - 10https://gerrit.wikimedia.org/r/245186 [20:23:52] (03PS3) 10Paladox: [AJAXPoll] Update jenkins tests [integration/config] - 10https://gerrit.wikimedia.org/r/245186 [20:27:47] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718449 (10Steinsplitter) @JustBerry: Seriously. Please stop stalking me. This is not funny at all :-(. And please stick on the exact trust, i haven't banned you from any channel on irc. Again: Please... [20:33:29] * JohnFLewis mumbles [20:34:46] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718457 (10JustBerry) For readers, I did not interact with the user after my last post. Unless the user's account on IRC had been compromised, the following channel log, which is now being published fo... [20:40:37] JohnFLewis: I had to address the issue, so I replied again. I've stated in the last sentence of my comment that no further extensive discussion should be pursued until Krenair or James_F|Away have discussed and resolved the policy issue, etc. [20:41:45] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718470 (10Steinsplitter) Errr.... I am not a op in this channel. You should stick on the truth. This kind of rhetorical games are not funny :-( :'-( I can't defend myself, because what you say is n... [20:45:36] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718471 (10Luke081515) Please just stay on the topic: We got a rights removal, and some people think, that this wasn't right, so we can reinstate the rights, if there is no protest. Your personal diffe... [20:48:16] 10Continuous-Integration-Infrastructure, 10pywikibot-core: run at least pep8 and pep257 for new changesets submitted to pywikibot/core for any user - https://phabricator.wikimedia.org/T87169#1718474 (10JanZerebecki) >>! In T87169#1711901, @jayvdb wrote: > Note, `detox` on pypi says it is Python 2 only, but [[h... [20:48:43] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718475 (10JustBerry) >>! In T115196#1718471, @Luke081515 wrote: > Please just stay on the topic: We got a rights removal, and some people think, that this wasn't right, so we can reinstate the rights,... [20:48:45] @ Luke081515 ^^ [20:51:15] 10Continuous-Integration-Infrastructure, 10pywikibot-core: run at least pep8 and pep257 for new changesets submitted to pywikibot/core for any user - https://phabricator.wikimedia.org/T87169#1718476 (10jayvdb) >>! In T87169#1718474, @JanZerebecki wrote: >>>! In T87169#1711901, @jayvdb wrote: >> Note, `detox` o... [20:52:39] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718477 (10Luke081515) Yes, I know. That's one of the reasons, why I'm wait, before I will do this action. [20:55:30] (03CR) 10JanZerebecki: [C: 032] [IMPROV] Don't use shell argument [integration/jenkins] - 10https://gerrit.wikimedia.org/r/243140 (owner: 10XZise) [20:56:18] (03Merged) 10jenkins-bot: [IMPROV] Don't use shell argument [integration/jenkins] - 10https://gerrit.wikimedia.org/r/243140 (owner: 10XZise) [21:01:17] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:02:54] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718488 (10MGChecker) I can confirm that Steinsplitter is right, because I was on the channel at this time. steinsplitter has hever banned JustBerry, it was another one, and i wonder why you are postin... [21:09:55] (03CR) 10JanZerebecki: [C: 04-1] "The phpunit job for extensions shouldn't be used for skins, see T68926." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:11:13] (03PS8) 10Paladox: Update BlueSky tests [integration/config] - 10https://gerrit.wikimedia.org/r/226635 [21:11:52] (03CR) 10Paladox: "Sorry noticed that. Ived removed it now." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:14:05] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718502 (10Luke081515) >>! In T115196#1718471, @Luke081515 wrote: > Please just stay on the topic: (...) Your personal differences doesn't matter. If we want to resolve this task, please stay on the to... [21:17:04] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718503 (10JustBerry) >>! In T115196#1718488, @MGChecker wrote: > I can confirm that Steinsplitter is right, because I was on the channel at this time. steinsplitter has hever banned JustBerry, it was... [21:19:44] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718507 (10Milad_A380) I have nothing to do with your discussion, but it seems that JustBerry has posted faked chat logs. I'm an op in the channel #wikipedia-de-jungwikipedianer (Steinsplitter isn't an... [21:23:21] (03CR) 10JanZerebecki: [C: 04-1] "Extension doesn't have composer test entry point." [integration/config] - 10https://gerrit.wikimedia.org/r/243683 (owner: 10Paladox) [21:25:24] (03CR) 10JanZerebecki: [C: 04-1] "Extension doesn't have composer entry point." [integration/config] - 10https://gerrit.wikimedia.org/r/244747 (owner: 10Paladox) [21:27:58] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718517 (10JustBerry) >>! In T115196#1718507, @Milad_A380 wrote: > I have nothing to do with your discussion, but it seems that JustBerry has posted faked chat logs. Do not accuse other people of lyin... [21:28:10] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718518 (10JustBerry) >>! In T115196#1718507, @Milad_A380 wrote: > I have nothing to do with your discussion, but it seems that JustBerry has posted faked chat logs. Do not accuse other people of lyin... [21:28:54] (03CR) 10JanZerebecki: [C: 04-1] "Extension doesn't have composer entry point." [integration/config] - 10https://gerrit.wikimedia.org/r/244945 (owner: 10Paladox) [21:29:30] (03CR) 10Paladox: "Hi I added them here https://gerrit.wikimedia.org/r/#/c/244750/" [integration/config] - 10https://gerrit.wikimedia.org/r/244747 (owner: 10Paladox) [21:30:29] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:30:42] (03CR) 10Paladox: "I added it here https://gerrit.wikimedia.org/r/#/c/243682/" [integration/config] - 10https://gerrit.wikimedia.org/r/243683 (owner: 10Paladox) [21:31:11] (03CR) 10jenkins-bot: [V: 04-1] Update BlueSky tests [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:31:12] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718527 (10Milad_A380) You posted a chat log, you didn't write something like "Steinsplitter kicked me", you wrote something in the style of "[21:13:18] =-= JustBerry was kicked out of #wikipedia-de-j... [21:31:23] (03CR) 10Paladox: "Ive added it here https://gerrit.wikimedia.org/r/#/c/244942/" [integration/config] - 10https://gerrit.wikimedia.org/r/244945 (owner: 10Paladox) [21:32:17] (03CR) 10Paladox: "Needs rechecking please." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:33:59] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718533 (10JustBerry) >>! In T115196#1718397, @JustBerry wrote: > From IRC, Steinsplitter commented "JustBerry: i commented on phabricator and i have nothing to add." Then, they had mentioned the follo... [21:36:17] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718534 (10Milad_A380) The faked log was in your post at 8:34 PM that you deleted later. I never said that this log was faked. [21:40:16] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718535 (10JustBerry) >>! In T115196#1718534, @Milad_A380 wrote: > The faked log was in your post at 8:34 PM that you deleted later. I never said that this log was faked. To clarify, those were not lo... [21:41:25] @ Luke081515 ^^ [21:41:27] Oy. [21:42:32] (03CR) 10JanZerebecki: [C: 04-1] "(Jenkins -1 is because phplint as a template does not exist.) I don't think it makes sense to remove the mw-checks template. Just add jshi" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:43:56] (03CR) 10JanZerebecki: [C: 04-1] "The extension doesn't have an composer test entry point." [integration/config] - 10https://gerrit.wikimedia.org/r/245186 (owner: 10Paladox) [21:45:51] (03PS9) 10Paladox: Update BlueSky tests [integration/config] - 10https://gerrit.wikimedia.org/r/226635 [21:46:39] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718548 (10Milad_A380) I'm ok with ending the discussion about faked logs here, as all that needs to be said has been said. As I do not have anything to do with Beta, I'll also leave the discussion here. [21:47:38] (03CR) 10JanZerebecki: [C: 04-1] "See my previous comment." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:48:48] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1717736 (10JustBerry) In light of resolving the issue, it should be noted that observing the actor of the action being discussed, as well as two supporters, commenting on this issue in a relatively sho... [21:49:13] (03CR) 10Paladox: "Well the new test I add do not depend on the mw-check test." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:49:20] (03PS10) 10Paladox: Update BlueSky tests [integration/config] - 10https://gerrit.wikimedia.org/r/226635 [21:50:05] (03CR) 10Paladox: "Added here https://gerrit.wikimedia.org/r/#/c/245185/" [integration/config] - 10https://gerrit.wikimedia.org/r/245186 (owner: 10Paladox) [21:51:13] (03CR) 10JanZerebecki: [C: 04-2] "This won't fix the job. The job is now failing and not voting, this would make the extension stop passing jenkins, which we don't want." [integration/config] - 10https://gerrit.wikimedia.org/r/245131 (owner: 10Paladox) [21:51:32] (03CR) 10JanZerebecki: [C: 04-1] "Err wrong vote." [integration/config] - 10https://gerrit.wikimedia.org/r/245131 (owner: 10Paladox) [21:53:39] Hello James_F, welcome back. [21:54:35] (03CR) 10Paladox: "Hi it would work this way." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:56:08] (03CR) 10Paladox: "Please recheck." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:59:15] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718560 (10Milad_A380) FYI: Someone (and this someone was not Steinsplitter) notified me in IRC that someone posted a faked log on phabricator, and as I had something to do with these logs, I decided t... [21:59:24] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [21:59:42] (03CR) 10JanZerebecki: "My previous comment still stands." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [22:00:24] (03CR) 10jenkins-bot: [V: 04-1] Update BlueSky tests [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [22:01:07] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718562 (10JustBerry) >>! In T115196#1718560, @Milad_A380 wrote: > FYI: Someone (and this someone was not Steinsplitter) notified me in IRC that someone posted a faked log on phabricator, and as I had... [22:03:05] (03PS11) 10Paladox: Update BlueSky tests [integration/config] - 10https://gerrit.wikimedia.org/r/226635 [22:05:00] (03PS1) 10Paladox: Add new phplint template [integration/config] - 10https://gerrit.wikimedia.org/r/245190 [22:05:07] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718565 (10JustBerry) A note to readers: Although I had mentioned that I would no longer comment on the IRC issue, it seemed necessary to clarify any discrepancies in statements through the subsequent... [22:05:25] (03CR) 10Paladox: "Ok done." [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [22:06:55] (03CR) 10Paladox: "And it shows this error in jenkins" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [22:07:07] (03CR) 10JanZerebecki: "recheck" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [22:13:20] (03CR) 10JanZerebecki: "Sorry, I forgot to check that first. The repo doesn't have jshint configured. So it might fail, not sure right now. Some sleep for me firs" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [22:15:36] (03CR) 10Paladox: "The repo has js a may be broken, maybe add it to experiment:" [integration/config] - 10https://gerrit.wikimedia.org/r/226635 (owner: 10Paladox) [22:17:58] PROBLEM - Puppet failure on deployment-cache-text04 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [22:38:36] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718583 (10scfc) @JustBerry: Just because you remove a comment doesn't mean it becomes unseen by the subscribers of this task who have received mail copies of it and probably now archived them. There... [22:39:28] PROBLEM - Puppet failure on deployment-cache-upload04 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [22:40:11] 10Continuous-Integration-Config, 10Wikidata: [Task] Add Wikidata to Jenkins job mediawiki-extensions-hhvm - https://phabricator.wikimedia.org/T96264#1718586 (10JanZerebecki) [22:46:17] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718605 (10Luke081515) But in fact, there are beta wikis used as test wikis, also by jenkins broswer tests, or test of modules, so I don't see a problem. [22:46:19] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718606 (10JustBerry) >>! In T115196#1718583, @scfc wrote: > @JustBerry: Just because you remove a comment doesn't mean it becomes unseen by the subscribers of this task who have received mail copies o... [23:08:59] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718639 (10Hydriz) Are there evidence that JustBerry is abusing the tools on the Beta cluster? I believe that is more relevant in the context here and would allow everyone to make a more informed choic... [23:11:27] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718640 (10JustBerry) >>! In T115196#1718639, @Hydriz wrote: > Are there evidence that JustBerry is abusing the tools on the Beta cluster? I believe that is more relevant in the context here and would... [23:17:46] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718645 (10Luke081515) I reviewed all actions (about 2000) at enwiki beta of him, and don't see any missuse. [23:31:36] 10Beta-Cluster-Infrastructure: Review rights removal at beta-enwiki - https://phabricator.wikimedia.org/T115196#1718649 (10JustBerry) After a general review of the discussion, the actions appear to be invalid mainly on the following grounds: # The inability and/or unwillingness of the actor to explain their a...