Review Request: NPE while migrating volume

2013-04-16 Thread Devdeep Singh

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

Review request for cloudstack and edison su.


Description
---

The state transitions on the volume that has to be migrated were incorrect. A 
volume to be migrated is in ready state and cannot be transitioned to copying 
state. Similarly, the duplicated volume is in ready state too and cannot be 
transitioned to creating state. Fixed it by transitioning the volume to 
migrating state when a migrateVolume api call is made. Also, a volume has to be 
destroyed first before it can be expunged. Fixed that too.


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


Diffs
-

  
engine/api/src/org/apache/cloudstack/engine/subsystem/api/storage/ObjectInDataStoreStateMachine.java
 f619ef4 
  
engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeObject.java
 ceadb25 
  
engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
 32e7d27 

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


Testing
---

1. migrated a volume from one storage pool to another and made sure it was 
successful. Attached it to a vm after migrating.
2. Also verified that the patch applies successfully.


Thanks,

Devdeep Singh



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

2013-04-16 Thread Prasanna Santhanam
Hey Kelven,

Will upgraded setups need to destroy/reboot console proxy VMs to have
the new encryption scheme take effect? Or this is entirely seamless?

Thanks,

-- 
Prasanna.,

On Tue, Apr 16, 2013 at 12:39:17AM +, Kelven Yang wrote:
> 
> ---
> 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: [DISCUSS] Granular Global Parameters

2013-04-16 Thread Abhinandan Prateek
On 16/04/13 12:14 PM, "Nitin Mehta"  wrote:

>Also as Mice asked do we plan to restart MS to update say config changes
>we make at zone/cluster level ?

That is how things are currently handled in MS. You need to restart MS
after any config change.

>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.


Here we are talking about dynamically updating the config and performance.

Dynamically updating the config requires a huge volume of change though
they are not complex in nature.

Performance I will not worry about that much as config updates do not
happen frequently and config is not read often.
As of now most of the config is read when the MS starts.

The granularity of parameter that the current spec covers is addressing a
different issue. So my hunch will be that we will be
Better off putting a separate spec to address the two other issues of
performance and dynamic update. Probably work can also go in
Parallel.

-abhi


> 
>
>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 

Latest master is failing at (apidocs)

2013-04-16 Thread Rajesh Battala
Hi All,
Am trying to build the latest master its failing at apidocs.

Check, is this api part of another build profile? Null value for key: 
addBaremetalHost preProcessedCommand=1
Traceback (most recent call last):
  File "/home/rajesh/hc_fzf/tools/apidoc/gen_toc.py", line 173, in 
category = choose_category(fn)
  File "/home/rajesh/hc_fzf/tools/apidoc/gen_toc.py", line 153, in 
choose_category
(fn, __file__))
Exception: Need to add a category for deleteCiscoAsa1000vResource.xml to 
/home/rajesh/hc_fzf/tools/apidoc/gen_toc.py:known_categories


Please let me know how to resolve this error?

Thanks
Rajesh   Battala


Re: Latest master is failing at (apidocs)

2013-04-16 Thread prasanna
Add a dictionary entry to tools/apidoc/gen_toc.py with 'Cisco' as the key

On 16 April 2013 13:26, Rajesh Battala  wrote:
> Hi All,
> Am trying to build the latest master its failing at apidocs.
>
> Check, is this api part of another build profile? Null value for key: 
> addBaremetalHost preProcessedCommand=1
> Traceback (most recent call last):
>   File "/home/rajesh/hc_fzf/tools/apidoc/gen_toc.py", line 173, in 
> category = choose_category(fn)
>   File "/home/rajesh/hc_fzf/tools/apidoc/gen_toc.py", line 153, in 
> choose_category
> (fn, __file__))
> Exception: Need to add a category for deleteCiscoAsa1000vResource.xml to 
> /home/rajesh/hc_fzf/tools/apidoc/gen_toc.py:known_categories
>
>
> Please let me know how to resolve this error?
>
> Thanks
> Rajesh   Battala


API naming conventions

2013-04-16 Thread Prasanna Santhanam
Is there a formal document somewhere describing the naming conventions
for our APIs? If not is it worthwhile starting one?

The document should describe:
1. valid action (verbs) to operate on an entity
2. entity being the desired resource, physical device, cloudstack
account etc.
3. what semantics an update API can take and what a configure API can
take etc.

Thanks,

-- 
Prasanna.,


Re: strange output on cs4.0.1 gui when restarted

2013-04-16 Thread Sebastien Goasguen
Hi Jason, did you get passed this ?

I have never had this issue. Did you select a special language ? Did you try on 
a different browser ?

-Sebastien

On Apr 9, 2013, at 7:49 PM, Jason Pavao  wrote:

> Hey Folks,
> I am seeing a really strange issue, after a reboot, I am seeing tons of % and 
> other characters throughout the display. An example-
> 
> 
> Anyone have any ideas as to why this happened and how to correct it? Has 
> anyone had this issue before?
> I appreciate any help.
> Thanks.
> -jason
> 
> 
> -- 
> Thanks.
> -Jason



inheritable componentContexts

2013-04-16 Thread Prasanna Santhanam
This is probably a question for the Spring community but I'll ask here
anyway. Every so often when the componentContext.xml.in changes I find
that I have to fix up simulatorComponentContext.xml.in similarly to
make the beans load appropriately and for the bvt tests to work.

Is there a way similar to the components-premium.xml we had before to
'inherit' Spring contexts so it's easier to maintain the xmls?

-- 
Prasanna.,


RE: Firewall and LB network elements in inline mode

2013-04-16 Thread Koushik Das
Thanks Chiradeep, Sheng for the info.

As I understand from the FS and also going over the code changes, only the LB 
element needs to modified to support inline mode and then the FW and LB inline 
scenario would be enabled on creating a network offering with inline mode 
enabled. There is no additional requirement on the FW element.
Sheng, can you confirm if this is correct?

> -Original Message-
> From: Sheng Yang [mailto:sh...@yasker.org]
> Sent: Tuesday, April 16, 2013 5:44 AM
> To: 
> Subject: Re: Firewall and LB network elements in inline mode
> 
> 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: [DOCS] Please review: VM Snapshots section (CLOUDSTACK-1561)

2013-04-16 Thread Mice Xia
Jessica,

Thanks for your work, the content looks fine to me; 
Whether this feature is enabled for Xenserver(and KVM) depends on other tasks' 
progress, and the content to be modified/added is not much if we finally decide 
to support them.

Chandan, 
Could you update test progress of Xenserver? We might need to propose a VOTE 
later to decide if Xenserver(and KVM) support are included in 4.2.

Regards
Mice

-Original Message-
From: Jessica Tomechak [mailto:jessica.tomec...@gmail.com] 
Sent: Tuesday, April 16, 2013 8:28 AM
To: dev@cloudstack.apache.org
Subject: [DOCS] Please review: VM Snapshots section (CLOUDSTACK-1561)

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.


Re: SSVM sets Management Server as localhost

2013-04-16 Thread Nitin Mehta
Yes, this is indeed a bug. Can you please file it against JIRA ?

During the first boot MS tries to run the code below, but for some of the
dev environments it finds  hostIpAdr as null and so persists the default
value which is localhost.
if (hostIpAdr != null) {
_configDao.update(Config.ManagementHostIPAdr.key(),
Config.ManagementHostIPAdr.getCategory(), hostIpAdr);
s_logger.debug("ConfigurationServer saved \"" + hostIpAdr
+ "\" as host.");
}


I guess the way it retrieves the hostIpAdr by running a bunch of linux
commands doesn't work for all the OS versions (like for mine). That needs
to be fixed 

On 15/04/13 10:35 PM, "Abhinandan Prateek"  wrote:

>In configuration table there is a entry by name "host" set it to MS ip and
>restart.
>You will need to recreate SSVM.
>
>The ssvm-check.sh should be around, probably a bug ? Can someone confirm.
>
>-abhi
>
>On 15/04/13 10:16 PM, "Adrien Montfort"  wrote:
>
>>Hello,
>>
>>My SSVM does not seem to work. I can't upload any template/iso.
>>I've been investigating a bit and I've found that in my SSVM
>>configuration
>>(/var/cache/cloud/cmdline) there's host=localhost which does not seem to
>>be
>>correct... How did this happen ? How can I overwrite it (my change are
>>erased by reboot) ?
>>As a side note, I could not find the script ssvm-check.sh in
>>/usr/local/cloud/systemvm/ssvm-check.sh
>>and anywhere on the VM. Is it expected ?
>>
>>Thanks a lot,
>>
>>Regards,
>>
>>Adrien
>
>



[ACS42][QA]Issues with latest Master Build

2013-04-16 Thread Sailaja Mada
Hi,

My observations with latest master build.


1)  System VM's are coming up but Default template is not got downloaded

2)  Can not view Console

3)  From XenCenter, Tried to login to system VM's with the predefined login 
credentials . But it failed to login. I am using new system templates.  Is 
there any change in behavior?

Has any one observed the similar issues?

Thanks,
Sailaja.M


[DISCUSS] labels for cloudstack university and GSoC

2013-04-16 Thread Sebastien Goasguen
Hi folks,

Some of you may have seen:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+University

I also emailed about the Google Summer of Code.

Kelcey mentioned creating an edu mailing list.

I have not particular feelings about this and I wanted to discuss it.

GSoC will be across dev and users. Students are expected to join the community, 
work on JIRA and submitted patches etc..
ACSEDU will be across dev/users and marketing.

Instead of creating a list maybe we can just use tags/labels in emails:

[GSOC] and [ACSEDU]

The only issue I see is that there will be information that is relevant to all 
lists and I don't want to leave anyone out.

Thoughts ?

-Sebastien

RE: [ACS42][QA]Issues with latest Master Build (Xenserver)

2013-04-16 Thread Sailaja Mada

Hi,

My observations with latest master build using Xenserver hyperviosor.

1)  System VM's are coming up but Default template is not got downloaded  - 
 There is a fix for https://issues.apache.org/jira/browse/CLOUDSTACK-1994 - 
Trying with newly generated System VM template

2)  Can not view Console  - Will recheck with new template

3)  From XenCenter, Tried to login to system VM's with the predefined login 
credentials . But it failed to login. I am using new system templates.  Is 
there any change in behavior?  -  Got the resoponse in IRC - password for 
system VM's got changed to password now


Thanks,
Sailaja.M


RE: [DISCUSS] Granular Global Parameters

2013-04-16 Thread Mice Xia
Not meant to stray from the topic, I raised this question because I think this 
is a good opportunity to refactor global configs.

Yes, they are two separate issues, but need think it through before work in 
parallel otherwise there will be many code conflicts..

+1 for Nitin's proposal:
Firstly introduce an interface and encapsulate the logic of fetching configs 
depends on context (zone/pod/cluster/account/..)
Then task like implementation of the interface, replacing class _variables and 
rewrite some daemon threads can work in parallel (hopefully)

Regards
Mice

-Original Message-
From: Abhinandan Prateek [mailto:cloudst...@aprateek.com] 
Sent: Tuesday, April 16, 2013 3:17 PM
To: dev@cloudstack.apache.org
Subject: Re: [DISCUSS] Granular Global Parameters

On 16/04/13 12:14 PM, "Nitin Mehta"  wrote:

>Also as Mice asked do we plan to restart MS to update say config 
>changes we make at zone/cluster level ?

That is how things are currently handled in MS. You need to restart MS after 
any config change.

>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.


Here we are talking about dynamically updating the config and performance.

Dynamically updating the config requires a huge volume of change though they 
are not complex in nature.

Performance I will not worry about that much as config updates do not happen 
frequently and config is not read often.
As of now most of the config is read when the MS starts.

The granularity of parameter that the current spec covers is addressing a 
different issue. So my hunch will be that we will be Better off putting a 
separate spec to address the two other issues of performance and dynamic 
update. Probably work can also go in Parallel.

-abhi


> 
>
>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+Gra

Re: [DISCUSS] Granular Global Parameters

2013-04-16 Thread Abhinandan Prateek


On 16/04/13 2:58 PM, "Mice Xia"  wrote:

>Not meant to stray from the topic, I raised this question because I think
>this is a good opportunity to refactor global configs.
>
>Yes, they are two separate issues, but need think it through before work
>in parallel otherwise there will be many code conflicts..
>
>+1 for Nitin's proposal:
>Firstly introduce an interface and encapsulate the logic of fetching
>configs depends on context (zone/pod/cluster/account/..)

+1, Hari has made some suggestions, I guess we need to review those and
see if they fit in the overall scheme of things.


>Then task like implementation of the interface, replacing class
>_variables and rewrite some daemon threads can work in parallel
>(hopefully)

+1 here too. Lets work out the details and take it from there.


>
>Regards
>Mice
>
>-Original Message-
>From: Abhinandan Prateek [mailto:cloudst...@aprateek.com]
>Sent: Tuesday, April 16, 2013 3:17 PM
>To: dev@cloudstack.apache.org
>Subject: Re: [DISCUSS] Granular Global Parameters
>
>On 16/04/13 12:14 PM, "Nitin Mehta"  wrote:
>
>>Also as Mice asked do we plan to restart MS to update say config
>>changes we make at zone/cluster level ?
>
>That is how things are currently handled in MS. You need to restart MS
>after any config change.
>
>>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.
>
>
>Here we are talking about dynamically updating the config and performance.
>
>Dynamically updating the config requires a huge volume of change though
>they are not complex in nature.
>
>Performance I will not worry about that much as config updates do not
>happen frequently and config is not read often.
>As of now most of the config is read when the MS starts.
>
>The granularity of parameter that the current spec covers is addressing a
>different issue. So my hunch will be that we will be Better off putting a
>separate spec to address the two other issues of performance and dynamic
>update. Probably work can also go in Parallel.
>
>-abhi
>
>
>> 
>>
>>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 cr

Re: API naming conventions

2013-04-16 Thread Nitin Mehta
+1 to this, but I guess this should be a subsection of a wiki called
"Adding a new api in Cloudstack - What all should I do ?"

Some of the subsections in it should be like (each with an example) :-

API naming conventions
Should the api be sync, async or asynccreate and what class should it
extend

API annotations and what they mean. Discuss ACL and parameters. Events to
write.
What to put in the API layer, manager layer, hypervisor layer
API Response - should I create a view (in case its a list command)
Testing expectations - Adding marvin tests

I guess this would be very useful to the devs, guys trying to understand
the flow. So lets create one consolidated wiki and add these subsections.

Thanks,
-Nitin

On 16/04/13 1:31 PM, "Prasanna Santhanam"  wrote:

>Is there a formal document somewhere describing the naming conventions
>for our APIs? If not is it worthwhile starting one?
>
>The document should describe:
>1. valid action (verbs) to operate on an entity
>2. entity being the desired resource, physical device, cloudstack
>account etc.
>3. what semantics an update API can take and what a configure API can
>take etc.
>
>Thanks,
>
>-- 
>Prasanna.,



RE: [ACS42][QA]Issues with latest Master Build (Xenserver)

2013-04-16 Thread Sailaja Mada
Tried with newly generated xenserver template. Templates got downloaded and 
Console view (Firefox) also worked fine .

Thanks,
Sailaja.M

-Original Message-
From: Sailaja Mada [mailto:sailaja.m...@citrix.com] 
Sent: Tuesday, April 16, 2013 2:47 PM
To: dev@cloudstack.apache.org
Subject: RE: [ACS42][QA]Issues with latest Master Build (Xenserver)


Hi,

My observations with latest master build using Xenserver hyperviosor.

1)  System VM's are coming up but Default template is not got downloaded  - 
 There is a fix for https://issues.apache.org/jira/browse/CLOUDSTACK-1994 - 
Trying with newly generated System VM template

2)  Can not view Console  - Will recheck with new template

3)  From XenCenter, Tried to login to system VM's with the predefined login 
credentials . But it failed to login. I am using new system templates.  Is 
there any change in behavior?  -  Got the resoponse in IRC - password for 
system VM's got changed to password now


Thanks,
Sailaja.M


Re: API naming conventions

2013-04-16 Thread Prasanna Santhanam
Oh that's more than I intend to chew :)

I only want the APIs to have some defined pattern - naming and
semantics. This if from a integration perspective than from the
perspective of a developer of cloudstack.

There is currently no documentation on what I should name my API and
how I choose b/w overloading an API vs creating a new API:

1. ldapConfig or configLdap?
2. addToLoadBalancerRule/removeFromLoadBalancerRule or simply
addLoadBalancerRule and removeLoadBalancerRule with overloaded
arguments for specifying the rule itself.

-- 
Prasanna.,

On Tue, Apr 16, 2013 at 10:40:44AM +, Nitin Mehta wrote:
> +1 to this, but I guess this should be a subsection of a wiki called
> "Adding a new api in Cloudstack - What all should I do ?"
> 
> Some of the subsections in it should be like (each with an example) :-
> 
> API naming conventions
> Should the api be sync, async or asynccreate and what class should it
> extend
> 
> API annotations and what they mean. Discuss ACL and parameters. Events to
> write.
> What to put in the API layer, manager layer, hypervisor layer
> API Response - should I create a view (in case its a list command)
> Testing expectations - Adding marvin tests
> 
> I guess this would be very useful to the devs, guys trying to understand
> the flow. So lets create one consolidated wiki and add these subsections.
> 
> Thanks,
> -Nitin
> 
> On 16/04/13 1:31 PM, "Prasanna Santhanam"  wrote:
> 
> >Is there a formal document somewhere describing the naming conventions
> >for our APIs? If not is it worthwhile starting one?
> >
> >The document should describe:
> >1. valid action (verbs) to operate on an entity
> >2. entity being the desired resource, physical device, cloudstack
> >account etc.
> >3. what semantics an update API can take and what a configure API can
> >take etc.
> >
> >Thanks,
> >
> >-- 
> >Prasanna.,



Review Request: CLOUDSTACK-1966 Primary Storage Limits are not validated while deploying the VM

2013-04-16 Thread Sanjay Tripathi

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

Review request for cloudstack, Devdeep Singh, Sateesh Chodapuneedi, and Min 
Chen.


Description
---

CLOUDSTACK-1966 Primary Storage Limits are not validated while deploying the VM.


This addresses bug CLOUDSTACK-1966.


Diffs
-

  server/src/com/cloud/vm/UserVmManagerImpl.java d281e5b 

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


Testing
---

Tests:
1. Create a user account.
2. Set the primary storage limit for this user account to 2GiB.
3. Log into the user account.
4. Deploy instance with template size > 2GiB
5. CloudStack displays the appropriate error message.


Thanks,

Sanjay Tripathi



RE: API naming conventions

2013-04-16 Thread Koushik Das
A fixed naming pattern with verb followed by noun looks good. If required the 
verbs can be categorized further like lifecycle verbs - create, delete etc.

> -Original Message-
> From: prasanna [mailto:srivatsav.prasa...@gmail.com] On Behalf Of
> Prasanna Santhanam
> Sent: Tuesday, April 16, 2013 4:34 PM
> To: dev@cloudstack.apache.org
> Subject: Re: API naming conventions
> 
> Oh that's more than I intend to chew :)
> 
> I only want the APIs to have some defined pattern - naming and semantics.
> This if from a integration perspective than from the perspective of a
> developer of cloudstack.
> 
> There is currently no documentation on what I should name my API and how
> I choose b/w overloading an API vs creating a new API:
> 
> 1. ldapConfig or configLdap?
> 2. addToLoadBalancerRule/removeFromLoadBalancerRule or simply
> addLoadBalancerRule and removeLoadBalancerRule with overloaded
> arguments for specifying the rule itself.
> 
> --
> Prasanna.,
> 
> On Tue, Apr 16, 2013 at 10:40:44AM +, Nitin Mehta wrote:
> > +1 to this, but I guess this should be a subsection of a wiki called
> > "Adding a new api in Cloudstack - What all should I do ?"
> >
> > Some of the subsections in it should be like (each with an example) :-
> >
> > API naming conventions
> > Should the api be sync, async or asynccreate and what class should it
> > extend
> >
> > API annotations and what they mean. Discuss ACL and parameters. Events
> > to write.
> > What to put in the API layer, manager layer, hypervisor layer API
> > Response - should I create a view (in case its a list command) Testing
> > expectations - Adding marvin tests
> >
> > I guess this would be very useful to the devs, guys trying to
> > understand the flow. So lets create one consolidated wiki and add these
> subsections.
> >
> > Thanks,
> > -Nitin
> >
> > On 16/04/13 1:31 PM, "Prasanna Santhanam"  wrote:
> >
> > >Is there a formal document somewhere describing the naming
> > >conventions for our APIs? If not is it worthwhile starting one?
> > >
> > >The document should describe:
> > >1. valid action (verbs) to operate on an entity 2. entity being the
> > >desired resource, physical device, cloudstack account etc.
> > >3. what semantics an update API can take and what a configure API can
> > >take etc.
> > >
> > >Thanks,
> > >
> > >--
> > >Prasanna.,



Re: SSVM sets Management Server as localhost

2013-04-16 Thread Adrien Montfort
Here is the JIRA issue :
https://issues.apache.org/jira/browse/CLOUDSTACK-2047

Regarding my other issue (/usr/local/cloud is empty) it was fixed with
apt-get install cloud-system-iso on the KVM host (as mentionned in a
previous 
email
).

Thanks guys.


On Tue, Apr 16, 2013 at 10:37 AM, Nitin Mehta wrote:

> Yes, this is indeed a bug. Can you please file it against JIRA ?
>
> During the first boot MS tries to run the code below, but for some of the
> dev environments it finds  hostIpAdr as null and so persists the default
> value which is localhost.
> if (hostIpAdr != null) {
> _configDao.update(Config.ManagementHostIPAdr.key(),
> Config.ManagementHostIPAdr.getCategory(), hostIpAdr);
> s_logger.debug("ConfigurationServer saved \"" + hostIpAdr
> + "\" as host.");
> }
>
>
> I guess the way it retrieves the hostIpAdr by running a bunch of linux
> commands doesn't work for all the OS versions (like for mine). That needs
> to be fixed
>
> On 15/04/13 10:35 PM, "Abhinandan Prateek" 
> wrote:
>
> >In configuration table there is a entry by name "host" set it to MS ip and
> >restart.
> >You will need to recreate SSVM.
> >
> >The ssvm-check.sh should be around, probably a bug ? Can someone confirm.
> >
> >-abhi
> >
> >On 15/04/13 10:16 PM, "Adrien Montfort" 
> wrote:
> >
> >>Hello,
> >>
> >>My SSVM does not seem to work. I can't upload any template/iso.
> >>I've been investigating a bit and I've found that in my SSVM
> >>configuration
> >>(/var/cache/cloud/cmdline) there's host=localhost which does not seem to
> >>be
> >>correct... How did this happen ? How can I overwrite it (my change are
> >>erased by reboot) ?
> >>As a side note, I could not find the script ssvm-check.sh in
> >>/usr/local/cloud/systemvm/ssvm-check.sh
> >>and anywhere on the VM. Is it expected ?
> >>
> >>Thanks a lot,
> >>
> >>Regards,
> >>
> >>Adrien
> >
> >
>
>


[DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Sebastien Goasguen
Hi,

There has been some terrific progress on the translations.

The easiest way to share it is to check the Transifex dashboard, below are some 
screenshots:

The runbook:
http://awesomescreenshot.com/0bd163zldd

The UI:
http://awesomescreenshot.com/089163zva2

The entire docs set:
http://awesomescreenshot.com/01c163zq63

Since 4.1 is not out yet, there is time to finish those last 3% on the UI 
translations.

Keep getting the word out, forward to friends and let's all make CloudStack a 
truly worldwide project.

Cheers,

-Sebastien

Re: API naming conventions

2013-04-16 Thread Rohit Yadav
Use what we have been using so far:

API := :

lowecase action := 
CamelCaseSubjects := 

So, based on such rule; the following makes sense:
- configLdap

About subjects, think what the API is trying to do. For example;

addToLoadBalancerRule does not make sense, what are we adding to
LoadBalancerRule, probably a rule right? So, in that
case addLoadBalancerRule makes sense.

Cheers.

On Tue, Apr 16, 2013 at 4:33 PM, Prasanna Santhanam  wrote:

> Oh that's more than I intend to chew :)
>
> I only want the APIs to have some defined pattern - naming and
> semantics. This if from a integration perspective than from the
> perspective of a developer of cloudstack.
>
> There is currently no documentation on what I should name my API and
> how I choose b/w overloading an API vs creating a new API:
>
> 1. ldapConfig or configLdap?
> 2. addToLoadBalancerRule/removeFromLoadBalancerRule or simply
> addLoadBalancerRule and removeLoadBalancerRule with overloaded
> arguments for specifying the rule itself.
>
> --
> Prasanna.,
>
> On Tue, Apr 16, 2013 at 10:40:44AM +, Nitin Mehta wrote:
> > +1 to this, but I guess this should be a subsection of a wiki called
> > "Adding a new api in Cloudstack - What all should I do ?"
> >
> > Some of the subsections in it should be like (each with an example) :-
> >
> > API naming conventions
> > Should the api be sync, async or asynccreate and what class should it
> > extend
> >
> > API annotations and what they mean. Discuss ACL and parameters. Events to
> > write.
> > What to put in the API layer, manager layer, hypervisor layer
> > API Response - should I create a view (in case its a list command)
> > Testing expectations - Adding marvin tests
> >
> > I guess this would be very useful to the devs, guys trying to understand
> > the flow. So lets create one consolidated wiki and add these subsections.
> >
> > Thanks,
> > -Nitin
> >
> > On 16/04/13 1:31 PM, "Prasanna Santhanam"  wrote:
> >
> > >Is there a formal document somewhere describing the naming conventions
> > >for our APIs? If not is it worthwhile starting one?
> > >
> > >The document should describe:
> > >1. valid action (verbs) to operate on an entity
> > >2. entity being the desired resource, physical device, cloudstack
> > >account etc.
> > >3. what semantics an update API can take and what a configure API can
> > >take etc.
> > >
> > >Thanks,
> > >
> > >--
> > >Prasanna.,
>
>


Re: Cloudstack.org domain

2013-04-16 Thread Noah Slater
I'm not sure how useful this is to you, but I just started a thread on
infrastruct...@apache.org with the subject "ASF DNS pointing to non-ASF
machines". It happens that I am involved with another project in a similar
predicament, and I would like to get some additional clarity on the policy.
If there's anything that seems relevant  I will loop back to this thread.


On 10 April 2013 07:55, Prasanna Santhanam  wrote:

> Time to find an alternate domain for jenkins then. :(
>
> On Tue, Apr 09, 2013 at 05:40:56PM +0100, Noah Slater wrote:
> > Ack :(
> >
> >
> > On 9 April 2013 17:38, David Nalley  wrote:
> >
> > > It was on infra-private as that was where I was told to carry the
> > > conversation when I asked in IRC.
> > > I don't think that there is any negotiation to be done, multiple infra
> > > contractors over a period of almost a year have echoed similar
> > > sentiments.
> > >
> > > --David
> > >
> > > On Tue, Apr 9, 2013 at 12:35 PM, Noah Slater 
> wrote:
> > > > Strange this was on infra-private, and not infra (which is also
> > > private). I
> > > > only subscribe to the latter. I found your message now.
> Disappointing. I
> > > > had not realised that this was policy. (I guess undocumented.)
> Wonder if
> > > > there's any room for negotiation...
> > > >
> > > >
> > > > On 9 April 2013 17:21, David Nalley  wrote:
> > > >
> > > >> On Tue, Apr 9, 2013 at 11:50 AM, Noah Slater 
> > > wrote:
> > > >> > Do you have a link to the discussion thread, David? I can't find
> > > anything
> > > >> > in my mail.
> > > >> >
> > > >>
> > > >> You'll need to use your member karma, as it's on infra-private:
> > > >> 097901ce30c3$c69ff210$53dfd630$@16degrees.com.au
> > > >>
> > > >
> > > >
> > > >
> > > > --
> > > > NS
> > >
> >
> >
> >
> > --
> > NS
>
> --
> Prasanna.,
>



-- 
NS


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

2013-04-16 Thread Jayapal Reddy Uradi
FS API section is updated.
Source NAT on the private gateway  can be enabled only while adding the private 
gateway.
when the private gateway is deleted source nat on private gateway deleted.
 
Please review the FS and provide your comments.

Thanks,
Jayapal

On 15-Apr-2013, at 3:38 PM, Jayapal Reddy Uradi  
wrote:

> 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: Cloudstack.org domain

2013-04-16 Thread Noah Slater
Okay, some success.

Here is a mail sent to infrastruct...@apache.org:

On 16 April 2013 13:44, Tony Stevenson  wrote:

> Noah Slater wrote on Tue, Apr 16, 2013 at 01:33:39PM +0100:
> > Hi,
> >
> > It recently came up on the CloudStack list that if cloudstack.org moves
> > over to ASF control, we would not be able to point foo.cloudstack.org at
> an
> > external VM. (Say, a committer is running a CI/build/whatever service.)
> Is
> > this the case? Are there any exceptions to this?
> >
> > I ask not because I want to drag up old discussions for the sake of it,
> but
> > because we have some ongoing trademark stuff with CouchDB, and one of the
> > things I am planning to propose is that we move couchdb.org to ASF
> control.
> >
> > But if the above is policy, that wouldn't work.
> > http://docs.couchdb.org/points at a third-party hosted service, for
> > instance. And we are working on
> > a Jenkins setup that is not on ASF hardware (because the ASF is unable to
> > provide us with the hardware we require at this time).
> >
> > Could someone clarify this for me?
>
> Noah we have several instances of DNS records pointing to non-ASF hosted
> services.  We would clearly very much prefer that we didnt have to do
> this, but I do not believe it is against policy.
>
> Our issue comes from the fact the machine is not under our control, and
> as such anyone could just add $bad-content within the apache.org
> namespace.  At the moment, from memory, all records that point to
> non-ASF hardware, are still machines we control.  Your asking for
> something above and beyond this, and I suspect if this is what the PMC
> wants then that would be done.  But it would need to be requested,
> following a vote thread and that link given to us so we can see the
> process :)
>

This is from a private list, but I have been granted permission to share it
with this one.

For those with the appropriate karma, the thread starts here:

http://s.apache.org/ZLE

So, if we put together a definitive plan of action for the domain, and the
services we want to host / point to, along with who is operating them, and
what the plan for that is, and vote on it, it looks like we can proceed.
(But we should be mindful of Infra's concerns, and take them into account
while drafting the proposal.)


On 16 April 2013 13:34, Noah Slater  wrote:

> I'm not sure how useful this is to you, but I just started a thread on
> infrastruct...@apache.org with the subject "ASF DNS pointing to non-ASF
> machines". It happens that I am involved with another project in a similar
> predicament, and I would like to get some additional clarity on the policy.
> If there's anything that seems relevant  I will loop back to this thread.
>
>
> On 10 April 2013 07:55, Prasanna Santhanam  wrote:
>
>> Time to find an alternate domain for jenkins then. :(
>>
>> On Tue, Apr 09, 2013 at 05:40:56PM +0100, Noah Slater wrote:
>> > Ack :(
>> >
>> >
>> > On 9 April 2013 17:38, David Nalley  wrote:
>> >
>> > > It was on infra-private as that was where I was told to carry the
>> > > conversation when I asked in IRC.
>> > > I don't think that there is any negotiation to be done, multiple infra
>> > > contractors over a period of almost a year have echoed similar
>> > > sentiments.
>> > >
>> > > --David
>> > >
>> > > On Tue, Apr 9, 2013 at 12:35 PM, Noah Slater 
>> wrote:
>> > > > Strange this was on infra-private, and not infra (which is also
>> > > private). I
>> > > > only subscribe to the latter. I found your message now.
>> Disappointing. I
>> > > > had not realised that this was policy. (I guess undocumented.)
>> Wonder if
>> > > > there's any room for negotiation...
>> > > >
>> > > >
>> > > > On 9 April 2013 17:21, David Nalley  wrote:
>> > > >
>> > > >> On Tue, Apr 9, 2013 at 11:50 AM, Noah Slater 
>> > > wrote:
>> > > >> > Do you have a link to the discussion thread, David? I can't find
>> > > anything
>> > > >> > in my mail.
>> > > >> >
>> > > >>
>> > > >> You'll need to use your member karma, as it's on infra-private:
>> > > >> 097901ce30c3$c69ff210$53dfd630$@16degrees.com.au
>> > > >>
>> > > >
>> > > >
>> > > >
>> > > > --
>> > > > NS
>> > >
>> >
>> >
>> >
>> > --
>> > NS
>>
>> --
>> Prasanna.,
>>
>
>
>
> --
> NS
>



-- 
NS


Re: [DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Gavin Lee
Yes, Traditional Chinese moving very quickly.
Hopefully, the other languages can have more contributors.

For the UI part, I saw the characters are not recognizable (browser
encoding setting: auto detect & Unicode UTF-8):
ja: http://snag.gy/AVsbU.jpg
zh_CN: http://snag.gy/MxbBS.jpg

The other languages are ok.


On Tue, Apr 16, 2013 at 8:25 PM, Sebastien Goasguen wrote:

> Hi,
>
> There has been some terrific progress on the translations.
>
> The easiest way to share it is to check the Transifex dashboard, below are
> some screenshots:
>
> The runbook:
> http://awesomescreenshot.com/0bd163zldd
>
> The UI:
> http://awesomescreenshot.com/089163zva2
>
> The entire docs set:
> http://awesomescreenshot.com/01c163zq63
>
> Since 4.1 is not out yet, there is time to finish those last 3% on the UI
> translations.
>
> Keep getting the word out, forward to friends and let's all make
> CloudStack a truly worldwide project.
>
> Cheers,
>
> -Sebastien




-- 
Gavin


database creator

2013-04-16 Thread Abhinandan Prateek

I have some queries regarding Database Creator:

Can this feature be tested on 4.1 ?

Could someone also provide more details on the following:
 
1. Outline the exact steps that are involved in rolling upgrade procedure?
2. Can you confirm if rolling upgrades are specific to only upgrade
procedure involving multiple management servers in a cluster?
3. Would rolling upgrades mean that there will be zero downtime for
customers when upgrading? Are we also dealing with NOT having to restart all
system Vms ? Currently restarting system Vms is part of our upgrade
procedure.

-abhi




Re: [DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Sebastien Goasguen

On Apr 16, 2013, at 9:41 AM, Gavin Lee  wrote:

> Yes, Traditional Chinese moving very quickly.
> Hopefully, the other languages can have more contributors.
> 
> For the UI part, I saw the characters are not recognizable (browser
> encoding setting: auto detect & Unicode UTF-8):
> ja: http://snag.gy/AVsbU.jpg
> zh_CN: http://snag.gy/MxbBS.jpg

Hum, we fixed some encodings so that it would work in the UI. Have you tried to 
build the 4.1 branch and tested the UI itself, rather than transifex.

Adding Milamber has he masters encoding for l10n.

-sebastien

> 
> The other languages are ok.
> 
> 
> On Tue, Apr 16, 2013 at 8:25 PM, Sebastien Goasguen wrote:
> 
>> Hi,
>> 
>> There has been some terrific progress on the translations.
>> 
>> The easiest way to share it is to check the Transifex dashboard, below are
>> some screenshots:
>> 
>> The runbook:
>> http://awesomescreenshot.com/0bd163zldd
>> 
>> The UI:
>> http://awesomescreenshot.com/089163zva2
>> 
>> The entire docs set:
>> http://awesomescreenshot.com/01c163zq63
>> 
>> Since 4.1 is not out yet, there is time to finish those last 3% on the UI
>> translations.
>> 
>> Keep getting the word out, forward to friends and let's all make
>> CloudStack a truly worldwide project.
>> 
>> Cheers,
>> 
>> -Sebastien
> 
> 
> 
> 
> -- 
> Gavin



Review Request: CLOUDSTACK-741: Granular Global Parameters

2013-04-16 Thread Harikrishna Patnala

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

Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin 
Mehta.


Description
---

CLOUDSTACK-741: Granular Global Parameters

Modified UpdateConfiguration API and ListConfiguration API by adding additional 
parameter scope and resource id (zone/cluster/pool/account).
Scope decides the parameter which level we can update the value and id is the 
corresponding id based on the scope.
This scope is defined in the config.java file where we are creating groups for 
each level(zone/cluster/pool/account).
Currently submitted the patch for the zone level parameter use.external.dns.

Included unit tests and marvin tests.


This addresses bug CLOUDSTACK-741.


Diffs
-

  api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java 
aabfd4a 
  api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java 
ffeb586 
  api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java 
PRE-CREATION 
  api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java 
PRE-CREATION 
  server/src/com/cloud/configuration/Config.java e69ea19 
  server/src/com/cloud/configuration/ConfigurationManager.java d7faf19 
  server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 
  server/src/com/cloud/configuration/ConfigurationParameterScope.java 
PRE-CREATION 
  server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 
  server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
ab91059 
  server/src/com/cloud/server/ConfigurationServer.java f441b1f 
  server/src/com/cloud/server/ConfigurationServerImpl.java 852c00b 
  server/src/com/cloud/server/ManagementServerImpl.java 5ed0c92 
  server/test/com/cloud/vpc/MockConfigurationManagerImpl.java a03e361 
  test/integration/smoke/test_UpdateCfg.py PRE-CREATION 

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


Testing
---

Tested locally
- tested the modified UpdateConfiguration API and ListConfiguration API by 
providing the zone scope and zone id
- tested the modified parameter to take the updated value at zone level(after 
updation).


Thanks,

Harikrishna Patnala



Re: Review Request: CLOUDSTACK-741: Granular Global Parameters

2013-04-16 Thread Harikrishna Patnala

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

(Updated April 16, 2013, 2:26 p.m.)


Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin 
Mehta.


Description
---

CLOUDSTACK-741: Granular Global Parameters

Modified UpdateConfiguration API and ListConfiguration API by adding additional 
parameter scope and resource id (zone/cluster/pool/account).
Scope decides the parameter which level we can update the value and id is the 
corresponding id based on the scope.
This scope is defined in the config.java file where we are creating groups for 
each level(zone/cluster/pool/account).
Currently submitted the patch for the zone level parameter use.external.dns.

Included unit tests and marvin tests.


This addresses bug CLOUDSTACK-741.


Diffs
-

  api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java 
aabfd4a 
  api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java 
ffeb586 
  api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java 
PRE-CREATION 
  api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java 
PRE-CREATION 
  server/src/com/cloud/configuration/Config.java e69ea19 
  server/src/com/cloud/configuration/ConfigurationManager.java d7faf19 
  server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 
  server/src/com/cloud/configuration/ConfigurationParameterScope.java 
PRE-CREATION 
  server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 
  server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
ab91059 
  server/src/com/cloud/server/ConfigurationServer.java f441b1f 
  server/src/com/cloud/server/ConfigurationServerImpl.java 852c00b 
  server/src/com/cloud/server/ManagementServerImpl.java 5ed0c92 
  server/test/com/cloud/vpc/MockConfigurationManagerImpl.java a03e361 
  test/integration/smoke/test_UpdateCfg.py PRE-CREATION 

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


Testing
---

Tested locally
- tested the modified UpdateConfiguration API and ListConfiguration API by 
providing the zone scope and zone id
- tested the modified parameter to take the updated value at zone level(after 
updation).


Thanks,

Harikrishna Patnala



Re: [DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Gavin Lee
The UI for 4.1 branch is ok, there is one case relate to the encoding issue:
https://issues.apache.org/jira/browse/CLOUDSTACK-870

Since messages_zh_CN.properties and messages_ja.properties file under resource
folder are not human readable, how to translate or modify the content?


On Tue, Apr 16, 2013 at 10:05 PM, Sebastien Goasguen wrote:

>
> On Apr 16, 2013, at 9:41 AM, Gavin Lee  wrote:
>
> > Yes, Traditional Chinese moving very quickly.
> > Hopefully, the other languages can have more contributors.
> >
> > For the UI part, I saw the characters are not recognizable (browser
> > encoding setting: auto detect & Unicode UTF-8):
> > ja: http://snag.gy/AVsbU.jpg
> > zh_CN: http://snag.gy/MxbBS.jpg
>
> Hum, we fixed some encodings so that it would work in the UI. Have you
> tried to build the 4.1 branch and tested the UI itself, rather than
> transifex.
>
> Adding Milamber has he masters encoding for l10n.
>
> -sebastien
>
> >
> > The other languages are ok.
> >
> >
> > On Tue, Apr 16, 2013 at 8:25 PM, Sebastien Goasguen  >wrote:
> >
> >> Hi,
> >>
> >> There has been some terrific progress on the translations.
> >>
> >> The easiest way to share it is to check the Transifex dashboard, below
> are
> >> some screenshots:
> >>
> >> The runbook:
> >> http://awesomescreenshot.com/0bd163zldd
> >>
> >> The UI:
> >> http://awesomescreenshot.com/089163zva2
> >>
> >> The entire docs set:
> >> http://awesomescreenshot.com/01c163zq63
> >>
> >> Since 4.1 is not out yet, there is time to finish those last 3% on the
> UI
> >> translations.
> >>
> >> Keep getting the word out, forward to friends and let's all make
> >> CloudStack a truly worldwide project.
> >>
> >> Cheers,
> >>
> >> -Sebastien
> >
> >
> >
> >
> > --
> > Gavin
>
>


-- 
Gavin


Re: [DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Sebastien Goasguen

On Apr 16, 2013, at 10:40 AM, Gavin Lee  wrote:

> The UI for 4.1 branch is ok, there is one case relate to the encoding issue:
> https://issues.apache.org/jira/browse/CLOUDSTACK-870

This was solved in https://issues.apache.org/jira/browse/CLOUDSTACK-1517
I see that 870 is closed now.

> 
> Since messages_zh_CN.properties and messages_ja.properties file under resource
> folder are not human readable, how to translate or modify the content?
> 
> 

I think milamber is using: http://www.cantamen.com/i18nedit.php to deal with 
the encoding.

the files needed to be encoded that way to render properly in the UI.

> On Tue, Apr 16, 2013 at 10:05 PM, Sebastien Goasguen wrote:
> 
>> 
>> On Apr 16, 2013, at 9:41 AM, Gavin Lee  wrote:
>> 
>>> Yes, Traditional Chinese moving very quickly.
>>> Hopefully, the other languages can have more contributors.
>>> 
>>> For the UI part, I saw the characters are not recognizable (browser
>>> encoding setting: auto detect & Unicode UTF-8):
>>> ja: http://snag.gy/AVsbU.jpg
>>> zh_CN: http://snag.gy/MxbBS.jpg
>> 
>> Hum, we fixed some encodings so that it would work in the UI. Have you
>> tried to build the 4.1 branch and tested the UI itself, rather than
>> transifex.
>> 
>> Adding Milamber has he masters encoding for l10n.
>> 
>> -sebastien
>> 
>>> 
>>> The other languages are ok.
>>> 
>>> 
>>> On Tue, Apr 16, 2013 at 8:25 PM, Sebastien Goasguen >> wrote:
>>> 
 Hi,
 
 There has been some terrific progress on the translations.
 
 The easiest way to share it is to check the Transifex dashboard, below
>> are
 some screenshots:
 
 The runbook:
 http://awesomescreenshot.com/0bd163zldd
 
 The UI:
 http://awesomescreenshot.com/089163zva2
 
 The entire docs set:
 http://awesomescreenshot.com/01c163zq63
 
 Since 4.1 is not out yet, there is time to finish those last 3% on the
>> UI
 translations.
 
 Keep getting the word out, forward to friends and let's all make
 CloudStack a truly worldwide project.
 
 Cheers,
 
 -Sebastien
>>> 
>>> 
>>> 
>>> 
>>> --
>>> Gavin
>> 
>> 
> 
> 
> -- 
> Gavin



CloudStack coming meetup in China

2013-04-16 Thread Gavin Lee
There were several meetups in China in Q1, 2013, mostly in Beijing and
Shanghai.
>From now on, I'll summerize the relate info(attendees, sharing materials,
etc) and keep dev list updating about the coming meetup.

The recent meetup is April 20th from 1:30PM to 5:30PM, there are 3 topics:
1. Update to attendees about CloudStack 4.1 features
2. CloudStack Best Practice in PPTV; PPTV OPS Architect will share their
real production environment which has more than 100 physical hosts.
3. Setup development environment and test on Devcloud

I'll also update to attendee about how to deeply involve in CloudStack
community.

For detailed info, please refer to(Sorry, only Chinese):
http://www.cloudstack-china.org/2013/04/1858.html

-- 
Gavin


Re: Review Request: Cloudstack-701 Support for non contiguous vlan ranges.

2013-04-16 Thread bharat kumar

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

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


Review request for cloudstack and Abhinandan Prateek.


Description
---

Cloudstack-701: Support for non contiguous vlan ranges.


Diffs (updated)
-

  api/src/com/cloud/network/NetworkService.java 066009b 
  api/src/com/cloud/network/PhysicalNetwork.java a2044a6 
  api/src/org/apache/cloudstack/api/ApiConstants.java 37cb59f 
  
api/src/org/apache/cloudstack/api/command/admin/network/UpdatePhysicalNetworkCmd.java
 06cf38d 
  server/src/com/cloud/api/ApiResponseHelper.java cfe0e00 
  server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 
  server/src/com/cloud/dc/dao/DataCenterVnetDao.java 79e91c4 
  server/src/com/cloud/dc/dao/DataCenterVnetDaoImpl.java 5ded0f4 
  server/src/com/cloud/network/ExternalFirewallDeviceManagerImpl.java c2038e5 
  server/src/com/cloud/network/NetworkServiceImpl.java 70d1d0d 
  server/src/com/cloud/network/dao/PhysicalNetworkVO.java e5ffcfb 
  server/src/com/cloud/network/guru/GuestNetworkGuru.java 9c0205a 
  server/test/com/cloud/network/MockNetworkManagerImpl.java 9042f03 
  server/test/com/cloud/network/UpdatePhysicalNetworkTest.java PRE-CREATION 
  server/test/com/cloud/vpc/MockNetworkManagerImpl.java 3a585ce 
  test/integration/smoke/test_non_contigiousvlan.py PRE-CREATION 

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


Testing
---

Tested on latest master.


Thanks,

bharat kumar



Re: CloudStack coming meetup in China

2013-04-16 Thread Joe Brockmeier
Hi Gavin, 

Can you also update the market...@cloudstack.apache.org list? 

On Tue, Apr 16, 2013, at 09:46 AM, Gavin Lee wrote:
> There were several meetups in China in Q1, 2013, mostly in Beijing and
> Shanghai.
> From now on, I'll summerize the relate info(attendees, sharing materials,
> etc) and keep dev list updating about the coming meetup.
> 
> The recent meetup is April 20th from 1:30PM to 5:30PM, there are 3
> topics:
> 1. Update to attendees about CloudStack 4.1 features
> 2. CloudStack Best Practice in PPTV; PPTV OPS Architect will share their
> real production environment which has more than 100 physical hosts.
> 3. Setup development environment and test on Devcloud
> 
> I'll also update to attendee about how to deeply involve in CloudStack
> community.
> 
> For detailed info, please refer to(Sorry, only Chinese):
> http://www.cloudstack-china.org/2013/04/1858.html
> 
> -- 
> Gavin


Best,

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


Re: Review Request: Cloudstack-701 Support for non contiguous vlan ranges.

2013-04-16 Thread bharat kumar


> On April 12, 2013, 9:53 a.m., Abhinandan Prateek wrote:
> > Ship It!
> 
> Abhinandan Prateek wrote:
> The patch does not apply, can you resubmit after checking it.

uploaded the new diff


- bharat


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


On April 16, 2013, 2:53 p.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10238/
> ---
> 
> (Updated April 16, 2013, 2:53 p.m.)
> 
> 
> Review request for cloudstack and Abhinandan Prateek.
> 
> 
> Description
> ---
> 
> Cloudstack-701: Support for non contiguous vlan ranges.
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/NetworkService.java 066009b 
>   api/src/com/cloud/network/PhysicalNetwork.java a2044a6 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 37cb59f 
>   
> api/src/org/apache/cloudstack/api/command/admin/network/UpdatePhysicalNetworkCmd.java
>  06cf38d 
>   server/src/com/cloud/api/ApiResponseHelper.java cfe0e00 
>   server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 
>   server/src/com/cloud/dc/dao/DataCenterVnetDao.java 79e91c4 
>   server/src/com/cloud/dc/dao/DataCenterVnetDaoImpl.java 5ded0f4 
>   server/src/com/cloud/network/ExternalFirewallDeviceManagerImpl.java c2038e5 
>   server/src/com/cloud/network/NetworkServiceImpl.java 70d1d0d 
>   server/src/com/cloud/network/dao/PhysicalNetworkVO.java e5ffcfb 
>   server/src/com/cloud/network/guru/GuestNetworkGuru.java 9c0205a 
>   server/test/com/cloud/network/MockNetworkManagerImpl.java 9042f03 
>   server/test/com/cloud/network/UpdatePhysicalNetworkTest.java PRE-CREATION 
>   server/test/com/cloud/vpc/MockNetworkManagerImpl.java 3a585ce 
>   test/integration/smoke/test_non_contigiousvlan.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10238/diff/
> 
> 
> Testing
> ---
> 
> Tested on latest master.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>



Re: [DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Milamber



Le 16/04/2013 13:41, Gavin Lee a ecrit :

Yes, Traditional Chinese moving very quickly.
Hopefully, the other languages can have more contributors.

For the UI part, I saw the characters are not recognizable (browser
encoding setting: auto detect&  Unicode UTF-8):
ja: http://snag.gy/AVsbU.jpg
zh_CN: http://snag.gy/MxbBS.jpg
This screenshots shows some characters with a incorrect encoding (try to 
display a char as a ISO-8859-1 (or japanese charset) but the encoding is 
a UTF-8, I think)


With Sebgoa, we have correct all UI ressource file to have only one 
encoding charset in this files (ASCII with unicode). The transifex data 
isn't up-to-date.


Sebgoa, I think we must upload the last version of this (all) resources 
files (except FR already done) from branch 4.1 to transifex.
The last version of resources files is ASCII with unicode for *all 
chars* in each file, and now transifex keep the unicode char (check with 
FR download for use)


Milamber





The other languages are ok.


On Tue, Apr 16, 2013 at 8:25 PM, Sebastien Goasguenwrote:


Hi,

There has been some terrific progress on the translations.

The easiest way to share it is to check the Transifex dashboard, below are
some screenshots:

The runbook:
http://awesomescreenshot.com/0bd163zldd

The UI:
http://awesomescreenshot.com/089163zva2

The entire docs set:
http://awesomescreenshot.com/01c163zq63

Since 4.1 is not out yet, there is time to finish those last 3% on the UI
translations.

Keep getting the word out, forward to friends and let's all make
CloudStack a truly worldwide project.

Cheers,

-Sebastien








Re: [DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Go Chiba
Gavin, I running 4.1 repository and all of languages of zh_CN are working
well as below.
  http://gyazo.com/6dbc68ecc1ee4b70feec9ff53c51df99

In the previous, ja_JP and zh_CN had a encoding problem as you said, but I
believe
newest 4.1 repository are fixed and you could running it on UTF-8 supported
borwser  :)

Go Chiba


On Tue, Apr 16, 2013 at 11:45 PM, Sebastien Goasguen wrote:

>
> On Apr 16, 2013, at 10:40 AM, Gavin Lee  wrote:
>
> > The UI for 4.1 branch is ok, there is one case relate to the encoding
> issue:
> > https://issues.apache.org/jira/browse/CLOUDSTACK-870
>
> This was solved in https://issues.apache.org/jira/browse/CLOUDSTACK-1517
> I see that 870 is closed now.
>
> >
> > Since messages_zh_CN.properties and messages_ja.properties file under
> resource
> > folder are not human readable, how to translate or modify the content?
> >
> >
>
> I think milamber is using: http://www.cantamen.com/i18nedit.php to deal
> with the encoding.
>
> the files needed to be encoded that way to render properly in the UI.
>
> > On Tue, Apr 16, 2013 at 10:05 PM, Sebastien Goasguen  >wrote:
> >
> >>
> >> On Apr 16, 2013, at 9:41 AM, Gavin Lee  wrote:
> >>
> >>> Yes, Traditional Chinese moving very quickly.
> >>> Hopefully, the other languages can have more contributors.
> >>>
> >>> For the UI part, I saw the characters are not recognizable (browser
> >>> encoding setting: auto detect & Unicode UTF-8):
> >>> ja: http://snag.gy/AVsbU.jpg
> >>> zh_CN: http://snag.gy/MxbBS.jpg
> >>
> >> Hum, we fixed some encodings so that it would work in the UI. Have you
> >> tried to build the 4.1 branch and tested the UI itself, rather than
> >> transifex.
> >>
> >> Adding Milamber has he masters encoding for l10n.
> >>
> >> -sebastien
> >>
> >>>
> >>> The other languages are ok.
> >>>
> >>>
> >>> On Tue, Apr 16, 2013 at 8:25 PM, Sebastien Goasguen  >>> wrote:
> >>>
>  Hi,
> 
>  There has been some terrific progress on the translations.
> 
>  The easiest way to share it is to check the Transifex dashboard, below
> >> are
>  some screenshots:
> 
>  The runbook:
>  http://awesomescreenshot.com/0bd163zldd
> 
>  The UI:
>  http://awesomescreenshot.com/089163zva2
> 
>  The entire docs set:
>  http://awesomescreenshot.com/01c163zq63
> 
>  Since 4.1 is not out yet, there is time to finish those last 3% on the
> >> UI
>  translations.
> 
>  Keep getting the word out, forward to friends and let's all make
>  CloudStack a truly worldwide project.
> 
>  Cheers,
> 
>  -Sebastien
> >>>
> >>>
> >>>
> >>>
> >>> --
> >>> Gavin
> >>
> >>
> >
> >
> > --
> > Gavin
>
>


-- 
千葉 豪  Go Chiba
E-mail:go.ch...@gmail.com


Re: Review Request: CLOUDSTACK-741: Granular Global Parameters

2013-04-16 Thread Nitin Mehta

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


The response Class needs a change too ?


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


So this is available only at zone level and not at the global level is it ?



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


Lets call it ScopeLevelConfigsMap



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


An English comment as to what u are trying to do would help



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


Do u need this ?



server/src/com/cloud/configuration/ConfigurationParameterScope.java


Can we not add this in Configuration interface itself ?



server/src/com/cloud/server/ConfigurationServerImpl.java


I would like to see a clean switch statement here...


- Nitin Mehta


On April 16, 2013, 2:26 p.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10548/
> ---
> 
> (Updated April 16, 2013, 2:26 p.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin 
> Mehta.
> 
> 
> Description
> ---
> 
> CLOUDSTACK-741: Granular Global Parameters
> 
> Modified UpdateConfiguration API and ListConfiguration API by adding 
> additional parameter scope and resource id (zone/cluster/pool/account).
> Scope decides the parameter which level we can update the value and id is the 
> corresponding id based on the scope.
> This scope is defined in the config.java file where we are creating groups 
> for each level(zone/cluster/pool/account).
> Currently submitted the patch for the zone level parameter use.external.dns.
> 
> Included unit tests and marvin tests.
> 
> 
> This addresses bug CLOUDSTACK-741.
> 
> 
> Diffs
> -
> 
>   api/src/org/apache/cloudstack/api/command/admin/config/ListCfgsByCmd.java 
> aabfd4a 
>   api/src/org/apache/cloudstack/api/command/admin/config/UpdateCfgCmd.java 
> ffeb586 
>   api/test/org/apache/cloudstack/api/command/test/ListCfgCmdTest.java 
> PRE-CREATION 
>   api/test/org/apache/cloudstack/api/command/test/UpdateCfgCmdTest.java 
> PRE-CREATION 
>   server/src/com/cloud/configuration/Config.java e69ea19 
>   server/src/com/cloud/configuration/ConfigurationManager.java d7faf19 
>   server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 
>   server/src/com/cloud/configuration/ConfigurationParameterScope.java 
> PRE-CREATION 
>   server/src/com/cloud/configuration/dao/ConfigurationDao.java c86c024 
>   server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java 
> ab91059 
>   server/src/com/cloud/server/ConfigurationServer.java f441b1f 
>   server/src/com/cloud/server/ConfigurationServerImpl.java 852c00b 
>   server/src/com/cloud/server/ManagementServerImpl.java 5ed0c92 
>   server/test/com/cloud/vpc/MockConfigurationManagerImpl.java a03e361 
>   test/integration/smoke/test_UpdateCfg.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10548/diff/
> 
> 
> Testing
> ---
> 
> Tested locally
> - tested the modified UpdateConfiguration API and ListConfiguration API by 
> providing the zone scope and zone id
> - tested the modified parameter to take the updated value at zone level(after 
> updation).
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>



Re: [DOCS][TRANSLATIONS] Upate

2013-04-16 Thread Milamber



Le 16/04/2013 14:45, Sebastien Goasguen a ecrit :

On Apr 16, 2013, at 10:40 AM, Gavin Lee  wrote:


The UI for 4.1 branch is ok, there is one case relate to the encoding issue:
https://issues.apache.org/jira/browse/CLOUDSTACK-870

This was solved in https://issues.apache.org/jira/browse/CLOUDSTACK-1517
I see that 870 is closed now.


Since messages_zh_CN.properties and messages_ja.properties file under resource
folder are not human readable, how to translate or modify the content?



I think milamber is using: http://www.cantamen.com/i18nedit.php to deal with 
the encoding.


Yes, I18Nedit to convert native encoding (like ISO-8859-1, EUC-JP, etc) 
to ASCII with unicode




the files needed to be encoded that way to render properly in the UI.


2 requirements for a good display of Web UI :

* resource file must be in ASCII with unicode (for all chars)
* (only for some language like French, Italian, Spanish, etc.) the quote 
' in sentence inside, must have a backslash before \'  (dictionary.jsp 
contains the key/value dictionary in a javascript code) This points 
isn't covering by I18Nedit, but a traditional search/replace in an editor.


Milamber





On Tue, Apr 16, 2013 at 10:05 PM, Sebastien Goasguenwrote:


On Apr 16, 2013, at 9:41 AM, Gavin Lee  wrote:


Yes, Traditional Chinese moving very quickly.
Hopefully, the other languages can have more contributors.

For the UI part, I saw the characters are not recognizable (browser
encoding setting: auto detect&  Unicode UTF-8):
ja: http://snag.gy/AVsbU.jpg
zh_CN: http://snag.gy/MxbBS.jpg

Hum, we fixed some encodings so that it would work in the UI. Have you
tried to build the 4.1 branch and tested the UI itself, rather than
transifex.

Adding Milamber has he masters encoding for l10n.

-sebastien


The other languages are ok.


On Tue, Apr 16, 2013 at 8:25 PM, Sebastien Goasguen
Hi,

There has been some terrific progress on the translations.

The easiest way to share it is to check the Transifex dashboard, below

are

some screenshots:

The runbook:
http://awesomescreenshot.com/0bd163zldd

The UI:
http://awesomescreenshot.com/089163zva2

The entire docs set:
http://awesomescreenshot.com/01c163zq63

Since 4.1 is not out yet, there is time to finish those last 3% on the

UI

translations.

Keep getting the word out, forward to friends and let's all make
CloudStack a truly worldwide project.

Cheers,

-Sebastien




--
Gavin




--
Gavin






Re: strange output on cs4.0.1 gui when restarted

2013-04-16 Thread Milamber

The text is display as URL encode (example: %20 == space char).

Empty your cache browser, close it and restart. Try now to view the 
admin UI?


Milamber

Le 16/04/2013 08:01, Sebastien Goasguen a ecrit :

Hi Jason, did you get passed this ?

I have never had this issue. Did you select a special language ? Did you try on 
a different browser ?

-Sebastien

On Apr 9, 2013, at 7:49 PM, Jason Pavao  wrote:


Hey Folks,
I am seeing a really strange issue, after a reboot, I am seeing tons of % and 
other characters throughout the display. An example-


Anyone have any ideas as to why this happened and how to correct it? Has anyone 
had this issue before?
I appreciate any help.
Thanks.
-jason


--
Thanks.
-Jason






[PATCH][ACS41] CLOUDSTACK-2048 - When Connectivity is added to a VpcOffering the provider is set to VPCVirtualRouter, should be NiciraNvp

2013-04-16 Thread Hugo Trippaers
Chip,

Can you apply this fix to 4.1?  There is a better fix for this in master, but 
that is more involved. This fix will allow NiciraNvp users to create VPC's 
without having to edit the database and has only minimal impact on 4.1 branch.

The patch is in the ticket as it can't be applied to master.

Cheers,

Hugo


[PATCH][ACS41] CLOUDSTACK-2033 Fix usage server startup and logging

2013-04-16 Thread Hugo Trippaers
Please cherry pick commits ae16f332132a5eb1e3ff85a0c435dbe25a1d6299 
574e716352290f9c34bfa7bd3134a13e0caa8da2.

Packaging fixes for usage server, but there are some code changes implemented 
to fix the startup.


Cheers,

Hugo


Re: strange output on cs4.0.1 gui when restarted

2013-04-16 Thread Jason Pavao

Thanks Sebastian for checking in with me.

It turned out that I or one of my jenkins users switched-
encode.api.response in global settings from false to true

This caused all that garbled output. Once we set it back and performed a 
reboot of the management server, we were back in business.


What is the purpose of encode.api.response ? I found no documentation on 
it's use/functionality.


Thanks again to everyone on the lists for the assistance!
-jason

On 4/16/2013 1:01 AM, Sebastien Goasguen wrote:

Hi Jason, did you get passed this ?

I have never had this issue. Did you select a special language ? Did you try on 
a different browser ?

-Sebastien

On Apr 9, 2013, at 7:49 PM, Jason Pavao  wrote:


Hey Folks,
I am seeing a really strange issue, after a reboot, I am seeing tons of % and 
other characters throughout the display. An example-


Anyone have any ideas as to why this happened and how to correct it? Has anyone 
had this issue before?
I appreciate any help.
Thanks.
-jason


--
Thanks.
-Jason


--
Thanks.
-Jason



Re: inheritable componentContexts

2013-04-16 Thread Kelven Yang


On 4/16/13 1:15 AM, "Prasanna Santhanam"  wrote:

>This is probably a question for the Spring community but I'll ask here
>anyway. Every so often when the componentContext.xml.in changes I find
>that I have to fix up simulatorComponentContext.xml.in similarly to
>make the beans load appropriately and for the bvt tests to work.
>
>Is there a way similar to the components-premium.xml we had before to
>'inherit' Spring contexts so it's easier to maintain the xmls?
>
>-- 
>Prasanna.,

Unfortunately, Spring does not provide an out-of-box solution to inherit
between XML configuration files currently, however it provides the feature
to merge several independent context files together.

What you may do is to have your simulator components in a separated
context XML file and have Spring merge it along with other common context
files (like we did for applicationContext.xml + componentContext.xml).

-Kelven
 
>



Re: strange output on cs4.0.1 gui when restarted

2013-04-16 Thread David Nalley
On Tue, Apr 16, 2013 at 12:16 PM, Jason Pavao  wrote:
> Thanks Sebastian for checking in with me.
>
> It turned out that I or one of my jenkins users switched-
> encode.api.response in global settings from false to true
>
> This caused all that garbled output. Once we set it back and performed a
> reboot of the management server, we were back in business.
>
> What is the purpose of encode.api.response ? I found no documentation on
> it's use/functionality.
>
> Thanks again to everyone on the lists for the assistance!
> -jason
>


Sounds like a bug - do you mind filing it?
http://issues.apache.org/jira

--David


Re: strange output on cs4.0.1 gui when restarted

2013-04-16 Thread Jason Pavao

Sure! It'll be my first bug! Exciting!

Issue CLOUDSTACK-2050 - encode.api.response in global settings when set 
to true will create garbled % output int he gui and json output has been 
successfully created.


Yay!

Thanks again for all help.
-jason

On 4/16/2013 10:03 AM, David Nalley wrote:

On Tue, Apr 16, 2013 at 12:16 PM, Jason Pavao  wrote:

Thanks Sebastian for checking in with me.

It turned out that I or one of my jenkins users switched-
encode.api.response in global settings from false to true

This caused all that garbled output. Once we set it back and performed a
reboot of the management server, we were back in business.

What is the purpose of encode.api.response ? I found no documentation on
it's use/functionality.

Thanks again to everyone on the lists for the assistance!
-jason



Sounds like a bug - do you mind filing it?
http://issues.apache.org/jira

--David


--
Thanks.
-Jason



RE: [DISCUSS] labels for cloudstack university and GSoC

2013-04-16 Thread Kelcey Damage (BT)
I think the concern here was in adding further volume to the dev list, which
should be primarily about just that, 'Development'. And EDU could be an
opt-in list. EDU is a wonderful initiative, however the more non-dev related
content we poor into dev, the more we pester the core development population
and dilute the purpose of the list. Besides, us Edu/Marketing people are
nothing without the devs focused on building an awesome product :).

-my 2 cents



>-Original Message-
>From: Sebastien Goasguen [mailto:run...@gmail.com]
>Sent: Tuesday, April 16, 2013 1:58 AM
>To: dev@cloudstack.apache.org
>Subject: [DISCUSS] labels for cloudstack university and GSoC
>
>Hi folks,
>
>Some of you may have seen:
>https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+Univer
>sity
>
>I also emailed about the Google Summer of Code.
>
>Kelcey mentioned creating an edu mailing list.
>
>I have not particular feelings about this and I wanted to discuss it.
>
>GSoC will be across dev and users. Students are expected to join the
>community, work on JIRA and submitted patches etc..
>ACSEDU will be across dev/users and marketing.
>
>Instead of creating a list maybe we can just use tags/labels in emails:
>
>[GSOC] and [ACSEDU]
>
>The only issue I see is that there will be information that is relevant to
all lists
>and I don't want to leave anyone out.
>
>Thoughts ?
>
>-Sebastien



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

2013-04-16 Thread Sangeetha Hariharan
We did have an issue where default network was not programmed correctly in 4.1 
which is now fixed:
CLOUDSTACK-1115 - In multiple shared network unable to login with default nic - 
KVM
It is possible you are hitting the same issue , not sure.

Both iptonetworklist and  networkIds cannot be used in deployVirtualMachine() . 
This is by design.
When using iptonetworklist parameter , iptonetworklist[0].networkid should be 
used as default network.


-Thanks
Sangeetha

-Original Message-
From: sx chen [mailto:cloudchen0...@gmail.com] 
Sent: Monday, April 15, 2013 7:28 PM
To: us...@cloudstack.apache.org; dev@cloudstack.apache.org
Subject: Re: How to set the default network when I specify IP use 
"iptonetworklist" parameter when deploy a vm?

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: [DISCUSS] labels for cloudstack university and GSoC

2013-04-16 Thread David Nalley
On Tue, Apr 16, 2013 at 4:57 AM, Sebastien Goasguen  wrote:
> Hi folks,
>
> Some of you may have seen:
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+University
>
> I also emailed about the Google Summer of Code.
>
> Kelcey mentioned creating an edu mailing list.
>
> I have not particular feelings about this and I wanted to discuss it.
>
> GSoC will be across dev and users. Students are expected to join the 
> community, work on JIRA and submitted patches etc..
> ACSEDU will be across dev/users and marketing.
>
> Instead of creating a list maybe we can just use tags/labels in emails:
>
> [GSOC] and [ACSEDU]
>
> The only issue I see is that there will be information that is relevant to 
> all lists and I don't want to leave anyone out.
>
> Thoughts ?
>
> -Sebastien

The goal of GSOC is to embed folks in the open source community they
are working in. Not to shuffle them off to some low-volume list to
isolate them from the flow of mail that is dev@. (as well as commits@,
issues@, and users@)

For better or worse dev@ is voluminous, and GSOC folks need to learn
to operate in that environment, and operate as a part of the dev
community.

ACSEDU might be slightly different, but I worry about fragmenting our
community. Dev@ should really be where things happen.

--David


Re: [DISCUSS] labels for cloudstack university and GSoC

2013-04-16 Thread Joe Brockmeier
On Tue, Apr 16, 2013, at 03:57 AM, Sebastien Goasguen wrote:
> Instead of creating a list maybe we can just use tags/labels in emails:
> 
> [GSOC] and [ACSEDU]

+1 

Suggestion: EDU efforts are probably more marketing than dev, so it
might be best to shunt those discussions to marketing unless there's a
reason not to. (At least that's my thought, I could be spectacularly
wrong.)

Best,

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


[VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Joe Brockmeier
Hi All,

I've created a 4.0.2 release, with the following artifacts up for a
vote:

Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.0
Commit: aa5081e1746c33eba6b3257602996e2aaa01ba0b

List of changes:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.0

Source release (checksums and signatures are available at the same
location):
http://people.apache.org/~jzb/cloudstack/dist/releases/4.0.2/

PGP release keys (signed using A0207CD4):
http://www.apache.org/dist/incubator/cloudstack/KEYS

Vote will be open for 72 hours.

For sanity in tallying the vote, can PMC members please be sure to
indicate "(binding)" with their vote?

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

Best,

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


RE: CLOUDSTACK-1747: fresh deploydb bug in 4.1 requires cherry-pick commits from master

2013-04-16 Thread Sangeetha Hariharan
I still see this issue  on master build , where after executing 
cloudstack-setup-databases DB version is still pointing to 4.0.0.

I have logged the following bug to track this issue:

 CLOUDSTACK-2052 - After executing cloudstack-setup-databases , DB version is 
still pointing to 4.0.0.


-Thanks
Sangeetha
-Original Message-
From: Alex Huang [mailto:alex.hu...@citrix.com] 
Sent: Thursday, March 21, 2013 11:56 AM
To: dev@cloudstack.apache.org; cloudstack-...@incubator.apache.org
Subject: RE: CLOUDSTACK-1747: fresh deploydb bug in 4.1 requires cherry-pick 
commits from master



> -Original Message-
> From: rohityada...@gmail.com [mailto:rohityada...@gmail.com] On Behalf 
> Of Rohit Yadav
> Sent: Wednesday, March 20, 2013 11:27 PM
> To: cloudstack-...@incubator.apache.org
> Subject: Re: CLOUDSTACK-1747: fresh deploydb bug in 4.1 requires 
> cherry- pick commits from master
> 
> On Thu, Mar 21, 2013 at 1:54 AM, Alex Huang 
> wrote:
> > Hi everyone,
> >
> > We had a discussion on irc.
> >
> > To implement something intermediary will be around the same effort 
> > as
> finishing the job and there are various reasons why just running the 
> create- schema.sql and update*.sql is not enough.  So we should just 
> decide on finishing the job in 4.1 or document the behavior.
> >
> > Any input on which way we should take.
> >
> > The engineer in me says just finish the job.  It's obviously working 
> > in dev
> setup on master so the risk of this introducing bugs is very low.  If 
> someone knows why it's difficult to change in cloud-setup-databases 
> script, please speak up.
> 
> Alex asks why it is difficult to change in cloud-setup-database script?
Rohit,

Actually, I really just meant was there technical challenges in updating the 
script that was already looked into.  Sounds like from your reply there isn't.

Appreciate your work on the various parts.  It was excellent.  Good luck on 
your next venture.

--Alex


[ACS41][Patch Request]

2013-04-16 Thread Marcus Sorensen
commit 202930fd554ddc9a8e78cf39ae54a4e3a56d3904
Author: Marcus Sorensen 
Date:   Tue Apr 16 12:07:15 2013 -0600

 CLOUDSTACK-2051 Allow KVM HA Monitor to verify that a NfsStoragePool is
valid before running the script that mounts it and touches the HA files.

Signed-off-by: Marcus Sorensen  1366135635
-0600


RE: [DISCUSS] labels for cloudstack university and GSoC

2013-04-16 Thread Kelcey Damage (BT)
I like Joe's proposition of potentially housing the [ACSEDU] tag under
marketing. I do agree that [GSoC] is 'Dev' related and belongs on that list.


>-Original Message-
>From: Joe Brockmeier [mailto:j...@zonker.net]
>Sent: Tuesday, April 16, 2013 10:35 AM
>To: dev@cloudstack.apache.org
>Subject: Re: [DISCUSS] labels for cloudstack university and GSoC
>
>On Tue, Apr 16, 2013, at 03:57 AM, Sebastien Goasguen wrote:
>> Instead of creating a list maybe we can just use tags/labels in emails:
>>
>> [GSOC] and [ACSEDU]
>
>+1
>
>Suggestion: EDU efforts are probably more marketing than dev, so it might
be
>best to shunt those discussions to marketing unless there's a reason not
to. (At
>least that's my thought, I could be spectacularly
>wrong.)
>
>Best,
>
>jzb
>--
>Joe Brockmeier
>j...@zonker.net
>Twitter: @jzb
>http://www.dissociatedpress.net/



RE: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Animesh Chaturvedi


> -Original Message-
> From: Joe Brockmeier [mailto:j...@zonker.net]
> Sent: Tuesday, April 16, 2013 10:53 AM
> To: dev@cloudstack.apache.org
> Subject: [VOTE] Apache CloudStack 4.0.2
> 
> Hi All,
> 
> I've created a 4.0.2 release, with the following artifacts up for a
> vote:
> 
> Git Branch and Commit SH:
> https://git-wip-
> us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.0
> Commit: aa5081e1746c33eba6b3257602996e2aaa01ba0b
> 
> List of changes:
> https://git-wip-
> us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.0
> 
> Source release (checksums and signatures are available at the same
> location):
> http://people.apache.org/~jzb/cloudstack/dist/releases/4.0.2/
> 
> PGP release keys (signed using A0207CD4):
> http://www.apache.org/dist/incubator/cloudstack/KEYS
> 
> Vote will be open for 72 hours.
> 
> For sanity in tallying the vote, can PMC members please be sure to indicate
> "(binding)" with their vote?
> 
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
> 
> Best,
> 
> jzb
[Animesh>]  +1

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


Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Jeronimo Garcia
+1


On Tue, Apr 16, 2013 at 1:48 PM, Animesh Chaturvedi <
animesh.chaturv...@citrix.com> wrote:

>
>
> > -Original Message-
> > From: Joe Brockmeier [mailto:j...@zonker.net]
> > Sent: Tuesday, April 16, 2013 10:53 AM
> > To: dev@cloudstack.apache.org
> > Subject: [VOTE] Apache CloudStack 4.0.2
> >
> > Hi All,
> >
> > I've created a 4.0.2 release, with the following artifacts up for a
> > vote:
> >
> > Git Branch and Commit SH:
> > https://git-wip-
> > us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.0
> > Commit: aa5081e1746c33eba6b3257602996e2aaa01ba0b
> >
> > List of changes:
> > https://git-wip-
> > us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.0
> >
> > Source release (checksums and signatures are available at the same
> > location):
> > http://people.apache.org/~jzb/cloudstack/dist/releases/4.0.2/
> >
> > PGP release keys (signed using A0207CD4):
> > http://www.apache.org/dist/incubator/cloudstack/KEYS
> >
> > Vote will be open for 72 hours.
> >
> > For sanity in tallying the vote, can PMC members please be sure to
> indicate
> > "(binding)" with their vote?
> >
> > [ ] +1  approve
> > [ ] +0  no opinion
> > [ ] -1  disapprove (and reason why)
> >
> > Best,
> >
> > jzb
> [Animesh>]  +1
>
> > --
> > Joe Brockmeier
> > j...@zonker.net
> > Twitter: @jzb
> > http://www.dissociatedpress.net/
>


Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Chiradeep Vittal
Folks, please follow the procedure here before voting
https://cwiki.apache.org/confluence/x/jgXVAQ

On 4/16/13 11:48 AM, "Animesh Chaturvedi" 
wrote:

>
>
>> -Original Message-
>> From: Joe Brockmeier [mailto:j...@zonker.net]
>> Sent: Tuesday, April 16, 2013 10:53 AM
>> To: dev@cloudstack.apache.org
>> Subject: [VOTE] Apache CloudStack 4.0.2
>> 
>> Hi All,
>> 
>> I've created a 4.0.2 release, with the following artifacts up for a
>> vote:
>> 
>> Git Branch and Commit SH:
>> https://git-wip-
>> us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.0
>> Commit: aa5081e1746c33eba6b3257602996e2aaa01ba0b
>> 
>> List of changes:
>> https://git-wip-
>> us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.0
>> 
>> Source release (checksums and signatures are available at the same
>> location):
>> http://people.apache.org/~jzb/cloudstack/dist/releases/4.0.2/
>> 
>> PGP release keys (signed using A0207CD4):
>> http://www.apache.org/dist/incubator/cloudstack/KEYS
>> 
>> Vote will be open for 72 hours.
>> 
>> For sanity in tallying the vote, can PMC members please be sure to
>>indicate
>> "(binding)" with their vote?
>> 
>> [ ] +1  approve
>> [ ] +0  no opinion
>> [ ] -1  disapprove (and reason why)
>> 
>> Best,
>> 
>> jzb
>[Animesh>]  +1
>
>> --
>> Joe Brockmeier
>> j...@zonker.net
>> Twitter: @jzb
>> http://www.dissociatedpress.net/



Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Wei ZHOU
Hi Joe,

Is this patch included in 4.0.2?
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=5ac59fbecbae069ab46048e9a73a6f7b8fb2d4b1

Kind regards,
Wei


2013/4/16 Joe Brockmeier 

> Hi All,
>
> I've created a 4.0.2 release, with the following artifacts up for a
> vote:
>
> Git Branch and Commit SH:
>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.0
> Commit: aa5081e1746c33eba6b3257602996e2aaa01ba0b
>
> List of changes:
>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.0
>
> Source release (checksums and signatures are available at the same
> location):
> http://people.apache.org/~jzb/cloudstack/dist/releases/4.0.2/
>
> PGP release keys (signed using A0207CD4):
> http://www.apache.org/dist/incubator/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
> Best,
>
> jzb
> --
> Joe Brockmeier
> j...@zonker.net
> Twitter: @jzb
> http://www.dissociatedpress.net/
>


Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread David Nalley
Wei - it is.
It was committed almost a month ago into the 4.0 branch.

--David

On Tue, Apr 16, 2013 at 3:34 PM, Wei ZHOU  wrote:
> Hi Joe,
>
> Is this patch included in 4.0.2?
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=5ac59fbecbae069ab46048e9a73a6f7b8fb2d4b1
>
> Kind regards,
> Wei
>
>
> 2013/4/16 Joe Brockmeier 
>
>> Hi All,
>>
>> I've created a 4.0.2 release, with the following artifacts up for a
>> vote:
>>
>> Git Branch and Commit SH:
>>
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.0
>> Commit: aa5081e1746c33eba6b3257602996e2aaa01ba0b
>>
>> List of changes:
>>
>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.0
>>
>> Source release (checksums and signatures are available at the same
>> location):
>> http://people.apache.org/~jzb/cloudstack/dist/releases/4.0.2/
>>
>> PGP release keys (signed using A0207CD4):
>> http://www.apache.org/dist/incubator/cloudstack/KEYS
>>
>> Vote will be open for 72 hours.
>>
>> For sanity in tallying the vote, can PMC members please be sure to
>> indicate "(binding)" with their vote?
>>
>> [ ] +1  approve
>> [ ] +0  no opinion
>> [ ] -1  disapprove (and reason why)
>>
>> Best,
>>
>> jzb
>> --
>> Joe Brockmeier
>> j...@zonker.net
>> Twitter: @jzb
>> http://www.dissociatedpress.net/
>>


grant/revokeAccess Question

2013-04-16 Thread Mike Tutkowski
Hi Edison,

I forgot to ask you about these methods when we talked yesterday. :)

It is my understanding that grant/revokeAccess will be called to grant
access to a host or revoke access from a host for a volume.

Our SAN has two ways of allowing access to volumes from hosts:

1) If the necessary CHAP credentials are in place on the host, the host can
access the volume.

2) If CHAP is not in use, the host's IQN needs to be listed in what we call
a Volume Access Group for the given volume.

I might be able to "get away" with just supporting number 2.

Is grant/revokeAccess the place where I would either talk to the hypervisor
to provide it with CHAP credentials or talk to our SAN to place the host's
IQN in the Volume Access Group for the volume?

If so, I assume it is OK for these methods to take a little time to execute
as a call would need to go out to the hypervisor or the SAN.

Thanks!

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Wei ZHOU
Thanks, David.
It is not in the list of changes.


2013/4/16 David Nalley 

> Wei - it is.
> It was committed almost a month ago into the 4.0 branch.
>
> --David
>
> On Tue, Apr 16, 2013 at 3:34 PM, Wei ZHOU  wrote:
> > Hi Joe,
> >
> > Is this patch included in 4.0.2?
> >
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=5ac59fbecbae069ab46048e9a73a6f7b8fb2d4b1
> >
> > Kind regards,
> > Wei
> >
> >
> > 2013/4/16 Joe Brockmeier 
> >
> >> Hi All,
> >>
> >> I've created a 4.0.2 release, with the following artifacts up for a
> >> vote:
> >>
> >> Git Branch and Commit SH:
> >>
> >>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.0
> >> Commit: aa5081e1746c33eba6b3257602996e2aaa01ba0b
> >>
> >> List of changes:
> >>
> >>
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.0
> >>
> >> Source release (checksums and signatures are available at the same
> >> location):
> >> http://people.apache.org/~jzb/cloudstack/dist/releases/4.0.2/
> >>
> >> PGP release keys (signed using A0207CD4):
> >> http://www.apache.org/dist/incubator/cloudstack/KEYS
> >>
> >> Vote will be open for 72 hours.
> >>
> >> For sanity in tallying the vote, can PMC members please be sure to
> >> indicate "(binding)" with their vote?
> >>
> >> [ ] +1  approve
> >> [ ] +0  no opinion
> >> [ ] -1  disapprove (and reason why)
> >>
> >> Best,
> >>
> >> jzb
> >> --
> >> Joe Brockmeier
> >> j...@zonker.net
> >> Twitter: @jzb
> >> http://www.dissociatedpress.net/
> >>
>


RE: [ASFCS42] Proposed schedule for our next release

2013-04-16 Thread Animesh Chaturvedi


> 
> >
> > 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.
> 
[Animesh>] Chip both additional time and  big focused effort is needed to help 
clear the backlog, you are right just one is not enough. Note that there is no 
change to feature freeze date for 4.2. It is the defect fixing phase that I am 
recommending extending. I agree to your comments on automated tests.



Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Joe Brockmeier
On Tue, Apr 16, 2013, at 03:40 PM, Wei ZHOU wrote:
> Thanks, David.
> It is not in the list of changes.

Sorry about that. I pulled the changes from Jira and it had no "fix
versions" tagged, so it didn't show up in the list of things that were
committed to 4.0.2. I've edited the Jira ticket to show 4.0.2 and 4.1.0.

Best,

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


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

2013-04-16 Thread Likitha Shetty
Hi Abhinav,

Thanks for the test plan. 

Could you add test cases to verify the usage related changes? 
Otherwise the plan looks good to me.

Thanks,
Likitha

>-Original Message-
>From: Abhinav Roy [mailto:abhinav@citrix.com]
>Sent: Monday, April 15, 2013 8:41 PM
>To: dev@cloudstack.apache.org
>Subject: [ACS42][QA] Test cases for Dedicated Resources - Public IP Addresses
>and VLANs per Tenant feature uploaded
>
>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



createAsync Question

2013-04-16 Thread Mike Tutkowski
Hi Edison,

I have another question regarding our discussion yesterday.

I'm trying to implement the createAsync method.

This is my impression for how I should start doing so:

Create a volume on our SAN.  I believe I can get the specs for this volume
from the Disk Offering like this:

DiskOfferingVO offering = diskOfferingDao
.findById(volume.getDiskOfferingId());

The next part is a bit hazy.  As we discussed yesterday, let's just deal
with XenServer for the moment.  :)

I'm thinking after I create my volume on the SAN, I should create an
instance of CreateStoragePoolCommand and send it to the hypervisor this way:

Answer answer = storageMgr.sendToPool(pool, null, cmd);

The type of the storage pool I would place in the CreateStoragePoolCommand
instance is StoragePoolType.IscsiLUN.

This would create a storage repository for me on the hypervisor.

I would then need to create an instance of CreateCommand and send it to the
hypervisor the same way to create a volume on the storage repository I just
created:

Answer answer = storageMgr.sendToPool(pool, null, cmd);

This volume on the storage repository would consume the entire SR.

Am I making decent sense here?

Thanks!

-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the
cloud
*™*


Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Marcus Sorensen
I've actually got a patch that *I believe* goes along with a previous
bugfix that is in 4.0.2 already, I'm trying to determine that and test
right now. I should be able to let you know within an hour or two whether
or not I think it should be applied.


On Tue, Apr 16, 2013 at 2:45 PM, Joe Brockmeier  wrote:

> On Tue, Apr 16, 2013, at 03:40 PM, Wei ZHOU wrote:
> > Thanks, David.
> > It is not in the list of changes.
>
> Sorry about that. I pulled the changes from Jira and it had no "fix
> versions" tagged, so it didn't show up in the list of things that were
> committed to 4.0.2. I've edited the Jira ticket to show 4.0.2 and 4.1.0.
>
> Best,
>
> jzb
> --
> Joe Brockmeier
> j...@zonker.net
> Twitter: @jzb
> http://www.dissociatedpress.net/
>


Fwd: Desktop as a Service solution

2013-04-16 Thread Iheb Kahloul
Hello, i m working on a DaaS solution based on open source IaaS.
Do you advise me to base my work on CloudStack?
And if so, what are the VDI protocoles that it's using?

Ps: I have a realy short amount of time to provide a Beta version.
Thank you,
Iheb Kahloul


[QA][ACS42] Test Plan for Enhanced Baremetal support for Cisco UCS

2013-04-16 Thread Parth Jagirdar
Test Plan @  
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Enhanced+Baremetal+support+for+Cisco+UCS

Requirements @ 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Enhanced+Baremetal+support+on+Cisco+UCS

FS @ 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Cisco+UCS+Integration+functional+spec


Ignore the Change Account Test Plan in above link.. that one got uploaded by 
mistake.


...Parth


apidocs build failing on master?

2013-04-16 Thread Vijayendra Bhamidipati
Hi,

I'm running into the following error when trying mvn install -P 
developer,systemvm -Dnonoss  on top of master:


[INFO] 
[INFO] Building Apache CloudStack apidocs 4.2.0-SNAPSHOT
[INFO] 
[INFO]
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-apidoc 
---
[INFO]
[INFO] --- exec-maven-plugin:1.2.1:exec (compile) @ cloud-apidoc ---
Check, is this api part of another build profile? Null value for key: 
addBaremetalHost preProcessedCommand=1
Traceback (most recent call last):
  File "/root/mywork/cloudstack/asf/latest/cloudstack/tools/apidoc/gen_toc.py", 
line 173, in 
category = choose_category(fn)
  File "/root/mywork/cloudstack/asf/latest/cloudstack/tools/apidoc/gen_toc.py", 
line 153, in choose_category
(fn, __file__))
Exception: Need to add a category for listCiscoVnmcResources.xml to 
/root/mywork/cloudstack/asf/latest/cloudstack/tools/apidoc/gen_toc.py:known_categories
[INFO] 
[INFO] Reactor Summary:


Is anyone else seeing the same issue?

Regards,
Vijay


RE: Desktop as a Service solution

2013-04-16 Thread Vijayendra Bhamidipati
Iheb,

You  mentioned DaaS, so if possible, please provide us with more details on the 
DaaS solution you are trying to implement. Since you mentioned VDI protocols, I 
assume you are looking to build a VDI solution on top of a cloud platform, 
spawning off VDI host VMs on demand, and deploying a connection broker to route 
the VDI to user assignment. Please correct me if this isn't the case, but if 
so, cloudstack would be a great fit, especially considering that you have 
little time for the beta version. You can use cloudstack APIs to upload/remove 
VDI images, use them to spawn off VMs, retrieve the VM uuids, store it in your 
own layer to map their life cycle/assignments etc. You can dynamically add 
disks if required to the guest VMs, or follow a simpler storage allocation 
model. You can take snapshots, create templates of the guest VMs as well in 
case you would want to provide users with templates containing pre-installed 
software, according to your organization's license requirements. You can also 
use various network deployment models that cloudstack offers for traffic 
isolation between VMs.

Cloudstack supports integration with NFS on iSCSI and SAN (Fibre Channel) as 
well as the backend, so you will have wider options for deployment of the 
backend storage arrays that will need to store the VDI images.

Compared to other IaaS solutions, cloudstack is vastly easier to setup, so 
starting with it would be a good idea.

As for VDI protocols, as I see it, the IaaS wouldn't know about them - they 
would logically sit on top of the IaaS.

Please let us know if you have any questions or need help with cloudstack 
itself.

Regards,
Vijay

-Original Message-
From: Iheb Kahloul [mailto:kahloul.i...@gmail.com] 
Sent: Tuesday, April 16, 2013 2:18 PM
To: dev@cloudstack.apache.org
Subject: Fwd: Desktop as a Service solution

Hello, i m working on a DaaS solution based on open source IaaS.
Do you advise me to base my work on CloudStack?
And if so, what are the VDI protocoles that it's using?

Ps: I have a realy short amount of time to provide a Beta version.
Thank you,
Iheb Kahloul


RE: Desktop as a Service solution

2013-04-16 Thread Vijayendra Bhamidipati
Ah, didn't see the subject line earlier - I was thinking of Data as a service 
apart from Desktop as a Service when I saw DaaS and was wondering! You can 
ignore that first line unless you want to add more :)

Regards,
Vijay

-Original Message-
From: Vijayendra Bhamidipati [mailto:vijayendra.bhamidip...@citrix.com] 
Sent: Tuesday, April 16, 2013 3:57 PM
To: dev@cloudstack.apache.org
Subject: RE: Desktop as a Service solution

Iheb,

You  mentioned DaaS, so if possible, please provide us with more details on the 
DaaS solution you are trying to implement. Since you mentioned VDI protocols, I 
assume you are looking to build a VDI solution on top of a cloud platform, 
spawning off VDI host VMs on demand, and deploying a connection broker to route 
the VDI to user assignment. Please correct me if this isn't the case, but if 
so, cloudstack would be a great fit, especially considering that you have 
little time for the beta version. You can use cloudstack APIs to upload/remove 
VDI images, use them to spawn off VMs, retrieve the VM uuids, store it in your 
own layer to map their life cycle/assignments etc. You can dynamically add 
disks if required to the guest VMs, or follow a simpler storage allocation 
model. You can take snapshots, create templates of the guest VMs as well in 
case you would want to provide users with templates containing pre-installed 
software, according to your organization's license requirements. You can also 
use various network deployment models that cloudstack offers for traffic 
isolation between VMs.

Cloudstack supports integration with NFS on iSCSI and SAN (Fibre Channel) as 
well as the backend, so you will have wider options for deployment of the 
backend storage arrays that will need to store the VDI images.

Compared to other IaaS solutions, cloudstack is vastly easier to setup, so 
starting with it would be a good idea.

As for VDI protocols, as I see it, the IaaS wouldn't know about them - they 
would logically sit on top of the IaaS.

Please let us know if you have any questions or need help with cloudstack 
itself.

Regards,
Vijay

-Original Message-
From: Iheb Kahloul [mailto:kahloul.i...@gmail.com] 
Sent: Tuesday, April 16, 2013 2:18 PM
To: dev@cloudstack.apache.org
Subject: Fwd: Desktop as a Service solution

Hello, i m working on a DaaS solution based on open source IaaS.
Do you advise me to base my work on CloudStack?
And if so, what are the VDI protocoles that it's using?

Ps: I have a realy short amount of time to provide a Beta version.
Thank you,
Iheb Kahloul


Re: [VOTE] Apache CloudStack 4.0.2

2013-04-16 Thread Marcus Sorensen
Alright, I've tested and verified that the patch/issue does NOT apply to
4.0 branch. I'll give this build a +1 (binding)


On Tue, Apr 16, 2013 at 3:31 PM, Marcus Sorensen wrote:

> I've actually got a patch that *I believe* goes along with a previous
> bugfix that is in 4.0.2 already, I'm trying to determine that and test
> right now. I should be able to let you know within an hour or two whether
> or not I think it should be applied.
>
>
> On Tue, Apr 16, 2013 at 2:45 PM, Joe Brockmeier  wrote:
>
>> On Tue, Apr 16, 2013, at 03:40 PM, Wei ZHOU wrote:
>> > Thanks, David.
>> > It is not in the list of changes.
>>
>> Sorry about that. I pulled the changes from Jira and it had no "fix
>> versions" tagged, so it didn't show up in the list of things that were
>> committed to 4.0.2. I've edited the Jira ticket to show 4.0.2 and 4.1.0.
>>
>> Best,
>>
>> jzb
>> --
>> Joe Brockmeier
>> j...@zonker.net
>> Twitter: @jzb
>> http://www.dissociatedpress.net/
>>
>
>


[RFC][FS]PVLAN for isolation within a VLAN

2013-04-16 Thread Sheng Yang
Hi all,

I am current working on a new mechanism to archive isolation for advance
shared network. It took advantage of PVLAN feature of Cisco switch, to
achieve isolation using a simpler way.

Here is the FS. You probably need to read references(in the link) to get an
idea of PVLAN first.

https://cwiki.apache.org/CLOUDSTACK/pvlan-for-isolation-within-a-vlan.html

Thanks!

--Sheng


RE: [ACS42][QA]Issues with latest Master Build (Xenserver)

2013-04-16 Thread Sangeetha Hariharan
I am not able to start user Vms successfully using the build from master. I am 
testing with Xenserver 6.0.2 hosts.
User Vm starts successfully but it gets to stopped state after few seconds.

deployVM reports success . But later on  cluster sync sees this Vm in stopped 
state and sends a StopCommand for this VM.

Is anyone seeing this issue?

Management server logs:

2013-04-16 16:21:13,805 DEBUG [xen.resource.CitrixResourceBase] 
(DirectAgent-21:null) 2. The VM i-2-10-VM is in Running state.
2013-04-16 16:21:13,806 DEBUG [agent.manager.DirectAgentAttache] 
(DirectAgent-21:null) Seq 5-1032650792: Response Received:
2013-04-16 16:21:13,806 DEBUG [agent.transport.Request] (DirectAgent-21:null) 
Seq 5-1032650792: Processing:  { Ans: , MgmtId: 7508777239729, via: 5, Ver: v1, 
Flags: 110, [
{"StartAnswer":{"vm":{"id":10,"name":"i-2-10-VM","bootloader":"PyGrub","type":"User","cpus":1,"minSpeed":100,"maxSpeed":100,"minRam":130023424,"maxRam":130023424,"arch":"x
86_64","os":"CentOS 5.3 
(64-bit)","bootArgs":"","rebootOnCrash":false,"enableHA":false,"limitCpuUse":false,"vncPassword":"10394489c81710b9","params":{},"uuid":"c0c0ee3a-af
91-4595-a1a3-4c3317489a03","disks":[{"id":10,"name":"ROOT-10","mountPoint":"/export/home/sangeetha/asf42-2/primary2","path":"eeeccffd-a86d-44ec-89f0-0e7923712ac2","size":8
589934592,"type":"ROOT","storagePoolType":"NetworkFilesystem","storagePoolUuid":"f483cba6-d7e9-38ad-ad5f-4c4971a2b06f","deviceId":0},{"id":10,"name":"test-123","size":0,"t
ype":"ISO","storagePoolType":"ISO","deviceId":3}],"nics":[{"deviceId":0,"networkRateMbps":200,"defaultNic":true,"uuid":"c726c8bf-3d17-4d9b-bae1-3e885461e4ab","ip":"10.1.1.
224","netmask":"255.255.255.0","gateway":"10.1.1.1","mac":"02:00:1a:b3:00:06","dns1":"72.52.126.11","dns2":"72.52.126.12","broadcastType":"Vlan","type":"Guest","broadcastU
ri":"vlan://2090","isolationUri":"vlan://2090","isSecurityGroupEnabled":false}]},"result":true,"wait":0}}]
 }
2013-04-16 16:21:13,806 DEBUG [agent.transport.Request] (Job-Executor-3:job-39) 
Seq 5-1032650792: Received:  { Ans: , MgmtId: 7508777239729, via: 5, Ver: v1, 
Flags: 110, {
 StartAnswer } }
2013-04-16 16:21:13,808 DEBUG [agent.manager.AgentAttache] 
(DirectAgent-21:null) Seq 5-1032650792: No more commands found
2013-04-16 16:21:13,838 DEBUG [cloud.network.NetworkModelImpl] 
(Job-Executor-3:job-39) Service SecurityGroup is not supported in the network 
id=205
2013-04-16 16:21:13,841 DEBUG [cloud.network.NetworkModelImpl] 
(Job-Executor-3:job-39) Service SecurityGroup is not supported in the network 
id=205
2013-04-16 16:21:13,846 DEBUG [cloud.capacity.CapacityManagerImpl] 
(Job-Executor-3:job-39) VM state transitted from :Starting to Running with 
event: OperationSucceededvm's
 original host id: 5 new host id: 5 host id before state transition: 5
2013-04-16 16:21:13,846 DEBUG [cloud.vm.VirtualMachineManagerImpl] 
(Job-Executor-3:job-39) Start completed for VM VM[User|again]


2013-04-16 16:22:04,873 DEBUG [agent.manager.AgentManagerImpl] 
(AgentManager-Handler-14:null) Ping from 4
2013-04-16 16:22:08,511 DEBUG [agent.manager.DirectAgentAttache] 
(DirectAgent-60:null) Ping from 5
2013-04-16 16:22:08,637 DEBUG [agent.manager.DirectAgentAttache] 
(DirectAgent-61:null) Ping from 1
2013-04-16 16:22:09,046 DEBUG [agent.manager.DirectAgentAttache] 
(DirectAgent-9:null) Seq 5-1032650756: Executing request
2013-04-16 16:22:09,289 WARN  [xen.resource.CitrixResourceBase] 
(DirectAgent-9:null) The VM is now missing marking it as Stopped i-2-10-VM
2013-04-16 16:22:09,290 DEBUG [agent.manager.DirectAgentAttache] 
(DirectAgent-9:null) Seq 5-1032650756: Response Received:
2013-04-16 16:22:09,290 DEBUG [agent.transport.Request] (DirectAgent-9:null) 
Seq 5-1032650756: Processing:  { Ans: , MgmtId: 7508777239729, via: 5, Ver: v1, 
Flags: 10, 
[{"ClusterSyncAnswer":{"_clusterId":2,"_newStates":{"i-2-10-VM":{"t":"b8f8a7e3-a6af-499f-9168-94691ca06b20","u":"Stopped"}},"_isExecuted":false,"result":true,"wait":0}}]
 }
2013-04-16 16:22:09,294 DEBUG [cloud.vm.VirtualMachineManagerImpl] 
(DirectAgent-9:null) VM i-2-10-VM: cs state = Running and realState = Stopped
2013-04-16 16:22:09,294 DEBUG [cloud.vm.VirtualMachineManagerImpl] 
(DirectAgent-9:null) VM i-2-10-VM: cs state = Running and realState = Stopped
2013-04-16 16:22:09,294 DEBUG [cloud.ha.HighAvailabilityManagerImpl] 
(DirectAgent-9:null) VM does not require investigation so I'm marking it as 
Stopped: VM[User|again]
2013-04-16 16:22:09,294 WARN  [apache.cloudstack.alerts] (DirectAgent-9:null)  
alertType:: 8 // dataCenterId:: 1 // podId:: 2 // clusterId:: null // message:: 
VM (name: again, id: 10) stopped unexpectedly on host id:5, availability zone 
id:1, pod id:2
2013-04-16 16:22:09,301 DEBUG [cloud.ha.HighAvailabilityManagerImpl] 
(DirectAgent-9:null) VM is not HA enabled so we're done.
2013-04-16 16:22:09,308 DEBUG [cloud.capacity.CapacityManagerImpl] 
(DirectAgent-9:null) VM state transitted from :Running to Stopping with event: 
StopRequestedvm's original host id: 

[PROPOSAL] [CLOUDSTACK-2056] DeploymentPlanner choice via ServiceOffering

2013-04-16 Thread Prachi Damle
I would like to propose a new feature - DeploymentPlanner choice via  
ServiceOffering

Jira ticket created: https://issues.apache.org/jira/browse/CLOUDSTACK-2056

This is to expose the choice of Deployment Planner via Service Offering and let 
Admin use a mix of Planners in a CS deployment.

The initial draft of the FS is here 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/DeploymentPlanner+choice+via+ServiceOffering

Please review and comment.

Thanks,
Prachi


Re: Firewall and LB network elements in inline mode

2013-04-16 Thread Sheng Yang
Basically, yes. Firewall got some minor changes as you seen in the code,
but in logic FW shouldn't concern about inline or not.

--Sheng


On Tue, Apr 16, 2013 at 1:30 AM, Koushik Das  wrote:

> Thanks Chiradeep, Sheng for the info.
>
> As I understand from the FS and also going over the code changes, only the
> LB element needs to modified to support inline mode and then the FW and LB
> inline scenario would be enabled on creating a network offering with inline
> mode enabled. There is no additional requirement on the FW element.
> Sheng, can you confirm if this is correct?
>
> > -Original Message-
> > From: Sheng Yang [mailto:sh...@yasker.org]
> > Sent: Tuesday, April 16, 2013 5:44 AM
> > To: 
> > Subject: Re: Firewall and LB network elements in inline mode
> >
> > 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
> > >
> > >
>


Create rbd primary storage fail in CS 4.0.1

2013-04-16 Thread Guangjian Liu
Create rbd primary storage fail in CS 4.0.1
Anybody can help about it!

Environment:
1. Server A: CS 4.0.1 OS: RHEL 6.2 x86-64
2. Server B: Ceph 0.56.4  OS: RHEL 6.2 x86-64
3. Server C: KVM/Qemu OS: Ubuntu 12.04
compile libvirt and Qemu as document
root@ubuntu:/usr/local/lib# virsh version
Compiled against library: libvirt 0.10.2
Using library: libvirt 0.10.2
Using API: QEMU 0.10.2
Running hypervisor: QEMU 1.0.0

Problem:
create primary storage fail with rbd device.

Fail log:
2013-04-16 16:27:14,224 DEBUG [cloud.storage.StorageManagerImpl]
(catalina-exec-9:null) createPool Params @ scheme - rbd storageHost -
10.0.0.41 hostPath - /cloudstack port - -1
2013-04-16 16:27:14,270 DEBUG [cloud.storage.StorageManagerImpl]
(catalina-exec-9:null) In createPool Setting poolId - 218 uuid -
5924a2df-d658-3119-8aba-f90307683206 zoneId - 4 podId - 4 poolName - ceph
2013-04-16 16:27:14,318 DEBUG [cloud.storage.StorageManagerImpl]
(catalina-exec-9:null) creating pool ceph on  host 18
2013-04-16 16:27:14,320 DEBUG [agent.transport.Request]
(catalina-exec-9:null) Seq 18-1625162275: Sending  { Cmd , MgmtId:
37528005876872, via: 18, Ver: v1, Flags: 100011,
[{"CreateStoragePoolCommand":{"add":true,"pool":{"id":218,"uuid":"5924a2df-d658-3119-8aba-f90307683206","host":"10.0.0.41","path":"cloudstack","userInfo":":","port":6789,"type":"RBD"},"localPath":"/mnt//3cf4f0e8-781d-39d8-b81c-9896da212335","wait":0}}]
}
2013-04-16 16:27:14,323 DEBUG [agent.transport.Request]
(AgentManager-Handler-2:null) Seq 18-1625162275: Processing:  { Ans: ,
MgmtId: 37528005876872, via: 18, Ver: v1, Flags: 10,
[{"Answer":{"result":true,"details":"success","wait":0}}] }
2013-04-16 16:27:14,323 DEBUG [agent.transport.Request]
(catalina-exec-9:null) Seq 18-1625162275: Received:  { Ans: , MgmtId:
37528005876872, via: 18, Ver: v1, Flags: 10, { Answer } }
2013-04-16 16:27:14,323 DEBUG [agent.manager.AgentManagerImpl]
(catalina-exec-9:null) Details from executing class
com.cloud.agent.api.CreateStoragePoolCommand: success
2013-04-16 16:27:14,323 DEBUG [cloud.storage.StorageManagerImpl]
(catalina-exec-9:null) In createPool Adding the pool to each of the hosts
2013-04-16 16:27:14,323 DEBUG [cloud.storage.StorageManagerImpl]
(catalina-exec-9:null) Adding pool ceph to  host 18
2013-04-16 16:27:14,326 DEBUG [agent.transport.Request]
(catalina-exec-9:null) Seq 18-1625162276: Sending  { Cmd , MgmtId:
37528005876872, via: 18, Ver: v1, Flags: 100011,
[{"ModifyStoragePoolCommand":{"add":true,"pool":{"id":218,"uuid":"5924a2df-d658-3119-8aba-f90307683206","host":"10.0.0.41","path":"cloudstack","userInfo":":","port":6789,"type":"RBD"},"localPath":"/mnt//3cf4f0e8-781d-39d8-b81c-9896da212335","wait":0}}]
}
2013-04-16 16:27:14,411 DEBUG [agent.transport.Request]
(AgentManager-Handler-6:null) Seq 18-1625162276: Processing:  { Ans: ,
MgmtId: 37528005876872, via: 18, Ver: v1, Flags: 10,
[{"Answer":{"result":false,"details":"java.lang.NullPointerException\n\tat
com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.createStoragePool(LibvirtStorageAdaptor.java:462)\n\tat
com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:57)\n\tat
com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.execute(LibvirtComputingResource.java:2087)\n\tat
com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(LibvirtComputingResource.java:1053)\n\tat
com.cloud.agent.Agent.processRequest(Agent.java:518)\n\tat
com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:831)\n\tat
com.cloud.utils.nio.Task.run(Task.java:83)\n\tat
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1146)\n\tat
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)\n\tat
java.lang.Thread.run(Thread.java:679)\n","wait":0}}] }
2013-04-16 16:27:14,412 DEBUG [agent.transport.Request]
(catalina-exec-9:null) Seq 18-1625162276: Received:  { Ans: , MgmtId:
37528005876872, via: 18, Ver: v1, Flags: 10, { Answer } }
2013-04-16 16:27:14,412 DEBUG [agent.manager.AgentManagerImpl]
(catalina-exec-9:null) Details from executing class
com.cloud.agent.api.ModifyStoragePoolCommand: java.lang.NullPointerException
at
com.cloud.hypervisor.kvm.storage.LibvirtStorageAdaptor.createStoragePool(LibvirtStorageAdaptor.java:462)
at
com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.createStoragePool(KVMStoragePoolManager.java:57)
at
com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.execute(LibvirtComputingResource.java:2087)
at
com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(LibvirtComputingResource.java:1053)
at com.cloud.agent.Agent.processRequest(Agent.java:518)
at com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:831)
at com.cloud.utils.nio.Task.run(Task.java:83)
at
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1146)
at
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExe

Re: [ACS41][DOCS] Generate diff of APIs changed/added/deleted?

2013-04-16 Thread David Nalley
On Tue, Apr 9, 2013 at 1:36 PM, Alena Prokharchyk <
alena.prokharc...@citrix.com> wrote:

> Joe, here is the file:
>
> ./server/src/com/cloud/api/doc/ApiXmlDocReader.java
>
>
> I used to run it this way:
>
> Java -cp  com.cloud.api.doc.ApiXmlDocReader -old  old xml file> -new  -d 
>
>
> -Alena.
>
> Thanks for this - I do think that jenkins was running this at one time,
but apparently not anymore, we should probably re-automate this, but in the
interim, I've added directions for doing this to the wiki page:

https://cwiki.apache.org/confluence/display/CLOUDSTACK/How+To+Generate+CloudStack+API+Documentation

--David


[ACS41] Can we resolve CLOUDSTACK-2027?

2013-04-16 Thread Chip Childers
Edison,

CLOUDSTACK-2027 is listed as critical, and tied to 4.1.0.  Any chance
you can review it please?  The description mentions that it might just
be a configuration error, however I'd like to be sure before cutting the
RC.

Thanks!

-chip


Re: [PATCH][ACS41] CLOUDSTACK-2048 - When Connectivity is added to a VpcOffering the provider is set to VPCVirtualRouter, should be NiciraNvp

2013-04-16 Thread Chip Childers
On Tue, Apr 16, 2013 at 03:36:14PM +, Hugo Trippaers wrote:
> Chip,
> 
> Can you apply this fix to 4.1?  There is a better fix for this in master, but 
> that is more involved. This fix will allow NiciraNvp users to create VPC's 
> without having to edit the database and has only minimal impact on 4.1 branch.
> 
> The patch is in the ticket as it can't be applied to master.
> 
> Cheers,
> 
> Hugo
>

Done


Re: [PATCH][ACS41] CLOUDSTACK-2033 Fix usage server startup and logging

2013-04-16 Thread Chip Childers
On Tue, Apr 16, 2013 at 03:42:09PM +, Hugo Trippaers wrote:
> Please cherry pick commits ae16f332132a5eb1e3ff85a0c435dbe25a1d6299 
> 574e716352290f9c34bfa7bd3134a13e0caa8da2.
> 
> Packaging fixes for usage server, but there are some code changes implemented 
> to fix the startup.
> 
> 
> Cheers,
> 
> Hugo
> 

Done


Re: [ACS41][Patch Request]

2013-04-16 Thread Chip Childers
On Tue, Apr 16, 2013 at 12:07:52PM -0600, Marcus Sorensen wrote:
> commit 202930fd554ddc9a8e78cf39ae54a4e3a56d3904
> Author: Marcus Sorensen 
> Date:   Tue Apr 16 12:07:15 2013 -0600
> 
>  CLOUDSTACK-2051 Allow KVM HA Monitor to verify that a NfsStoragePool is
> valid before running the script that mounts it and touches the HA files.
> 
> Signed-off-by: Marcus Sorensen  1366135635
> -0600

Done


Re: [ACS41][DOCS] Generate diff of APIs changed/added/deleted?

2013-04-16 Thread Jessica Tomechak
On Tue, Apr 16, 2013 at 5:02 PM, David Nalley  wrote:

> On Tue, Apr 9, 2013 at 1:36 PM, Alena Prokharchyk <
> alena.prokharc...@citrix.com> wrote:
>
> > Joe, here is the file:
> >
> > ./server/src/com/cloud/api/doc/ApiXmlDocReader.java
> >
> >
> > I used to run it this way:
> >
> > Java -cp  com.cloud.api.doc.ApiXmlDocReader -old  > old xml file> -new  -d 
> >
> >
> > -Alena.
> >
> > Thanks for this - I do think that jenkins was running this at one time,
> but apparently not anymore, we should probably re-automate this, but in the
> interim, I've added directions for doing this to the wiki page:
>
>
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/How+To+Generate+CloudStack+API+Documentation
>
> --David
>

David, thanks for updating the wiki. It's very helpful.

Jessica T.


Re: CloudStack coming meetup in China

2013-04-16 Thread Gavin Lee
Done.
Thanks for reminding me.



On Tue, Apr 16, 2013 at 10:53 PM, Joe Brockmeier  wrote:

> Hi Gavin,
>
> Can you also update the market...@cloudstack.apache.org list?
>
> On Tue, Apr 16, 2013, at 09:46 AM, Gavin Lee wrote:
> > There were several meetups in China in Q1, 2013, mostly in Beijing and
> > Shanghai.
> > From now on, I'll summerize the relate info(attendees, sharing materials,
> > etc) and keep dev list updating about the coming meetup.
> >
> > The recent meetup is April 20th from 1:30PM to 5:30PM, there are 3
> > topics:
> > 1. Update to attendees about CloudStack 4.1 features
> > 2. CloudStack Best Practice in PPTV; PPTV OPS Architect will share their
> > real production environment which has more than 100 physical hosts.
> > 3. Setup development environment and test on Devcloud
> >
> > I'll also update to attendee about how to deeply involve in CloudStack
> > community.
> >
> > For detailed info, please refer to(Sorry, only Chinese):
> > http://www.cloudstack-china.org/2013/04/1858.html
> >
> > --
> > Gavin
>
>
> Best,
>
> jzb
> --
> Joe Brockmeier
> j...@zonker.net
> Twitter: @jzb
> http://www.dissociatedpress.net/
>



-- 
Gavin


CS 4.0.1 | KVM | CentOS 6.3

2013-04-16 Thread Maurice Lawler
Greetings,

I am having this FLOOD my log. I will have you know, this is ONE server ONE 
HOST. KVM | CentOS 6.3 | Cloudstack 4.0.1

I have found restarting cloud management solves the issue for awhile, then it 
starts again.

Please advise.

- Maurice

 Eror Log 

2013-04-16 20:58:38,993 DEBUG [cloud.consoleproxy.ConsoleProxyManagerImpl] 
(consoleproxy-1:null) Zone 1 is ready to launch console proxy
2013-04-16 20:58:39,137 DEBUG [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Detected management node left, id:1, 
nodeIP:MYIPADDRESSHERE
2013-04-16 20:58:39,137 INFO  [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Trying to connect to MYIPADDRESSHERE
2013-04-16 20:58:39,138 INFO  [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Management node 1 is detected inactive by timestamp 
but is pingable
2013-04-16 20:58:39,264 DEBUG 
[network.router.VirtualNetworkApplianceManagerImpl] 
(RouterStatusMonitor-1:null) Found 0 routers.
2013-04-16 20:58:40,637 DEBUG [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Detected management node left, id:1, 
nodeIP:MYIPADDRESSHERE
2013-04-16 20:58:40,637 INFO  [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Trying to connect to MYIPADDRESSHERE
2013-04-16 20:58:40,637 INFO  [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Management node 1 is detected inactive by timestamp 
but is pingable
2013-04-16 20:58:42,062 DEBUG [agent.manager.AgentManagerImpl] 
(AgentManager-Handler-13:null) Ping from 3
2013-04-16 20:58:42,063 DEBUG [agent.manager.AgentManagerImpl] 
(AgentManager-Handler-14:null) Ping from 4
2013-04-16 20:58:42,138 DEBUG [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Detected management node left, id:1, 
nodeIP:MYIPADDRESSHERE
2013-04-16 20:58:42,138 INFO  [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Trying to connect to MYIPADDRESSHERE
2013-04-16 20:58:42,138 INFO  [cloud.cluster.ClusterManagerImpl] 
(Cluster-Heartbeat-1:null) Management node 1 is detected inactive by timestamp 
but is pingable

===

Re: Review Request: 4.1 deb packaging tomcat fixes

2013-04-16 Thread Animesh Chaturvedi


> On March 31, 2013, 3:03 p.m., Wido den Hollander wrote:
> > Ship It!

Wido is this in master now?


- Animesh


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


On March 2, 2013, 11:34 p.m., Noa Resare wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9714/
> ---
> 
> (Updated March 2, 2013, 11:34 p.m.)
> 
> 
> Review request for cloudstack and Wido den Hollander.
> 
> 
> Description
> ---
> 
> Changes isolated to the deb package build files, to fix
> failure to start management server from deb built packages
> 
> * Adds the tomcat6 bootstrap jar to outer classpath
> * Removes install of cloud-server-4.1.0-SNAPSHOT.jar
>   in /usr/share/cloudstack-management/lib. This causes
>   /usr/share/cloudstack-management structure to be in the
>   way tomcat expects it to
> * Update paths to logfiles, which restores log4j functionality
> 
> CLOUDSTACK-1490: 4.1 deb management fails to start
> 
> 
> This addresses bug CLOUDSTACK-1490.
> 
> 
> Diffs
> -
> 
>   debian/rules ba9056e 
>   packaging/debian/replace.properties fee1def 
> 
> Diff: https://reviews.apache.org/r/9714/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Noa Resare
> 
>



Re: Review Request: Documentation for the new features: Optional Public IP assignment for EIP with Basic Zone, Enable L4-L7 network services in the shared network in the advanced zone, and User-provid

2013-04-16 Thread Animesh Chaturvedi


> On March 4, 2013, 9:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> > Ship It!
> 
> Radhika PC wrote:
> will merge once Murali merges the code into Master.

Radhika both 312 and 265 are closed now, can this review be closed now


- Animesh


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


On Jan. 15, 2013, 12:34 p.m., Radhika PC wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8180/
> ---
> 
> (Updated Jan. 15, 2013, 12:34 p.m.)
> 
> 
> Review request for cloudstack, Chip Childers, Venkata Siva Vijayendra 
> Bhamidipati, Jessica Tomechak, Murali Reddy, and Joe Brockmeier.
> 
> 
> Description
> ---
> 
> This review request is for the documentation written for the following 
> features:
> 
> Optional Public IP assignment for EIP with Basic Zone
> User-provided host name in vCenter 
> Enable L4-L7 network services in the shared network in the advanced zone 
> 
> 
> Diffs
> -
> 
>   docs/en-US/Installation_Guide.xml 2f60aca 
>   docs/en-US/about-working-with-vms.xml 47153e2 
>   docs/en-US/append-displayname-vms.xml PRE-CREATION 
>   docs/en-US/creating-network-offerings.xml ab56920 
>   docs/en-US/elastic-ip.xml PRE-CREATION 
>   docs/en-US/network-offerings.xml c1fd79d 
>   docs/en-US/networks.xml a7b9ea1 
>   docs/en-US/set-up-network-for-users.xml 2b40923 
>   docs/en-US/using-multiple-guest-networks.xml 9076a81 
>   docs/en-US/using-netscaler-load-balancers.xml c2044de 
>   docs/en-US/virtual-machines.xml 7c74932 
> 
> Diff: https://reviews.apache.org/r/8180/diff/
> 
> 
> Testing
> ---
> 
> Patch cleanly applies. doc is reviewed by QA
> 
> 
> Thanks,
> 
> Radhika PC
> 
>



Thoughts on API changes in release notes

2013-04-16 Thread David Nalley
Hi folks,

I've played with a table and itemizedlist, and still not happy with how to
represent changes in the API..

Here's some plain text of what I am trying to figure out how to format.


createVlanIpRange  Request:  New parameters: endipv6 (optional),
ip6cidr (optional), ip6gateway (optional), startipv6 (optional)
Changed parameters: startip (old version - required, new version -
optional)   Response:  New parameters: endipv6, ip6cidr, ip6gateway,
startipv6

listNetworkOfferings  Response:  New parameters: ispersistent
registerTemplate  Response:  New parameters: sshkeyenabled

 addTrafficMonitor  Request:  New parameters: excludezones (optional),
includezones (optional)   Response:  Removed parameters:
privateinterface, privatezone, publicinterface, publiczone,
usageinterface, username


RE: Thoughts on API changes in release notes

2013-04-16 Thread Radhika Puthiyetath
How about itemized list within a table of six rows:

API | Description|  Request: Changed Parameter| Request: New Parameter 
|Response: Changed Parameter| Response: New Parameter 

Or

API | Description|  Changed Parameter| New Parameter |

Under Changed/ New Parameter, itemized lists for both response and request
 

-Original Message-
From: David Nalley [mailto:da...@gnsa.us] 
Sent: Wednesday, April 17, 2013 8:26 AM
To: dev@cloudstack.apache.org
Subject: Thoughts on API changes in release notes

Hi folks,

I've played with a table and itemizedlist, and still not happy with how to 
represent changes in the API..

Here's some plain text of what I am trying to figure out how to format.


createVlanIpRange  Request:  New parameters: endipv6 (optional), ip6cidr 
(optional), ip6gateway (optional), startipv6 (optional) Changed parameters: 
startip (old version - required, new version -
optional)   Response:  New parameters: endipv6, ip6cidr, ip6gateway,
startipv6

listNetworkOfferings  Response:  New parameters: ispersistent registerTemplate  
Response:  New parameters: sshkeyenabled

 addTrafficMonitor  Request:  New parameters: excludezones (optional),
includezones (optional)   Response:  Removed parameters:
privateinterface, privatezone, publicinterface, publiczone, usageinterface, 
username


RE: Review Request: Documentation for the new features: Optional Public IP assignment for EIP with Basic Zone, Enable L4-L7 network services in the shared network in the advanced zone, and User-provid

2013-04-16 Thread Radhika Puthiyetath
Hi Animesh/ Murali

Thank you for the nudge.

Will submit once I am back to work.

-Radhika

-Original Message-
From: Animesh Chaturvedi [mailto:nore...@reviews.apache.org] On Behalf Of 
Animesh Chaturvedi
Sent: Wednesday, April 17, 2013 7:42 AM
To: Joe Brockmeier; Vijayendra Bhamidipati; Chip Childers; Jessica Tomechak; 
Murali Reddy
Cc: Radhika Puthiyetath; Animesh Chaturvedi; cloudstack
Subject: Re: Review Request: Documentation for the new features: Optional 
Public IP assignment for EIP with Basic Zone, Enable L4-L7 network services in 
the shared network in the advanced zone, and User-provided hostname in vCenter



> On March 4, 2013, 9:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> > Ship It!
> 
> Radhika PC wrote:
> will merge once Murali merges the code into Master.

Radhika both 312 and 265 are closed now, can this review be closed now


- Animesh


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


On Jan. 15, 2013, 12:34 p.m., Radhika PC wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8180/
> ---
> 
> (Updated Jan. 15, 2013, 12:34 p.m.)
> 
> 
> Review request for cloudstack, Chip Childers, Venkata Siva Vijayendra 
> Bhamidipati, Jessica Tomechak, Murali Reddy, and Joe Brockmeier.
> 
> 
> Description
> ---
> 
> This review request is for the documentation written for the following 
> features:
> 
> Optional Public IP assignment for EIP with Basic Zone User-provided 
> host name in vCenter Enable L4-L7 network services in the shared 
> network in the advanced zone
> 
> 
> Diffs
> -
> 
>   docs/en-US/Installation_Guide.xml 2f60aca 
>   docs/en-US/about-working-with-vms.xml 47153e2 
>   docs/en-US/append-displayname-vms.xml PRE-CREATION 
>   docs/en-US/creating-network-offerings.xml ab56920 
>   docs/en-US/elastic-ip.xml PRE-CREATION 
>   docs/en-US/network-offerings.xml c1fd79d 
>   docs/en-US/networks.xml a7b9ea1 
>   docs/en-US/set-up-network-for-users.xml 2b40923 
>   docs/en-US/using-multiple-guest-networks.xml 9076a81 
>   docs/en-US/using-netscaler-load-balancers.xml c2044de 
>   docs/en-US/virtual-machines.xml 7c74932
> 
> Diff: https://reviews.apache.org/r/8180/diff/
> 
> 
> Testing
> ---
> 
> Patch cleanly applies. doc is reviewed by QA
> 
> 
> Thanks,
> 
> Radhika PC
> 
>



Re: [ACS41][Patch Request]

2013-04-16 Thread Marcus Sorensen
Thanks, here's one more related to it...

commit 2057221f4f1fd5afde422b367fc416d4e44275cb
Author: Marcus Sorensen 
Date:   Tue Apr 16 22:18:38 2013 -0600

 CLOUDSTACK-2051 - The previous patch ensures that holes in deleting
storage
pools the proper way won't cause problems for KVM HA Monitor, this
patch closes
holes. Call the KVMStoragePool deleteStoragePool that properly removes
it from
the KVMHA hashmap, instead of the pools direct delete() call.

Signed-off-by: Marcus Sorensen  1366172318
-0600


On Tue, Apr 16, 2013 at 6:38 PM, Chip Childers wrote:

> On Tue, Apr 16, 2013 at 12:07:52PM -0600, Marcus Sorensen wrote:
> > commit 202930fd554ddc9a8e78cf39ae54a4e3a56d3904
> > Author: Marcus Sorensen 
> > Date:   Tue Apr 16 12:07:15 2013 -0600
> >
> >  CLOUDSTACK-2051 Allow KVM HA Monitor to verify that a
> NfsStoragePool is
> > valid before running the script that mounts it and touches the HA
> files.
> >
> > Signed-off-by: Marcus Sorensen  1366135635
> > -0600
>
> Done
>


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

2013-04-16 Thread Abhinav Roy
Sure, I will add those tests as well. Thanks for your comments.


Regards,
Abhinav

-Original Message-
From: Likitha Shetty [mailto:likitha.she...@citrix.com] 
Sent: Wednesday, April 17, 2013 2:22 AM
To: dev@cloudstack.apache.org
Subject: RE: [ACS42][QA] Test cases for Dedicated Resources - Public IP 
Addresses and VLANs per Tenant feature uploaded

Hi Abhinav,

Thanks for the test plan. 

Could you add test cases to verify the usage related changes? 
Otherwise the plan looks good to me.

Thanks,
Likitha

>-Original Message-
>From: Abhinav Roy [mailto:abhinav@citrix.com]
>Sent: Monday, April 15, 2013 8:41 PM
>To: dev@cloudstack.apache.org
>Subject: [ACS42][QA] Test cases for Dedicated Resources - Public IP 
>Addresses and VLANs per Tenant feature uploaded
>
>Hi,
>
>
>
>I've uploaded some test cases for the feature here:
>https://cwiki.apache.org/confluence/display/CLOUDSTACK/Dedicated+Resour
>ces
>+-+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: Review Request: Cloudstack-701 Support for non contiguous vlan ranges.

2013-04-16 Thread Abhinandan Prateek


> On April 12, 2013, 9:53 a.m., Abhinandan Prateek wrote:
> > Ship It!
> 
> Abhinandan Prateek wrote:
> The patch does not apply, can you resubmit after checking it.
> 
> bharat kumar wrote:
> uploaded the new diff

committed


- Abhinandan


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


On April 16, 2013, 2:53 p.m., bharat kumar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10238/
> ---
> 
> (Updated April 16, 2013, 2:53 p.m.)
> 
> 
> Review request for cloudstack and Abhinandan Prateek.
> 
> 
> Description
> ---
> 
> Cloudstack-701: Support for non contiguous vlan ranges.
> 
> 
> Diffs
> -
> 
>   api/src/com/cloud/network/NetworkService.java 066009b 
>   api/src/com/cloud/network/PhysicalNetwork.java a2044a6 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 37cb59f 
>   
> api/src/org/apache/cloudstack/api/command/admin/network/UpdatePhysicalNetworkCmd.java
>  06cf38d 
>   server/src/com/cloud/api/ApiResponseHelper.java cfe0e00 
>   server/src/com/cloud/configuration/ConfigurationManagerImpl.java c9ae854 
>   server/src/com/cloud/dc/dao/DataCenterVnetDao.java 79e91c4 
>   server/src/com/cloud/dc/dao/DataCenterVnetDaoImpl.java 5ded0f4 
>   server/src/com/cloud/network/ExternalFirewallDeviceManagerImpl.java c2038e5 
>   server/src/com/cloud/network/NetworkServiceImpl.java 70d1d0d 
>   server/src/com/cloud/network/dao/PhysicalNetworkVO.java e5ffcfb 
>   server/src/com/cloud/network/guru/GuestNetworkGuru.java 9c0205a 
>   server/test/com/cloud/network/MockNetworkManagerImpl.java 9042f03 
>   server/test/com/cloud/network/UpdatePhysicalNetworkTest.java PRE-CREATION 
>   server/test/com/cloud/vpc/MockNetworkManagerImpl.java 3a585ce 
>   test/integration/smoke/test_non_contigiousvlan.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/10238/diff/
> 
> 
> Testing
> ---
> 
> Tested on latest master.
> 
> 
> Thanks,
> 
> bharat kumar
> 
>



Re: [ACS41] Can we resolve CLOUDSTACK-2027?

2013-04-16 Thread Chiradeep Vittal
According to 
https://issues.apache.org/jira/browse/CLOUDSTACK-934

The bug reporter (Angie) is trying to use a hand-rolled version of RHEL
(libvirt and ovs).
I'd agree with Edison's assessment that we should only support whatever
the vendor supports.

On 4/16/13 5:36 PM, "Chip Childers"  wrote:

>Edison,
>
>CLOUDSTACK-2027 is listed as critical, and tied to 4.1.0.  Any chance
>you can review it please?  The description mentions that it might just
>be a configuration error, however I'd like to be sure before cutting the
>RC.
>
>Thanks!
>
>-chip



Re: CS 4.0.1 | KVM | CentOS 6.3

2013-04-16 Thread Chiradeep Vittal
Did the management server ip change? It somehow thinks there's 2
management servers.

On 4/16/13 7:00 PM, "Maurice Lawler"  wrote:

>Greetings,
>
>I am having this FLOOD my log. I will have you know, this is ONE server
>ONE HOST. KVM | CentOS 6.3 | Cloudstack 4.0.1
>
>I have found restarting cloud management solves the issue for awhile,
>then it starts again.
>
>Please advise.
>
>- Maurice
>
> Eror Log 
>
>2013-04-16 20:58:38,993 DEBUG
>[cloud.consoleproxy.ConsoleProxyManagerImpl] (consoleproxy-1:null) Zone 1
>is ready to launch console proxy
>2013-04-16 20:58:39,137 DEBUG [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Detected management node left, id:1,
>nodeIP:MYIPADDRESSHERE
>2013-04-16 20:58:39,137 INFO  [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Trying to connect to MYIPADDRESSHERE
>2013-04-16 20:58:39,138 INFO  [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Management node 1 is detected inactive by
>timestamp but is pingable
>2013-04-16 20:58:39,264 DEBUG
>[network.router.VirtualNetworkApplianceManagerImpl]
>(RouterStatusMonitor-1:null) Found 0 routers.
>2013-04-16 20:58:40,637 DEBUG [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Detected management node left, id:1,
>nodeIP:MYIPADDRESSHERE
>2013-04-16 20:58:40,637 INFO  [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Trying to connect to MYIPADDRESSHERE
>2013-04-16 20:58:40,637 INFO  [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Management node 1 is detected inactive by
>timestamp but is pingable
>2013-04-16 20:58:42,062 DEBUG [agent.manager.AgentManagerImpl]
>(AgentManager-Handler-13:null) Ping from 3
>2013-04-16 20:58:42,063 DEBUG [agent.manager.AgentManagerImpl]
>(AgentManager-Handler-14:null) Ping from 4
>2013-04-16 20:58:42,138 DEBUG [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Detected management node left, id:1,
>nodeIP:MYIPADDRESSHERE
>2013-04-16 20:58:42,138 INFO  [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Trying to connect to MYIPADDRESSHERE
>2013-04-16 20:58:42,138 INFO  [cloud.cluster.ClusterManagerImpl]
>(Cluster-Heartbeat-1:null) Management node 1 is detected inactive by
>timestamp but is pingable
>
>===



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

2013-04-16 Thread Venkata Siva Vijayendra Bhamidipati


> On April 16, 2013, 4:31 a.m., Prasanna Santhanam wrote:
> > tools/marvin/marvin/cloudstackConnection.py, line 61
> > 
> >
> > 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.

I saw that make_request() was calling into make_request_with_auth(), so I put 
in the default value for httpmethod as GET, but missed the fact that 
make_request_with_auth() is calling itself when retrying - thanks for catching 
that - I've put in a default value of GET for this parameter. When a unittest 
calls this function, it would be expected to pass in all arguments in case it 
wants to use POST, but existing tests that simply use GET don't need to 
change.. does that look ok? Or is there a better way for me to implement this?


> On April 16, 2013, 4:31 a.m., Prasanna Santhanam wrote:
> > tools/marvin/marvin/cloudstackConnection.py, line 77
> > 
> >
> > Are you sure you don't want to urlencode the POST data? 
> >

Yes that is definitely better than doing requestUrl = "&".join(["=".join([r[0], 
urllib.quote_plus(str(r[1]))]) for r in request]) , which won't take care of 
spaces either.. have made the change here and even in 
make_request_without_auth()..


> On April 16, 2013, 4:31 a.m., Prasanna Santhanam wrote:
> > tools/marvin/marvin/cloudstackConnection.py, line 79
> > 
> >
> > 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.

I changed both cases to use urllib2.Request(), but still it's still the same 
number of lines :) Is there a way I can merge those two separate calls to 
urllib2.Request(url) and urllib2.Request(url, requestUrl) into a single call?


- Venkata Siva Vijayendra


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


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: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-16 Thread Rohit Yadav
On Wed, Apr 17, 2013 at 11:00 AM, Venkata Siva Vijayendra Bhamidipati <
vijayendra.bhamidip...@citrix.com> wrote:

>
>
> > On April 16, 2013, 4:31 a.m., Prasanna Santhanam wrote:
> > > tools/marvin/marvin/cloudstackConnection.py, line 61
> > > <
> https://reviews.apache.org/r/10294/diff/4/?file=281521#file281521line61>
> > >
> > > 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.
>
> I saw that make_request() was calling into make_request_with_auth(), so I
> put in the default value for httpmethod as GET, but missed the fact that
> make_request_with_auth() is calling itself when retrying - thanks for
> catching that - I've put in a default value of GET for this parameter. When
> a unittest calls this function, it would be expected to pass in all
> arguments in case it wants to use POST, but existing tests that simply use
> GET don't need to change.. does that look ok? Or is there a better way for
> me to implement this?
>
>
> > On April 16, 2013, 4:31 a.m., Prasanna Santhanam wrote:
> > > tools/marvin/marvin/cloudstackConnection.py, line 77
> > > <
> https://reviews.apache.org/r/10294/diff/4/?file=281521#file281521line77>
> > >
> > > Are you sure you don't want to urlencode the POST data?
> > >
>
> Yes that is definitely better than doing requestUrl =
> "&".join(["=".join([r[0], urllib.quote_plus(str(r[1]))]) for r in request])
> , which won't take care of spaces either.. have made the change here and
> even in make_request_without_auth()..
>
>
> > On April 16, 2013, 4:31 a.m., Prasanna Santhanam wrote:
> > > tools/marvin/marvin/cloudstackConnection.py, line 79
> > > <
> https://reviews.apache.org/r/10294/diff/4/?file=281521#file281521line79>
> > >
> > > 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.
>
> I changed both cases to use urllib2.Request(), but still it's still the
> same number of lines :) Is there a way I can merge those two separate calls
> to urllib2.Request(url) and urllib2.Request(url, requestUrl) into a single
> call?
>


If it does not cost much, let's introduce a new dependency for requesting
stuff (get or post etc.) using "requests" [1] for both marvin and
cloudmonkey. Vijay, you can write your own requester, as it's not much
effort required to implement your own requester once you how signature is
calculated.

IMO, cloudstackConnection could be rewritten using latest ways and better
libraries as it's not much code and is totally do-able with much less
effort. A lot of flows for auth or non-auth are inter-dependent, the
json->obj and obj->json parts and requesting logic is not straight forward
(each method does *only* one thing). We need to make it generic by
providing both raw and processed outputs, for example the result expects to
return an obj and not json (maybe I need json and not an obj), one reason
why I chose to write my own for cloudmonkey after I tried to reuse it.
Lastly, host marvin on pypi so it's easier for any developer to get it off
the shelf and start hacking some cool clients.

Cheers.

[1] http://docs.python-requests.org/en/latest/


>
>
> - Venkata Siva Vijayendra
>
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10294/#review19245
> ---
>
>
> 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+enhancementsfor
>  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/Use

  1   2   >