Re: [ANNOUNCE] Switched to Java 7

2014-01-30 Thread Hugo Trippaers
Heya,

All Java 7 related issues should be resolved by now for the master builds. I’ve 
created a number of JDK settings in Jenkins and each build can now select which 
JVM to use.

Cheers,

Hugp


On 30 jan. 2014, at 06:49, Prasanna Santhanam t...@apache.org wrote:

 On Mon, Jan 27, 2014 at 03:03:09PM +0100, Hugo Trippaers wrote:
 I have some trouble getting into the rpmbuilders, so i can?t
 configure those. I need root permissions for those boxes and i don?t
 have that.
 
 
 Hey Hugo - just saw this. I see the build failed in the last run but
 can't tell if it used Java7 JDK or not on rpmbuilder-2. Did you get a
 chance to fix this?
 
 The cloudstack-slaves will be fixed to support jdk 7. 
 
 
 -- 
 Prasanna.,
 
 
 Powered by BigRock.com
 



Re: cloudstack-agent on ubuntu

2014-01-30 Thread Wido den Hollander

On 01/29/2014 08:24 PM, Anirban Chakraborty wrote:

Hi All,

I am trying to install cloudstack-agent on ubuntu 12.04 and it is failing with 
openjdk-6-jre dependency. Following is the error message:

The following packages have unmet dependencies:
  cloudstack-agent : Depends: openjdk-6-jre but it is not installable or
  openjdk-7-jre but it is not going to be installed
 Depends: cloudstack-common (= 4.2.1) but it is not going 
to be installed
 Depends: ethtool but it is not installable
E: Unable to correct problems, you have held broken packages.
——
Has any one seen this, and if so, how do I get around it? Thanks much.



Can you verify if you can install ethtool manually on your machine?

Would be weird if you couldn't, but still worth checking out.

Wido


Anirban Chakraborty





Re: cloudstack-agent on ubuntu

2014-01-30 Thread Wei ZHOU
ethtool is used
in 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
so this package is required in cloudstack-agent installation.
If you use dpkg -i to install the deb packages and meet error
about dependencies, you can use apt-get install or apt-get install
ethtool to fix it.


2014-01-29 Anirban Chakraborty abc...@juniper.net

 Hi All,

 I am trying to install cloudstack-agent on ubuntu 12.04 and it is failing
 with openjdk-6-jre dependency. Following is the error message:
 
 The following packages have unmet dependencies:
  cloudstack-agent : Depends: openjdk-6-jre but it is not installable or
  openjdk-7-jre but it is not going to be
 installed
 Depends: cloudstack-common (= 4.2.1) but it is not
 going to be installed
 Depends: ethtool but it is not installable
 E: Unable to correct problems, you have held broken packages.
 
 Has any one seen this, and if so, how do I get around it? Thanks much.

 Anirban Chakraborty



4.3 commit cherry pick

2014-01-30 Thread Jayapal Reddy Uradi
Hi Animesh,

Can you please cherry pick commit to 7255e50f2051aff1cb5e1991e2dde4ef48f57454 
4.3 branch 
from 4.3-forward.

Thanks,
Jayapal

[DISCUSS][findbugs] proposing fixes or just do it

2014-01-30 Thread Daan Hoogland
H Alex and others,

I hesitate to ask you as it seems obvious, but in the awsapi project
in com.cloud.bridge.model.SObjectItem.equals(Object other) a
comparison is made
if 
(!theObject.getId().equals(((SObjectItemVO)other).getTheObject()))
which i think should be
if 
(!theObject.getId().equals(((SObjectItemVO)other).getTheObject().getId()))
The obvious question would be, 'am i correct?', or 'can i make the change?'.

Now my real question is should I just go ahead and fix the scariest
bugs that findbugs comes up with or should I consult with package
maintainers for each part of the system on errors like these?

(to comfort all of you, i restricted myself to doing this in master)

regards,
Daan


Re: Add new KVM hypervisor host into CloudStack

2014-01-30 Thread Nux!

On 30.01.2014 08:57, Indra Pramana wrote:

Hi,

It's time for me to add new KVM hypervisor host into our CloudStack 
and I

read these requirements:

http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.2.0/html/Installation_Guide/hypervisor-kvm-install-flow.html
- Same distribution version, is it referring to same OS and version? 
E.g.

Ubuntu 12.04 (Precise)?
- All hosts must be homogenous, does it mean that the new server has 
to be
exactly the same chassis, motherboard, type of CPU and RAM? Can I use 
a
different type of chassis and motherboard, but use the same type of 
CPU?


Thank you.


Yes, I think the CPU is the important bit here and imho it should be 
equal or better than what you have if you want certain stuff like live 
migration to work. This is especially important if you use 
host-passthrough for CPU type. I do not think anyone expect you to have 
indentical servers throughout your setup; it's normal to bring in better 
servers as time passes.


Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


RE: API documentation

2014-01-30 Thread Paul Angus
Hey Radhika,

Yes, I'm happy to do what I can (probably mainly harassing other people)

CloudMonkey says that there are 431 API calls (in 4.3 RC1) I recon there are 
409 listed on 
http://cloudstack.apache.org/docs/api/apidocs-4.2/TOC_Root_Admin.html

I'll try to find which ones are missing.

I think in terms of the API documentation page, the first thing to do is put 
this headings in alphabetical order!

Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
paul.an...@shapeblue.com

-Original Message-
From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com]
Sent: 30 January 2014 07:48
To: dev@cloudstack.apache.org
Subject: RE: API documentation

Hi Paul,

Yes, API documentation requires a serious revisit. I had initiated a discussion 
months back, and invited volunteers.

A wiki page is created at 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Documentation+Sprint+for+Enhancing+API+Documentation
, but nothing beyond that.

If you are interested to volunteer, let's begin.

-Radhika

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Thursday, January 30, 2014 12:43 PM
To: dev@cloudstack.apache.org
Subject: API documentation

DEVs,

New features are generally cool, but given that the 'engine' of CloudStack is 
the API it seems ridiculous that as a minimum the API documentation isn't kept 
up to date.

The specific example that I'm thinking of is

addvmwaredc

Only by using the UI and following the API calls sent could I find that it 
existed, and then tracked it down in the design documents in the wiki.

I've filed a bug for this one 
(https://issues.apache.org/jira/browse/CLOUDSTACK-5984)

But please can everyone make sure we can all use the features you create by 
documenting them.

Regards

Paul Angus
Senior Consultant / Cloud Architect

[cid:image002.png@01CE1071.C6CC9C10]

S: +44 20 3603 0540tel:+442036030540 | M: +4tel:+44796816158147711418784 | 
T: @CloudyAngus paul.an...@shapeblue.commailto:paul.an...@shapeblue.com | 
www.shapeblue.comhttp://www.shapeblue.com/ | 
Twitter:@shapebluehttps://twitter.com/
ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS

Need Enterprise Grade Support for Apache CloudStack?
Our CloudStack Infrastructure 
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers the 
best 24/7 SLA for CloudStack Environments.

Apache CloudStack Bootcamp training courses

**NEW!** CloudStack 4.2.1 traininghttp://shapeblue.com/cloudstack-training/
18th-19th February 2014, Brazil. 
Classroomhttp://shapeblue.com/cloudstack-training/
17th-23rd March 2014, Region A. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/
24th-28th March 2014, Region B. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/
16th-20th June 2014, Region A. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/
23rd-27th June 2014, Region B. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England  Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue is a registered trademark.
This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England  Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue is a registered trademark.


Re: BVT report on 4.3 branch

2014-01-30 Thread Daan Hoogland
H Talluri,

in the report summary for xen it says
Fail :5
but i only see these 3
test_loadbalance/ 0   3   0

am i missing something? (the kvm failures seem to match the summary)

when I folow the links I see 12 and 3 failures respectively. I'm sure
I'm just being paranoid but can you explain, please?

On Thu, Jan 30, 2014 at 6:40 AM, Prasanna Santhanam t...@apache.org wrote:
 This basically means a +1 on 4.3? I think there were couple of issues
 that emerged in the RC particularly the missing response params in the
 list* API that Alena reported. We should probably include a test for
 that in the integration suite.

 I would've done it had I had some bandwidth, but thought I'd bring
 this up as a way to backfill missing tests.

 On Wed, Jan 29, 2014 at 12:14:18PM +, Srikanteswararao Talluri wrote:
 Here is the report of automated smoke tests results on Xen and KVM.
 
 On Xenserver 6.2/Test Run: #1174 and 1175
 
 This report is consolidation of runs #1174 and #1175
 3 failures are not product failures but ssh time outs in the scripts.
 
 Link to reports:
 http://jenkins.buildacloud.org/view/cloudstack-qa-4.3/job/test-smoke-matri
 x-4.3/1174/testReport/
 
 http://jenkins.buildacloud.org/view/cloudstack-qa-4.3/job/test-smoke-matri
 x-4.3/1175/testReport/
 
 Total:97
 Fail :5
 Skip :2
 
 
 name   passfailskip
 test_public_ip_range/ 1   0   0
 test_nic/ 1   0   0
 test_global_settings/ 1   0   0
 test_deploy_vms_with_varied_deploymentplanners/   3   0   0
 test_regions/ 1   0   0
 test_ssvm/   10   0   0
 test_vpc_vpn/ 2   0   0
 test_affinity_groups/ 1   0   0
 test_deploy_vm/   1   0   0
 test_deploy_vm_with_userdata/ 2   0   0
 test_routers/ 9   0   0
 test_loadbalance/ 0   3   0
 test_portable_publicip/   2   0   0
 test_iso/ 5   0   1
 test_internal_lb/ 1   0   0
 test_vm_snapshots/3   0   0
 test_network/ 7   0   0
 test_disk_offerings/  3   0   0
 test_non_contigiousvlan/  1   0   0
 test_templates/   7   0   1
 test_network_acl/ 1   0   0
 test_multipleips_per_nic/ 1   0   0
 test_reset_vm_on_reboot/  1   0   0
 test_scale_vm/1   0   0
 test_pvlan/   1   0   0
 test_vm_life_cycle/  10   0   0
 test_privategw_acl/   1   0   0
 test_resource_detail/ 1   0   0
 test_guest_vlan_range/1   0   0
 test_volumes/ 9   0   0
 test_service_offerings/   4   0   0
 
 
 
 On KVM /RHEL 6.3 hosts/ Test Run: #1173
 ---
 4 of the test failures are not valid because snapshots and scaling VM are
 not supported on KVM Hypervisor
 
 Link to report:
 http://jenkins.buildacloud.org/job/test-smoke-matrix-4.3/1173/testReport/
 
 
 Total:97
 Fail :8
 Skip :2
 
 
 name   passfailskip
 test_public_ip_range/ 1   0   0
 test_nic/ 1   0   0
 test_global_settings/ 1   0   0
 test_deploy_vms_with_varied_deploymentplanners/   3   0   0
 test_regions/ 1   0   0
 test_ssvm/   10   0   0
 test_vpc_vpn/ 2   0   0
 test_affinity_groups/ 1   0   0
 test_deploy_vm/   1   0   0
 test_deploy_vm_with_userdata/ 2   0   0
 test_routers/ 9   0   0
 test_loadbalance/ 0   3   0
 

RE: [VOTE] Apache CloudStack 4.3.0 (second round)

2014-01-30 Thread Nux!

On 29.01.2014 22:52, Geoff Higginbottom wrote:

-1

I am unable to manage Firewall, Port Forwarding or Load Balancing
rules via the UI, the interfaces no longer exist.  I did create a Port
Forwarding Rule via the API successfully to prove it's just a UI
issue.

VMs assigned to VPC's fail to deploy and are then deleted.

I was unable to complete my remaining tests as these required a
working VPC and the ability to create Firewall, Port Forwarding and
Load Balancing Rules for Isolated Networks.


Yeah, there are some problems with the new UI. I've also reported some 
issues:

https://issues.apache.org/jira/browse/CLOUDSTACK-5906
https://issues.apache.org/jira/browse/CLOUDSTACK-5907 (not UI related)


--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro


Re: Findbugs report on 4.3-forward

2014-01-30 Thread Daan Hoogland
Sorry Rajani,

I had seen no reaction to Ian's explenation  and the request by
Animesh to pull it so I just did. let me look into it for a minute

On Thu, Jan 30, 2014 at 5:36 AM, Rajani Karuturi
rajani.karut...@citrix.com wrote:
 I see that the commit 9776e1af1c92486f5081b1ee8fa95cf0edb86b97 is already 
 pushed to 4.3. I don't see any response on my concern as well.
 Is it just me or anyone else sees a security issue with the generate password 
 change?
 Ian/Animesh/Daan, can you please respond?

 Thanks,
 ~Rajani



 On 29-Jan-2014, at 10:59 am, Rajani Karuturi rajani.karut...@citrix.com 
 wrote:

 Hi Ian,
 Before it is pushed to 4.3, can you fix the generate password change like i 
 suggested in the other mail? This current change would make it less secure.

 Thanks,
 ~Rajani



 On 29-Jan-2014, at 8:03 am, Ian Duffy i...@ianduffy.ie wrote:

 Hi Animesh,

 Tested all those changes to detail. Those lines were removed due to
 unexpected behavior that I had not spotted until now.

 They were suppose to allow for better fall over between multiple domain
 controllers, how ever they were causing caching to occur. This meant if a
 users password was reset in LDAP the old password was still allowing login
 for a limited time.

 Please pull the changes forward,
 Thanks

 Ian.
 On 29 Jan 2014 00:07, Animesh Chaturvedi animesh.chaturv...@citrix.com
 wrote:

 If I look at this commit for example


 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=92b4f66d73562e4211d2d787554ff229dbeb5705

 It removes the two lines from LdapContextFactory.java

 environment.put(com.sun.jndi.ldap.read.timeout, 500);-
 environment.put(com.sun.jndi.ldap.connect.pool, true);

 Is that reported by find bug? I don't know this code  so not sure if it is
 intentional or not ?

 The point is there may be unintended risks in allowing late changes.



 -Original Message-
 From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
 Sent: Tuesday, January 28, 2014 3:35 PM
 To: dev@cloudstack.apache.org
 Subject: RE: Findbugs report on 4.3-forward

 Are you sure all of the ones are needed. A quick look at 20+ commits from
 Daan  show many formatting changes that may not be necessary and hinder
 quick review.

 -Original Message-
 From: Hugo Trippaers [mailto:trip...@gmail.com]
 Sent: Tuesday, January 28, 2014 3:16 PM
 To: dev@cloudstack.apache.org
 Subject: Re: Findbugs report on 4.3-forward



 Sent from my iPhone

 On 28 jan. 2014, at 23:50, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com wrote:


 -Original Message-
 From: Hugo Trippaers [mailto:trip...@gmail.com]
 Sent: Tuesday, January 28, 2014 2:37 PM
 To: dev@cloudstack.apache.org
 Cc: dev@cloudstack.apache.org
 Subject: Re: Findbugs report on 4.3-forward

 Hey Animesh,

 Not in agreement here. These are squashed bugs and we want as less bugs
 in the release as possible.
 [Animesh] I understand but once we enter RC phase we only limit
 important fixes. I have pulled in  2 commits from yours and 1 from Daan.

 We limited our fixes to only the important issues that we found. The other
 6000 issues between coverity and findbugs are still being triaged and will
 probably not make it into this release.


 This is why we test any RC before we release it.
 [Animesh] Of course but timing is a bit off, if this was done a month
 back it would have been fine.
 I say include all the big fixes we have in the release. If that means
 more testing before we cut the RC then that is what it is. I can't
 rightfully vote for a release with known issues with existing fixes.
 [Animesh] Any release will have known issues, if we have fixes but can't
 be sure on regression impact then we have to make a choice.

 Agreed, but we just don't agree on what that choice should be yet ;-)

 Quality over release schedule would be my vote then.
 [Animesh] But why so late? Why was this activity not planned early on? I
 have been reminding community to call out issues early on since like
 mid-December.

 On November 4 I sent the mail to the dev list that static code analysis
 (coverity) found 6000+ issues that needed to be triaged. I worked on quite
 a few with my colleagues, but it's a big task for just the four of us.
 Findbugs just helped us to quickly identify the real scary issues among
 them.
 So I agree that the timing is less than ideal, but we should do our utmost
 best to ship the highest quality piece of software we can.


 Cheers,
 Hugo

 Sent from my iPhone

 On 28 jan. 2014, at 18:48, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com wrote:

 Folks these issues reported by find-bugs have existed for some time. I
 am not confident in picking them up now for 4.3 as it may break code that
 assumed old way of working. We can take them up for 4.3 maintenance
 release. I wish we had done this exercise and not waited until now.

 I will pick Hugo's commit for which he called -1.

 Thanks
 Animesh

 -Original Message-
 From: Trippie 

Re: API documentation

2014-01-30 Thread Daan Hoogland
Hey Paul,

Great that you are concerned with the documentation. I don't think you
can expect that people implementing stuff because they need it can be
expected to document it. Unfortunately the second person to use a new
feature falls the responsibility to interrogate the guilty party and
produce a report on the results thereof. I say unfortunately with kind
of a smile as I think it is a natural way to 'grow' software. So
please harass me if my name comes up in git blame.

regards,
lazy dev

On Thu, Jan 30, 2014 at 10:53 AM, Paul Angus paul.an...@shapeblue.com wrote:
 Hey Radhika,

 Yes, I'm happy to do what I can (probably mainly harassing other people)

 CloudMonkey says that there are 431 API calls (in 4.3 RC1) I recon there are 
 409 listed on 
 http://cloudstack.apache.org/docs/api/apidocs-4.2/TOC_Root_Admin.html

 I'll try to find which ones are missing.

 I think in terms of the API documentation page, the first thing to do is put 
 this headings in alphabetical order!

 Regards,

 Paul Angus
 Cloud Architect
 S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
 paul.an...@shapeblue.com

 -Original Message-
 From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com]
 Sent: 30 January 2014 07:48
 To: dev@cloudstack.apache.org
 Subject: RE: API documentation

 Hi Paul,

 Yes, API documentation requires a serious revisit. I had initiated a 
 discussion months back, and invited volunteers.

 A wiki page is created at 
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Documentation+Sprint+for+Enhancing+API+Documentation
 , but nothing beyond that.

 If you are interested to volunteer, let's begin.

 -Radhika

 From: Paul Angus [mailto:paul.an...@shapeblue.com]
 Sent: Thursday, January 30, 2014 12:43 PM
 To: dev@cloudstack.apache.org
 Subject: API documentation

 DEVs,

 New features are generally cool, but given that the 'engine' of CloudStack is 
 the API it seems ridiculous that as a minimum the API documentation isn't 
 kept up to date.

 The specific example that I'm thinking of is

 addvmwaredc

 Only by using the UI and following the API calls sent could I find that it 
 existed, and then tracked it down in the design documents in the wiki.

 I've filed a bug for this one 
 (https://issues.apache.org/jira/browse/CLOUDSTACK-5984)

 But please can everyone make sure we can all use the features you create by 
 documenting them.

 Regards

 Paul Angus
 Senior Consultant / Cloud Architect

 [cid:image002.png@01CE1071.C6CC9C10]

 S: +44 20 3603 0540tel:+442036030540 | M: +4tel:+44796816158147711418784 
 | T: @CloudyAngus paul.an...@shapeblue.commailto:paul.an...@shapeblue.com | 
 www.shapeblue.comhttp://www.shapeblue.com/ | 
 Twitter:@shapebluehttps://twitter.com/
 ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS

 Need Enterprise Grade Support for Apache CloudStack?
 Our CloudStack Infrastructure 
 Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers the 
 best 24/7 SLA for CloudStack Environments.

 Apache CloudStack Bootcamp training courses

 **NEW!** CloudStack 4.2.1 traininghttp://shapeblue.com/cloudstack-training/
 18th-19th February 2014, Brazil. 
 Classroomhttp://shapeblue.com/cloudstack-training/
 17th-23rd March 2014, Region A. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/
 24th-28th March 2014, Region B. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/
 16th-20th June 2014, Region A. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/
 23rd-27th June 2014, Region B. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/

 This email and any attachments to it may be confidential and are intended 
 solely for the use of the individual to whom it is addressed. Any views or 
 opinions expressed are solely those of the author and do not necessarily 
 represent those of Shape Blue Ltd or related companies. If you are not the 
 intended recipient of this email, you must neither take any action based upon 
 its contents, nor copy or show it to anyone. Please contact the sender if you 
 believe you have received this email in error. Shape Blue Ltd is a company 
 incorporated in England  Wales. ShapeBlue Services India LLP is a company 
 incorporated in India and is operated under license from Shape Blue Ltd. 
 Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
 operated under license from Shape Blue Ltd. ShapeBlue is a registered 
 trademark.
 This email and any attachments to it may be confidential and are intended 
 solely for the use of the individual to whom it is addressed. Any views or 
 opinions expressed are solely those of the author and do not necessarily 
 represent those of Shape Blue Ltd or related companies. If you are not the 
 intended recipient of this email, you must neither take any action based upon 
 its contents, nor copy or show it to anyone. Please contact the sender if you 
 believe you have received this email in error. 

Re: API documentation

2014-01-30 Thread sebgoa

On Jan 30, 2014, at 10:53 AM, Paul Angus paul.an...@shapeblue.com wrote:

 Hey Radhika,
 
 Yes, I'm happy to do what I can (probably mainly harassing other people)
 
 CloudMonkey says that there are 431 API calls (in 4.3 RC1) I recon there are 
 409 listed on 
 http://cloudstack.apache.org/docs/api/apidocs-4.2/TOC_Root_Admin.html
 

We talked about this recently, the discrepancy seems to come from the fact that 
the apidocs are built for the oss dist and that the missing apis are from the 
nonredist build.

-sebastien

 I'll try to find which ones are missing.
 
 I think in terms of the API documentation page, the first thing to do is put 
 this headings in alphabetical order!
 
 Regards,
 
 Paul Angus
 Cloud Architect
 S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
 paul.an...@shapeblue.com
 
 -Original Message-
 From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com]
 Sent: 30 January 2014 07:48
 To: dev@cloudstack.apache.org
 Subject: RE: API documentation
 
 Hi Paul,
 
 Yes, API documentation requires a serious revisit. I had initiated a 
 discussion months back, and invited volunteers.
 
 A wiki page is created at 
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Documentation+Sprint+for+Enhancing+API+Documentation
 , but nothing beyond that.
 
 If you are interested to volunteer, let's begin.
 
 -Radhika
 
 From: Paul Angus [mailto:paul.an...@shapeblue.com]
 Sent: Thursday, January 30, 2014 12:43 PM
 To: dev@cloudstack.apache.org
 Subject: API documentation
 
 DEVs,
 
 New features are generally cool, but given that the 'engine' of CloudStack is 
 the API it seems ridiculous that as a minimum the API documentation isn't 
 kept up to date.
 
 The specific example that I'm thinking of is
 
 addvmwaredc
 
 Only by using the UI and following the API calls sent could I find that it 
 existed, and then tracked it down in the design documents in the wiki.
 
 I've filed a bug for this one 
 (https://issues.apache.org/jira/browse/CLOUDSTACK-5984)
 
 But please can everyone make sure we can all use the features you create by 
 documenting them.
 
 Regards
 
 Paul Angus
 Senior Consultant / Cloud Architect
 
 [cid:image002.png@01CE1071.C6CC9C10]
 
 S: +44 20 3603 0540tel:+442036030540 | M: +4tel:+44796816158147711418784 
 | T: @CloudyAngus paul.an...@shapeblue.commailto:paul.an...@shapeblue.com | 
 www.shapeblue.comhttp://www.shapeblue.com/ | 
 Twitter:@shapebluehttps://twitter.com/
 ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
 
 Need Enterprise Grade Support for Apache CloudStack?
 Our CloudStack Infrastructure 
 Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers the 
 best 24/7 SLA for CloudStack Environments.
 
 Apache CloudStack Bootcamp training courses
 
 **NEW!** CloudStack 4.2.1 traininghttp://shapeblue.com/cloudstack-training/
 18th-19th February 2014, Brazil. 
 Classroomhttp://shapeblue.com/cloudstack-training/
 17th-23rd March 2014, Region A. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/
 24th-28th March 2014, Region B. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/
 16th-20th June 2014, Region A. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/
 23rd-27th June 2014, Region B. Instructor led, 
 On-linehttp://shapeblue.com/cloudstack-training/
 
 This email and any attachments to it may be confidential and are intended 
 solely for the use of the individual to whom it is addressed. Any views or 
 opinions expressed are solely those of the author and do not necessarily 
 represent those of Shape Blue Ltd or related companies. If you are not the 
 intended recipient of this email, you must neither take any action based upon 
 its contents, nor copy or show it to anyone. Please contact the sender if you 
 believe you have received this email in error. Shape Blue Ltd is a company 
 incorporated in England  Wales. ShapeBlue Services India LLP is a company 
 incorporated in India and is operated under license from Shape Blue Ltd. 
 Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
 operated under license from Shape Blue Ltd. ShapeBlue is a registered 
 trademark.
 This email and any attachments to it may be confidential and are intended 
 solely for the use of the individual to whom it is addressed. Any views or 
 opinions expressed are solely those of the author and do not necessarily 
 represent those of Shape Blue Ltd or related companies. If you are not the 
 intended recipient of this email, you must neither take any action based upon 
 its contents, nor copy or show it to anyone. Please contact the sender if you 
 believe you have received this email in error. Shape Blue Ltd is a company 
 incorporated in England  Wales. ShapeBlue Services India LLP is a company 
 incorporated in India and is operated under license from Shape Blue Ltd. 
 Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
 operated under license 

Re: RSA Keys Deployment on Windows

2014-01-30 Thread Chiradeep Vittal
Looks specific to Windows. What is your user id on the Wiki?

On 1/29/14 4:22 PM, Alex Hitchins alex.hitch...@shapeblue.com wrote:

All,

I've rebuilt my development environment from the 4.3 branch. In
rebuilding, I hit upon a number of errors namely around ssh keys
(id_rsa.cloud  id_rsa.cloud.pub). When running the Jetty command there
were errors due to the file not existing. It then later looks for the
file it's tried to create to insert this into the database.

I manually created the keys as the mvn script was set to use ssh-keygen
under /bin/bash - also, as the folder was named beginning with a . I
had to create the directory with admin permissions.

Not sure if this is a bug or just needs writing up in the dev wiki under
the setting up a windows environment. Happy to do so if I have the
permissions to edit the site. If not, I'll blog about it so someone can
find some help in the future.


Alex Hitchins
--
E: a...@alexhitchins.commailto:a...@alexhitchins.com
M: 07788 423 969


Need Enterprise Grade Support for Apache CloudStack?
Our CloudStack Infrastructure
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers
the best 24/7 SLA for CloudStack Environments.

Apache CloudStack Bootcamp training courses

**NEW!** CloudStack 4.2.1
traininghttp://shapeblue.com/cloudstack-training/
18th-19th February 2014, Brazil.
Classroomhttp://shapeblue.com/cloudstack-training/
17th-23rd March 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
24th-28th March 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
16th-20th June 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
23rd-27th June 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/

This email and any attachments to it may be confidential and are intended
solely for the use of the individual to whom it is addressed. Any views
or opinions expressed are solely those of the author and do not
necessarily represent those of Shape Blue Ltd or related companies. If
you are not the intended recipient of this email, you must neither take
any action based upon its contents, nor copy or show it to anyone. Please
contact the sender if you believe you have received this email in error.
Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
Services India LLP is a company incorporated in India and is operated
under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is
a company incorporated in Brasil and is operated under license from Shape
Blue Ltd. ShapeBlue is a registered trademark.



Re: BVT report on 4.3 branch

2014-01-30 Thread Srikanteswararao Talluri
Hey Daan,

It is a mistake while clubbing two reports #1174 and #1175. Thanks for
pointing out. Initially there are 12 failures in #1174 and when I reran
#1175 there are indeed only 3 failures overall.


Thanks,
~Talluri

On 30/01/14 3:48 pm, Daan Hoogland daan.hoogl...@gmail.com wrote:

H Talluri,

in the report summary for xen it says
Fail :5
but i only see these 3
test_loadbalance/ 0   3   0

am i missing something? (the kvm failures seem to match the summary)

when I folow the links I see 12 and 3 failures respectively. I'm sure
I'm just being paranoid but can you explain, please?

On Thu, Jan 30, 2014 at 6:40 AM, Prasanna Santhanam t...@apache.org
wrote:
 This basically means a +1 on 4.3? I think there were couple of issues
 that emerged in the RC particularly the missing response params in the
 list* API that Alena reported. We should probably include a test for
 that in the integration suite.

 I would've done it had I had some bandwidth, but thought I'd bring
 this up as a way to backfill missing tests.

 On Wed, Jan 29, 2014 at 12:14:18PM +, Srikanteswararao Talluri
wrote:
 Here is the report of automated smoke tests results on Xen and KVM.
 
 On Xenserver 6.2/Test Run: #1174 and 1175
 
 This report is consolidation of runs #1174 and #1175
 3 failures are not product failures but ssh time outs in the scripts.
 
 Link to reports:
 
http://jenkins.buildacloud.org/view/cloudstack-qa-4.3/job/test-smoke-ma
tri
 x-4.3/1174/testReport/
 
 
http://jenkins.buildacloud.org/view/cloudstack-qa-4.3/job/test-smoke-ma
tri
 x-4.3/1175/testReport/
 
 Total:97
 Fail :5
 Skip :2
 
 
 name   passfail
skip
 test_public_ip_range/ 1   0
0
 test_nic/ 1   0
0
 test_global_settings/ 1   0
0
 test_deploy_vms_with_varied_deploymentplanners/   3   0
0
 test_regions/ 1   0
0
 test_ssvm/   10   0
0
 test_vpc_vpn/ 2   0
0
 test_affinity_groups/ 1   0
0
 test_deploy_vm/   1   0
0
 test_deploy_vm_with_userdata/ 2   0
0
 test_routers/ 9   0
0
 test_loadbalance/ 0   3
0
 test_portable_publicip/   2   0
0
 test_iso/ 5   0
1
 test_internal_lb/ 1   0
0
 test_vm_snapshots/3   0
0
 test_network/ 7   0
0
 test_disk_offerings/  3   0
0
 test_non_contigiousvlan/  1   0
0
 test_templates/   7   0
1
 test_network_acl/ 1   0
0
 test_multipleips_per_nic/ 1   0
0
 test_reset_vm_on_reboot/  1   0
0
 test_scale_vm/1   0
0
 test_pvlan/   1   0
0
 test_vm_life_cycle/  10   0
0
 test_privategw_acl/   1   0
0
 test_resource_detail/ 1   0
0
 test_guest_vlan_range/1   0
0
 test_volumes/ 9   0
0
 test_service_offerings/   4   0
0
 
 
 
 On KVM /RHEL 6.3 hosts/ Test Run: #1173
 ---
 4 of the test failures are not valid because snapshots and scaling VM
are
 not supported on KVM Hypervisor
 
 Link to report:
 
http://jenkins.buildacloud.org/job/test-smoke-matrix-4.3/1173/testRepor
t/
 
 
 Total:97
 Fail :8
 Skip :2
 
 
 name   passfail
skip
 test_public_ip_range/ 1   0
0
 test_nic/ 1   0
0
 test_global_settings/ 1   0
0
 test_deploy_vms_with_varied_deploymentplanners/   3   0
0
 test_regions/ 1   0
0
 test_ssvm/   10   0
0
 test_vpc_vpn/ 2   0
0
 test_affinity_groups/ 1   0
0
 test_deploy_vm/   1   0
0
 test_deploy_vm_with_userdata/ 2   0
0
 test_routers/ 9   0
0
 test_loadbalance/   

Re: bof for denver

2014-01-30 Thread Chiradeep Vittal
Nice, like it

On 1/29/14 9:31 PM, Daan Hoogland daan.hoogl...@gmail.com wrote:

H,

I submitted a bof proposal: continuous delivery the apache way

the apache release process is based on high quality and delivering
good quality software. for a big project as cloudstack this means
unsure delivery schedules and long release cycles. The goal of this
BoF is to find ways to hasten delivery whilst at least maintaining and
preferably improving code quality.

the apache release process is based on high quality and delivering
good quality software. for a big project as cloudstack this means
unsure delivery schedules and long release cycles. The goal of this
BoF is to find ways to hasten delivery whilst at least maintaining and
preferably improving code quality.

committers to any middle size to large project will be interested to
participate. Also (board)members or managers interested in speed and
quality as possibly contradicting aspects of code delivery.


any takers?



Re: [DISCUSS][findbugs] proposing fixes or just do it

2014-01-30 Thread Chiradeep Vittal
Just do it. Although, I suspect that piece of code may be generated from
the API WSDL

On 1/30/14 9:38 AM, Daan Hoogland daan.hoogl...@gmail.com wrote:

H Alex and others,

I hesitate to ask you as it seems obvious, but in the awsapi project
in com.cloud.bridge.model.SObjectItem.equals(Object other) a
comparison is made
if 
(!theObject.getId().equals(((SObjectItemVO)other).getTheObject()))
which i think should be
if 
(!theObject.getId().equals(((SObjectItemVO)other).getTheObject().getId()))
The obvious question would be, 'am i correct?', or 'can i make the
change?'.

Now my real question is should I just go ahead and fix the scariest
bugs that findbugs comes up with or should I consult with package
maintainers for each part of the system on errors like these?

(to comfort all of you, i restricted myself to doing this in master)

regards,
Daan



Review Request 17531: CLOUDSTACK-5440 - Added new guest OS types.

2014-01-30 Thread Alexander Hitchins

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17531/
---

Review request for cloudstack.


Repository: cloudstack-git


Description
---

Added new guest OS types in upgrade script.


Diffs
-

  setup/db/db/schema-421to430.sql afad06c 

Diff: https://reviews.apache.org/r/17531/diff/


Testing
---

Ran new SQL, no errors, checked table, contains correct data.


Thanks,

Alexander Hitchins



Re: [DISCUSS][findbugs] proposing fixes or just do it

2014-01-30 Thread Daan Hoogland
;/that would mean the code generator has a serious bug. Beter waste time in
that code

mobile bilingual spell checker used
Op 30 jan. 2014 12:03 schreef Chiradeep Vittal 
chiradeep.vit...@citrix.com:

 Just do it. Although, I suspect that piece of code may be generated from
 the API WSDL

 On 1/30/14 9:38 AM, Daan Hoogland daan.hoogl...@gmail.com wrote:

 H Alex and others,
 
 I hesitate to ask you as it seems obvious, but in the awsapi project
 in com.cloud.bridge.model.SObjectItem.equals(Object other) a
 comparison is made
 if
 (!theObject.getId().equals(((SObjectItemVO)other).getTheObject()))
 which i think should be
 if
 (!theObject.getId().equals(((SObjectItemVO)other).getTheObject().getId()))
 The obvious question would be, 'am i correct?', or 'can i make the
 change?'.
 
 Now my real question is should I just go ahead and fix the scariest
 bugs that findbugs comes up with or should I consult with package
 maintainers for each part of the system on errors like these?
 
 (to comfort all of you, i restricted myself to doing this in master)
 
 regards,
 Daan




Re: API documentation

2014-01-30 Thread Gaurav Aradhye
I found following 3 APIs related to adding load balancer devices absent in
the documentation.

1. addExternalLoadBalancer
2. addF5LoadBalancer
3. addNetscalerLoadBalancer

Regards,
Gaurav


On Thu, Jan 30, 2014 at 4:09 PM, sebgoa run...@gmail.com wrote:


 On Jan 30, 2014, at 10:53 AM, Paul Angus paul.an...@shapeblue.com wrote:

  Hey Radhika,
 
  Yes, I'm happy to do what I can (probably mainly harassing other people)
 
  CloudMonkey says that there are 431 API calls (in 4.3 RC1) I recon there
 are 409 listed on
 http://cloudstack.apache.org/docs/api/apidocs-4.2/TOC_Root_Admin.html
 

 We talked about this recently, the discrepancy seems to come from the fact
 that the apidocs are built for the oss dist and that the missing apis are
 from the nonredist build.

 -sebastien

  I'll try to find which ones are missing.
 
  I think in terms of the API documentation page, the first thing to do is
 put this headings in alphabetical order!
 
  Regards,
 
  Paul Angus
  Cloud Architect
  S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
  paul.an...@shapeblue.com
 
  -Original Message-
  From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com]
  Sent: 30 January 2014 07:48
  To: dev@cloudstack.apache.org
  Subject: RE: API documentation
 
  Hi Paul,
 
  Yes, API documentation requires a serious revisit. I had initiated a
 discussion months back, and invited volunteers.
 
  A wiki page is created at
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Documentation+Sprint+for+Enhancing+API+Documentation
  , but nothing beyond that.
 
  If you are interested to volunteer, let's begin.
 
  -Radhika
 
  From: Paul Angus [mailto:paul.an...@shapeblue.com]
  Sent: Thursday, January 30, 2014 12:43 PM
  To: dev@cloudstack.apache.org
  Subject: API documentation
 
  DEVs,
 
  New features are generally cool, but given that the 'engine' of
 CloudStack is the API it seems ridiculous that as a minimum the API
 documentation isn't kept up to date.
 
  The specific example that I'm thinking of is
 
  addvmwaredc
 
  Only by using the UI and following the API calls sent could I find that
 it existed, and then tracked it down in the design documents in the wiki.
 
  I've filed a bug for this one (
 https://issues.apache.org/jira/browse/CLOUDSTACK-5984)
 
  But please can everyone make sure we can all use the features you create
 by documenting them.
 
  Regards
 
  Paul Angus
  Senior Consultant / Cloud Architect
 
  [cid:image002.png@01CE1071.C6CC9C10]
 
  S: +44 20 3603 0540tel:+442036030540 | M:
 +4tel:+44796816158147711418784 | T: @CloudyAngus
 paul.an...@shapeblue.commailto:paul.an...@shapeblue.com |
 www.shapeblue.comhttp://www.shapeblue.com/ | Twitter:@shapeblue
 https://twitter.com/
  ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
 
  Need Enterprise Grade Support for Apache CloudStack?
  Our CloudStack Infrastructure Support
 http://shapeblue.com/cloudstack-infrastructure-support/ offers the best
 24/7 SLA for CloudStack Environments.
 
  Apache CloudStack Bootcamp training courses
 
  **NEW!** CloudStack 4.2.1 training
 http://shapeblue.com/cloudstack-training/
  18th-19th February 2014, Brazil. Classroom
 http://shapeblue.com/cloudstack-training/
  17th-23rd March 2014, Region A. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
  24th-28th March 2014, Region B. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
  16th-20th June 2014, Region A. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
  23rd-27th June 2014, Region B. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
 
  This email and any attachments to it may be confidential and are
 intended solely for the use of the individual to whom it is addressed. Any
 views or opinions expressed are solely those of the author and do not
 necessarily represent those of Shape Blue Ltd or related companies. If you
 are not the intended recipient of this email, you must neither take any
 action based upon its contents, nor copy or show it to anyone. Please
 contact the sender if you believe you have received this email in error.
 Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
 Services India LLP is a company incorporated in India and is operated under
 license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is a
 company incorporated in Brasil and is operated under license from Shape
 Blue Ltd. ShapeBlue is a registered trademark.
  This email and any attachments to it may be confidential and are
 intended solely for the use of the individual to whom it is addressed. Any
 views or opinions expressed are solely those of the author and do not
 necessarily represent those of Shape Blue Ltd or related companies. If you
 are not the intended recipient of this email, you must neither take any
 action based upon its contents, nor copy or show it to anyone. Please
 contact the sender if you believe you have received this email in error.
 

Re: 4.3 commit cherry pick

2014-01-30 Thread Hiroaki KAWAI
Hi,

I found that our plugin was broken, so please cherry pick
db3dc2ee17c699f1abf42ea2fbe09ec2748caa4a in 4.3-forward.


