Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

Fix CI: Postgresql install was missing apt-get update #516

Merged
merged 1 commit into from
May 25, 2022

Conversation

atomicules
Copy link
Collaborator

@atomicules atomicules commented Apr 23, 2022

Github actions recently switched from focal to jammy and for some reason
this caused the Postgresql install to fail. According to the wiki Jammy
is supported, but the install script was missing an apt-get update.
Adding that seems to fix things.

Leaving a commented out lsb_release for debugging.

Required for all PRs:

  • N/A CHANGELOG.md updated (feel free to wait until changes have been reviewed by a maintainer)
  • N/A README.md updated (if needed)

@atomicules atomicules marked this pull request as draft April 23, 2022 14:07
@atomicules atomicules force-pushed the fix-postgresql-tests branch 2 times, most recently from 87cac9a to 4ffd293 Compare April 23, 2022 14:14
@atomicules atomicules changed the title Github actions have switched from focal to jammy Fix CI: Postgresql install was missing apt-get update Apr 23, 2022
@atomicules atomicules marked this pull request as ready for review April 23, 2022 14:17
Github actions recently switched from focal to jammy and for some reason
this caused the Postgresql install to fail. According to [the wiki][1] Jammy
is supported, but the install script was missing an `apt-get update`.
Adding that seems to fix things.

Leaving a commented out `lsb_release` for debugging.

[1]: https://wiki.postgresql.org/wiki/Apt
Copy link
Contributor

@pete-may pete-may left a comment

Choose a reason for hiding this comment

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

LGTM

@atomicules
Copy link
Collaborator Author

Thanks!

@atomicules atomicules merged commit 631f9ad into master May 25, 2022
@atomicules atomicules deleted the fix-postgresql-tests branch May 25, 2022 11:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants