Alon Bar-Lev has posted comments on this change. Change subject: aaa: add namespace for directory entities ......................................................................
Patch Set 7: I fixed the search query, as it should work with profiles not directory names. but this is still not working... I do not know why we need to fetch the user again when we do AddUser/AddGroup.... as we do have all the data... we can just put it in database. But if we need to fetch again, we need to have the id and the namespace. I've put some non-compiling changes... please modify as you see fit. I prefer not to fetch anything, just add to database. But if you think we should fetch, please make sure the name space is transmitted with user and back to the AddXXXX commands. Thanks! -- To view, visit http://gerrit.ovirt.org/28217 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I99b5611d4a05a18b271bde61a9d5be14007fd9a4 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
