automatically by randomly generating
changes each region.
Thanks,
Alex Ough
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
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
, 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
O
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, Kish
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
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 proce
ally asked Chiradeep 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
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
wrote:
>This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20099/
>
> Alex,
>
this far, 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 spri
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:
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
Hi Alena,
Can you confirm if this fix is correct?
@Parameter(name = ApiConstants.ORIGINATED_REGION_ID, type =
CommandType.INTEGER, description = "Region where this account is created.",
since = "4.5")
private Integer originatedRegionId;
Thanks
Alex Ough
On Wed, J
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
wrote:
> Alex,
>
> As Alena mentioned, it is admin’s responsibility to keep ids same across
> Regions. Ids sh
a solution for this now.
Thanks
Alex Ough
On Tue, Jun 24, 2014 at 9:59 PM, Alex Ough 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 r
nique
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.
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.
Th
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.co
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
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
tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
Alex Ough
done manually and also automatically by randomly generating
changes each region.
Thanks,
Alex Ough
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
iber] found for
dependency: expected at least 1 bean which qualifies as autowire candidate for
this dependency. Dependency annotations: {@javax.inject.Inject()}
Can you give me how to resolve this?
- Alex
---
This is an au
e in the CS code like
> account/domain/user/event objects. Please remove it from the event. Or come
> up with a different solution that doesn't touch the event.
>
WHAT DO YOU MEAN??
>
> - Alena Prokharchyk
>
>
> On June 15, 2014, 9:40 p.m., Alex Oug
network connection
issues.
3. The tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
Alex Ough
by randomly generating
changes each region.
Thanks,
Alex Ough
; It would be helpful if you explain the process in more details using
> regionA/regionB analogy.
>
> Thanks,
> Alena.
>
>
> On 6/4/14, 3:27 PM, "Alex Ough" wrote:
>
> >I just found out an issue when storing 'originatedRegionUuid' in
> >user/ac
#x27;CallContext.current()' 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> wro
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,
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
re created in the local region.
4. 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 Prokh
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 ti
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 wrote:
> I know. That's why I asked before Alex Huang to let me know when he's
> available after he's
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 to
thod not to generate
event?
Let me know if I'm missing 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 yo
Alena,
I think we should talk, so please let me know when you're available.
Thanks
Alex Ough
On Wed, May 14, 2014 at 1:36 PM, Alena Prokharchyk <
alena.prokharc...@citrix.com> wrote:
> Alex, we do understand how “Full Scan” works and we know that your
> component/other comp
Are we going to rolling this out?
On Thu, May 8, 2014 at 2:28 PM, Alex Ough 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 P
ack is actually in their own billing database.
> The fact that #2 doesn’t 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...@sung
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 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,
ing invitation to me and Alex.
>
> From: Alex Ough
> Date: Wednesday, May 14, 2014 at 11:33 AM
>
> To: Alena Prokharchyk
> Cc: Alex Huang , Murali Reddy <
> murali.re...@citrix.com>, Kishan Kavala , "
> dev@cloudstack.apache.org"
> Subject: Re: Con
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 wrote:
> Alen
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
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 appro
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 remainin
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 wrote:
> Alex and Alena,
>
&
.
3. The tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
Alex Ough
fered a path forward already. Please 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.apach
use the events, then I'll
follow your suggestion.
Thanks
Alex Ough
On Sat, May 10, 2014 at 11:55 AM, Alex Ough 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 witho
e
command classes are located. Please let me know if I'm missing any.
- Alex
---
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
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
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
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
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 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:
>
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
; 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
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:
>
> -
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
.
Thanks,
Alex Ough
like network connection
issues.
3. The tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
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
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
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
.
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
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
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
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/
> >-
automatically by randomly generating
changes each region.
Thanks,
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
. 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
automatically by randomly generating
changes each region.
Thanks,
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
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
connection
issues.
3. The tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
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
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
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
done manually and also automatically by randomly generating
changes each region.
Thanks,
Alex Ough
es.
3. The tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
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
>
Can anyone show me what needs to be fixed?
Thanks
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
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
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
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
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
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
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
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
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
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
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.
>
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
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?
>
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
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?
&
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
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
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
>
1 - 100 of 184 matches
Mail list logo