(2014/01/30 18:06), Jayapal Reddy Uradi wrote:
 Hi Animesh,
 
 Can you please cherry pick commit to 7255e50f2051aff1cb5e1991e2dde4ef48f57454 
 4.3 branch 
 from 4.3-forward.
 
 Thanks,
 Jayapal
 



RE: RSA Keys Deployment on Windows

2014-01-30 Thread Alex Hitchins
My username is _alex_hitchins

If I could have edit access I can get this in to the page.


Alex Hitchins
+44 7788 423 969

-Original Message-
From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
Sent: 30 January 2014 10:57
To: dev@cloudstack.apache.org
Subject: Re: RSA Keys  Deployment on Windows

Looks specific to Windows. What is your user id on the Wiki?

On 1/29/14 4:22 PM, Alex Hitchins alex.hitch...@shapeblue.com wrote:

All,

I've rebuilt my development environment from the 4.3 branch. In
rebuilding, I hit upon a number of errors namely around ssh keys
(id_rsa.cloud  id_rsa.cloud.pub). When running the Jetty command there
were errors due to the file not existing. It then later looks for the
file it's tried to create to insert this into the database.

I manually created the keys as the mvn script was set to use ssh-keygen
under /bin/bash - also, as the folder was named beginning with a . I
had to create the directory with admin permissions.

Not sure if this is a bug or just needs writing up in the dev wiki
under the setting up a windows environment. Happy to do so if I have
the permissions to edit the site. If not, I'll blog about it so someone
can find some help in the future.


Alex Hitchins
--
E: a...@alexhitchins.commailto:a...@alexhitchins.com
M: 07788 423 969


Need Enterprise Grade Support for Apache CloudStack?
Our CloudStack Infrastructure
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers
the best 24/7 SLA for CloudStack Environments.

Apache CloudStack Bootcamp training courses

**NEW!** CloudStack 4.2.1
traininghttp://shapeblue.com/cloudstack-training/
18th-19th February 2014, Brazil.
Classroomhttp://shapeblue.com/cloudstack-training/
17th-23rd March 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
24th-28th March 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
16th-20th June 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
23rd-27th June 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/

This email and any attachments to it may be confidential and are
intended solely for the use of the individual to whom it is addressed.
Any views or opinions expressed are solely those of the author and do
not necessarily represent those of Shape Blue Ltd or related companies.
If you are not the intended recipient of this email, you must neither
take any action based upon its contents, nor copy or show it to anyone.
Please contact the sender if you believe you have received this email in error.
Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
Services India LLP is a company incorporated in India and is operated
under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
is a company incorporated in Brasil and is operated under license from
Shape Blue Ltd. ShapeBlue is a registered trademark.

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England  Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue is a registered trademark.


Re: Review Request 17142: CLOUDSTACK-5916: associateIpAddress leaves an IP in allocating state

2014-01-30 Thread ASF Subversion and Git Services

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17142/#review33213
---


Commit 85d0f8f93aaf118e0597fcbba0f4cf06efa9df1f in branch refs/heads/master 
from Saksham Srivastava
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=85d0f8f ]

CLOUDSTACK-5916: associateIpAddress leaves an IP in allocating state


- ASF Subversion and Git Services


On Jan. 21, 2014, 11:34 a.m., Saksham Srivastava wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17142/
 ---
 
 (Updated Jan. 21, 2014, 11:34 a.m.)
 
 
 Review request for cloudstack and Murali Reddy.
 
 
 Bugs: 5916
 https://issues.apache.org/jira/browse/5916
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 associateIpAddress leaves an IP in allocating state (user_ip_address table), 
 although the API command is executed on incorrectly.
 
 Steps to repro :
 1) create a vpc tier.
 2) Execute associateIpAddress API on the vpc tier but do not specify the vpc 
 id.
 #cloudmonkey
 associate ipaddress networkid=09ffc45f-beba-4690-8be7-425891915d44
 Async job ea020246-d0e8-4e58-ac84-fccb55c3b646 failed
 Error 530, Can't assign ip to the network directly when network belongs to 
 VPC.Specify vpcId to associate ip address to VPC
 accountid = a6ba35b3-7e76-11e3-8490-7614eba325e6
 cmd = org.apache.cloudstack.api.command.user.address.AssociateIPAddrCmd
 created = 2014-01-21T10:46:46+0530
 jobid = ea020246-d0e8-4e58-ac84-fccb55c3b646
 jobprocstatus = 0
 jobresult:
 errorcode = 530
 errortext = Can't assign ip to the network directly when network belongs to 
 VPC.Specify vpcId to associate ip address to VPC
 jobresultcode = 530
 jobresulttype = object
 jobstatus = 2
 userid = a6ba5844-7e76-11e3-8490-7614eba325e6
 
 Expected behavior:
 There should be no allocation of IP .
 
 Actual behaviour:
 The public IP remains in 'Allocating' state
 
 The fix releases the IP incase of exception.
 
 
 Diffs
 -
 
   server/src/com/cloud/network/NetworkServiceImpl.java 056190f 
 
 Diff: https://reviews.apache.org/r/17142/diff/
 
 
 Testing
 ---
 
 Tested the following scenarios:
 ip:8096/client/api?command=associateIpAddressnetworkid=09ffc45f-beba-4690-8be7-425891915d44
 There is no allocation of public IP (no IP is put in Allocating state in the 
 db).
  
 ip:8096/client/api?command=associateIpAddressnetworkid=09ffc45f-beba-4690-8be7-425891915d44vpcid=1
 Successfully associates the IP.
 
 Build passes successfully.
 Patch applies cleanly.
 
 
 Thanks,
 
 Saksham Srivastava
 




Re: Review Request 17142: CLOUDSTACK-5916: associateIpAddress leaves an IP in allocating state

2014-01-30 Thread Murali Reddy

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17142/#review33214
---

Ship it!


85d0f8f93aaf118e0597fcbba0f4cf06efa9df1f

- Murali Reddy


On Jan. 21, 2014, 11:34 a.m., Saksham Srivastava wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17142/
 ---
 
 (Updated Jan. 21, 2014, 11:34 a.m.)
 
 
 Review request for cloudstack and Murali Reddy.
 
 
 Bugs: 5916
 https://issues.apache.org/jira/browse/5916
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 associateIpAddress leaves an IP in allocating state (user_ip_address table), 
 although the API command is executed on incorrectly.
 
 Steps to repro :
 1) create a vpc tier.
 2) Execute associateIpAddress API on the vpc tier but do not specify the vpc 
 id.
 #cloudmonkey
 associate ipaddress networkid=09ffc45f-beba-4690-8be7-425891915d44
 Async job ea020246-d0e8-4e58-ac84-fccb55c3b646 failed
 Error 530, Can't assign ip to the network directly when network belongs to 
 VPC.Specify vpcId to associate ip address to VPC
 accountid = a6ba35b3-7e76-11e3-8490-7614eba325e6
 cmd = org.apache.cloudstack.api.command.user.address.AssociateIPAddrCmd
 created = 2014-01-21T10:46:46+0530
 jobid = ea020246-d0e8-4e58-ac84-fccb55c3b646
 jobprocstatus = 0
 jobresult:
 errorcode = 530
 errortext = Can't assign ip to the network directly when network belongs to 
 VPC.Specify vpcId to associate ip address to VPC
 jobresultcode = 530
 jobresulttype = object
 jobstatus = 2
 userid = a6ba5844-7e76-11e3-8490-7614eba325e6
 
 Expected behavior:
 There should be no allocation of IP .
 
 Actual behaviour:
 The public IP remains in 'Allocating' state
 
 The fix releases the IP incase of exception.
 
 
 Diffs
 -
 
   server/src/com/cloud/network/NetworkServiceImpl.java 056190f 
 
 Diff: https://reviews.apache.org/r/17142/diff/
 
 
 Testing
 ---
 
 Tested the following scenarios:
 ip:8096/client/api?command=associateIpAddressnetworkid=09ffc45f-beba-4690-8be7-425891915d44
 There is no allocation of public IP (no IP is put in Allocating state in the 
 db).
  
 ip:8096/client/api?command=associateIpAddressnetworkid=09ffc45f-beba-4690-8be7-425891915d44vpcid=1
 Successfully associates the IP.
 
 Build passes successfully.
 Patch applies cleanly.
 
 
 Thanks,
 
 Saksham Srivastava
 




Review Request 17532: CLOUDSTACK-5992: default values of configuraiton parameters in configuration table are set NULL on fresh setup

2014-01-30 Thread Harikrishna Patnala

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17532/
---

Review request for cloudstack and Kishan Kavala.


Bugs: CLOUDSTACK-5992
https://issues.apache.org/jira/browse/CLOUDSTACK-5992


Repository: cloudstack-git


Description
---

CLOUDSTACK-5992: default values of configuraiton parameters in configuration 
table are set NULL on fresh setup 
Some configuration parameters have Component names different from fresh and 
upgrade setup


Diffs
-

  engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java 
4d925fd 
  
framework/config/src/org/apache/cloudstack/framework/config/impl/ConfigDepotImpl.java
 4cefdaf 
  server/src/com/cloud/server/ConfigurationServerImpl.java 7c6743a 

Diff: https://reviews.apache.org/r/17532/diff/


Testing
---


Thanks,

Harikrishna Patnala



Re: RSA Keys Deployment on Windows

2014-01-30 Thread Chiradeep Vittal
Done

On 1/30/14 11:35 AM, Alex Hitchins alex.hitch...@shapeblue.com wrote:

My username is _alex_hitchins

If I could have edit access I can get this in to the page.


Alex Hitchins
+44 7788 423 969

-Original Message-
From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
Sent: 30 January 2014 10:57
To: dev@cloudstack.apache.org
Subject: Re: RSA Keys  Deployment on Windows

Looks specific to Windows. What is your user id on the Wiki?

On 1/29/14 4:22 PM, Alex Hitchins alex.hitch...@shapeblue.com wrote:

All,

I've rebuilt my development environment from the 4.3 branch. In
rebuilding, I hit upon a number of errors namely around ssh keys
(id_rsa.cloud  id_rsa.cloud.pub). When running the Jetty command there
were errors due to the file not existing. It then later looks for the
file it's tried to create to insert this into the database.

I manually created the keys as the mvn script was set to use ssh-keygen
under /bin/bash - also, as the folder was named beginning with a . I
had to create the directory with admin permissions.

Not sure if this is a bug or just needs writing up in the dev wiki
under the setting up a windows environment. Happy to do so if I have
the permissions to edit the site. If not, I'll blog about it so someone
can find some help in the future.


Alex Hitchins
--
E: a...@alexhitchins.commailto:a...@alexhitchins.com
M: 07788 423 969


Need Enterprise Grade Support for Apache CloudStack?
Our CloudStack Infrastructure
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers
the best 24/7 SLA for CloudStack Environments.

Apache CloudStack Bootcamp training courses

**NEW!** CloudStack 4.2.1
traininghttp://shapeblue.com/cloudstack-training/
18th-19th February 2014, Brazil.
Classroomhttp://shapeblue.com/cloudstack-training/
17th-23rd March 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
24th-28th March 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
16th-20th June 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
23rd-27th June 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/

This email and any attachments to it may be confidential and are
intended solely for the use of the individual to whom it is addressed.
Any views or opinions expressed are solely those of the author and do
not necessarily represent those of Shape Blue Ltd or related companies.
If you are not the intended recipient of this email, you must neither
take any action based upon its contents, nor copy or show it to anyone.
Please contact the sender if you believe you have received this email in
error.
Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
Services India LLP is a company incorporated in India and is operated
under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
is a company incorporated in Brasil and is operated under license from
Shape Blue Ltd. ShapeBlue is a registered trademark.

This email and any attachments to it may be confidential and are intended
solely for the use of the individual to whom it is addressed. Any views
or opinions expressed are solely those of the author and do not
necessarily represent those of Shape Blue Ltd or related companies. If
you are not the intended recipient of this email, you must neither take
any action based upon its contents, nor copy or show it to anyone. Please
contact the sender if you believe you have received this email in error.
Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
Services India LLP is a company incorporated in India and is operated
under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is
a company incorporated in Brasil and is operated under license from Shape
Blue Ltd. ShapeBlue is a registered trademark.



Re: RSA Keys Deployment on Windows

2014-01-30 Thread Chiradeep Vittal
Anything

On 1/30/14 12:42 PM, Alex Hitchins alex.hitch...@shapeblue.com wrote:

Thank you. I'll get this updated later tonight. I shall put it in as a
note rather than a step instruction.

Can I now amend any area of the Wiki or just that one page? I do
sometimes see small things that I could easily tidy up.



Alex Hitchins
+44 7788 423 969

-Original Message-
From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
Sent: 30 January 2014 12:38
To: dev@cloudstack.apache.org
Subject: Re: RSA Keys  Deployment on Windows

Done

On 1/30/14 11:35 AM, Alex Hitchins alex.hitch...@shapeblue.com wrote:

My username is _alex_hitchins

If I could have edit access I can get this in to the page.


Alex Hitchins
+44 7788 423 969

-Original Message-
From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
Sent: 30 January 2014 10:57
To: dev@cloudstack.apache.org
Subject: Re: RSA Keys  Deployment on Windows

Looks specific to Windows. What is your user id on the Wiki?

On 1/29/14 4:22 PM, Alex Hitchins alex.hitch...@shapeblue.com wrote:

All,

I've rebuilt my development environment from the 4.3 branch. In
rebuilding, I hit upon a number of errors namely around ssh keys
(id_rsa.cloud  id_rsa.cloud.pub). When running the Jetty command
there were errors due to the file not existing. It then later looks
for the file it's tried to create to insert this into the database.

I manually created the keys as the mvn script was set to use
ssh-keygen under /bin/bash - also, as the folder was named beginning
with a . I had to create the directory with admin permissions.

Not sure if this is a bug or just needs writing up in the dev wiki
under the setting up a windows environment. Happy to do so if I have
the permissions to edit the site. If not, I'll blog about it so
someone can find some help in the future.


Alex Hitchins
--
E: a...@alexhitchins.commailto:a...@alexhitchins.com
M: 07788 423 969


Need Enterprise Grade Support for Apache CloudStack?
Our CloudStack Infrastructure
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/
offers the best 24/7 SLA for CloudStack Environments.

Apache CloudStack Bootcamp training courses

**NEW!** CloudStack 4.2.1
traininghttp://shapeblue.com/cloudstack-training/
18th-19th February 2014, Brazil.
Classroomhttp://shapeblue.com/cloudstack-training/
17th-23rd March 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
24th-28th March 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
16th-20th June 2014, Region A. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/
23rd-27th June 2014, Region B. Instructor led,
On-linehttp://shapeblue.com/cloudstack-training/

This email and any attachments to it may be confidential and are
intended solely for the use of the individual to whom it is addressed.
Any views or opinions expressed are solely those of the author and do
not necessarily represent those of Shape Blue Ltd or related companies.
If you are not the intended recipient of this email, you must neither
take any action based upon its contents, nor copy or show it to anyone.
Please contact the sender if you believe you have received this email
in error.
Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
Services India LLP is a company incorporated in India and is operated
under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
is a company incorporated in Brasil and is operated under license from
Shape Blue Ltd. ShapeBlue is a registered trademark.

This email and any attachments to it may be confidential and are
intended solely for the use of the individual to whom it is addressed.
Any views or opinions expressed are solely those of the author and do
not necessarily represent those of Shape Blue Ltd or related companies.
If you are not the intended recipient of this email, you must neither
take any action based upon its contents, nor copy or show it to anyone.
Please contact the sender if you believe you have received this email in
error.
Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
Services India LLP is a company incorporated in India and is operated
under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda
is a company incorporated in Brasil and is operated under license from
Shape Blue Ltd. ShapeBlue is a registered trademark.

This email and any attachments to it may be confidential and are intended
solely for the use of the individual to whom it is addressed. Any views
or opinions expressed are solely those of the author and do not
necessarily represent those of Shape Blue Ltd or related companies. If
you are not the intended recipient of this email, you must neither take
any action based upon its contents, nor copy or show it to anyone. Please
contact the sender if you believe you have received this email in error.
Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue
Services 

EC2 Rest Servlet domain cannot be null!! Error In Advanced Networking Mode

2014-01-30 Thread Jeff Hair
Hi,

We're trying to query the EC2 REST API. Most of it works, but we're running
into an issue when calling describe addresses. It returns a fault with
domain cannot be null!! as the error text. I've traced this down to
CloudStack not setting the EC2 domain (standard or vpc) in the
EC2SoapServiceImpl#toDescribeAddressesResponse static method. The domain
property is not being set, and then it explodes on serialization with the
above error.

So, my questions are these:

1. Is this a bug in CloudStack with the EC2 API and Advanced Networking? Or
are we missing some configuration/setting?
2. If this is actually a bug, what is the best way to work around it
temporarily, and also where to start fixing it for real in the CS codebase?

Thanks,

Jeff


Review Request 17535: Added few fixes for CLOUDSTACK-5973

2014-01-30 Thread Santhosh Edukulla

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17535/
---

Review request for cloudstack and Girish Shilamkar.


Bugs: CLOUDSTACK-5973
https://issues.apache.org/jira/browse/CLOUDSTACK-5973


Repository: cloudstack-git


Description
---

Added the below changes to topic branch.
1. Disabled network offering before delete, we followed this in few cases and 
missed at a case.
2. Removed few deletes for network.
3. The log messages at place seems to out of context for with, leads to 
exception,added that.
4. Fixed an and issue. Added few exception levels and to sshClient


Diffs
-

  test/integration/component/test_add_remove_network.py 6eef71e 
  tools/marvin/marvin/sshClient.py 588c978 

Diff: https://reviews.apache.org/r/17535/diff/


Testing
---

Tested pep8 for sshClient and python import for both. Will test on topic branch 
post other changes.


Thanks,

Santhosh Edukulla



RE: Xen StorageMotion Fails

2014-01-30 Thread Francois Gaudreault
I used the UI for the vm migration. I can get you the exact api call but
isnt the call in the pastebin output already..

Another thing, if there is a vm running on the same network on the
destination host, I dont have the exception but the volume stay in migrate
state forever.

Thanks

Francois
On Jan 30, 2014 2:19 AM, Srikanteswararao Talluri 
srikanteswararao.tall...@citrix.com wrote:

 Yes, I am able to migrate a VM on local storage(host) to another local
 storage(host).

 Can you paste the API call you tried?

 Thanks,
 ~Talluri

 -Original Message-
 From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com]
 Sent: Wednesday, 29 January 2014 6:31 PM
 To: dev@cloudstack.apache.org
 Subject: Xen StorageMotion Fails

 Hi,

 I posted this on the users@ mailing-list yesterday, but giving it more
 thoughts I think this should have been posted here. We are running 4.2.1
 with XenServer 6.2SP1. I tried to run a storage motion from a local storage
 to another, and I am getting this error. Xen returns a VM_REQUIRES_NETWORK
 when attempting the migration. This error, according to the javadoc means:

 You attempted to run a VM on a host which doesn't have a PIF on a Network
 needed by the VM. The VM has at least one VIF attached to the Network.

 http://pastebin.com/cHcAyPtP

 I believe this is supposed to be supported since CloudStack let me do it
 from the UI. Right?

 Thanks!

 --
 Francois Gaudreault
 Architecte de Solution Cloud | Cloud Solutions Architect
 fgaudrea...@cloudops.com
 514-629-6775
 - - -
 CloudOps
 420 rue Guy
 Montréal QC  H3J 1S6
 www.cloudops.com
 @CloudOps_




RE: API documentation

2014-01-30 Thread Radhika Puthiyetath
There are many such issues. Can we chart down something concrete ? 

My suggestion is simple: Volunteer can pick up one feature each and fix all 
issues related to the API documentation: Description, build (can take help from 
infra), how-to info

Experts, please advise.

-Original Message-
From: Gaurav Aradhye [mailto:gaurav.arad...@clogeny.com] 
Sent: Thursday, January 30, 2014 5:00 PM
To: dev@cloudstack.apache.org
Subject: Re: API documentation

I found following 3 APIs related to adding load balancer devices absent in the 
documentation.

1. addExternalLoadBalancer
2. addF5LoadBalancer
3. addNetscalerLoadBalancer

Regards,
Gaurav


On Thu, Jan 30, 2014 at 4:09 PM, sebgoa run...@gmail.com wrote:


 On Jan 30, 2014, at 10:53 AM, Paul Angus paul.an...@shapeblue.com wrote:

  Hey Radhika,
 
  Yes, I'm happy to do what I can (probably mainly harassing other 
  people)
 
  CloudMonkey says that there are 431 API calls (in 4.3 RC1) I recon 
  there
 are 409 listed on
 http://cloudstack.apache.org/docs/api/apidocs-4.2/TOC_Root_Admin.html
 

 We talked about this recently, the discrepancy seems to come from the 
 fact that the apidocs are built for the oss dist and that the missing 
 apis are from the nonredist build.

 -sebastien

  I'll try to find which ones are missing.
 
  I think in terms of the API documentation page, the first thing to 
  do is
 put this headings in alphabetical order!
 
  Regards,
 
  Paul Angus
  Cloud Architect
  S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
  paul.an...@shapeblue.com
 
  -Original Message-
  From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com]
  Sent: 30 January 2014 07:48
  To: dev@cloudstack.apache.org
  Subject: RE: API documentation
 
  Hi Paul,
 
  Yes, API documentation requires a serious revisit. I had initiated a
 discussion months back, and invited volunteers.
 
  A wiki page is created at
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Documentation+S
 print+for+Enhancing+API+Documentation
  , but nothing beyond that.
 
  If you are interested to volunteer, let's begin.
 
  -Radhika
 
  From: Paul Angus [mailto:paul.an...@shapeblue.com]
  Sent: Thursday, January 30, 2014 12:43 PM
  To: dev@cloudstack.apache.org
  Subject: API documentation
 
  DEVs,
 
  New features are generally cool, but given that the 'engine' of
 CloudStack is the API it seems ridiculous that as a minimum the API 
 documentation isn't kept up to date.
 
  The specific example that I'm thinking of is
 
  addvmwaredc
 
  Only by using the UI and following the API calls sent could I find 
  that
 it existed, and then tracked it down in the design documents in the wiki.
 
  I've filed a bug for this one (
 https://issues.apache.org/jira/browse/CLOUDSTACK-5984)
 
  But please can everyone make sure we can all use the features you 
  create
 by documenting them.
 
  Regards
 
  Paul Angus
  Senior Consultant / Cloud Architect
 
  [cid:image002.png@01CE1071.C6CC9C10]
 
  S: +44 20 3603 0540tel:+442036030540 | M:
 +4tel:+44796816158147711418784 | T: @CloudyAngus
 paul.an...@shapeblue.commailto:paul.an...@shapeblue.com | 
 www.shapeblue.comhttp://www.shapeblue.com/ | Twitter:@shapeblue 
 https://twitter.com/
  ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
 
  Need Enterprise Grade Support for Apache CloudStack?
  Our CloudStack Infrastructure Support
 http://shapeblue.com/cloudstack-infrastructure-support/ offers the 
 best
 24/7 SLA for CloudStack Environments.
 
  Apache CloudStack Bootcamp training courses
 
  **NEW!** CloudStack 4.2.1 training
 http://shapeblue.com/cloudstack-training/
  18th-19th February 2014, Brazil. Classroom
 http://shapeblue.com/cloudstack-training/
  17th-23rd March 2014, Region A. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
  24th-28th March 2014, Region B. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
  16th-20th June 2014, Region A. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
  23rd-27th June 2014, Region B. Instructor led, On-line
 http://shapeblue.com/cloudstack-training/
 
  This email and any attachments to it may be confidential and are
 intended solely for the use of the individual to whom it is addressed. 
 Any views or opinions expressed are solely those of the author and do 
 not necessarily represent those of Shape Blue Ltd or related 
 companies. If you are not the intended recipient of this email, you 
 must neither take any action based upon its contents, nor copy or show 
 it to anyone. Please contact the sender if you believe you have received this 
 email in error.
 Shape Blue Ltd is a company incorporated in England  Wales. ShapeBlue 
 Services India LLP is a company incorporated in India and is operated 
 under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda 
 is a company incorporated in Brasil and is operated under license from 
 Shape Blue Ltd. ShapeBlue is a registered trademark.
  This email 

Re: Findbugs report on 4.3-forward

2014-01-30 Thread Daan Hoogland
Guys,

I found two reported issues of category 'scary'. To satify Rajuri's
concerns I would like to revert Ian's commit and checkin two changes
that change returning
byte[].toString()
into
Arrays.toString(byte[])
on return statements of generatePassword methods.

if no objections come in within a few...,
Daan

On Thu, Jan 30, 2014 at 1:53 PM, Daan Hoogland daan.hoogl...@gmail.com wrote:
 Animesh, Ian,

 Can you comment on this?

 I couldn't find any findbugs issues of the the scariest kind in
 yesterdays version of the 4.3 branch. What was solved that needs to go
 in in spite of Rajuri's reservations?

 thanks,
 Daan

 On Thu, Jan 30, 2014 at 11:21 AM, Daan Hoogland daan.hoogl...@gmail.com 
 wrote:
 Sorry Rajani,

 I had seen no reaction to Ian's explenation  and the request by
 Animesh to pull it so I just did. let me look into it for a minute

 On Thu, Jan 30, 2014 at 5:36 AM, Rajani Karuturi
 rajani.karut...@citrix.com wrote:
 I see that the commit 9776e1af1c92486f5081b1ee8fa95cf0edb86b97 is already 
 pushed to 4.3. I don't see any response on my concern as well.
 Is it just me or anyone else sees a security issue with the generate 
 password change?
 Ian/Animesh/Daan, can you please respond?

 Thanks,
 ~Rajani



 On 29-Jan-2014, at 10:59 am, Rajani Karuturi rajani.karut...@citrix.com 
 wrote:

 Hi Ian,
 Before it is pushed to 4.3, can you fix the generate password change like 
 i suggested in the other mail? This current change would make it less 
 secure.

 Thanks,
 ~Rajani



 On 29-Jan-2014, at 8:03 am, Ian Duffy i...@ianduffy.ie wrote:

 Hi Animesh,

 Tested all those changes to detail. Those lines were removed due to
 unexpected behavior that I had not spotted until now.

 They were suppose to allow for better fall over between multiple domain
 controllers, how ever they were causing caching to occur. This meant if a
 users password was reset in LDAP the old password was still allowing login
 for a limited time.

 Please pull the changes forward,
 Thanks

 Ian.
 On 29 Jan 2014 00:07, Animesh Chaturvedi animesh.chaturv...@citrix.com
 wrote:

 If I look at this commit for example


 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=92b4f66d73562e4211d2d787554ff229dbeb5705

 It removes the two lines from LdapContextFactory.java

 environment.put(com.sun.jndi.ldap.read.timeout, 500);-
 environment.put(com.sun.jndi.ldap.connect.pool, true);

 Is that reported by find bug? I don't know this code  so not sure if it 
 is
 intentional or not ?

 The point is there may be unintended risks in allowing late changes.



 -Original Message-
 From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
 Sent: Tuesday, January 28, 2014 3:35 PM
 To: dev@cloudstack.apache.org
 Subject: RE: Findbugs report on 4.3-forward

 Are you sure all of the ones are needed. A quick look at 20+ commits from
 Daan  show many formatting changes that may not be necessary and hinder
 quick review.

 -Original Message-
 From: Hugo Trippaers [mailto:trip...@gmail.com]
 Sent: Tuesday, January 28, 2014 3:16 PM
 To: dev@cloudstack.apache.org
 Subject: Re: Findbugs report on 4.3-forward



 Sent from my iPhone

 On 28 jan. 2014, at 23:50, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com wrote:


 -Original Message-
 From: Hugo Trippaers [mailto:trip...@gmail.com]
 Sent: Tuesday, January 28, 2014 2:37 PM
 To: dev@cloudstack.apache.org
 Cc: dev@cloudstack.apache.org
 Subject: Re: Findbugs report on 4.3-forward

 Hey Animesh,

 Not in agreement here. These are squashed bugs and we want as less bugs
 in the release as possible.
 [Animesh] I understand but once we enter RC phase we only limit
 important fixes. I have pulled in  2 commits from yours and 1 from Daan.

 We limited our fixes to only the important issues that we found. The 
 other
 6000 issues between coverity and findbugs are still being triaged and 
 will
 probably not make it into this release.


 This is why we test any RC before we release it.
 [Animesh] Of course but timing is a bit off, if this was done a month
 back it would have been fine.
 I say include all the big fixes we have in the release. If that means
 more testing before we cut the RC then that is what it is. I can't
 rightfully vote for a release with known issues with existing fixes.
 [Animesh] Any release will have known issues, if we have fixes but can't
 be sure on regression impact then we have to make a choice.

 Agreed, but we just don't agree on what that choice should be yet ;-)

 Quality over release schedule would be my vote then.
 [Animesh] But why so late? Why was this activity not planned early on? I
 have been reminding community to call out issues early on since like
 mid-December.

 On November 4 I sent the mail to the dev list that static code analysis
 (coverity) found 6000+ issues that needed to be triaged. I worked on 
 quite
 a few with my colleagues, but it's a big task for just the four of us.
 Findbugs just helped us to quickly 

Re: Add new KVM hypervisor host into CloudStack

2014-01-30 Thread Sugandh S
Hi Indra,

- Same distribution version, is it referring to same OS and version? E.g.
Ubuntu 12.04 (Precise)?
I think you need to have the same distribution and same version on all the 
hosts.

- All hosts must be homogenous, does it mean that the new server has to be
exactly the same chassis, motherboard, type of CPU and RAM? Can I use a
different type of chassis and motherboard, but use the same type of CPU?
According to the instructions on the official CS documentation, the processor 
and RAM should be homogeneous in nature across all the hosts. But, during my 
test run, I managed to setup CS agents on hosts which had different processors 
and different amount of RAM. 

I am sure someone here can give much better answers to your questions than me.

Best,
Sugandh 





On Thursday, 30 January 2014 2:27 PM, Indra Pramana in...@sg.or.id wrote:
 
Hi,

It's time for me to add new KVM hypervisor host into our CloudStack and I
read these requirements:

http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.2.0/html/Installation_Guide/hypervisor-kvm-install-flow.html

===

   - Within a single cluster, the hosts must be of the same distribution
   version.
   - All hosts within a cluster must be homogenous. The CPUs must be of the
   same type, count, and feature flags.

===
- Same distribution version, is it referring to same OS and version? E.g.
Ubuntu 12.04 (Precise)?
- All hosts must be homogenous, does it mean that the new server has to be
exactly the same chassis, motherboard, type of CPU and RAM? Can I use a
different type of chassis and motherboard, but use the same type of CPU?

Thank you.

[Responsiveness report] users 2014w03

2014-01-30 Thread Daan Hoogland
http://markmail.org/message/6zkqyqf4qg3e6lgx Xen crash big problem
with disk by Paolo De Marco
http://markmail.org/message/tx6xp4afgzijcqxx cs 4.2.1 vm vmware wrong
volume type thin and NOT thick by Domenico De Monte


for an explanation of this report see
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Responsiveness+report


[4.3]more limited ldap fix

2014-01-30 Thread Daan Hoogland
Animesh,

at your convenience please pull

5651081ac0a32ba88c7446f079a5d3a8dca47feb 'Revert Fix findbug issues
within LDAP authenticator'

(or revert 9776e1af1c92486f5081b1ee8fa95cf0edb86b97)

and then pull in

ce7c6d23575d455a8273b77702a56b7bb419b37b

hopefully this mitigates Rajani's concerns.

regards,
Daan


dropping [Responsiveness report] dev list

2014-01-30 Thread Daan Hoogland
H everyone,

I've been putting out weekly reports on both the user - and the dev
lists about unanswerred questions. I feel like dropping the one on the
dev list. As I had not much feedback on them I was thinking of
dropping both but forcing myself to scan the old users mail still
seems like a valid excercise for myself.

Anyone still interested?

regards,
Daan


Re: dropping [Responsiveness report] dev list

2014-01-30 Thread Sebastien Goasguen

On Jan 30, 2014, at 10:10 AM, Daan Hoogland daan.hoogl...@gmail.com wrote:

 H everyone,
 
 I've been putting out weekly reports on both the user - and the dev
 lists about unanswerred questions. I feel like dropping the one on the
 dev list. As I had not much feedback on them I was thinking of
 dropping both but forcing myself to scan the old users mail still
 seems like a valid excercise for myself.
 
 Anyone still interested?
 

Definitely, somehow we have to find a way to make sure that all questions get 
answered.

 regards,
 Daan



Re: EC2 Rest Servlet domain cannot be null!! Error In Advanced Networking Mode

2014-01-30 Thread Chiradeep Vittal
AFAIK, it works with a very specific version of the API tools
http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.0.2/html/Instal
lation_Guide/aws-ec2-user-setup.html


On 1/30/14 1:42 PM, Jeff Hair j...@greenqloud.com wrote:

I should clarify that we are using 4.2.1 stable release with VMWare.


On Thu, Jan 30, 2014 at 1:07 PM, Jeff Hair j...@greenqloud.com wrote:

 Hi,

 We're trying to query the EC2 REST API. Most of it works, but we're
 running into an issue when calling describe addresses. It returns a
fault
 with domain cannot be null!! as the error text. I've traced this down
to
 CloudStack not setting the EC2 domain (standard or vpc) in the
 EC2SoapServiceImpl#toDescribeAddressesResponse static method. The domain
 property is not being set, and then it explodes on serialization with
the
 above error.

 So, my questions are these:

 1. Is this a bug in CloudStack with the EC2 API and Advanced Networking?
 Or are we missing some configuration/setting?
 2. If this is actually a bug, what is the best way to work around it
 temporarily, and also where to start fixing it for real in the CS
codebase?

 Thanks,

 Jeff




Re: dropping [Responsiveness report] dev list

2014-01-30 Thread Daan Hoogland
On Thu, Jan 30, 2014 at 4:17 PM, Sebastien Goasguen run...@gmail.com wrote:
 Definitely, somehow we have to find a way to make sure that all questions get 
 answered.

I agree but don't have the feeling that my weekly exercise is serving this goal.


Re: userVmResponse no longer returns some parameters

2014-01-30 Thread Animesh Chaturvedi
Daan

Did you review this thread?

Thanks
Animesh

On Jan 29, 2014, at 3:50 PM, Animesh Chaturvedi 
animesh.chaturv...@citrix.com wrote:

 Daan
 
 This is from the recent commit that you cherry-picked into 4.3. These 
 unintended consequences was my reason not to change so late. Anyhow please 
 review
 
 Animesh
 
 -Original Message-
 From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com] 
 Sent: Wednesday, January 29, 2014 2:32 PM
 To: dev@cloudstack.apache.org
 Cc: Daan Hoogland
 Subject: Re: userVmResponse no longer returns some parameters
 
 dcda79e2c02fda4e0fe52f61bf61712742e1f0db
 
 On 1/29/14, 2:22 PM, Animesh Chaturvedi animesh.chaturv...@citrix.com
 wrote:
 
 Alena do you know the commit ID
 
 -Original Message-
 From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
 Sent: Wednesday, January 29, 2014 2:03 PM
 To: dev@cloudstack.apache.org
 Cc: Daan Hoogland
 Subject: Re: userVmResponse no longer returns some parameters
 
 So the problem exists in 4.3 branch as well?
 
 =alena.
 
 On 1/29/14, 1:59 PM, Sheng Yang sh...@yasker.org wrote:
 
 That's the reason of:
 http://www.mail-archive.com/dev@cloudstack.apache.org/msg21948.html
 
 --Sheng
 
 
 On Wed, Jan 29, 2014 at 1:53 PM, Alena Prokharchyk  
 alena.prokharc...@citrix.com wrote:
 
 Hi Daan,
 
 In the latest master build I observed that listVirtualMachines 
 command no  longer returns parameters like:
 
 State
 displayName
 Account
 
 Its due to the commit that you've cherry-picked
 (fae05560218ef3bcb962bd424ae5dbf5cb8ede68)
 
 Setting corresponding fields in UserVmJoinVO to become final, 
 causing them  to always return NULL in the response.
 
 Can you please fix it? Current behavior will break all integration 
 code.
 
 Thanks,
 Alena.
 


Re: dropping [Responsiveness report] dev list

2014-01-30 Thread Chiradeep Vittal
It gives me an idea of the volume of such requests.

On 1/30/14 3:34 PM, Daan Hoogland daan.hoogl...@gmail.com wrote:

On Thu, Jan 30, 2014 at 4:17 PM, Sebastien Goasguen run...@gmail.com
wrote:
 Definitely, somehow we have to find a way to make sure that all
questions get answered.

I agree but don't have the feeling that my weekly exercise is serving
this goal.



Re: [4.3.0] CloudStack diff of Commit 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

2014-01-30 Thread Chiradeep Vittal
Attachments are stripped from this mailing list.

From: Wilder Rodrigues 
wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com
Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
Date: Thursday, January 30, 2014 7:56 AM
To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
Subject: [4.3.0] CloudStack diff of Commit 
2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

Hi folks,

We were going through the procedures in order to test the 4.3.0 RC and got some 
problems with the diff.

We got this last commit from Jessica/Animesh

CLOUDSTACK-5970: UI  network menu  guest network section  select non-VPC 
isolated... 
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a
 
4.3https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3

author

Jessica 
Wanghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Jessica+Wang;st=author
 
jessicaw...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=jessicaw...@apache.org;st=author


Wed, 29 Jan 2014 23:40:14 + (15:40 -0800)

committer

Animesh 
Chaturvedihttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Animesh+Chaturvedi;st=committer
 
anim...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=anim...@apache.org;st=committer


Wed, 29 Jan 2014 23:57:01 + (15:57 -0800)

commit

2f2adad50827f2249353d0258f3b5642a849479a

tree

29c688ca264e37de3caf0569ec8b06229d9270cfhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a

treehttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a
 | 
snapshothttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=snapshot;h=2f2adad50827f2249353d0258f3b5642a849479a;sf=tgz

parent

d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213

commithttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
 | 
diffhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a;hp=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213


And after downloading the source and pulling the repo, we did:

diff -r ~/dev/temp/cloudstack/apache-cloudstack-4.3.0-src 
~/dev/temp/cloudstack/tree

The outcome is in the attached file.

What are we doing wrong here, guys?

Thanks in advance.

Cheers,
Wilder


StratosphereSsp

2014-01-30 Thread Alena Prokharchyk
Hi Hiroaki-san,

I’ve seen that you’ve added 2 new api commands to 4.3-forward branch: 
addStratosphereSsp and deleteStratosphereSsp. There is one thing that needs to 
be fixed – the response name is not set in those commands. Check how “s_name” 
parameter is defined in other API commands, and please apply the same to your 
commands.

Also in CS you should be able to list the entries you add with add/create 
command, using corresponding list* command. I wonder why listStratosphereSsp 
command wasn’t introduced along with add/delete?

Thanks,
Alena.




RE: [4.3.0] CloudStack diff of Commit 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

2014-01-30 Thread Animesh Chaturvedi


-Original Message-
From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] 
Sent: Thursday, January 30, 2014 9:48 AM
To: dev@cloudstack.apache.org
Subject: Re: [4.3.0] CloudStack diff of Commit 
2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

Attachments are stripped from this mailing list.

From: Wilder Rodrigues 
wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com
Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
Date: Thursday, January 30, 2014 7:56 AM
To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
Subject: [4.3.0] CloudStack diff of Commit 
2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

Hi folks,

We were going through the procedures in order to test the 4.3.0 RC and got some 
problems with the diff.

We got this last commit from Jessica/Animesh

CLOUDSTACK-5970: UI  network menu  guest network section  select non-VPC 
isolated... 
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a
 
4.3https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3

author

Jessica 
Wanghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Jessica+Wang;st=author
 
jessicaw...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=jessicaw...@apache.org;st=author


Wed, 29 Jan 2014 23:40:14 + (15:40 -0800)

committer

Animesh 
Chaturvedihttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Animesh+Chaturvedi;st=committer
 
anim...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=anim...@apache.org;st=committer


Wed, 29 Jan 2014 23:57:01 + (15:57 -0800)

commit

2f2adad50827f2249353d0258f3b5642a849479a

tree

29c688ca264e37de3caf0569ec8b06229d9270cfhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a

treehttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a
 | 
snapshothttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=snapshot;h=2f2adad50827f2249353d0258f3b5642a849479a;sf=tgz

parent

d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213

commithttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
 | 
diffhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a;hp=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213


And after downloading the source and pulling the repo, we did:

diff -r ~/dev/temp/cloudstack/apache-cloudstack-4.3.0-src 
~/dev/temp/cloudstack/tree


[Animesh] Need more details on the error. Jessica's change has not been pulled 
into the 2nd RC I published. The currently published RC is with commit 
66d0b3d45bda73205b59693baeeefdf27e6c9c66 from Marcus

The outcome is in the attached file.

What are we doing wrong here, guys?

Thanks in advance.

Cheers,
Wilder


RE: userVmResponse no longer returns some parameters

2014-01-30 Thread Animesh Chaturvedi
Alena

Is the offending change restricted to this file only

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=server/src/com/cloud/api/query/vo/UserVmJoinVO.java;h=2bb0783595e7f6b5957cb94588a3cdd035baff55;hp=25079ca59f5d0ce0dabffcccb0b05972ea9e7a35;hb=dcda79e2c02fda4e0fe52f61bf61712742e1f0db;hpb=1fb32b58eb76d0c963bee7cc238d0f37a6dd8393



Animesh

-Original Message-
From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] 
Sent: Thursday, January 30, 2014 7:44 AM
To: dev@cloudstack.apache.org
Cc: daan.hoogl...@gmail.com
Subject: Re: userVmResponse no longer returns some parameters

Daan

Did you review this thread?

Thanks
Animesh

On Jan 29, 2014, at 3:50 PM, Animesh Chaturvedi 
animesh.chaturv...@citrix.com wrote:

 Daan
 
 This is from the recent commit that you cherry-picked into 4.3. These 
 unintended consequences was my reason not to change so late. Anyhow 
 please review
 
 Animesh
 
 -Original Message-
 From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
 Sent: Wednesday, January 29, 2014 2:32 PM
 To: dev@cloudstack.apache.org
 Cc: Daan Hoogland
 Subject: Re: userVmResponse no longer returns some parameters
 
 dcda79e2c02fda4e0fe52f61bf61712742e1f0db
 
 On 1/29/14, 2:22 PM, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com
 wrote:
 
 Alena do you know the commit ID
 
 -Original Message-
 From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
 Sent: Wednesday, January 29, 2014 2:03 PM
 To: dev@cloudstack.apache.org
 Cc: Daan Hoogland
 Subject: Re: userVmResponse no longer returns some parameters
 
 So the problem exists in 4.3 branch as well?
 
 =alena.
 
 On 1/29/14, 1:59 PM, Sheng Yang sh...@yasker.org wrote:
 
 That's the reason of:
 http://www.mail-archive.com/dev@cloudstack.apache.org/msg21948.html
 
 --Sheng
 
 
 On Wed, Jan 29, 2014 at 1:53 PM, Alena Prokharchyk  
 alena.prokharc...@citrix.com wrote:
 
 Hi Daan,
 
 In the latest master build I observed that listVirtualMachines 
 command no  longer returns parameters like:
 
 State
 displayName
 Account
 
 Its due to the commit that you've cherry-picked
 (fae05560218ef3bcb962bd424ae5dbf5cb8ede68)
 
 Setting corresponding fields in UserVmJoinVO to become final, 
 causing them  to always return NULL in the response.
 
 Can you please fix it? Current behavior will break all integration 
 code.
 
 Thanks,
 Alena.
 


Re: userVmResponse no longer returns some parameters

2014-01-30 Thread Alena Prokharchyk
No. This particular commit has changes in may files, and changes are so
random, so its hard to say the purpose of each.
I would advise to:

1) revert the commit altogether
2) Split changes into several separate commits based on purpose
3) Define clear purpose in the commit description before pushing it to the
repo.


-Alena.

On 1/30/14, 11:16 AM, Animesh Chaturvedi animesh.chaturv...@citrix.com
wrote:

Alena

Is the offending change restricted to this file only

https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=serv
er/src/com/cloud/api/query/vo/UserVmJoinVO.java;h=2bb0783595e7f6b5957cb945
88a3cdd035baff55;hp=25079ca59f5d0ce0dabffcccb0b05972ea9e7a35;hb=dcda79e2c0
2fda4e0fe52f61bf61712742e1f0db;hpb=1fb32b58eb76d0c963bee7cc238d0f37a6dd839
3



Animesh

-Original Message-
From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
Sent: Thursday, January 30, 2014 7:44 AM
To: dev@cloudstack.apache.org
Cc: daan.hoogl...@gmail.com
Subject: Re: userVmResponse no longer returns some parameters

Daan

Did you review this thread?

Thanks
Animesh

On Jan 29, 2014, at 3:50 PM, Animesh Chaturvedi
animesh.chaturv...@citrix.com wrote:

 Daan
 
 This is from the recent commit that you cherry-picked into 4.3. These
 unintended consequences was my reason not to change so late. Anyhow
 please review
 
 Animesh
 
 -Original Message-
 From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
 Sent: Wednesday, January 29, 2014 2:32 PM
 To: dev@cloudstack.apache.org
 Cc: Daan Hoogland
 Subject: Re: userVmResponse no longer returns some parameters
 
 dcda79e2c02fda4e0fe52f61bf61712742e1f0db
 
 On 1/29/14, 2:22 PM, Animesh Chaturvedi
 animesh.chaturv...@citrix.com
 wrote:
 
 Alena do you know the commit ID
 
 -Original Message-
 From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
 Sent: Wednesday, January 29, 2014 2:03 PM
 To: dev@cloudstack.apache.org
 Cc: Daan Hoogland
 Subject: Re: userVmResponse no longer returns some parameters
 
 So the problem exists in 4.3 branch as well?
 
 =alena.
 
 On 1/29/14, 1:59 PM, Sheng Yang sh...@yasker.org wrote:
 
 That's the reason of:
 http://www.mail-archive.com/dev@cloudstack.apache.org/msg21948.html
 
 --Sheng
 
 
 On Wed, Jan 29, 2014 at 1:53 PM, Alena Prokharchyk 
 alena.prokharc...@citrix.com wrote:
 
 Hi Daan,
 
 In the latest master build I observed that listVirtualMachines
 command no  longer returns parameters like:
 
 State
 displayName
 Account
 
 Its due to the commit that you've cherry-picked
 (fae05560218ef3bcb962bd424ae5dbf5cb8ede68)
 
 Setting corresponding fields in UserVmJoinVO to become final,
 causing them  to always return NULL in the response.
 
 Can you please fix it? Current behavior will break all integration
 code.
 
 Thanks,
 Alena.
 



Re: Findbugs report on 4.3-forward

2014-01-30 Thread Ian Duffy
Hi,

Sorry about the delay in replying to this have been doing exams at uni all
week.

Daan's change looks to change Rajani concern.

Might be me being naive but I fail to understand the concern fully...
The given character selection was roughly 72 with a string of length 20. If
my math is correct thats 72^20 different possible combinations...

Anyways, thanks for taking care of this Daan.

On 30 January 2014 13:28, Daan Hoogland daan.hoogl...@gmail.com wrote:

 Guys,

 I found two reported issues of category 'scary'. To satify Rajuri's
 concerns I would like to revert Ian's commit and checkin two changes
 that change returning
 byte[].toString()
 into
 Arrays.toString(byte[])
 on return statements of generatePassword methods.

 if no objections come in within a few...,
 Daan

 On Thu, Jan 30, 2014 at 1:53 PM, Daan Hoogland daan.hoogl...@gmail.com
 wrote:
  Animesh, Ian,
 
  Can you comment on this?
 
  I couldn't find any findbugs issues of the the scariest kind in
  yesterdays version of the 4.3 branch. What was solved that needs to go
  in in spite of Rajuri's reservations?
 
  thanks,
  Daan
 
  On Thu, Jan 30, 2014 at 11:21 AM, Daan Hoogland daan.hoogl...@gmail.com
 wrote:
  Sorry Rajani,
 
  I had seen no reaction to Ian's explenation  and the request by
  Animesh to pull it so I just did. let me look into it for a minute
 
  On Thu, Jan 30, 2014 at 5:36 AM, Rajani Karuturi
  rajani.karut...@citrix.com wrote:
  I see that the commit 9776e1af1c92486f5081b1ee8fa95cf0edb86b97 is
 already pushed to 4.3. I don't see any response on my concern as well.
  Is it just me or anyone else sees a security issue with the generate
 password change?
  Ian/Animesh/Daan, can you please respond?
 
  Thanks,
  ~Rajani
 
 
 
  On 29-Jan-2014, at 10:59 am, Rajani Karuturi 
 rajani.karut...@citrix.com wrote:
 
  Hi Ian,
  Before it is pushed to 4.3, can you fix the generate password change
 like i suggested in the other mail? This current change would make it less
 secure.
 
  Thanks,
  ~Rajani
 
 
 
  On 29-Jan-2014, at 8:03 am, Ian Duffy i...@ianduffy.ie wrote:
 
  Hi Animesh,
 
  Tested all those changes to detail. Those lines were removed due to
  unexpected behavior that I had not spotted until now.
 
  They were suppose to allow for better fall over between multiple
 domain
  controllers, how ever they were causing caching to occur. This meant
 if a
  users password was reset in LDAP the old password was still allowing
 login
  for a limited time.
 
  Please pull the changes forward,
  Thanks
 
  Ian.
  On 29 Jan 2014 00:07, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com
  wrote:
 
  If I look at this commit for example
 
 
 
 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=92b4f66d73562e4211d2d787554ff229dbeb5705
 
  It removes the two lines from LdapContextFactory.java
 
  environment.put(com.sun.jndi.ldap.read.timeout, 500);-
  environment.put(com.sun.jndi.ldap.connect.pool, true);
 
  Is that reported by find bug? I don't know this code  so not sure
 if it is
  intentional or not ?
 
  The point is there may be unintended risks in allowing late changes.
 
 
 
  -Original Message-
  From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com]
  Sent: Tuesday, January 28, 2014 3:35 PM
  To: dev@cloudstack.apache.org
  Subject: RE: Findbugs report on 4.3-forward
 
  Are you sure all of the ones are needed. A quick look at 20+
 commits from
  Daan  show many formatting changes that may not be necessary and
 hinder
  quick review.
 
  -Original Message-
  From: Hugo Trippaers [mailto:trip...@gmail.com]
  Sent: Tuesday, January 28, 2014 3:16 PM
  To: dev@cloudstack.apache.org
  Subject: Re: Findbugs report on 4.3-forward
 
 
 
  Sent from my iPhone
 
  On 28 jan. 2014, at 23:50, Animesh Chaturvedi 
  animesh.chaturv...@citrix.com wrote:
 
 
  -Original Message-
  From: Hugo Trippaers [mailto:trip...@gmail.com]
  Sent: Tuesday, January 28, 2014 2:37 PM
  To: dev@cloudstack.apache.org
  Cc: dev@cloudstack.apache.org
  Subject: Re: Findbugs report on 4.3-forward
 
  Hey Animesh,
 
  Not in agreement here. These are squashed bugs and we want as less
 bugs
  in the release as possible.
  [Animesh] I understand but once we enter RC phase we only limit
  important fixes. I have pulled in  2 commits from yours and 1 from
 Daan.
 
  We limited our fixes to only the important issues that we found.
 The other
  6000 issues between coverity and findbugs are still being triaged
 and will
  probably not make it into this release.
 
 
  This is why we test any RC before we release it.
  [Animesh] Of course but timing is a bit off, if this was done a
 month
  back it would have been fine.
  I say include all the big fixes we have in the release. If that
 means
  more testing before we cut the RC then that is what it is. I can't
  rightfully vote for a release with known issues with existing fixes.
  [Animesh] Any release will have known issues, if we have fixes but
 can't
  be 

Re: git commit: updated refs/heads/master to cb073e0

2014-01-30 Thread Wei ZHOU
Alena,

Should the line
 sb.and(zoneId, sb.entity().getZoneId(), SearchCriteria.Op.LIKE);
be  SearchCriteria.Op.EQ ?




2014-01-30 alena1...@apache.org

 Updated Branches:
   refs/heads/master 13e6c2fe7 - cb073e063


 listAutoscaleVmProfiles: added zoneId to the list of optional api
 parameters


 Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
 Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/cb073e06
 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/cb073e06
 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/cb073e06

 Branch: refs/heads/master
 Commit: cb073e063450097f348fd2afd4006487ca99919a
 Parents: 13e6c2f
 Author: Alena Prokharchyk alena.prokharc...@citrix.com
 Authored: Thu Jan 30 11:57:12 2014 -0800
 Committer: Alena Prokharchyk alena.prokharc...@citrix.com
 Committed: Thu Jan 30 12:36:56 2014 -0800

 --
  .../command/user/autoscale/ListAutoScaleVmProfilesCmd.java   | 8 
  server/src/com/cloud/network/as/AutoScaleManagerImpl.java| 6 ++
  2 files changed, 14 insertions(+)
 --



 http://git-wip-us.apache.org/repos/asf/cloudstack/blob/cb073e06/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleVmProfilesCmd.java
 --
 diff --git
 a/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleVmProfilesCmd.java
 b/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleVmProfilesCmd.java
 index 34c2543..5b4a46e 100644
 ---
 a/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleVmProfilesCmd.java
 +++
 b/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleVmProfilesCmd.java
 @@ -27,6 +27,7 @@ import
 org.apache.cloudstack.api.response.AutoScaleVmProfileResponse;
  import org.apache.cloudstack.api.response.ListResponse;
  import org.apache.cloudstack.api.response.ServiceOfferingResponse;
  import org.apache.cloudstack.api.response.TemplateResponse;
 +import org.apache.cloudstack.api.response.ZoneResponse;
  import org.apache.log4j.Logger;

  import com.cloud.network.as.AutoScaleVmProfile;
 @@ -53,6 +54,9 @@ public class ListAutoScaleVmProfilesCmd extends
 BaseListProjectAndAccountResourc
  @Parameter(name = ApiConstants.OTHER_DEPLOY_PARAMS, type =
 CommandType.STRING, description = the otherdeployparameters of the
 autoscale vm profile)
  private String otherDeployParams;

 +@Parameter(name = ApiConstants.ZONE_ID, type = CommandType.UUID,
 entityType = ZoneResponse.class, since = 4.4, description = availability
 zone for the auto deployed virtual machine)
 +private Long zoneId;
 +
  // ///
  // / Accessors ///
  // ///
 @@ -73,10 +77,14 @@ public class ListAutoScaleVmProfilesCmd extends
 BaseListProjectAndAccountResourc
  return serviceOffId;
  }

 +public Long getZoneId() {
 +return zoneId;
 +}
  // ///
  // / API Implementation///
  // ///

 +
  @Override
  public String getCommandName() {
  return s_name;


 http://git-wip-us.apache.org/repos/asf/cloudstack/blob/cb073e06/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 --
 diff --git a/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 b/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 index 7492c9a..7eeec9c 100644
 --- a/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 +++ b/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 @@ -438,6 +438,7 @@ public class AutoScaleManagerImplType extends
 ManagerBase implements AutoScale
  Long templateId = cmd.getTemplateId();
  String otherDeployParams = cmd.getOtherDeployParams();
  Long serviceOffId = cmd.getServiceOfferingId();
 +Long zoneId = cmd.getZoneId();

  SearchWrapperAutoScaleVmProfileVO searchWrapper = new
 SearchWrapperAutoScaleVmProfileVO(_autoScaleVmProfileDao,
 AutoScaleVmProfileVO.class, cmd, cmd.getId());
  SearchBuilderAutoScaleVmProfileVO sb =
 searchWrapper.getSearchBuilder();
 @@ -446,6 +447,7 @@ public class AutoScaleManagerImplType extends
 ManagerBase implements AutoScale
  sb.and(templateId, sb.entity().getTemplateId(),
 SearchCriteria.Op.EQ);
  sb.and(serviceOfferingId, sb.entity().getServiceOfferingId(),
 SearchCriteria.Op.EQ);
  sb.and(otherDeployParams, sb.entity().getOtherDeployParams(),
 SearchCriteria.Op.LIKE);
 +sb.and(zoneId, sb.entity().getZoneId(), SearchCriteria.Op.LIKE);
  

Re: git commit: updated refs/heads/master to cb073e0

2014-01-30 Thread Alena Prokharchyk
Yes, will fix that in a min.

-Alena.

On 1/30/14, 12:52 PM, Wei ZHOU ustcweiz...@gmail.com wrote:

Alena,

Should the line
 sb.and(zoneId, sb.entity().getZoneId(), SearchCriteria.Op.LIKE);
be  SearchCriteria.Op.EQ ?




2014-01-30 alena1...@apache.org

 Updated Branches:
   refs/heads/master 13e6c2fe7 - cb073e063


 listAutoscaleVmProfiles: added zoneId to the list of optional api
 parameters


 Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
 Commit: 
http://git-wip-us.apache.org/repos/asf/cloudstack/commit/cb073e06
 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/cb073e06
 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/cb073e06

 Branch: refs/heads/master
 Commit: cb073e063450097f348fd2afd4006487ca99919a
 Parents: 13e6c2f
 Author: Alena Prokharchyk alena.prokharc...@citrix.com
 Authored: Thu Jan 30 11:57:12 2014 -0800
 Committer: Alena Prokharchyk alena.prokharc...@citrix.com
 Committed: Thu Jan 30 12:36:56 2014 -0800

 --
  .../command/user/autoscale/ListAutoScaleVmProfilesCmd.java   | 8

  server/src/com/cloud/network/as/AutoScaleManagerImpl.java| 6 ++
  2 files changed, 14 insertions(+)
 --



 
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/cb073e06/api/src/o
rg/apache/cloudstack/api/command/user/autoscale/ListAutoScaleVmProfilesCm
d.java
 --
 diff --git
 
a/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleV
mProfilesCmd.java
 
b/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleV
mProfilesCmd.java
 index 34c2543..5b4a46e 100644
 ---
 
a/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleV
mProfilesCmd.java
 +++
 
b/api/src/org/apache/cloudstack/api/command/user/autoscale/ListAutoScaleV
mProfilesCmd.java
 @@ -27,6 +27,7 @@ import
 org.apache.cloudstack.api.response.AutoScaleVmProfileResponse;
  import org.apache.cloudstack.api.response.ListResponse;
  import org.apache.cloudstack.api.response.ServiceOfferingResponse;
  import org.apache.cloudstack.api.response.TemplateResponse;
 +import org.apache.cloudstack.api.response.ZoneResponse;
  import org.apache.log4j.Logger;

  import com.cloud.network.as.AutoScaleVmProfile;
 @@ -53,6 +54,9 @@ public class ListAutoScaleVmProfilesCmd extends
 BaseListProjectAndAccountResourc
  @Parameter(name = ApiConstants.OTHER_DEPLOY_PARAMS, type =
 CommandType.STRING, description = the otherdeployparameters of the
 autoscale vm profile)
  private String otherDeployParams;

 +@Parameter(name = ApiConstants.ZONE_ID, type = CommandType.UUID,
 entityType = ZoneResponse.class, since = 4.4, description =
availability
 zone for the auto deployed virtual machine)
 +private Long zoneId;
 +
  // ///
  // / Accessors ///
  // ///
 @@ -73,10 +77,14 @@ public class ListAutoScaleVmProfilesCmd extends
 BaseListProjectAndAccountResourc
  return serviceOffId;
  }

 +public Long getZoneId() {
 +return zoneId;
 +}
  // ///
  // / API Implementation///
  // ///

 +
  @Override
  public String getCommandName() {
  return s_name;


 
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/cb073e06/server/sr
c/com/cloud/network/as/AutoScaleManagerImpl.java
 --
 diff --git a/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 b/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 index 7492c9a..7eeec9c 100644
 --- a/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 +++ b/server/src/com/cloud/network/as/AutoScaleManagerImpl.java
 @@ -438,6 +438,7 @@ public class AutoScaleManagerImplType extends
 ManagerBase implements AutoScale
  Long templateId = cmd.getTemplateId();
  String otherDeployParams = cmd.getOtherDeployParams();
  Long serviceOffId = cmd.getServiceOfferingId();
 +Long zoneId = cmd.getZoneId();

  SearchWrapperAutoScaleVmProfileVO searchWrapper = new
 SearchWrapperAutoScaleVmProfileVO(_autoScaleVmProfileDao,
 AutoScaleVmProfileVO.class, cmd, cmd.getId());
  SearchBuilderAutoScaleVmProfileVO sb =
 searchWrapper.getSearchBuilder();
 @@ -446,6 +447,7 @@ public class AutoScaleManagerImplType extends
 ManagerBase implements AutoScale
  sb.and(templateId, sb.entity().getTemplateId(),
 SearchCriteria.Op.EQ);
  sb.and(serviceOfferingId, sb.entity().getServiceOfferingId(),
 SearchCriteria.Op.EQ);
  sb.and(otherDeployParams, sb.entity().getOtherDeployParams(),
 

Re: dropping [Responsiveness report] dev list

2014-01-30 Thread Marty Sweet
I find the report useful, unfortunately I find most requests from one off
users end up never being responding to by the op (which I would say is more
a mailing list issue - I had this issue when I sent my first question).



On Thu, Jan 30, 2014 at 5:48 PM, Chiradeep Vittal 
chiradeep.vit...@citrix.com wrote:

 It gives me an idea of the volume of such requests.

 On 1/30/14 3:34 PM, Daan Hoogland daan.hoogl...@gmail.com wrote:

 On Thu, Jan 30, 2014 at 4:17 PM, Sebastien Goasguen run...@gmail.com
 wrote:
  Definitely, somehow we have to find a way to make sure that all
 questions get answered.
 
 I agree but don't have the feeling that my weekly exercise is serving
 this goal.




volume lifecycle issue expunging

2014-01-30 Thread Marcus
I think there's a hole in the volume lifecycle.  I've been noticing
volumes lingering that should have been cleaned up, and it seems to be
a bug in the state machine for the volumes:

s_fsm.addTransition(Destroy, Event.ExpungingRequested, Expunging);
s_fsm.addTransition(Expunging, Event.ExpungingRequested, Expunging);
s_fsm.addTransition(Expunging, Event.OperationSucceeded, Expunged);
s_fsm.addTransition(Expunging, Event.OperationFailed, Expunging);

If a volume is in Destroy state, it goes to Expunging when the delete
operation is requested. If the delete fails, it remains in expunging.
The storage garbage collector will never try to clean up that volume
again, since it only lists volumes in 'Destroy' and attempts those.
You can only get to Expunging from Destroy, it makes sense to change
that last line to revert the volume state back to Destroy if the
expunge operation failed, so that it will try again next time.


RE: dropping [Responsiveness report] dev list

2014-01-30 Thread Donal Lafferty
+1

Measurable implies manageable.

 -Original Message-
 From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
 Sent: 30 January 2014 17:49
 To: dev@cloudstack.apache.org
 Subject: Re: dropping [Responsiveness report] dev list
 
 It gives me an idea of the volume of such requests.
 
 On 1/30/14 3:34 PM, Daan Hoogland daan.hoogl...@gmail.com wrote:
 
 On Thu, Jan 30, 2014 at 4:17 PM, Sebastien Goasguen run...@gmail.com
 wrote:
  Definitely, somehow we have to find a way to make sure that all
 questions get answered.
 
 I agree but don't have the feeling that my weekly exercise is serving
 this goal.



Re: volume lifecycle issue expunging

2014-01-30 Thread Mike Tutkowski
I agree, Marcus.


On Thu, Jan 30, 2014 at 2:42 PM, Marcus shadow...@gmail.com wrote:

 I think there's a hole in the volume lifecycle.  I've been noticing
 volumes lingering that should have been cleaned up, and it seems to be
 a bug in the state machine for the volumes:

 s_fsm.addTransition(Destroy, Event.ExpungingRequested,
 Expunging);
 s_fsm.addTransition(Expunging, Event.ExpungingRequested,
 Expunging);
 s_fsm.addTransition(Expunging, Event.OperationSucceeded,
 Expunged);
 s_fsm.addTransition(Expunging, Event.OperationFailed,
 Expunging);

 If a volume is in Destroy state, it goes to Expunging when the delete
 operation is requested. If the delete fails, it remains in expunging.
 The storage garbage collector will never try to clean up that volume
 again, since it only lists volumes in 'Destroy' and attempts those.
 You can only get to Expunging from Destroy, it makes sense to change
 that last line to revert the volume state back to Destroy if the
 expunge operation failed, so that it will try again next time.




-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*


Re: volume lifecycle issue expunging

2014-01-30 Thread Marcus
My one hesitation is that it may cause problems by surfacing other
bugs. For example, say a delete operation succeeds in deleting the
image file, but fails in some subsequent code. The next time it runs,
it may fail in deleting the image file (file not found), and thus get
stuck in an endless loop trying to clean up.  I think it's ultimately
good to surface those (better than potentially orphaning disks on
storage), but it may be something to watch for.

On Thu, Jan 30, 2014 at 2:56 PM, Mike Tutkowski
mike.tutkow...@solidfire.com wrote:
 I agree, Marcus.


 On Thu, Jan 30, 2014 at 2:42 PM, Marcus shadow...@gmail.com wrote:

 I think there's a hole in the volume lifecycle.  I've been noticing
 volumes lingering that should have been cleaned up, and it seems to be
 a bug in the state machine for the volumes:

 s_fsm.addTransition(Destroy, Event.ExpungingRequested,
 Expunging);
 s_fsm.addTransition(Expunging, Event.ExpungingRequested,
 Expunging);
 s_fsm.addTransition(Expunging, Event.OperationSucceeded,
 Expunged);
 s_fsm.addTransition(Expunging, Event.OperationFailed,
 Expunging);

 If a volume is in Destroy state, it goes to Expunging when the delete
 operation is requested. If the delete fails, it remains in expunging.
 The storage garbage collector will never try to clean up that volume
 again, since it only lists volumes in 'Destroy' and attempts those.
 You can only get to Expunging from Destroy, it makes sense to change
 that last line to revert the volume state back to Destroy if the
 expunge operation failed, so that it will try again next time.




 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the
 cloudhttp://solidfire.com/solution/overview/?video=play
 *(tm)*


Re: [4.3.0] CloudStack diff of Commit 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

2014-01-30 Thread Wilder Rodrigues
Thanks for the remark, Animesh. 

However, after the -1 from Geoff with regards to the problem with firewalling, 
port forwarding and load balancing on the UI, it seems a bit pointless to test 
the release without Jessica fixes, imho.

Cheers,
Wilder

Sent from my iPhone

 On 30 jan. 2014, at 20:06, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com wrote:
 
 
 
 -Original Message-
 From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] 
 Sent: Thursday, January 30, 2014 9:48 AM
 To: dev@cloudstack.apache.org
 Subject: Re: [4.3.0] CloudStack diff of Commit 
 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC
 
 Attachments are stripped from this mailing list.
 
 From: Wilder Rodrigues 
 wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com
 Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
 dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
 Date: Thursday, January 30, 2014 7:56 AM
 To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
 dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
 Subject: [4.3.0] CloudStack diff of Commit 
 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC
 
 Hi folks,
 
 We were going through the procedures in order to test the 4.3.0 RC and got 
 some problems with the diff.
 
 We got this last commit from Jessica/Animesh
 
 CLOUDSTACK-5970: UI  network menu  guest network section  select non-VPC 
 isolated... 
 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a
  
 4.3https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
 
 author
 
 Jessica 
 Wanghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Jessica+Wang;st=author
  
 jessicaw...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=jessicaw...@apache.org;st=author
 
 
 Wed, 29 Jan 2014 23:40:14 + (15:40 -0800)
 
 committer
 
 Animesh 
 Chaturvedihttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Animesh+Chaturvedi;st=committer
  
 anim...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=anim...@apache.org;st=committer
 
 
 Wed, 29 Jan 2014 23:57:01 + (15:57 -0800)
 
 commit
 
 2f2adad50827f2249353d0258f3b5642a849479a
 
 tree
 
 29c688ca264e37de3caf0569ec8b06229d9270cfhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a
 
 treehttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a
  | 
 snapshothttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=snapshot;h=2f2adad50827f2249353d0258f3b5642a849479a;sf=tgz
 
 parent
 
 d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
 
 commithttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
  | 
 diffhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a;hp=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
 
 
 And after downloading the source and pulling the repo, we did:
 
 diff -r ~/dev/temp/cloudstack/apache-cloudstack-4.3.0-src 
 ~/dev/temp/cloudstack/tree
 
 
 [Animesh] Need more details on the error. Jessica's change has not been 
 pulled into the 2nd RC I published. The currently published RC is with commit 
 66d0b3d45bda73205b59693baeeefdf27e6c9c66 from Marcus
 
 The outcome is in the attached file.
 
 What are we doing wrong here, guys?
 
 Thanks in advance.
 
 Cheers,
 Wilder


Re: volume lifecycle issue expunging

2014-01-30 Thread Mike Tutkowski
Yeah, unfortunately our code should be a bit more robust in this area. The
second (or more) time around, the failed deletion of a file should probably
not cause us trouble if, in fact, the file is gone (regardless of whether
or not your delete command actually deleted the file or it was gone for
some other reason).

I think, as you say, it's probably good to expose those other issues,
though.


On Thu, Jan 30, 2014 at 3:29 PM, Marcus shadow...@gmail.com wrote:

 My one hesitation is that it may cause problems by surfacing other
 bugs. For example, say a delete operation succeeds in deleting the
 image file, but fails in some subsequent code. The next time it runs,
 it may fail in deleting the image file (file not found), and thus get
 stuck in an endless loop trying to clean up.  I think it's ultimately
 good to surface those (better than potentially orphaning disks on
 storage), but it may be something to watch for.

 On Thu, Jan 30, 2014 at 2:56 PM, Mike Tutkowski
 mike.tutkow...@solidfire.com wrote:
  I agree, Marcus.
 
 
  On Thu, Jan 30, 2014 at 2:42 PM, Marcus shadow...@gmail.com wrote:
 
  I think there's a hole in the volume lifecycle.  I've been noticing
  volumes lingering that should have been cleaned up, and it seems to be
  a bug in the state machine for the volumes:
 
  s_fsm.addTransition(Destroy, Event.ExpungingRequested,
  Expunging);
  s_fsm.addTransition(Expunging, Event.ExpungingRequested,
  Expunging);
  s_fsm.addTransition(Expunging, Event.OperationSucceeded,
  Expunged);
  s_fsm.addTransition(Expunging, Event.OperationFailed,
  Expunging);
 
  If a volume is in Destroy state, it goes to Expunging when the delete
  operation is requested. If the delete fails, it remains in expunging.
  The storage garbage collector will never try to clean up that volume
  again, since it only lists volumes in 'Destroy' and attempts those.
  You can only get to Expunging from Destroy, it makes sense to change
  that last line to revert the volume state back to Destroy if the
  expunge operation failed, so that it will try again next time.
 
 
 
 
  --
  *Mike Tutkowski*
  *Senior CloudStack Developer, SolidFire Inc.*
  e: mike.tutkow...@solidfire.com
  o: 303.746.7302
  Advancing the way the world uses the
  cloudhttp://solidfire.com/solution/overview/?video=play
  *(tm)*




-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*


XenServer SR Question

2014-01-30 Thread Mike Tutkowski
Hi,

Does anyone know how a XenServer SR could (correctly) say more space is
being used than is allocated?

This is what the Size field of one of my shared SRs says:

Size: 20.1 GB used of 80 GB total (20 GB allocated)

I would have expected used to be = allocated at all times (typically I'd
expect it to be  allocated most of the time).

Any thoughts on this?

Thanks!

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*


Font question in GUI

2014-01-30 Thread Mike Tutkowski
Hi,

This is a minor issue.

Has anyone else noticed when you return to the GUI from another window that
sometimes the GUI's font has changed?

To get the GUI to switch back to the expected font, I have to refresh it by
clicking on one of the buttons in the left-hand pane.

Typically I use Chrome.

Thanks!

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*


Re: dropping [Responsiveness report] dev list

2014-01-30 Thread Daan Hoogland
I have the same experience as Marty several times. This is why I
started taking up this job, hoping we could structurally improve on
it. But the only use I have had of my work is for my own good and
interest. So some doubts creeped in. I think I have seen enough
positive reactions in this thread to keep up the (good) work.
I would very much appreciate it if someone came up with a genious idea
on how to use the stuff. It is hardly scientific data and as a
response guard I am usually very late.
whineSo i feel I am not doing a very good job and I like having the
feeling of doing effective and efficient work./whine

regards

On Thu, Jan 30, 2014 at 10:53 PM, Donal Lafferty
donal.laffe...@citrix.com wrote:
 +1

 Measurable implies manageable.

 -Original Message-
 From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
 Sent: 30 January 2014 17:49
 To: dev@cloudstack.apache.org
 Subject: Re: dropping [Responsiveness report] dev list

 It gives me an idea of the volume of such requests.

 On 1/30/14 3:34 PM, Daan Hoogland daan.hoogl...@gmail.com wrote:

 On Thu, Jan 30, 2014 at 4:17 PM, Sebastien Goasguen run...@gmail.com
 wrote:
  Definitely, somehow we have to find a way to make sure that all
 questions get answered.
 
 I agree but don't have the feeling that my weekly exercise is serving
 this goal.



Re: XenServer SR Question

2014-01-30 Thread Tim Mackey
I'm assuming that's the value from XenCenter. What does the cli say? I
could see this being just a formatting question.
On Jan 30, 2014 5:59 PM, Mike Tutkowski mike.tutkow...@solidfire.com
wrote:

 Hi,

 Does anyone know how a XenServer SR could (correctly) say more space is
 being used than is allocated?

 This is what the Size field of one of my shared SRs says:

 Size: 20.1 GB used of 80 GB total (20 GB allocated)

 I would have expected used to be = allocated at all times (typically I'd
 expect it to be  allocated most of the time).

 Any thoughts on this?

 Thanks!

 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the
 cloudhttp://solidfire.com/solution/overview/?video=play
 *(tm)*



Re: Font question in GUI

2014-01-30 Thread Marcus
yep


On Thu, Jan 30, 2014 at 4:01 PM, Mike Tutkowski
mike.tutkow...@solidfire.com wrote:
 Hi,

 This is a minor issue.

 Has anyone else noticed when you return to the GUI from another window that
 sometimes the GUI's font has changed?

 To get the GUI to switch back to the expected font, I have to refresh it by
 clicking on one of the buttons in the left-hand pane.

 Typically I use Chrome.

 Thanks!

 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the
 cloudhttp://solidfire.com/solution/overview/?video=play
 *(tm)*


Re: XenServer SR Question

2014-01-30 Thread Mike Tutkowski
Hi Tim,

You are correct that I was looking at XenCenter. Below is the output from
xe.

It looks like physical-size (below) corresponds to total in XenCenter,
physical-utilization (below) corresponds to used in XenCenter, and
virtual-allocation (below) corresponds to allocated in XenCenter.

Does that look right?

Thanks!

[root@XenServer-6 ~]# xe sr-list uuid=ec8db98c-8338-7bc7-75aa-f45207c32a83
params=all
uuid ( RO): ec8db98c-8338-7bc7-75aa-f45207c32a83
  name-label ( RW): /iqn.2010-01.com.solidfire:3y8w.vol-1.126/0
name-description ( RW): /iqn.2010-01.com.solidfire:3y8w.vol-1.126/0
host ( RO): shared
  allowed-operations (SRO): VDI.create; VDI.snapshot; PBD.create;
PBD.destroy; plug; update; VDI.destroy; scan; VDI.clone; VDI.resize; unplug
  current-operations (SRO):
VDIs (SRO): 8f14728f-938f-4e02-bc9c-8f67e86d5c86
PBDs (SRO): c4766cdc-7975-9ae7-5953-6fd831a5c654;
7ef960f0-4074-ef18-b17b-1f9a1ae8a6ff
  virtual-allocation ( RO): 21525168128
physical-utilisation ( RO): 21529362432
   physical-size ( RO): 85886763008
type ( RO): lvmoiscsi
content-type ( RO): user
  shared ( RW): true
   introduced-by ( RO): not in database
other-config (MRW):
   sm-config (MRO): allocation: thick; use_vhd: true;
multipathable: true; devserial: scsi-36f47acc133793877007e
   blobs ( RO):
 local-cache-enabled ( RO): false
tags (SRW):


On Thu, Jan 30, 2014 at 4:08 PM, Tim Mackey tmac...@gmail.com wrote:

 I'm assuming that's the value from XenCenter. What does the cli say? I
 could see this being just a formatting question.
 On Jan 30, 2014 5:59 PM, Mike Tutkowski mike.tutkow...@solidfire.com
 wrote:

  Hi,
 
  Does anyone know how a XenServer SR could (correctly) say more space is
  being used than is allocated?
 
  This is what the Size field of one of my shared SRs says:
 
  Size: 20.1 GB used of 80 GB total (20 GB allocated)
 
  I would have expected used to be = allocated at all times (typically I'd
  expect it to be  allocated most of the time).
 
  Any thoughts on this?
 
  Thanks!
 
  --
  *Mike Tutkowski*
  *Senior CloudStack Developer, SolidFire Inc.*
  e: mike.tutkow...@solidfire.com
  o: 303.746.7302
  Advancing the way the world uses the
  cloudhttp://solidfire.com/solution/overview/?video=play
  *(tm)*
 




-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*


Re: Urgent : Network stuck in implementing state

2014-01-30 Thread Bjoern Teipel
Thanks, I got one step further and the NPE is not coming up but I guess I
need to change the overall network state :?

014-01-30 15:23:16,159 DEBUG [cloud.network.NetworkManagerImpl]
(Job-Executor-67:job-1218 = [ 7de934e0-2b42-476b-83da-cf03109331fa ])
Network is not implemented: Ntwk[212|Guest|16]
2014-01-30 15:23:16,160 DEBUG [cloud.network.NetworkManagerImpl]
(Job-Executor-67:job-1218 = [ 7de934e0-2b42-476b-83da-cf03109331fa ])
Network is not not in the correct state to be destroyed: Implementing
2014-01-30 15:23:16,164 DEBUG [cloud.async.AsyncJobManagerImpl]
(Job-Executor-67:job-1218 = [ 7de934e0-2b42-476b-83da-cf03109331fa ])
Complete async job-1218 = [ 7de934e0-2b42-476b-83da-cf03109331fa ],
jobStatus: 2, resultCode: 530, result: Error Code: 530 Error text: Failed
to delete network



On Mon, Jan 27, 2014 at 10:57 PM, Sanjeev Neelarapu 
sanjeev.neelar...@citrix.com wrote:

 Hi Bjoern,

 Since the nic's instance_id is NULL you can set the removed field with
 some timestamp (e.g. now()). Also check the external IP address state in
 user_ip_address table. If this ip address state is in allocated set it to
 NULL and try delete network again.

 -Sanjeev

 -Original Message-
 From: Bjoern Teipel [mailto:bjoern.tei...@gmail.com]
 Sent: Monday, January 27, 2014 12:44 PM
 To: users; dev@cloudstack.apache.org
 Subject: Urgent : Network stuck in implementing state

 Hi guys,

 I had trouble while I was creating a new VLAN and now I can't delete it
 anymore, because it's stuck in implementing state.
 I guess that happened after the VR did not came up and I restarted the
 management server so I could delete the VR and all addresses.

 If I would spin up a guest, the VR and the guest would come up but I sill
 want to get rid of it since I have an error in the network service offering.
 After I cleaned all up though the GUI I can only see an external IP for
 the old VR is still associated with the VLAN but can't be deleted (I don't
 get it offered)

 In the logs I found this and the NPE matches the table content :



 2014-01-26 13:53:02,857 DEBUG [cloud.network.NetworkManagerImpl]
 (Job-Executor-2:job-1127 = [ f90b84ab-0d4a-45ce-9cd1-d73b63e24020 ])
 Network id=212 is destroyed successfully, cleaning up corresponding
 resources  now.
 2014-01-26 13:53:02,871 DEBUG [network.guru.DirectNetworkGuru]
 (Job-Executor-2:job-1127 = [ f90b84ab-0d4a-45ce-9cd1-d73b63e24020 ])
 Releasing ip 10.16.48.1 of placeholder nic Nic[189-null-null-10.16.48.1]
 2014-01-26 13:53:02,872 DEBUG [db.Transaction.Transaction]
 (Job-Executor-2:job-1127 = [ f90b84ab-0d4a-45ce-9cd1-d73b63e24020 ])
 Rolling back the transaction: Time = 14 Name =
  -AsyncJobManagerImpl$1.run:494-Exec

 utors$RunnableAdapter.call:471-FutureTask.run:262-ThreadPoolExecutor.runWorker:1145-ThreadPoolExecutor$Worker.run:615-Thread.run:744;
 called by
 -Transaction.rollback:897-Transaction.removeUpTo:840-Transaction.cl

 ose:664-TransactionContextBuilder.interceptException:63-ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept:133-NetworkManagerImpl.destroyNetwork:3144-ComponentInstantiationPostProcessor$Intercep

 torDispatcher.intercept:125-NetworkServiceImpl.deleteNetwork:1767-ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept:125-DeleteNetworkCmd.execute:70-ApiDispatcher.dispatch:158-AsyncJobManagerImp
 l$1.run:531
 2014-01-26 13:53:02,877 ERROR [cloud.async.AsyncJobManagerImpl]
 (Job-Executor-2:job-1127 = [ f90b84ab-0d4a-45ce-9cd1-d73b63e24020 ])
 Unexpected exception while executing
 org.apache.cloudstack.api.command.user.ne
 twork.DeleteNetworkCmd
 java.lang.NullPointerException
 at
 com.cloud.network.guru.DirectNetworkGuru.trash(DirectNetworkGuru.java:311)
 at

 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
 at

 com.cloud.network.NetworkManagerImpl.destroyNetwork(NetworkManagerImpl.java:3144)
 at

 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
 at

 com.cloud.network.NetworkServiceImpl.deleteNetwork(NetworkServiceImpl.java:1767)
 at

 com.cloud.utils.component.ComponentInstantiationPostProcessor$InterceptorDispatcher.intercept(ComponentInstantiationPostProcessor.java:125)
 at

 org.apache.cloudstack.api.command.user.network.DeleteNetworkCmd.execute(DeleteNetworkCmd.java:70)
 at com.cloud.api.ApiDispatcher.dispatch(ApiDispatcher.java:158)
 at
 com.cloud.async.AsyncJobManagerImpl$1.run(AsyncJobManagerImpl.java:531)
 at
 java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
 at java.util.concurrent.FutureTask.run(FutureTask.java:262)
 at

 java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
 at

 java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
 at 

RE: [4.3.0] CloudStack diff of Commit 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

2014-01-30 Thread Animesh Chaturvedi


-Original Message-
From: Wilder Rodrigues [mailto:wrodrig...@schubergphilis.com] 
Sent: Thursday, January 30, 2014 2:31 PM
To: dev@cloudstack.apache.org
Cc: dev@cloudstack.apache.org
Subject: Re: [4.3.0] CloudStack diff of Commit 
2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC

Thanks for the remark, Animesh. 

However, after the -1 from Geoff with regards to the problem with firewalling, 
port forwarding and load balancing on the UI, it seems a bit pointless to test 
the release without Jessica fixes, imho.
[Animesh] Her fix will be in for next RC. Thanks for the feedback

Cheers,
Wilder

Sent from my iPhone

 On 30 jan. 2014, at 20:06, Animesh Chaturvedi 
 animesh.chaturv...@citrix.com wrote:
 
 
 
 -Original Message-
 From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] 
 Sent: Thursday, January 30, 2014 9:48 AM
 To: dev@cloudstack.apache.org
 Subject: Re: [4.3.0] CloudStack diff of Commit 
 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC
 
 Attachments are stripped from this mailing list.
 
 From: Wilder Rodrigues 
 wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com
 Reply-To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
 dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
 Date: Thursday, January 30, 2014 7:56 AM
 To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org 
 dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
 Subject: [4.3.0] CloudStack diff of Commit 
 2f2adad50827f2249353d0258f3b5642a849479a vs 4.3.0-SRC
 
 Hi folks,
 
 We were going through the procedures in order to test the 4.3.0 RC and got 
 some problems with the diff.
 
 We got this last commit from Jessica/Animesh
 
 CLOUDSTACK-5970: UI  network menu  guest network section  select non-VPC 
 isolated... 
 https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a
  
 4.3https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
 
 author
 
 Jessica 
 Wanghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Jessica+Wang;st=author
  
 jessicaw...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=jessicaw...@apache.org;st=author
 
 
 Wed, 29 Jan 2014 23:40:14 + (15:40 -0800)
 
 committer
 
 Animesh 
 Chaturvedihttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=Animesh+Chaturvedi;st=committer
  
 anim...@apache.orghttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=search;h=2f2adad50827f2249353d0258f3b5642a849479a;s=anim...@apache.org;st=committer
 
 
 Wed, 29 Jan 2014 23:57:01 + (15:57 -0800)
 
 commit
 
 2f2adad50827f2249353d0258f3b5642a849479a
 
 tree
 
 29c688ca264e37de3caf0569ec8b06229d9270cfhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a
 
 treehttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tree;h=29c688ca264e37de3caf0569ec8b06229d9270cf;hb=2f2adad50827f2249353d0258f3b5642a849479a
  | 
 snapshothttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=snapshot;h=2f2adad50827f2249353d0258f3b5642a849479a;sf=tgz
 
 parent
 
 d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
 
 commithttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
  | 
 diffhttps://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2f2adad50827f2249353d0258f3b5642a849479a;hp=d6bbfbbc59ee3956fd0e1c45d9d73a2f97d33213
 
 
 And after downloading the source and pulling the repo, we did:
 
 diff -r ~/dev/temp/cloudstack/apache-cloudstack-4.3.0-src 
 ~/dev/temp/cloudstack/tree
 
 
 [Animesh] Need more details on the error. Jessica's change has not been 
 pulled into the 2nd RC I published. The currently published RC is with commit 
 66d0b3d45bda73205b59693baeeefdf27e6c9c66 from Marcus
 
 The outcome is in the attached file.
 
 What are we doing wrong here, guys?
 
 Thanks in advance.
 
 Cheers,
 Wilder


Re: Review Request 17348: VPC support for contrail plugin

2014-01-30 Thread Sheng Yang

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17348/#review33274
---



plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManagerImpl.java
https://reviews.apache.org/r/17348/#comment62673

I just notice that you didn't follow Java function/variable style. The 
first character of variable/functions should be lower case. You may want to use 
refactor function of IDE to correct them all(but send in an separate patch for 
the existed file please).


Others looks good to me. Please correct the name style issue in this patch, 
rebase to latest master and generate the patch using: git-format-patch.

Thanks.

- Sheng Yang


On Jan. 25, 2014, 1:26 a.m., Suresh Balineni wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17348/
 ---
 
 (Updated Jan. 25, 2014, 1:26 a.m.)
 
 
 Review request for cloudstack.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 VPC Element implementation in contrail plugin
 
 
 Diffs
 -
 
   api/src/com/cloud/network/Network.java 62151a4 
   
 plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/spring-contrail-context.xml
  4614af7 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailGuru.java
  704dedd 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManager.java
  6853d1e 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManagerImpl.java
  ef76e4c 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVpcElementImpl.java
  PRE-CREATION 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServerDBSyncImpl.java
  b1f4718 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/NetworkPolicyModel.java
  8ec2ff7 
   server/src/com/cloud/configuration/ConfigurationManagerImpl.java 9d148fa 
   server/src/com/cloud/network/vpc/VpcManagerImpl.java 2face90 
 
 Diff: https://reviews.apache.org/r/17348/diff/
 
 
 Testing
 ---
 
 This has been verified in 4.3 and porting the same in master branch
 
 
 Thanks,
 
 Suresh Balineni
 




Re: XenServer SR Question

2014-01-30 Thread Tim Mackey
I'll need to confirm with engineering, but that makes sense. I'll also see
if there is a different format specifier in the XenCenter string
On Jan 30, 2014 6:27 PM, Mike Tutkowski mike.tutkow...@solidfire.com
wrote:

 Hi Tim,

 You are correct that I was looking at XenCenter. Below is the output from
 xe.

 It looks like physical-size (below) corresponds to total in XenCenter,
 physical-utilization (below) corresponds to used in XenCenter, and
 virtual-allocation (below) corresponds to allocated in XenCenter.

 Does that look right?

 Thanks!

 [root@XenServer-6 ~]# xe sr-list uuid=ec8db98c-8338-7bc7-75aa-f45207c32a83
 params=all
 uuid ( RO): ec8db98c-8338-7bc7-75aa-f45207c32a83
   name-label ( RW): /iqn.2010-01.com.solidfire:3y8w.vol-1.126/0
 name-description ( RW): /iqn.2010-01.com.solidfire:3y8w.vol-1.126/0
 host ( RO): shared
   allowed-operations (SRO): VDI.create; VDI.snapshot; PBD.create;
 PBD.destroy; plug; update; VDI.destroy; scan; VDI.clone; VDI.resize; unplug
   current-operations (SRO):
 VDIs (SRO): 8f14728f-938f-4e02-bc9c-8f67e86d5c86
 PBDs (SRO): c4766cdc-7975-9ae7-5953-6fd831a5c654;
 7ef960f0-4074-ef18-b17b-1f9a1ae8a6ff
   virtual-allocation ( RO): 21525168128
 physical-utilisation ( RO): 21529362432
physical-size ( RO): 85886763008
 type ( RO): lvmoiscsi
 content-type ( RO): user
   shared ( RW): true
introduced-by ( RO): not in database
 other-config (MRW):
sm-config (MRO): allocation: thick; use_vhd: true;
 multipathable: true; devserial: scsi-36f47acc133793877007e
blobs ( RO):
  local-cache-enabled ( RO): false
 tags (SRW):


 On Thu, Jan 30, 2014 at 4:08 PM, Tim Mackey tmac...@gmail.com wrote:

  I'm assuming that's the value from XenCenter. What does the cli say? I
  could see this being just a formatting question.
  On Jan 30, 2014 5:59 PM, Mike Tutkowski mike.tutkow...@solidfire.com
  wrote:
 
   Hi,
  
   Does anyone know how a XenServer SR could (correctly) say more space is
   being used than is allocated?
  
   This is what the Size field of one of my shared SRs says:
  
   Size: 20.1 GB used of 80 GB total (20 GB allocated)
  
   I would have expected used to be = allocated at all times (typically
 I'd
   expect it to be  allocated most of the time).
  
   Any thoughts on this?
  
   Thanks!
  
   --
   *Mike Tutkowski*
   *Senior CloudStack Developer, SolidFire Inc.*
   e: mike.tutkow...@solidfire.com
   o: 303.746.7302
   Advancing the way the world uses the
   cloudhttp://solidfire.com/solution/overview/?video=play
   *(tm)*
  
 



 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the
 cloudhttp://solidfire.com/solution/overview/?video=play
 *(tm)*



Re: XenServer SR Question

2014-01-30 Thread Mike Tutkowski
Great - thanks!

I was just trying to make sense of the numbers. :)


On Thu, Jan 30, 2014 at 4:36 PM, Tim Mackey tmac...@gmail.com wrote:

 I'll need to confirm with engineering, but that makes sense. I'll also see
 if there is a different format specifier in the XenCenter string
 On Jan 30, 2014 6:27 PM, Mike Tutkowski mike.tutkow...@solidfire.com
 wrote:

  Hi Tim,
 
  You are correct that I was looking at XenCenter. Below is the output from
  xe.
 
  It looks like physical-size (below) corresponds to total in XenCenter,
  physical-utilization (below) corresponds to used in XenCenter, and
  virtual-allocation (below) corresponds to allocated in XenCenter.
 
  Does that look right?
 
  Thanks!
 
  [root@XenServer-6 ~]# xe sr-list
 uuid=ec8db98c-8338-7bc7-75aa-f45207c32a83
  params=all
  uuid ( RO): ec8db98c-8338-7bc7-75aa-f45207c32a83
name-label ( RW):
 /iqn.2010-01.com.solidfire:3y8w.vol-1.126/0
  name-description ( RW):
 /iqn.2010-01.com.solidfire:3y8w.vol-1.126/0
  host ( RO): shared
allowed-operations (SRO): VDI.create; VDI.snapshot; PBD.create;
  PBD.destroy; plug; update; VDI.destroy; scan; VDI.clone; VDI.resize;
 unplug
current-operations (SRO):
  VDIs (SRO): 8f14728f-938f-4e02-bc9c-8f67e86d5c86
  PBDs (SRO): c4766cdc-7975-9ae7-5953-6fd831a5c654;
  7ef960f0-4074-ef18-b17b-1f9a1ae8a6ff
virtual-allocation ( RO): 21525168128
  physical-utilisation ( RO): 21529362432
 physical-size ( RO): 85886763008
  type ( RO): lvmoiscsi
  content-type ( RO): user
shared ( RW): true
 introduced-by ( RO): not in database
  other-config (MRW):
 sm-config (MRO): allocation: thick; use_vhd: true;
  multipathable: true; devserial: scsi-36f47acc133793877007e
 blobs ( RO):
   local-cache-enabled ( RO): false
  tags (SRW):
 
 
  On Thu, Jan 30, 2014 at 4:08 PM, Tim Mackey tmac...@gmail.com wrote:
 
   I'm assuming that's the value from XenCenter. What does the cli say? I
   could see this being just a formatting question.
   On Jan 30, 2014 5:59 PM, Mike Tutkowski 
 mike.tutkow...@solidfire.com
   wrote:
  
Hi,
   
Does anyone know how a XenServer SR could (correctly) say more space
 is
being used than is allocated?
   
This is what the Size field of one of my shared SRs says:
   
Size: 20.1 GB used of 80 GB total (20 GB allocated)
   
I would have expected used to be = allocated at all times (typically
  I'd
expect it to be  allocated most of the time).
   
Any thoughts on this?
   
Thanks!
   
--
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*
   
  
 
 
 
  --
  *Mike Tutkowski*
  *Senior CloudStack Developer, SolidFire Inc.*
  e: mike.tutkow...@solidfire.com
  o: 303.746.7302
  Advancing the way the world uses the
  cloudhttp://solidfire.com/solution/overview/?video=play
  *(tm)*
 




-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*


CLOUDSTACK-5406

2014-01-30 Thread Francois Gaudreault

Hi,

I saw this bug as been fixed for the secondary storage limits in 4.3:
CLOUDSTACK-5406

But it seems to impact the primary storage limit count as well, at least 
in 4.2.1. Is the primary storage count fixed in 4.3 (I can't test right 
now) as well?


Thanks!

--
Francois Gaudreault
Architecte de Solution Cloud | Cloud Solutions Architect
fgaudrea...@cloudops.com
514-629-6775
- - -
CloudOps
420 rue Guy
Montréal QC  H3J 1S6
www.cloudops.com
@CloudOps_



RE: CLOUDSTACK-5406

2014-01-30 Thread Animesh Chaturvedi
Copying the folks that worked on it

-Original Message-
From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com] 
Sent: Thursday, January 30, 2014 3:49 PM
To: dev@cloudstack.apache.org
Subject: CLOUDSTACK-5406

Hi,

I saw this bug as been fixed for the secondary storage limits in 4.3:
CLOUDSTACK-5406

But it seems to impact the primary storage limit count as well, at least in 
4.2.1. Is the primary storage count fixed in 4.3 (I can't test right
now) as well?

Thanks!

--
Francois Gaudreault
Architecte de Solution Cloud | Cloud Solutions Architect 
fgaudrea...@cloudops.com
514-629-6775
- - -
CloudOps
420 rue Guy
Montréal QC  H3J 1S6
www.cloudops.com
@CloudOps_



Re: Review Request 17348: VPC support for contrail plugin

2014-01-30 Thread Suresh Balineni

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17348/
---

(Updated Jan. 31, 2014, 12:19 a.m.)


Review request for cloudstack.


Changes
---

Hi Sheng,

I have created a new diff with your suggested changes.
Please review the same.

Thanks,
Suresh


Repository: cloudstack-git


Description
---

VPC Element implementation in contrail plugin


Diffs (updated)
-

  api/src/com/cloud/network/Network.java 62151a4 
  
plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/spring-contrail-context.xml
 4614af7 
  
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailGuru.java
 704dedd 
  
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManager.java
 6853d1e 
  
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManagerImpl.java
 ef76e4c 
  
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVpcElementImpl.java
 PRE-CREATION 
  
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServerDBSyncImpl.java
 b1f4718 
  
plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/NetworkPolicyModel.java
 8ec2ff7 
  server/src/com/cloud/configuration/ConfigurationManagerImpl.java 2003716 
  server/src/com/cloud/network/vpc/VpcManagerImpl.java 51e476f 

Diff: https://reviews.apache.org/r/17348/diff/


Testing
---

This has been verified in 4.3 and porting the same in master branch


Thanks,

Suresh Balineni



Jenkins build is back to normal : cloudstack-4.3-maven-build #459

2014-01-30 Thread jenkins
See http://jenkins.buildacloud.org/job/cloudstack-4.3-maven-build/459/changes



Review Request 17576: Detect log unknown parameters for CS API and refactoring

2014-01-30 Thread Antonio Fornie

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17576/
---

Review request for cloudstack, daan Hoogland and Hugo Trippaers.


Repository: cloudstack-git


Description
---

Detect  log unknown parameters for CS API and refactoring. There are plenty of 
changes for just refactoring purpose, but they are all related, most of them 
interdependant


Diffs
-

  api/src/org/apache/cloudstack/api/ApiConstants.java d500303 
  api/src/org/apache/cloudstack/api/BaseCmd.java 4229ec9 
  api/src/org/apache/cloudstack/api/BaseListCmd.java c1a4b4c 
  
api/src/org/apache/cloudstack/api/command/user/autoscale/CreateAutoScaleVmProfileCmd.java
 bee1b22 
  server/src/com/cloud/api/ApiAsyncJobDispatcher.java 71ac616 
  server/src/com/cloud/api/ApiDispatcher.java 55ef53a 
  server/src/com/cloud/api/ApiServer.java d715db6 
  server/src/com/cloud/api/ApiServlet.java 46f7eba 
  server/src/com/cloud/api/ParameterHandler.java PRE-CREATION 
  server/src/com/cloud/api/dispatch/DispatchChainFactory.java PRE-CREATION 
  server/src/com/cloud/api/dispatch/DispatchWorker.java PRE-CREATION 
  server/src/com/cloud/api/dispatch/ParamGenericValidationWorker.java 
PRE-CREATION 
  server/src/com/cloud/api/dispatch/ParamProcessWorker.java PRE-CREATION 
  server/src/com/cloud/api/dispatch/ParamSemanticValidationWorker.java 
PRE-CREATION 
  server/src/com/cloud/network/as/AutoScaleManagerImpl.java a2cc32e 
  server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
bf2516a 
  server/src/com/cloud/storage/snapshot/SnapshotSchedulerImpl.java 3159059 
  server/test/com/cloud/api/ApiDispatcherTest.java 7314a57 

Diff: https://reviews.apache.org/r/17576/diff/


Testing
---

mvn full build and manual test of functionalities


Thanks,

Antonio Fornie



RE: 4.3 commit cherry pick

2014-01-30 Thread Animesh Chaturvedi
Pulling in to 4.3 now

-Original Message-
From: Jayapal Reddy Uradi 
Sent: Thursday, January 30, 2014 6:10 AM
To: dev@cloudstack.apache.org
Cc: Animesh Chaturvedi
Subject: Re: 4.3 commit cherry pick

Hi Animesh,

Please cherry pick the following commits from 4.3-forward to 4.3 branch.

7255e50f2051aff1cb5e1991e2dde4ef48f57454
888b90677623da745a01db0a5087272f0dabe42c

Thanks,
Jayapal
On 30-Jan-2014, at 2:36 PM, Jayapal Reddy jayapalreddy.ur...@citrix.com wrote:

 Hi Animesh,
 
 Can you please cherry pick commit to 
 7255e50f2051aff1cb5e1991e2dde4ef48f57454 4.3 branch from 4.3-forward.
 
 Thanks,
 Jayapal



RE: 4.3 commit cherry pick

2014-01-30 Thread Animesh Chaturvedi
Hiroaki I am waiting on resolution on concern from Alena in other thread

-Original Message-
From: Hiroaki KAWAI [mailto:ka...@stratosphere.co.jp] 
Sent: Thursday, January 30, 2014 3:35 AM
To: dev@cloudstack.apache.org
Subject: Re: 4.3 commit cherry pick

Hi,

I found that our plugin was broken, so please cherry pick 
db3dc2ee17c699f1abf42ea2fbe09ec2748caa4a in 4.3-forward.


(2014/01/30 18:06), Jayapal Reddy Uradi wrote:
 Hi Animesh,
 
 Can you please cherry pick commit to 
 7255e50f2051aff1cb5e1991e2dde4ef48f57454 4.3 branch from 4.3-forward.
 
 Thanks,
 Jayapal
 



RE: StratosphereSsp

2014-01-30 Thread Animesh Chaturvedi
Hiroaki-san

I will wait on your comment from Alena when I review your commit 
db3dc2ee17c699f1abf42ea2fbe09ec2748caa4a  it's adding new APIs now which is 
late for 4.3

Animesh



-Original Message-
From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com] 
Sent: Thursday, January 30, 2014 10:48 AM
To: ka...@stratosphere.co.jp; dev@cloudstack.apache.org
Subject: StratosphereSsp

Hi Hiroaki-san,

I've seen that you've added 2 new api commands to 4.3-forward branch: 
addStratosphereSsp and deleteStratosphereSsp. There is one thing that needs to 
be fixed - the response name is not set in those commands. Check how s_name 
parameter is defined in other API commands, and please apply the same to your 
commands.

Also in CS you should be able to list the entries you add with add/create 
command, using corresponding list* command. I wonder why listStratosphereSsp 
command wasn't introduced along with add/delete?

Thanks,
Alena.




Re: Review Request 17348: VPC support for contrail plugin

2014-01-30 Thread Sheng Yang

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17348/#review33282
---

Ship it!


Applied to master.

Thanks!

- Sheng Yang


On Jan. 31, 2014, 12:19 a.m., Suresh Balineni wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17348/
 ---
 
 (Updated Jan. 31, 2014, 12:19 a.m.)
 
 
 Review request for cloudstack.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 VPC Element implementation in contrail plugin
 
 
 Diffs
 -
 
   api/src/com/cloud/network/Network.java 62151a4 
   
 plugins/network-elements/juniper-contrail/resources/META-INF/cloudstack/contrail/spring-contrail-context.xml
  4614af7 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailGuru.java
  704dedd 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManager.java
  6853d1e 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailManagerImpl.java
  ef76e4c 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailVpcElementImpl.java
  PRE-CREATION 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ServerDBSyncImpl.java
  b1f4718 
   
 plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/NetworkPolicyModel.java
  8ec2ff7 
   server/src/com/cloud/configuration/ConfigurationManagerImpl.java 2003716 
   server/src/com/cloud/network/vpc/VpcManagerImpl.java 51e476f 
 
 Diff: https://reviews.apache.org/r/17348/diff/
 
 
 Testing
 ---
 
 This has been verified in 4.3 and porting the same in master branch
 
 
 Thanks,
 
 Suresh Balineni
 




Re: StratosphereSsp

2014-01-30 Thread Hiroaki KAWAI
Hi,

The response name is obtained from the annotation,
because I really did not want to put typo in String itself.
Note s_name is usually private, and getCommandName() responds.

list* command is not there because simply we did not need it.
It might be better to have list* command, but it will take
some time to implement it.

(2014/01/31 3:47), Alena Prokharchyk wrote:
 Hi Hiroaki-san,
 
 I’ve seen that you’ve added 2 new api commands to 4.3-forward branch: 
 addStratosphereSsp and deleteStratosphereSsp. There is one thing that needs 
 to be fixed – the response name is not set in those commands. Check how 
 “s_name” parameter is defined in other API commands, and please apply the 
 same to your commands.
 
 Also in CS you should be able to list the entries you add with add/create 
 command, using corresponding list* command. I wonder why listStratosphereSsp 
 command wasn’t introduced along with add/delete?
 
 Thanks,
 Alena.
 
 
 



[CANCELLED] [VOTE] Apache CloudStack 4.3.0 (second round)

2014-01-30 Thread Animesh Chaturvedi
Formally cancelling this VOTE.

I have picked up a number of issues from 4.3-forward besides the SSL offload 
API response issue. I am almost ready to push the 3rd RC. 

Thanks
Animesh

-Original Message-
From: Animesh Chaturvedi 
Sent: Tuesday, January 28, 2014 8:54 PM
To: dev@cloudstack.apache.org
Subject: [VOTE] Apache CloudStack 4.3.0 (second round)

Hi All,
 
I've created a 4.3.0 release, with the following artifacts up for a
vote:
 
Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3
  
Commit: 66d0b3d45bda73205b59693baeeefdf27e6c9c66

 
List of changes:
Release notes are still in progress in a different repo, so providing links 
here for now New Features in 4.3: 
https://issues.apache.org/jira/issues/?filter=12325248
Improvement in 4.3: https://issues.apache.org/jira/issues/?filter=12325249
Issues fixed in 4.3 https://issues.apache.org/jira/issues/?filter=12326161
Known Issues in 4.3: https://issues.apache.org/jira/issues/?filter=12326162

 
Source release (checksums and signatures are available at the same
location):
https://dist.apache.org/repos/dist/dev/cloudstack/4.3.0/   
 
PGP release keys (signed using 94BE0D7C):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS   
 
Testing instructions are here:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+test+procedure
 
Vote will be open for 72 hours (excluding weekend). I will count the VOTE 
Sunday night PST.
 
For sanity in tallying the vote, can PMC members please be sure to indicate 
(binding) with their vote?


[ ] +1  approve
[ ] +0  no opinion
[ ] -1  disapprove (and reason why)



Re: Font question in GUI

2014-01-30 Thread Mike Tutkowski
Anyone know if there is already a JIRA ticket open for this issue?

If not, I can open one.

Thanks!


On Thu, Jan 30, 2014 at 4:20 PM, Marcus shadow...@gmail.com wrote:

 yep


 On Thu, Jan 30, 2014 at 4:01 PM, Mike Tutkowski
 mike.tutkow...@solidfire.com wrote:
  Hi,
 
  This is a minor issue.
 
  Has anyone else noticed when you return to the GUI from another window
 that
  sometimes the GUI's font has changed?
 
  To get the GUI to switch back to the expected font, I have to refresh it
 by
  clicking on one of the buttons in the left-hand pane.
 
  Typically I use Chrome.
 
  Thanks!
 
  --
  *Mike Tutkowski*
  *Senior CloudStack Developer, SolidFire Inc.*
  e: mike.tutkow...@solidfire.com
  o: 303.746.7302
  Advancing the way the world uses the
  cloudhttp://solidfire.com/solution/overview/?video=play
  *(tm)*




-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloudhttp://solidfire.com/solution/overview/?video=play
*(tm)*


RE: CLOUDSTACK-5406

2014-01-30 Thread Sanjay Tripathi
Hi Francois,

Can you please explain what is the impact you are seeing in primary storage 
count. Is it the negative value of primary storage count or something else.

--Sanjay

-Original Message-
From: Animesh Chaturvedi 
Sent: Friday, January 31, 2014 5:23 AM
To: dev@cloudstack.apache.org; fgaudrea...@cloudops.com
Cc: Sangeetha Hariharan; Sanjay Tripathi
Subject: RE: CLOUDSTACK-5406

Copying the folks that worked on it

-Original Message-
From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com] 
Sent: Thursday, January 30, 2014 3:49 PM
To: dev@cloudstack.apache.org
Subject: CLOUDSTACK-5406

Hi,

I saw this bug as been fixed for the secondary storage limits in 4.3:
CLOUDSTACK-5406

But it seems to impact the primary storage limit count as well, at least in 
4.2.1. Is the primary storage count fixed in 4.3 (I can't test right
now) as well?

Thanks!

--
Francois Gaudreault
Architecte de Solution Cloud | Cloud Solutions Architect 
fgaudrea...@cloudops.com
514-629-6775
- - -
CloudOps
420 rue Guy
Montréal QC  H3J 1S6
www.cloudops.com
@CloudOps_



RE: EC2 Rest Servlet domain cannot be null!! Error In Advanced Networking Mode

2014-01-30 Thread Likitha Shetty
Jeff, what you have reported is a bug. Created an issue to track it in in JIRA 
- https://issues.apache.org/jira/browse/CLOUDSTACK-5998 .

BTW you mentioned that you trying EC2 Query API against a setup with Advanced 
networking? I just want to point out that AFAIK, we have never tested EC2 
REST/SOAP API compatibility in a setup with Advanced networking.

Thanks,
Likitha

-Original Message-
From: Jeff Hair [mailto:j...@greenqloud.com]
Sent: Thursday, January 30, 2014 6:38 PM
To: dev@cloudstack.apache.org
Subject: EC2 Rest Servlet domain cannot be null!! Error In Advanced
Networking Mode

Hi,

We're trying to query the EC2 REST API. Most of it works, but we're running 
into
an issue when calling describe addresses. It returns a fault with domain 
cannot
be null!! as the error text. I've traced this down to CloudStack not setting 
the
EC2 domain (standard or vpc) in the
EC2SoapServiceImpl#toDescribeAddressesResponse static method. The domain
property is not being set, and then it explodes on serialization with the above
error.

So, my questions are these:

1. Is this a bug in CloudStack with the EC2 API and Advanced Networking? Or are
we missing some configuration/setting?
2. If this is actually a bug, what is the best way to work around it 
temporarily,
and also where to start fixing it for real in the CS codebase?

Thanks,

Jeff


RE: CLOUDSTACK-5406

2014-01-30 Thread Francois Gaudreault
No, its the same thing as the ticket describe. I tried to lunch a VM, and
the resource limit was reached for my account even if there was no volume
running at all.

FG
On Jan 31, 2014 12:15 AM, Sanjay Tripathi sanjay.tripa...@citrix.com
wrote:

 Hi Francois,

 Can you please explain what is the impact you are seeing in primary
 storage count. Is it the negative value of primary storage count or
 something else.

 --Sanjay

 -Original Message-
 From: Animesh Chaturvedi
 Sent: Friday, January 31, 2014 5:23 AM
 To: dev@cloudstack.apache.org; fgaudrea...@cloudops.com
 Cc: Sangeetha Hariharan; Sanjay Tripathi
 Subject: RE: CLOUDSTACK-5406

 Copying the folks that worked on it

 -Original Message-
 From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com]
 Sent: Thursday, January 30, 2014 3:49 PM
 To: dev@cloudstack.apache.org
 Subject: CLOUDSTACK-5406

 Hi,

 I saw this bug as been fixed for the secondary storage limits in 4.3:
 CLOUDSTACK-5406

 But it seems to impact the primary storage limit count as well, at least
 in 4.2.1. Is the primary storage count fixed in 4.3 (I can't test right
 now) as well?

 Thanks!

 --
 Francois Gaudreault
 Architecte de Solution Cloud | Cloud Solutions Architect
 fgaudrea...@cloudops.com
 514-629-6775
 - - -
 CloudOps
 420 rue Guy
 Montréal QC  H3J 1S6
 www.cloudops.com
 @CloudOps_




RE: Xen StorageMotion Fails

2014-01-30 Thread Devdeep Singh
Hi Francois,

This looks like an issue in cloudstack. Can you file a bug with the details?

Regards,
Devdeep

-Original Message-
From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com] 
Sent: Thursday, January 30, 2014 6:48 PM
To: Srikanteswararao Talluri
Cc: dev@cloudstack.apache.org
Subject: RE: Xen StorageMotion Fails

I used the UI for the vm migration. I can get you the exact api call but isnt 
the call in the pastebin output already..

Another thing, if there is a vm running on the same network on the destination 
host, I dont have the exception but the volume stay in migrate state forever.

Thanks

Francois
On Jan 30, 2014 2:19 AM, Srikanteswararao Talluri  
srikanteswararao.tall...@citrix.com wrote:

 Yes, I am able to migrate a VM on local storage(host) to another local 
 storage(host).

 Can you paste the API call you tried?

 Thanks,
 ~Talluri

 -Original Message-
 From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com]
 Sent: Wednesday, 29 January 2014 6:31 PM
 To: dev@cloudstack.apache.org
 Subject: Xen StorageMotion Fails

 Hi,

 I posted this on the users@ mailing-list yesterday, but giving it more 
 thoughts I think this should have been posted here. We are running 
 4.2.1 with XenServer 6.2SP1. I tried to run a storage motion from a 
 local storage to another, and I am getting this error. Xen returns a 
 VM_REQUIRES_NETWORK when attempting the migration. This error, according to 
 the javadoc means:

 You attempted to run a VM on a host which doesn't have a PIF on a 
 Network needed by the VM. The VM has at least one VIF attached to the Network.

 http://pastebin.com/cHcAyPtP

 I believe this is supposed to be supported since CloudStack let me do 
 it from the UI. Right?

 Thanks!

 --
 Francois Gaudreault
 Architecte de Solution Cloud | Cloud Solutions Architect 
 fgaudrea...@cloudops.com
 514-629-6775
 - - -
 CloudOps
 420 rue Guy
 Montréal QC  H3J 1S6
 www.cloudops.com
 @CloudOps_




Re: volume lifecycle issue expunging

2014-01-30 Thread Marcus
CLOUDSTACK-6000

Sounds like a vacuum cleaner.

On Thu, Jan 30, 2014 at 3:38 PM, Mike Tutkowski
mike.tutkow...@solidfire.com wrote:
 Yeah, unfortunately our code should be a bit more robust in this area. The
 second (or more) time around, the failed deletion of a file should probably
 not cause us trouble if, in fact, the file is gone (regardless of whether
 or not your delete command actually deleted the file or it was gone for
 some other reason).

 I think, as you say, it's probably good to expose those other issues,
 though.


 On Thu, Jan 30, 2014 at 3:29 PM, Marcus shadow...@gmail.com wrote:

 My one hesitation is that it may cause problems by surfacing other
 bugs. For example, say a delete operation succeeds in deleting the
 image file, but fails in some subsequent code. The next time it runs,
 it may fail in deleting the image file (file not found), and thus get
 stuck in an endless loop trying to clean up.  I think it's ultimately
 good to surface those (better than potentially orphaning disks on
 storage), but it may be something to watch for.

 On Thu, Jan 30, 2014 at 2:56 PM, Mike Tutkowski
 mike.tutkow...@solidfire.com wrote:
  I agree, Marcus.
 
 
  On Thu, Jan 30, 2014 at 2:42 PM, Marcus shadow...@gmail.com wrote:
 
  I think there's a hole in the volume lifecycle.  I've been noticing
  volumes lingering that should have been cleaned up, and it seems to be
  a bug in the state machine for the volumes:
 
  s_fsm.addTransition(Destroy, Event.ExpungingRequested,
  Expunging);
  s_fsm.addTransition(Expunging, Event.ExpungingRequested,
  Expunging);
  s_fsm.addTransition(Expunging, Event.OperationSucceeded,
  Expunged);
  s_fsm.addTransition(Expunging, Event.OperationFailed,
  Expunging);
 
  If a volume is in Destroy state, it goes to Expunging when the delete
  operation is requested. If the delete fails, it remains in expunging.
  The storage garbage collector will never try to clean up that volume
  again, since it only lists volumes in 'Destroy' and attempts those.
  You can only get to Expunging from Destroy, it makes sense to change
  that last line to revert the volume state back to Destroy if the
  expunge operation failed, so that it will try again next time.
 
 
 
 
  --
  *Mike Tutkowski*
  *Senior CloudStack Developer, SolidFire Inc.*
  e: mike.tutkow...@solidfire.com
  o: 303.746.7302
  Advancing the way the world uses the
  cloudhttp://solidfire.com/solution/overview/?video=play
  *(tm)*




 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the
 cloudhttp://solidfire.com/solution/overview/?video=play
 *(tm)*


Disks in ACS Vmware

2014-01-30 Thread Girish Shilamkar
Hello,

In ACS KVM the volumes attached are seen as normal disk /dev/vda /dev/vdb and 
so on. But with vmware it seems Volume Groups are being used.
The root device is /dev/mapper/VolGroup00-LogVol100 which previously it was 
/dev/hda.
So now when I attach a new datadisk to vm running on vmware what device/vg is 
created ?

Regards,
Girish



RE: CLOUDSTACK-5406

2014-01-30 Thread Sanjay Tripathi
Francois,

Bug CLOUDSTACK-5406 was related to snapshot size and secondary storage count.

For your issue, can you paste the response of listAccounts API for the account 
you are trying to deploy VM with; this will help us to check the resource_limit 
set for your account and what is the resource_count that account is having.

Also paste the error message you are getting during the VM deployment.

--Sanjay

From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com]
Sent: Friday, January 31, 2014 10:58 AM
To: Sanjay Tripathi
Cc: dev@cloudstack.apache.org; Sangeetha Hariharan; Animesh Chaturvedi
Subject: RE: CLOUDSTACK-5406


No, its the same thing as the ticket describe. I tried to lunch a VM, and the 
resource limit was reached for my account even if there was no volume running 
at all.

FG
On Jan 31, 2014 12:15 AM, Sanjay Tripathi 
sanjay.tripa...@citrix.commailto:sanjay.tripa...@citrix.com wrote:
Hi Francois,

Can you please explain what is the impact you are seeing in primary storage 
count. Is it the negative value of primary storage count or something else.

--Sanjay

-Original Message-
From: Animesh Chaturvedi
Sent: Friday, January 31, 2014 5:23 AM
To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org; 
fgaudrea...@cloudops.commailto:fgaudrea...@cloudops.com
Cc: Sangeetha Hariharan; Sanjay Tripathi
Subject: RE: CLOUDSTACK-5406

Copying the folks that worked on it

-Original Message-
From: Francois Gaudreault 
[mailto:fgaudrea...@cloudops.commailto:fgaudrea...@cloudops.com]
Sent: Thursday, January 30, 2014 3:49 PM
To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
Subject: CLOUDSTACK-5406

Hi,

I saw this bug as been fixed for the secondary storage limits in 4.3:
CLOUDSTACK-5406

But it seems to impact the primary storage limit count as well, at least in 
4.2.1. Is the primary storage count fixed in 4.3 (I can't test right
now) as well?

Thanks!

--
Francois Gaudreault
Architecte de Solution Cloud | Cloud Solutions Architect 
fgaudrea...@cloudops.commailto:fgaudrea...@cloudops.com
514-629-6775tel:514-629-6775
- - -
CloudOps
420 rue Guy
Montréal QC  H3J 1S6
www.cloudops.comhttp://www.cloudops.com
@CloudOps_


Review Request 17583: [Hyper-v] can't get console access after vm live migration for 3 minutes

2014-01-30 Thread Anshul Gangwar

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17583/
---

Review request for cloudstack, Devdeep Singh and Rajesh Battala.


Bugs: CLOUDSTACK-6001
https://issues.apache.org/jira/browse/CLOUDSTACK-6001


Repository: cloudstack-git


Description
---

[Hyper-v] can't get console access after vm live migration for 3 minutes.


Diffs
-

  
services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyRdpClient.java
 d5a3fcd 

Diff: https://reviews.apache.org/r/17583/diff/


Testing
---

verified by opening console and then doing migration and agin opening console. 
It is working fine now.


Thanks,

Anshul Gangwar