A question on vm migrations when hosts are set into a maintenance mode.

2013-09-18 Thread Alex Ough
the actual migrations in parallel? Thanks Alex Ough

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-09-23 Thread Alex Ough
7;TRUE'? Any answers/comments will be very appreciated. Thanks Alex Ough On Wed, Sep 18, 2013 at 10:22 AM, Alex Ough wrote: > I checked the vm migration when their host is set to a maintenance mode > and found that even if the orchestration layer fires the each vm migration > at the

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-09-24 Thread Alex Ough
For a resolution without breaking possible flows, I'd like to add the value of 'executeInSequence' to the global setting. Is there any reason not to do this? Thanks Alex Ough On Mon, Sep 23, 2013 at 12:57 PM, Alex Ough wrote: > All, > > After a little more investi

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-09-24 Thread Alex Ough
> > On Tue, Sep 24, 2013 at 9:58 AM, Chip Childers > wrote: > > Hey Kelven - This topic was discussed briefly in the past [1]. Are you > > able to provide any thoughts on Alex's ideas below? > > > > -chip > > > > > > [1] http://markmail.org/messag

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-10-02 Thread Alex Ough
Marcus/Kelven, Any thoughts on my suggestions? Thanks Alex Ough On Tue, Sep 24, 2013 at 12:09 PM, Alex Ough wrote: > Oh, sorry for the confusion. I must have reversed the flags. > As Kelven pointed, it is set as 'TRUE', which makes the process as > sequential. > > So

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-10-02 Thread Alex Ough
Thanks for the reply, Marcus. What about the option #3, which is to make it as a global setting? I think it can prevent side effects if exist. Alex Ough On Wed, Oct 2, 2013 at 11:37 AM, Marcus Sorensen wrote: > Not sure. I don't know the history well enough to know if there were >

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-10-03 Thread Alex Ough
Hi Koushik, Thanks for your reply, but unfortunately, this setting does NOT cover 'MigrateCommand'. As you specified, it seems to be effective only in Start/Stop/CopyCommand. So can we include 'MigrateCommand' in that setting? Thanks Alex Ough On Wed, Oct 2, 2013 at 1

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-10-03 Thread Alex Ough
I'm not sure what else commands 'MigrateCommand' actually execute in addition to 'Start/Stop/CopyCommand', but can we include 'MigrateCommand' if it consists of only those 3 commands? Thanks Alex Ough On Thu, Oct 3, 2013 at 11:31 AM, Alena Prokharchyk &l

Resource Configuration Failure in Simulator mode (v4.2)

2013-10-17 Thread Alex Ough
working ok before getting the latest. "mvn -Pdeveloper,marvin.setup -Dmarvin.config=setup/dev/advanced.cfg -pl :cloud-marvin integration-test" I'm wondering if this is happening only to me or something is missing. Your help will be very appreciated. Thanks Alex Ough

Fwd: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-10-17 Thread Alex Ough
from each management server, updates from each region can be propagated to the rest regions and they can be executed accordingly. I hope you guys have a chance to think about this and give some feedbacks if interested. Thanks in advance. Alex Ough

Re: Resource Configuration Failure in Simulator mode (v4.2)

2013-10-18 Thread Alex Ough
Thanks Amogh, that solved the problem. Btw, don't we need to update the repository to fix this? Thanks Alex Ough On Thu, Oct 17, 2013 at 10:21 PM, Amogh Vasekar wrote: > Can you check the main pom.xml to ensure "tools" is not commented out as a > module from developer pr

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-10-29 Thread Alex Ough
I created a jira for this feature. https://issues.apache.org/jira/browse/CLOUDSTACK-4992 But it doesn't allow for me to assign it to myself, so any permission do I need for this? If so, can anyone give me this permission? If there is anything missing, let me know. Thanks Alex Ough On Fri

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-10-30 Thread Alex Ough
And I'd like to write the design document in the wiki page, but I don't seem to have a permission to create pages. So can anyone give me the permission? My account in the wiki is alex.o...@sungard.com Thanks in advance. Alex Ough On Tue, Oct 29, 2013 at 3:38 PM, Alex Ough wrote: &

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-10-31 Thread Alex Ough
icated, but the only issue is to distinguish the events from either manual jobs or triggered automatic processing, which I'm currently working on. Your comments/recommendations will be very appreciated. Thanks Alex Ough On Wed, Oct 30, 2013 at 12:19 PM, Alex Ough wrote: > And I

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-10-31 Thread Alex Ough
Thanks Chip, and can you also give a permission in Jira so that I can assign myself in its jira? Alex Ough On Thu, Oct 31, 2013 at 2:00 PM, Chip Childers wrote: > Permission added. > > On Wed, Oct 30, 2013 at 12:19:23PM -0500, Alex Ough wrote: > > And I'd like to write th

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-10-31 Thread Alex Ough
Great! Thanks a lot, Daan. On Thu, Oct 31, 2013 at 4:58 PM, Daan Hoogland wrote: > you are added to jira, Alex > > On Thu, Oct 31, 2013 at 8:31 PM, Alex Ough wrote: > > Thanks Chip, and can you also give a permission in Jira so that I can > > assign myself in its ji

