[05:24:03] 10GitLab (Account Approval), 10Release-Engineering-Team: Requesting GitLab account activation for USER[S] - https://phabricator.wikimedia.org/T337534 (10Nik.xyz.in) [07:41:17] hashar: I need MW to make an API call to itself while running the api-testing suite. I am trying to have it auto-configure based on $wgServer and $wgRestPath, but I'M getting a 404. The URL it is trying is: http://127.0.0.1:9413/w/rest.php/127.0.0.1/v3/transform/wikitext/to/html/Talk%3AFlow_QA [07:41:53] hashar: what would be the correct URL of rest.php in CI while running api-testing? [07:46:12] hmm I have absolutely no idea. I know install.php is called with an explicit scriptpath [07:46:23] and I would guess the $wgRestPath is infered from $wgScriptPath somehow [07:46:28] or that never got implemented [07:47:05] the builds should have attached the LocalSettings.php generated by the installer as well as the custom LocalSettings.php crafted by Quibble which is included in the former [08:14:06] duesen: `$wgServer = "http://127.0.0.1:9413";` and `$wgScriptPath = "";` [08:14:55] Quibble also set some environment variables `MW_SERVER` (most probably set from `$wgServer` `http://127.0.0.1.9413` and `MW_SCRIPT_PATH='/'`) [08:15:18] hashar: doesn't seem to work either... failed: 404 Base URL: http://127.0.0.1:9413/rest.php/ [08:16:27] It's possibel that there is another reason for the 404, but the base URL being wrong seems the most likely. [08:16:47] Given that it's a 404 and not a connection failure, the host and port are probably wright. [08:20:09] Actually... [08:20:40] hashar: where do I find the api-testing.config.json used in CI? It should have the correct base URL. [08:25:49] in mediawiki/core there is tests/api-testing/.api-testing-quibble.json [08:26:12] which has a different port [08:29:05] the config is generated dynamically by quibble inc commands.py, based on the url parameter. In the log I see /usr/local/bin/quibble --web-backend=external --web-url=http://127.0.0.1:9413 --reporting-url=https://earlywarningbot.toolforge.org --packages-source vendor --db mysql --db-dir /workspace/db --git-parallel=8 --reporting-url=https://earlywarningbot.toolforge.org --run api-testing [08:29:13] So that *should* work. [08:29:24] strange. [08:29:41] * hashar tries locally with `quibble -c bash` [08:30:27] Well, --web-url=http://127.0.0.1:9413 indicates that the script path is indeed "". [08:31:38] Oh. Wait. I think I know what's happening. Silly me. [08:32:33] :D [08:32:37] Yea. My assumption that the parsoid API is bundled with core is wrong. The code is bundled, but the endpoint needs to be enabled. [08:32:46] ohhh [08:33:04] So... is there a way to have wfLoadExtension( 'Parsoid' ); in LocalSettings for the api-testing run? [08:33:36] https://phabricator.wikimedia.org/T227352 [08:33:46] quibble has some hack in the localsettings [08:35:03] This is turning nasty. I may come up with a hack to load extra settings when running in CI. THis is temporary, these are baseline tests for a refactoring. After the refactoring, the API call will be gone. [08:35:21] so that if mediawiki/services/parsoid is cloned, some code load it as an extension which cause some glue code in parsoid to load and expose it [08:35:22] From inside PHP code, can I know whether it's running in CI? [08:35:27] cscott would know :] [08:35:59] Quibble sets the environment variable MW_QUIBBLE_CI [08:36:17] and the generated localsettings.php has `define( 'MW_QUIBBLE_CI', true );` [08:36:47] ok cool. I may be able to make use of that. I think GrowthExperiments does. It's terrible, but as a temporary hack it would be ok [08:37:42] It's all made complicated by the fact that Parsoid isn't a "real" extension... [08:37:47] *sigh* [08:39:30] hm... generally, how are extension dependencies handled? If extension A needs extension B, how does quibble know? Will it automaticalyl clone and enable the extension? [08:41:17] nop [08:41:27] (oh good, local_settings.php.tpl has hacks for both Flow and Parsoid, this is getting better and better) [08:41:45] One passe the repos project to clone as arguments to the quibble command line [08:41:54] which then git clone each of them under extensions skins and service [08:42:08] then Quibble injects them in LocalSettings.php with wfLoadExtension() [08:42:16] can I do that in the zuul config? [08:42:27] and when mediawiki runs the ExtensionRegistry will validate the dependency tree and bails out if a dependency is missing [08:42:34] so quibble must have all dependencies injected [08:43:04] * hashar on CI the tree is maintained centrlly in integration/config zuul/parameters_functions.py in a form of a dict `A: [dependencies]` [08:53:43] hashar: that might be an option. but I'm seeing this hack in quibble: [08:53:51] $parsoidDir = $IP . '/services/parsoid'; [08:53:52] if ( is_dir( $parsoidDir ) ) { ... } [08:54:08] This is supposed to automatically enable to parsoid extension, as far as I can tell. [08:54:17] But that path seems wrong. That's not where the bundled parsoid lives. [08:54:17] oh I did implement dependency resolution in Quibble though which crawls through the extension.json/skin.json recursively cloning all of them: `quibble --resolve-requires` [08:54:40] but that param is experimental/poc and is not meant to be used, there is a long tail of things that should be done for it [08:54:41] oh nice, yet another option :) [08:55:02] Anyway, $IP . '/services/parsoid'; is strange [08:55:14] there is no services/ directory in mediawiki [08:55:17] Is there in CI? [08:55:59] yeah when mediawiki/services/parsoid is added as a repo to clone, Quibble maps it to be cloned at `$IP/services/parsoid` [08:56:09] The correct path afaik should be '/vendor/wikimedia/parsoid/extension.json', but that should only be used if extensions/Parsoid doesn't exist [08:56:58] So... [08:57:05] hm... [08:57:28] if that repo is cloned, it's enabled as an extension. but if parsoid is pulled in with composer, it isn't. [08:57:32] I... guess that kind of makes sense. [08:57:58] So now I need to make Flow depend on the Parsoid extension? And tell Zuul about it? [08:58:26] And all this to actually *remove* the dependency on the parsoid extension.... so i'll have to undo all that again... [08:58:28] *sigh* [08:58:34] I want a quick hack :) [08:58:46] * duesen goes out to walk the dog [09:00:53] ah here is the hack in LocalSettings.php https://gerrit.wikimedia.org/r/plugins/gitiles/integration/quibble/+/refs/heads/master/quibble/mediawiki/local_settings.php.tpl#73 [09:01:08] and there is something else in parsoid itself to enable the end points and make the lib behave as an extension [09:01:15] should ask to #content-transformers :) [09:11:05] 10GitLab (Account Approval), 10Release-Engineering-Team: Requesting GitLab account activation for Nik.xyz.in - https://phabricator.wikimedia.org/T337534 (10Peachey88) [09:18:32] 10Release-Engineering-Team, 10API Platform, 10AQS2.0, 10Platform Engineering, and 5 others: Define a procedure/pattern to populate test environments - https://phabricator.wikimedia.org/T334851 (10Sfaci) Hi @Htriedman, first of all, thank you for your help to run your experiment! Currently I'm able to run... [09:37:24] hashar: thanks, I'll poke around. [09:40:49] there is something with https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/services/parsoid/+/refs/heads/master/extension/src/RegistrationHookHandler.php [09:41:00] which is in parsoid extension.json "callback": "\\MWParsoid\\RegistrationHookHandler::onRegistration", [09:41:54] and I think https://www.mediawiki.org/wiki/Parsoid#Linking_a_developer_checkout_of_Parsoid [09:41:57] is the relevant doc for the hack [09:43:53] 10Release-Engineering-Team (They Live 🕶️🧟), 10Scap: Handle spurious failures in Scap's self-installer - https://phabricator.wikimedia.org/T337394 (10jnuche) 05In progress→03Resolved Closing tentatively. Will reopen if problem shows up again in the future. [09:50:42] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (They Live 🕶️🧟): Replace deprecated `CI_JOB_JWT` CI variable in Kokkuri - https://phabricator.wikimedia.org/T337474 (10jnuche) 05Open→03In progress [10:14:06] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (They Live 🕶️🧟): Replace deprecated `CI_JOB_JWT` CI variable in Kokkuri - https://phabricator.wikimedia.org/T337474 (10jnuche) [10:14:53] 10GitLab (CI & Job Runners), 10Release-Engineering-Team (They Live 🕶️🧟): Replace deprecated `CI_JOB_JWT` CI variable in Kokkuri - https://phabricator.wikimedia.org/T337474 (10jnuche) @brennen I updated the description. It seems this won't prevent us from upgrading to GitLab 16 after all. [10:18:27] 10Beta-Cluster-Infrastructure: deployment-prep Puppet CA about to expire on 2023-05-25 - https://phabricator.wikimedia.org/T335689 (10jbond) @Jgiannelos. im not sure if thats related however the certificate has no expired ad of May 25 20:43:07 2023 GMT [10:21:41] 10Beta-Cluster-Infrastructure: deployment-prep Puppet CA about to expire on 2023-05-25 - https://phabricator.wikimedia.org/T335689 (10jbond) T236277 is the task where we handled this for production, specifically https://flyingcircus.io/blog/how-to-renew-puppet-ca-and-server-certificates-in-place/ [10:22:03] hi all, wonderign if anyone is working on T335689, if not i can take a lok [10:22:04] T335689: deployment-prep Puppet CA about to expire on 2023-05-25 - https://phabricator.wikimedia.org/T335689 [10:36:30] 10Beta-Cluster-Infrastructure: deployment-prep Puppet CA about to expire on 2023-05-25 - https://phabricator.wikimedia.org/T335689 (10jbond) FYI im trying to fix this, i have taken a backup of the current ssl dir in /root/ssl.backup.tar.gz [10:58:22] 10Beta-Cluster-Infrastructure: deployment-prep Puppet CA about to expire on 2023-05-25 - https://phabricator.wikimedia.org/T335689 (10jbond) i have now extended the certificate on deployment-puppetmaster04 and deployed the fix to that systaem as well as deployment-puppetdb03 and deployment-deploy03. To fix othe... [11:19:09] 10Beta-Cluster-Infrastructure: deployment-prep Puppet CA about to expire on 2023-05-25 - https://phabricator.wikimedia.org/T335689 (10jbond) 05Open→03Resolved a:03jbond I have extended the certificate and fixed all instances i could in deployment-prep. there was one node that failed because of ssh issues... [11:19:53] 10Project-Admins: Add 'developer-experience' Phabricator tag - https://phabricator.wikimedia.org/T334126 (10Aklapper) 05Stalled→03Declined @Volker_E: No reply; declining for the time being. [11:31:44] jbond: thanks is for the Puppet CA / cert fixes!!! :] [11:31:46] those are painful [11:34:20] (03CR) 10Hashar: [C: 03+2] wm-patch-demo: link to other patches [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/922882 (https://phabricator.wikimedia.org/T332474) (owner: 10Hashar) [11:34:26] (03PS1) 10Daniel Kinzler: Enable the parsoid extension when testing Flow [integration/config] - 10https://gerrit.wikimedia.org/r/923562 [11:34:28] (03CR) 10Hashar: [C: 03+2] wm-patch-demo: use WARNING to prevent chipset collapsing [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923418 (https://phabricator.wikimedia.org/T332474) (owner: 10Hashar) [11:34:38] hashar: no problem [11:34:38] hashar: --^^ [11:34:52] (03Merged) 10jenkins-bot: wm-patch-demo: link to other patches [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/922882 (https://phabricator.wikimedia.org/T332474) (owner: 10Hashar) [11:34:56] (03Merged) 10jenkins-bot: wm-patch-demo: use WARNING to prevent chipset collapsing [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923418 (https://phabricator.wikimedia.org/T332474) (owner: 10Hashar) [11:36:36] hashar: It kind of looks like asking for parsoid to be installed as a dependency should work, but nothing else does it... I see a hack for parsoid to be befound even though the repo is under mediawiki/services. And I see the hack that enabled the extension if parsoid was cloned into the services directory. [11:36:47] But I'm not sure how it would actually get cloned into the services directory... [11:37:13] duesen: as I said earlier, we should loop cscott/ Content-transformers cause I can't remember all the details :D [11:37:53] cscott removed the parsoid dependency with https://gerrit.wikimedia.org/r/c/integration/config/+/803990 [11:38:25] I'd ask kostajh but he is in vacation this week :/ [11:38:50] Hm, right [11:38:56] I will try to ping C.Scott [11:45:37] hashar: i pinged cscott, but can we somehow try if my patch would Just Work (tm)? [11:47:47] (03CR) 10Hashar: [C: 04-1] "Daniel and I talked about it today." [integration/config] - 10https://gerrit.wikimedia.org/r/923562 (owner: 10Daniel Kinzler) [11:48:04] duesen: no no it is broken :] [11:48:22] I commented on the task. Adding 'parsoid' as a dependency was the hack we used last year [11:48:27] and it is no more how it should be done nowadays [11:48:42] and I don't even think that is going to work due to some clash with how mediawiki behave nowadays [11:49:31] cause the wiring is now in mediawiki/core itself https://gerrit.wikimedia.org/r/c/mediawiki/core/+/764922 as I get it [11:49:40] hashar: I know how to enable parsoid as an extension, I use that seup locally all the time. I need a way to tell quibble to do it. [11:50:25] The canonical way to do it in LocalSettings, using the code in the library from vendor, is wfLoadExtension( 'Parsoid', MW_INSTALL_PATH . '/vendor/wikimedia/parsoid/extension.json' ); [11:50:44] I can put that into quiubble, but should it be done unconditionally, so the extension is always enabled? [11:55:07] I don't know :/ [11:56:48] hashar: so... in what way is my patch broken? [11:57:48] I think it should still work... can we try it somehow? [12:15:49] (03PS1) 10Hashar: wm-patch-demo: do not return runs when there are no wikis [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923589 (https://phabricator.wikimedia.org/T332474) [12:20:03] (03CR) 10Hashar: [C: 03+2] wm-patch-demo: do not return runs when there are no wikis [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923589 (https://phabricator.wikimedia.org/T332474) (owner: 10Hashar) [12:20:43] (03Merged) 10jenkins-bot: wm-patch-demo: do not return runs when there are no wikis [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923589 (https://phabricator.wikimedia.org/T332474) (owner: 10Hashar) [12:24:18] 10Gerrit, 10Patch-For-Review: [wm-checks-api] Create a new gerrit bot for Patch Demo - https://phabricator.wikimedia.org/T332474 (10hashar) 05Open→03Resolved I have merged the two patches I have made earlier and the results are now reported as WARNING which makes them stand out a bit. I added a last minu... [12:42:47] (03CR) 10Daniel Kinzler: Enable the parsoid extension when testing Flow (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/923562 (owner: 10Daniel Kinzler) [13:12:31] 10GitLab (Project Migration), 10Release-Engineering-Team (They Live 🕶️🧟), 10serviceops-collab: Provide mechanism to publish to doc.wikimedia.org from GitLab CI - https://phabricator.wikimedia.org/T336168 (10hashar) 05Resolved→03Open Coming from https://gerrit.wikimedia.org/r/c/integration/docroot/+/92254... [13:13:03] 10GitLab (Project Migration), 10Release-Engineering-Team (Priority Backlog 📥), 10API Platform, 10Anti-Harassment, and 19 others: Migrate PipelineLib repos to GitLab - https://phabricator.wikimedia.org/T332953 (10hashar) [13:13:14] (03CR) 10Hashar: "Note the old doc is still at https://doc.wikimedia.org/scap/ which does not redirect to the new path. I have left some details on the task" [integration/docroot] - 10https://gerrit.wikimedia.org/r/922540 (https://phabricator.wikimedia.org/T336168) (owner: 10Ahmon Dancy) [13:15:02] (03CR) 10Hashar: [C: 03+2] Move 'StandardDialog' from 'phan_dependencies' to 'dependencies' [integration/config] - 10https://gerrit.wikimedia.org/r/923343 (owner: 10Hslater) [13:15:30] 10GitLab (Upstream pit of despair 🕳️), 10Release-Engineering-Team (Radar), 10User-brennen: GitLab MR reviews throw 500 errors when commenting on some specific lines - https://phabricator.wikimedia.org/T337514 (10dcaro) Thanks! [13:16:24] (03Merged) 10jenkins-bot: Move 'StandardDialog' from 'phan_dependencies' to 'dependencies' [integration/config] - 10https://gerrit.wikimedia.org/r/923343 (owner: 10Hslater) [13:16:59] (03CR) 10Hashar: [C: 03+2] "Deployed" [integration/config] - 10https://gerrit.wikimedia.org/r/923343 (owner: 10Hslater) [13:27:25] 10GitLab (Project Migration), 10Release-Engineering-Team (They Live 🕶️🧟), 10User-brennen: Define a permissions model for the /repos/mediawiki/ namespace on GitLab - https://phabricator.wikimedia.org/T336807 (10Mvolz) One thing I wanted to flag up is that main branches can be protected so that only developer... [13:29:18] 10Phabricator (Upstream), 10Upstream: "Unhandled Exception: Call to a member function getAppliedTransactionPHIDs() on bool" when viewing old Herald Transcript - https://phabricator.wikimedia.org/T294860 (10Aklapper) p:05Triage→03Low a:03Aklapper [13:36:08] 10Gerrit, 10Release-Engineering-Team: Investigate Gerrit git repositories having invalid references - https://phabricator.wikimedia.org/T337508 (10hashar) Ah thanks for actually running the code! So I was off by one slash and `refs/master` is accepted. So I don't know from where the funny name error comes :/... [13:58:32] 10Gerrit, 10Release-Engineering-Team (Development services), 10Release-Engineering-Team-TODO, 10Upstream: Can't delete weird ref using git in Gerrit - https://phabricator.wikimedia.org/T275946 (10hashar) To the best I can tell, Gerrit JGit accepts receiving `refs/master` but the replication is done with cg... [14:06:51] 10Gerrit, 10Release-Engineering-Team: Investigate Gerrit git repositories having invalid references - https://phabricator.wikimedia.org/T337508 (10hashar) When the replication plugin uses SSH for transport it establishes a ssh connection (on port 22, not the Gerrit daemon ssh server on 29418) and then use `git... [14:09:58] 10Gerrit: org.apache.http.client.protocol.ResponseProcessCookies : Invalid cookie header for WMF-Last-Access - https://phabricator.wikimedia.org/T273605 (10hashar) I had an untested patch for jgit which would maybe make it able to read those cookies: **org.eclipse.jgit.http.apache/src/org/eclipse/jgit/transport... [14:17:06] 10Project-Admins: Create a #wmcs-superset tag - https://phabricator.wikimedia.org/T336940 (10rook) [14:23:19] 10Gerrit, 10Release-Engineering-Team: Investigate Gerrit git repositories having invalid references - https://phabricator.wikimedia.org/T337508 (10hashar) From https://gerrit.googlesource.com/jgit in `org.eclipse.jgit.test/tst/org/eclipse/jgit/lib/ValidRefNameTest.java` I have added a test: ` @Test pub... [14:24:54] 10Release-Engineering-Team, 10SRE, 10Security-Team, 10serviceops-collab, 10Security: Add github.com/wikimedia as an SCM for Semgrep Cloud - https://phabricator.wikimedia.org/T337561 (10jbond) @sbassett im not sure who manages the github account. for noa ill tag #serviceops-collab who manage gitlab and #... [14:26:36] 10Release-Engineering-Team, 10SRE, 10Security-Team, 10serviceops-collab, 10Security: Add github.com/wikimedia as an SCM for Semgrep Cloud - https://phabricator.wikimedia.org/T337561 (10sbassett) Hey @jbond - I had been talking to @Clement_Goubert on Slack about this, so that's why I tagged them/SRE. In... [14:27:38] thcipriani: thank you for the java code testing `refs/master`. It seems to be a dispredancy between cgit refusing those refs while jgit happilly accepts them :] [14:27:44] I am off for the week-end & [14:36:30] 10Project-Admins: Create a #wmcs-superset tag - https://phabricator.wikimedia.org/T336940 (10nskaggs) @Aklapper https://wikitech.wikimedia.org/wiki/Superset has a disambiguation on it, if that helps. Simply put Data Engineering runs a restricted superset instance; superset.wikimedia.org, while WMCS has launched... [14:38:24] 10Project-Admins: Create a #wmcs-superset tag - https://phabricator.wikimedia.org/T336940 (10nskaggs) It's possible you may wish to force both tags to have prefixes or suffices on them. And updating the tags to have public and NDA/restricted prefixes/suffices might make more sense in the longterm rather than att... [14:52:18] (03CR) 10Ahmon Dancy: Update scap URLs (031 comment) [integration/docroot] - 10https://gerrit.wikimedia.org/r/922540 (https://phabricator.wikimedia.org/T336168) (owner: 10Ahmon Dancy) [14:57:13] 10Project-Admins: Create a project for the WMF Movement Insights team - https://phabricator.wikimedia.org/T337518 (10Aklapper) 05Open→03Resolved a:03Aklapper Requested public project #WMF-Movement-Insights has been created: https://phabricator.wikimedia.org/project/view/6566/ Please add a link to a wiki p... [15:00:05] 10GitLab (Project Migration), 10Release-Engineering-Team (They Live 🕶️🧟), 10User-brennen: Define a permissions model for the /repos/mediawiki/ namespace on GitLab - https://phabricator.wikimedia.org/T336807 (10dancy) >>! In T336807#8882990, @Mvolz wrote: > One thing I wanted to flag up is that main branches... [15:02:18] (03PS1) 10Ilias Sarantopoulos: ml-services: rename ores-migration pipelines [integration/config] - 10https://gerrit.wikimedia.org/r/923647 [15:04:52] 10Release-Engineering-Team, 10SRE, 10Security-Team, 10serviceops-collab, 10Security: Add github.com/wikimedia as an SCM for Semgrep Cloud - https://phabricator.wikimedia.org/T337561 (10Dzahn) We don't really have a relation to the github Wikimedia organization. [15:10:35] (03CR) 10Hashar: [C: 03+2] "Neat! I have deployed the Jenkins job and will reload Zuul as soon as this is merged. You can then CR+2 the change in the source repositor" [integration/config] - 10https://gerrit.wikimedia.org/r/923647 (owner: 10Ilias Sarantopoulos) [15:11:39] (03CR) 10Ilias Sarantopoulos: ml-services: rename ores-migration pipelines (031 comment) [integration/config] - 10https://gerrit.wikimedia.org/r/923647 (owner: 10Ilias Sarantopoulos) [15:11:49] (03Merged) 10jenkins-bot: ml-services: rename ores-migration pipelines [integration/config] - 10https://gerrit.wikimedia.org/r/923647 (owner: 10Ilias Sarantopoulos) [15:12:46] !log Reloaded Zuul for https://gerrit.wikimedia.org/r/c/integration/config/+/923647 [15:12:47] Logged the message at https://wikitech.wikimedia.org/wiki/Release_Engineering/SAL [15:15:18] 10Phabricator (Upstream), 10Upstream: "Unhandled Exception: Call to a member function getAppliedTransactionPHIDs() on bool" when viewing old Herald Transcript - https://phabricator.wikimedia.org/T294860 (10Aklapper) [15:15:53] 10Phabricator, 10Release-Engineering-Team (They Live 🕶️🧟), 10Patch-For-Review: Reports: Age histogram shows nonsensical additional bucket after "Older" for projects created in last days - https://phabricator.wikimedia.org/T336152 (10Aklapper) [15:15:59] 10Phabricator, 10Release-Engineering-Team (They Live 🕶️🧟), 10Patch-For-Review: Custom default project menu items have wrong description, typos, use outdated code repo URL - https://phabricator.wikimedia.org/T337297 (10Aklapper) [15:47:29] 10Release-Engineering-Team (They Live 🕶️🧟), 10serviceops-collab, 10Patch-For-Review: upgrade gerrit servers to bullseye - https://phabricator.wikimedia.org/T334521 (10Dzahn) Thank you as well @hashar :) Sounds good. One follow-up: We did not actually merge https://gerrit.wikimedia.org/r/c/operations/puppet/... [15:49:09] 10Gerrit, 10Patch-For-Review: [wm-checks-api] Create a new gerrit bot for Patch Demo - https://phabricator.wikimedia.org/T332474 (10Esanders) Looks good, thanks again! [16:01:51] 10GitLab (Integrations), 10Phabricator: Get GitLab to render `T{\d}+` in MR overviews, comments, etc. as links to Phabricator - https://phabricator.wikimedia.org/T337570 (10Jdforrester-WMF) [16:09:19] 10Release-Engineering-Team, 10Scap, 10PyBal, 10SRE, and 4 others: High rate of errors and increased latency on uncached MediaWiki requests due to infrastructure outage - https://phabricator.wikimedia.org/T337497 (10jcrespo) An initial draft of a postmortem for this issue has been posted at: https://wikitec... [16:41:51] 10GitLab (Integrations): Get GitLab to render `T{\d}+` in MR overviews, comments, etc. as links to Phabricator - https://phabricator.wikimedia.org/T337570 (10Aklapper) [16:43:53] 10GitLab (Integrations), 10Phabricator: Get GitLab to render `T{\d}+` in MR overviews, comments, etc. as links to Phabricator - https://phabricator.wikimedia.org/T337570 (10Jdforrester-WMF) This is relevant to Phabricator, as without this kind of integration we'll need to dump it and use GitLab issues. [16:50:18] 10GitLab (Integrations), 10Phabricator: Get GitLab to render `T{\d}+` in MR overviews, comments, etc. as links to Phabricator - https://phabricator.wikimedia.org/T337570 (10bd808) A potential solution would be to work upstream to implement a Phabricator [[https://docs.gitlab.com/ee/integration/external-issue-t... [17:05:05] 10GitLab (Integrations), 10Phabricator: Get GitLab to render `T{\d}+` in MR overviews, comments, etc. as links to Phabricator - https://phabricator.wikimedia.org/T337570 (10Aklapper) After [removing Phabricator code, metrics, and documentation](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/117649) I'm... [18:04:53] 10Project-Admins: Create a #openstack-magnum - https://phabricator.wikimedia.org/T336941 (10nskaggs) 05Open→03Resolved a:03nskaggs I added a description and created this sub-project here: https://phabricator.wikimedia.org/project/view/6568/ [18:07:33] (03PS1) 10Hashar: wm-checks-api: add support for DUCT [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923688 (https://phabricator.wikimedia.org/T331651) [18:17:50] 10Gerrit, 10Abstract Wikipedia team, 10Tool-ducttape, 10WikiLambda, 10Patch-For-Review: [wm-checks-api] support kindrobot - https://phabricator.wikimedia.org/T331651 (10hashar) I have done the implementation :) Example rendering on https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikiLambda/+/90480... [18:26:29] 10Project-Admins: Create a #wmcs-superset tag - https://phabricator.wikimedia.org/T336940 (10rook) [18:31:15] 10Project-Admins, 10WMF-Movement-Insights: Create a project for the WMF Movement Insights team - https://phabricator.wikimedia.org/T337518 (10Mayakp.wiki) [18:33:31] (03PS2) 10Hashar: wm-checks-api: add support for DUCT [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923688 (https://phabricator.wikimedia.org/T331651) [18:37:31] (03CR) 10Hashar: "See T331651#8883870 and below for the rendering ;)" [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923688 (https://phabricator.wikimedia.org/T331651) (owner: 10Hashar) [18:38:12] 10Gerrit, 10Abstract Wikipedia team, 10Tool-ducttape, 10WikiLambda, 10Patch-For-Review: [wm-checks-api] support kindrobot - https://phabricator.wikimedia.org/T331651 (10hashar) {F37070595 layout=right, float, width=600} On https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikiLambda/+/923669 I notice... [18:42:04] (03CR) 10Jforrester: "Neat!" [software/gerrit] (deploy/wmf/stable-3.5) - 10https://gerrit.wikimedia.org/r/923688 (https://phabricator.wikimedia.org/T331651) (owner: 10Hashar) [18:42:29] James_F: time for a friday deploy? ;) [18:57:43] 10GitLab (Account Approval), 10Release-Engineering-Team, 10Abstract Wikipedia team: Requesting GitLab account activation for Nik.xyz.in - https://phabricator.wikimedia.org/T337534 (10Jdforrester-WMF) [19:04:41] hashar: Tsk. Not before a three day weekend, please. :-) [19:12:37] James_F: yeah very true :] [19:12:42] & [20:38:22] 10Project-Admins, 10WMF-Movement-Insights: Create a project for the WMF Movement Insights team - https://phabricator.wikimedia.org/T337518 (10nshahquinn-wmf) @Aklapper thank you! Is there a reason you went with "WMF-Movement-Insights" rather than just "Movement-Insights"? I thought the latter made sense given... [21:59:53] 10Project-Admins, 10WMF-Movement-Insights: Create a project for the WMF Movement Insights team - https://phabricator.wikimedia.org/T337518 (10Aklapper) No real reason, I guess I just didn't read carefully enough. Feel free to rename. [22:30:43] 10Project-Admins, 10Movement-Insights: Create a project for the WMF Movement Insights team - https://phabricator.wikimedia.org/T337518 (10nshahquinn-wmf) Okay, makes sense! [23:51:23] 10GitLab: Cannot log into GitLab due to email address conflict - https://phabricator.wikimedia.org/T337598 (10nshahquinn-wmf) [23:51:32] 10GitLab: Cannot log into GitLab due to email address conflict - https://phabricator.wikimedia.org/T337598 (10nshahquinn-wmf) a:05nshahquinn-wmf→03None