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
jewel: build/ops: install-deps.sh based on /etc/os-release #12405
Conversation
@jan--f Please get rid of the merge commit. |
@dachary IIRC there is nothing in the integration test suites that exercises install-deps.sh, correct? |
9e0ac66
to
362b7af
Compare
@jan--f Please rebase to the latest jewel (i.e. something like |
The tox test failure is benign. |
362b7af
to
a3e9148
Compare
This avoids initial distribution detection (before lsb_release is available) based on the presence of a specific package manager. This caused an issue with openSUSE Tumbleweed before since both zypper and apt-get are available. Signed-off-by: Jan Fajerski <jfajerski@suse.com> (cherry picked from commit 85a370e)
85a370e introduced the DISTRO variable whose value was obtained by parsing /etc/os-release like this: DISTRO=$(grep "^ID=" /etc/os-release | sed "s/ID=//") This unfortunately picks up the double-quotes, so on a CentOS system DISTRO will be equal to '"centos"'. Signed-off-by: Nathan Cutler <ncutler@suse.com> (cherry picked from commit fd37e87)
a3e9148
to
240431b
Compare
Jenkins, retest this please |
@jan--f Please cherry-pick 266f4d4 into this PR. When you do please remind me to:
|
Avoids this error on SLES systems: > ./install-deps.sh sles is unknown, dependencies will have to be installed manually. Signed-off-by: Nitin A Kamble <Nitin.Kamble@Teradata.com> (cherry picked from commit 266f4d4) Conflict in install-deps.sh due to indentation. Trivial resolution.
Signed-off-by: Jan Fajerski <jfajerski@suse.com>
Jenkins, retest this please |
@jan--f Excellent, thanks! |
http://tracker.ceph.com/issues/18199
http://tracker.ceph.com/issues/18466