library/rados_user.py modify

[karasjoh000]

https://github.com/ceph/ceph-ansible/blob/b40e4bfe60cb14a8eac225086f60d5b170636b6d/library/radosgw_user.py#L268-L269

If display_name != name and display_name is not provided as a param, then display_name is changed to name although the expectation will be that display_name should not change.

Note: I did not test this, but it seems that L268 and L269 need to be removed.

[asm0deuz]

display_name is a mandatory parameter to radosgw-admin command. If you do not set the display_name, it fails creating the user with could not create user: unable to create user, no display name specified. This is probably why L268 and L269 are there for but indeed if the user already exists, its display_name will be overwritten with name.

Seems to be an easy fix, I will take a look at it. Thx

[github-actions[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[karasjoh000]

Ill create a PR for this

[github-actions[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[guits]

@karasjoh000 what is the status on this? should we take over?

[guits]

@asm0deuz @karasjoh000 🙂

[karasjoh000]

This should be a 2 line fix I think.

[github-actions[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[github-actions[bot]]

This issue has been automatically closed due to inactivity. Please re-open if this still requires investigation.