Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-12-07 Thread Rajani Karuturi
> On Jan. 24, 2014, 6:52 a.m., Prachi Damle wrote: > > I see many files unrelated to this fix have been updated and present in the > > patch. > > Please can you update the patch with changes needed for the fix only? > > > > Following are some such files. > > api/src/com/cloud/user/AccountServic

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-12-05 Thread Rohit Yadav
> On Jan. 24, 2014, 6:52 a.m., Prachi Damle wrote: > > I see many files unrelated to this fix have been updated and present in the > > patch. > > Please can you update the patch with changes needed for the fix only? > > > > Following are some such files. > > api/src/com/cloud/user/AccountServic

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-08-13 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17120/ --- (Updated Aug. 13, 2014, 10:31 a.m.) Review request for cloudstack, Abhinandan P

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-02-11 Thread Rajani Karuturi
> On Jan. 24, 2014, 6:52 a.m., Prachi Damle wrote: > > I see many files unrelated to this fix have been updated and present in the > > patch. > > Please can you update the patch with changes needed for the fix only? > > > > Following are some such files. > > api/src/com/cloud/user/AccountServic

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-29 Thread Prachi Damle
> On Jan. 24, 2014, 6:52 a.m., Prachi Damle wrote: > > I see many files unrelated to this fix have been updated and present in the > > patch. > > Please can you update the patch with changes needed for the fix only? > > > > Following are some such files. > > api/src/com/cloud/user/AccountServic

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-28 Thread Rajani Karuturi
> On Jan. 24, 2014, 6:52 a.m., Prachi Damle wrote: > > I see many files unrelated to this fix have been updated and present in the > > patch. > > Please can you update the patch with changes needed for the fix only? > > > > Following are some such files. > > api/src/com/cloud/user/AccountServic

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-28 Thread Prachi Damle
> On Jan. 24, 2014, 6:52 a.m., Prachi Damle wrote: > > I see many files unrelated to this fix have been updated and present in the > > patch. > > Please can you update the patch with changes needed for the fix only? > > > > Following are some such files. > > api/src/com/cloud/user/AccountServic

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-23 Thread Rajani Karuturi
> On Jan. 24, 2014, 6:52 a.m., Prachi Damle wrote: > > I see many files unrelated to this fix have been updated and present in the > > patch. > > Please can you update the patch with changes needed for the fix only? > > > > Following are some such files. > > api/src/com/cloud/user/AccountServic

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-23 Thread Prachi Damle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17120/#review32702 --- I see many files unrelated to this fix have been updated and present

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-21 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17120/ --- (Updated Jan. 22, 2014, 7:24 a.m.) Review request for cloudstack, Abhinandan Pr

Re: Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-21 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17120/ --- (Updated Jan. 22, 2014, 5:05 a.m.) Review request for cloudstack, Abhinandan Pr

Review Request 17120: added an extra column to mark the imported ldap users as from ldap

2014-01-20 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17120/ --- Review request for cloudstack, Abhinandan Prateek and Ian Duffy. Bugs: CLOUDSTA