Skip to content

ci: only use requested images; enable opcache#706

Merged
smbader merged 1 commit intoncstate-delta:mainfrom
jrchamp:ci/performance
Mar 26, 2026
Merged

ci: only use requested images; enable opcache#706
smbader merged 1 commit intoncstate-delta:mainfrom
jrchamp:ci/performance

Conversation

@jrchamp
Copy link
Copy Markdown
Collaborator

@jrchamp jrchamp commented Mar 23, 2026

  • conditionally enable only the active database (inspired by the moodle/moodle workflow)
  • switch to current version of checkout action (v4 is emitting a deprecation warning)
  • enable opcache for CLI (saves ~30 seconds during Moodle install)
  • bumps the postgres image to 16, which is required by Moodle main/5.3

I'll be pushing for similar changes directly in moodle core or moodle-plugin-ci, but I am already using these for my tests so I wanted to share them with y'all. ❤️

@jrchamp jrchamp requested a review from a team March 23, 2026 16:59
@jrchamp jrchamp self-assigned this Mar 23, 2026
@jrchamp jrchamp added the bug Fixes problems or reduces technical debt label Mar 23, 2026
Copy link
Copy Markdown
Contributor

@smbader smbader left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow! This looks great, thank you for keeping us up-to-date.

@smbader smbader merged commit 97e7109 into ncstate-delta:main Mar 26, 2026
6 checks passed
@jrchamp jrchamp deleted the ci/performance branch March 28, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes problems or reduces technical debt

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants