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: upstart: radosgw-all does not start on boot if ceph-base is not installed #13777

Closed
wants to merge 1 commit into from

Conversation

smithfarm
Copy link
Contributor

Prior to this change, the radosgw-all service depended on the ceph-all
service in order to trigger a start operation. ceph-all.conf ships in
the ceph-base package. This means that "radosgw-all" Upstart service
would not start on boot unless the "ceph-base" package was also
installed.

Break this dependency by starting radosgw-all according to the system's
runlevel, rather than only listening for the ceph-all starting signal.

This change causes RGW to start on boot even when ceph-base is not
installed.

Preserve the original "on starting ceph-all" condition here as well, so
that users can still start RGW with the documented "start ceph-all"
command.

(Note: This has the side effect that if a user has a "ceph-all.override
-> manual" file on their system because they intentionally wanted to
prevent all Ceph-related services from starting during boot, RGW will
still start on boot when it did not previously. Users will need to
explicitly create a "radosgw-all.override -> manual" file as well if
they want to prevent RGW from starting on boot.)

Fixes: http://tracker.ceph.com/issues/18313

Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
(cherry picked from commit 230b9d5)
@smithfarm smithfarm self-assigned this Mar 3, 2017
@smithfarm smithfarm added this to the hammer milestone Mar 3, 2017
@smithfarm smithfarm requested a review from ktdreyer March 3, 2017 13:43
@smithfarm
Copy link
Contributor Author

Hammer is EOL.

@smithfarm smithfarm closed this Sep 3, 2017
@smithfarm smithfarm deleted the wip-18854-hammer branch September 3, 2017 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants