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

Missing sanity check: user "cephadm" must exist on all minions that have cephadm role #372

Closed
smithfarm opened this issue Sep 4, 2020 · 2 comments
Assignees

Comments

@smithfarm
Copy link
Contributor

Although the act of installing the cephadm package can be expected to create the cephadm user, this does not give us 100% certitude that the cephadm user will exist on all nodes where we expect it to. Users might delete the cephadm user, for example.

@smithfarm smithfarm self-assigned this Sep 4, 2020
@ricardoasmarques
Copy link
Contributor

This can only happen on ceph-salt status, because on ceph-salt apply/update/reboot the cephadm user will be recreated.

Relates to #379

@ricardoasmarques
Copy link
Contributor

Note that ceph-salt explicitly creates the cephadm user [1], which means it will be recreated, if needed, everytime cephadm apply is executed.`

I don't see a reason to add this check, so I'm now closing the issue.

Feel free to reopen it if I'm missing something.

[1] https://github.com/ceph/ceph-salt/blob/master/ceph-salt-formula/salt/ceph-salt/common/sshkey.sls#L9-L15

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

No branches or pull requests

2 participants