Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: macos-latest: now without PostgreSQL #488

Merged
merged 1 commit into from
May 22, 2024

Conversation

jcflack
Copy link
Contributor

@jcflack jcflack commented May 21, 2024

Per the 'roadmap' announced in actions/runner-images#9255, macos-latest now points to macos-14 as of Q2, and the macos-14 image does not supply any version of PostgreSQL (or any database; its much-shorter installed-software list might not be an oversight). Bad news for the quick workflow that relies on the runner's installed version.

Limit the MacOS CI to the macos-12 runner for now. That should hold until the image is removed in Q4. A workflow that installs a specified PostgreSQL version may be needed after that.

Also, shorten the name of the workflow. It was too long to read in the online display of checks.

And add a restrictive permissions: entry, which was missing, and its absence is not secure-by-default.

Per the 'roadmap' announced in actions/runner-images#9255,
macos-latest now points to macos-14 as of Q2, and the macos-14
image does not supply any version of PostgreSQL. Bad news for
the quick workflow that relies on the runner's installed version.

Limit the MacOS CI to the macos-12 runner for now. That should
hold until the image is removed in Q4. A workflow that installs
a specified PostgreSQL version may be needed after that.

Also, shorten the name of the workflow. It was too long to read
in the online display of checks.

And add a restrictive permissions: entry, which was missing, and
its absence is not secure-by-default.
@jcflack jcflack merged commit 61699f3 into REL1_6_STABLE May 22, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant