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

library/radosgw_user.py: fix user update #7321

Merged
merged 1 commit into from
Oct 10, 2022

Conversation

karasjoh000
Copy link
Contributor

Removes the case when display_name was defined prevously but was not provided when modifying. Without this change the module will change display_name to name even if display_name was not name originally. See #7296

Removes the case when display_name was defined prevously but
was not provided when modifying. Without this change the module
will change display_name to name even if display_name was not name
originally. See ceph#7296

Signed-off-by: John Karasev <john.karasev@intel.com>
@guits
Copy link
Collaborator

guits commented Oct 5, 2022

jenkins test centos-non_container-lvm_osds

@guits
Copy link
Collaborator

guits commented Oct 5, 2022

jenkins test centos-non_container-update

@guits
Copy link
Collaborator

guits commented Oct 6, 2022

jenkins test centos-container-all_daemons

@guits
Copy link
Collaborator

guits commented Oct 6, 2022

jenkins test centos-container-purge

@guits
Copy link
Collaborator

guits commented Oct 6, 2022

jenkins test centos-container-update

2 similar comments
@guits
Copy link
Collaborator

guits commented Oct 7, 2022

jenkins test centos-container-update

@guits
Copy link
Collaborator

guits commented Oct 7, 2022

jenkins test centos-container-update

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.

None yet

2 participants