Hi,
ssia...
I was working on implementing
https://fedorahosted.org/sssd/ticket/2637
and while for most occurences it was quite easy to set domain status
instead of back end status, I hit the sdap_id_op module..
The sdap_id_op operation sets the offline status directly, in a bit of
a hackish way by accessing the be_ctx through sdap_id_ctx. It's not
trivial to change the request to operate on domains, except changing
sdap_id_op_create() to also accept domain as a parameter..but we already
have 37 calls to sdap_id_op_create().
A better way would be to change the whole sdap_id_op request to return an
error code and don't touch the backend status, but that's even more work
-- we need to do that and it's something we need to do in the next weeks
anyway, I just think it's not something that should land in 1.13..
Thoughts? Should I go ahead and change sdap_id_op_create() to accept
domain in 1.13 or do refactor the sdap_id_op for 1.14? I would
personally prefer the latter..
btw my work in progress is here:
https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/log/?h=subdomfo