Re: haPlanners error in fresh 4.2 checkout

2013-11-04 Thread Alex Ough
I still have this issue Any update on this? Thanks Alex Ough On Fri, Nov 1, 2013 at 4:41 AM, sebgoa wrote: > I deleted my local 4.2 branch and did a brand new checkout, same result. > > latest 4.2 does not work for me -same error as below - > > On Oct 31, 2

Re: haPlanners error in fresh 4.2 checkout

2013-11-04 Thread Alex Ough
I added the property and the server starts ok after being rebuilt. Thank Prachi for your help. Alex Ough On Mon, Nov 4, 2013 at 4:02 PM, Prachi Damle wrote: > I don't see this on 4.2 anymore, so it's hard to debug. But when I saw > this, the 'haPlanners' property w

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-04 Thread Alex Ough
+Regions Please review them and let me know what you think if you're interested! Thanks Alex Ough On Thu, Oct 31, 2013 at 6:51 PM, Alex Ough wrote: > Great! Thanks a lot, Daan. > > > On Thu, Oct 31, 2013 at 4:58 PM, Daan Hoogland wrote: > >> you are added to jira,

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-05 Thread Alex Ough
OK. 1) Do you mean the plug-in? If so, let me find out how to develop a plug-in and work on this. 2) Sure, let me add more information in the document. Thanks for your suggestions. Alex Ough On Tue, Nov 5, 2013 at 11:41 AM, Chip Childers wrote: > Alex, > > I've moved your page t

What is the cloudstack branch that has same codes with CloudPlatform-4.2.0-2-rhel6.4?

2013-11-06 Thread Alex Ough
Is it master or any other branch? Thanks Alex Ough

Re: What is the cloudstack branch that has same codes with CloudPlatform-4.2.0-2-rhel6.4?

2013-11-06 Thread Alex Ough
OK, thanks! On Wed, Nov 6, 2013 at 11:25 AM, Chip Childers wrote: > 4.2 but specifically the 4.2.0 tag > > > On Nov 6, 2013, at 9:18 AM, Alex Ough wrote: > > > > Is it master or any other branch? > > > > Thanks > > Alex Ough > >

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-06 Thread Alex Ough
I'm having a difficulty finding documents about how to develop a plug-in. Anyone to help me find one? Thanks in advance. Alex Ough On Tue, Nov 5, 2013 at 2:18 PM, Alex Ough wrote: > OK. > > 1) Do you mean the plug-in? If so, let me find out how to develop a > plug-in and

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-07 Thread Alex Ough
I don't mind implementing it in the core server unless there is anyone who thinks differently. Anyone with another thought? Thank Daan for your suggestion. Alex Ough On Thu, Nov 7, 2013 at 11:23 AM, Daan Hoogland wrote: > Alex, > > Why would you want to make this a plugin? It

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-11 Thread Alex Ough
sideration of the support as a plug-in in resolving the #1 restriction. Please review them and let me know what you think. Thanks Alex Ough On Sat, Nov 9, 2013 at 8:20 AM, Daan Hoogland wrote: > H Guys, > > Can you shoot at my claims below, please? > > syncing being optional does n

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-11-12 Thread Alex Ough
Thanks a lot for your confirmation, Marcus. I'll create a review request unless anyone has an objection. Thanks Alex Ough On Tue, Nov 12, 2013 at 3:37 PM, Marcus Sorensen wrote: > I have done parallel KVM migrations without issue, it's "supposed to > work". Reall

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-11-12 Thread Alex Ough
queued in reversed orders, which is hopefully rare. And to support consistency in the AP system, I'd like to provide a full sync up, which will sync up all data in all region servers by selecting a region as a master and force its data to other regions. Let me know what you think. Thanks Alex

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-11-13 Thread Alex Ough
/src/com/cloud/vm' from 'server/src/com/cloud/vm'. Am I missing something related with this or do we stop supporting this option in 4.3? I'm a little confused, so please help me resolve this. Thanks Alex Ough On Tue, Nov 12, 2013 at 4:20 PM, Alex Ough wrote: > Thanks a

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-11-13 Thread Alex Ough
but can you check this for me? Thanks Alex Ough On Wed, Nov 13, 2013 at 11:18 AM, Marcus Sorensen wrote: > I'm not sure. I know in the past when I've seen files change locations > it has also clobbered updates to that file. Someone branched, did the > reorganization work, and mer

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-11-14 Thread Alex Ough
Not sure if Alex Huang checked this, but can anyone help to resolve this? Thanks Alex Ough On Wed, Nov 13, 2013 at 11:39 AM, Alex Ough wrote: > It sounds a little scary... > > I looked at the history and found these. > > 8/9/ : file moved to engine by Al

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-11-15 Thread Alex Ough
I hate to sending the same emails over and over again, but I really need to finalize this feature to be included in the next code freeze because this feature is very critical in our inside project. Anyone who can help, please? Thanks Alex Ough On Thu, Nov 14, 2013 at 1:27 PM, Alex Ough wrote

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-11-18 Thread Alex Ough
Thank Parasanna & Sebastien, I also got his email and sent an email. Waiting for his reply... Thanks Alex Ough On Sat, Nov 16, 2013 at 3:05 PM, Sebastien Goasguen wrote: > cc Alex Huang to get his attention: > > > On Nov 15, 2013, at 10:17 PM, Prasanna Santhanam wrote: >

Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-02-06 Thread Alex Ough
randomly generating changes each region. Thanks, Alex Ough

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2014-02-06 Thread Alex Ough
All, I just sent a review request, so please take a look at it and let me know if you have any comments/suggests. https://reviews.apache.org/r/17790/ Thanks Alex Ough On Mon, Jan 13, 2014 at 11:17 AM, Alex Ough wrote: > All, > > I'd like to have some suggestion about 2 thing

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2014-02-06 Thread Alex Ough
fects. Thanks Alex Ough On Thu, Feb 6, 2014 at 1:35 PM, Chiradeep Vittal < chiradeep.vit...@citrix.com> wrote: > I am uncomfortable with changes to GenericDaoBase. Was this really > necessary? This feature was supposed to be "outside" CloudStack as much as > possible and opt

Re: [ACS 4.3] RC update

