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

[DPE-6395] Update to Postgresql 14.15 #72

Merged
merged 3 commits into from
Jan 17, 2025
Merged

[DPE-6395] Update to Postgresql 14.15 #72

merged 3 commits into from
Jan 17, 2025

Conversation

dragomirp
Copy link
Contributor

@dragomirp dragomirp commented Jan 16, 2025

Update snapped Postgresql to 14.15

@dragomirp dragomirp marked this pull request as ready for review January 16, 2025 14:51
@@ -12,11 +12,11 @@ on:
jobs:
lint:
name: Lint
uses: canonical/data-platform-workflows/.github/workflows/lint.yaml@v23.1.0
uses: canonical/data-platform-workflows/.github/workflows/lint.yaml@v24.0.6
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Version that works on 24.04

Copy link
Contributor

Choose a reason for hiding this comment

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

would recommend v29, since it includes this fix: canonical/data-platform-workflows#269

Comment on lines +29 to +31
- name: Uninstall Postgresql
run: |
sudo apt --purge remove postgresql*
Copy link
Contributor Author

@dragomirp dragomirp Jan 16, 2025

Choose a reason for hiding this comment

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

Workaround for 24.04, which has preinstalled PG 16

Choose a reason for hiding this comment

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

Do you mean GH runner 24.04?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.

@dragomirp dragomirp requested review from a team, taurus-forever, marceloneppel and lucasgameiroborges and removed request for a team January 16, 2025 14:53
Comment on lines +29 to +31
- name: Uninstall Postgresql
run: |
sudo apt --purge remove postgresql*

Choose a reason for hiding this comment

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

Do you mean GH runner 24.04?

@@ -12,11 +12,11 @@ on:
jobs:
lint:
name: Lint
uses: canonical/data-platform-workflows/.github/workflows/lint.yaml@v23.1.0
uses: canonical/data-platform-workflows/.github/workflows/lint.yaml@v24.0.6
Copy link
Contributor

Choose a reason for hiding this comment

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

would recommend v29, since it includes this fix: canonical/data-platform-workflows#269

@dragomirp dragomirp merged commit 067da15 into 14/edge Jan 17, 2025
6 checks passed
@dragomirp dragomirp deleted the pg1415 branch January 17, 2025 10:01
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.

3 participants