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

Rgw fixes #2428

Merged
merged 5 commits into from
Mar 2, 2018
Merged

Rgw fixes #2428

merged 5 commits into from
Mar 2, 2018

Conversation

leseb
Copy link
Member

@leseb leseb commented Mar 1, 2018

No description provided.

@leseb leseb force-pushed the rgw-fixes branch 3 times, most recently from b05d06d to 2951ced Compare March 2, 2018 11:01
@leseb
Copy link
Member Author

leseb commented Mar 2, 2018

@@ -1,6 +1,7 @@
---
- name: update period
command: radosgw-admin period update --commit
command: radosgw-admin --cluster {{ cluster }} period update --commit
delegate_to: "{{ groups[mon_group_name][0] }}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

run_once: true ?

Copy link
Member Author

Choose a reason for hiding this comment

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

affirmatif

Copy link
Collaborator

Choose a reason for hiding this comment

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

ok maggle

In case the admin wasn't copied over to the node this command would
fail. So it's safer to run it from a monitor directly.

Signed-off-by: Sébastien Han <seb@redhat.com>
If we now set copy_admin_key while running a containerized scenario, the
ceph admin key will be copied on the node.

Signed-off-by: Sébastien Han <seb@redhat.com>
Signed-off-by: Sébastien Han <seb@redhat.com>
If the cluster name is different than 'ceph', the command will fail so
we need to pass the cluster name.

Signed-off-by: Sébastien Han <seb@redhat.com>
The command doesn't exist on Red Hat systems so it's better to skip it
instead of ignoring the error.

Signed-off-by: Sébastien Han <seb@redhat.com>
@guits guits merged commit cb0f598 into master Mar 2, 2018
@guits guits deleted the rgw-fixes branch March 2, 2018 20:59
@guits guits mentioned this pull request Mar 2, 2018
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.

2 participants