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

hammer: ceph init script unconditionally sources /lib/lsb/init-functions #7797

Merged
1 commit merged into from Mar 4, 2016

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm self-assigned this Feb 25, 2016
@smithfarm smithfarm added this to the hammer milestone Feb 25, 2016
@ktdreyer
Copy link
Member

LGTM

ghost pushed a commit that referenced this pull request Feb 26, 2016
…urces /lib/lsb/init-functions

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Mar 3, 2016

this passed a run of the upgrade suite ( http://tracker.ceph.com/issues/14692#note-29 ) but make check failed (the logs are gone though). @smithfarm could you please rebase & repush to get a another run ?

On OSX/FreeBSD, /lib/lsb/init-functions does not exist

Signed-off-by: Yan, Zheng <zyan@redhat.com>
(cherry picked from commit bb1fa7f)
ghost pushed a commit that referenced this pull request Mar 4, 2016
…urces /lib/lsb/init-functions

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Mar 4, 2016
hammer: ceph init script unconditionally sources /lib/lsb/init-functions

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
@ghost ghost merged commit 522e3e8 into ceph:hammer Mar 4, 2016
@smithfarm smithfarm deleted the wip-14861-hammer branch March 4, 2016 11:42
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants