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

upstart: start ceph-all after static-network-up #11631

Merged
1 commit merged into from Nov 23, 2016
Merged

Conversation

wolsen
Copy link
Contributor

@wolsen wolsen commented Oct 24, 2016

Starting on runlevel [2345] allows the ceph services to
start before all of the networking is configured. This
introduces a race condition which allows a service to start
before the network it binds to is available.

Add the static-network-up event as a dependency for the start
on directive in order to start the ceph services after all of
the network stanzas have been processed and executed in the
/etc/network/interfaces and /etc/network/interfaces.d/*.conf
files.

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

Signed-off-by: Billy Olsen billy.olsen@gmail.com

Starting on runlevel [2345] allows the ceph services to
start before all of the networking is configured. This
introduces a race condition which allows a service to start
before the network it binds to is available.

Add the static-network-up event as a dependency for the start
on directive in order to start the ceph services after all of
the network stanzas have been processed and executed in the
/etc/network/interfaces and /etc/network/interfaces.d/*.conf
files.

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

Signed-off-by: Billy Olsen <billy.olsen@gmail.com>
@wolsen
Copy link
Contributor Author

wolsen commented Oct 28, 2016

retest this please

@ghost
Copy link

ghost commented Nov 23, 2016

@javacruft care to review ?

Copy link
Contributor

@javacruft javacruft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ghost
Copy link

ghost commented Nov 23, 2016

@javacruft thanks !

@ghost ghost merged commit f1b7a06 into ceph:master Nov 23, 2016
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
2 participants