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

replace ceph-disk-* with ceph-disk analogs #218

Open
wants to merge 64 commits into
base: master
Choose a base branch
from

Conversation

fzylogic
Copy link

With infernalis, ceph-disk-[prepare|activate] are gone and
were just dumb wrappers anyway. Replace calls to them with
appropriate calls to ceph-disk.

fzylogic and others added 30 commits October 13, 2015 23:49
With infernalis, ceph-disk-[prepare|activate] are gone and
were just dumb wrappers anyway. Replace calls to them with
appropriate calls to ceph-disk.
…for repository, release key and autobuild key download
daemon now runs as "ceph" user.
ceph daemons now run as ceph:ceph
Parts of the conf recipe reference the "ceph" user, which comes from the
package.  Therefore, the packages need to be installed before running
the conf recipe.
Install the packages early on
When chown-ing files/folder (during ceph::conf), use the UID instead of
the name.  This should fix a race condition we're seeing between package
installs and config file deploys.
Use UID instead of name for chown
bumps default version to jewel
Packages must be installed before ceph::conf is called.  ceph::conf
references the ceph user, which comes from the package and as such, will
never be able to complete unless the packages are installed first.
oddomatik and others added 30 commits February 16, 2017 12:22
use ceph user to run mon create etc commands.
Revert "use ceph user to run mon create etc commands."
update user, owner, group for jewel
merge Ipv6test back into master
mon host won't be defined if ['ceph']['config']['global']['mon host'] or ['ceph']['config']['global']['mon_host'] is
revert mistaken apache version increment too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants