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

initialAdminGroupName should honor being changed from nil to a value #842

Open
Fryguy opened this issue Jun 28, 2022 · 1 comment
Open

Comments

@Fryguy
Copy link
Member

Fryguy commented Jun 28, 2022

initialAdminGroupName isn't required on installation, so it can be created with an empty value. However, we don't reconcile that field again. It is understandable to not reconcile when created with a value X and changing it to Y because we don't know what to do with X in that case. However, in the case of it being initially empty, it seems reasonable to support someone adding it at later point in time (for example, if they forgot on initial creation of the CR).

Alternatively, maybe we should require an initialAdminGroupName, or perhaps default it to something like EvmGroup-super-administrator?

@miq-bot add-label enhancement

@miq-bot
Copy link
Member

miq-bot commented Feb 27, 2023

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants