manageiq
manageiq copied to clipboard
[WIP] Remove MiqLdap
WIP because I'm still finding tentacles, plus I still want to run cross-repo
Fixes #20017
cc @jvlcek @chessbyte @bdunne
Not sure what to do with these:
https://github.com/ManageIQ/manageiq/blob/0590341e394117fd2a2de706750ce56eb484fa38/tools/miq_config_sssd_ldap/configure_database.rb#L35-L40
Also still trying to figure out if any settings can be dropped.
Yay. I remember starting this removal 6 years ago with GM. so happy it is down to the last remains
I do seem to remember that these tests would only run on lucy's computer. So we had them mostly stubbed out except on an individual's computer that had ldap configured.
@jvlcek do you remember if ldap specs currently run in ci, or if they only run locally on a machine with ldap configured?
fwiw/ the failure is related to tools/miqldap_to_sssd/cli_convert.rb
which was just deleted.
I'm not sure if this tool is used for miq ldap, or used to help us configure external ldap
Is this PR going to also touch the ui? https://github.com/ManageIQ/manageiq-ui-classic/pull/7230#issuecomment-665048014
The initial intent of this tool was to convert configurations from MiqLdap to external auth with SSSD for LDAP. However it can also be used to do initial configurations of external auth with SSSD for LDAP.
For example, the below could be run from the command line on a newly deployed appliance:
miq_config_sssd --domain="example.com" --bind-dn="cn=Manager,dc=example,dc=com" --bind-pwd="secret" --mode="ldap" --ldaphost="my-openldap.example.redhat.com" --ldap-role --user-type="dn-cn" --user-suffix="ou=people,ou=prod,dc=example,dc=com"
This could be integrated into the UI or the httpd_configmap_generator
The initial intent of this tool was to convert configurations from
MiqLdap
[...] However it can also be used to do initial configurations of external auth
Seems like we want to keep that tool.
Seems like we want to keep that tool.
Or move it inside appliance_console.
This pull request has been automatically closed because it has not been updated for at least 3 months.
Feel free to reopen this pull request if these changes are still valid.
Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.
I need to bring this back to life
This pull request has been automatically closed because it has not been updated for at least 3 months.
Feel free to reopen this pull request if these changes are still valid.
Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.
This pull request has been automatically closed because it has not been updated for at least 3 months.
Feel free to reopen this pull request if these changes are still valid.
Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.
This pull request has been automatically marked as stale because it has not been updated for at least 3 months.
If these changes are still valid, please remove the stale
label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).