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

2014-06-27 Thread Alex Ough
Kishan, 1. Why Long instead of Integer : You replied that it should be Integer 2. @Encrypt : Does it both encrypt decrypt? Is there anything necessary to make it work because it doesn't seem to work when I trace the persist. Thanks Alex Ough On Fri, Jun 27, 2014 at 7:39 AM, Kishan Kavala

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

2014-06-27 Thread Alex Ough
, so if the order is reversed with some reason, the creations will fail, but they will be covered by FullScan. 6. It sounds like not related with this project. 7. The interval for FullScan processing is configurable in the global setting, 'region.full.scan.interval'. Thanks Alex Ough On Fri, Jun

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

2014-06-27 Thread Alex Ough
Hi John, Thanks a lot for your detailed feedback, but I strongly suggest to continue to discuss about them after this development is wrapped up because this feature is provided as a plugin and and will not work unless you turn it on in case you don't want to use. Thanks Alex Ough On Fri, Jun

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

2014-06-27 Thread Alex Ough
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-06-26 Thread Alex Ough
Kishan, The type of region id is Integer, not Long, so I'm wondering why it should be Long. Alex Ough On Thu, Jun 26, 2014 at 2:08 AM, Kishan Kavala kishan.kav...@citrix.com wrote: This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20099/ Alex

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

2014-06-26 Thread Alex Ough
and you to review them, which is 3/10/14. Kishan, I posted 2 questions, so please guide me for the questions. Thanks Alex Ough On Thu, Jun 26, 2014 at 12:57 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, By “huge” I’ve meant that there was a lot of repetitive hardcoded

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

2014-06-26 Thread Alex Ough
Alena, Didn't you say that you guys already did logic review in the previous email? Thanks Alex Ough On Thu, Jun 26, 2014 at 2:59 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, sorry to hear that it took so long to get on the review process. The question still remains

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

2014-06-26 Thread Alex Ough
Sounds like it goes back to what I said I wish they have been involved more actively from the start. Thanks but really making me tired. Alex Ough On Thu, Jun 26, 2014 at 5:17 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: I did logic review according to the FS assuming

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

2014-06-25 Thread Alex Ough
Thanks Kishan, but there seems to be lots of 'db.properties' files, so which one should be referenced? Alex Ough On Wed, Jun 25, 2014 at 2:25 AM, Kishan Kavala kishan.kav...@citrix.com wrote: Alex, As Alena mentioned, it is admin’s responsibility to keep ids same across Regions. Ids

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

2014-06-25 Thread Alex Ough
Thanks Alena, and I'm glad if they spend time for the review, but could it be a little earlier for you to ask them to review instead of at the last moment? I'm really exhausted with repeatedly added items whenever I post a review. Thanks Alex Ough On Wed, Jun 25, 2014 at 7:44 PM, Alena

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

2014-06-25 Thread Alex Ough
Alena, Don't get me wrong. What I'm saying is that it would have been better if you asked the review to whomever you thought was important when you started the review. Thanks Alex Ough On Wed, Jun 25, 2014 at 9:45 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, I did my

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

2014-06-25 Thread Alex Ough
, but it was also very painful for me. Thanks Alex Ough On Wed, Jun 25, 2014 at 10:41 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, In the beginning the code was not very well organazied, didn't match coding standarts (no use of spring, misleading names, not segregated to its own

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

2014-06-24 Thread Alex Ough
. 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-06-24 Thread Alex Ough
. But the ids of regions are not same across the regions and currently the regions do not have uuids(they will not be same either if we add them to regions), so I'd like to use names. Please let me know what you think. Thanks Alex Ough On Tue, Jun 24, 2014 at 7:05 PM, Animesh Chaturvedi animesh.chaturv

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

2014-06-24 Thread Alex Ough
What I'm trying to say is that when we pass the ids of regions, the receivers do not know what the originated region is and the id of each region is not same across all the regions. Thanks Alex Ough On Tue, Jun 24, 2014 at 7:35 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex

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

2014-06-24 Thread Alex Ough
We can use the same ids names, but we don't have to use the same ids if we use names, which is a little easier because names are user readable but ids are not, so we don't need to memorize/check all the ids when we add new regions in multiple regions, which can be confusing. Thanks Alex Ough

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

