Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2015-04-06 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/#review78992 --- Thank you for submitting your CloudStack contribution through

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-12-05 Thread Rohit Yadav
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/#review64047 --- ping. any update on this? - Rohit Yadav On June 27, 2014, 5:04

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-12-05 Thread Alena Prokharchyk
...@apache.org Subject: Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin) --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/#review64047

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-12-05 Thread Rohit Yadav
To: cloudstack dev@cloudstack.apache.org, Alex Ough alex.o...@sungard.com, Rohit Yadav bhais...@apache.org Subject: Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-12-05 Thread Alena Prokharchyk
@cloudstack.apache.orgmailto:dev@cloudstack.apache.org, Alex Ough alex.o...@sungard.commailto:alex.o...@sungard.com, John Burwell jburw...@basho.commailto:jburw...@basho.com Subject: Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin) On Sat, Dec 6, 2014 at 3:39 AM, Alena Prokharchyk

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-12-05 Thread John Burwell
...@apache.org Date: Friday, December 5, 2014 at 10:41 AM To: cloudstack dev@cloudstack.apache.org, Alex Ough alex.o...@sungard.com, Rohit Yadav bhais...@apache.org Subject: Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-06-24 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated June 24, 2014, 3:52 p.m.) Review request for cloudstack. Changes

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-06-20 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated June 20, 2014, 3:49 p.m.) Review request for cloudstack. Changes

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-06-15 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated June 15, 2014, 9:48 p.m.) Review request for cloudstack. Changes

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-15 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/#review42512 --- Looks good to me, Alex. Lets wait for the resolution on

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-13 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated May 7, 2014, 10:46 p.m.) Review request for cloudstack. Repository:

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-06 Thread Alex Ough
On May 5, 2014, 6:28 p.m., Alena Prokharchyk wrote: Alex, Move all new API commands/responses you've added (ListSyncAccountsCmdByAdmin.java, SyncAccountResponse.java), to your plugin's multiregion folder. Look how its done for other CS plugins - internallb for example I checked

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-06 Thread Alena Prokharchyk
On May 5, 2014, 6:28 p.m., Alena Prokharchyk wrote: Alex, Move all new API commands/responses you've added (ListSyncAccountsCmdByAdmin.java, SyncAccountResponse.java), to your plugin's multiregion folder. Look how its done for other CS plugins - internallb for example Alex

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-05-04 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated May 4, 2014, 9:18 p.m.) Review request for cloudstack. Repository:

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-04-16 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated April 16, 2014, 7:07 p.m.) Review request for cloudstack. Changes

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-04-07 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated April 7, 2014, 7:14 p.m.) Review request for cloudstack. Changes

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions (New Plugin)

2014-04-07 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/ --- (Updated April 7, 2014, 7:17 p.m.) Review request for cloudstack. Changes