[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 11: Build Failed http://jenkins.ovirt.org/job/ovirt-engine_master_dao-unit-tests_merged/9708/ : SUCCESS ht

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-19 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 11: * Update tracker::#1092744::OK * Set MODIFIED::bug 1092744#1092744IGNORE, not all related patches are

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-19 Thread oourfali
Oved Ourfali has submitted this change and it was merged. Change subject: aaa: remove user/group sync .. aaa: remove user/group sync Since all queries are performed using the engine sessions table, the users groups are always d

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-19 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/35893 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-19 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 11: Build Successful http://jenkins.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit/22609/ : SUCCESS ht

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-19 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 11: Code-Review+1 Verified+1 rebase (reorder) on top of master, verified, can be merged without waiting for mla changes.

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-19 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 11: * Update tracker::#1092744::OK * Check Bug-Url::OK * Check Public Bug::#1092744::OK, public bug * Check Produ

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-17 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 10: Code-Review-1 Verified-1 Build Failed http://jenkins.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit/

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-17 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 10: * Update tracker::#1092744::OK * Check Bug-Url::OK * Check Public Bug::#1092744::OK, public bug * Check Produ

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-11 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 9: -Code-Review -Verified Build Successful http://jenkins.ovirt.org/job/ovirt-engine_master_create-rpms-quic

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-11 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 9: * Update tracker::#1092744::OK * Check Bug-Url::OK * Check Public Bug::#1092744::OK, public bug * Check Produc

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-10 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 8: Code-Review-1 Verified-1 Build Unstable http://jenkins.ovirt.org/job/ovirt-engine_master_unit-tests_gerri

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-10 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 7: Build Successful http://jenkins.ovirt.org/job/ovirt-engine_master_unit-tests_gerrit/22554/ : SUCCESS ht

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-10 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 8: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/35893 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-10 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 8: * Update tracker::#1092744::OK * Check Bug-Url::OK * Check Public Bug::#1092744::OK, public bug * Check Produc

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-10 Thread rnori
Ravi Nori has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 7: @Alon I will handle in subsequent patch -- To view, visit http://gerrit.ovirt.org/35893 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-10 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 7: shouldn't we remove the groups column from the users? so actually this patch should contain the restapi query the Get

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-10 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 7: * Update tracker::#1092744::OK * Check Bug-Url::OK * Check Public Bug::#1092744::OK, public bug * Check Produc

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-07 Thread emesika
Eli Mesika has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/35893 To unsubscribe, visit http://gerrit.ovirt.org/settings G

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 6: it was possible before as well... especially after 3.4 aaa cleanups. we had no time to focus in that... so I left it

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 6: Well, the code looks OK, butsorry for still insisting - as the flat_groups was already touched - I think now is the

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 6: I am pretty sure I have asked in the past why do we still keep the periodic sync. What have changed since the sessi

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 6: Code-Review+1 ok, great, now we can remove the useless fields within users and groups, I hope. -- To view, visit htt

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 6: Build Successful http://jenkins.ovirt.org/job/ovirt-engine_master_unit-tests_gerrit/22270/ : SUCCESS ht

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 6: * Update tracker::#1092744::OK * Check Bug-Url::OK * Check Public Bug::#1092744::OK, public bug * Check Produc

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: > is this the behavior we want? yes, of course, the current behavior is a limitation. you want to see all groups ava

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-05 Thread rnori
Ravi Nori has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: (2 comments) @Alon After the change to GetAuthzGroupsByUserIdQuery when I select alonbl in UI Users tab and choose "Di

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-04 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: Build Successful http://jenkins.ovirt.org/job/ovirt-engine_master_unit-tests_gerrit/22248/ : SUCCESS ht

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-04 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: TODO: remove groups column of user. -- To view, visit http://gerrit.ovirt.org/35893 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-04 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: it is working for me, great work! -- To view, visit http://gerrit.ovirt.org/35893 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-04 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: (1 comment) http://gerrit.ovirt.org/#/c/35893/5/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/a

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-04 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: (3 comments) http://gerrit.ovirt.org/#/c/35893/5/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/

[Engine-patches] Change in ovirt-engine[master]: aaa: remove user/group sync

2014-12-04 Thread automation
automat...@ovirt.org has posted comments on this change. Change subject: aaa: remove user/group sync .. Patch Set 5: * Update tracker::#1092744::OK * Check Bug-Url::OK * Check Public Bug::#1092744::OK, public bug * Check Produc