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

Add manageiq user to allowed_uids for sssd #220

Merged

Conversation

jrafanie
Copy link
Member

When we moved to using a manageiq user, we need to add this user so it has permission in sssd.conf.

See also ManageIQ/manageiq-documentation#1743

When we moved to using a manageiq user, we need to add this user so it has permission in sssd.conf.

See also ManageIQ/manageiq-documentation#1743
@miq-bot
Copy link
Member

miq-bot commented Sep 22, 2023

Checked commit jrafanie@f219254 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. 👍

@Fryguy
Copy link
Member

Fryguy commented Sep 22, 2023

I think there should also be a corresponding change in the ManageIQ operator, since we have these configs there as well.

jrafanie added a commit to jrafanie/manageiq that referenced this pull request Sep 25, 2023
jrafanie added a commit to jrafanie/manageiq that referenced this pull request Sep 25, 2023
When we moved to using a manageiq user, we need to add this user so it has permission in sssd.conf.

See also:
ManageIQ/manageiq-documentation#1743
ManageIQ/manageiq-appliance_console#220
jrafanie added a commit to jrafanie/httpd_configmap_generator that referenced this pull request Sep 25, 2023
When we moved to using a manageiq user, we need to add this user so it has permission in sssd.conf.

See also:
ManageIQ/manageiq#22715
ManageIQ/manageiq-documentation#1743
ManageIQ/manageiq-appliance_console#220
jrafanie added a commit to jrafanie/guides that referenced this pull request Sep 25, 2023
@jrafanie
Copy link
Member Author

I think there should also be a corresponding change in the ManageIQ operator, since we have these configs there as well.

I think that's here: ManageIQ/httpd_configmap_generator#77. Right?

Do you know how it's pulled in? I assume we'll need a new tag in httpd_configmap_generator and pull that in.

@jrafanie
Copy link
Member Author

I think that's here: ManageIQ/httpd_configmap_generator#77. Right?

Do you know how it's pulled in? I assume we'll need a new tag in httpd_configmap_generator and pull that in.

bump... @bdunne maybe you know? Do we need to do something to pull in the new configmap_generator? If we want to backport this, do we need a new version there and here?

@Fryguy Fryguy merged commit 0b5f92d into ManageIQ:master Sep 29, 2023
4 checks passed
@Fryguy
Copy link
Member

Fryguy commented Sep 29, 2023

I think I was thinking of https://github.com/ManageIQ/manageiq-pods/blob/master/manageiq-operator/api/v1alpha1/helpers/miq-components/httpd_conf.go ? The httpd_configmap_generator change I think covers what I was thinking.

@jrafanie jrafanie deleted the add_manageiq_user_to_sssd_allowed_uids branch January 31, 2024 16:39
Fryguy added a commit that referenced this pull request Feb 7, 2024
Fixed
- Fix sporadic test failure [#204]
- Remove MIQ specific gem source [#209]
- Double escape @ in realm to avoid shell interpretation [#211]
- Move gem name loader to proper namespaced location [#208]
- Separate kerberos from service principal name and use correctly [#215]
- Add manageiq user to allowed_uids for sssd [#220]
- Remove warning about using pg_dump [#221]
- Fix specs where AwesomeSpawn private interface changed [#224]
- Change the Name of the CA from something to ApplianceCA [#228]
- Fix YAML.load_file failing on aliases [#234]

Added
- Make backward compatible changes to work with repmgr13 - version 5.2.1 [#192]
- Support Ruby 3.0 [#206]
- Support Ruby 3.1 [#227]
- Allow rails 7 gems in gemspec [#226]

Changed
- Update to Highline 2.1.0 [#201]
- Clean up test output (highline and stdout messages) [#210]

Removed
- Drop Ruby 2.7 [#223]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants