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
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
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
.
3. The tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
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 wrote:
> Alex and Alena,
>
&
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
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
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 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
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
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,
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
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
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
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
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
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
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
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
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
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,
#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
; 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
by randomly generating
changes each region.
Thanks,
Alex Ough
network connection
issues.
3. The tests were done manually and also automatically by randomly generating
changes each region.
Thanks,
Alex Ough
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
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
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
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
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
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
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
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
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.
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
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
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 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
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:
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
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,
>
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
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
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
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
, 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
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
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
ncerTest') during building
'Server' project.
I'm not familiar with 'inject' tag, so I'm not sure what the problem is,
but just defining a variable using 'inject' tag causes the failure.
Your help will be very appreciate.
Thanks
Alex Ough
;Unknown column 'disk_offering.cache_mode'
in 'field list' even if I re-built the database.
Is the schema change part of your changes or something other developer
changed?
If that is from any other developer, can you fix this?
Thanks
Alex Ough
On Tue, Nov 19, 2013 at 7:25 PM, Alex Huang wr
be null, so I just changed the column to allow null to
move on.
So you may include the sql file and allow null in that column.
Thanks
Alex Ough
On Wed, Nov 20, 2013 at 3:45 PM, Alex Huang wrote:
> Wido,
>
>
>
> Looks like you didn’t update your schema file or forgot to add a sc
I see. Thanks for the information, Alex.
Alex Ough
On Thu, Nov 21, 2013 at 9:22 AM, Alex Huang wrote:
> It’s because we switched over to a new way to declare configuration
> variables. Here’s some information about it.
>
>
>
> https://cwiki.apache.org/confluence/
engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 3a3de70
setup/db/db/schema-410to420.sql 522ccc4
setup/db/db/schema-430to440.sql 9bb9602
Diff: https://reviews.apache.org/r/15763/diff/
Testing
---
Tested in the simulator mode and Xen servers.
Thanks,
Alex Ough
All,
I'm very interested in converting the current DAO framework to an ORM. I
didn't have any experience with java related ORMs, but I've done quite lots
of works with Django and LINQ. So can you add me if this project is started?
Thanks
Alex Ough
On Fri, Nov 22, 2013 at 7:06 AM
Thanks to help from many developers,
I sent my first review request in the cloudstack,
https://reviews.apache.org/r/15763/,
so please take a look at it and let me know if there is anything
missing/incorrect.
Thanks again.
Alex Ough
On Thu, Nov 21, 2013 at 10:09 AM, Alex Ough wrote:
> Hi W
All,
It's been a while since this review was requested, so can anyone review
this to move on?
Thanks in advance.
Alex Ough
On Fri, Nov 22, 2013 at 11:51 AM, Alex Ough wrote:
> Thanks to help from many developers,
> I sent my first review request in the cloudsta
vate AccountDao _accountDao;' in 'ActionEventInterceptor' class and the
same injection was already included in 'ActionEventUtils' class that is in
the same package.
It is quite strange to me, so can you give me some light on this?
Thanks
Alex Ough
On Thu, Nov 28, 2013 at
l 522ccc4
setup/db/db/schema-430to440.sql 9bb9602
Diff: https://reviews.apache.org/r/15763/diff/
Testing
---
Tested in the simulator mode and Xen servers.
Thanks,
Alex Ough
Thank Koushik, and I modified what you pointed out (for 'MigrateCommand'
text) and updated the diffs.
Let me know if there is anything missing/incorrect.
Thanks
Alex Ough
On Tue, Dec 3, 2013 at 2:46 AM, Koushik Das wrote:
> I have posted my review comments. Except for a few m
fs (updated)
-
engine/api/src/com/cloud/vm/VirtualMachineManager.java c78942f
setup/db/db/schema-410to420.sql 522ccc4
Diff: https://reviews.apache.org/r/15763/diff/
Testing
---
Tested in the simulator mode and Xen servers.
Thanks,
Alex Ough
Hi Koushik,
This time I reloaded only the 2 changes.
To make it clean, I tried to remove the previous incorrect one, but I
couldn't find how.
So let me know if it is better to discard this review and create a new one.
Thanks
Alex Ough
On Wed, Dec 4, 2013 at 12:04 AM, Koushik Das
a blocker of what I'm
working on for
CLOUDSTACK-4992<https://issues.apache.org/jira/browse/CLOUDSTACK-4992>
.
Thanks
Alex Ough
On Wed, Nov 20, 2013 at 1:37 PM, Nitin Mehta wrote:
> David - CallContext gets created during the entry point of the API.
> I haven't had the chanc
3/diff/
Testing
---
Tested in the simulator mode and Xen servers.
Thanks,
Alex Ough
Hi Koushik,
I followed your instructions and this is the merged commit.
https://reviews.apache.org/r/15763/diff/
Let me know if there is anything missing/incorrect.
Thanks
Alex Ough
On Wed, Dec 4, 2013 at 11:44 PM, Koushik Das wrote:
> Alex,
>
> This is what you need to d
12:45:38 -0600","status":"Completed","description":"Successfully completed
deleting User. UserId:
5","event":"USER.DELETE","entityuuid":"67888f40-80f3-4ec0-9a45-e1f729cf7b36","entity":"com.cloud.user.User","account":"32907226-5dd5-11e3-a6af-d073d0b82aa6","user":"3295ad2c-5dd5-11e3-a6af-d073d0b82aa6"}'
Thanks,
Alex Ough
All,
I submitted the review request, so please review it and let me know if
there is anything missing/incorrect.
Thanks
Alex Ough
On Wed, Dec 4, 2013 at 11:29 PM, Murali Reddy wrote:
> On 05/12/13 12:01 AM, "Alex Ough" wrote:
>
> >All,
> >
> >I made a comme
.
I think it is appropriate to follow the same implementation with yours, so
can I get the review request information of this implementation so that I
can reference your implementation if possible?
Thanks.
Alex Ough
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15763/#review29872
---
Ship it!
Ship It!
- Alex Ough
On Dec. 5, 2013, 3:28 p.m., Alex
Hi Sheng, good to hear from you!
Btw, can I get the review request id of the RvR so that I can get
information of how it was implemented?
Thanks
Alex Ough
On Thu, Dec 5, 2013 at 6:53 PM, Sheng Yang wrote:
> I meant, VPC and redundant router. :)
>
> --Sheng
>
>
> On Thu, D
uot;:"32907226-5dd5-11e3-a6af-d073d0b82aa6","user":"3295ad2c-5dd5-11e3-a6af-d073d0b82aa6"}'
[x]
'management-server.ActionEvent.USER-DELETE.User.*':'{"eventDateTime":"2013-12-05
12:45:38 -0600","status":"Completed","description":"Successfully completed
deleting User. UserId:
5","event":"USER.DELETE","entityuuid":"67888f40-80f3-4ec0-9a45-e1f729cf7b36","entity":"com.cloud.user.User","account":"32907226-5dd5-11e3-a6af-d073d0b82aa6","user":"3295ad2c-5dd5-11e3-a6af-d073d0b82aa6"}'
Thanks,
Alex Ough
when the management server is started?
Your reply will be very appreciated.
Thanks
Alex Ough
I modified the fix to make a little simpler, so can you review it please?
I'd like to finalize this as soon as possible to move on with CLOUDSTACK-4992.
Thanks
Alex Ough
On Thu, Dec 5, 2013 at 1:32 PM, Alex Ough wrote:
> All,
>
> I submitted the review request, so please review
Hi Muali,
Thanks for your information, but in the sample codes,
'EventNotificationSubscriberImpl' implements an interface,
'EventNotificationsSubscriber', but I can't find
'EventNotificationsSubscriber' in the project.
Can you show me where it is located?
Tha
But I think there seems to be a missing package because I can't find
the package, 'utils', that includes these 2 classes either.
import com.cloud.utils.component.Adapters;
import com.cloud.utils.component.ComponentLocator;
FYI, I'm using the master.
Thanks
Alex Ough
On Mo
Hi Murali,
Where will be the right place to add my implemented EventSubscriber to
the EventBus?
Thanks
Alex Ough
On Wed, Dec 11, 2013 at 5:39 AM, Murali Reddy wrote:
>
> Sorry my bad. Sample code is using old way of loading beans before spring
> is brought in. P
NVM, I think I figured out.
Thanks
Alex Ough
On Wed, Dec 11, 2013 at 11:19 AM, Alex Ough wrote:
> Hi Murali,
>
> Where will be the right place to add my implemented EventSubscriber to
> the EventBus?
>
> Thanks
> Alex Ough
>
> On Wed, Dec 11, 2013 at 5:39 AM, Murali R
ntext' to store the entity uuids and types in
'publishOnEventBus' method.
Correct me if I'm wrong.
Thanks
Alex Ough
On Thu, Dec 12, 2013 at 10:46 AM, David Grizzanti
wrote:
> Murali/All,
>
> Opening this discussion back up to decide how to approach fixing this. I
> looked o
A little confusion here because 'setEntityDetails' in 'UserContext' is
no longer used in master.
Is this functionality only for 4.2 and not supported in master?
Thanks
Alex Ough
On Mon, Dec 16, 2013 at 8:35 AM, Murali Reddy wrote:
> David,
>
> Your analysis is right
any cases.
Thanks for your interest and your feedback will be very helpful.
Alex Ough
On Tue, Nov 12, 2013 at 6:00 PM, Alex Ough wrote:
> Good point, Chiradeep,
>
> I'm not sure if you reviewed my design doc in the wiki, but my design is to
> just skip any actions for target res
Hi Murali,
I created the 'spring-event-bus-context.xml' under
'server/resources/META-INF/cloudstack/core', and it works ok,
so you may move the context file over there and try.
Thanks
Alex Ough
On Wed, Dec 18, 2013 at 3:40 AM, Murali Reddy wrote:
> Darren,
>
&
and submit the
review requests for this implementation.
Let me know if you have any comments.
Thanks
Alex Ough
On Wed, Dec 18, 2013 at 3:39 PM, Alex Ough wrote:
> All,
>
> I updated the wiki after some logic changes, so please review them,
> especially "Full Scan", whic
luding login info of each region.
I was planning to use a table as a repository, but I think it is better to
store it in the config file to make the access a little lighter.
Any recommendation on this?
Your reply with directions & comments will be very appreciated.
Thanks
Alex Ough
On Wed, Jan
101 - 184 of 184 matches
Mail list logo
