remove openldap from TW container

dmwm / CRABServer

15 stars 37 forks source link

remove openldap from TW container #8439

Open belforte opened 1 month ago

belforte commented 1 month ago

we now use CRIC to reselve e-groups content. No need for openldap and its dependencies.

belforte commented 2 weeks ago

hmm... looks like code still uses LDAP both in TW and scheduler. Also for high priority users @novicecpp do you remember why we did not change ? there are 2 use cases:

group --> lit of users (for high prio)
user --> groups they belong to (in several places, I still have to understand exactly what for)

novicecpp commented 2 weeks ago

What do you mean? This one https://github.com/dmwm/CRABServer/issues/7560#issuecomment-1923531953 ? We want to remove the LDAP deps but keep high prio system, no?

I forgot about the schedd part and I do not know about the second bullet. I can have a look later.

novicecpp commented 2 weeks ago

FYI, this task is stuck because I want to use egroup name from highPrioEgroups config to get the group object id in CRIC and then list all user there later. But it look like CRIC does not have such API.

belforte commented 2 weeks ago

I see. OK, this is consistent with what I saw. I am not sure I understand your question though. Maybe we have too many names for the same thing ? Leave the second bullet to me, I want to find out what the purpose is.