- Issue created by @catch
- Status changed to Needs work
8 months ago 7:35am 24 July 2024 - Status changed to Needs review
8 months ago 7:54am 24 July 2024 - 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Yeah this came from how we test on circle ci where spin up is faster, agree on current gitlab setup it's hurting us in overall resection time
- 🇬🇧United Kingdom catch
Only realised this after working on 📌 Try to store the phpstan result cache as an artifact Active and seeing just how much time we were spending on spinning up vs doing things. With the combination of the two issues, the newly combined lint stage finishes in around a minute. Once we add cspell/phpcs/eslint cache support, it should be consistently under a minute hopefully.
- Status changed to Needs work
8 months ago 12:22pm 25 July 2024 The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇬🇧United Kingdom catch
Just a straightforward rebase.
However I noticed since we added when: manual to the performance job, the job is blocked - needs to allow fail too it looks like.
- Status changed to Needs review
8 months ago 1:03pm 25 July 2024 - 🇬🇧United Kingdom catch
Latest pipeline on the MR finished in 6 minutes and 52 seconds: https://git.drupalcode.org/project/drupal/-/pipelines/233855
- 🇬🇧United Kingdom catch
This will also help the performance test run finish quicker. At the moment we're building composer and yarn to use the cache in performance tests, but it's much quicker to just run performance install in the actual test job before running the tests. Should save ~90s on those jobs.
- 🇬🇧United Kingdom catch
Since this takes a full minute off every core MR pipeline, bumping to major.
Combining this with some other changes, was able to get the minimum time for a full MR pipeline run down to 5m37s https://git.drupalcode.org/project/drupal/-/pipelines/234735
- 🇳🇱Netherlands spokje
The nightwatch job failed, not sure if fluke or bad, and can't rerun since MR was created by a core committer.
- 🇬🇧United Kingdom catch
Shouldn't affect the nightwatch test at all but re-ran it just in case.
- Status changed to RTBC
8 months ago 1:22pm 26 July 2024 - 🇳🇱Netherlands spokje
Thanks, tests are now green, as expected.
Code changes make sense.RTBC for me.
- Status changed to Downport
8 months ago 10:30pm 26 July 2024 - 🇫🇷France nod_ Lille
- 🇬🇧United Kingdom catch
OK the commit conflicts are because we haven't done 📌 Resync .gitlab-ci.yml and .gitignore following Yarn 4 in 11.x Needs review yet. Put up an MR there. Once that's in, this should be a clean(er) cherry-pick.
- 🇬🇧United Kingdom catch
Double checked the performance pipeline on 11.x and it's down to 8m30 now with the combination of this and 📌 Consolidate Umami performance tests Fixed , was previously closer to 13 minutes.
https://git.drupalcode.org/project/drupal/-/pipelines/235402
- Assigned to spokje
- Issue was unassigned.
- 🇳🇱Netherlands spokje
And as was said in #21 📌 Merge the build and lint stages in core MR pipelines Downport , lets postpone this on 📌 Resync .gitlab-ci.yml and .gitignore following Yarn 4 in 11.x Needs review to make our lives easier.
- Merge request !8960Merge the build and lint stages in core MR pipelines for 10.4.x → (Closed) created by nod_
- Status changed to Needs review
8 months ago 1:26pm 28 July 2024 - Status changed to RTBC
8 months ago 9:28am 29 July 2024 - 🇬🇧United Kingdom catch
Backport looks good.
Opened an issue for gitlab_templates since we might see a similar improvement there 📌 Consider merging the build and validate stages Active .
- 🇫🇷France nod_ Lille
adding back the
-s
for yarn 1 that got squeezed from https://git.drupalcode.org/project/drupal/-/commit/19c608d552a3bbedc554a... - Status changed to Fixed
8 months ago 10:01am 29 July 2024 - 🇫🇷France nod_ Lille
Committed and pushed 963de90958 to 10.4.x and d170a1e09d to 10.3.x. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Thanks for 📌 Consider merging the build and validate stages Active ! 🙏