2014-02-10 Thread Alex Ough
All, I'm not sure what the current status for the issue of "Could not find corresponding resource manager for simulator" during deploying infra in simulator mode. It looks like I'm still getting this error in 4.3. Your help will be very appreciated. Thanks Alex Ough On Th

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-02-10 Thread Alex Ough
uring real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-02-13 Thread Alex Ough
Hi Daan, Does it look good to you? Thanks Alex Ough On Mon, Feb 10, 2014 at 1:07 PM, Alex Ough wrote: >This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17790/ > Review request for cloudstack. > By Alex Ough. > > *Updated Feb.

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-02-13 Thread Alex Ough
We're just about to add unit tests, so I'll add them once completed. Thanks Alex Ough On Thu, Feb 13, 2014 at 3:12 PM, Daan Hoogland wrote: > h Alex, > > looks alright (i did mean with newlines but I don't know if this is > actually caught by our checkstyle run)

Re: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2014-02-19 Thread Alex Ough
g the real time synchronization. Let me know if this is not clear. Thanks Alex Ough On Wed, Feb 19, 2014 at 1:47 PM, David Grizzanti < david.grizza...@sungard.com> wrote: > Hi Alex, > > One thing I wanted to ask about/mention on this was regarding the > restriction you have mention

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-02-27 Thread Alex Ough
Hi Daan, We started to work on implementing unit tests and checked in some of them, so please review them and let us know if there is any comments/suggestions. https://github.com/alexoughsg/Albatross/commit/fca94db36054ee35f5da85c69c2471430e0eebfe Thanks Alex Ough On Thu, Feb 13, 2014 at 3:15

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-02-28 Thread Alex Ough
t will automatically create/update/remove domains/accounts/users with that interval, which will trigger the scheduled full scan to sync the newly changed resources. Let me know if you need more info. Thanks Alex Ough On Thu, Feb 27, 2014 at 3:50 PM, Daan Hoogland wrote: > I see, great. >

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-01 Thread Alex Ough
Can you tell me when the deadline for 4.4 is? Thanks Alex Ough On Sat, Mar 1, 2014 at 2:26 PM, Daan Hoogland wrote: > Ha Alex, > > branch 4.3 is going to be a problem. We will be able to release this > with 4.4 if we hurry. > > On Fri, Feb 28, 2014 at 4:01 PM, Alex Ough

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-04 Thread Alex Ough
Do you think it is ok to wrap up the implementation until the end of this week to make this included in 4.4 release? Thanks Alex Ough On Sun, Mar 2, 2014 at 2:10 PM, Daan Hoogland wrote: > two weeks from now, but under debate. Let's go with that, though. > > On Sun, Mar 2, 2

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-04 Thread Alex Ough
Yes, by 3/7. Thanks Alex Ough On Tue, Mar 4, 2014 at 12:30 PM, Daan Hoogland wrote: > This week? should be ok. end of next week is too late! > > On Tue, Mar 4, 2014 at 6:17 PM, Alex Ough wrote: > > Do you think it is ok to wrap up the implementation until the end of this > &

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-05 Thread Alex Ough
What branch needs to be based to create a review request of this implementation that will be included in 4.4 release? 4.3 or 4.3-forward? Thanks Alex Ough On Tue, Mar 4, 2014 at 1:25 PM, Alex Ough wrote: > Yes, by 3/7. > > Thanks > Alex Ough > > > On Tue, Mar 4, 2

Unit test failing in master

2014-03-06 Thread Alex Ough
All, Anyone who can fix this failure in "cloudstack/plugins/alert-handlers/snmp-alerts"? Failed tests: appendTest(org.apache.cloudstack.alert.snmp.SnmpTrapAppenderTest): error snmpHelper list size not as expected expected:<0> but was:<2> Thanks Alex Ough

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-08 Thread Alex Ough
scans to synchronize resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-08 Thread Alex Ough
resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-09 Thread Alex Ough
uring real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-10 Thread Alex Ough
Please see my reply/question in blue. Thanks Alex Ough On Mon, Mar 10, 2014 at 1:25 PM, Chiradeep Vittal < chiradeep.vit...@citrix.com> wrote: > I haven¹t looked at it because it is huge. But preliminary scan: > > - there are unit tests missing for changes to core CS code >

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-11 Thread Alex Ough
I worked on a couple of items, so can you give me the confirmation for the rest items so that I can wrap this up? I really want to include this into 4.4. Thanks Alex Ough On Mon, Mar 10, 2014 at 3:41 PM, Alex Ough wrote: > Please see my reply/question in blue. > > Thanks > Alex O

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-11 Thread Alex Ough
Hi Daan, I didn't update the patch after the couple of works because I wanted to do it with others Chiradeep asked if any. Let me know when you want me to. Thanks Alex Ough On Tue, Mar 11, 2014 at 4:37 PM, Daan Hoogland wrote: > I will call the merge in a moment, but won't do i

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-12 Thread Alex Ough
hope I can get the reply soon. Thanks Alex Ough On Wed, Mar 12, 2014 at 4:13 AM, Daan Hoogland wrote: > H Alex, I considder Chiradeeps comments quite valid and serious enough > to anticipate not making friday 14:00 CET. That would mean no merge > before 4.4. Can you live with that? &

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-12 Thread Alex Ough
synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-12 Thread Alex Ough
I uploaded the patch A that includes only core changes, so please review it and let me know if you have any comments. Thanks Alex Ough On Wed, Mar 12, 2014 at 8:24 AM, Alex Ough wrote: > Hi Chiradeep, > > Can you give me the example of the Singleton service class you mentioned? >

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-12 Thread Alex Ough
resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-12 Thread Alex Ough
And I also uploaded the patch B that includes new implementation to support multi regions. Thanks Alex Ough On Wed, Mar 12, 2014 at 10:17 AM, Alex Ough wrote: > I uploaded the patch A that includes only core changes, so please review > it and let me know if you have any comments. >

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
Chiradeep, Any comments on them? Thanks Alex Ough On Wed, Mar 12, 2014 at 10:58 AM, Alex Ough wrote: > And I also uploaded the patch B that includes new implementation to > support multi regions. > > Thanks > Alex Ough > > > On Wed, Mar 12, 2014 at 10:17 AM, Alex Ough

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
of requests to the remote regions because the syncing is always using the api requests a lot to get information of domains/accounts/users from remote regions. And let me change the concatenation into using StringBuilder. Thanks a lot for your reply. Alex Ough On Thu, Mar 13, 2014 at 2:41 PM, Alena Pro

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
essor - RemoteUserEventProcessor, RemoteAccountEventProcessor, RemoteDomainEventProcessor => Do you recommend to refactor them into defining interfaces and creating one class implementing all methods related with each user, account and domain? 3. Let me work on changing names. Thanks Alex Ough On Thu, Mar

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
emoteEventProcessor - RemoteUserEventProcessor, RemoteAccountEventProcessor, RemoteDomainEventProcessor => no change Thanks Alex Ough On Thu, Mar 13, 2014 at 3:58 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > You extract stuff into interfaces when the methods are mean

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-13 Thread Alex Ough
along with username and password. Please see 'RegionVO' under 'engine/schema/src/org/apache/cloudstack/region'. Thanks Alex Ough On Thu, Mar 13, 2014 at 5:49 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > Alex, > > There are so many classes, a

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-16 Thread Alex Ough
Hi Alena, It looks like this was not included in version 4.4. Anyway, do you have any comments on my reply? Thanks Alex Ough On Thu, Mar 13, 2014 at 9:48 PM, Alex Ough wrote: > What I think based on your comments are > > 1. Well, I have a different thought. I'd rather have sep

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-03-26 Thread Alex Ough
nize resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-26 Thread Alex Ough
r objects separately. 4. Done 5. Same 6. I removed 'domain path' from AccountResponse & UserResponse. Thanks Alex Ough On Thu, Mar 13, 2014 at 9:48 PM, Alex Ough wrote: > What I think based on your comments are > > 1. Well, I have a different thought. I'd rather have

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-26 Thread Alex Ough
Sure, I'm looking forward to it. Thanks Alex Ough On Wed, Mar 26, 2014 at 9:39 AM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > Alex, I'm attending a conference today, will review the patch tomorrow. > > -Alena > > From: Alex Ough > Date: We

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-26 Thread Alex Ough
Sorry, my bad. I mis-read your comment. Thanks for pointing it out. Alex Ough On Wed, Mar 26, 2014 at 12:25 PM, Chiradeep Vittal < chiradeep.vit...@citrix.com> wrote: > I didn't say "do not use auto wiring". I said the convention is to use > @Inject which you did

How to use @Inject

2014-03-31 Thread Alex Ough
> Can anyone show me what needs to be fixed? Thanks Alex Ough

Re: How to use @Inject

2014-03-31 Thread Alex Ough
It seems like I wrote a wrong type in ComponentContext. getComponent(RmapDao.class). It should be ComponentContext.getComponent(DomainDao.class) Thanks Alex Ough On Mon, Mar 31, 2014 at 5:53 PM, Alex Ough wrote: > All, > > I'm trying to change "ComponentContext.getComponent

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-04-03 Thread Alex Ough
es. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-04-03 Thread Alex Ough
done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-04-03 Thread Alex Ough
All, I updated the patches as per Alena's request. Let me know if there is anything missing/incorrect. Thanks Alex Ough On Wed, Mar 26, 2014 at 1:40 PM, Alex Ough wrote: > Sorry, my bad. I mis-read your comment. > > Thanks for pointing it out. > Alex Ough > > > On

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-04-03 Thread Alex Ough
Forgot to mention this, but it was a great help from Darren. Thanks again, Darren! Alex Ough On Thu, Apr 3, 2014 at 11:56 AM, Alex Ough wrote: > All, > > I updated the patches as per Alena's request. > > Let me know if there is anything missing/incorrect. > Thanks > A

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-04-03 Thread Alex Ough
en's help from. I think this info would be useful for all CS > committers. > > Thank you, > Alena. > > From: Alex Ough > Date: Thursday, April 3, 2014 at 9:22 AM > To: Chiradeep Vittal , Alena Prokharchyk < > alena.prokharc...@citrix.com>, Darren Shepher

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-04-05 Thread Alex Ough
connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 17790: Domain-Account-User Sync Up Among Multiple Regions

2014-04-05 Thread Alex Ough
synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-07 Thread Alex Ough
. Successfully tested full scans to synchronize resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-07 Thread Alex Ough
automatically by randomly generating changes each region. Thanks, Alex Ough

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

2014-04-07 Thread Alex Ough
. Successfully tested full scans to synchronize resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

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

2014-04-07 Thread Alex Ough
synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

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

2014-04-07 Thread Alex Ough
automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-07 Thread Alex Ough
e confirm, > Alena. > > On 4/7/14, 12:13 PM, "Alex Ough" wrote: > > > > >--- > >This is an automatically generated e-mail. To reply, visit: > >https://reviews.apache.org/r/20099/ > >-

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-07 Thread Alex Ough
bout the times, but there is no way to guarantee #1 not to be failed, so #2 is very important to be supported. Please let me know what you think and we can have a conference call if you want. Thanks Alex Ough On Mon, Apr 7, 2014

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-07 Thread Alex Ough
And all 4 of the recommendations Alex Hwang gave were already implemented to support the real time synchronization (#1), but like I said in the previous email, we need to support the full scan (#2) to cover any failures during the synchronization. Thanks Alex Ough On Tue, Apr 8, 2014 at 12:10

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-08 Thread Alex Ough
. Let me try to resolve those 2 and let you know once done, so please let me know now if there is anything else needed. Thanks Alex Ough On Tue, Apr 8, 2014 at 1:02 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > Alex, thank you for the explanation. But I think there

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-08 Thread Alex Ough
iginal datetime. Let me think about the #1 more closely and let you know what I find. So please let me know if you have any other suggestion related with this. Thanks Alex Ough On Tue, Apr 8, 2014 at 2:09 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > Sorry, Alex, for n

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-08 Thread Alex Ough
r dao interfaces, and the values are assigned only during the full scan to preserve the original value when they are missed. Thanks Alex Ough On Tue, Apr 8, 2014 at 2:29 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > > > On 4/8/14, 11:26 AM, "Alex Ough" wro

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-08 Thread Alex Ough
g/updating the resources in their manager methods to guarantee the atomicity? Thanks Alex Ough On Tue, Apr 8, 2014 at 2:45 PM, Alena Prokharchyk < alena.prokharc...@citrix.com> wrote: > Alex, > > Take for example the Removed field. Its not being managed through none of > the CS i

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-16 Thread Alex Ough
like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

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

2014-04-16 Thread Alex Ough
. Thanks, Alex Ough

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-04-16 Thread Alex Ough
Hi Alena, It was a nice to actually meet you during the conference and I updated the review requests, so please review them and let me know if there is anything not expected. Thanks Alex Ough On Thu, Apr 3, 2014 at 1:01 PM, Alex Ough wrote: > Well, I'm not sure about that because the

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-28 Thread Alex Ough
x ------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20099/#review40690 --- On April 16, 2014, 7:06 p.m., Alex Ough wrote: > > -

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-28 Thread Alex Ough
; make, how CS apis would or wouldn¹t get affected - before you do the > coding? It would definitely save us time, and won¹t lead to further > frustration. I¹m more than willing to help. > > On 4/28/14, 6:59 AM, "Alex Ough" wrote: > > > > > > >> On Apri

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-28 Thread Alex Ough
gt; -alena. > > On 4/28/14, 10:35 AM, "Alex Ough" wrote: > > >when & how do you want to have it? > > > > > >On Mon, Apr 28, 2014 at 1:18 PM, Alena Prokharchyk < > >alena.prokharc...@citrix.com> wrote: > > > >> Alex, pleas

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-04-28 Thread Alex Ough
re follow up > notes with the community. The best approach would be - to update the FS > with the design changes for this particular part of the code. > > Will be waiting for the webex invitation from you. > > -Alena. > > On 4/28/14, 11:23 AM, "Alex Ough" wrote: >

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-04 Thread Alex Ough
resources that were missed during real time synchronization because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

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

2014-05-04 Thread Alex Ough
because of any reasons like network connection issues. 3. The tests were done manually and also automatically by randomly generating changes each region. Thanks, Alex Ough

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alex Ough
at your code doesn't actually need. It > > would make the changes list much shorter and easier for review. for 3) AccountManager * updateUser in 'state' : it is necessary during the Full Sync when an object's information is not same with one in other regions. * new s

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-05-05 Thread Alex Ough
count, String newAccountName, String > > newNetworkDomain, final Map details, Account.State state); > > > > As updateAccount doesn't allow account state update, please remove > > Account.State from the method signature > > > > > > 4) ApiResponseHelper.jav

  1   2   >