[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2013-01-02 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Delete DbUserBase .. Patch Set 2: Verified; Looks good to me, approved Rebased, no changes since acked -- To view, visit http://gerrit.ovirt.org/10505 To

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2013-01-02 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Delete DbUserBase .. core: Delete DbUserBase This class is empty, and no one is using it, so delete it and have sub-classes extend IVdcQueryable d

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2012-12-31 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: core: Delete DbUserBase .. Patch Set 1: to answer your question, DbUser is used as a wrapper for groups sent to the ui by the searchbackend, I guess that this w

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2012-12-31 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: core: Delete DbUserBase .. Patch Set 1: Yair, there is no references to the class anywhere -- To view, visit http://gerrit.ovirt.org/10505 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2012-12-31 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: core: Delete DbUserBase .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10505 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2012-12-31 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: Delete DbUserBase .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10505 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2012-12-31 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Delete DbUserBase .. Patch Set 1: You should also try to add several users and groups to your setup and see them get presented when you handle permission

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2012-12-31 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Delete DbUserBase .. Patch Set 1: Did you verify with UI that if you search for let's say C* (in users & groups) you get both user and groups? -- To vie

[Engine-patches] Change in ovirt-engine[master]: core: Delete DbUserBase

2012-12-31 Thread mkolesni
Mike Kolesnik has uploaded a new change for review. Change subject: core: Delete DbUserBase .. core: Delete DbUserBase This class is empty, and no one is using it, so delete it and have sub-classes extend IVdcQueryable directly