2014-06-24 Thread Alex Ough
identifier. It's been a long way to come this far with several reasons, so I really want to wrap this up as soon as possible, and this doesn't seem to be a major obstacle, so let me just use 'id' as a parameter if there is no one with a different thought until tomorrow morning. Thanks Alex Ough On Tue

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

2014-06-24 Thread Alex Ough
for this now. Thanks Alex Ough On Tue, Jun 24, 2014 at 9:59 PM, Alex Ough alex.o...@sungardas.com wrote: I agree with that the ids are unique identifier, but they are usually internal purpose not exposed to the users. So it is a little strange to ask users to assign ids when they add new regions

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

2014-06-20 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-06-20 Thread Alex Ough
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-06-18 Thread Alex Ough
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-06-16 Thread Alex Ough
. WHAT DO YOU MEAN?? - Alena Prokharchyk On June 15, 2014, 9:40 p.m., 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-06-15 Thread Alex Ough
by randomly generating changes each region. Thanks, Alex Ough

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

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

Re: Control event publishing in multi region setups

2014-06-04 Thread Alex Ough
. The API interfaces to manage the user/account/domain will have an additional input param, 'originated_region_uuid', to support this change. Please let me know if you have any comments. Thanks Alex Ough On Mon, Jun 2, 2014 at 12:52 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Yes

Re: Control event publishing in multi region setups

2014-06-04 Thread Alex Ough
I thought about this and I think it is better to save the local region uuid because all resources are sure to be created in the local region, which is #4. Thanks Alex Ough On Wed, Jun 4, 2014 at 12:28 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, one more bullet is needed

Re: Control event publishing in multi region setups

2014-06-04 Thread Alex Ough
Prokharchyk alena.prokharc...@citrix.com wrote: But what if those resources are synced around regions prior to the upgrade? With the approach you suggest, the same resource will have different region id in each region¹s DB. -Alena. On 6/4/14, 9:33 AM, Alex Ough alex.o...@sungardas.com wrote

Re: Control event publishing in multi region setups

2014-06-04 Thread Alex Ough
()' as the user/account/domain objects are stored when they have been changed instead of storing it in their tables. Let me know if you have any issue with this. Thanks Alex Ough On Wed, Jun 4, 2014 at 1:15 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: On 6/4/14, 9:42 AM, Alex Ough

Re: Control event publishing in multi region setups

2014-06-04 Thread Alex Ough
using regionA/regionB analogy. Thanks, Alena. On 6/4/14, 3:27 PM, Alex Ough alex.o...@sungardas.com wrote: I just found out an issue when storing 'originatedRegionUuid' in user/account/domainVO in case of removing them because the record is actually removed and it is not recommended

Re: Control event publishing in multi region setups

2014-06-02 Thread Alex Ough
Hi Alena, Did you get back from the vacation? If so, let me know when it is the good time to discuss this. Thanks Alex Ough On Thu, May 15, 2014 at 9:02 AM, Alex Ough alex.o...@sungardas.com wrote: I know. That's why I asked before Alex Huang to let me know when he's available after he's

Re: Control event publishing in multi region setups

2014-06-02 Thread Alex Ough
Hi Alex Huang, Can you tell me when you're available? Thanks Alex Ough On Mon, Jun 2, 2014 at 12:52 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Yes, I’m back. Please check with Alex Huang what time he can be on the call with you. I can join any time today/tomorrow. -Alena

Re: Control event publishing in multi region setups

2014-05-16 Thread Alex Ough
I know. That's why I asked before Alex Huang to let me know when he's available after he's coming back next week. Have a good vacation. Thanks Alex Ough On Wed, May 14, 2014 at 4:21 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, I’m on vacation tomorrow; leaving today at 2

Re: Control event publishing in multi region setups

2014-05-15 Thread Alex Ough
generate an event would mean this process would be broken for them. This is the regression that causes the blocker. --Alex *From:* Alex Ough [mailto:alex.o...@sungardas.com] *Sent:* Thursday, May 8, 2014 11:02 AM *To:* Alex Huang *Cc:* Murali Reddy; Alena Prokharchyk; Kishan Kavala

Re: Control event publishing in multi region setups

2014-05-15 Thread Alex Ough
Are we going to rolling this out? On Thu, May 8, 2014 at 2:28 PM, Alex Ough alex.o...@sungardas.com wrote: That's why there are 2 methods, one is that generates events and the other not and there are already a few public methods without event decoration. On Thu, May 8, 2014 at 2:25 PM

Re: Control event publishing in multi region setups

2014-05-15 Thread Alex Ough
any. Thanks Alex Ough On Tue, May 13, 2014 at 12:56 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, how do you know that the data is useless? Only the recipient can make this judgement. In your case you know that the recipient – your local region – doesn’t need this data

Re: Control event publishing in multi region setups

2014-05-14 Thread Alex Ough
I think I forgot to mention this, but I think we should talk with Alex Huang also because you need his approval. So let me know when you guys are available and let's just stop sending emails back and forth. Thanks Alex Ough On Wed, May 14, 2014 at 2:30 PM, Alex Ough alex.o...@sungardas.com

Re: Control event publishing in multi region setups

2014-05-14 Thread Alex Ough
and Alex. From: Alex Ough alex.o...@sungardas.com Date: Wednesday, May 14, 2014 at 11:33 AM To: Alena Prokharchyk alena.prokharc...@citrix.com Cc: Alex Huang alex.hu...@citrix.com, Murali Reddy murali.re...@citrix.com, Kishan Kavala kishan.kav...@citrix.com, dev@cloudstack.apache.org dev

Re: Control event publishing in multi region setups

2014-05-14 Thread Alex Ough
My meeting is being delayed, so let me know when you guys are available from tomorrow. Thanks Alex Ough On Wed, May 14, 2014 at 3:05 PM, Alex Ough alex.o...@sungardas.com wrote: I have a meeting in 20 min which is estimated to end 1pm PST, so I'll let you know once it is over. On Wed, May

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

2014-05-13 Thread Alex Ough
in one region. 2. 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: Control event publishing in multi region setups

2014-05-12 Thread Alex Ough
Alex, It looks like I'd better wait until you're back because I'm afraid Alena seems to need your approval based on what I've been through. Let me know once you're back. Thanks Alex Ough On Sat, May 10, 2014 at 12:50 PM, Alex Huang alex.hu...@citrix.com wrote: Alex and Alena, Perhaps

Re: Control event publishing in multi region setups

2014-05-12 Thread Alex Ough
That is not good, but I'm wondering if you can approve after our conversation without consulting with Alex Hwang. Thanks Alex Ough On Mon, May 12, 2014 at 2:37 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: We do have to come to conclusion for this remaining issue before

Re: Control event publishing in multi region setups

2014-05-12 Thread Alex Ough
I'm not really sure why you think it is a bug. And why do you want to send data that is absolutely useless to the destination? Thanks Alex Ough On Mon, May 12, 2014 at 6:19 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: Alex, I can’t approve the current approach you use in your

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

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

Re: Control event publishing in multi region setups

2014-05-10 Thread Alex Ough
, then I'll follow your suggestion. Thanks Alex Ough On Sat, May 10, 2014 at 11:55 AM, Alex Ough alex.o...@sungardas.com wrote: I really don't know why you guys are making it complicated. The class has two different methods, one with 'event' decorator and the other without it. So the callers

Re: Control event publishing in multi region setups

2014-05-10 Thread Alex Ough
reconsider that. --Alex *From:* Alex Ough [mailto:alex.o...@sungardas.com] *Sent:* Friday, May 9, 2014 9:14 PM *To:* Alex Huang *Cc:* Murali Reddy; Alena Prokharchyk; Kishan Kavala; dev@cloudstack.apache.org *Subject:* Re: Control event publishing in multi region setups Are we

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

2014-05-06 Thread Alex Ough
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17790/#review42177 --- On May 4, 2014, 9:18 p.m., Alex Ough wrote: --- This is an automatically generated e-mail

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

2014-05-05 Thread Alex Ough
-mail. To reply, visit: https://reviews.apache.org/r/20099/#review42175 --- On May 4, 2014, 9:12 p.m., Alex Ough wrote: --- This is an automatically generated e-mail. To reply, visit

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-04-28 Thread Alex Ough
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 alex.o...@sungard.com wrote: On April 17, 2014, 8:10 p.m., Alena Prokharchyk wrote: 1) Alex, I

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

