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

Upgrade to PostgreSQL 14 #110

Merged
merged 1 commit into from
Nov 2, 2021

Conversation

rgl
Copy link
Contributor

@rgl rgl commented Oct 12, 2021

Description

Upgrade to PostgreSQL 14.

Why is this needed

Please see tinkerbell/tink#552

Fixes: tinkerbell/tink#551

@rgl rgl mentioned this pull request Oct 12, 2021
3 tasks
@jacobweinstock
Copy link
Member

Hey @rgl, looks like our DCO bot is not happy. Would you be able to maybe re-push or rebase or git ammend?

Signed-off-by: Rui Lopes <rgl@ruilopes.com>
@rgl rgl force-pushed the rgl-upgrade-to-postgres-14 branch from 204dffe to bffaa7b Compare October 19, 2021 17:52
@rgl
Copy link
Contributor Author

rgl commented Oct 19, 2021

@jacobweinstock, it should be fixed now. I'm not sure what happened.

@jacobweinstock jacobweinstock merged commit e3f8d28 into tinkerbell:main Nov 2, 2021
mergify bot added a commit to tinkerbell/tink that referenced this pull request Nov 2, 2021
## Description

Upgrade to PostgreSQL 14.

## Why is this needed

Besides all the major improvements made from PostgreSQL 10 to 14, there is one particular feature that I wanted to use:

[A nicer way to work with JSON](https://www.postgresql.org/docs/14/datatype-json.html#JSONB-SUBSCRIPTING), like the hardware data model :-)

Fixes: #551

The sandbox MR is at tinkerbell/playground#110.

## How Has This Been Tested?

Besides the unit tests executed by CI, I've successfully tested this locally in a sandbox, and PostgreSQL 14 is now running:

```console
root@provisioner:~# docker exec -i compose-db-1 psql -U tinkerbell -A -c 'select version()'
version
PostgreSQL 14.0 on x86_64-pc-linux-musl, compiled by gcc (Alpine 10.3.1_git20210424) 10.3.1 20210424, 64-bit
(1 row)
```

## How are existing users impacted? What migration steps/scripts do we need?

<!--- Fixes a bug, unblocks installation, removes a component of the stack etc -->
<!--- Requires a DB migration script, etc. -->

## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
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.

Migrate to PostgreSQL 14
2 participants