-
Notifications
You must be signed in to change notification settings - Fork 217
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
Conversation
37c2a00
to
14a51c7
Compare
d1c8b48
to
4aba7bd
Compare
Pulled in newest container-common-scripts. |
We can't build 9.6 image at all, which is expected probably. Otherwise it looks good to me. |
Ah, |
What about to have |
Right, since the README was changed after the PR was created. Will make sure it uses correct versions. |
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. |
Per @hhorak 's note, the rh-postgresql96 packages already live here: Perhaps we could enable the repository temporarily to have at least one container built. @pkubatrh WDYT? |
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" && \ |
Otherwise looks good now, thanks. |
Good idea, thanks. I have enabled the centos7 build with testing rpms |
[test] |
LGTM |
Tests are passing now, merging. Thanks for review |
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.