Re: Review Request 18623: CLOUDSTACK-6147: Adding first set of test cases for Dynamic Compute Offering feature

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18623/ --- (Updated March 7, 2014, 2:38 p.m.) Review request for cloudstack, Santhosh Eduk

Re: Just checked in a minor change to the DB

Thanks for the heads up Mike, As discussed in your thread [1]. You are eating your own dogfood here. I was checking in a minor change to the upgrade handling yesterday and decided not to bother everyone with it. evil me. In that thread we didn't dicide on a tag, yet. I would suggest [DB-CHANGE] a

RE: Just checked in a minor change to the DB

Hi Mike, This change is for master right? Thanks Rajesh Battala -Original Message- From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Friday, March 7, 2014 1:36 AM To: dev@cloudstack.apache.org Subject: Just checked in a minor change to the DB Hi everyone, Just wanted to

Re: Review Request 18016: CLOUDSTACK-5626: Simplifying VM Migrate code

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18016/ --- (Updated March 7, 2014, 3:06 p.m.) Review request for cloudstack, Girish Shilam

Re: Review Request 18844: Amended review request (origin: https://reviews.apache.org/r/17531/)

no problem, thanks On Thu, Mar 6, 2014 at 11:06 PM, Alex Hitchins wrote: > Ah, sorry - I'll delete this one and add to the original. > > > Regards > > Alex Hitchins > > D: +44 1892 523 587 | S: +44 2036 030 540 | M: +44 7788 423 969 > > alex.hitch...@shapeblue.com > > -Original Message- >

Re: CLOUDSTACK-5641

Apologies for having missed this. Please find details in the bug. Thanks, -Nitin From: Namita Chaudhari mailto:namita.chaudh...@sungard.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Thursday 6 March 2014 11:19 PM To: "dev@

Review Request 18902: CLOUDSTACK-6211: Xenserver - HA - SSVM fails to start due to running out of management Ip ranges when testing host down scenarios

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18902/ --- Review request for cloudstack and Koushik Das. Bugs: CLOUDSTACK-6211 https:

Re: Review Request 18902: CLOUDSTACK-6211: Xenserver - HA - SSVM fails to start due to running out of management Ip ranges when testing host down scenarios

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18902/ --- (Updated March 7, 2014, 11:04 a.m.) Review request for cloudstack and Koushik D

Re: jenkins job for the developer build

Ah, thanks for the tip. On Thu, Mar 6, 2014 at 9:21 AM, Hugo Trippaers wrote: > The Jenkins builds do mvn -Pawsapi,systemvm clean install. > > I'm not using the developer profile there thats just for Marvin and Apidocs. > Those are built separately. You can have a fully functional cloud dev > e

Re: [DISCUSS] realhostip.com going away

Hi all. Are we going to have a solution for older versions like 4.1.1? I think we can already change that domain to something different currently in settings. Hopefully it's not "hardcoded" anywhere else. Is it? I think it's the right thing to move away from such solution in future versions,

Re: Just checked in a minor change to the DB

Good point! :) I should have said what branch it was in. You are correct it was master. On Fri, Mar 7, 2014 at 2:33 AM, Rajesh Battala wrote: > Hi Mike, > This change is for master right? > > Thanks > Rajesh Battala > > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@sol

Re: Just checked in a minor change to the DB

I like either tag. Since DB changes are the most likely to lead to breaking other people's environments, maybe we should go with that one: [DB-CHANGE] Ideally we'd provide the branch, commit SHA, and a manual way around it, if there is something simple (like in my case yesterday, simply deleting

Re: Review Request 17888: Dispatcher corrections, refactoring and tests. Corrects problems from previous attempt

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17888/ --- (Updated March 7, 2014, 3:38 p.m.) Review request for cloudstack, Alena Prokhar

Re: Review Request 17888: Dispatcher corrections, refactoring and tests. Corrects problems from previous attempt

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17888/ --- (Updated March 7, 2014, 3:41 p.m.) Review request for cloudstack, Alena Prokhar

Re: Review Request 17888: Dispatcher corrections, refactoring and tests. Corrects problems from previous attempt

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17888/ --- (Updated March 7, 2014, 3:45 p.m.) Review request for cloudstack, Alena Prokhar

Re: Review Request 17888: Dispatcher corrections, refactoring and tests. Corrects problems from previous attempt

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17888/ --- (Updated March 7, 2014, 3:59 p.m.) Review request for cloudstack, Alena Prokhar

Re: Just checked in a minor change to the DB

Absolutaly, But I meant for us to use both tags. I checked in a commit (that I didn't give a head up on) for a colleague in which a CloudRuntimeException was removed to be more consistent with the rest of the code. It was only effecting th upgrade process in this case but still; [BEHAVIORAL-CHANGE

Re: 4.3 vote

I have a review request sitting at https://reviews.apache.org/r/18392/ - that works for me but I don’t know if it’s breaking AWSAPI functionality. Would love it if somebody more familiar with that module could test. I’d rather not just check that in and see what happens. Animesh, I know you put

RE: Just checked in a minor change to the DB

That's really great idea. In fact long back I had started a similar thread to inform any schema changes so that ppl can changed the schema manually without disturbing the setups. It would be great if all of us follow with [DB-CHANGE] subject tag. Thanks Rajesh Battala -Original Message---

Re: Review Request 18894: CLOUDSTACK-6210 LDAP:listLdapUsers api throws exception when we click on "Add LDAP Account"

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18894/#review36534 --- Ship it! Ship It! - Ian Duffy On March 7, 2014, 6:10 a.m., Rajan

Re: Review Request 18894: CLOUDSTACK-6210 LDAP:listLdapUsers api throws exception when we click on "Add LDAP Account"

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18894/#review36535 --- Commit 4552ec632201e7432afae7770f5854aaa244267c in cloudstack's bran

devcloud2 java

Hi guys, Trying to get back into the Xen devcloud for a few things, and following the instructions it seems that the existing devcloud2 is missing java 7. Do our docs point to an old image, or are we just installing that manually?

Re: Just checked in a minor change to the DB

That sounds good, Daan. Let's use both tags. Thanks! On Fri, Mar 7, 2014 at 9:33 AM, Daan Hoogland wrote: > Absolutaly, > > But I meant for us to use both tags. I checked in a commit (that I > didn't give a head up on) for a colleague in which a > CloudRuntimeException was removed to be more c

Re: Just checked in a minor change to the DB

Does anyone know if our e-mail tags are documented on the Wiki? On Fri, Mar 7, 2014 at 11:36 AM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > That sounds good, Daan. > > Let's use both tags. > > Thanks! > > > On Fri, Mar 7, 2014 at 9:33 AM, Daan Hoogland wrote: > >> Absolutaly, >> >>

Re: devcloud2 java

I manually installed java 7, but I'm beginning to think I've got the wrong version of devcloud, as I can't add the host. Caused by: java.io.IOException: Error during SCP transfer. at com.trilead.ssh2.SCPClient.put(SCPClient.java:592) at com.trilead.ssh2.SCPClient.put(SCPClient.java:547) at com.tri

Re: Review Request 16385: Fix for CloudStack JIRA 4406

On Fri, Mar 7, 2014 at 7:31 PM, Alena Prokharchyk wrote: > And the defaults should be false, I don't agree, The true case does nothing if no fields are recognized as sensitive, but it the flase case skips sensitive data containing log messages. The only consquence of true as default is a perform

Re: Just checked in a minor change to the DB

No, but I do know that it might be useful to use them in commit comments as well. If you so feel inclined you can refer to that as well On Fri, Mar 7, 2014 at 7:41 PM, Mike Tutkowski wrote: > Does anyone know if our e-mail tags are documented on the Wiki? > > > On Fri, Mar 7, 2014 at 11:36 AM, Mi

Re: Just checked in a minor change to the DB

I had a quick look. Nothing found but a lot of hits on tags of course. I'll try to find a place to create the page and start chapters on Dev and Users On Fri, Mar 7, 2014 at 7:54 PM, Daan Hoogland wrote: > No, but I do know that it might be useful to use them in commit > comments as well. If you

Re: Just checked in a minor change to the DB

I created https://cwiki.apache.org/confluence/display/CLOUDSTACK/mail+tags+to+use+to+help+each+other below the path Apache Cloudstack / Home / Community / Getting Help On Fri, Mar 7, 2014 at 8:03 PM, Daan Hoogland wrote: > I had a quick look. Nothing found but a lot of hits on tags of course. >

Re: [DISCUSS] realhostip.com going away

On 07.03.2014 14:55, France wrote: Hi all. Are we going to have a solution for older versions like 4.1.1? I think we can already change that domain to something different currently in settings. Hopefully it's not "hardcoded" anywhere else. Is it? I think it's the right thing to move away from s

Re: Review Request 16385: Fix for CloudStack JIRA 4406

Daan, if the default comes as true for the command, I assume that the user won¹t see the command logged at all? Unless he overrides it. I assume sensitive=³true² means not ³analyze the command² but rather ³don¹t log the command². That doesn¹t seem right to me. True would seem right to me if the pa

Re: [DISCUSS] realhostip.com going away

Hi, The design on https://cwiki.apache.org/confluence/display/CLOUDSTACK/Realhost+IP+changes (big thanks to Kelven!) mentions a third work mode, that would use a load balancer as the front for CPVMs. Since this is an exact address, it would not need a wildcard cert and (hopefully) make it a littl

Re: Review Request 16385: Fix for CloudStack JIRA 4406

Alena, I can see I am not being clear because what you say is the sensible way and apart from the parameter level exactly what happens. The parameter thing is an enhancement that we can make on top of this. At the moment it only obfuscate a set of parameters with a fixed set of names. We will have

Re: Review Request 16385: Fix for CloudStack JIRA 4406

Ok, got it, somehow missed the “hardcoded” parameters part. In this case true is fine as the parameter in @ApiCommand just triggers the validation. We only have to fix one part - instead of hardcoding the parameter(s) to hide, we have to come up with the new parameter in @Parameter to trigger the e

Re: Just checked in a minor change to the DB

Great! Thanks, Daan I also agree that it would be useful info to have in commit messages. On Fri, Mar 7, 2014 at 12:14 PM, Daan Hoogland wrote: > I created > https://cwiki.apache.org/confluence/display/CLOUDSTACK/mail+tags+to+use+to+help+each+other > below the path > Apache Cloudstack / Home /

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

Any suggestion? Do we go forward assuming that the correct parameter for resize on deploy is: deployVirtualMachine&details[0].rootdisksize=3 or do we change it to deployVirtualMachine&rootdisksize=3 On Tue, Mar 4, 2014 at 4:14 PM, Marcus wrote: > Ok, sounds like there needs to be some work don

Re: [DISCUSS] realhostip.com going away

You could use this article as well: http://support.citrix.com/article/CTX133468 Does not require you to use the java solution. On 3/7/14, 11:19 AM, "Nux!" wrote: >On 07.03.2014 14:55, France wrote: >> Hi all. >> >> Are we going to have a solution for older versions like 4.1.1? >> I think we can

Re: [DISCUSS] realhostip.com going away

Soo…I’d recommend against something like Nux’s suggestion below. I’ve only looked briefly at VirtualDNS.java, and it looks fine from a glance, but I’m willing to bet I can a) DOS it, and b) use it for a reflection attack. I could be wrong, don’t really have time to look closely, but based on it

Re: Just checked in a minor change to the DB

I went ahead and added to the Wiki page you created a few more tags that I've seen us use from time to time. On Fri, Mar 7, 2014 at 12:37 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Great! Thanks, Daan > > I also agree that it would be useful info to have in commit messages. > >

Re: [DISCUSS] realhostip.com going away

On 07.03.2014 20:28, John Kinsella wrote: Soo…I’d recommend against something like Nux’s suggestion below. I’ve only looked briefly at VirtualDNS.java, and it looks fine from a glance, but I’m willing to bet I can a) DOS it, and b) use it for a reflection attack. I could be wrong, don’t really ha

Re: [DISCUSS] realhostip.com going away

On 07.03.2014 20:19, Chiradeep Vittal wrote: You could use this article as well: http://support.citrix.com/article/CTX133468 Does not require you to use the java solution. Thanks, I've updated my post with it. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro

Build failed in Jenkins: build-master-slowbuild #379

See Changes: [Daan Hoogland] Dispatcher corrections, refactoring and tests [Daan Hoogland] rats [Alena Prokharchyk] Removed unused methods from BaseCmd class. Moved some helper methods to AccountManagerImpl class [brian.f

RE: 4.3 vote

> -Original Message- > From: John Kinsella [mailto:j...@stratosec.co] > Sent: Friday, March 07, 2014 8:41 AM > To: dev@cloudstack.apache.org > Subject: Re: 4.3 vote > > I have a review request sitting at https://reviews.apache.org/r/18392/ - that > works for me but I don't know if it's b

Re: [DISCUSS] realhostip.com going away

You should require 1 record for the ACTUAL public IP used by the ConsoleProxy VM. On 3/7/14, 12:39 PM, "Nux!" wrote: >On 07.03.2014 20:28, John Kinsella wrote: >> SooŠI¹d recommend against something like Nux¹s suggestion below. I¹ve >> only looked briefly at VirtualDNS.java, and it looks fine fr

Re: [DISCUSS] realhostip.com going away

On 07.03.2014 21:30, Chiradeep Vittal wrote: You should require 1 record for the ACTUAL public IP used by the ConsoleProxy VM. I know, but if the VM is destroyed or an additional one is spawned it could have ANY IP.. I mean, I really don't want to track this kind of stuff, doesn't "scale". Ma

Re: Review Request 16385: Fix for CloudStack JIRA 4406

no problem, glad we agree. On Fri, Mar 7, 2014 at 8:38 PM, Alena Prokharchyk wrote: > Ok, got it, somehow missed the "hardcoded" parameters part. In this case > true is fine as the parameter in @ApiCommand just triggers the validation. > We only have to fix one part - instead of hardcoding the pa

Re: Just checked in a minor change to the DB

saw, good, will do some more if I think of them as well, as well as add to the descriptions On Fri, Mar 7, 2014 at 9:29 PM, Mike Tutkowski wrote: > I went ahead and added to the Wiki page you created a few more tags that > I've seen us use from time to time. > > > On Fri, Mar 7, 2014 at 12:37 PM,

Re: devcloud2 java

Marcus, I saw a template definition in the tools/appliance directory for devcloud. I have build it succesfully but never ran it yet. Millage unknown. Daan On Fri, Mar 7, 2014 at 7:46 PM, Marcus wrote: > I manually installed java 7, but I'm beginning to think I've got the > wrong version of devcl

Re: Review Request 16385: Fix for CloudStack JIRA 4406

And here is the Jira ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-6213 "Add new field to API @Parameter indicating if the param should be skipped from logs” -Alena. On 3/7/14, 1:47 PM, "Daan Hoogland" wrote: >no problem, glad we agree. > >On Fri, Mar 7, 2014 at 8:38 PM, Alena Prok

Re: Review Request 16385: Fix for CloudStack JIRA 4406

Mandar, you want to take it? On Fri, Mar 7, 2014 at 11:12 PM, Alena Prokharchyk wrote: > And here is the Jira ticket: > > https://issues.apache.org/jira/browse/CLOUDSTACK-6213 > > "Add new field to API @Parameter indicating if the param should be skipped > from logs" > > -Alena. > > On 3/7/14, 1:

Fwd: Build failed in Jenkins: cloudstack-rat-master #4166

Hi, I'm not that familiar with reading these reports, but it claims there was a Jenkins build failure due to code I checked in. I ran a full build locally using mvn -P developer,systemvm clean install and then tested before checking the code in. Perhaps there is a regression test on Jenkins that

Re: Build failed in Jenkins: cloudstack-rat-master #4166

There is a RAT build failure. Check the link [1] for the file that misses license header. Checking whether RAT build works for your machine at [2] [1] https://builds.apache.org/job/cloudstack-rat-master/4166/artifact/target/ra t.txt [2] https://cwiki.apache.org/confluence/display/CLOUDSTACK/How+

Re: Build failed in Jenkins: cloudstack-rat-master #4166

Link says this new file is missing license: Unapproved licenses: api/src/com/cloud/offering/DiskOfferingInfo.java On Fri, Mar 7, 2014 at 4:14 PM, Nitin Mehta wrote: > There is a RAT build failure. Check the link [1] for the file that misses > license header. Checking whether RAT build works f

libvirt-java upgrade

Wido, I'm seeing this in Ubuntu 12.04 after commit 2014-02-10 01:19:16,793 DEBUG [kvm.resource.LibvirtComputingResource] (agentRequest-Handler-2:null) Volume /mnt/2fe9a944-505e-38cb-bf87-72623634be4a/e47e6501-c8ae-41a7-9abc-0f7fdad5fb30 can be resized by libvirt. Asking libvirt to resize the

Re: libvirt-java upgrade

Hrm... sent instead of pasted. Commit commit 3989d6c48118f31464c87c71b6279a11eb13eb35 Author: Wido den Hollander Date: Mon Feb 3 17:04:11 2014 +0100 kvm: Resize volumes using libvirt virsh blockresize works on this system, so I can only assume that the libvirt.so.0.9.8 that ships with Ubu

master issue calling volume resize via libvirt - Was: libvirt-java upgrade

On Fri, Mar 7, 2014 at 4:30 PM, Marcus wrote: > Hrm... sent instead of pasted. Commit > > commit 3989d6c48118f31464c87c71b6279a11eb13eb35 > Author: Wido den Hollander > Date: Mon Feb 3 17:04:11 2014 +0100 > > kvm: Resize volumes using libvirt > > virsh blockresize works on this system, so I

Re: Build failed in Jenkins: cloudstack-rat-master #4166

Great, guys - thanks! That makes sense. I added a small class to wrap some commonly passed parameters and didn't add the license header. On Fri, Mar 7, 2014 at 4:21 PM, Marcus wrote: > Link says this new file is missing license: > > Unapproved licenses: > > api/src/com/cloud/offering/DiskOff

Re: Build failed in Jenkins: cloudstack-rat-master #4166

It would make sense to have one single command which could do it all for the devs, before committing code. On 07/03/14 3:41 PM, "Mike Tutkowski" wrote: >Great, guys - thanks! > >That makes sense. I added a small class to wrap some commonly passed >parameters and didn't add the license header. >

Build failed in Jenkins: build-master-slowbuild #380

See Changes: [mike.tutkowski] CLOUDSTACK-6170 -- [...truncated 18117 lines...] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 [INFO] [INFO] <<< cobertura-maven-plugin:2.6:cobertur

unable to schedule async job, duplicate primary key

I've been randomly hitting this with my master testing, and I'm unsure if it's something strange in my test environment, or if somebody has been working on the async job handler and there's some sort of race condition. I saw it first when trying to destroy a vm, then during a test run with volume d

Re: [DISCUSS] realhostip.com going away

I planned on looking at the Lua part of PowerDNS for this :-) -- Erik Weber 7. mars 2014 22:48 skrev "Nux!" følgende: > On 07.03.2014 21:30, Chiradeep Vittal wrote: > >> You should require 1 record for the ACTUAL public IP used by the >> ConsoleProxy VM. >> > > I know, but if the VM is destroye

cherry-pick to 4.3 request

Animesh, could you please cherry-pick one more commit for VPC, from forward to 4.3 branch? 164ea3e84f6f282006e66725f22cd2246f0be8f8 CloudStackCLOUDSTACK-6214 VPC: when guest network is in Setup state, on its initial nicPlug to the VR, corresponding network rules are not getting applied -Alena.

Build failed in Jenkins: build-master-slowbuild #381

See Changes: [mike.tutkowski] Adding license header [nitin.mehta] CLOUDSTACK-5478: Enable publishing uuid for all the async apis in the CallContext. -- [...truncated 18097 lines...]

RE: Just checked in a minor change to the DB

Just a thought, instead of sending mail manually, can't we have a git hook to send out mail to dev list when there is a commit to schema files. So that git will take care of informing the dev about new schema changes.! Thanks Rajesh Battala -Original Message- From: Daan Hoogland [mailto:

Build failed in Jenkins: build-master-slowbuild #382

See -- [...truncated 18097 lines...] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 [INFO] [INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ cloud-awsapi <<< [INFO] [INFO] -

Re: Just checked in a minor change to the DB

We could, but some schema changes won't impact others. For example, if I removed a column and took out the corresponding Java code and you updated your code, the new code wouldn't fail against your old schema. People might also update the schema and check it in, but not update the Java code until

system vm disk space issue in ACS 4.3

Hi All, I am seeing system vm disk has no space left after running for few days. Cloudstack UI shows the agent in v-2-VM in alert state, while agent state of s-1-VM shows blank (hyphen in the UI). Both the system vms are running and ssh-able from the host. The log in s-1-Vm shows following erro

RE: system vm disk space issue in ACS 4.3

AFAIK, log roation is enabled in the systemvm. Can you check whether the logs are getting zipped .? -Original Message- From: Anirban Chakraborty [mailto:abc...@juniper.net] Sent: Saturday, March 8, 2014 12:46 PM To: dev@cloudstack.apache.org Subject: system vm disk space issue in ACS 4.3