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

cephadm: bootstrap: avoid repeat chars in generated password #32332

Merged
merged 1 commit into from
Dec 20, 2019

Conversation

liewegas
Copy link
Member

INFO:cephadm:Non-zero exit code 22 from /usr/bin/podman run --rm --net=host -e CONTAINER_IMAGE=ceph/daemon-base:latest-master-devel -e NODE_NAME=gnit -v /var/log/ceph/00000000-0000-0000-0000-0000deadbeef:/var/log/ceph:z -v /tmp/ceph-tmp6if8iaha:/etc/ceph/ceph.client.admin.keyring:z -v /tmp/ceph-tmp4vyvfiio:/etc/ceph/ceph.conf:z --entrypoint /usr/bin/ceph ceph/daemon-base:latest-master-devel dashboard ac-user-create admin oaaacwf4ul administrator
INFO:cephadm:/usr/bin/ceph:stderr Error EINVAL: Password cannot contain repetitive characters.

Signed-off-by: Sage Weil sage@redhat.com

@sebastian-philipp
Copy link
Contributor

sebastian-philipp commented Dec 19, 2019

Ignore the password complexity checks, since we might (randomly) generate
something that fails whatever the current criteria are.

INFO:cephadm:Non-zero exit code 22 from /usr/bin/podman run --rm --net=host -e CONTAINER_IMAGE=ceph/daemon-base:latest-master-devel -e NODE_NAME=gnit -v /var/log/ceph/00000000-0000-0000-0000-0000deadbeef:/var/log/ceph:z -v /tmp/ceph-tmp6if8iaha:/etc/ceph/ceph.client.admin.keyring:z -v /tmp/ceph-tmp4vyvfiio:/etc/ceph/ceph.conf:z --entrypoint /usr/bin/ceph ceph/daemon-base:latest-master-devel dashboard ac-user-create admin oaaacwf4ul administrator
INFO:cephadm:/usr/bin/ceph:stderr Error EINVAL: Password cannot contain repetitive characters.

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-cephadm-bootstrap-password branch from 319d0cf to bb3750d Compare December 19, 2019 19:23
@liewegas
Copy link
Member Author

updated to --force-passowrd.. much better! the criteria are a moving target.

liewegas added a commit that referenced this pull request Dec 20, 2019
* refs/pull/32332/head:
	cephadm: bootstrap: force password choice

Reviewed-by: Sebastian Wagner <swagner@suse.com>
Reviewed-by: Michael Fritch <mfritch@suse.com>
@liewegas liewegas merged commit bb3750d into ceph:master Dec 20, 2019
@liewegas liewegas deleted the wip-cephadm-bootstrap-password branch December 20, 2019 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants