On Fri, 27 Sep 2013, Sumit Bose wrote:
On Mon, Sep 23, 2013 at 06:04:22PM +0300, Alexander Bokovoy wrote:
On Mon, 23 Sep 2013, Alexander Bokovoy wrote:
>On Mon, 23 Sep 2013, Alexander Bokovoy wrote:
>>On Mon, 23 Sep 2013, Martin Kosek wrote:
>>>>>However, we don't have trust type available so it needs to discovered
>>>>>every time. This doesn't play well with the framework, it is simply not
>>>>>expecting dynamic containers.
>>>>This doesn't sound like a big obstacle to me. Right now the trust_type 
>>>>is done in trust_show.execute() for some reason, which is not the best 
place to
>>>>do it IMHO. Doing it in trust.get_dn() instead should simplify things 
enough to
>>>>make parent_object work.
>>>Yup, get_dn() is the method where object DN lookup should be done. See for
>>>example host.py plugin get_dn method, we also do a dynamic lookup for correct
>>>host name.
>>I'll see if that would work.
>>>the best way to implement dynamic DN gathering is the get_dn() method. That
>>>way, it could be implemented in one place and all commands could take 
>>>of it instead of re-implementing it several times in pre_callback - this is
>>>just hackish.
>>I'd suggest you look into the code. The commands use pre_callback for a
>>different purpose than implementing dynamic DN gathering.
>>>I think it would have been very useful to have a design page before sending a
>>>patch. It is then easier to make design decisions without having to dig into
>>>the patch.
>>The design page is there for long time:
>Ok, here is new version of the patch and updated version of my 0117
>patch as Sumit noticed I've sent wrong version.
Ok, here is updated 0118 which fixes API.txt change for trustdomain_add
-- I renamed trustdomain_create to trustdomain_add but forgot to rerun

I haven't see any issue during build or using the latest version of 0117
(please note the latest version of 0117 is 2 emails earlier in this
thread) and 0118. So it is an ACK from me for the functional part. But
since there were some discussion about the usage of the internal Python
API it would be nice if someone can check the python side as well.
I've rewrote the patches to follow what we discussed. New patchset is
sent separately, includes patches 0117-0122.

/ Alexander Bokovoy

Freeipa-devel mailing list

Reply via email to