incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Animesh Chaturvedi <animesh.chaturv...@citrix.com>
Subject RE: Review Request: Proposal & BUG-ID: CLOUDSTACK-1252 Summary: Failed to download default template in VMware
Date Thu, 14 Mar 2013 20:56:21 GMT
Folks please review and apply this patch. It is  for an important 4.1 fix 

> -----Original Message-----
> From: Pradeep Soundararajan [mailto:noreply@reviews.apache.org] On
> Behalf Of Pradeep Soundararajan
> Sent: Wednesday, March 13, 2013 5:47 AM
> To: David Nalley; Noa Resare; Hugo Trippaers; Alex Huang; Wido den
> Hollander; Chip Childers
> Cc: cloudstack; Pradeep Soundararajan
> Subject: Review Request: Proposal & BUG-ID: CLOUDSTACK-1252 Summary:
> Failed to download default template in VMware
> 
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9900/
> -----------------------------------------------------------
> 
> Review request for cloudstack, David Nalley, Chip Childers, Hugo Trippaers,
> Alex Huang, Wido den Hollander, and Noa Resare.
> 
> 
> Description
> -------
> 
> BUG-ID: CLOUDSTACK-1252 Summary: Failed to download default template
> in VMware
> 
> Proposing to modify package.sh script. Package as per OSS or NONOSS
> arguments.
> 
> Signed-off-by: Pradeep <pradeep.soundararajan@citrix.com>
> --------------------------------------------------------------------------------------
> 
> Now package.sh will act based on the below usage i.e. with or without
> arguments:
> 
>  ./package.sh -h
> 
> usage: ./package.sh [-p|--pack] [-h|--help] [ARGS]
> 
> The commonly used Arguments are:
> oss|OSS         To package OSS specific
> nonoss|NONOSS   To package NONOSS specific
> 
> Examples: ./package.sh -p|--pack oss|OSS
>           ./package.sh -p|--pack nonoss|NONOSS
>           ./package.sh (Default OSS)
> 
> ---------------------------------------------------------------------------------------
> 
> 
> Diffs
> -----
> 
>   packaging/centos63/cloud.spec
> 1b2c9a9633e161081809c3aa25574fdb863d4d0b
>   packaging/centos63/package.sh
> 2515ecba11fee3ea8dd5f85884861585392a26cd
> 
> Diff: https://reviews.apache.org/r/9900/diff/
> 
> 
> Testing
> -------
> 
> I have tested these changes. They are working well. Please review and
> submit it if you are fine.
> 
> In future, we can even change the mvn commands as per our need from
> cloud.spec.
> 
> 
> Thanks,
> 
> Pradeep Soundararajan

Mime
View raw message