2014-04-28 Thread Alex Ough
, 10:35 AM, Alex Ough alex.o...@sungardas.com 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, please see the answers inline. May be it would be a good idea if we discuss the high level architecture

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

2014-04-28 Thread Alex Ough
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 alex.o...@sungardas.com wrote: what do have any web conferencing app

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 alex.o...@sungardas.com wrote: Well, I'm not sure about

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

2014-04-08 Thread Alex Ough
. 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 not providing the input earlier, I understand that requests like this shouldn't come the last

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

2014-04-08 Thread Alex Ough
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 alex.o...@sungardas.com wrote: Yes, I

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
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
, 12:13 PM, Alex Ough alex.o...@sungard.com wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20099/ --- (Updated April 7, 2014

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

2014-04-07 Thread Alex Ough
#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 at 8:39 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote

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 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

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 alex.o...@sungard.com wrote: Sorry, my bad. I mis-read your comment. Thanks for pointing it out. Alex Ough On Wed, Mar 26

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 alex.o...@sungardas.com wrote: All, I updated the patches as per Alena's request. Let me know if there is anything missing/incorrect. Thanks Alex Ough

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

2014-04-03 Thread Alex Ough
this info would be useful for all CS committers. Thank you, Alena. From: Alex Ough alex.o...@sungardas.com Date: Thursday, April 3, 2014 at 9:22 AM To: Chiradeep Vittal chiradeep.vit...@citrix.com, Alena Prokharchyk alena.prokharc...@citrix.com, Darren Shepherd darren.sheph

How to use @Inject

2014-03-31 Thread Alex Ough
=com.cloud.domain.dao.DomainDaoImpl / bean id=accountDaoImpl class=com.cloud.user.dao.AccountDaoImpl / bean id=userDaoImpl class=com.cloud.user.dao.UserDaoImpl / /beans 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 alex.o...@sungard.com wrote: All, I'm trying to change ComponentContext.getComponent

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

2014-03-26 Thread Alex Ough
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
classes to process domain/account/user 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 alex.o...@sungard.com wrote: What I think based on your comments are 1. Well, I have a different

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 alex.o...@sungard.com Date: Wednesday, March 26

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 didn't have. From: Alena

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 alex.o...@sungard.com 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-13 Thread Alex Ough
Chiradeep, Any comments on them? Thanks Alex Ough On Wed, Mar 12, 2014 at 10:58 AM, Alex Ough alex.o...@sungard.com 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 alex.o

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

2014-03-13 Thread Alex Ough
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 Prokharchyk alena.prokharc

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

2014-03-13 Thread Alex Ough
- 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 13, 2014

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

2014-03-13 Thread Alex Ough
- 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 meant to be called from different classes

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

2014-03-13 Thread Alex Ough
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, and it makes it hard to see/review the feature. Can

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

2014-03-12 Thread Alex Ough
I can get the reply soon. Thanks Alex Ough On Wed, Mar 12, 2014 at 4:13 AM, Daan Hoogland daan.hoogl...@gmail.comwrote: 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

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 alex.o...@sungard.com wrote: Hi Chiradeep, Can you give me the example of the Singleton service class you

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 alex.o...@sungard.com wrote: I uploaded the patch A that includes only core changes, so please review it and let me know if you have any

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 alex.o...@sungard.com wrote: Please see my reply/question in blue. Thanks

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 daan.hoogl...@gmail.comwrote: I will call the merge in a moment

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 Unit tests

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

2014-03-09 Thread Alex Ough
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
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

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-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 alex.o...@sungard.com wrote: Yes, by 3/7. Thanks Alex Ough On Tue, Mar 4, 2014 at 12:30 PM

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 daan.hoogl...@gmail.comwrote: two weeks from now, but under debate. Let's go with that, though. On Sun, Mar 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 daan.hoogl...@gmail.comwrote: This week? should be ok. end of next week is too late! On Tue, Mar 4, 2014 at 6:17 PM, Alex Ough alex.o...@sungard.com wrote: Do you think it is ok to wrap up the implementation

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 daan.hoogl...@gmail.comwrote: 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 alex.o

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

2014-02-28 Thread Alex Ough
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 daan.hoogl...@gmail.comwrote: I see, great. I

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

  1   2   >