[10:51:23] 10Quibble, 10MediaWiki-Core-Tests, 10Platform Team Initiatives (API Integration Tests), 10ci-test-error: api-testing: "should ignore duplicate redirect source and target if both pages are a match" test fails on SQLite - https://phabricator.wikimedia.org/T302706 (10kostajh) [10:54:24] 10Quibble, 10MediaWiki-Core-Tests, 10Patch-For-Review, 10Platform Team Initiatives (API Integration Tests), 10ci-test-error: api-testing: "should ignore duplicate redirect source and target if both pages are a match" test fails on SQLite - https://phabricator.wikimedia.org/T302706 (10kostajh) Fixing the... [10:59:12] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10ci-test-error: QUnit: The value provided to `assert.throws` in "ve.dm.MWGraphModel.static" was not a function. - https://phabricator.wikimedia.org/T302707 (10kostajh) [11:01:17] hi hashar [11:01:31] I think those two jjb patches would unblock Quibble CI. 🤞 [11:01:48] hii [11:02:22] kostajh: I am not sure how you have managed to track those down :] [11:02:24] impressive [11:03:16] I would guess there is a race condition in the js loading [11:03:45] well, I wish I had tracked down the root cause, but I don't see anything obvious so I think we should just skip them [11:06:20] the Graph extension does have some integration with VE [11:06:39] who knows what ended up being broken though :\ [11:08:25] I have updated the two jobs and +2ed the CI config patches [11:09:50] ah, I can reproduce the sqlite one locally at least. [11:10:07] but I will leave debugging that for another time [11:10:30] (03CR) 10Kosta Harlan: "recheck" [integration/quibble] - 10https://gerrit.wikimedia.org/r/764419 (https://phabricator.wikimedia.org/T302226) (owner: 10Kosta Harlan) [11:24:21] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10ci-test-error: QUnit: The value provided to `assert.throws` in "ve.dm.MWGraphModel.static" was not a function. - https://phabricator.wikimedia.org/T302707 (10hashar) I can not reproduce with: | Graph + VisualEditor | php built in server | MySQL Ma... [11:53:05] 10Quibble, 10MediaWiki-Core-Tests, 10Platform Team Initiatives (API Integration Tests), 10SQLite, 10ci-test-error: api-testing: "should ignore duplicate redirect source and target if both pages are a match" test fails on SQLite - https://phabricator.wikimedia.org/T302706 (10kostajh) The difference betwee... [11:58:39] 10Quibble, 10MediaWiki-Core-Tests, 10Platform Team Initiatives (API Integration Tests), 10SQLite, 10ci-test-error: api-testing: "should ignore duplicate redirect source and target if both pages are a match" test fails on SQLite - https://phabricator.wikimedia.org/T302706 (10kostajh) Adding some people wh... [11:58:45] (03CR) 10Kosta Harlan: [C: 03+2] "per Antoine" [integration/quibble] - 10https://gerrit.wikimedia.org/r/764419 (https://phabricator.wikimedia.org/T302226) (owner: 10Kosta Harlan) [11:58:49] hashar: yay, it worked [11:58:58] hurrah! [12:00:14] (03CR) 10Hashar: [C: 03+2] Remove Dockerfile [integration/quibble] - 10https://gerrit.wikimedia.org/r/757888 (owner: 10Kosta Harlan) [12:00:52] now we just need to wait for the backend.py patch to merge [12:00:55] then I guess we can cut a new quibble version [12:01:06] Cause the SQLite jobs will fail until that one gets in [12:43:36] (03Merged) 10jenkins-bot: backend: Check if key exists before deleting [integration/quibble] - 10https://gerrit.wikimedia.org/r/764419 (https://phabricator.wikimedia.org/T302226) (owner: 10Kosta Harlan) [12:44:16] (03Merged) 10jenkins-bot: Remove Dockerfile [integration/quibble] - 10https://gerrit.wikimedia.org/r/757888 (owner: 10Kosta Harlan) [12:46:31] (03CR) 10Kosta Harlan: "recheck" [integration/quibble] - 10https://gerrit.wikimedia.org/r/760948 (owner: 10Kosta Harlan) [12:46:35] (03CR) 10Kosta Harlan: "recheck" [integration/quibble] - 10https://gerrit.wikimedia.org/r/758783 (https://phabricator.wikimedia.org/T235449) (owner: 10Kosta Harlan) [13:26:00] (03CR) 10jerkins-bot: [V: 04-1] Parallelize phpunit -databaseless, -standalone and -database [integration/quibble] - 10https://gerrit.wikimedia.org/r/758783 (https://phabricator.wikimedia.org/T235449) (owner: 10Kosta Harlan) [13:29:45] (03CR) 10Kosta Harlan: "Shaves several minutes off the ci-fullrun and extensions jobs, but looks like some tests have issues when this is run in SQLite." [integration/quibble] - 10https://gerrit.wikimedia.org/r/758783 (https://phabricator.wikimedia.org/T235449) (owner: 10Kosta Harlan) [13:47:07] (03CR) 10Kosta Harlan: "Looks like it might save 1-2 minutes overall." [integration/quibble] - 10https://gerrit.wikimedia.org/r/760948 (owner: 10Kosta Harlan) [13:53:12] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10ci-test-error: QUnit: The value provided to `assert.throws` in "ve.dm.MWGraphModel.static" was not a function. - https://phabricator.wikimedia.org/T302707 (10Krinkle) This shouldn't be possible with any valid set of repositories. Perhaps something... [13:53:38] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10ci-test-error: quibble-fullrun-extensions: Fails on ve.dm.MWGraphModel.static test. - https://phabricator.wikimedia.org/T302707 (10Krinkle) [13:55:55] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10ci-test-error: quibble-fullrun-extensions: Fails on ve.dm.MWGraphModel.static test. - https://phabricator.wikimedia.org/T302707 (10kostajh) >>! In T302707#7741395, @Krinkle wrote: > This shouldn't be possible with any valid set of repositories. Per... [14:07:06] back [14:26:56] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10ci-test-error: quibble-fullrun-extensions: Fails on ve.dm.MWGraphModel.static test. - https://phabricator.wikimedia.org/T302707 (10hashar) The Quibble build started on Feb 26, 2022 at 17:26 UTC. it does: ` INFO:zuul.Cloner.mediawiki/extensions/Grap... [14:41:24] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10ci-test-error: quibble-fullrun-extensions: Fails on ve.dm.MWGraphModel.static test. - https://phabricator.wikimedia.org/T302707 (10hashar) The build has `ZUUL_URL=git://contint2001.wikimedia.org` (which is normal) and on that server we can still se... [14:46:18] (03PS1) 10Hashar: ci-fullrun: do not use ZUUL_REF [integration/quibble] - 10https://gerrit.wikimedia.org/r/766774 (https://phabricator.wikimedia.org/T302707) [14:46:38] kostajh: ^ that is for Graph being checked out with an obsolete version of the master branch :D [14:47:02] which will surely fix a few the broken build we had for the last few days [14:47:14] cause the utils/ci-fullrun script were fetching arbitrary codes from the repository [14:47:21] respositories [14:47:22] fun [14:47:23] sorry about that [14:47:34] yeah fun :D [14:47:46] I haven't caught that one when I approved the scripts doing ZUUL_REF=master [14:48:05] so most probably we can revert the two changes made this morning to skip qunit or api-testing with sqlite [14:48:15] they were most probably due to some incompatibilities between the branch [14:48:27] or checking out an obsolete mediawiki/core [14:48:27] (03CR) 10Kosta Harlan: [C: 03+2] ci-fullrun: do not use ZUUL_REF (031 comment) [integration/quibble] - 10https://gerrit.wikimedia.org/r/766774 (https://phabricator.wikimedia.org/T302707) (owner: 10Hashar) [14:49:05] no, the sqlite one is a problem in core, I dug into it a little bit. [14:49:36] and posted some comments in T302706#7741178 [14:49:42] the qunit skipping patch can be reverted though [14:50:55] oh so that is a bug in core!? :D [14:52:16] sometimes tests are useful, what can i say [14:52:26] (assuminmg it is, in fact, a bug. it looks like an inconsistency, anyway. [15:30:20] (03Merged) 10jenkins-bot: ci-fullrun: do not use ZUUL_REF [integration/quibble] - 10https://gerrit.wikimedia.org/r/766774 (https://phabricator.wikimedia.org/T302707) (owner: 10Hashar) [15:35:26] 10Quibble, 10MediaWiki-extensions-Graph, 10VisualEditor, 10Patch-For-Review, 10ci-test-error: quibble-fullrun-extensions: Fails on ve.dm.MWGraphModel.static test. - https://phabricator.wikimedia.org/T302707 (10hashar) 05Open→03Resolved a:03hashar >>! In T302707#7741395, @Krinkle wrote: > This shoul... [16:15:49] (03PS2) 10Hashar: ci-fullrun-extensions: Run npm install in parallel [integration/quibble] - 10https://gerrit.wikimedia.org/r/760948 (owner: 10Kosta Harlan) [16:16:36] (03CR) 10Hashar: "Rebased and fixed the trivial conflict. lets see how the build behave and then I guess we can merge it ;)" [integration/quibble] - 10https://gerrit.wikimedia.org/r/760948 (owner: 10Kosta Harlan) [17:07:30] (03CR) 10jerkins-bot: [V: 04-1] ci-fullrun-extensions: Run npm install in parallel [integration/quibble] - 10https://gerrit.wikimedia.org/r/760948 (owner: 10Kosta Harlan) [17:19:58] 10Quibble, 10MediaWiki-Core-Tests, 10Platform Team Initiatives (API Integration Tests), 10SQLite, 10ci-test-error: api-testing: "should ignore duplicate redirect source and target if both pages are a match" test fails on SQLite - https://phabricator.wikimedia.org/T302706 (10nnikkhoui) I think what is hap... [19:46:29] 10Quibble, 10MediaWiki-Core-Tests, 10Patch-For-Review, 10Platform Team Initiatives (API Integration Tests), and 2 others: api-testing: "should ignore duplicate redirect source and target if both pages are a match" test fails on SQLite - https://phabricator.wikimedia.org/T302706 (10kostajh) 05Open→03Reso...