Alex,
As Alena mentioned, it is admin’s responsibility to keep ids same across
Regions. Ids should be used as unique identifier. Region name is merely
descriptive name and its mostly associated with geographic location.
Also note that region name can be updated anytime using updateRegion API.
On Jun 24, 2014, at 10:19 PM, Mike Tutkowski mike.tutkow...@solidfire.com
wrote:
Thanks, Seif!
Seif and I have been communicating pretty much on a daily basis with
regards to his GSoC project.
He's new to CloudStack, so there was a lot of startup-related overhead.
Once he got past
On Jun 24, 2014, at 12:13 AM, Ian Duffy i...@ianduffy.ie wrote:
Hi Folks,
At the end of this week (27th) we will be at the mid term evaluation stage
of the Google Summer of code programme.
It has been great working with the Cloudstack Community again. I'm very
happy with the progress
Hi,
We're trying to get some sort of accounting going on and we're looking at
checking the network stats, but while listVirtualMachines will list some
minimal stats[1] for running VMs, we were hoping listUsageRecords types 4 5
(
NETWORK_BYTES_SENT RECEIVED) would have some more data, but
I also noticed the cloud_usage DB doesn't have any network stats either:
mysql select * from cloud_usage where usage_type=4;
Empty set (0.30 sec)
mysql select * from cloud_usage where usage_type=5;
Empty set (0.29 sec)
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
On Jun 25, 2014, at 5:37 AM, Nux! n...@li.nux.ro wrote:
Hi,
We're trying to get some sort of accounting going on and we're looking at
checking the network stats, but while listVirtualMachines will list some
minimal stats[1] for running VMs, we were hoping listUsageRecords types 4
5 (
We have a use-case where we will patch an existing 4.3 installation with our
plugin.
We are facing the same issue .
In 4.2, we used to disable the entry for ExternalNetworkGuru in
componentContext.xml as part of installing the patch.
How do we do this in 4.3 (on an existing installation) ?
H,
I am slowly moving through the list of criticals starting with those
that were not updated the longest, marking them as future instead of
4.4.0
If you feel that criticals should be fixed update and search for
solutions. If you feel they are blockers discuss them on list, please.
regards,
--
Hi guys,
Checked the other emails in users@ re usage, but it does not seem to be my
problem (I have a working connection to mysql, tables are populated, all tables
are there etc). However, the data for types 4 and 5 (NETWORK that is) is
missing completely from the DB as I said earlier.
Here's
Let me wrap in Loic, he is running 4.3 with KVM and has usage going.
@Loic, do you see usage of type 4 in your cloud usage DB ? the network data ?
thanks,
-sebastien
On Jun 25, 2014, at 12:29 PM, Nux! n...@li.nux.ro wrote:
Hi guys,
Checked the other emails in users@ re usage, but it does
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
Hi there,
Thanks to Min, Rohit and Prachi for the details about the current
implementation.
On the if-else thing, that's not a problem to have them. At some point in time
you have to have if-else in the code. The point is just to not abuse of it. But
no worries, because there is no abuse of
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21773/#review46616
---
But in debian wheezy sysctl.conf is not got loaded on boot time.
Team,
With the help of Vagrant boxes and simulator, We can now get a readily
available Cloudtack with data center. With this facility, automatically, It
will provision a thin ubuntu vm, required software and Cloudstack are
installed, followed by a deploy data center. Please check the below
Santhosh,
This looks cool.
(Just a timing coincidence) I've been doing the same thing but with chef.
https://github.com/imduffy15/GSoC-2014/blob/master/simulator/Vagrantfile
Does yours take long to come up? I find cloning the repo and compiling the
codebase can be slow.
On 25 June 2014 13:27,
Hi Guys,
We¹re running 4.3+ (KVM, Basic Networking). I confirm that we do not have
any record of type 4 in the usage DB.
Best Regards,
--
Loic Lambiel
EXOSCALE Head of Operations
Tel : +41 58 668 56 00
Mail : loic.lamb...@exoscale.ch
Swiss Cloud Services
www.exoscale.ch
--
Thanks and Regards,
Chandra Prakash Mishra
Looks great, thanks for sharing Santhosh!
Will try it in the evenings or during weekends.
Cheers.
On Wed, Jun 25, 2014 at 5:57 PM, Santhosh Edukulla
santhosh.eduku...@citrix.com wrote:
Team,
With the help of Vagrant boxes and simulator, We can now get a readily
available Cloudtack with
Hi Wilder,
On Wed, Jun 25, 2014 at 4:57 PM, Wilder Rodrigues
wrodrig...@schubergphilis.com wrote:
Hi there,
Thanks to Min, Rohit and Prachi for the details about the current
implementation.
On the if-else thing, that's not a problem to have them. At some point in
time you have to have
Welcome! Feel free to subscribe to the mailing list by sending an
email to dev-subscr...@cloudstack.apache.org. You will then have to
reply to the confirmation email you receive.
On Wed, Jun 25, 2014 at 3:37 AM, chandraprakash mishra
cpmishra...@gmail.com wrote:
--
Thanks and Regards,
Chandra
It seems not (we’re purging old usage data)
Loic Lambiel
EXOSCALE Head of Operations
Tel : +41 58 668 56 00
Mail : loic.lamb...@exoscale.ch
Swiss Cloud Services
www.exoscale.ch http://www.exoscale.ch/
exoscale is a brand of Veltigroup SA
Av. de Provence 4 CH 1007 Lausanne
Nitin, can you have a look at your SnapshotPolicyDetails files. Three
files have a license problem
On Wed, Jun 25, 2014 at 3:01 PM, Apache Jenkins Server
jenk...@builds.apache.org wrote:
See https://builds.apache.org/job/cloudstack-rat-master/4947/changes
Changes:
[daan] CLOUDSTACK-6747 test
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22955/
---
Review request for cloudstack and Girish Shilamkar.
Repository: cloudstack-git
hi mike,
now this updateStoragePool API would be really helpful to update the
backend storagepool and other third party features.
question on individual CloudStack volumes
is it feasible to associate resize volume and changing IOPS into one API ?
if so we might need to add more intelligence in
Sebastien,
CLOUDSTACK-6747, entered by Erik Weber a while ago and discussed on
some list (this or user@ or both) has some implementation in master
now. We are running 4.3 and though we would prefer to go to 4.4 it
might be worthwhile backporting it to 4.3.1
Do you have a ..-forward policy for
Hello,
Related to my previous thread, I've opened the following:
https://issues.apache.org/jira/browse/CLOUDSTACK-6993
I can also confirm that Advanced zones are not affected by this bug (or at
least I can see plenty of table entries in cloud_usage like usage_type=4).
Can anyone shed some more
Opened https://issues.apache.org/jira/browse/CLOUDSTACK-6993
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
From: Loic Lambiel loic.lamb...@exoscale.ch
To: Sebastien Goasguen run...@gmail.com
Cc: dev@cloudstack.apache.org
Sent: Wednesday,
if you know the commit, then feel free to merge it in the 4.3 branch with the
CLOUDSTACK-6747 name in the commit message.
On Jun 25, 2014, at 9:52 AM, Daan Hoogland daan.hoogl...@gmail.com wrote:
Sebastien,
CLOUDSTACK-6747, entered by Erik Weber a while ago and discussed on
some list
Alex,
You can refer to the code from initDataSource method in Transaction.java.
Properties file can be loaded using the following:
File dbPropsFile = PropertiesUtil.findConfigFile(propsFileName);
From: Alex Ough [mailto:alex.o...@sungardas.com]
Sent: Wednesday, 25 June 2014 4:31 PM
To: Kishan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22721/#review46630
---
Ship it!
Commit c468228fe807c621decc5919dadae9bcbb38c753
- Kishan
See http://jenkins.buildacloud.org/job/build-master/1031/changes
Changes:
[Daan Hoogland] CLOUDSTACK-6747 test for test to allow all cidrs on other end
of a vpc
--
[...truncated 450 lines...]
at
See http://jenkins.buildacloud.org/job/build-master/1032/changes
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22805/
---
(Updated June 25, 2014, 3:24 p.m.)
Review request for cloudstack and Girish
it is in three commits, sorry for the mess.
On Wed, Jun 25, 2014 at 4:51 PM, Sebastien Goasguen run...@gmail.com wrote:
if you know the commit, then feel free to merge it in the 4.3 branch with the
CLOUDSTACK-6747 name in the commit message.
On Jun 25, 2014, at 9:52 AM, Daan Hoogland
LS,
As the critical issue list is starting to go down and no one has
discussed any bug as being blocking for a while, I would like to make
a release candidate on Friday.
As requested earlier, please triage your bug lists and send me any
cherry-pick requests for fixes you would like to go in
On June 24, 2014, 5:27 a.m., Santhosh Edukulla wrote:
tools/marvin/marvin/configGenerator.py, line 216
https://reviews.apache.org/r/22554/diff/3/?file=615070#file615070line216
All looks good now, please do a dry run on both master and 4.4-forward
i.e., deploy a datacenter atleast
I'm trying to get a handle on whats new with CM 5.1.x
Anybody point me in the direction of the release notes or anywhere else that
summarises ?
thanks
Kind Regards
Giles
Giles Sirett
CEO
[Description: Mail Logo Bottom Align]
D: +44 20 3603 0541 | S: +44 20 3603 0540 | M: +44 796 111 2055
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22554/#review46635
---
Ship it!
Ship It!
- Santhosh Edukulla
On June 23, 2014, 3:53
Hi Giles,
The changelog can be see at
https://github.com/apache/cloudstack-cloudmonkey/blob/master/CHANGES
On 25 June 2014 16:46, Giles Sirett giles.sir...@shapeblue.com wrote:
I’m trying to get a handle on whats new with CM 5.1.x
Anybody point me in the direction of the release notes or
Hi guys,
Can anyone tell me which changes I should make in order to increase password
length for instances?
Currently I get something like tK2yptbby which might have been secure 10
years ago, but now it's way too short.
Thanks!
Lucian
--
Sent from the Delta quadrant using Borg technology!
Hi Punith,
Yes, that is exactly the approach I was planning on taking: enhance the
resizeVolume API to allow for IOPS control and to extend what it can do
with volume sizes.
Talk to you later!
Mike
On Wed, Jun 25, 2014 at 7:44 AM, Punith S punit...@cloudbyte.com wrote:
hi mike,
now this
I have one minor issue I'd like to solve before a RC, but Friday should be
no problem.
It's kind of a weird issue where label.SR.name shows up literally in the
Add Primary Storage dialog (as opposed to the English word(s) that should
be substituted for it).
I've looked in the relevant files to
Thanks, Daan! I appreciate the effort in helping to get 4.4 out in the
field. :)
On Wed, Jun 25, 2014 at 4:16 AM, Daan Hoogland daan.hoogl...@gmail.com
wrote:
H,
I am slowly moving through the list of criticals starting with those
that were not updated the longest, marking them as future
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22709/#review46647
---
Ship it!
Ship It!
- Santhosh Edukulla
On June 24, 2014, 12:38
Hi Wilder,
#3 in your list is definitely something we need to do to reuse command
code and avoid the mistake of changing one place and forgetting the other
one. But I don't quite get how to avoid passing response view parameter in
ApiResponseGenerator in the execute() method using your
Hi Lucian,
Take a look at server/src/com/cloud/server/ManagementServerImpl.java
Line 895 - 898
@Override
public String generateRandomPassword() {
return PasswordGenerator.generateRandomPassword(6);
}
On 25 June 2014 17:16, Nux! n...@li.nux.ro wrote:
Hi guys,
Can
Hi,
I noticed a couple i18n-related issues in the Add Primary Storage dialog:
1) label.path shows up literally as label.path (instead of Path)
2) label.SR.name shows up literally as label.SR.name (instead of SR
Name-Label)
I looked at the applicable i18n/l10n files and all looked well.
Once I
Oh, sounds good, Sebastien.
I've asked Seif to create a Review Board account and we can go through that
process.
Thanks
On Wed, Jun 25, 2014 at 1:52 AM, sebgoa run...@gmail.com wrote:
On Jun 24, 2014, at 10:19 PM, Mike Tutkowski mike.tutkow...@solidfire.com
wrote:
Thanks, Seif!
Seif
Thanks, Ian,
This means I need to modify the source, rebuild the RPMs and update, right? (ie
it's not something that I can just modify on the mgmt server right now).
Lucian
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
From: Ian Duffy
Afaik yes. (Will to be corrected on this but it appears to be hard coded)
On 25 June 2014 18:06, Nux! n...@li.nux.ro wrote:
Thanks, Ian,
This means I need to modify the source, rebuild the RPMs and update,
right? (ie it's not something that I can just modify on the mgmt server
right now).
Lovely,
Thank you Daan :-)
Erik
25. juni 2014 15:53 skrev Daan Hoogland daan.hoogl...@gmail.com
følgende:
Sebastien,
CLOUDSTACK-6747, entered by Erik Weber a while ago and discussed on
some list (this or user@ or both) has some implementation in master
now. We are running 4.3 and though we
I should submit a bug report to rewrite ACS in a scripting language.
Cheers :)
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
From: Ian Duffy i...@ianduffy.ie
To: CloudStack Dev dev@cloudstack.apache.org
Sent: Wednesday, 25 June, 2014
You should ask if this can be done as global setting variable - not hard
coded.
This should be an easy one.
On 6/25/14, 10:14 AM, Nux! wrote:
I should submit a bug report to rewrite ACS in a scripting language.
Cheers :)
--
Sent from the Delta quadrant using Borg technology!
Nux!
And have some bastard user from hell (or operator) change it?
Daan
mobile bi-lingual spell checker used.
On 25 Jun 2014 19:18, Nux! n...@li.nux.ro wrote:
I should submit a bug report to rewrite ACS in a scripting language.
Cheers :)
--
Sent from the Delta quadrant using Borg technology!
ilya - Agreed.
On 25 June 2014 18:33, Daan Hoogland daan.hoogl...@gmail.com wrote:
And have some bastard user from hell (or operator) change it?
Daan
mobile bi-lingual spell checker used.
On 25 Jun 2014 19:18, Nux! n...@li.nux.ro wrote:
I should submit a bug report to rewrite ACS in a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22554/#review46656
---
Commit 477a812a6f7aaab74122c11488713f417dfe4d89 in cloudstack's
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22554/#review46657
---
Ship it!
477a812a6f7aaab74122c11488713f417dfe4d89 on master
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22554/#review46658
---
Commit 2498f65683bd529b2b03bac9a6cfd2fdbf65aca2 in cloudstack's
Volunteer to do it in time for 4.4?
Lucian
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
From: ilya musayev ilya.mailing.li...@gmail.com
To: dev@cloudstack.apache.org
Sent: Wednesday, 25 June, 2014 6:30:25 PM
Subject: Re: What to modify to
On Wed, Jun 25, 2014 at 7:58 PM, Santhosh Edukulla
santhosh.eduku...@citrix.com wrote:
...
Team uses 4.4-forward marvin to test the changes.
You should really not test 4.4-forward, but 4.4!
--
Daan
I don't really have any input on this. It should work in general, I just
don't know about the ppp device setup specifics. I've only ever tried over
VPN where the endpoints weren't on the KVM or mgmt server. I'd just treat
the ppp device like the mgmt IP/interface of the KVM host, but I don't know
Ho Rohit and Min,
My apologies, but sometimes I just write the things I see in my head, as I know
they would work, but don't give enough details so other people can see the same
picture. Thanks for the heads up, I will prepare it.
In terms of what it solves I would say: nothing. And why that?
Just pushed a change for this to the 4.4-forward branch.
Daan, will you review / cherrypick? 96412e3e58fd1ced9d269e4552aaa6410bedf556
Testing done:
Brought up simulator.
Changed password flag for the builtin template.
Brought up VM, password was displayed at length of 6. Stopped the VM, reset
See
http://jenkins.buildacloud.org/job/cloudstack-4.4-forward-build/292/changes
Changes:
[Ian Duffy] Allow for length of password on VMs to be set via global settings
--
[...truncated 1179 lines...]
[INFO] Using 'UTF-8' encoding to copy filtered
Unused import removed. should be resolved shortly.
On 25 June 2014 19:55, jenk...@cloudstack.org wrote:
See
http://jenkins.buildacloud.org/job/cloudstack-4.4-forward-build/292/changes
Changes:
[Ian Duffy] Allow for length of password on VMs to be set via global
settings
See
http://jenkins.buildacloud.org/job/cloudstack-4.4-forward-build/293/changes
Hi,
I've been intending to do this for a while, but decided it has to happen this
summer so here goes:
http://www.openvm.eu - A repository of Cloudstack images and appliances!
Do not be fooled by it's current façade, it will get better.
It only has one template for now, but I expect this
c70cadff0e3d70f625e7160a3aee600a3198cc67 in 4.4 it had conflicts in the sql:(
On Wed, Jun 25, 2014 at 8:53 PM, Ian Duffy i...@ianduffy.ie wrote:
Just pushed a change for this to the 4.4-forward branch.
Daan, will you review / cherrypick? 96412e3e58fd1ced9d269e4552aaa6410bedf556
Testing done:
:( Get them solved?
Fixed a styecheck issue in 2bb12f2a85c8497a15cd7b17099aefbd7c486f9b needed
to remove unused imports.
On 25 June 2014 20:24, Daan Hoogland daan.hoogl...@gmail.com wrote:
c70cadff0e3d70f625e7160a3aee600a3198cc67 in 4.4 it had conflicts in the
sql:(
On Wed, Jun 25, 2014 at
Hi Giles,
Not a lot -- it added support for a newer signature scheme (version 3,
something better than plain and simple hmac signature validation algorithm
that reduces chance of API replay attacks) and letting users use CloudStack
username/password in addition to using api/secret keys for auth
yeah, so I should probably get that one in as well?
On Wed, Jun 25, 2014 at 9:28 PM, Ian Duffy i...@ianduffy.ie wrote:
:( Get them solved?
Fixed a styecheck issue in 2bb12f2a85c8497a15cd7b17099aefbd7c486f9b needed
to remove unused imports.
On 25 June 2014 20:24, Daan Hoogland
See http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/366/changes
Changes:
[Daan Hoogland] Allow for length of password on VMs to be set via global
settings
--
[...truncated 1179 lines...]
[INFO] Using 'UTF-8' encoding to copy filtered
pulled both
On Wed, Jun 25, 2014 at 9:33 PM, Ian Duffy i...@ianduffy.ie wrote:
Yeah sorry, just noticed and fixed it myself.
On 25 June 2014 20:32, Daan Hoogland daan.hoogl...@gmail.com wrote:
yeah, so I should probably get that one in as well?
On Wed, Jun 25, 2014 at 9:28 PM, Ian Duffy
Thanks! Turns out its a java version issue - I installed java 1.7 but eclipse
still refers to 1.6. Removing 1.6 entirely fix the problem.
KC
On Jun 24, 2014, at 6:10 PM, Mike Tutkowski mike.tutkow...@solidfire.com
wrote:
Are you seeing errors in Maven POM files? If so, those are business
See http://jenkins.buildacloud.org/job/cloudstack-4.4-maven-build/367/changes
It looks like these issues were introduced in
182c31899bb353eac66a43ca4e81117c4fd06332 by vetrivelc with regards to
externalizing hardcoded strings.
My guess is that this substitution was done in an automated fashion and
some unintended consequences of the substitution logic occurred.
vetrivelc
Mike,
I guest you did that already restart the jvm of the management
server? I went to this problem once, changing and adding labels and to had
to restart cloudstack to have the label working.
Le mercredi 25 juin 2014, Mike Tutkowski mike.tutkow...@solidfire.com a
écrit :
I have one minor issue
Hi guys,
I finish the first version of design document:
https://docs.google.com/document/d/1kbPQJrBC87ZtR-t7LwHFDzAmT436ShtjwKE84FVfByM/pub
.
Someone could give me access to put design documents in wiki? Bellow the
username of people work with Cloudstack in Globo.com and need access.
-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
Sent: Thursday, May 29, 2014 10:45 AM
To: dev
Subject: Re: [ACS45][ACS50][PROPOSAL] move forward feature freeze
Seeing what objections there might be I want to amend my proposal to
1. move forward feature
you three have access to the wiki
On Wed, Jun 25, 2014 at 10:38 PM, Silvano Nogueira Buback
silv...@corp.globo.com wrote:
Hi guys,
I finish the first version of design document:
https://docs.google.com/document/d/1kbPQJrBC87ZtR-t7LwHFDzAmT436ShtjwKE84FVfByM/pub
.
Someone could give
On Wed, Jun 25, 2014 at 4:38 PM, Silvano Nogueira Buback
silv...@corp.globo.com wrote:
Hi guys,
I finish the first version of design document:
https://docs.google.com/document/d/1kbPQJrBC87ZtR-t7LwHFDzAmT436ShtjwKE84FVfByM/pub
.
Someone could give me access to put design documents in
No certainly not, that would mean yet another bigger release and
another extra shift. It is now at 19 July And I was proposing to move
it forward to last 19th of june.
On Wed, Jun 25, 2014 at 10:46 PM, Animesh Chaturvedi
animesh.chaturv...@citrix.com wrote:
-Original Message-
From:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22939/
---
(Updated June 25, 2014, 9:02 p.m.)
Review request for cloudstack.
Changes
I saw you mentioning mid august as well so there is some confusion
Animesh
-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
Sent: Wednesday, June 25, 2014 1:52 PM
To: dev
Subject: Re: [ACS45][ACS50][PROPOSAL] move forward feature freeze
No certainly not,
In response to Brocade I see your response
feature should be done (in it's branch) by 19th july. merging and fixing
issues may take to mid august that essentially means feature freeze (cutting
the branch) by mid august
Animesh
-Original Message-
From: Animesh Chaturvedi
Wunderbar! Thanks guys!
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
From: Daan Hoogland daan.hoogl...@gmail.com
To: dev dev@cloudstack.apache.org
Sent: Wednesday, 25 June, 2014 8:35:00 PM
Subject: Re: What to modify to increase instance
By the way, what I was referring to with my proposed hack was just to fix
the two situations (SR Name-Label and Path) by hardcoding the English
back in.
On Wed, Jun 25, 2014 at 2:34 PM, Mike Tutkowski
mike.tutkow...@solidfire.com wrote:
It looks like these issues were introduced in
Cool --thanks, Lucian!
Just so I understand, this is a supporting repository and not an official
CloudStack project resource, correct?
If so, totally fine: please do continue as intended. If not, I suspect this
might need to reside somewhere in the cloudstack.apache.org/* home.
I'd love
This is awesome on Nux's part. I am thrilled to see this come to fruition.
This is not an official CloudStack deployment. There are a number of
reasons why this needs to be external (for instance, much of what he
has up for download is GPL-licensed; it's not our software, etc) In
addition, Infra
Alex, looks fine to me. Make sure that you put the regionId validation as our
in-built API validation won’t work in this case because there is no UUID field
support for the Region object. You can check how validation is begin done in
updateRegion/deleteRegion scenarios.
Kishan/Murali, can you
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22695/
---
(Updated June 25, 2014, 11:54 p.m.)
Review request for cloudstack, Girish
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
Alex,
I did my best to review the code, made sure it came in shape with the CS
guidelines and java code style There was no way to anticipate all the things to
fix originally, as every subsequent review update added more things to fix as
the review code was new/refactored.
And I don’t see
Hi Tim and Mike,
On Tue, Jun 24, 2014 at 7:30 PM, Tim Mackey tmac...@gmail.com wrote:
Hieu,
It would also be good in your design doc and testing to validate which
XenServer versions you expect this to work with, and don't forget about
xcp-xapi in pure Linux as part of that. If you have
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
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
plugin), and the code base was unneccessary huge.
All of the above it very hard to review and understand the code logic from the
beginning
Thanks! I'm also curious about manual (or automated) regression testing you
may have performed.
On Wed, Jun 25, 2014 at 7:52 PM, Hieu LE hieul...@gmail.com wrote:
Hi Tim and Mike,
On Tue, Jun 24, 2014 at 7:30 PM, Tim Mackey tmac...@gmail.com wrote:
Hieu,
It would also be good in your
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22695/#review46711
---
defed59b97324ff51360a24edc470b5e6efda7ac 4.4-forward
-
Alena,
I understand that you have been helping a lot to make my codes to match the
coding standards, but I'm not sure what you mean by the code base was
unnecessary huge.
The initial implementation was to support the synchronization inside the CS
because this feature is missing in the current
Shouldn't the code handle the scenario if vm.password.length is missing from
db? In this case the upgrade script will put it but if someone accidentally
removes it.
-Original Message-
From: Nux! [mailto:n...@li.nux.ro]
Sent: Thursday, 26 June 2014 2:55 AM
To: dev@cloudstack.apache.org
1 - 100 of 102 matches
Mail list logo