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

Update dev environment #966

Merged
merged 2 commits into from
Oct 4, 2024
Merged

Conversation

gocallag
Copy link
Contributor

This commit removes the obsolete version tag in the docker-compose file and upgrades the postgres database container to 13.16 to match the client used in ovirt-engine

Fixes issue #965

Changes introduced with this PR

  • modifications to docker-compose file to upgrade to pgsql 13.16

Are you the owner of the code you are sending in, or do you have permission of the owner?

n

@sandrobonazzola
Copy link
Member

@gocallag is "n" a typo as answer to the question "Are you the owner of the code you are sending in, or do you have permission of the owner?" ? Aren't you the owner of the patch you're submitting?

docker-compose.yml Outdated Show resolved Hide resolved
@gocallag
Copy link
Contributor Author

gocallag commented Oct 4, 2024

@gocallag is "n" a typo as answer to the question "Are you the owner of the code you are sending in, or do you have permission of the owner?" ? Aren't you the owner of the patch you're submitting?

yes typo.

This commit removes the obsolete version tag in the docker-compose file and
upgrades the postgres database container to 13.16 to match the client used in ovirt-engine

Signed-off-by: geoff o'callaghan <[email protected]>
Avoids having to upgrade on centos stream updates. Only needs to change if/when
ovirt-engine requires postgres:14 or above.

Signed-off-by: geoff o'callaghan <[email protected]>
@sandrobonazzola sandrobonazzola merged commit 1e10b6e into oVirt:master Oct 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants