Re: Review Request: Storage motion changes for xenserver

2013-04-15 Thread Devdeep Singh

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

(Updated April 15, 2013, 7:24 a.m.)


Review request for cloudstack, Abhinandan Prateek, edison su, Alex Huang, and 
anthony xu.


Changes
---

Updated the patch after merging with the latest master.


Description
---

Storage motion for Xenserver. FS for the feature 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Enabling+Storage+XenMotion+for+XenServer
1. Implemented Api findStoragePoolsForMigration. Added a new response 
objects to list storage pools available for migration.
2. Updated migrateVolume api for allowing migrating volumes of running vms. 
These changes are integrated into the latest storage refactoring changes.
3. Added the implementation for findHostsForMigration api. It lists the 
hosts to which an instance can be migrated, including hosts from within and 
across clusters to which an instance may be migrated with storage motion. The 
work of migrating a volume of a running vm is also done in copyAsync.
4. Updated the listHosts api for backward compatibility.
5. Added the implementation for migrateVirtualMachineWithVolume api. It 
migrates an instance with its volumes within a cluster and also across 
clusters. Also introduced a new XenServerStorageMotionStrategy for migrating 
volumes of a vm. When a vm is being migrated with its volumes, the vm is put in 
migrating state and a request is send to the volume manager to migrate the vm 
and its volumes. Volume manager calls into the volume service which forwards 
the request to data motion service after moving all the volumes to migrating 
state. Data motion service enumerates the strategies and the request reaches 
the XenServerStorageMotionStrategy. It calls in to the resource to complete the 
operation.
6. Resolved an issue where storage xenmotion of 2nd VM created from the 
same template to a host was failing with duplicate_vm exception. Made changes 
to remove the mac_seed key value pair from other_config when vms are created. 
This is was storage motion to fail.
7. Updated the db upgrade schema script.
8. Added the right permissions in commands.properties
9. Marvin tests for testing storage motion. Following scenarios are tested.
9.1. A virtual machine is migrated to another host. Its volumes are also 
migrated to another storage pool.
9.2. Just the volumes of a vm are migrated to another storage pool while 
the vm continues to run on the same host.
10. Unit tests for testing migration of a vm with its volumes.


This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-659.


Diffs (updated)
-

  api/src/com/cloud/agent/api/MigrateWithStorageAnswer.java PRE-CREATION 
  api/src/com/cloud/agent/api/MigrateWithStorageCommand.java PRE-CREATION 
  api/src/com/cloud/agent/api/MigrateWithStorageCompleteAnswer.java 
PRE-CREATION 
  api/src/com/cloud/agent/api/MigrateWithStorageCompleteCommand.java 
PRE-CREATION 
  api/src/com/cloud/agent/api/MigrateWithStorageReceiveAnswer.java PRE-CREATION 
  api/src/com/cloud/agent/api/MigrateWithStorageReceiveCommand.java 
PRE-CREATION 
  api/src/com/cloud/agent/api/MigrateWithStorageSendAnswer.java PRE-CREATION 
  api/src/com/cloud/agent/api/MigrateWithStorageSendCommand.java PRE-CREATION 
  api/src/com/cloud/agent/api/storage/MigrateVolumeAnswer.java PRE-CREATION 
  api/src/com/cloud/agent/api/storage/MigrateVolumeCommand.java PRE-CREATION 
  api/src/com/cloud/hypervisor/HypervisorCapabilities.java aff81b0 
  api/src/com/cloud/server/ManagementService.java 6e6dbc3 
  api/src/com/cloud/vm/UserVmService.java d963b74 
  api/src/org/apache/cloudstack/api/ApiConstants.java b08e992 
  api/src/org/apache/cloudstack/api/ResponseGenerator.java c0dd57e 
  
api/src/org/apache/cloudstack/api/command/admin/host/FindHostsForMigrationCmd.java
 PRE-CREATION 
  api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java 
29844c3 
  
api/src/org/apache/cloudstack/api/command/admin/storage/FindStoragePoolsForMigrationCmd.java
 PRE-CREATION 
  
api/src/org/apache/cloudstack/api/command/admin/vm/MigrateVirtualMachineWithVolumeCmd.java
 PRE-CREATION 
  api/src/org/apache/cloudstack/api/command/user/volume/MigrateVolumeCmd.java 
287241a 
  api/src/org/apache/cloudstack/api/response/HostForMigrationResponse.java 
PRE-CREATION 
  api/src/org/apache/cloudstack/api/response/HostResponse.java f5aa8f9 
  
api/src/org/apache/cloudstack/api/response/StoragePoolForMigrationResponse.java 
PRE-CREATION 
  api/src/org/apache/cloudstack/api/response/StoragePoolResponse.java 0b16226 
  client/tomcatconf/applicationContext.xml.in 15cd6fe 
  client/tomcatconf/commands.properties.in 798d226 
  core/src/com/cloud/hypervisor/HypervisorCapabilitiesVO.java fafc0a3 
  
engine/api/src/org/apache/cloudstack/engine/subsystem/api/storage/ObjectInDataSt

Re: Review Request: (CLOUDSTACK-2017) implement network bandwidth rate limits for VMs for KVM

2013-04-15 Thread Wei Zhou


> On April 12, 2013, 1:27 p.m., Chip Childers wrote:
> > Wei,
> > 
> > Any chance you can provide unit tests and / or a marvin test that covers 
> > this functionality? I know it's a bit of an odd part of the code to have an 
> > automated test for, but we'd love to see something added with each 
> > incremental functional change we make.
> > 
> > -chip
> 
> Prasanna Santhanam wrote:
> Wido has setup some basic tests over which one can build further 
> unit-tests. Checkout LibvirtVMDefTest.java in 
> plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource

Prasanna, Thanks.

Chip, it is just an implementation, not a new feature. For new features in 
future, I will provide unit tests for automated testing.


- Wei


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


On April 12, 2013, 6:33 a.m., Wei Zhou wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10428/
> ---
> 
> (Updated April 12, 2013, 6:33 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Description
> ---
> 
> Adding bandwidth inbound and outbound limitations in VM XML file to implement 
> network bandwidth rate limits for VMs for KVM.
> It is supported since libvirt 0.9.4.
> 
> The patch only applies on CloudStack 4.0.1 branch.
> If commited, I will change it to support 4.1 and master branch.
> 
> 
> This addresses bug CLOUDSTACK-2017.
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java
>  f406c86 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java
>  b622b6d 
>   
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java
>  ba6c715 
> 
> Diff: https://reviews.apache.org/r/10428/diff/
> 
> 
> Testing
> ---
> 
> Testing manually ok.
> The VMs need to stop and start to make network bandwidth limits effective.
> 
> 
> Thanks,
> 
> Wei Zhou
> 
>



RE: [QA]question on cloudstack-usage service

2013-04-15 Thread Hugo Trippaers
Heya,

I just checked my qa system. Before starting the usage server I copy the 
db.properties and log4j.xml from /etc/cloudstack/management to 
/etc/cloudstack/usage.  Without these two files the usage server will have 
trouble starting. The db.properties not being present could be the cause for 
the DAO failure.

Can you test with this "fix"? If that works I'll fix the usage server to also 
look in /etc/cloudstack/managent for those files and package example 
configuration in the rpm. (In case people want to install the usage server on 
another server than the management server)

Cheers,

Hugo

> -Original Message-
> From: Abhinav Roy [mailto:abhinav@citrix.com]
> Sent: Monday, April 15, 2013 7:55 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Same issue is seen in my setup also.
> 
> Thanks and regards,
> Abhinav
> 
> -Original Message-
> From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> Sent: Monday, April 15, 2013 11:15 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Wido/Kelven,
> 
> This issue  noticed on master build please find few more details below:.
>  http://jenkins-ccp.citrix.com/job/ASF-build-master-nonoss-rhel63/211/
> 
> 
> root@centos63 usage]# which jsvc
> /usr/bin/jsvc
> [root@centos63 usage]# /usr/bin/jsvc status
> 15/04/2013 06:53:11 14280 jsvc error: Cannot locate Java Home Manually set
> the JAVA_HOME classpath but still see the problem
> 
> [root@centos63 usage]# export JAVA_HOME=/usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# echo $JAVA_HOME
> /usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# /usr/bin/jsvc status
> [root@centos63 usage]# /usr/bin/jsvc start
> 
> Content of  Cloudstack-usage file (FYI)
> 
> SCP=""
> DCP=""
> UCP=`ls /usr/share/cloudstack-usage/cloud-usage-*.jar`":"`ls
> /usr/share/cloudstack-usage/lib/* | tr '\n' ':'`
> JCP="/usr/share/java/commons-daemon.jar"
> 
> # We need to append the JSVC daemon JAR to the classpath # AgentShell
> implements the JSVC daemon methods export
> CLASSPATH="$SCP:$DCP:$UCP:$JCP:/etc/cloudstack/usage"
> 
> 
> Its looks like it's similar to the following issue:
> 
>  https://issues.apache.org/jira/browse/CLOUDSTACK-1746
> 
> 
> regards
> sadhu
> 
> 
> 
> 
> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 13 April 2013 00:11
> To: dev@cloudstack.apache.org
> Subject: Re: [QA]question on cloudstack-usage service
> 
> It looks like to UsageDao has some trouble to initialize itself. On which 
> branch
> does this problem happen?
> 
> Kelven
> 
> On 4/12/13 11:05 AM, "Wido den Hollander"  wrote:
> 
> >
> >
> >On 04/12/2013 01:33 PM, Suresh Sadhu wrote:
> >> HI,
> >>
> >>
> >> Any idea why usage server is stopped with unexpected exception(is
> >>there any issue exits/logged already),do we need to add any additional
> >>packages/rpm/configurations other than installing the usage server
> >>monitor"
> >> Seen this issue on master build.
> >>
> >
> >Hmm, that is weird. Could you print the classpath what is submitted to
> >the JSVC process?
> >
> >Wondering what it is generating.
> >
> >Wido
> >
> >>
> >> [root@rhel631 ~]# service cloudstack-usage start
> >> Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
> >> [root@rhel631 ~]# service cloudstack-usage status cloudstack-usage
> >> (pid  20999) is running...
> >> [root@rhel631 ~]# service cloudstack-usage status cloudstack-usage is
> >> stopped
> >>
> >>
> >> Content of cloudstack-usage.err:
> >> *
> >>
> >> log4j:WARN No appenders could be found for logger
> >>(org.springframework.core.env.StandardEnvironment).
> >> log4j:WARN Please initialize the log4j system properly.
> >> log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig
> >>for more info.
> >> java.lang.reflect.InvocationTargetException
> >>  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> >>  at
> >>sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorIm
> pl.j
> >>ava
> >>:57)
> >>  at
> >>sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAcc
> ess
> >>orI
> >>mpl.java:43)
> >>  at java.lang.reflect.Method.invoke(Method.java:616)
> >>  at
> >>org.apache.commons.daemon.support.DaemonLoader.start(DaemonLoa
> der.java
> >>:17
> >>7)
> >> Caused by: org.springframework.beans.factory.BeanCreationException:
> >>Error creating bean with name 'portForwardingUsageParser': Injection
> >>of autowired dependencies failed; nested exception is
> >>org.springframework.beans.factory.BeanCreationException: Could not
> >>autowire field: private com.cloud.usage.dao.UsageDao
> >>com.cloud.usage.parser.PortForwardingUsageParser._usageDao; nested
> >>exception is org.springframework.beans.factory.BeanCreationException:
> >>Error creating bean with name 'usageDaoImpl' defined in URL
> >>[jar:file:/usr/share/c

RE: [ACS41] Final push to the first RC!

2013-04-15 Thread Hugo Trippaers
Hey,

If there are no objections I would like to have commit 
accfccd83e11d0b12eccebde940c705e49eec913 included. It's a small thing, but it 
would certainly help a lot with setting up automated testing environments and 
it would be nice to have it available as a standard feature in 4.1 already.

Cheers,

Hugo

> -Original Message-
> From: Chip Childers [mailto:chip.child...@sungard.com]
> Sent: Friday, April 12, 2013 10:04 PM
> To: dev@cloudstack.apache.org
> Subject: [ACS41] Final push to the first RC!
> 
> Hi!
> 
> Fantastic job over the last 24 hours.  We're now done with the known Blocker
> and Critical *functional* bugs.
> 
> Outstanding work:
> 
> Complete the release notes - How's that going Joe?  Need / want help?
> 
> Upload the IPv6 enabled system VM template to Wido's server (Chiradeep is
> putting it somewhere for me to push to Wido's server), and update the
> documentation to reference the download location.  I'll handle these items.
> 
> Last, but not least, Ilya asked if CLOUDSTACK-524 could be put into 4.1.
> The fix looks reasonable to me, and I pinged Chiradeep to see if he wouldn't
> mind providing a 4.1 specific patch or cherry-pick request.
> 
> Are there any other outstanding items that others want to see complete
> before I cut the RC?
> 
> The only other outstanding question is if we want to get 4.0.2 out of the way
> *first*, or if we do them concurrently.  IMO we can do them concurrently.
> 
> Unless anyone objects, or raises other items for inclusion, I'll start the
> process as soon as we complete the items noted above.
> 
> -chip


RE: New Committer: Bruno Demion (a.k.a milamber)

2013-04-15 Thread Hugo Trippaers
Welcome and congrats :-)

> -Original Message-
> From: Milamber [mailto:milam...@apache.org]
> Sent: Friday, April 12, 2013 9:49 PM
> To: dev@cloudstack.apache.org
> Subject: Re: New Committer: Bruno Demion (a.k.a milamber)
> 
> Thanks Community CS !
> 
> Hopefully I bring some good things on CloudStack!
> 
> Milamber
> 
> Le 12/04/2013 16:02, Sebastien Goasguen a ecrit :
> > The Project Management Committee (PMC) for Apache CloudStack has
> asked Bruno Demion (a.k.a milamber) to become a committer and we are
> pleased to announced that they have accepted.
> >
> > Bruno has contributed to the french translation of the UI and the Docs, as
> well as fixed several issues with encoding of the localization support of the
> UI.  For information, Bruno is also a committer and PMC member of the
> Apache Jmeter project.
> > Being a committer allows many contributors to contribute more
> > autonomously. For developers, it makes it easier to submit changes and
> > eliminates the need to have contributions reviewed via the patch
> > submission process. Whether contributions are development-related or
> > otherwise, it is a recognition of a contributor's participation in the
> > project and commitment to the project and the Apache Way.
> > Please join me in congratulating Bruno,
> >
> > -Sebastien, on behalf of  the Apache CloudStack PMC



RE: [QA]question on cloudstack-usage service

2013-04-15 Thread Suresh Sadhu
Still see the problem even after copying the files


[root@rhel631 management]# cp db.properties /etc/cloudstack/usage/
[root@rhel631 management]# cp log4j-cloud.xml /etc/cloudstack/usage/
[root@rhel631 management]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 management]# service cloudstack-usage status
cloudstack-usage (pid  13320) is running...
[root@rhel631 management]# service cloudstack-usage status
cloudstack-usage is stopped

total 68

[root@rhel631 usage]# chmod 755 *
[root@rhel631 usage]# ll
-rwxr-xr-x. 1 root root 64626 Apr 15 09:31 cloudstack-usage.err
-rwxr-xr-x. 1 root root 0 Mar 29 09:16 cloudstack-usage.out
[root@rhel631 usage]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 usage]# service cloudstack-usage status
cloudstack-usage is stopped



regards
sadhu

-Original Message-
From: Hugo Trippaers [mailto:htrippa...@schubergphilis.com] 
Sent: 15 April 2013 13:15
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Heya,

I just checked my qa system. Before starting the usage server I copy the 
db.properties and log4j.xml from /etc/cloudstack/management to 
/etc/cloudstack/usage.  Without these two files the usage server will have 
trouble starting. The db.properties not being present could be the cause for 
the DAO failure.

Can you test with this "fix"? If that works I'll fix the usage server to also 
look in /etc/cloudstack/managent for those files and package example 
configuration in the rpm. (In case people want to install the usage server on 
another server than the management server)

Cheers,

Hugo

> -Original Message-
> From: Abhinav Roy [mailto:abhinav@citrix.com]
> Sent: Monday, April 15, 2013 7:55 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Same issue is seen in my setup also.
> 
> Thanks and regards,
> Abhinav
> 
> -Original Message-
> From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> Sent: Monday, April 15, 2013 11:15 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Wido/Kelven,
> 
> This issue  noticed on master build please find few more details below:.
>  http://jenkins-ccp.citrix.com/job/ASF-build-master-nonoss-rhel63/211/
> 
> 
> root@centos63 usage]# which jsvc
> /usr/bin/jsvc
> [root@centos63 usage]# /usr/bin/jsvc status
> 15/04/2013 06:53:11 14280 jsvc error: Cannot locate Java Home Manually 
> set the JAVA_HOME classpath but still see the problem
> 
> [root@centos63 usage]# export JAVA_HOME=/usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# echo $JAVA_HOME
> /usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# /usr/bin/jsvc status
> [root@centos63 usage]# /usr/bin/jsvc start
> 
> Content of  Cloudstack-usage file (FYI)
> 
> SCP=""
> DCP=""
> UCP=`ls /usr/share/cloudstack-usage/cloud-usage-*.jar`":"`ls
> /usr/share/cloudstack-usage/lib/* | tr '\n' ':'` 
> JCP="/usr/share/java/commons-daemon.jar"
> 
> # We need to append the JSVC daemon JAR to the classpath # AgentShell 
> implements the JSVC daemon methods export 
> CLASSPATH="$SCP:$DCP:$UCP:$JCP:/etc/cloudstack/usage"
> 
> 
> Its looks like it's similar to the following issue:
> 
>  https://issues.apache.org/jira/browse/CLOUDSTACK-1746
> 
> 
> regards
> sadhu
> 
> 
> 
> 
> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 13 April 2013 00:11
> To: dev@cloudstack.apache.org
> Subject: Re: [QA]question on cloudstack-usage service
> 
> It looks like to UsageDao has some trouble to initialize itself. On 
> which branch does this problem happen?
> 
> Kelven
> 
> On 4/12/13 11:05 AM, "Wido den Hollander"  wrote:
> 
> >
> >
> >On 04/12/2013 01:33 PM, Suresh Sadhu wrote:
> >> HI,
> >>
> >>
> >> Any idea why usage server is stopped with unexpected exception(is 
> >>there any issue exits/logged already),do we need to add any 
> >>additional packages/rpm/configurations other than installing the 
> >>usage server monitor"
> >> Seen this issue on master build.
> >>
> >
> >Hmm, that is weird. Could you print the classpath what is submitted 
> >to the JSVC process?
> >
> >Wondering what it is generating.
> >
> >Wido
> >
> >>
> >> [root@rhel631 ~]# service cloudstack-usage start
> >> Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
> >> [root@rhel631 ~]# service cloudstack-usage status cloudstack-usage 
> >> (pid  20999) is running...
> >> [root@rhel631 ~]# service cloudstack-usage status cloudstack-usage 
> >> is stopped
> >>
> >>
> >> Content of cloudstack-usage.err:
> >> *
> >>
> >> log4j:WARN No appenders could be found for logger 
> >>(org.springframework.core.env.StandardEnvironment).
> >> log4j:WARN Please initialize the log4j system properly.
> >> log4j:WARN See 
> >>http://logging.apache.org/log4j/1.2/f

RE: [QA]question on cloudstack-usage service

2013-04-15 Thread Abhinav Roy
Hi Sadhu,

I just tried it, we need to give permissions to the files db.properties and 
log4j-cloud.xml after copying them to the /etc/cloudstack/usage directory. 
After that just restart the cloudstack-usage service and the issue won't be 
seen.

Thanks and regards,
Abhinav

-Original Message-
From: Suresh Sadhu [mailto:suresh.sa...@citrix.com] 
Sent: Monday, April 15, 2013 1:36 PM
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Still see the problem even after copying the files


[root@rhel631 management]# cp db.properties /etc/cloudstack/usage/
[root@rhel631 management]# cp log4j-cloud.xml /etc/cloudstack/usage/
[root@rhel631 management]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 management]# service cloudstack-usage status cloudstack-usage 
(pid  13320) is running...
[root@rhel631 management]# service cloudstack-usage status cloudstack-usage is 
stopped

total 68

[root@rhel631 usage]# chmod 755 *
[root@rhel631 usage]# ll
-rwxr-xr-x. 1 root root 64626 Apr 15 09:31 cloudstack-usage.err
-rwxr-xr-x. 1 root root 0 Mar 29 09:16 cloudstack-usage.out
[root@rhel631 usage]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 usage]# service cloudstack-usage status cloudstack-usage is 
stopped



regards
sadhu

-Original Message-
From: Hugo Trippaers [mailto:htrippa...@schubergphilis.com]
Sent: 15 April 2013 13:15
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Heya,

I just checked my qa system. Before starting the usage server I copy the 
db.properties and log4j.xml from /etc/cloudstack/management to 
/etc/cloudstack/usage.  Without these two files the usage server will have 
trouble starting. The db.properties not being present could be the cause for 
the DAO failure.

Can you test with this "fix"? If that works I'll fix the usage server to also 
look in /etc/cloudstack/managent for those files and package example 
configuration in the rpm. (In case people want to install the usage server on 
another server than the management server)

Cheers,

Hugo

> -Original Message-
> From: Abhinav Roy [mailto:abhinav@citrix.com]
> Sent: Monday, April 15, 2013 7:55 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Same issue is seen in my setup also.
> 
> Thanks and regards,
> Abhinav
> 
> -Original Message-
> From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> Sent: Monday, April 15, 2013 11:15 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Wido/Kelven,
> 
> This issue  noticed on master build please find few more details below:.
>  http://jenkins-ccp.citrix.com/job/ASF-build-master-nonoss-rhel63/211/
> 
> 
> root@centos63 usage]# which jsvc
> /usr/bin/jsvc
> [root@centos63 usage]# /usr/bin/jsvc status
> 15/04/2013 06:53:11 14280 jsvc error: Cannot locate Java Home Manually 
> set the JAVA_HOME classpath but still see the problem
> 
> [root@centos63 usage]# export JAVA_HOME=/usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# echo $JAVA_HOME
> /usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# /usr/bin/jsvc status
> [root@centos63 usage]# /usr/bin/jsvc start
> 
> Content of  Cloudstack-usage file (FYI)
> 
> SCP=""
> DCP=""
> UCP=`ls /usr/share/cloudstack-usage/cloud-usage-*.jar`":"`ls
> /usr/share/cloudstack-usage/lib/* | tr '\n' ':'` 
> JCP="/usr/share/java/commons-daemon.jar"
> 
> # We need to append the JSVC daemon JAR to the classpath # AgentShell 
> implements the JSVC daemon methods export 
> CLASSPATH="$SCP:$DCP:$UCP:$JCP:/etc/cloudstack/usage"
> 
> 
> Its looks like it's similar to the following issue:
> 
>  https://issues.apache.org/jira/browse/CLOUDSTACK-1746
> 
> 
> regards
> sadhu
> 
> 
> 
> 
> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 13 April 2013 00:11
> To: dev@cloudstack.apache.org
> Subject: Re: [QA]question on cloudstack-usage service
> 
> It looks like to UsageDao has some trouble to initialize itself. On 
> which branch does this problem happen?
> 
> Kelven
> 
> On 4/12/13 11:05 AM, "Wido den Hollander"  wrote:
> 
> >
> >
> >On 04/12/2013 01:33 PM, Suresh Sadhu wrote:
> >> HI,
> >>
> >>
> >> Any idea why usage server is stopped with unexpected exception(is 
> >>there any issue exits/logged already),do we need to add any 
> >>additional packages/rpm/configurations other than installing the 
> >>usage server monitor"
> >> Seen this issue on master build.
> >>
> >
> >Hmm, that is weird. Could you print the classpath what is submitted 
> >to the JSVC process?
> >
> >Wondering what it is generating.
> >
> >Wido
> >
> >>
> >> [root@rhel631 ~]# service cloudstack-usage start
> >> Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
> >> [root@rhel631 ~]# service c

RE: [QA]question on cloudstack-usage service

2013-04-15 Thread Suresh Sadhu
Thanks Abhinav. Its looks like  mistakenly I have applied permissions on source 
folder.

Hugo: your work around is working fine. After copying  the 2 files in 
/etc/cloudstack/usage directory  ,gave access permission (755) and restart the 
cloud-usage service. Then issue won't see again.




Regards
Sadhu


-Original Message-
From: Abhinav Roy [mailto:abhinav@citrix.com] 
Sent: 15 April 2013 13:39
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Hi Sadhu,

I just tried it, we need to give permissions to the files db.properties and 
log4j-cloud.xml after copying them to the /etc/cloudstack/usage directory. 
After that just restart the cloudstack-usage service and the issue won't be 
seen.

Thanks and regards,
Abhinav

-Original Message-
From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
Sent: Monday, April 15, 2013 1:36 PM
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Still see the problem even after copying the files


[root@rhel631 management]# cp db.properties /etc/cloudstack/usage/
[root@rhel631 management]# cp log4j-cloud.xml /etc/cloudstack/usage/
[root@rhel631 management]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 management]# service cloudstack-usage status cloudstack-usage 
(pid  13320) is running...
[root@rhel631 management]# service cloudstack-usage status cloudstack-usage is 
stopped

total 68

[root@rhel631 usage]# chmod 755 *
[root@rhel631 usage]# ll
-rwxr-xr-x. 1 root root 64626 Apr 15 09:31 cloudstack-usage.err
-rwxr-xr-x. 1 root root 0 Mar 29 09:16 cloudstack-usage.out
[root@rhel631 usage]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 usage]# service cloudstack-usage status cloudstack-usage is 
stopped



regards
sadhu

-Original Message-
From: Hugo Trippaers [mailto:htrippa...@schubergphilis.com]
Sent: 15 April 2013 13:15
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Heya,

I just checked my qa system. Before starting the usage server I copy the 
db.properties and log4j.xml from /etc/cloudstack/management to 
/etc/cloudstack/usage.  Without these two files the usage server will have 
trouble starting. The db.properties not being present could be the cause for 
the DAO failure.

Can you test with this "fix"? If that works I'll fix the usage server to also 
look in /etc/cloudstack/managent for those files and package example 
configuration in the rpm. (In case people want to install the usage server on 
another server than the management server)

Cheers,

Hugo

> -Original Message-
> From: Abhinav Roy [mailto:abhinav@citrix.com]
> Sent: Monday, April 15, 2013 7:55 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Same issue is seen in my setup also.
> 
> Thanks and regards,
> Abhinav
> 
> -Original Message-
> From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> Sent: Monday, April 15, 2013 11:15 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Wido/Kelven,
> 
> This issue  noticed on master build please find few more details below:.
>  http://jenkins-ccp.citrix.com/job/ASF-build-master-nonoss-rhel63/211/
> 
> 
> root@centos63 usage]# which jsvc
> /usr/bin/jsvc
> [root@centos63 usage]# /usr/bin/jsvc status
> 15/04/2013 06:53:11 14280 jsvc error: Cannot locate Java Home Manually 
> set the JAVA_HOME classpath but still see the problem
> 
> [root@centos63 usage]# export JAVA_HOME=/usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# echo $JAVA_HOME
> /usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# /usr/bin/jsvc status
> [root@centos63 usage]# /usr/bin/jsvc start
> 
> Content of  Cloudstack-usage file (FYI)
> 
> SCP=""
> DCP=""
> UCP=`ls /usr/share/cloudstack-usage/cloud-usage-*.jar`":"`ls
> /usr/share/cloudstack-usage/lib/* | tr '\n' ':'` 
> JCP="/usr/share/java/commons-daemon.jar"
> 
> # We need to append the JSVC daemon JAR to the classpath # AgentShell 
> implements the JSVC daemon methods export 
> CLASSPATH="$SCP:$DCP:$UCP:$JCP:/etc/cloudstack/usage"
> 
> 
> Its looks like it's similar to the following issue:
> 
>  https://issues.apache.org/jira/browse/CLOUDSTACK-1746
> 
> 
> regards
> sadhu
> 
> 
> 
> 
> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 13 April 2013 00:11
> To: dev@cloudstack.apache.org
> Subject: Re: [QA]question on cloudstack-usage service
> 
> It looks like to UsageDao has some trouble to initialize itself. On 
> which branch does this problem happen?
> 
> Kelven
> 
> On 4/12/13 11:05 AM, "Wido den Hollander"  wrote:
> 
> >
> >
> >On 04/12/2013 01:33 PM, Suresh Sadhu wrote:
> >> HI,
> >>
> >>
> >> Any idea why usage server is stopped with unexpected exception(is 
> >>there any issue exits/

{QA}Not able to login with default admin credentials in latest master build

2013-04-15 Thread Suresh Sadhu
HI All,

Seen a problem with latest maser build ,i.e not able to login with default 
admin credentials.it looks like by default we are putting admin user state in 
disable mode. Even after updating the state to enable mode in db ,still see the 
problem. We even updated password with MD5 hash  but still see the problem i.e  
" Not abkle to login with admin credentials"

* 
CLOUDSTACK-2028 not able 
to login cloudstack using default admin credentials


Regards
Sadhu



Re: GSoC 2013 Task Mentor

2013-04-15 Thread Sebastien Goasguen

On Apr 14, 2013, at 4:53 PM, Radu Calin  wrote:

> Hello all,
> 
> 
> 
> I'm Radu , a 1st year MS student willing to work for the CloudStack project
> through GSoC this summer.
> 
> 
> 
> I would like to know whether there's any mentor assigned for the DevCloud
> Cloud in a Box project
> (https://issues.apache.org/jira/browse/CLOUDSTACK-1781) yet? 
> 
> 

Hi Radu, yes for now it's me.

> 
> I'm asking this because I want to find out more details about the project,
> like the mentor's vision about the end product functionality/UX and how the
> workload is going to be structured throughout the given timeframe.  
> 

You should start looking at https://cwiki.apache.org/CLOUDSTACK/devcloud.html
and:
https://cwiki.apache.org/CLOUDSTACK/quickcloud.html

I am also on IRC : cloudstack and cloudstack-dev on freenode.

> 
> 
> Thanks for reading this,
> 
> Radu Calin
> 



RE: Fixed function list for Apache CloudStack 4.1

2013-04-15 Thread Sudha Ponnaganti
Hi Kimi,

>From the JIRA link, you  can review features which are closed/resolved. That 
>list gives you complete scope that made it into 4.1.  There are minor number 
>of  open and in progress items which did not make it to 4.1 release. Release 
>is getting closed shortly and possibly the open / in progress items would move 
>to next release as the window to check-in has been closed already. 

Below are the features / improvements that made it into 4.1 as per JIRA which 
is accurate:  Each one of the JIRA items would list all the artifacts related 
to the user story. 
https://issues.apache.org/jira/issues/#?jql=project%20%3D%20CLOUDSTACK%20AND%20issuetype%20in%20(Improvement%2C%20%22New%20Feature%22)%20AND%20fixVersion%20%3D%20%224.1.0%22%20AND%20status%20in%20(Reopened%2C%20Resolved%2C%20Closed%2C%20%22Ready%20To%20Review%22)%20ORDER%20BY%20priority%20DESC%2C%20reporter%20ASC%2C%20status%20DESC%2C%20assignee%20ASC


Key Summary Status
CLOUDSTACK-729  Provide RPM spec file without depending on waf and system 
packages  Closed
CLOUDSTACK-1184 Localization - Add Korean label to all locales  Closed
CLOUDSTACK-177  Quickview and table widget redesign Closed
CLOUDSTACK-176  Tooltip widget  Closed
CLOUDSTACK-436  User and Domain admin can change his password   Resolved
CLOUDSTACK-437  User and Domain admin can create his API key and secret Resolved
CLOUDSTACK-662  Advanced Search UI  Closed
CLOUDSTACK-1076 Legal documentation improvements to separate source 
distributions from packaged distributions.  Closed
CLOUDSTACK-740  Refactor networking core for clarityClosed
CLOUDSTACK-594  Update the java binding used to the latest one that came with 
XenServer 6.1 Closed
CLOUDSTACK-297  Reset SSH Key to access VM (similar to reset password)  Closed
CLOUDSTACK-241  AWS Style Regions   Closed
CLOUDSTACK-780  Additional VMX Settings Closed
CLOUDSTACK-726  Implement L3 Router functionality in Nicira Nvp Plugin  Resolved
CLOUDSTACK-1708 CloudMonkey Profiles for Multiple CS Environments   Resolved
CLOUDSTACK-574  Cloudstack Ceph RBD Setup   Closed
CLOUDSTACK-299  Egress firewall rules for guest network Closed
CLOUDSTACK-537  UI of "Security Group support in Advance zone" feature  Closed
CLOUDSTACK-509  S3-backed Secondary Storage Reopened
CLOUDSTACK-197  Support for EC2 Query API (rest interface)  Closed
CLOUDSTACK-706  Persistent Networks without running a VMClosed
CLOUDSTACK-686  Allow for same vlan on different physical nics  Closed
CLOUDSTACK-644  Resize volumes feature  Resolved
CLOUDSTACK-618  API request throttling to avoid malicious attacks on MS per 
account through frequent API request.   Closed
CLOUDSTACK-820  Events framework to publish/subscribe to CloudStack events  
Closed
CLOUDSTACK-421  Open up the integration api port on the developer profile   
Closed
CLOUDSTACK-926  ApiDiscoverService: Implement a plugin mechanism that exposes 
the list of APIs through a discovery service on the management server Closed
CLOUDSTACK-132  Mash up marvin into an interactive auto-completing API shell 
for CloudStack Closed
CLOUDSTACK-101  OVS support in KVM  Reopened
CLOUDSTACK-151  Configure Maven goals to replace ant deploy/debug on tomcat, 
deploydb commands  Closed
CLOUDSTACK-192  XenServer updates notifications Closed
CLOUDSTACK-306  Support SRX & F5 inline modeClosed
CLOUDSTACK-727  Add support for Nicira NVP to KVM hypervisor orchestration  
Resolved
CLOUDSTACK-965  When a detailview action is prohibited, the operation dialog 
box should not show up in the mean timeResolved
CLOUDSTACK-313  Fix cross references throughout documentation   Reopened
CLOUDSTACK-399  Document vSphere / ESXi patch installation procedureResolved
CLOUDSTACK-933  CglibThrowableRendererTest writing stack traces to terminal 
Closed
CLOUDSTACK-744  maven optimization for awsapi - prevent transitive dependency 
and injection of broken repo info Closed
CLOUDSTACK-997  Improve documentation of assignVirtualMachine API   Resolved
CLOUDSTACK-637  AutoScale   Closed

Thanks
/sudha

-Original Message-
From: Kimihiko Kitase [mailto:kimihiko.kit...@citrix.co.jp] 
Sent: Sunday, April 14, 2013 10:35 PM
To: dev@cloudstack.apache.org
Subject: RE: Fixed function list for Apache CloudStack 4.1

Nitin, Sudha,

Thank you very much.

My understanding is the design documents is exactly not same as feature lits of 
ACS. 
I saw the following link.
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Design+Documents+Not+Committed+to+a+Release

According to this link, when function is committed, its moved to release page 
like the following page.
https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+4.1+Release
https://issues.apache.org/jira/issues/?filter=12323168

But I thought the number of feature in this list is few.. That's why I asked.

According to Sudha Ponnaganti, now I understood I can get from JIRA with 
filtering "improvement

Re: [PROPOSAL][CLOUDSTACK-1828] NAT on private gateway

2013-04-15 Thread Jayapal Reddy Uradi
Hi Chiradeep, 

Can we  enable/disable source nat on private gateway while creating it ?
We can add a new optional parameter to the 'createPrivateGateway' API to set 
source nat.

Enable/Disable source NAT on private gateway while creating is enough  or it 
needs to be  configurable after creating private gateway also.

Please comment on the approach.

According to me, In future we can set static NAT/LB support for the private 
gateway  while  creating private gateway using the createPrivateGateway API. 

Thanks,
Jayapal

On 30-Mar-2013, at 4:05 AM, Chiradeep Vittal  
wrote:

> Thanks for this. 
> It seems a reasonable approach, except that it reflects a failure of
> previous modeling of the private gateway feature.
> So, today we want to provide source nat,
> Tomorrow perhaps static NAT?
> The day after PF and LB ?
> 
> The difference between the public network and the private gateway (or the
> network behind the private gateway) is not much (if any).
> 
> On 3/27/13 10:42 PM, "Jayapal Reddy Uradi" 
> wrote:
> 
>> I would like to propose feature NAT on private gateway.
>> This feature is sub feature of nTier2.0 apps.
>> 
>> Using this feature we can enable/disable the source NAT on the vpc
>> private gateway .
>> 
>> 
>> Jira ticket:
>> 
>> https://issues.apache.org/jira/browse/CLOUDSTACK-1828
>> 
>> 
>> FS :
>> 
>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/NAT+on+private+gate
>> way
>> 
>> 
>> 
>> Please provide your comments on above FS.
>> 
>> 
>> 
>> Thanks,
>> 
>> Jayapal
>> 
> 



RE: [QA]question on cloudstack-usage service

2013-04-15 Thread Suresh Sadhu
Its look like we require log4j-cloud_ussge.xml file for successful usage stats 
collection. Right now usage server is running but not able to collect the 
stats(usage job not triggered and no log generated for usage). We will raise 
the bug.

Regards
Sadhu




-Original Message-
From: Suresh Sadhu 
Sent: 15 April 2013 13:51
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Thanks Abhinav. Its looks like  mistakenly I have applied permissions on source 
folder.

Hugo: your work around is working fine. After copying  the 2 files in 
/etc/cloudstack/usage directory  ,gave access permission (755) and restart the 
cloud-usage service. Then issue won't see again.




Regards
Sadhu


-Original Message-
From: Abhinav Roy [mailto:abhinav@citrix.com]
Sent: 15 April 2013 13:39
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Hi Sadhu,

I just tried it, we need to give permissions to the files db.properties and 
log4j-cloud.xml after copying them to the /etc/cloudstack/usage directory. 
After that just restart the cloudstack-usage service and the issue won't be 
seen.

Thanks and regards,
Abhinav

-Original Message-
From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
Sent: Monday, April 15, 2013 1:36 PM
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Still see the problem even after copying the files


[root@rhel631 management]# cp db.properties /etc/cloudstack/usage/
[root@rhel631 management]# cp log4j-cloud.xml /etc/cloudstack/usage/
[root@rhel631 management]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 management]# service cloudstack-usage status cloudstack-usage 
(pid  13320) is running...
[root@rhel631 management]# service cloudstack-usage status cloudstack-usage is 
stopped

total 68

[root@rhel631 usage]# chmod 755 *
[root@rhel631 usage]# ll
-rwxr-xr-x. 1 root root 64626 Apr 15 09:31 cloudstack-usage.err
-rwxr-xr-x. 1 root root 0 Mar 29 09:16 cloudstack-usage.out
[root@rhel631 usage]# service cloudstack-usage start
Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
[root@rhel631 usage]# service cloudstack-usage status cloudstack-usage is 
stopped



regards
sadhu

-Original Message-
From: Hugo Trippaers [mailto:htrippa...@schubergphilis.com]
Sent: 15 April 2013 13:15
To: dev@cloudstack.apache.org
Subject: RE: [QA]question on cloudstack-usage service

Heya,

I just checked my qa system. Before starting the usage server I copy the 
db.properties and log4j.xml from /etc/cloudstack/management to 
/etc/cloudstack/usage.  Without these two files the usage server will have 
trouble starting. The db.properties not being present could be the cause for 
the DAO failure.

Can you test with this "fix"? If that works I'll fix the usage server to also 
look in /etc/cloudstack/managent for those files and package example 
configuration in the rpm. (In case people want to install the usage server on 
another server than the management server)

Cheers,

Hugo

> -Original Message-
> From: Abhinav Roy [mailto:abhinav@citrix.com]
> Sent: Monday, April 15, 2013 7:55 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Same issue is seen in my setup also.
> 
> Thanks and regards,
> Abhinav
> 
> -Original Message-
> From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> Sent: Monday, April 15, 2013 11:15 AM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Wido/Kelven,
> 
> This issue  noticed on master build please find few more details below:.
>  http://jenkins-ccp.citrix.com/job/ASF-build-master-nonoss-rhel63/211/
> 
> 
> root@centos63 usage]# which jsvc
> /usr/bin/jsvc
> [root@centos63 usage]# /usr/bin/jsvc status
> 15/04/2013 06:53:11 14280 jsvc error: Cannot locate Java Home Manually 
> set the JAVA_HOME classpath but still see the problem
> 
> [root@centos63 usage]# export JAVA_HOME=/usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# echo $JAVA_HOME
> /usr/lib/jvm/jre-1.6.0
> [root@centos63 usage]# /usr/bin/jsvc status
> [root@centos63 usage]# /usr/bin/jsvc start
> 
> Content of  Cloudstack-usage file (FYI)
> 
> SCP=""
> DCP=""
> UCP=`ls /usr/share/cloudstack-usage/cloud-usage-*.jar`":"`ls
> /usr/share/cloudstack-usage/lib/* | tr '\n' ':'` 
> JCP="/usr/share/java/commons-daemon.jar"
> 
> # We need to append the JSVC daemon JAR to the classpath # AgentShell 
> implements the JSVC daemon methods export 
> CLASSPATH="$SCP:$DCP:$UCP:$JCP:/etc/cloudstack/usage"
> 
> 
> Its looks like it's similar to the following issue:
> 
>  https://issues.apache.org/jira/browse/CLOUDSTACK-1746
> 
> 
> regards
> sadhu
> 
> 
> 
> 
> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 13 April 2013 00:11
> To: dev@cloudstack.apache

Re: [DISCUSS] Granular Global Parameters

2013-04-15 Thread Abhinandan Prateek
For Granular params, I am proposing that we use updateConfiguration
command with two additional parameters i.e. scope_type and scope_id.
Where scope_type can be zone, account, cluster or pool  and scope_id will
be the corresponding id of that scope.

We also similarly overload listConfiguration API with these two new params.

-abhi

On 11/04/13 9:06 AM, "Abhinandan Prateek" 
wrote:

>
>
>On 10/04/13 6:30 PM, "David Nalley"  wrote:
>
>>On Wed, Apr 10, 2013 at 7:08 AM, Harikrishna Patnala
>> wrote:
>>> Hi all,
>>>
>>> There are many global parameters which are used to set
>>>values/limits/boolean for various operations, but these parameters
>>>effects all zones/clusters/accounts/storage based on the parameter.
>>> Here I would like to discuss on granulising these parameters so that
>>>these parameters can be customised at different levels
>>>(zone/cluster/account/storage).
>>> New APIs are introduced to update these parameters based on the level
>>>listed in the FS below.
>>> During the creation of zone/cluster/account/storage default values for
>>>the granular parameters are set from the normal global parameters and
>>>later these can be updated using the corresponding API.
>>>
>>>
>>> This proposal for Global granular parameters is detailed in the FS
>>>here: 
>>>https://cwiki.apache.org/confluence/display/CLOUDSTACK/FS+for+Granular+G
>>>l
>>>obal+Configuration+Parameters
>>> The JIRA ticket for tracking is
>>>https://issues.apache.org/jira/browse/CLOUDSTACK-741
>>>
>>> Please review this and provide me comments/suggestions.
>>>
>>> Thanks
>>> Harikrishna
>>
>>
>>Hi Harikrisha:
>>
>>First - the title is a bit confusing; granular and global seems
>>contradictory. Regardless - this is a great move forward.
>>
>>I don't understand why we would inject a new API command
>>(update{storage,cluster,zone,account}levelparamater) instead of just
>>using updateZone, updateAccount, updateStoragePool, etc. For instance,
>>I would expect that listZone would tell me the status of the
>>zone-level options. So why wouldn't updateZone be capable of setting
>>the options
>
>
>Good question. Whether to overload an existing API or create a new one is
>always debatable.
>In this case one of the reason is that the existing update APIs were
>returning a {Zone, Account,..}Response that is not required when you
>change a granular config param. Moreover, all the existing update APIs are
>already heavily overloaded, not a strong reason to avoid further
>overloading though apart from that the response grows in size more you
>overload.
>
>We will also need an API to query the config params at these various
>granularities, that is not mentioned in the FS.
>
>-abhi
>
>
>




VMware volume snapshot improvements Test Plan

2013-04-15 Thread Kiran Koneti

Hi,

VMware volume snapshot improvements Test Plan is posted @  
https://cwiki.apache.org/confluence/display/CLOUDSTACK/VMware+volume+snapshot+improvements
Please review the same and send me the comments or any additional cases needed 
to be added.

Thanks and Regards,
Kiran Koneti


Review Request: populate mode in the "services" dictionary from the network type in the list zones API response

2013-04-15 Thread SrikanteswaraRao Talluri

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

Review request for cloudstack and Prasanna Santhanam.


Description
---

Instead of hard coding the networking type of the zone in "services" dictionary 
, populate it from the zone.networktype 


This addresses bug CLOUDSTACK-2032.


Diffs
-

  test/integration/smoke/test_iso.py 5bd7bb3 
  test/integration/smoke/test_network.py e78cc43 
  test/integration/smoke/test_routers.py 435c7e4 
  test/integration/smoke/test_templates.py 663b174 
  test/integration/smoke/test_vm_life_cycle.py 564f6e8 
  test/integration/smoke/test_volumes.py 7d910d4 

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


Testing
---

TESTED


Thanks,

SrikanteswaraRao Talluri



Re: Review Request: populate mode in the "services" dictionary from the network type in the list zones API response

2013-04-15 Thread Prasanna Santhanam

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

Ship it!


commit e7090e6933f7fa2731c817ead10dfafd650a8e85
Author: Srikanteswararao Talluri 
Date:   Mon Apr 15 16:47:40 2013 +0530

CLOUDSTACK-2032: populate mode in the services dict from the list zones 
instead of hard coding

Signed-off-by: Prasanna Santhanam 


- Prasanna Santhanam


On April 15, 2013, 12:10 p.m., SrikanteswaraRao Talluri wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10477/
> ---
> 
> (Updated April 15, 2013, 12:10 p.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Description
> ---
> 
> Instead of hard coding the networking type of the zone in "services" 
> dictionary , populate it from the zone.networktype 
> 
> 
> This addresses bug CLOUDSTACK-2032.
> 
> 
> Diffs
> -
> 
>   test/integration/smoke/test_iso.py 5bd7bb3 
>   test/integration/smoke/test_network.py e78cc43 
>   test/integration/smoke/test_routers.py 435c7e4 
>   test/integration/smoke/test_templates.py 663b174 
>   test/integration/smoke/test_vm_life_cycle.py 564f6e8 
>   test/integration/smoke/test_volumes.py 7d910d4 
> 
> Diff: https://reviews.apache.org/r/10477/diff/
> 
> 
> Testing
> ---
> 
> TESTED
> 
> 
> Thanks,
> 
> SrikanteswaraRao Talluri
> 
>



Re: [DISCUSS] Granular Global Parameters

2013-04-15 Thread Harikrishna Patnala
Yes Abhi I agree with you, this approach will eliminate the usage of multiple 
APIs.

We can specify scope for each configuration parameter both in config.java file 
and configuration table.
We will not set the default values during the creation of resource 
(Zone/cluster/pool/account).

Whenever we need to update an entity we call updateConfig API with name, value, 
scope and resource ID. This does following,
- validates the scope of the parameter
- checks the resource details table and updates there, if not present then will 
fetch from the global configuration parameters and create entry in the details 
table.

API:  updateConfiguration
Parameters: name, value, scope, entityId

listConfiguration also fetches based on the scope.
API: listConfiguration
Parameters: category, name, scope, entityId



-Harikrishna



On 15-Apr-2013, at 4:36 PM, Abhinandan Prateek 
mailto:cloudst...@aprateek.com>>
 wrote:

For Granular params, I am proposing that we use updateConfiguration
command with two additional parameters i.e. scope_type and scope_id.
Where scope_type can be zone, account, cluster or pool  and scope_id will
be the corresponding id of that scope.

We also similarly overload listConfiguration API with these two new params.

-abhi

On 11/04/13 9:06 AM, "Abhinandan Prateek" 
mailto:abhinandan.prat...@citrix.com>>
wrote:



On 10/04/13 6:30 PM, "David Nalley" mailto:da...@gnsa.us>> wrote:

On Wed, Apr 10, 2013 at 7:08 AM, Harikrishna Patnala
mailto:harikrishna.patn...@citrix.com>> wrote:
Hi all,

There are many global parameters which are used to set
values/limits/boolean for various operations, but these parameters
effects all zones/clusters/accounts/storage based on the parameter.
Here I would like to discuss on granulising these parameters so that
these parameters can be customised at different levels
(zone/cluster/account/storage).
New APIs are introduced to update these parameters based on the level
listed in the FS below.
During the creation of zone/cluster/account/storage default values for
the granular parameters are set from the normal global parameters and
later these can be updated using the corresponding API.


This proposal for Global granular parameters is detailed in the FS
here:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/FS+for+Granular+G
l
obal+Configuration+Parameters
The JIRA ticket for tracking is
https://issues.apache.org/jira/browse/CLOUDSTACK-741

Please review this and provide me comments/suggestions.

Thanks
Harikrishna


Hi Harikrisha:

First - the title is a bit confusing; granular and global seems
contradictory. Regardless - this is a great move forward.

I don't understand why we would inject a new API command
(update{storage,cluster,zone,account}levelparamater) instead of just
using updateZone, updateAccount, updateStoragePool, etc. For instance,
I would expect that listZone would tell me the status of the
zone-level options. So why wouldn't updateZone be capable of setting
the options


Good question. Whether to overload an existing API or create a new one is
always debatable.
In this case one of the reason is that the existing update APIs were
returning a {Zone, Account,..}Response that is not required when you
change a granular config param. Moreover, all the existing update APIs are
already heavily overloaded, not a strong reason to avoid further
overloading though apart from that the response grows in size more you
overload.

We will also need an API to query the config params at these various
granularities, that is not mentioned in the FS.

-abhi








RE: [QA]question on cloudstack-usage service

2013-04-15 Thread Hugo Trippaers
Suresh,

Can you test with the latest build on master? I've updated the usage package to 
fix this problem. See commit ae16f332132a5eb1e3ff85a0c435dbe25a1d6299.

If your tests are ok I can ask Chip to pull it into the 4.1 branch

Cheers,

Hugo

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

> -Original Message-
> From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> Sent: Monday, April 15, 2013 12:47 PM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Its look like we require log4j-cloud_ussge.xml file for successful usage stats
> collection. Right now usage server is running but not able to collect the
> stats(usage job not triggered and no log generated for usage). We will raise
> the bug.
> 
> Regards
> Sadhu
> 
> 
> 
> 
> -Original Message-
> From: Suresh Sadhu
> Sent: 15 April 2013 13:51
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Thanks Abhinav. Its looks like  mistakenly I have applied permissions on
> source folder.
> 
> Hugo: your work around is working fine. After copying  the 2 files in
> /etc/cloudstack/usage directory  ,gave access permission (755) and restart
> the cloud-usage service. Then issue won't see again.
> 
> 
> 
> 
> Regards
> Sadhu
> 
> 
> -Original Message-
> From: Abhinav Roy [mailto:abhinav@citrix.com]
> Sent: 15 April 2013 13:39
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Hi Sadhu,
> 
> I just tried it, we need to give permissions to the files db.properties and
> log4j-cloud.xml after copying them to the /etc/cloudstack/usage directory.
> After that just restart the cloudstack-usage service and the issue won't be
> seen.
> 
> Thanks and regards,
> Abhinav
> 
> -Original Message-
> From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> Sent: Monday, April 15, 2013 1:36 PM
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Still see the problem even after copying the files
> 
> 
> [root@rhel631 management]# cp db.properties /etc/cloudstack/usage/
> [root@rhel631 management]# cp log4j-cloud.xml /etc/cloudstack/usage/
> [root@rhel631 management]# service cloudstack-usage start
> Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
> [root@rhel631 management]# service cloudstack-usage status cloudstack-
> usage (pid  13320) is running...
> [root@rhel631 management]# service cloudstack-usage status cloudstack-
> usage is stopped
> 
> total 68
> 
> [root@rhel631 usage]# chmod 755 *
> [root@rhel631 usage]# ll
> -rwxr-xr-x. 1 root root 64626 Apr 15 09:31 cloudstack-usage.err
> -rwxr-xr-x. 1 root root 0 Mar 29 09:16 cloudstack-usage.out
> [root@rhel631 usage]# service cloudstack-usage start
> Starting CloudStack Usage Monitor cloudstack-usage [  OK  ]
> [root@rhel631 usage]# service cloudstack-usage status cloudstack-usage is
> stopped
> 
> 
> 
> regards
> sadhu
> 
> -Original Message-
> From: Hugo Trippaers [mailto:htrippa...@schubergphilis.com]
> Sent: 15 April 2013 13:15
> To: dev@cloudstack.apache.org
> Subject: RE: [QA]question on cloudstack-usage service
> 
> Heya,
> 
> I just checked my qa system. Before starting the usage server I copy the
> db.properties and log4j.xml from /etc/cloudstack/management to
> /etc/cloudstack/usage.  Without these two files the usage server will have
> trouble starting. The db.properties not being present could be the cause for
> the DAO failure.
> 
> Can you test with this "fix"? If that works I'll fix the usage server to also 
> look
> in /etc/cloudstack/managent for those files and package example
> configuration in the rpm. (In case people want to install the usage server on
> another server than the management server)
> 
> Cheers,
> 
> Hugo
> 
> > -Original Message-
> > From: Abhinav Roy [mailto:abhinav@citrix.com]
> > Sent: Monday, April 15, 2013 7:55 AM
> > To: dev@cloudstack.apache.org
> > Subject: RE: [QA]question on cloudstack-usage service
> >
> > Same issue is seen in my setup also.
> >
> > Thanks and regards,
> > Abhinav
> >
> > -Original Message-
> > From: Suresh Sadhu [mailto:suresh.sa...@citrix.com]
> > Sent: Monday, April 15, 2013 11:15 AM
> > To: dev@cloudstack.apache.org
> > Subject: RE: [QA]question on cloudstack-usage service
> >
> > Wido/Kelven,
> >
> > This issue  noticed on master build please find few more details below:.
> >  http://jenkins-ccp.citrix.com/job/ASF-build-master-nonoss-rhel63/211/
> >
> >
> > root@centos63 usage]# which jsvc
> > /usr/bin/jsvc
> > [root@centos63 usage]# /usr/bin/jsvc status
> > 15/04/2013 06:53:11 14280 jsvc error: Cannot locate Java Home Manually
> > set the JAVA_HOME classpath but still see the problem
> >
> > [root@centos63 usage]# export JAVA_HOME=/usr/lib/jvm/jre-1.6.0
> > [root@centos63 usage]# echo $JAVA_HOME
> > /usr/lib/jvm/jre-1.6.0
> > [root@centos63 usag

Re: [MERGE] ASA 1000v as external firewall in isolated guest networks

2013-04-15 Thread Chip Childers
On Mon, Apr 15, 2013 at 10:43:25AM +0530, Prasanna Santhanam wrote:
> On Fri, Apr 12, 2013 at 12:54:17PM -0400, Chip Childers wrote:
> > > >
> > > >Koushik,
> > > >
> > > >I think that it will require working with Prasanna to get a Marvin test
> > > >suite for the feature ready to go.  I'd personally like to see that
> > > >*prior* to the merge into master.  We need to "up our game" on automated
> > > >testing of new features, so that we stop digging ourselves deeper and
> > > >deeper into a hole.
> > > >
> > > >-chip
> > > 
> > > But this approach isn't scalable either.
> > > 
> > > Every new whiz bang network device will require infrastructure / licenses
> > > / etc. 
> > > 
> > > 
> > > I think if there's component level tests of the code introduced (at least
> > > the network elements) that should be a good first step.
> > > 
> > > --
> > > Chiradeep
> > 
> > Agreed that it would be a good first step, absolutely.  We really do
> > need to think about how we deal with all of these integrations...  and
> > how we do ongoing testing.  It's going to be a challenge either way.
> > 
> > The point of having a marvin test, IMO, isn't necessarily tied to the CI
> > infrastructure.  Instead, it allows for a regression test to be run when
> > and if there is an appropriate target infrastructure.  I'm sort of
> > breaking the problem into two parts: having tests to run if you have the
> > required setup, and figuring out how we get the required setup.
> > 
> 
> In general the way we've taken care of external devices for other
> network appliances, NetScaler for instance, is to assume the
> deployment contains the network element present (inline, side-by-side,
> thingamajig mode) and test the services it provides as a tenant would
> use it. We don't test the actual deployment itself and assume the
> admin has provisioned and registered the appliance appropriately.
> 
> -- 
> Prasanna.,
>

Makes sense, and that jives with my point:  having the tests is
important, because then if you have the gear you know how to confirm the
functionality via automated tests.



RE: [MERGE] ASA 1000v as external firewall in isolated guest networks

2013-04-15 Thread Koushik Das
Added the required tests and merged to master

-Koushik

> -Original Message-
> From: Koushik Das [mailto:koushik@citrix.com]
> Sent: Friday, April 12, 2013 10:30 PM
> To: dev@cloudstack.apache.org
> Subject: RE: [MERGE] ASA 1000v as external firewall in isolated guest
> networks
> 
> 
> 
> > -Original Message-
> > From: Chip Childers [mailto:chip.child...@sungard.com]
> > Sent: Friday, April 12, 2013 7:27 PM
> > To: dev@cloudstack.apache.org
> > Subject: Re: [MERGE] ASA 1000v as external firewall in isolated guest
> > networks
> >
> > On Fri, Apr 12, 2013 at 06:55:56PM +0530, Prasanna Santhanam wrote:
> > > On Fri, Apr 12, 2013 at 09:20:39AM -0400, Chip Childers wrote:
> > > > On Fri, Apr 12, 2013 at 06:39:29AM +, Koushik Das wrote:
> > > > > Currently there are no functional tests, did e2e tests manually.
> > > > > There are some external setup that is required which then needs
> > > > > to be
> > added to Cloudstack like Nexus 1000v, VNMC, ASA 1000v appliance
> > details. I don't see any existing tests related to Vmware/Nexus stuff
> > which I can reuse, so implementing all these steps would need some time.
> > > > >
> > > > > So wanted to check if it is ok to merge now and add the
> > > > > functional tests
> > subsequently?
> > > >
> > > > Before we merge it in, can we first discuss how we want to hand
> > > > this sort of thing?  It seems that at least a marvin test, not run
> > > > by default, would be helpful here.  Certainly documentation on how
> > > > to setup an environment to support the test is also needed.
> > > >
> > > > Do other's have thoughts on this?
> > >
> > > +1 - I can spare some hardware to host virtual instances of the ASA
> > > appliance. We do NetScaler VPXs in this fashion. But I'm not sure
> > > what kind of pre-programming will be required on the ASA to run the
> test.
> > > So some documentation would certainly help.
> >
> > Koushik,
> >
> > I think that it will require working with Prasanna to get a Marvin
> > test suite for the feature ready to go.  I'd personally like to see
> > that
> > *prior* to the merge into master.  We need to "up our game" on
> > automated testing of new features, so that we stop digging ourselves
> > deeper and deeper into a hole.
> 
> Sure will work with Prasanna on this.
> 
> Ideally I would like to write tests for network offering creation with VNMC as
> provider and the newly added APIs for this feature and would like to reuse
> existing code for testing the following features:
> 
> - Vmware cluster setup with Nexus 1000v
> - Network creation
> - Vm deployment
> - Acquiring public IP
> - Static nat/pf rule creation
> 
> Any help on the above would be appreciated.
> 
> 
> >
> > -chip


[ACS41][Patch Request] NiciraNvp implements IpDeployer

2013-04-15 Thread Hugo Trippaers
Please cherry pick 1f0b804dd3972f373c196799fc48929a7bc3f473 to 4.1

Fixes a bug where ips would not apply to a Nicira L3 gateway.


Cheers,

Hugo

> -Original Message-
> From: h...@apache.org [mailto:h...@apache.org]
> Sent: Monday, April 15, 2013 3:43 PM
> To: comm...@cloudstack.apache.org
> Subject: git commit: updated refs/heads/master to 1f0b804
> 
> Updated Branches:
>   refs/heads/master e94c70254 -> 1f0b804dd
> 
> 
> NiciraNvpElement is also implements IpDeployer.
> 
> IpDeployer is required for the L3 components.
> 
> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
> Commit: http://git-wip-
> us.apache.org/repos/asf/cloudstack/commit/1f0b804d
> Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/1f0b804d
> Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/1f0b804d
> 
> Branch: refs/heads/master
> Commit: 1f0b804dd3972f373c196799fc48929a7bc3f473
> Parents: e94c702
> Author: Hugo Trippaers 
> Authored: Mon Apr 15 15:42:10 2013 +0200
> Committer: Hugo Trippaers 
> Committed: Mon Apr 15 15:42:40 2013 +0200
> 
> --
>  client/tomcatconf/componentContext.xml.in   |1 +
>  client/tomcatconf/nonossComponentContext.xml.in |1 +
>  2 files changed, 2 insertions(+), 0 deletions(-)
> --
> 
> 
> http://git-wip-
> us.apache.org/repos/asf/cloudstack/blob/1f0b804d/client/tomcatconf/com
> ponentContext.xml.in
> --
> diff --git a/client/tomcatconf/componentContext.xml.in
> b/client/tomcatconf/componentContext.xml.in
> index 92838fd..23284ab 100644
> --- a/client/tomcatconf/componentContext.xml.in
> +++ b/client/tomcatconf/componentContext.xml.in
> @@ -197,6 +197,7 @@
>
>
>
> +  
>
>  
>
> 
> http://git-wip-
> us.apache.org/repos/asf/cloudstack/blob/1f0b804d/client/tomcatconf/nono
> ssComponentContext.xml.in
> --
> diff --git a/client/tomcatconf/nonossComponentContext.xml.in
> b/client/tomcatconf/nonossComponentContext.xml.in
> index aac3b89..a2182d1 100644
> --- a/client/tomcatconf/nonossComponentContext.xml.in
> +++ b/client/tomcatconf/nonossComponentContext.xml.in
> @@ -293,6 +293,7 @@
>
>
>
> +  
>
>  
>



Re: [ACS41][Patch Request] NiciraNvp implements IpDeployer

2013-04-15 Thread Chip Childers
On Mon, Apr 15, 2013 at 01:45:51PM +, Hugo Trippaers wrote:
> Please cherry pick 1f0b804dd3972f373c196799fc48929a7bc3f473 to 4.1
> 
> Fixes a bug where ips would not apply to a Nicira L3 gateway.
> 
> 
> Cheers,
> 
> Hugo

Done:

commit dbaeda0c211a842fb3a72b5d1f433de7737a47b2
Author: Hugo Trippaers 
Date:   Mon Apr 15 15:42:10 2013 +0200

NiciraNvpElement is also implements IpDeployer.

IpDeployer is required for the L3 components.


Firewall and LB network elements in inline mode

2013-04-15 Thread Koushik Das
The network orchestration component currently supports elements in a 
side-by-side mode. Compatible network elements can be used in this mode to 
offer various network services. There are also scenarios where an inline mode 
would be desirable. For e.g. firewall followed by LB. Wanted to check out on 
the pros/cons of having support for network elements in inline mode in the core 
component itself.

-Koushik


Re: [DOCS] Documentation focused committers, and review processes

2013-04-15 Thread Chip Childers
On Sun, Apr 14, 2013 at 06:05:48PM -0500, Joe Brockmeier wrote:
> On Sun, Apr 14, 2013, at 02:19 AM, Radhika Puthiyetath wrote:
> > Any penalties for people who BLOCK documentation development ?
> 
> Are there rewards for the folks who aren't blocking documentation
> development? It's not like we can dock their volunteer pay. ;-) 
> 
> In all seriousness - I'm sure that it's frustrating to do the work and
> wait on feedback that is slow in coming. But every project suffers from
> more work than hands to do it. I don't think we should be looking to
> penalize or shame anybody, we just need to find ways to help people do
> their work more efficiently.
> 
> We *could* revert features where the developer has not reviewed
> documentation, but who does that help? Assuming a feature is complete
> and isn't buggy, is it better for the user not to have a feature because
> someone didn't have enough time to review docs? Probably not. 
> 
> In this case, I think Chip's proposal is the right one: docs folks
> should do the docs to the best of their ability, commit, and raise the
> new docs to the attention of the developer(s) specifically responsible
> for the feature and the dev@ list in general. (And perhaps even the
> user@ list.)

+1 to what Joe said.  Do the best you can with the information you have,
and harass developers until you get a response.

-chip


Re: [ASFCS42] Proposed schedule for our next release

2013-04-15 Thread Chip Childers
On Thu, Apr 11, 2013 at 02:50:02PM -0700, Animesh Chaturvedi wrote:
> Based on the community discussions of having 4 month cadence I am proposing 
> the following schedule:
> 
> =
>  4.2 detailed schedule proposal:
>  =
> 
> 
>  2013-05-31
>Feature Freeze
>All new feature need to have been merged into master by this date.
>Release branch will be cut on this date.
>Jenkins updated to include new release branch builds.
>  
>  2013-06-01 through 2013-06-30
>Testing/Bug Fixes (testing against jenkins artifacts)
>Documentation Finalization
>  
>  2013-06-30
>Docs Freeze (except release notes and translations)
>Release Branch moves to limited updates only (only commits allowed
>  in would be release blockers fixes, translation updates, etc...)
>  
>  2013-07-01 through 2013-07-22
>Translation Development and Integration (should be ongoing, but
>  focused effort)
>Final regression testing / bug fixes / doc fixes
>  
>  2013-07-22
>4.2.0-RC1 is created, and project level VOTE is called
> 

+1 to the plan above.

> 
> I want to call out my concern on technical debt we have accumulated so far.
> 
>  I did an analysis on JIRA bugs yesterday night PST on "Affects Version = 
> 4.1" and created since Dec 2012 
> 
> Total records : 429
> Resolution Type (Invalid, Duplicate, Cannot reproduce etc.) : 87 (30 
> Blockers, 27 Critical, 27 Major, 4 Minor)
> Valid Defects  : 429-87= 342
> Fixed : 246 (60 Blockers, 70 Critical, 99 Majors) out of which 217 were fixed 
> since Feb
> Unresolved : 96 (1 Blocker, 8 Critical, 64 Major)
> 
> With this data it looks like we have fixed 2/3 of valid defects in little 
> over 2 months and pretty much deferring around 1/3 rd of issues for future 
> release.
> 
> I also looked at overall backlog of bugs (Critical, Major and Blockers only)  
> as of 4/10/2013 - 10:0PM PST. 
> 
> 284 open (18 Blocker, 38 Critical, 228 Major) ; 
> By Fix version
>   -  Release 4.0.x and prior: 13
>   -  4.1: 70
>   -  4.2 : 97
>   -  Future: 8
>   -  No version: 107
> 
> Looking at that we fixed 217 bugs in roughly 2 months during 4.1 cycle,  
> fixing the backlog of bug  will probably take us 2 months.  Should we extend 
> the 4.2 test cycle by 2 months [Original Schedule: 6/1 - 7/22, Extended 
> Schedule: 6/1-9/22] to reduce the technical debt significantly? I would like 
> to hear how community wants to address technical debt. Based on the input and 
> consensus I will publish the agreed schedule next week.
> 
> 

I don't think that an extension of time changes bug counts really.  IMO,
we need to pull together to have some bug-fix focused effort applied to
the code-base.  It's also another reason that I'm so big on making sure
that automated tests come in with the new features.  That doesn't
address test scenarios that human testers can come up with, but if a
developer spends the time to think about testing the basic feature and
codifies that, we should at least avoid the "this actually doesn't work
at all" types of bugs.

There's a school of thought that says, don't build another feature until
you have sorted out the known bugs in the current features.  I don't
think we could really pull that off, but perhaps a different thread to
rally people around the bug backlog is in order?

-chip


RE: [DOCS] Documentation focused committers, and review processes

2013-04-15 Thread Radhika Puthiyetath
Thanks everyone! I appreciate the suggestions/ reviews/ comments etc. 

Let me  clarify that my intention was not to penalize anyone-I am worried about 
those users who end up following  any  incomplete / half-cooked piece of doc of 
a superb feature and failing to achieve the purpose



-Original Message-
From: Joe Brockmeier [mailto:j...@zonker.net] 
Sent: Monday, April 15, 2013 4:36 AM
To: dev@cloudstack.apache.org
Subject: Re: [DOCS] Documentation focused committers, and review processes

On Sun, Apr 14, 2013, at 02:19 AM, Radhika Puthiyetath wrote:
> Any penalties for people who BLOCK documentation development ?

Are there rewards for the folks who aren't blocking documentation development? 
It's not like we can dock their volunteer pay. ;-) 

In all seriousness - I'm sure that it's frustrating to do the work and wait on 
feedback that is slow in coming. But every project suffers from more work than 
hands to do it. I don't think we should be looking to penalize or shame 
anybody, we just need to find ways to help people do their work more 
efficiently.

We *could* revert features where the developer has not reviewed documentation, 
but who does that help? Assuming a feature is complete and isn't buggy, is it 
better for the user not to have a feature because someone didn't have enough 
time to review docs? Probably not. 

In this case, I think Chip's proposal is the right one: docs folks should do 
the docs to the best of their ability, commit, and raise the new docs to the 
attention of the developer(s) specifically responsible for the feature and the 
dev@ list in general. (And perhaps even the user@ list.) 

Best,

jzb
--
Joe Brockmeier
j...@zonker.net
Twitter: @jzb
http://www.dissociatedpress.net/


RE: [DOCS] Documentation focused committers, and review processes

2013-04-15 Thread Rajesh Battala


> -Original Message-
> From: Chip Childers [mailto:chip.child...@sungard.com]
> Sent: Monday, April 15, 2013 7:47 PM
> To: dev@cloudstack.apache.org
> Subject: Re: [DOCS] Documentation focused committers, and review processes
> 
> On Sun, Apr 14, 2013 at 06:05:48PM -0500, Joe Brockmeier wrote:
> > On Sun, Apr 14, 2013, at 02:19 AM, Radhika Puthiyetath wrote:
> > > Any penalties for people who BLOCK documentation development ?
> >
> > Are there rewards for the folks who aren't blocking documentation
> > development? It's not like we can dock their volunteer pay. ;-)
> >
> > In all seriousness - I'm sure that it's frustrating to do the work and
> > wait on feedback that is slow in coming. But every project suffers
> > from more work than hands to do it. I don't think we should be looking
> > to penalize or shame anybody, we just need to find ways to help people
> > do their work more efficiently.
> >
> > We *could* revert features where the developer has not reviewed
> > documentation, but who does that help? Assuming a feature is complete
> > and isn't buggy, is it better for the user not to have a feature
> > because someone didn't have enough time to review docs? Probably not.
> >
> > In this case, I think Chip's proposal is the right one: docs folks
> > should do the docs to the best of their ability, commit, and raise the
> > new docs to the attention of the developer(s) specifically responsible
> > for the feature and the dev@ list in general. (And perhaps even the
> > user@ list.)
> 
> +1 to what Joe said.  Do the best you can with the information you have,
> and harass developers until you get a response.
> 
> -chip

+1 for joe and chip :) 


Re: [1/2] Adding issues fixed in 4.1.0

2013-04-15 Thread Milamber

Hello jzb,

A lot of description seems have lost the last characters:

+Vmware network labels are ignored when creating a Zone using basic 
networkin

+ UI provides an option to reconnect a disconnected host - 
ServerApiException is thrown on an attemp


+When Datacenter name in VCenter has spaces Primary Storage (VMFS) discovery 
will fai


Etc.

Milamber


Le 15/04/2013 04:09, j...@apache.org a ecrit :

Updated Branches:
   refs/heads/4.1 5c542ea15 ->  1eceaf3a8


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/1eceaf3a/docs/en-US/Release_Notes.xml
--
diff --git a/docs/en-US/Release_Notes.xml b/docs/en-US/Release_Notes.xml
index b8c5c01..5b9571e 100644
--- a/docs/en-US/Release_Notes.xml
+++ b/docs/en-US/Release_Notes.xml
@@ -27,6 +27,3942 @@ under the License.
  This document contains information specific to this release of&PRODUCT;, including upgrade instructions from prior releases, new features added 
to&PRODUCT;, API changes, and issues fixed in the release. For installation instructions, please see thehttp://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.1.0/html/Installation_Guide/index.html";>Installation Guide. For usage and 
administration instructions, please see thehttp://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.1.0/html/Admin_Guide/index.html";>&PRODUCT; Administrator's Guide. Developers 
and users who wish to work with the API will find instruction in thehttp://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.0.1-incubating/html/API_Developers_Guide/index.html";>&PRODUCT; API Developer's 
Guide
  If you find any errors or problems in this guide, please see. We hope you enjoy working with&PRODUCT;!
  
+
+Version 4.1.0
+
+What’s New in 4.1
+Apache CloudStack 4.1.0 includes many new features. This section covers the 
most prominent new features and changes.
+
+Localization
+The 4.1.0 release adds partial User Interface (UI) support for Catalan, 
Chinese, French, German, Italian, Japanese, Korean, Norwegian, Portuguese, Russian, and 
Spanish. Not all languages are complete.
+The 4.1.0 release also adds documentation translations for Chinese, Chinese 
(Taiwan), Italian, Japanese, Korean, and Portuguese.
+
+
+
+Issues Fixed in 4.1.0
+Apache CloudStack useshttps://issues.apache.org/jira/browse/CLOUDSTACK";>Jira
+to track its issues. All new features and bugs for 4.1.0 have 
been tracked in Jira, and have
+a standard naming convention of "CLOUDSTACK-" where "" is 
the issue number.
+This section includes a summary of known issues against 4.0.0 that were fixed 
in 4.1.0. Approximately 470 bugs were resolved or closed in the 4.1.0 cycle.
+
+
+
+
+
+
+
+Defect
+
+
+Description
+
+
+
+
+
+
+CLOUDSTACK-46
+
+
+Remnants of mycloud remain.
+
+
+
+
+CLOUDSTACK-70
+
+
+Improve Network Restart Behaviour for Basic Zone: Restarting Network 
Fail
+
+
+
+
+CLOUDSTACK-94
+
+
+"API command, listIsos documentation clarity
+
+
+
+
+CLOUDSTACK-95
+
+
+IP address allocation not working when a user tries to allocate IP addresses 
in a Project
+
+
+
+
+CLOUDSTACK-97
+
+
+Vmware network labels are ignored when creating a Zone using basic 
networkin
+
+
+
+
+CLOUDSTACK-108
+
+
+VM should not be allowed to be deployed on two Isolated Networks of an Account 
that were created from DefaultNetworkOfferingwithSourceNATService
+
+
+
+
+CLOUDSTACK-118
+
+
+"Status of host resorce stuck in ""ErrorInMaintenance""
+
+
+
+
+CLOUDSTACK-119
+
+
+Move Agent-Simulator in to the hypervisor plugin mode
+
+
+
+
+CLOUDSTACK-130
+
+
+Clarify docs on tags parameter in API referenc
+
+
+
+
+CLOUDSTACK-152
+
+
+Routes on the User VM are programmed incorrectly on a VM present on both 
Isolated and Shared Guest Network
+
+
+
+
+CLOUDSTACK-178
+
+
+Expose name parameter of VM in list Vm view
+
+
+
+
+CLOUDSTACK-198
+
+
+vpn:failto add VPN Users deletes all the existing Vpn use
+
+
+
+
+CLOUDSTACK-222
+
+
+Admin UI prompts to restart Management server with cancel edit 
operatio
+
+
+
+
+CLOUDSTACK-225
+
+
+API Docs: Request params repeated with different description
+
+
+
+
+CLOUDSTACK-226
+
+
+UpdatePhysicalNetworkcommand failed due to java.sql.BatchUpdateException ; 
Tried to extend the existing Guest VLAN Range of one physical network into the Guest VLAN 
range of the other physical networ
+
+
+
+
+CLOUDSTACK-227
+
+
+ReconnectHostCmd: NullPointerException: Unable to get host Information for 
XenServer 6.0.2 host - on intentionally changing the traffic labels on the physical 
networ
+
+
+
+
+CLOUDSTACK-228
+
+
+UI provides an option to reconnect a disconnected host - ServerApiException is 
thrown on an attemp
+
+
+
+
+CLOUDSTACK-232
+
+
+Zone infrastructure chart -- disable resource total displa
+
+
+
+
+CLOUDSTACK-235
+
+
+Network rate can be set in 2 places. Clarify docs on how this 
works
+
+
+
+
+CLOUDSTACK-249
+
+
+Add host id to failed VM deploy alert
+
+
+
+
+CLOUDSTACK-250
+
+
+Inc

How to set the default network when I specify IP use "iptonetworklist" parameter when deploy a vm?

2013-04-15 Thread sx chen
Hi,
   as far as I know, I can specify the default network by put it at the
first when use "*networkids*" paramenter when deploy a vm.
   but if I use "iptonetworklist" parameter specify the IP address,I find I
can't specify the default network.I've test put it at the first and put it
at the last.
   So if I want to specify the default network as well as the IP
address,what should I do?
appreciate your help!


Re: [DOC] Opening vncextras firewall ports in ESXi 5.1

2013-04-15 Thread Jason Cadmus
What page on the wiki should I create my documentation so I  can create a
doc task to get it ported?


On Thu, Apr 11, 2013 at 12:41 PM, Joe Brockmeier  wrote:

> On 04/10/2013 04:23 PM, Jason Cadmus wrote:
>
>> I have some documentation on how to accomplish this task and would like to
>> contribute it to the ACS documentation for CS 4.1.
>>
>
> Awesome!
>
>
>  I am going through the following link below, trying to comprehend how to
>> accomplish this. I have downloaded the source for 4.1, compiled to
>> documentation with publican, and see what section the information needs to
>> added to.
>> https://cwiki.apache.org/**CLOUDSTACK/cloudstack-**
>> documentation-contributors-**overview.html
>>
>> I do not have a developer background what so every so please
>> have patience with my questions.
>>
>
> Ask away, we're very happy to help folks who want to contribute. Speaking
> as a non-developer who's also working on docs, I understand that it
> involves a bit of a learning curve.
>
>
>  Should create a new xml doc or do I edit and existing xml doc?
>>
>
> It depends on whether what you want to add fits with an existing section,
> or whether it needs its own section.
>
> Another option would be to write the docs on the wiki and create a doc
> task to port the documentation into the appropriate guide. For a first-time
> docs contribution, that might be an easy way to go. Then you'd have an
> template to work off of for the next time.
>
> We're usually on IRC during the day as well if you have questions about
> how to work in Publican, or anything else.
>
>
>  Is there a easier way for someone to contribute with having a Sys Admin
>> background?
>>
>
> We can use a lot of help getting the wiki in shape and if you have
> experience with CloudStack and want to add to the wiki information, that'd
> be awesome as well.
>
> I'm sure other folks have ideas too.
>
> Thanks!
>
> Joe
>



-- 
Thanks,
JC


RE: [DOC] Opening vncextras firewall ports in ESXi 5.1

2013-04-15 Thread Musayev, Ilya
Jason,

Please point me to where you begin this effort.

For VNC and vSphere:
This is hardcoded within CS. First 64 port from  5900-5964 and if all used, 
goes to higher 59000-6..

I did an analysis of all systems VMs and ports used with CS4.1 and Vsphere, I 
will post the details - hopefully this week.


Regards
ilya





> -Original Message-
> From: Jason Cadmus [mailto:jdcad...@gmail.com]
> Sent: Monday, April 15, 2013 10:58 AM
> To: dev@cloudstack.apache.org
> Subject: Re: [DOC] Opening vncextras firewall ports in ESXi 5.1
> 
> What page on the wiki should I create my documentation so I  can create a
> doc task to get it ported?
> 
> 
> On Thu, Apr 11, 2013 at 12:41 PM, Joe Brockmeier  wrote:
> 
> > On 04/10/2013 04:23 PM, Jason Cadmus wrote:
> >
> >> I have some documentation on how to accomplish this task and would
> >> like to contribute it to the ACS documentation for CS 4.1.
> >>
> >
> > Awesome!
> >
> >
> >  I am going through the following link below, trying to comprehend how
> > to
> >> accomplish this. I have downloaded the source for 4.1, compiled to
> >> documentation with publican, and see what section the information
> >> needs to added to.
> >> https://cwiki.apache.org/**CLOUDSTACK/cloudstack-**
> >> documentation-contributors-
> **overview.html >> LOUDSTACK/cloudstack-documentation-contributors-overview.html>
> >>
> >> I do not have a developer background what so every so please have
> >> patience with my questions.
> >>
> >
> > Ask away, we're very happy to help folks who want to contribute.
> > Speaking as a non-developer who's also working on docs, I understand
> > that it involves a bit of a learning curve.
> >
> >
> >  Should create a new xml doc or do I edit and existing xml doc?
> >>
> >
> > It depends on whether what you want to add fits with an existing
> > section, or whether it needs its own section.
> >
> > Another option would be to write the docs on the wiki and create a doc
> > task to port the documentation into the appropriate guide. For a
> > first-time docs contribution, that might be an easy way to go. Then
> > you'd have an template to work off of for the next time.
> >
> > We're usually on IRC during the day as well if you have questions
> > about how to work in Publican, or anything else.
> >
> >
> >  Is there a easier way for someone to contribute with having a Sys
> > Admin
> >> background?
> >>
> >
> > We can use a lot of help getting the wiki in shape and if you have
> > experience with CloudStack and want to add to the wiki information,
> > that'd be awesome as well.
> >
> > I'm sure other folks have ideas too.
> >
> > Thanks!
> >
> > Joe
> >
> 
> 
> 
> --
> Thanks,
> JC



Re: [ACS41][DOCS] Summarize Javelin?

2013-04-15 Thread Chip Childers
On Mon, Apr 15, 2013 at 12:23:33AM -0500, Joe Brockmeier wrote:
> Hi all, 
> 
> For the release notes, can someone summarize what *exactly* Javelin is
> and how it impacts the user in, say, two paragraphs or less? (In other
> words, what's the layperson description that will make sense to
> non-developers?) 
> 
> Best,
> 
> jzb
> -- 
> Joe Brockmeier
> j...@zonker.net
> Twitter: @jzb
> http://www.dissociatedpress.net/
>

Joe,

There were actually several things "embedded" in the Javelin work, most
of which were pulled out into their own features / improvements.

The one that remains, AFAIK, is the switch to Spring.

Here's how I'd summarize:

Starting with release 4.1.0, Apache CloudStack now makes use of the
Spring Framework for component injection, instead of the custom
injection framework used in previous versions of the software.  While
these changes primarily impact developers of the software (and related
plugins), users need to understand the new configuration approach if
they have customized their applicationContext.xml or
componentContext.xml files.  Details on how to configure custom plugins
or select different plugins can be found on the Apache CloudStack wiki
here:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Using+Spring+in+CloudStack


RE: [DISCUSS] VM appliance with CS pre-installed

2013-04-15 Thread Musayev, Ilya
Sailaja,

This is going to differ from DevCloud slightly. 

1) CS is going to be preinstalled and started
2) NFS will be-preconfigured

I will most likely roll non-oss versions since it will cover all use cases.

Regards
ilya

> -Original Message-
> From: Sailaja Mada [mailto:sailaja.m...@citrix.com]
> Sent: Sunday, April 14, 2013 9:25 PM
> To: dev@cloudstack.apache.org; Musayev, Ilya
> Subject: RE: [DISCUSS] VM appliance with CS pre-installed
> 
> Hi,
> 
> Can you please share how it is going to be different from DevCloud.   Will you
> be automating the installation process of CS(oss & non-oss), So that it can be
> used with any hypervisor?
> 
> 
> Thanks,
> Sailaja.M
> 
> -Original Message-
> From: Musayev, Ilya [mailto:imusa...@webmd.net]
> Sent: Monday, April 15, 2013 12:50 AM
> To: dev@cloudstack.apache.org
> Subject: [DISCUSS] VM appliance with CS pre-installed
> 
> I'd like to simplify the CS onboarding expirience even further by providing vm
> images with CS pre-installed in appliance format (perhaps including system
> vms as bundle and nfs preconfigured)
> 
> I can automate this process completely hands off with or without vagrant
> (not a fan of vbox).
> 
> Im curious what people think? Should this be hosted externally?
> 



Re: [DOC] Opening vncextras firewall ports in ESXi 5.1

2013-04-15 Thread Jason Cadmus
Ilya, please review
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Configure+vncextras+and+nfsClient+ESXi+firewall+ports

Thanks
JC


On Mon, Apr 15, 2013 at 11:12 AM, Musayev, Ilya  wrote:

> Jason,
>
> Please point me to where you begin this effort.
>
> For VNC and vSphere:
> This is hardcoded within CS. First 64 port from  5900-5964 and if all
> used, goes to higher 59000-6..
>
> I did an analysis of all systems VMs and ports used with CS4.1 and
> Vsphere, I will post the details - hopefully this week.
>
>
> Regards
> ilya
>
>
>
>
>
> > -Original Message-
> > From: Jason Cadmus [mailto:jdcad...@gmail.com]
> > Sent: Monday, April 15, 2013 10:58 AM
> > To: dev@cloudstack.apache.org
> > Subject: Re: [DOC] Opening vncextras firewall ports in ESXi 5.1
> >
> > What page on the wiki should I create my documentation so I  can create a
> > doc task to get it ported?
> >
> >
> > On Thu, Apr 11, 2013 at 12:41 PM, Joe Brockmeier  wrote:
> >
> > > On 04/10/2013 04:23 PM, Jason Cadmus wrote:
> > >
> > >> I have some documentation on how to accomplish this task and would
> > >> like to contribute it to the ACS documentation for CS 4.1.
> > >>
> > >
> > > Awesome!
> > >
> > >
> > >  I am going through the following link below, trying to comprehend how
> > > to
> > >> accomplish this. I have downloaded the source for 4.1, compiled to
> > >> documentation with publican, and see what section the information
> > >> needs to added to.
> > >> https://cwiki.apache.org/**CLOUDSTACK/cloudstack-**
> > >> documentation-contributors-
> > **overview.html > >> LOUDSTACK/cloudstack-documentation-contributors-overview.html>
> > >>
> > >> I do not have a developer background what so every so please have
> > >> patience with my questions.
> > >>
> > >
> > > Ask away, we're very happy to help folks who want to contribute.
> > > Speaking as a non-developer who's also working on docs, I understand
> > > that it involves a bit of a learning curve.
> > >
> > >
> > >  Should create a new xml doc or do I edit and existing xml doc?
> > >>
> > >
> > > It depends on whether what you want to add fits with an existing
> > > section, or whether it needs its own section.
> > >
> > > Another option would be to write the docs on the wiki and create a doc
> > > task to port the documentation into the appropriate guide. For a
> > > first-time docs contribution, that might be an easy way to go. Then
> > > you'd have an template to work off of for the next time.
> > >
> > > We're usually on IRC during the day as well if you have questions
> > > about how to work in Publican, or anything else.
> > >
> > >
> > >  Is there a easier way for someone to contribute with having a Sys
> > > Admin
> > >> background?
> > >>
> > >
> > > We can use a lot of help getting the wiki in shape and if you have
> > > experience with CloudStack and want to add to the wiki information,
> > > that'd be awesome as well.
> > >
> > > I'm sure other folks have ideas too.
> > >
> > > Thanks!
> > >
> > > Joe
> > >
> >
> >
> >
> > --
> > Thanks,
> > JC
>
>


-- 
Thanks,
JC


[DISCUSS] Getting rid of premium

2013-04-15 Thread Hugo Trippaers
Heya,

As we don't have a premium distribution anymore I would like to get rid of the 
premium configuration value. At the moment it seems to be unintentionally used 
to distinguish between the nonss and oss build so I think it can go. It will 
likely only confuse people that are new to the project and doesn't bring 
anything usefull so it seems.

I'm willing to trawl through the code and remove any references to premium and 
replace it with another flag where needed (for example a flag to indicate 
vmware support is available).

Any thoughts about this?

Cheers,

Hugo


Re: [DISCUSS] Getting rid of premium

2013-04-15 Thread Chip Childers
On Mon, Apr 15, 2013 at 03:53:59PM +, Hugo Trippaers wrote:
> Heya,
> 
> As we don't have a premium distribution anymore I would like to get rid of 
> the premium configuration value. At the moment it seems to be unintentionally 
> used to distinguish between the nonss and oss build so I think it can go. It 
> will likely only confuse people that are new to the project and doesn't bring 
> anything usefull so it seems.
> 
> I'm willing to trawl through the code and remove any references to premium 
> and replace it with another flag where needed (for example a flag to indicate 
> vmware support is available).
> 
> Any thoughts about this?
> 
> Cheers,
> 
> Hugo
>

+1 - Just be sure to create a Jira "Improvement" to track it please!


RE: [DOC] Opening vncextras firewall ports in ESXi 5.1

2013-04-15 Thread Musayev, Ilya
I would ask another VMWare person to confirm this, but from what I've seen in 
past and based on the source code, CS automatically creates firewall rules and 
executes them directly on VCenter - it does so repeatedly - on various 
instances (add host to clusters, disable or enable  host or cluster, etc..)

So while the config is helpful, in my opinion, it's not necessary. 

> -Original Message-
> From: Jason Cadmus [mailto:jdcad...@gmail.com]
> Sent: Monday, April 15, 2013 11:47 AM
> To: dev@cloudstack.apache.org
> Subject: Re: [DOC] Opening vncextras firewall ports in ESXi 5.1
> 
> Ilya, please review
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Configure+vncex
> tras+and+nfsClient+ESXi+firewall+ports
> 
> Thanks
> JC
> 
> 
> On Mon, Apr 15, 2013 at 11:12 AM, Musayev, Ilya 
> wrote:
> 
> > Jason,
> >
> > Please point me to where you begin this effort.
> >
> > For VNC and vSphere:
> > This is hardcoded within CS. First 64 port from  5900-5964 and if all
> > used, goes to higher 59000-6..
> >
> > I did an analysis of all systems VMs and ports used with CS4.1 and
> > Vsphere, I will post the details - hopefully this week.
> >
> >
> > Regards
> > ilya
> >
> >
> >
> >
> >
> > > -Original Message-
> > > From: Jason Cadmus [mailto:jdcad...@gmail.com]
> > > Sent: Monday, April 15, 2013 10:58 AM
> > > To: dev@cloudstack.apache.org
> > > Subject: Re: [DOC] Opening vncextras firewall ports in ESXi 5.1
> > >
> > > What page on the wiki should I create my documentation so I  can
> > > create a doc task to get it ported?
> > >
> > >
> > > On Thu, Apr 11, 2013 at 12:41 PM, Joe Brockmeier 
> wrote:
> > >
> > > > On 04/10/2013 04:23 PM, Jason Cadmus wrote:
> > > >
> > > >> I have some documentation on how to accomplish this task and
> > > >> would like to contribute it to the ACS documentation for CS 4.1.
> > > >>
> > > >
> > > > Awesome!
> > > >
> > > >
> > > >  I am going through the following link below, trying to comprehend
> > > > how to
> > > >> accomplish this. I have downloaded the source for 4.1, compiled
> > > >> to documentation with publican, and see what section the
> > > >> information needs to added to.
> > > >> https://cwiki.apache.org/**CLOUDSTACK/cloudstack-**
> > > >> documentation-contributors-
> > > **overview.html > > >> LOUDSTACK/cloudstack-documentation-contributors-overview.html>
> > > >>
> > > >> I do not have a developer background what so every so please have
> > > >> patience with my questions.
> > > >>
> > > >
> > > > Ask away, we're very happy to help folks who want to contribute.
> > > > Speaking as a non-developer who's also working on docs, I
> > > > understand that it involves a bit of a learning curve.
> > > >
> > > >
> > > >  Should create a new xml doc or do I edit and existing xml doc?
> > > >>
> > > >
> > > > It depends on whether what you want to add fits with an existing
> > > > section, or whether it needs its own section.
> > > >
> > > > Another option would be to write the docs on the wiki and create a
> > > > doc task to port the documentation into the appropriate guide. For
> > > > a first-time docs contribution, that might be an easy way to go.
> > > > Then you'd have an template to work off of for the next time.
> > > >
> > > > We're usually on IRC during the day as well if you have questions
> > > > about how to work in Publican, or anything else.
> > > >
> > > >
> > > >  Is there a easier way for someone to contribute with having a Sys
> > > > Admin
> > > >> background?
> > > >>
> > > >
> > > > We can use a lot of help getting the wiki in shape and if you have
> > > > experience with CloudStack and want to add to the wiki
> > > > information, that'd be awesome as well.
> > > >
> > > > I'm sure other folks have ideas too.
> > > >
> > > > Thanks!
> > > >
> > > > Joe
> > > >
> > >
> > >
> > >
> > > --
> > > Thanks,
> > > JC
> >
> >
> 
> 
> --
> Thanks,
> JC



Review Request: CLOUDSTACK-1871 : domainId parameter to uploadVolume not working

2013-04-15 Thread Isaac Chiang

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

Review request for cloudstack and Pranav Saxena.


Description
---

CLOUDSTACK-1871 : domainId parameter to uploadVolume not working

1. Remove duplicated lines for setting domainId.
2. Set domainId with owner's domain if the owner is specified ( with account 
and domainid passed in)


This addresses bug CLOUDSTACK-1871.


Diffs
-

  server/src/com/cloud/storage/VolumeManagerImpl.java 1e8edaf 

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


Testing
---

Test from UI and curl request, both work well


Thanks,

Isaac Chiang



Re: How to set the default network when I specify IP use "iptonetworklist" parameter when deploy a vm?

2013-04-15 Thread Alena Prokharchyk

NetworkIds parameter has to be specify regardless of iptonetworklist
parameter presence. So you have to define both parameters, and the first
network from the networkIds list will be used as a default.

-Alena.

On 4/15/13 7:44 AM, "sx chen"  wrote:

>Hi,
>   as far as I know, I can specify the default network by put it at the
>first when use "*networkids*" paramenter when deploy a vm.
>   but if I use "iptonetworklist" parameter specify the IP address,I find
>I
>can't specify the default network.I've test put it at the first and put it
>at the last.
>   So if I want to specify the default network as well as the IP
>address,what should I do?
>appreciate your help!
>




RE: [Add/Remove Network to VM] Multiple NICs on same Guest Network

2013-04-15 Thread Saksham Srivastava
We have a feature Multiple IPs on the same NIC 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Multiple+IP+address+per+NIC

Isn't the functionality covered by this feature.

I do not see a strong use case for having both the features (Multi NIC in same 
network and Multi IP per NIC).
Should we allow only one of them, as they both tend to serve the same purpose, 
or am I missing something?

Thanks,
Saksham

From: Marcus Sorensen [shadow...@gmail.com]
Sent: Saturday, April 13, 2013 9:03 PM
To: Saksham Srivastava
Cc: dev@cloudstack.apache.org
Subject: Re: [Add/Remove Network to VM] Multiple NICs on same Guest Network


I believe it was allowed because one can also create multiple NICs on the same 
network while deploying a VM.

There may be people doing that to get multiple IPS auto-assigned to a VM.

On Apr 13, 2013 3:28 AM, "Saksham Srivastava" 
mailto:saksham.srivast...@citrix.com>> wrote:
Hi,

Using addNicToVirtualMachine API user can add multiple NICs to a VM

I am also able to add multiple NICs to a VM on the same isolated guest network.

Is this a valid scenario??
If yes what could be the use case for the same?

Thanks,
Saksham



Re: [Add/Remove Network to VM] Multiple NICs on same Guest Network

2013-04-15 Thread Marcus Sorensen
The functionality was duplicated from deploy allowing multiple NICs, and I
wasn't involved in the discusion as to why that was decided. There could be
other reasons why that was put in place, I don't know. At any rate it
doesn't hurt anything in particular, and I'm not sure how many are using
the capability in 4.0 and earlier, but they may have gotten used to it.


On Mon, Apr 15, 2013 at 10:32 AM, Saksham Srivastava <
saksham.srivast...@citrix.com> wrote:

>   We have a feature Multiple IPs on the same NIC
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Multiple+IP+address+per+NIC
>
> Isn't the functionality covered by this feature.
>
> I do not see a strong use case for having both the features (Multi NIC in
> same network and Multi IP per NIC).
> Should we allow only one of them, as they both tend to serve the same
> purpose, or am I missing something?
>
> Thanks,
> Saksham
>  --
> *From:* Marcus Sorensen [shadow...@gmail.com]
> *Sent:* Saturday, April 13, 2013 9:03 PM
> *To:* Saksham Srivastava
> *Cc:* dev@cloudstack.apache.org
> *Subject:* Re: [Add/Remove Network to VM] Multiple NICs on same Guest
> Network
>
>   I believe it was allowed because one can also create multiple NICs on
> the same network while deploying a VM.
>
> There may be people doing that to get multiple IPS auto-assigned to a VM.
> On Apr 13, 2013 3:28 AM, "Saksham Srivastava" <
> saksham.srivast...@citrix.com> wrote:
>
>>  Hi,
>>
>> Using addNicToVirtualMachine API user can add multiple NICs to a VM
>>
>> I am also able to add multiple NICs to a VM on the same isolated guest
>> network.
>>
>> Is this a valid scenario??
>> If yes what could be the use case for the same?
>>
>> Thanks,
>> Saksham
>>
>>


Re: [DOC] Opening vncextras firewall ports in ESXi 5.1

2013-04-15 Thread Joe Brockmeier
On Mon, Apr 15, 2013, at 09:58 AM, Jason Cadmus wrote:
> What page on the wiki should I create my documentation so I  can create a
> doc task to get it ported?

Good question. I started a wiki page for this generally:

https://cwiki.apache.org/confluence/display/CLOUDSTACK/Docs+in+Process

And for this specific feature:

https://cwiki.apache.org/confluence/display/CLOUDSTACK/Opening+Firewall+Ports+in+ESXi+5.1

Once you're happy with the docs as written, just open a Jira ticket
under the Docs component and send a note to dev@ that you'd like someone
to convert to Publican. Please let me know if you need anything else,
and thanks!

Joe
 
Best,

jzb
-- 
Joe Brockmeier
j...@zonker.net
Twitter: @jzb
http://www.dissociatedpress.net/


Re: [DOC] Opening vncextras firewall ports in ESXi 5.1

2013-04-15 Thread Jason Cadmus
Thanks Joe

I have been talking Ilya and wanted to do a bit more testing prior to
posting my documentation. I did start a page at the following link although
it can be that can be deleted. "My confluence permissions do not allow me
to delete the page I added."
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Configure+vncextras+(page+can+be+deleted)

Thanks
JC




On Mon, Apr 15, 2013 at 12:52 PM, Joe Brockmeier  wrote:

> On Mon, Apr 15, 2013, at 09:58 AM, Jason Cadmus wrote:
> > What page on the wiki should I create my documentation so I  can create a
> > doc task to get it ported?
>
> Good question. I started a wiki page for this generally:
>
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Docs+in+Process
>
> And for this specific feature:
>
>
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Opening+Firewall+Ports+in+ESXi+5.1
>
> Once you're happy with the docs as written, just open a Jira ticket
> under the Docs component and send a note to dev@ that you'd like someone
> to convert to Publican. Please let me know if you need anything else,
> and thanks!
>
> Joe
>
> Best,
>
> jzb
> --
> Joe Brockmeier
> j...@zonker.net
> Twitter: @jzb
> http://www.dissociatedpress.net/
>



-- 
Thanks,
JC


Re: Review Request: Fix command for executing injectkeys.sh

2013-04-15 Thread Chiradeep Vittal

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

Ship it!


Ship It!

- Chiradeep Vittal


On April 12, 2013, 9:11 p.m., Phong Nguyen wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10437/
> ---
> 
> (Updated April 12, 2013, 9:11 p.m.)
> 
> 
> Review request for cloudstack and Hugo Trippaers.
> 
> 
> Description
> ---
> 
> Management server unable to start (from rpm) due to invalid call to Script 
> for injectkeys.sh. 
> 
> 2013-04-12 15:02:13,496 WARN  [cloud.server.ConfigurationServerImpl] 
> (Timer-1:null) Failed to inject generated public key into systemvm iso 
> java.io.IOException: Cannot run program "/bin/bash 
> /usr/share/cloudstack-common/scripts/vm/systemvm/injectkeys.sh": error=2, No 
> such file or directory
>   at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
>   at com.cloud.utils.script.Script.execute(Script.java:183)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/server/ConfigurationServerImpl.java 596387f 
> 
> Diff: https://reviews.apache.org/r/10437/diff/
> 
> 
> Testing
> ---
> 
> Rebuilt rpm and tested management server startup -- success
> 
> 
> Thanks,
> 
> Phong Nguyen
> 
>



Re: Review Request: Fix command for executing injectkeys.sh

2013-04-15 Thread Chip Childers
Do I need a patch for this in 4.1?

On Mon, Apr 15, 2013 at 05:02:07PM +, Chiradeep Vittal wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10437/#review19193
> ---
> 
> Ship it!
> 
> 
> Ship It!
> 
> - Chiradeep Vittal
> 
> 
> On April 12, 2013, 9:11 p.m., Phong Nguyen wrote:
> > 
> > ---
> > This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/10437/
> > ---
> > 
> > (Updated April 12, 2013, 9:11 p.m.)
> > 
> > 
> > Review request for cloudstack and Hugo Trippaers.
> > 
> > 
> > Description
> > ---
> > 
> > Management server unable to start (from rpm) due to invalid call to Script 
> > for injectkeys.sh. 
> > 
> > 2013-04-12 15:02:13,496 WARN  [cloud.server.ConfigurationServerImpl] 
> > (Timer-1:null) Failed to inject generated public key into systemvm iso 
> > java.io.IOException: Cannot run program "/bin/bash 
> > /usr/share/cloudstack-common/scripts/vm/systemvm/injectkeys.sh": error=2, 
> > No such file or directory
> > at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
> > at com.cloud.utils.script.Script.execute(Script.java:183)
> > 
> > 
> > Diffs
> > -
> > 
> >   server/src/com/cloud/server/ConfigurationServerImpl.java 596387f 
> > 
> > Diff: https://reviews.apache.org/r/10437/diff/
> > 
> > 
> > Testing
> > ---
> > 
> > Rebuilt rpm and tested management server startup -- success
> > 
> > 
> > Thanks,
> > 
> > Phong Nguyen
> > 
> >
> 


Re: Firewall and LB network elements in inline mode

2013-04-15 Thread Chiradeep Vittal
Isn't this discussed here:
https://cwiki.apache.org/confluence/x/ggjVAQ

On 4/15/13 6:51 AM, "Koushik Das"  wrote:

>The network orchestration component currently supports elements in a
>side-by-side mode. Compatible network elements can be used in this mode
>to offer various network services. There are also scenarios where an
>inline mode would be desirable. For e.g. firewall followed by LB. Wanted
>to check out on the pros/cons of having support for network elements in
>inline mode in the core component itself.
>
>-Koushik



Re: [DISCUSS] Getting rid of premium

2013-04-15 Thread Chiradeep Vittal


On 4/15/13 9:01 AM, "Chip Childers"  wrote:

>On Mon, Apr 15, 2013 at 03:53:59PM +, Hugo Trippaers wrote:
>> Heya,
>> 
>> As we don't have a premium distribution anymore I would like to get rid
>>of the premium configuration value. At the moment it seems to be
>>unintentionally used to distinguish between the nonss and oss build so I
>>think it can go. It will likely only confuse people that are new to the
>>project and doesn't bring anything usefull so it seems.
>> 
>> I'm willing to trawl through the code and remove any references to
>>premium and replace it with another flag where needed (for example a
>>flag to indicate vmware support is available).
>> 
>> Any thoughts about this?
>> 
>> Cheers,
>> 
>> Hugo
>>
>
>+1 - Just be sure to create a Jira "Improvement" to track it please!

+1. 



Re: [DISCUSS] Getting rid of premium

2013-04-15 Thread Kelven Yang


On 4/15/13 8:53 AM, "Hugo Trippaers"  wrote:

>Heya,
>
>As we don't have a premium distribution anymore I would like to get rid
>of the premium configuration value. At the moment it seems to be
>unintentionally used to distinguish between the nonss and oss build so I
>think it can go. It will likely only confuse people that are new to the
>project and doesn't bring anything usefull so it seems.
>
>I'm willing to trawl through the code and remove any references to
>premium and replace it with another flag where needed (for example a flag
>to indicate vmware support is available).
>
>Any thoughts about this?
>
>Cheers,
>
>Hugo

+1


>



Re: [1/2] Adding issues fixed in 4.1.0

2013-04-15 Thread Joe Brockmeier
On Mon, Apr 15, 2013, at 09:36 AM, Milamber wrote:
> Hello jzb,
> 
> A lot of description seems have lost the last characters:

Thanks. I'll try to address that... (p.s., you might want to trim
replies - no need to send the entire list!)

Best,

jzb
-- 
Joe Brockmeier
j...@zonker.net
Twitter: @jzb
http://www.dissociatedpress.net/


RE: [DISCUSS] Getting rid of premium

2013-04-15 Thread Rajesh Battala


> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: Monday, April 15, 2013 10:43 PM
> To: dev@cloudstack.apache.org
> Subject: Re: [DISCUSS] Getting rid of premium
> 
> 
> 
> On 4/15/13 8:53 AM, "Hugo Trippaers" 
> wrote:
> 
> >Heya,
> >
> >As we don't have a premium distribution anymore I would like to get rid
> >of the premium configuration value. At the moment it seems to be
> >unintentionally used to distinguish between the nonss and oss build so
> >I think it can go. It will likely only confuse people that are new to
> >the project and doesn't bring anything usefull so it seems.
> >
> >I'm willing to trawl through the code and remove any references to
> >premium and replace it with another flag where needed (for example a
> >flag to indicate vmware support is available).
> >
> >Any thoughts about this?
> >
> >Cheers,
> >
> >Hugo
> 
> +1
> 
> 
> >
 
+1


Add/Remove Network for VM : VMware Support

2013-04-15 Thread Saksham Srivastava
Following the discussion on bug: 
https://issues.apache.org/jira/browse/CLOUDSTACK-645 , there needs to be some 
work done for enhancing  VMware support for the feature Add/Remove Network on 
VM.

I will be updating the FS soon and will send out for review.

Thanks,
Saksham


Review Request: Add LXC to hypervisor.list configuration value on fresh install

2013-04-15 Thread Phong Nguyen

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

Review request for cloudstack and edison su.


Description
---

Add LXC to hypervisor.list configuration value on fresh install


Diffs
-

  server/src/com/cloud/configuration/Config.java e3e3053 

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


Testing
---

Built rpm and tested fresh install, verified configuration for hypervisor.list 
contained LXC.


Thanks,

Phong Nguyen



Re: Issue launching VM on VMware

2013-04-15 Thread Kelven Yang
>From the log, you seem to use a offering that directs to create volume in
local storage and there is no such storage available in your setup

Kelven

On 4/13/13 4:14 PM, "Mike Tutkowski"  wrote:

>That's weird...I have three local storages (one from DevCloud2 by default,
>one from one ESXi host, and one from another ESXi host).  I also have
>several shared storages (iSCSI-based datastores that were configured in CS
>as VMFS-based primary storages).  I tagged the shared primary storage I
>wanted to use, referenced that storage tag from a compute offering, and
>executed the compute offering.
>
>
>On Sat, Apr 13, 2013 at 12:30 PM, Ahmad Emneina 
>wrote:
>
>> looks as though your zone is set to use shared storage, and youre only
>> providing local storage.
>>
>>1. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
>>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) Checking if
>>storage pool is suitable, name: devcloud Local Storage ,poolId: 200
>>2. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
>>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) Is
>>localStorageAllocationNeeded? false
>>3. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
>>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) Is storage
>>pool
>>shared? false
>>4. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
>>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) StoragePool is
>> not
>>of correct type, skipping this pool
>>
>>
>>
>> On Fri, Apr 12, 2013 at 10:19 PM, Mike Tutkowski <
>> mike.tutkow...@solidfire.com> wrote:
>>
>> > Hi,
>> >
>> > Thanks for the comments.
>> >
>> > In my case, I am just running a Basic Zone that was originally set up
>> with
>> > DevCloud2.  I added a Pod and a Cluster with two ESXi Hosts in the
>> Cluster.
>> >
>> > I am able to create iSCSI-based VMFS Datastores and Primary Storages
>>in
>> CS
>> > from them.
>> >
>> > However, when I try to execute a Compute Offering that is storage
>>tagged
>> to
>> > only use Primary Storage that is based on one of my VMFS Datastores, I
>> get
>> > a failure message.
>> >
>> > I've attached the log here:
>> >
>> > http://paste.cloudstack.org/rst0/
>> >
>> > Thanks!!
>> >
>> >
>> > On Fri, Apr 12, 2013 at 2:49 PM, Musayev, Ilya 
>> wrote:
>> >
>> > > Mike,
>> > >
>> > > I've done many vmware install, I don't believe your issue is with VC
>> > > running on the same hypervisor you are trying to manage - since one
>>of
>> my
>> > > setups is doing just that.
>> > >
>> > > Also, while it maybe stated somewhere that you should dedicate this
>> > > hypervisor to CS, CS does not know anything about other VMs it did
>>not
>> > > create. It maybe a best practice to dedicate your hypervisors,
>>however,
>> > its
>> > > not a must requirement (please correct me if I'm wrong).
>> > >
>> > > Even when you place ESXi into maintenance mode in CS, it does not
>> really
>> > > take hypervisor into maintenance mode, it just evacuvates its VMs to
>> > other
>> > > hypervisors.
>> > >
>> > > Typical reasons for your problem:
>> > > 1) you've changed "Management Network" port group
>> > > 2) you are not using default vswitch0
>> > > (both of the above can be changed in settings)
>> > > 3) you attempting to use DVS, not yet supported in 4.1, supported in
>> 4.2
>> > >  - I will release my version of ACS of 4.1 with DVS integrated as
>>soon
>> as
>> > > 4.1 goes out.
>> > >
>> > > If none of the above are true, please submit a log through
>> > > paste.cloudstack.org, the exception you've posted show the end
>>result
>> > > error - but not the original cause.
>> > >
>> > > Regards
>> > > ilya
>> > >
>> > > > -Original Message-
>> > > > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
>> > > > Sent: Friday, April 12, 2013 3:18 PM
>> > > > To: dev@cloudstack.apache.org
>> > > > Cc: cloudstack-...@incubator.apache.org
>> > > > Subject: Re: Issue launching VM on VMware
>> > > >
>> > > > So, could this be the problem:
>> > > >
>> > > > In the interest of efficiency, I am running a virtual appliance
>>with
>> > > vCenter
>> > > > inside of it on one of the two ESXi hosts I'm using in CloudStack.
>> > > >
>> > > > Technically we're not supposed to have any VMs running that we're
>>not
>> > > > started by CloudStack, right?
>> > > >
>> > > > Could this be what's causing my troubles?
>> > > >
>> > > >
>> > > > On Fri, Apr 12, 2013 at 12:56 PM, Mike Tutkowski <
>> > > > mike.tutkow...@solidfire.com> wrote:
>> > > >
>> > > > > Looks like it might be because no virtual router is running?
>> > > > >
>> > > > > 2013-04-12 12:50:27,994 DEBUG
>> > > > > [network.router.VirtualNetworkApplianceManagerImpl]
>> > > > > (RouterStatusMonitor-1:null) Found 0 routers to update status.
>> > > > > 2013-04-12 12:50:27,995 DEBUG
>> > > > > [network.router.VirtualNetworkApplianceManagerImpl]
>> > > > > (RouterStatusMonitor-1:null) Found 0 networks to update RvR
>>status.
>> > > > > 2013-04-12 12:50:28,026 DEBUG
>> > > > > [network.router.V

RE: {QA}Not able to login with default admin credentials in latest master build

2013-04-15 Thread Chandan Purushothama
Suresh,

The root cause for the  bug that you logged is 
https://issues.apache.org/jira/browse/CLOUDSTACK-2013 . Hence closing your bug 
as the duplicate of this bug,

Thank you,
Chandan.

-Original Message-
From: Suresh Sadhu [mailto:suresh.sa...@citrix.com] 
Sent: Monday, April 15, 2013 1:31 AM
To: dev@cloudstack.apache.org
Subject: {QA}Not able to login with default admin credentials in latest master 
build

HI All,

Seen a problem with latest maser build ,i.e not able to login with default 
admin credentials.it looks like by default we are putting admin user state in 
disable mode. Even after updating the state to enable mode in db ,still see the 
problem. We even updated password with MD5 hash  but still see the problem i.e  
" Not abkle to login with admin credentials"

* 
CLOUDSTACK-2028 not able 
to login cloudstack using default admin credentials


Regards
Sadhu



Re: Review Request: Fix command for executing injectkeys.sh

2013-04-15 Thread Phong Nguyen
No, only need for master. I just checked 4.1 and it does not have the
changes from commit 5cd3608.


On Mon, Apr 15, 2013 at 1:07 PM, Chip Childers wrote:

> Do I need a patch for this in 4.1?
>
> On Mon, Apr 15, 2013 at 05:02:07PM +, Chiradeep Vittal wrote:
> >
> > ---
> > This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/10437/#review19193
> > ---
> >
> > Ship it!
> >
> >
> > Ship It!
> >
> > - Chiradeep Vittal
> >
> >
> > On April 12, 2013, 9:11 p.m., Phong Nguyen wrote:
> > >
> > > ---
> > > This is an automatically generated e-mail. To reply, visit:
> > > https://reviews.apache.org/r/10437/
> > > ---
> > >
> > > (Updated April 12, 2013, 9:11 p.m.)
> > >
> > >
> > > Review request for cloudstack and Hugo Trippaers.
> > >
> > >
> > > Description
> > > ---
> > >
> > > Management server unable to start (from rpm) due to invalid call to
> Script for injectkeys.sh.
> > >
> > > 2013-04-12 15:02:13,496 WARN  [cloud.server.ConfigurationServerImpl]
> (Timer-1:null) Failed to inject generated public key into systemvm iso
> java.io.IOException: Cannot run program "/bin/bash
> /usr/share/cloudstack-common/scripts/vm/systemvm/injectkeys.sh": error=2,
> No such file or directory
> > > at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
> > > at com.cloud.utils.script.Script.execute(Script.java:183)
> > >
> > >
> > > Diffs
> > > -
> > >
> > >   server/src/com/cloud/server/ConfigurationServerImpl.java 596387f
> > >
> > > Diff: https://reviews.apache.org/r/10437/diff/
> > >
> > >
> > > Testing
> > > ---
> > >
> > > Rebuilt rpm and tested management server startup -- success
> > >
> > >
> > > Thanks,
> > >
> > > Phong Nguyen
> > >
> > >
> >
>


Re: Review Request: Fix command for executing injectkeys.sh

2013-04-15 Thread Chiradeep Vittal
Doesn't look like it

On 4/15/13 10:07 AM, "Chip Childers"  wrote:

>Do I need a patch for this in 4.1?
>
>On Mon, Apr 15, 2013 at 05:02:07PM +, Chiradeep Vittal wrote:
>> 
>> ---
>> This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/10437/#review19193
>> ---
>> 
>> Ship it!
>> 
>> 
>> Ship It!
>> 
>> - Chiradeep Vittal
>> 
>> 
>> On April 12, 2013, 9:11 p.m., Phong Nguyen wrote:
>> > 
>> > ---
>> > This is an automatically generated e-mail. To reply, visit:
>> > https://reviews.apache.org/r/10437/
>> > ---
>> > 
>> > (Updated April 12, 2013, 9:11 p.m.)
>> > 
>> > 
>> > Review request for cloudstack and Hugo Trippaers.
>> > 
>> > 
>> > Description
>> > ---
>> > 
>> > Management server unable to start (from rpm) due to invalid call to
>>Script for injectkeys.sh.
>> > 
>> > 2013-04-12 15:02:13,496 WARN  [cloud.server.ConfigurationServerImpl]
>>(Timer-1:null) Failed to inject generated public key into systemvm iso
>>java.io.IOException: Cannot run program "/bin/bash
>>/usr/share/cloudstack-common/scripts/vm/systemvm/injectkeys.sh":
>>error=2, No such file or directory
>> >at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
>> >at com.cloud.utils.script.Script.execute(Script.java:183)
>> > 
>> > 
>> > Diffs
>> > -
>> > 
>> >   server/src/com/cloud/server/ConfigurationServerImpl.java 596387f
>> > 
>> > Diff: https://reviews.apache.org/r/10437/diff/
>> > 
>> > 
>> > Testing
>> > ---
>> > 
>> > Rebuilt rpm and tested management server startup -- success
>> > 
>> > 
>> > Thanks,
>> > 
>> > Phong Nguyen
>> > 
>> >
>> 



Re: Review Request: Fix command for executing injectkeys.sh

2013-04-15 Thread Hugo Trippaers

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

Ship it!


Ship It!

- Hugo Trippaers


On April 12, 2013, 9:11 p.m., Phong Nguyen wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10437/
> ---
> 
> (Updated April 12, 2013, 9:11 p.m.)
> 
> 
> Review request for cloudstack and Hugo Trippaers.
> 
> 
> Description
> ---
> 
> Management server unable to start (from rpm) due to invalid call to Script 
> for injectkeys.sh. 
> 
> 2013-04-12 15:02:13,496 WARN  [cloud.server.ConfigurationServerImpl] 
> (Timer-1:null) Failed to inject generated public key into systemvm iso 
> java.io.IOException: Cannot run program "/bin/bash 
> /usr/share/cloudstack-common/scripts/vm/systemvm/injectkeys.sh": error=2, No 
> such file or directory
>   at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
>   at com.cloud.utils.script.Script.execute(Script.java:183)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/server/ConfigurationServerImpl.java 596387f 
> 
> Diff: https://reviews.apache.org/r/10437/diff/
> 
> 
> Testing
> ---
> 
> Rebuilt rpm and tested management server startup -- success
> 
> 
> Thanks,
> 
> Phong Nguyen
> 
>



Re: Review Request: Fix command for executing injectkeys.sh

2013-04-15 Thread Hugo Trippaers


> On April 15, 2013, 6:43 p.m., Hugo Trippaers wrote:
> > Ship It!

Commit on master 794e653c1325f6ba78b700f62e75c5a7b502c4eb

Thanks for fixing my bug ;-)


- Hugo


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


On April 12, 2013, 9:11 p.m., Phong Nguyen wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10437/
> ---
> 
> (Updated April 12, 2013, 9:11 p.m.)
> 
> 
> Review request for cloudstack and Hugo Trippaers.
> 
> 
> Description
> ---
> 
> Management server unable to start (from rpm) due to invalid call to Script 
> for injectkeys.sh. 
> 
> 2013-04-12 15:02:13,496 WARN  [cloud.server.ConfigurationServerImpl] 
> (Timer-1:null) Failed to inject generated public key into systemvm iso 
> java.io.IOException: Cannot run program "/bin/bash 
> /usr/share/cloudstack-common/scripts/vm/systemvm/injectkeys.sh": error=2, No 
> such file or directory
>   at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
>   at com.cloud.utils.script.Script.execute(Script.java:183)
> 
> 
> Diffs
> -
> 
>   server/src/com/cloud/server/ConfigurationServerImpl.java 596387f 
> 
> Diff: https://reviews.apache.org/r/10437/diff/
> 
> 
> Testing
> ---
> 
> Rebuilt rpm and tested management server startup -- success
> 
> 
> Thanks,
> 
> Phong Nguyen
> 
>



Re: [DOCS] Documentation focused committers, and review processes

2013-04-15 Thread Jessica Tomechak
On Mon, Apr 15, 2013 at 7:32 AM, Rajesh Battala
wrote:

>
>
> > -Original Message-
> > From: Chip Childers [mailto:chip.child...@sungard.com]
> > Sent: Monday, April 15, 2013 7:47 PM
> > To: dev@cloudstack.apache.org
> > Subject: Re: [DOCS] Documentation focused committers, and review
> processes
> >
> > On Sun, Apr 14, 2013 at 06:05:48PM -0500, Joe Brockmeier wrote:
> > > On Sun, Apr 14, 2013, at 02:19 AM, Radhika Puthiyetath wrote:
> > > > Any penalties for people who BLOCK documentation development ?
> > >
> > > Are there rewards for the folks who aren't blocking documentation
> > > development? It's not like we can dock their volunteer pay. ;-)
> > >
> > > In all seriousness - I'm sure that it's frustrating to do the work and
> > > wait on feedback that is slow in coming. But every project suffers
> > > from more work than hands to do it. I don't think we should be looking
> > > to penalize or shame anybody, we just need to find ways to help people
> > > do their work more efficiently.
> > >
> > > We *could* revert features where the developer has not reviewed
> > > documentation, but who does that help? Assuming a feature is complete
> > > and isn't buggy, is it better for the user not to have a feature
> > > because someone didn't have enough time to review docs? Probably not.
> > >
> > > In this case, I think Chip's proposal is the right one: docs folks
> > > should do the docs to the best of their ability, commit, and raise the
> > > new docs to the attention of the developer(s) specifically responsible
> > > for the feature and the dev@ list in general. (And perhaps even the
> > > user@ list.)
>

The suggestion to get the users list involved is a very good one.


> >
> > +1 to what Joe said.  Do the best you can with the information you have,
> > and harass developers until you get a response.
>

I'm sure this is exactly what is already being done. I believe the problem
Radhika raises is that sometimes, there is no response from the developers,
no matter how many requests are made or in what imaginative fasion they are
issued to try to get attention.


> >
> > -chip
>
> +1 for joe and chip


Please pardon my ignorance, but I have a couple of questions. When
developers commit code to the repo, it's required to go through a testing
phase before it's included in a release, is that correct? Or do we put out
some code that's half-baked, and if so, does it have maybe a comment at the
top like "Work in progress"?

I'm just trying to get at whether the expectations for code and docs are
the same, and even whether they ought to be the same, or not. I am fine
with Chip's and Joe's suggestions that the doc committers go ahead and
commit whatever documentation we have to the repo, even as we're still
waiting for an accuracy review (analogous to the testing phase for
code).That's what I already do anyhow.

I'm just wondering how we want to handle the case where review never
happens, or not in time for a scheduled release. I agree with Joe that
there's no point reverting an entire feature just because its docs were not
reviewed, but I'd be in favor of either pulling out the docs or at least
stamping them with some kind of caveat like "Preliminary draft, accuracy
not guaranteed." What do folks think about that?

Jessica T.


Re: Issue launching VM on VMware

2013-04-15 Thread Mike Tutkowski
Interesting...when I review one of my Compute Offerings, it says it is of
Storage Type "shared" and has a Storage Tag of "SF_iSCSI_1" and that
Storage Tag is only used by a Primary Storage of the same name that is
based on an iSCSI-backed VMFS datastore.


On Mon, Apr 15, 2013 at 11:53 AM, Kelven Yang wrote:

> From the log, you seem to use a offering that directs to create volume in
> local storage and there is no such storage available in your setup
>
> Kelven
>
> On 4/13/13 4:14 PM, "Mike Tutkowski"  wrote:
>
> >That's weird...I have three local storages (one from DevCloud2 by default,
> >one from one ESXi host, and one from another ESXi host).  I also have
> >several shared storages (iSCSI-based datastores that were configured in CS
> >as VMFS-based primary storages).  I tagged the shared primary storage I
> >wanted to use, referenced that storage tag from a compute offering, and
> >executed the compute offering.
> >
> >
> >On Sat, Apr 13, 2013 at 12:30 PM, Ahmad Emneina 
> >wrote:
> >
> >> looks as though your zone is set to use shared storage, and youre only
> >> providing local storage.
> >>
> >>1. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
> >>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) Checking if
> >>storage pool is suitable, name: devcloud Local Storage ,poolId: 200
> >>2. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
> >>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) Is
> >>localStorageAllocationNeeded? false
> >>3. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
> >>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) Is storage
> >>pool
> >>shared? false
> >>4. 2013-04-12 11:35:20,292 DEBUG [storage.allocator.
> >>AbstractStoragePoolAllocator] (Job-Executor-11:job-11) StoragePool is
> >> not
> >>of correct type, skipping this pool
> >>
> >>
> >>
> >> On Fri, Apr 12, 2013 at 10:19 PM, Mike Tutkowski <
> >> mike.tutkow...@solidfire.com> wrote:
> >>
> >> > Hi,
> >> >
> >> > Thanks for the comments.
> >> >
> >> > In my case, I am just running a Basic Zone that was originally set up
> >> with
> >> > DevCloud2.  I added a Pod and a Cluster with two ESXi Hosts in the
> >> Cluster.
> >> >
> >> > I am able to create iSCSI-based VMFS Datastores and Primary Storages
> >>in
> >> CS
> >> > from them.
> >> >
> >> > However, when I try to execute a Compute Offering that is storage
> >>tagged
> >> to
> >> > only use Primary Storage that is based on one of my VMFS Datastores, I
> >> get
> >> > a failure message.
> >> >
> >> > I've attached the log here:
> >> >
> >> > http://paste.cloudstack.org/rst0/
> >> >
> >> > Thanks!!
> >> >
> >> >
> >> > On Fri, Apr 12, 2013 at 2:49 PM, Musayev, Ilya 
> >> wrote:
> >> >
> >> > > Mike,
> >> > >
> >> > > I've done many vmware install, I don't believe your issue is with VC
> >> > > running on the same hypervisor you are trying to manage - since one
> >>of
> >> my
> >> > > setups is doing just that.
> >> > >
> >> > > Also, while it maybe stated somewhere that you should dedicate this
> >> > > hypervisor to CS, CS does not know anything about other VMs it did
> >>not
> >> > > create. It maybe a best practice to dedicate your hypervisors,
> >>however,
> >> > its
> >> > > not a must requirement (please correct me if I'm wrong).
> >> > >
> >> > > Even when you place ESXi into maintenance mode in CS, it does not
> >> really
> >> > > take hypervisor into maintenance mode, it just evacuvates its VMs to
> >> > other
> >> > > hypervisors.
> >> > >
> >> > > Typical reasons for your problem:
> >> > > 1) you've changed "Management Network" port group
> >> > > 2) you are not using default vswitch0
> >> > > (both of the above can be changed in settings)
> >> > > 3) you attempting to use DVS, not yet supported in 4.1, supported in
> >> 4.2
> >> > >  - I will release my version of ACS of 4.1 with DVS integrated as
> >>soon
> >> as
> >> > > 4.1 goes out.
> >> > >
> >> > > If none of the above are true, please submit a log through
> >> > > paste.cloudstack.org, the exception you've posted show the end
> >>result
> >> > > error - but not the original cause.
> >> > >
> >> > > Regards
> >> > > ilya
> >> > >
> >> > > > -Original Message-
> >> > > > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
> >> > > > Sent: Friday, April 12, 2013 3:18 PM
> >> > > > To: dev@cloudstack.apache.org
> >> > > > Cc: cloudstack-...@incubator.apache.org
> >> > > > Subject: Re: Issue launching VM on VMware
> >> > > >
> >> > > > So, could this be the problem:
> >> > > >
> >> > > > In the interest of efficiency, I am running a virtual appliance
> >>with
> >> > > vCenter
> >> > > > inside of it on one of the two ESXi hosts I'm using in CloudStack.
> >> > > >
> >> > > > Technically we're not supposed to have any VMs running that we're
> >>not
> >> > > > started by CloudStack, right?
> >> > > >
> >> > > > Could this be what's causing my troubles?
> >> > > >
> >> > > >
> >> > > > On Fri, Apr 12, 2013 at 12:56 PM, Mike Tutko

Re: [DOC] Opening vncextras firewall ports in ESXi 5.1

2013-04-15 Thread Jessica Tomechak
On Mon, Apr 15, 2013 at 9:58 AM, Jason Cadmus  wrote:

> Thanks Joe
>
> I have been talking Ilya and wanted to do a bit more testing prior to
> posting my documentation. I did start a page at the following link although
> it can be that can be deleted. "My confluence permissions do not allow me
> to delete the page I added."
>
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Configure+vncextras+(page+can+be+deleted)
>
> Thanks
> JC
>
>
>
Jason, I deleted that page for you. Thanks very much for your help on
documentation!

Jessica T.


>
>
> On Mon, Apr 15, 2013 at 12:52 PM, Joe Brockmeier  wrote:
>
> > On Mon, Apr 15, 2013, at 09:58 AM, Jason Cadmus wrote:
> > > What page on the wiki should I create my documentation so I  can
> create a
> > > doc task to get it ported?
> >
> > Good question. I started a wiki page for this generally:
> >
> > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Docs+in+Process
> >
> > And for this specific feature:
> >
> >
> >
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Opening+Firewall+Ports+in+ESXi+5.1
> >
> > Once you're happy with the docs as written, just open a Jira ticket
> > under the Docs component and send a note to dev@ that you'd like someone
> > to convert to Publican. Please let me know if you need anything else,
> > and thanks!
> >
> > Joe
> >
> > Best,
> >
> > jzb
> > --
> > Joe Brockmeier
> > j...@zonker.net
> > Twitter: @jzb
> > http://www.dissociatedpress.net/
> >
>
>
>
> --
> Thanks,
> JC
>


Re: [DOCS] Documentation focused committers, and review processes

2013-04-15 Thread Joe Brockmeier
On Mon, Apr 15, 2013, at 02:31 PM, Jessica Tomechak wrote:
> I'm just wondering how we want to handle the case where review never
> happens, or not in time for a scheduled release. I agree with Joe that
> there's no point reverting an entire feature just because its docs were
> not reviewed, but I'd be in favor of either pulling out the docs or at least
> stamping them with some kind of caveat like "Preliminary draft, accuracy
> not guaranteed." What do folks think about that?

I'm concerned that would be putting a lot of responsibility on
documentation contributors to continually follow up and verify whether
something was reviewed. 

If we wanted to, I suppose we could include "draft" on sections that are
new and need review, and ask the reviewer to remove that. Not sure that
is desirable, though - If there was a vote on something like that, I'd
probably +0 it: I don't feel strongly enough to -1 it, but it doesn't
seem necessary to me either. 

It's my hope that folks who are developing features care enough to
eventually find time to review the docs written for their features and
any pain points we're currently experiencing are largely related to
finding a good rhythm for working on time-based releases. 

Best,

jzb
-- 
Joe Brockmeier
j...@zonker.net
Twitter: @jzb
http://www.dissociatedpress.net/


RE: [DOCS] Documentation focused committers, and review processes

2013-04-15 Thread Animesh Chaturvedi


> -Original Message-
> From: Chip Childers [mailto:chip.child...@sungard.com]
> Sent: Thursday, April 11, 2013 6:05 AM
> To: dev@cloudstack.apache.org
> Cc: Radhika Puthiyetath
> Subject: [DOCS] Documentation focused committers, and review processes
> 
> Hi all,
> 
> I've observed Radhika struggling to get reviews of her feature docs, and
> having to work to provide patch after patch for reviewboard submissions.
> 
> I have a proposal (which I think Joe hinted at in another thread):
> 
> Doc writers with commit rights shouldn't use reviewboard to elicit reviews
> from feature developers.  They have access to the repo directly.
> Instead, commit away anything that would have previously been posted to
> reviewboard.  There's a reason that we try to move folks from contributor to
> committer on the project, right? ;-)
[Animesh>] +1
> 
> Radhika, perhaps you just commit what you have for review if you feel that
> it's close enough to publish.  Point people to jenkins.cs.o jobs to review the
> content if required.  Questions that block even the initial doc authoring
> process obviously continue to be raised on this list.
> 
> Would this make life easier for you Radhika?
> 
> -chip


RE: How to set the default network when I specify IP use "iptonetworklist" parameter when deploy a vm?

2013-04-15 Thread Sangeetha Hariharan
When using iptonetworklist  when deploying Vms ,first entry in the 
iptonetworklist ( iptonetworklist[0].networkid) becomes the default network.

-Thanks
Sangeetha

-Original Message-
From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com] 
Sent: Monday, April 15, 2013 9:32 AM
To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
Subject: Re: How to set the default network when I specify IP use 
"iptonetworklist" parameter when deploy a vm?


NetworkIds parameter has to be specify regardless of iptonetworklist parameter 
presence. So you have to define both parameters, and the first network from the 
networkIds list will be used as a default.

-Alena.

On 4/15/13 7:44 AM, "sx chen"  wrote:

>Hi,
>   as far as I know, I can specify the default network by put it at the 
>first when use "*networkids*" paramenter when deploy a vm.
>   but if I use "iptonetworklist" parameter specify the IP address,I 
>find I can't specify the default network.I've test put it at the first 
>and put it at the last.
>   So if I want to specify the default network as well as the IP 
>address,what should I do?
>appreciate your help!
>




[BUG][ACS41] vm_data updates router VM .htaccess file with duplicate RewriteRules

2013-04-15 Thread Musayev, Ilya
++Chiradeep, as I see your name referenced in the script.

Noticed an odd behavior on ACS41 and vSphere, where each time you deploy a VM, 
on router vm under /var/www/html/latest/ the .htaccess file gets duplicate set 
of rewrite rules appended.

Vm_data is called each time VM is deployed, and each time redundant 
RewriteRules are added to .htaccess file - per each run/vm demployment.

Before I begin troubleshooting and fixing this issue (I can help rewrite the 
bash logic for checking if entry exists), could someone please give me a 
context as to how and when vm_data file is used.


Also, for some reason metadata/meta-data Rewrite Rule is missing, I would like 
to understand where the call is made and how $folder variable is set.

Thanks
ilya


Re: [BUG][ACS41] vm_data updates router VM .htaccess file with duplicate RewriteRules

2013-04-15 Thread Chiradeep Vittal
I think if you run
git blame  ./patches/systemvm/debian/config/root/userdata.sh

There's several changes done to this particular piece of code.

The vm_data file is used for AWS-style "metadata" and user data
http://s.apache.org/amX
http://s.apache.org/PJH


From: , Ilya mailto:imusa...@webmd.net>>
Date: Monday, April 15, 2013 3:47 PM
To: "dev@cloudstack.apache.org" 
mailto:dev@cloudstack.apache.org>>, Chiradeep Vittal 
mailto:chiradeep.vit...@citrix.com>>
Subject: [BUG][ACS41] vm_data updates router VM .htaccess file with duplicate 
RewriteRules

++Chiradeep, as I see your name referenced in the script.

Noticed an odd behavior on ACS41 and vSphere, where each time you deploy a VM, 
on router vm under /var/www/html/latest/ the .htaccess file gets duplicate set 
of rewrite rules appended.

Vm_data is called each time VM is deployed, and each time redundant 
RewriteRules are added to .htaccess file – per each run/vm demployment.

Before I begin troubleshooting and fixing this issue (I can help rewrite the 
bash logic for checking if entry exists), could someone please give me a 
context as to how and when vm_data file is used.


Also, for some reason metadata/meta-data Rewrite Rule is missing, I would like 
to understand where the call is made and how $folder variable is set.

Thanks
ilya


RE: deployDataCenter.py doesn't work for me on master

2013-04-15 Thread Anthony Xu
>>The XenServer (6.0.2) I am using already came bundled with vhd-util, I 
>>did not have to download it. I have not read the detail of

Hi Soheil,

We added one function in vhd-util, which allows vhd-util specifiy parent when 
do coalesce, basically you can coalesce a VHD file to other VHD file. 
CS uses this vhd-util function in API creating template/volume from snapshot.  
These APIs might not work with bundled vhd-util in XenServer.


Anthony



-Original Message-
From: Soheil Eizadi [mailto:seiz...@infoblox.com] 
Sent: Saturday, April 13, 2013 11:05 PM
To: dev@cloudstack.apache.org
Subject: Re: deployDataCenter.py doesn't work for me on master

Downloaded the vhd-util to my environment and had the same problem. I Debugged 
this further by running the copy_vhd_from_secondarystorage.sh on the XenServer. 
It looks like the copy_vhd_from_secondarystorage.sh is looking for vhd-util in 
/opt/xensource/bin/ rather than /usr/bin/

I looked at this further and looks like what is built in the client/target 
directory is not consistent with what is in the ./scripts/vm in my tree.
There are two versions of copy_vhd_from_secondarystorage.sh.

I removed the client/target directory and rebuilt, but had the same files 
appear again. I am not sure how they are getting pulled in to the client/target 
directory. Any ideas why my build is broken and how to proceed to fix this?

For now I patched this on my XenServer and was able to get ssvm and cproxyvm 
running and brought up my CloudStack zone completely.
-Soheil


Administrators-MacBook-Pro-7:cloudstack seizadi$ find . -name 
copy_vhd_from_secondarystorage.sh 
./client/target/cloud-client-ui-4.2.0-SNAPSHOT/WEB-INF/classes/scripts/vm/h
ypervisor/xenserver/copy_vhd_from_secondarystorage.sh
./client/target/cloud-client-ui-4.2.0-SNAPSHOT/WEB-INF/classes/scripts/vm/h
ypervisor/xenserver/xcposs/copy_vhd_from_secondarystorage.sh
./client/target/generated-webapp/WEB-INF/classes/scripts/vm/hypervisor/xens
erver/copy_vhd_from_secondarystorage.sh
./client/target/generated-webapp/WEB-INF/classes/scripts/vm/hypervisor/xens
erver/xcposs/copy_vhd_from_secondarystorage.sh
./scripts/vm/hypervisor/xenserver/copy_vhd_from_secondarystorage.sh
./scripts/vm/hypervisor/xenserver/xcposs/copy_vhd_from_secondarystorage.sh


Administrators-MacBook-Pro-7:cloudstack seizadi$ find . -name 
copy_vhd_from_secondarystorage.sh -exec diff 
./client/target/generated-webapp/WEB-INF/classes/scripts/vm/hypervisor/xens
erver/copy_vhd_from_secondarystorage.sh {} \;
92c92
< VHDUTIL="/opt/xensource/bin/vhd-util"
---
> VHDUTIL="/usr/bin/vhd-util"
113c113
< if [ $type != "nfs" -a $type != "ext" ]; then
---
> if [ $type != "nfs" -a $type != "ext" -a $type != "file" ]; then
143c143
<   desvhd=/var/run/sr-mount/$sruuid/$uuid.vhd
---
>   desvhd=/run/sr-mount/$sruuid/$uuid.vhd
160a161,166
> elif [ $type == "file" ]; then
>   pbd=`xe sr-param-list uuid=$sruuid |grep PBDs | awk '{print $3}'`
>   path=`xe pbd-param-list uuid=$pbd |grep device-config |awk '{print 
>$4}'`
>   desvhd=$path/$uuid.vhd
>   copyvhd $desvhd $vhdfile 0 $type
> 
92c92
< VHDUTIL="/opt/xensource/bin/vhd-util"
---
> VHDUTIL="/usr/bin/vhd-util"
113c113
< if [ $type != "nfs" -a $type != "ext" ]; then
---
> if [ $type != "nfs" -a $type != "ext" -a $type != "file" ]; then
143c143
<   desvhd=/var/run/sr-mount/$sruuid/$uuid.vhd
---
>   desvhd=/run/sr-mount/$sruuid/$uuid.vhd
160a161,166
> elif [ $type == "file" ]; then
>   pbd=`xe sr-param-list uuid=$sruuid |grep PBDs | awk '{print $3}'`
>   path=`xe pbd-param-list uuid=$pbd |grep device-config |awk '{print 
>$4}'`
>   desvhd=$path/$uuid.vhd
>   copyvhd $desvhd $vhdfile 0 $type
> 
Š.


[root@xenserver-devcloud /]# copy_vhd_from_secondarystorage.sh 
172.16.197.134:/opt/storage/secondary/template/tmpl/1/1/
7e934fee-96b5-b297-2c25-b20a14139fd9 DEVTEST
/opt/xensource/bin/copy_vhd_from_secondarystorage.sh: line 133:
/opt/xensource/bin/vhd-util: No such file or directory
Error: Failed to parse field 'virtual-size': expecting an integer (possibly 
with suffix) 9#can not create vdi in sr 7e934fee-96b5-b297-2c25-b20a14139fd9

[root@xenserver-devcloud /]# cp /usr/sbin/vhd-util /opt/xensource/bin/vhd-util 
[root@xenserver-devcloud /]# copy_vhd_from_secondarystorage.sh 
172.16.197.134:/opt/storage/secondary/template/tmpl/1/1/
7e934fee-96b5-b297-2c25-b20a14139fd9 DEVTEST
1001+1 records in
1001+1 records out
2101252608 bytes (2.1 GB) copied, 178.662 seconds, 11.8 MB/s
0#c9cb5877-9434-4f3e-93b7-2f895cc62642






On 4/12/13 5:04 PM, "Chiradeep Vittal"  wrote:

>Because of this
>http://s.apache.org/Nsd
>
>On 4/12/13 4:32 PM, "Soheil Eizadi"  wrote:
>
>>That patch is about downloading vhd-util to XenServer.
>>
>>The XenServer (6.0.2) I am using already came bundled with vhd-util, I 
>>did not have to download it. I have not read the detail of 
>>copy_vhd_from_secondarystorage.sh which throws the error to figure out 
>>exactly why it is failing in my environment. I have used this same 
>>XenServer ima

Re: [BUG][ACS41] vm_data updates router VM .htaccess file with duplicate RewriteRules

2013-04-15 Thread Musayev, Ilya
Chiradeep,

Thanks for the feedback,

We are aware what this is for. There is an issue we found where "boto", aws 
component used by cloud-init looks for meta-data dir. In ACS 4.1, the rewrite 
rule for meta-data dir is never added to router vm's .htaccess file - however 
we do see the code for it in vm_data.sh

Hence we trying to understand - why its not added.

Ill dig more tomorrow, if you can explain the context behind vm_data.sh, whats 
its for and why its executed - it would really speed up our debug efforts.

Thanks
Ilya



 Original message 
From: Chiradeep Vittal 
Date:
To: "Musayev, Ilya" ,dev@cloudstack.apache.org
Subject: Re: [BUG][ACS41] vm_data updates router VM .htaccess file with 
duplicate RewriteRules


I think if you run
git blame  ./patches/systemvm/debian/config/root/userdata.sh

There's several changes done to this particular piece of code.

The vm_data file is used for AWS-style "metadata" and user data
http://s.apache.org/amX
http://s.apache.org/PJH


From: , Ilya mailto:imusa...@webmd.net>>
Date: Monday, April 15, 2013 3:47 PM
To: "dev@cloudstack.apache.org" 
mailto:dev@cloudstack.apache.org>>, Chiradeep Vittal 
mailto:chiradeep.vit...@citrix.com>>
Subject: [BUG][ACS41] vm_data updates router VM .htaccess file with duplicate 
RewriteRules

++Chiradeep, as I see your name referenced in the script.

Noticed an odd behavior on ACS41 and vSphere, where each time you deploy a VM, 
on router vm under /var/www/html/latest/ the .htaccess file gets duplicate set 
of rewrite rules appended.

Vm_data is called each time VM is deployed, and each time redundant 
RewriteRules are added to .htaccess file – per each run/vm demployment.

Before I begin troubleshooting and fixing this issue (I can help rewrite the 
bash logic for checking if entry exists), could someone please give me a 
context as to how and when vm_data file is used.


Also, for some reason metadata/meta-data Rewrite Rule is missing, I would like 
to understand where the call is made and how $folder variable is set.

Thanks
ilya


Re: Firewall and LB network elements in inline mode

2013-04-15 Thread Sheng Yang
Support for SRX + F5 big ip is already in.

--Sheng


On Mon, Apr 15, 2013 at 10:07 AM, Chiradeep Vittal <
chiradeep.vit...@citrix.com> wrote:

> Isn't this discussed here:
> https://cwiki.apache.org/confluence/x/ggjVAQ
>
> On 4/15/13 6:51 AM, "Koushik Das"  wrote:
>
> >The network orchestration component currently supports elements in a
> >side-by-side mode. Compatible network elements can be used in this mode
> >to offer various network services. There are also scenarios where an
> >inline mode would be desirable. For e.g. firewall followed by LB. Wanted
> >to check out on the pros/cons of having support for network elements in
> >inline mode in the core component itself.
> >
> >-Koushik
>
>


Re: [BUG][ACS41] vm_data updates router VM .htaccess file with duplicate RewriteRules

2013-04-15 Thread Chiradeep Vittal
For Citrix Xenserver, the vm_data.sh script gets executed on dom0 every time a 
vm is added to the network.
For VMWare, it executes the userdata.py script which in turn calls the 
userdata.sh script.

The agent api that drives this process is VMDataCommand


From: , Ilya mailto:imusa...@webmd.net>>
Reply-To: "Musayev, Ilya" mailto:imusa...@webmd.net>>
Date: Monday, April 15, 2013 4:56 PM
To: Chiradeep Vittal 
mailto:chiradeep.vit...@citrix.com>>, 
"dev@cloudstack.apache.org" 
mailto:dev@cloudstack.apache.org>>
Subject: Re: [BUG][ACS41] vm_data updates router VM .htaccess file with 
duplicate RewriteRules

Chiradeep,

Thanks for the feedback,

We are aware what this is for. There is an issue we found where "boto", aws 
component used by cloud-init looks for meta-data dir. In ACS 4.1, the rewrite 
rule for meta-data dir is never added to router vm's .htaccess file - however 
we do see the code for it in vm_data.sh

Hence we trying to understand - why its not added.

Ill dig more tomorrow, if you can explain the context behind vm_data.sh, whats 
its for and why its executed - it would really speed up our debug efforts.

Thanks
Ilya



 Original message 
From: Chiradeep Vittal 
mailto:chiradeep.vit...@citrix.com>>
Date:
To: "Musayev, Ilya" 
mailto:imusa...@webmd.net>>,dev@cloudstack.apache.org
Subject: Re: [BUG][ACS41] vm_data updates router VM .htaccess file with 
duplicate RewriteRules


I think if you run
git blame  ./patches/systemvm/debian/config/root/userdata.sh

There's several changes done to this particular piece of code.

The vm_data file is used for AWS-style "metadata" and user data
http://s.apache.org/amX
http://s.apache.org/PJH


From: , Ilya mailto:imusa...@webmd.net>>
Date: Monday, April 15, 2013 3:47 PM
To: "dev@cloudstack.apache.org" 
mailto:dev@cloudstack.apache.org>>, Chiradeep Vittal 
mailto:chiradeep.vit...@citrix.com>>
Subject: [BUG][ACS41] vm_data updates router VM .htaccess file with duplicate 
RewriteRules

++Chiradeep, as I see your name referenced in the script.

Noticed an odd behavior on ACS41 and vSphere, where each time you deploy a VM, 
on router vm under /var/www/html/latest/ the .htaccess file gets duplicate set 
of rewrite rules appended.

Vm_data is called each time VM is deployed, and each time redundant 
RewriteRules are added to .htaccess file – per each run/vm demployment.

Before I begin troubleshooting and fixing this issue (I can help rewrite the 
bash logic for checking if entry exists), could someone please give me a 
context as to how and when vm_data file is used.


Also, for some reason metadata/meta-data Rewrite Rule is missing, I would like 
to understand where the call is made and how $folder variable is set.

Thanks
ilya


[DOCS] Please review: VM Snapshots section (CLOUDSTACK-1561)

2013-04-15 Thread Jessica Tomechak
I have posted a review draft at the URL below [1]. Please take a look. A
few more replies to Mice's input inline:


>
>
> *From:* Mice Xia [mailto:mice_...@tcloudcomputing.com]
> *Sent:* Tuesday, April 09, 2013 10:25 PM
> *To:* Jessica Tomechak; Vijayendra Bhamidipati
> *Cc:* Animesh Chaturvedi
> *Subject:* RE: Doc questions on VM Snapshots (CLOUDSTACK-684)
>
> ** **
>
> Hi Jessica
>
> ** **
>
> Quiesce option is not implemented, it is always ‘not quiesce’
>

I added this under "Limitations".


> 
>
> vmsnapshot.expunge.interval and vmsnapshot.expunge.workers are not used
> anymore and I have removed them from global config.
>

Also not in the docs! Thanks for letting me know.


> 
>
> ** **
>
> And per Animesh’s suggestion, i.e, enable/disable this feature  on a per
> hypervisor basis,  this has been implemented in hypervisor_capabilities,
> and by default Xenserver and VMware are enabled. But the UI part needs
> further work, I will contact UI gurus for details.
>

I had thought this was VMware only for 4.2. Please advise if XenServer will
work in this release. For now, the docs just say "VMware only."


> 
>
> ** **
>
> *For the doc:*
>
> VM snapshots can't be kept if you change the VM's service offering. Any
> existing VM snapshots will be discarded.
>
> VM snapshots can't be kept if you change the VM's service offering. Any
> existing VM snapshots *of type ‘DiskAndMemory’* will be discarded.
>

I made a change for this; please check it out.


> 
>
> ** **
>
> And if possible, it is preferred to add some explanations about ‘current’
> and ‘parent’ attribute of a snapshot.
>

I added a paragraph as follows:

VM snapshots can have a parent/child relationship. Each successive snapshot
of the same VM is the child of the snapshot that came before it. Each time
you take an additional snapshot of the same VM, it saves only the
differences between the current state of the VM and the state stored in the
most recent previous snapshot. The previous snapshot becomes a parent, and
the new snapshot is its child. It is possible to create a long chain of
these parent/child snapshots, which amount to a "redo" record leading from
the current state of the VM back to the original.


> 
>
> ** **
>
> Thanks and Regards
>
> Mice
>
> ** **
>
[1]
http://people.apache.org/~jtomechak/CLOUDSTACK-1561/VM_Snapshots_Review_Draft.html

Thanks,
Jessica T.


Review Request: Improve console access security with 128-bit AES encryption and securely-randomized key generation

2013-04-15 Thread Kelven Yang

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

Review request for cloudstack and Chip Childers.


Description
---

Improve console access security with 128-bit AES encryption and 
securely-randomized key generation


This addresses bug CLOUDSTACK-2039.


Diffs
-

  
console-proxy/src/com/cloud/consoleproxy/ConsoleProxyPasswordBasedEncryptor.java
 29826f0 
  server/src/com/cloud/configuration/Config.java 0815a77 
  server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java 168ac0e 
  server/src/com/cloud/server/ManagementServer.java 5c34dee 
  server/src/com/cloud/server/ManagementServerImpl.java 9cddb77 
  server/src/com/cloud/servlet/ConsoleProxyPasswordBasedEncryptor.java 2638c8b 
  server/src/com/cloud/servlet/ConsoleProxyServlet.java c4b9334 

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


Testing
---

Manually tested


Thanks,

Kelven Yang



Re: Review Request: Improve console access security with 128-bit AES encryption and securely-randomized key generation

2013-04-15 Thread Chip Childers

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

Ship it!


Committed to 4.1:

commit 73ed03baea3f6718f74703b752a61c4314246a69
Author: Kelven Yang 
Date:   Mon Apr 15 16:51:50 2013 -0700

CLOUDSTACK-2039: Improve console access security with 128-bit AES 
encryption and securely-randomized key generation

- Chip Childers


On April 16, 2013, 12:39 a.m., Kelven Yang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10533/
> ---
> 
> (Updated April 16, 2013, 12:39 a.m.)
> 
> 
> Review request for cloudstack and Chip Childers.
> 
> 
> Description
> ---
> 
> Improve console access security with 128-bit AES encryption and 
> securely-randomized key generation
> 
> 
> This addresses bug CLOUDSTACK-2039.
> 
> 
> Diffs
> -
> 
>   
> console-proxy/src/com/cloud/consoleproxy/ConsoleProxyPasswordBasedEncryptor.java
>  29826f0 
>   server/src/com/cloud/configuration/Config.java 0815a77 
>   server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java 168ac0e 
>   server/src/com/cloud/server/ManagementServer.java 5c34dee 
>   server/src/com/cloud/server/ManagementServerImpl.java 9cddb77 
>   server/src/com/cloud/servlet/ConsoleProxyPasswordBasedEncryptor.java 
> 2638c8b 
>   server/src/com/cloud/servlet/ConsoleProxyServlet.java c4b9334 
> 
> Diff: https://reviews.apache.org/r/10533/diff/
> 
> 
> Testing
> ---
> 
> Manually tested
> 
> 
> Thanks,
> 
> Kelven Yang
> 
>



Re: [DISCUSS] Granular Global Parameters

2013-04-15 Thread Chip Childers
On Thu, Apr 11, 2013 at 03:36:59AM +, Abhinandan Prateek wrote:
> 
> 
> On 10/04/13 6:30 PM, "David Nalley"  wrote:
> 
> >On Wed, Apr 10, 2013 at 7:08 AM, Harikrishna Patnala
> > wrote:
> >> Hi all,
> >>
> >> There are many global parameters which are used to set
> >>values/limits/boolean for various operations, but these parameters
> >>effects all zones/clusters/accounts/storage based on the parameter.
> >> Here I would like to discuss on granulising these parameters so that
> >>these parameters can be customised at different levels
> >>(zone/cluster/account/storage).
> >> New APIs are introduced to update these parameters based on the level
> >>listed in the FS below.
> >> During the creation of zone/cluster/account/storage default values for
> >>the granular parameters are set from the normal global parameters and
> >>later these can be updated using the corresponding API.
> >>
> >>
> >> This proposal for Global granular parameters is detailed in the FS
> >>here: 
> >>https://cwiki.apache.org/confluence/display/CLOUDSTACK/FS+for+Granular+Gl
> >>obal+Configuration+Parameters
> >> The JIRA ticket for tracking is
> >>https://issues.apache.org/jira/browse/CLOUDSTACK-741
> >>
> >> Please review this and provide me comments/suggestions.
> >>
> >> Thanks
> >> Harikrishna
> >
> >
> >Hi Harikrisha:
> >
> >First - the title is a bit confusing; granular and global seems
> >contradictory. Regardless - this is a great move forward.
> >
> >I don't understand why we would inject a new API command
> >(update{storage,cluster,zone,account}levelparamater) instead of just
> >using updateZone, updateAccount, updateStoragePool, etc. For instance,
> >I would expect that listZone would tell me the status of the
> >zone-level options. So why wouldn't updateZone be capable of setting
> >the options
> 
> 
> Good question. Whether to overload an existing API or create a new one is
> always debatable.
> In this case one of the reason is that the existing update APIs were
> returning a {Zone, Account,..}Response that is not required when you
> change a granular config param. Moreover, all the existing update APIs are
> already heavily overloaded, not a strong reason to avoid further
> overloading though apart from that the response grows in size more you
> overload.

+1 - and keeping it logically distinct will help if / when we spend the
time to transition to a more RESTful API model.

> 
> We will also need an API to query the config params at these various
> granularities, that is not mentioned in the FS.
> 
> -abhi



Re: Need project repo for Hyper-V ServerResource written in C#/.NET code

2013-04-15 Thread Chip Childers
On Wed, Apr 10, 2013 at 12:32:36PM -0400, David Nalley wrote:
> On Wed, Apr 10, 2013 at 11:23 AM, Donal Lafferty
>  wrote:
> >
> >> -Original Message-
> >> From: Chip Childers [mailto:chip.child...@sungard.com]
> >> Sent: 10 April 2013 2:17 PM
> >> To: 
> >> Subject: Re: Need project repo for Hyper-V ServerResource written in
> >> C#/.NET code
> >>
> >> On Apr 10, 2013, at 9:09 AM, David Nalley  wrote:
> >>
> >> > On Wed, Apr 10, 2013 at 5:35 AM, Wido den Hollander 
> >> wrote:
> >> >> On 04/09/2013 10:58 PM, Donal Lafferty wrote:
> >> >>>
> >> >>> I brought up some problems related to a ServerResource written in C#
> >> >>> on the mailing list [1].
> >> >>>
> >> >>> WRT to the source location, the conclusion seemed to be to create an
> >> >>> Apache CloudStack *project* repo for the source.  E.g. ChipC & DaveN
> >> >>> suggested a separate repo[2][3].
> >> >>>
> >> >>> Who can I approach to get this repo created?  Any committer?
> >> >>>
> >> >>
> >> >> Good one! I think someone from the PMC (like me) should contact Infra
> >> >> to set up a new repository.
> >> >>
> >> >> I'm not sure though since I've never done this before. Any other PMC
> >> >> member how knows how this works?
> >> >>
> >> >> Wido
> >> >>
> >> >
> >> > Yes, from a resource standpoint, just create the git repo request in
> >> > infra's jira. Feel free to assign it to me, and I'll process it.
> >> >
> >> > From a procedural standpoint, we probably have some due diligence
> >> > necessary on the PMC's part, before we commit to that repo
> >> > (Establishing provenance, ensuring we have paperwork, populating the
> >> > repo with things like LICENSE and NOTICE files)
> >> >
> >> Yes.  If this is a donation, we need to go through IP clearance. Now that
> >> we've graduated, I can run the process for us. The git repo is just the 
> >> target
> >> repo to accept the new code.
> > [Donal Lafferty]
> >
> > Does this process apply if I am talking about new code written specifically 
> > for Apache CloudStack by Citrix, and not previously distributed anywhere?  
> > I mean, aren't I already covered by a Citrix to Apache CloudStack agreement?
> >
> >>
> 
> Does the code exist yet? If so, where can we find it? If so, are you
> the sole author?
> If it doesn't yet exist, it will be a slightly different process.
> Keep in mind that there is no Corporate CLA between Citrix and the
> ASF, there has been a very specific Software Grant, but it doesn't
> cover this work.
> 
> If the code exists, it will need to come in via IP Clearance, which
> should be relatively painless, just a process step.
> 
> --David
>

Donal,

Can you reply to David's questions below?  Do you want to try to get
this into the project before our 4.2 feature cut-off date (end of May,
not that far away).

-chip


Review Request: CLOUDSTACK-1894 User is not able to deploy VMs in a project[permission denied to access the guest network)

2013-04-15 Thread mice xia

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

Review request for cloudstack and Chip Childers.


Description
---

this is merged from cbcb83b086f4a87aa0703e18a03938b808ed044f in master


This addresses bug CS-1894.


Diffs
-

  server/src/com/cloud/network/NetworkModelImpl.java d39214a 

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


Testing
---


Thanks,

mice xia



Re: How to set the default network when I specify IP use "iptonetworklist" parameter when deploy a vm?

2013-04-15 Thread sx chen
Thanks,Alena and Sangeetha
My environment is CS 3.02,I want make the default nic is 10.10.4.41,I've
made these test:
1.
iptonetworklist[0].ip=10.10.4.41&iptonetworklist[0].networkid=233&iptonetworklist[1].ip=10.10.15.91&iptonetworklist[1].networkid=234&iptonetworklist[2].ip=10.10.16.91&iptonetworklist[2].networkid=235
the default nic is 10.10.16.91
2.
iptonetworklist[0].ip=10.10.16.91&iptonetworklist[0].networkid=235&iptonetworklist[1].ip=10.10.15.91&iptonetworklist[1].networkid=234&iptonetworklist[2].ip=10.10.4.41&iptonetworklist[2].networkid=233
the default nic is 10.10.16.91
3.
iptonetworklist[0].ip=10.10.4.41&iptonetworklist[0].networkid=233&iptonetworklist[1].ip=10.10.15.91&iptonetworklist[1].networkid=234&iptonetworklist[2].ip=10.10.16.91&iptonetworklist[2].networkid=235&networkids=233
response:ipToNetworkMap can't be specified along with networkIds or
ipAddress

So,It seems the iptonetworklist can't use with networkIds.
also I can't specify the default network by set it in iptonetworklist[0].

maybe a commit after 3.02 has solve this problem?



2013/4/16 Sangeetha Hariharan 

> When using iptonetworklist  when deploying Vms ,first entry in the
> iptonetworklist ( iptonetworklist[0].networkid) becomes the default network.
>
> -Thanks
> Sangeetha
>
> -Original Message-
> From: Alena Prokharchyk [mailto:alena.prokharc...@citrix.com]
> Sent: Monday, April 15, 2013 9:32 AM
> To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
> Subject: Re: How to set the default network when I specify IP use
> "iptonetworklist" parameter when deploy a vm?
>
>
> NetworkIds parameter has to be specify regardless of iptonetworklist
> parameter presence. So you have to define both parameters, and the first
> network from the networkIds list will be used as a default.
>
> -Alena.
>
> On 4/15/13 7:44 AM, "sx chen"  wrote:
>
> >Hi,
> >   as far as I know, I can specify the default network by put it at the
> >first when use "*networkids*" paramenter when deploy a vm.
> >   but if I use "iptonetworklist" parameter specify the IP address,I
> >find I can't specify the default network.I've test put it at the first
> >and put it at the last.
> >   So if I want to specify the default network as well as the IP
> >address,what should I do?
> >appreciate your help!
> >
>
>
>


Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-15 Thread Venkata Siva Vijayendra Bhamidipati

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

(Updated April 16, 2013, 2:53 a.m.)


Review request for cloudstack, Chip Childers, Hugo Trippaers, Kelven Yang, and 
Min Chen.


Changes
---

Please find the latest diff attached with the following changes:
- Marvin client API changes to support POST requests by clients
- Inclusion of unittest files missing from earlier patch 
(UserVmDaoTestConfiguration.java and UserVMDaoTestContext.xml)
- Fixed a bug discovered during testing w.r.t httpmethod value being populated 
in the parameter map in the integration port code path

The marvin HTTP API itself is still to be put in. I'm working on it, will put 
it in at the earliest. Tests already exist for DeployVirtualMachineCmd. I will 
need to make appropriate changes to add to those tests, a case where userdata > 
2K is fired using a POST request.


Description
---

Please refer to 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/DeployVirtualMachine+userdata+enhancements
 for a background on the requirements driving this patch.

This patch hasn't been extensively tested yet, and I will update this request 
with more info. I am uploading a first diff for initial review/comments.


This addresses bug CLOUDSTACK-1086.


Diffs (updated)
-

  api/src/com/cloud/vm/UserVmService.java d963b74 
  api/src/org/apache/cloudstack/api/BaseCmd.java 42c0680 
  api/src/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java 77ba9fe 
  core/src/com/cloud/vm/UserVmVO.java a16eaf9 
  server/src/com/cloud/api/ApiDispatcher.java 925d90a 
  server/src/com/cloud/api/ApiServer.java d842819 
  server/src/com/cloud/api/ApiServerService.java 12d8b52 
  server/src/com/cloud/api/ApiServlet.java 03bfb5f 
  server/src/com/cloud/vm/UserVmManagerImpl.java d281e5b 
  server/test/com/cloud/vm/MockUserVmManagerImpl.java fd826d9 
  server/test/com/cloud/vm/dao/UserVmDaoImplTest.java 0936180 
  server/test/com/cloud/vm/dao/UserVmDaoTestConfiguration.java PRE-CREATION 
  server/test/resources/UserVMDaoTestContext.xml PRE-CREATION 
  setup/db/db/schema-410to420.sql fb760bf 
  tools/marvin/marvin/cloudstackConnection.py 1caeef3 

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


Testing
---


Thanks,

Venkata Siva Vijayendra Bhamidipati



Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-15 Thread Venkata Siva Vijayendra Bhamidipati


> On April 11, 2013, 8:55 p.m., Min Chen wrote:
> > In your description, you mentioned that it is not extensively tested yet. 
> > Is this still true? If not, please revise your description to avoid 
> > confusing people.

Not true anymore - apologies for the confusion. I'll put in details of all 
testing done when I upload the new diffs (wip).


- Venkata Siva Vijayendra


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


On April 16, 2013, 2:53 a.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10294/
> ---
> 
> (Updated April 16, 2013, 2:53 a.m.)
> 
> 
> Review request for cloudstack, Chip Childers, Hugo Trippaers, Kelven Yang, 
> and Min Chen.
> 
> 
> Description
> ---
> 
> Please refer to 
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/DeployVirtualMachine+userdata+enhancements
>  for a background on the requirements driving this patch.
> 
> This patch hasn't been extensively tested yet, and I will update this request 
> with more info. I am uploading a first diff for initial review/comments.
> 
> 
> This addresses bug CLOUDSTACK-1086.
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/vm/UserVmService.java d963b74 
>   api/src/org/apache/cloudstack/api/BaseCmd.java 42c0680 
>   api/src/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java 77ba9fe 
>   core/src/com/cloud/vm/UserVmVO.java a16eaf9 
>   server/src/com/cloud/api/ApiDispatcher.java 925d90a 
>   server/src/com/cloud/api/ApiServer.java d842819 
>   server/src/com/cloud/api/ApiServerService.java 12d8b52 
>   server/src/com/cloud/api/ApiServlet.java 03bfb5f 
>   server/src/com/cloud/vm/UserVmManagerImpl.java d281e5b 
>   server/test/com/cloud/vm/MockUserVmManagerImpl.java fd826d9 
>   server/test/com/cloud/vm/dao/UserVmDaoImplTest.java 0936180 
>   server/test/com/cloud/vm/dao/UserVmDaoTestConfiguration.java PRE-CREATION 
>   server/test/resources/UserVMDaoTestContext.xml PRE-CREATION 
>   setup/db/db/schema-410to420.sql fb760bf 
>   tools/marvin/marvin/cloudstackConnection.py 1caeef3 
> 
> Diff: https://reviews.apache.org/r/10294/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>



[ACS42][QA] Test cases for Dedicated Resources - Public IP Addresses and VLANs per Tenant feature uploaded

2013-04-15 Thread Abhinav Roy
Hi,



I've uploaded some test cases for the feature here: 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Dedicated+Resources+-+Public+IP+Addresses+and+VLANs+per+Tenant+Test+Plan



 Jira id tracking the feature is: 
https://issues.apache.org/jira/browse/CLOUDSTACK-704



Thanks and regards,

Abhinav



RE: [DISCUSS] Getting rid of premium

2013-04-15 Thread Sateesh Chodapuneedi
> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 15 April 2013 22:43
> To: dev@cloudstack.apache.org
> Subject: Re: [DISCUSS] Getting rid of premium
> 
> 
> 
> On 4/15/13 8:53 AM, "Hugo Trippaers" 
> wrote:
> 
> >Heya,
> >
> >As we don't have a premium distribution anymore I would like to get rid
> >of the premium configuration value. At the moment it seems to be
> >unintentionally used to distinguish between the nonss and oss build so
> >I think it can go. It will likely only confuse people that are new to
> >the project and doesn't bring anything usefull so it seems.
> >
> >I'm willing to trawl through the code and remove any references to
> >premium and replace it with another flag where needed (for example a
> >flag to indicate vmware support is available).
> >
> >Any thoughts about this?
> >
> >Cheers,
> >
> >Hugo
> 
> +1

+1



[ACS402] Almost Ready: Please Check!

2013-04-15 Thread Joe Brockmeier
Hey all, 

I plan on starting the vote for this tomorrow, but you can find the
source packages here:

http://people.apache.org/~jzb/SRC/

Will copy these over to our dist space tomorrow after we get it created. 

Please take a look and if you find any wonkiness, please let me know.
Thanks!

Best,

jzb
-- 
Joe Brockmeier
j...@zonker.net
Twitter: @jzb
http://www.dissociatedpress.net/


Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-15 Thread Prasanna Santhanam

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



tools/marvin/marvin/cloudstackConnection.py


How and when does marvin decide to use POST? Which commands will call the 
POST httpmethod? May be you can provide a more complete example when your test 
is ready.



tools/marvin/marvin/cloudstackConnection.py


Are you sure you don't want to urlencode the POST data? 




tools/marvin/marvin/cloudstackConnection.py


Minor nitpick: urllib2.Request() can be used for both GET and POST and you 
can set the data argument only when you intend to use POST. That will reduce 
some code duplication here.


- Prasanna Santhanam


On April 16, 2013, 2:53 a.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10294/
> ---
> 
> (Updated April 16, 2013, 2:53 a.m.)
> 
> 
> Review request for cloudstack, Chip Childers, Hugo Trippaers, Kelven Yang, 
> and Min Chen.
> 
> 
> Description
> ---
> 
> Please refer to 
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/DeployVirtualMachine+userdata+enhancements
>  for a background on the requirements driving this patch.
> 
> This patch hasn't been extensively tested yet, and I will update this request 
> with more info. I am uploading a first diff for initial review/comments.
> 
> 
> This addresses bug CLOUDSTACK-1086.
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/vm/UserVmService.java d963b74 
>   api/src/org/apache/cloudstack/api/BaseCmd.java 42c0680 
>   api/src/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java 77ba9fe 
>   core/src/com/cloud/vm/UserVmVO.java a16eaf9 
>   server/src/com/cloud/api/ApiDispatcher.java 925d90a 
>   server/src/com/cloud/api/ApiServer.java d842819 
>   server/src/com/cloud/api/ApiServerService.java 12d8b52 
>   server/src/com/cloud/api/ApiServlet.java 03bfb5f 
>   server/src/com/cloud/vm/UserVmManagerImpl.java d281e5b 
>   server/test/com/cloud/vm/MockUserVmManagerImpl.java fd826d9 
>   server/test/com/cloud/vm/dao/UserVmDaoImplTest.java 0936180 
>   server/test/com/cloud/vm/dao/UserVmDaoTestConfiguration.java PRE-CREATION 
>   server/test/resources/UserVMDaoTestContext.xml PRE-CREATION 
>   setup/db/db/schema-410to420.sql fb760bf 
>   tools/marvin/marvin/cloudstackConnection.py 1caeef3 
> 
> Diff: https://reviews.apache.org/r/10294/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>



Re: [QA][PROPOSAL][ACS4.2] Test plan review: GSLB (Global Server Load Balancing)

2013-04-15 Thread Murali Reddy

Thanks for test plan. Test plan looks good to me.For the below questions
you raised in the test plan please open a bug.

1. Provide a mechanism for updating information about existing Netscaler
device that is enabled for GSLB (like change isGslbProvider from True to
False and vice versa etc..,)
2. Prvide a mechanism for reprogramming GSLB info on the network devices
in case of any changes or configuration lost /messed up situations.



On 12/04/13 8:51 AM, "Venkata SwamyBabu Budumuru"
 wrote:

>
>Hi Murali,
>
>Please review the test plan [1] for feature " GSLB (Global Server Load
>Balancing)" and let me know the review comments. The test cases are
>mentioned in an excel sheet attached to the page.
>
>[1] Test Plan  :
>https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=31817730
>[2] Functional Spec:
>https://cwiki.apache.org/CLOUDSTACK/gslb-global-server-load-balancing-func
>tional-specification-and-design-document.html
>[3] Bug Id   :
>https://issues.apache.org/jira/browse/CLOUDSTACK-894
>
>
>Thanks,
>SWAMY
>
>




RE: [Add/Remove Network to VM] Multiple NICs on same Guest Network

2013-04-15 Thread Jayapal Reddy Uradi
The reason to choose multiple ips is  there is a limit (I think it is 7) on the 
hypervisor about number of nic can VM have.
For a nic you can have more than 7 ip addresses.
The multiple ips feature is supported in the basic zone also.

I think multiple nics feature is more useful when user need a nic in different 
network.

Thanks,
Jayapal

> -Original Message-
> From: Marcus Sorensen [mailto:shadow...@gmail.com]
> Sent: Monday, 15 April 2013 10:13 PM
> To: Saksham Srivastava
> Cc: dev@cloudstack.apache.org
> Subject: Re: [Add/Remove Network to VM] Multiple NICs on same Guest
> Network
> 
> The functionality was duplicated from deploy allowing multiple NICs, and I
> wasn't involved in the discusion as to why that was decided. There could be
> other reasons why that was put in place, I don't know. At any rate it doesn't
> hurt anything in particular, and I'm not sure how many are using the
> capability in 4.0 and earlier, but they may have gotten used to it.
> 
> 
> On Mon, Apr 15, 2013 at 10:32 AM, Saksham Srivastava <
> saksham.srivast...@citrix.com> wrote:
> 
> >   We have a feature Multiple IPs on the same NIC
> >
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Multiple+IP+add
> > ress+per+NIC
> >
> > Isn't the functionality covered by this feature.
> >
> > I do not see a strong use case for having both the features (Multi NIC
> > in same network and Multi IP per NIC).
> > Should we allow only one of them, as they both tend to serve the same
> > purpose, or am I missing something?
> >
> > Thanks,
> > Saksham
> >  --
> > *From:* Marcus Sorensen [shadow...@gmail.com]
> > *Sent:* Saturday, April 13, 2013 9:03 PM
> > *To:* Saksham Srivastava
> > *Cc:* dev@cloudstack.apache.org
> > *Subject:* Re: [Add/Remove Network to VM] Multiple NICs on same
> Guest
> > Network
> >
> >   I believe it was allowed because one can also create multiple NICs
> > on the same network while deploying a VM.
> >
> > There may be people doing that to get multiple IPS auto-assigned to a VM.
> > On Apr 13, 2013 3:28 AM, "Saksham Srivastava" <
> > saksham.srivast...@citrix.com> wrote:
> >
> >>  Hi,
> >>
> >> Using addNicToVirtualMachine API user can add multiple NICs to a VM
> >>
> >> I am also able to add multiple NICs to a VM on the same isolated
> >> guest network.
> >>
> >> Is this a valid scenario??
> >> If yes what could be the use case for the same?
> >>
> >> Thanks,
> >> Saksham
> >>
> >>


[QA][ACS42] Test Plan for Change Account Membership CS-1390

2013-04-15 Thread Parth Jagirdar
All,

Test Plan @ 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Change+account+membership+Test+Plan

CS Bug @ https://issues.apache.org/jira/browse/CLOUDSTACK-1390

Requirements @ 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Change+account+membership

FS & UseCases @ 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Change+Account+Membership+-+VM


…Parth



RE: [DISCUSS] Getting rid of premium

2013-04-15 Thread Mice Xia
+1

-Original Message-
From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com] 
Sent: Tuesday, April 16, 2013 12:16 PM
To: dev@cloudstack.apache.org
Subject: RE: [DISCUSS] Getting rid of premium

> -Original Message-
> From: Kelven Yang [mailto:kelven.y...@citrix.com]
> Sent: 15 April 2013 22:43
> To: dev@cloudstack.apache.org
> Subject: Re: [DISCUSS] Getting rid of premium
> 
> 
> 
> On 4/15/13 8:53 AM, "Hugo Trippaers" 
> wrote:
> 
> >Heya,
> >
> >As we don't have a premium distribution anymore I would like to get 
> >rid of the premium configuration value. At the moment it seems to be 
> >unintentionally used to distinguish between the nonss and oss build 
> >so I think it can go. It will likely only confuse people that are new 
> >to the project and doesn't bring anything usefull so it seems.
> >
> >I'm willing to trawl through the code and remove any references to 
> >premium and replace it with another flag where needed (for example a 
> >flag to indicate vmware support is available).
> >
> >Any thoughts about this?
> >
> >Cheers,
> >
> >Hugo
> 
> +1

+1



Re: [DISCUSS] Getting rid of premium

2013-04-15 Thread Rohit Yadav
+1

On Tue, Apr 16, 2013 at 10:59 AM, Mice Xia wrote:

> +1
>
> -Original Message-
> From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com]
> Sent: Tuesday, April 16, 2013 12:16 PM
> To: dev@cloudstack.apache.org
> Subject: RE: [DISCUSS] Getting rid of premium
>
> > -Original Message-
> > From: Kelven Yang [mailto:kelven.y...@citrix.com]
> > Sent: 15 April 2013 22:43
> > To: dev@cloudstack.apache.org
> > Subject: Re: [DISCUSS] Getting rid of premium
> >
> >
> >
> > On 4/15/13 8:53 AM, "Hugo Trippaers" 
> > wrote:
> >
> > >Heya,
> > >
> > >As we don't have a premium distribution anymore I would like to get
> > >rid of the premium configuration value. At the moment it seems to be
> > >unintentionally used to distinguish between the nonss and oss build
> > >so I think it can go. It will likely only confuse people that are new
> > >to the project and doesn't bring anything usefull so it seems.
> > >
> > >I'm willing to trawl through the code and remove any references to
> > >premium and replace it with another flag where needed (for example a
> > >flag to indicate vmware support is available).
> > >
> > >Any thoughts about this?
> > >
> > >Cheers,
> > >
> > >Hugo
> >
> > +1
>
> +1
>
>


Re: [DISCUSS] Getting rid of premium

2013-04-15 Thread Nitin Mehta
+1 provided we test it thoroughly and get it reviewed as well to ensure
that we don't break the master.

Thanks,
-Nitin

On 16/04/13 11:01 AM, "Rohit Yadav"  wrote:

>+1
>
>On Tue, Apr 16, 2013 at 10:59 AM, Mice Xia
>wrote:
>
>> +1
>>
>> -Original Message-
>> From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com]
>> Sent: Tuesday, April 16, 2013 12:16 PM
>> To: dev@cloudstack.apache.org
>> Subject: RE: [DISCUSS] Getting rid of premium
>>
>> > -Original Message-
>> > From: Kelven Yang [mailto:kelven.y...@citrix.com]
>> > Sent: 15 April 2013 22:43
>> > To: dev@cloudstack.apache.org
>> > Subject: Re: [DISCUSS] Getting rid of premium
>> >
>> >
>> >
>> > On 4/15/13 8:53 AM, "Hugo Trippaers" 
>> > wrote:
>> >
>> > >Heya,
>> > >
>> > >As we don't have a premium distribution anymore I would like to get
>> > >rid of the premium configuration value. At the moment it seems to be
>> > >unintentionally used to distinguish between the nonss and oss build
>> > >so I think it can go. It will likely only confuse people that are new
>> > >to the project and doesn't bring anything usefull so it seems.
>> > >
>> > >I'm willing to trawl through the code and remove any references to
>> > >premium and replace it with another flag where needed (for example a
>> > >flag to indicate vmware support is available).
>> > >
>> > >Any thoughts about this?
>> > >
>> > >Cheers,
>> > >
>> > >Hugo
>> >
>> > +1
>>
>> +1
>>
>>



Re: [Discuss] API name alias

2013-04-15 Thread Nitin Mehta
In case we need different response tags we can also extend the existing
api class
 


Thanks,
-Nitin

On 09/04/13 2:29 PM, "Kishan Kavala"  wrote:

>Thanks everyone for your inputs.
>We should avoid making changes to the APIs which have only one name. So
>they'll continue to have the annotation:
>
>@APICommand(name="apiName1")
>
>For APIs with multiple names, we should support String[] as suggested by
>John. Preferred and deprecated would be required in case of multiple API
>names. Preferred and deprecated params should also support String[].
>@APICommand(name={'Foo', 'Bar', 'FooBar'}, preferred='Foo',
>deprecated={'Bar','FooBar'})
>
>
>> -Original Message-
>> From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
>> Sent: Tuesday, 9 April 2013 12:05 PM
>> To: dev@cloudstack.apache.org
>> Subject: Re: [Discuss] API name alias
>> 
>> Rohit, that's like saying we won't fix a leaky faucet today because
>>we're
>> moving in a couple of years anyway. It isn't a big deal to add this and
>>it
>> clarifies to the end user what is the correct alias to use.
>> 
>> On 4/8/13 9:52 PM, "Rohit Yadav"  wrote:
>> 
>> >If we are still on our previously discussed plan (search a wiki shared
>> >by
>> >Min) on deprecating the whole query based API (Server), it's just
>> >unnecessary work on present APIs. I think for now just changing the
>> >name attribute type from String to String[] would just work, as John
>> suggested.
>> >
>> >Adding other fields, may not hold much use for now. Except for few of
>> >them the rest of 300 apis will have preferred == current apiname and
>> >deprecated as blank string.
>> >
>> >I think we should just deprecated all cmd classes (query based) over a
>> >long period say 1 or 2 years, and/or find a way to rewrite/refactor
>> >them so as to reuse them with our new rest based API server for future.
>> >
>> >Cheers.
>> >
>> >On Tue, Apr 9, 2013 at 3:18 AM, Chiradeep Vittal <
>> >chiradeep.vit...@citrix.com> wrote:
>> >
>> >>
>> >>
>> >> On 4/8/13 11:18 AM, "Rohit Yadav"  wrote:
>> >>
>> >> >On Mon, Apr 8, 2013 at 7:10 PM, John Burwell 
>> >>wrote:
>> >> >
>> >> >> Rohit,
>> >> >>
>> >> >> Why would we need to rewrite any Java code to switch to an array?
>> >>As I
>> >> >> mentioned, array annotation values are treated as varargs.
>> >>Therefore,
>> >> >> annotations with single values can be left alone.  Only cases
>> >> >> where multiple values are required would require the addition of
>> >> >> curly
>> >>braces.
>> >> >>  For example, for an APICommand with one name, the following would
>> >> >>continue  to work:
>> >> >>
>> >> >
>> >> >Hey John, you're right but it's a matter of code styling, I prefer
>> >> >writing;
>> >> >
>> >> >@APICommand(name = {'name1'}) even if it's only one name, this way
>> >> >any subsequent api author would know that name is an array and they
>> >> >can
>> >>have
>> >> >aliases which is what Kishan wants.
>> >> >
>> >> >
>> >> >>
>> >> >> @APICommand(name="apiName1")
>> >> >>
>> >> >> While for an APICommand with multiple names, the following form
>> >> >>would now  be supported:
>> >> >>
>> >> >> @APICommand(name={"apiName1", "apiName2",
>>"apiName3"})@APICommand(name={"apiName1", "apiName2",
>>"apiName3"})@APICommand(name={"apiName1", "apiName2", "apiName3"})
>> >> >>
>> >> >> I am not familiar with the Marvin code, so I can not speak to the
>> >>impact
>> >> >> of this change to it.  However, the only changes to the Java code
>> >> >>should be  the API commands with multiple names and the classes
>> >> >>that actually consume  the value of this annotation.
>> >> >>
>> >> >
>> >> >Yes, you're right we can skip the code styling I referred then it
>> >> >won't consume much coding, we just need to fix these;
>> >> >
>> >> >- Fix APICommand annotation interface definition to accept name as
>> >> >an array
>> >> >- Fix method in ApiServer which creates apiname->cmdclass map to
>> >> >care
>> >>for
>> >> >aliases
>> >> >- Fix ApiDiscovery to take care of the aliases (create different
>> >>response
>> >> >objs for the same cmd class)
>> >> >- Fix apidocs XmlWriter class to take of the aliases and the api
>> >> >html generator to process the same.
>> >> >- Fix Marvin's codegenerator.py to take care of the aliases (which
>> >>means
>> >> >create apiname related modules which would contain cmd and response
>> >> >boilterplate fields/class)
>> >> >
>> >> >Cheers.
>> >> >
>> >> >
>> >> >>
>> >> >> Thanks,
>> >> >> -John
>> >> >>
>> >> >> On Apr 8, 2013, at 9:22 AM, Rohit Yadav 
>>wrote:
>> >> >>
>> >> >> > On Mon, Apr 8, 2013 at 6:33 PM, Kishan Kavala
>> >> >>> >> >> >wrote:
>> >> >> >
>> >> >> >> APICommand annotation in API Cmd object has a name parameter.
>> >> >>Currently
>> >> >> >> name parameter takes only one value. I plan to enhance this to
>> >> >>support
>> >> >> >> comma separated values. This will allow multiple API names for
>> >> >> >> the
>> >> >>same
>> >> >> API
>> >> >> >> Cmd object.
>> >> >> >>
>> >> >> >> Current:
>> >> >> >>

Re: [DISCUSS] Granular Global Parameters

2013-04-15 Thread Nitin Mehta
Also as Mice asked do we plan to restart MS to update say config changes
we make at zone/cluster level ?
For now I would suggest to stop using the class variables (which get
loaded during the class initiation time) and introduce a generic interface
with methods (input as name and scope id) to retrieve the appropriate
value for the config. The implementation of the method should ideally use
an in memory cache which gets dynamically updated or is refreshed every so
often(clustered MS can be an issue). But we can also use DB for now though
it would be highly non performant. In future all the configs should start
using this. 

Some food for thought ?

If we have enough consensus here and the discussion below please go ahead
and update the FS for the same.

Thanks,
-Nitin

On 15/04/13 6:23 PM, "Harikrishna Patnala"
 wrote:

>Yes Abhi I agree with you, this approach will eliminate the usage of
>multiple APIs.
>
>We can specify scope for each configuration parameter both in config.java
>file and configuration table.



Don¹t think you need to store scope in the configuration table. You can
just keep it in Config.java.



>We will not set the default values during the creation of resource
>(Zone/cluster/pool/account).
>
>Whenever we need to update an entity we call updateConfig API with name,
>value, scope and resource ID. This does following,
>- validates the scope of the parameter
>- checks the resource details table and updates there, if not present
>then will fetch from the global configuration parameters and create entry
>in the details table.
>
>API:  updateConfiguration
>Parameters: name, value, scope, entityId
>
>listConfiguration also fetches based on the scope.
>API: listConfiguration
>Parameters: category, name, scope, entityId
>
>
>
>-Harikrishna
>
>
>
>On 15-Apr-2013, at 4:36 PM, Abhinandan Prateek
>mailto:cloudst...@aprateek.com>>
> wrote:
>
>For Granular params, I am proposing that we use updateConfiguration
>command with two additional parameters i.e. scope_type and scope_id.
>Where scope_type can be zone, account, cluster or pool  and scope_id will
>be the corresponding id of that scope.
>
>We also similarly overload listConfiguration API with these two new
>params.
>
>-abhi
>
>On 11/04/13 9:06 AM, "Abhinandan Prateek"
>mailto:abhinandan.prat...@citrix.com>>
>wrote:
>
>
>
>On 10/04/13 6:30 PM, "David Nalley" mailto:da...@gnsa.us>>
>wrote:
>
>On Wed, Apr 10, 2013 at 7:08 AM, Harikrishna Patnala
>mailto:harikrishna.patn...@citrix.com>>
>wrote:
>Hi all,
>
>There are many global parameters which are used to set
>values/limits/boolean for various operations, but these parameters
>effects all zones/clusters/accounts/storage based on the parameter.
>Here I would like to discuss on granulising these parameters so that
>these parameters can be customised at different levels
>(zone/cluster/account/storage).
>New APIs are introduced to update these parameters based on the level
>listed in the FS below.
>During the creation of zone/cluster/account/storage default values for
>the granular parameters are set from the normal global parameters and
>later these can be updated using the corresponding API.
>
>
>This proposal for Global granular parameters is detailed in the FS
>here:
>https://cwiki.apache.org/confluence/display/CLOUDSTACK/FS+for+Granular+G
>l
>obal+Configuration+Parameters
>The JIRA ticket for tracking is
>https://issues.apache.org/jira/browse/CLOUDSTACK-741
>
>Please review this and provide me comments/suggestions.
>
>Thanks
>Harikrishna
>
>
>Hi Harikrisha:
>
>First - the title is a bit confusing; granular and global seems
>contradictory. Regardless - this is a great move forward.
>
>I don't understand why we would inject a new API command
>(update{storage,cluster,zone,account}levelparamater) instead of just
>using updateZone, updateAccount, updateStoragePool, etc. For instance,
>I would expect that listZone would tell me the status of the
>zone-level options. So why wouldn't updateZone be capable of setting
>the options
>
>
>Good question. Whether to overload an existing API or create a new one is
>always debatable.
>In this case one of the reason is that the existing update APIs were
>returning a {Zone, Account,..}Response that is not required when you
>change a granular config param. Moreover, all the existing update APIs are
>already heavily overloaded, not a strong reason to avoid further
>overloading though apart from that the response grows in size more you
>overload.
>
>We will also need an API to query the config params at these various
>granularities, that is not mentioned in the FS.
>
>-abhi
>
>
>
>
>