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

Add postgresql version 9.6 #182

Merged
merged 4 commits into from
Aug 21, 2017
Merged

Add postgresql version 9.6 #182

merged 4 commits into from
Aug 21, 2017

Conversation

pkubatrh
Copy link
Member

Uses sclorg/container-common-scripts#20 . Should not be merged until this PR is merged.

Packages for 9.6 not yet available so the build has been disabled for this version for now.

@pkubatrh
Copy link
Member Author

Pulled in newest container-common-scripts.

@praiskup
Copy link
Contributor

We can't build 9.6 image at all, which is expected probably. Otherwise it looks good to me.

@praiskup
Copy link
Contributor

Ah, git grep 95 latest and git grep 9.5 latest don't give me empty output yet.

@praiskup
Copy link
Contributor

praiskup commented Aug 18, 2017

What about to have TARGET=f26_copr for Dockerfile.f26_copr; and install packages from https://copr.fedorainfracloud.org/coprs/g/db-sig/sclo-postgresql96 ? (Doesn't have to be done within this PR of course). edit: maybe rather TARGET=fedora_copr to allow us to have only one fedora dockerfile till we have some templating system on place ...

@pkubatrh
Copy link
Member Author

Ah, git grep 95 latest and git grep 9.5 latest don't give me empty output yet.

Right, since the README was changed after the PR was created. Will make sure it uses correct versions.

@pkubatrh
Copy link
Member Author

edit: maybe rather TARGET=fedora_copr to allow us to have only one fedora dockerfile till we have some templating system on place ...

Might be better to just use the regular fedora target for this (for now), since we do not have a fedora Dockerfile for 9.6 yet anyway. And this way we would not need to make any additional changes to the common scripts until we know what kind of templating system to use in future.

@praiskup
Copy link
Contributor

Per @hhorak 's note, the rh-postgresql96 packages already live here:
https://cbs.centos.org/repos/sclo7-rh-postgresql96-rh-candidate/x86_64/os/

Perhaps we could enable the repository temporarily to have at least one container built. @pkubatrh WDYT?

@praiskup
Copy link
Contributor

Without paying too much attention to the patch, this did the trick for me:

 RUN yum install -y centos-release-scl-rh && \
+    yum-config-manager --add-repo https://cbs.centos.org/repos/sclo7-rh-postgresql96-rh-candidate/x86_64/os/ && \
+    echo gpgcheck=0 >> /etc/yum.repos.d/cbs.centos.org_repos_sclo7-rh-postgresql96-rh-candidate_x86_64_os_.repo && \
     INSTALL_PKGS="rsync tar gettext bind-utils rh-postgresql96 rh-postgresql96-postgresql-contrib nss_wrapper" && \

@praiskup
Copy link
Contributor

Otherwise looks good now, thanks.

@pkubatrh
Copy link
Member Author

Good idea, thanks. I have enabled the centos7 build with testing rpms

@pkubatrh
Copy link
Member Author

[test]

@praiskup
Copy link
Contributor

LGTM

@pkubatrh
Copy link
Member Author

Tests are passing now, merging. Thanks for review

@pkubatrh pkubatrh merged commit 8c2b6aa into sclorg:master Aug 21, 2017
@pkubatrh pkubatrh deleted the 9.6 branch August 21, 2017 09:53
@pkubatrh pkubatrh mentioned this pull request Sep 20, 2017
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.

2 participants