cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nitin Mehta <>
Subject Re: Regarding ssvm-check script
Date Fri, 14 Jun 2013 06:38:32 GMT
Hi Rajesh,
Please find my comments inline

On 13/06/13 10:40 PM, "Rajesh Battala" <> wrote:

>Hi All,
>While fixing an issue ( )in
>ssvm-check script I figured out some issues.
>1.There are two ssvm_check scripts(duplicates).
>When building the code, these scripts will go to,
> will be packaged into systemvm.iso.
>systemvm-descriptor.xml will define what all the scripts should package.
>As per the descriptor xml,  the ssvm-check script under console-proxy is
>getting into

Shouldn't it be the other way round ? I mean the ssvm script under
secondary-storage should have come in ?

>I had verified the ssvm-check script with the fix under console-proxy.
>The is getting update properly and making into systemvm.iso.
>And ssvm is getting the right script now.
>Changes made in script under
>./services/secondary-storage/scripts/ is not getting into
>I feel the script is redundant and creating confusion.
>Can we remove the script in one location?

I would remove it from console-proxy for the sake of consistency and make
sure the one under secondary-storage gets in. Also while doing so
hopefully the final location (folder structure) of the script is not
disturbed in the ssvm

>Rajesh Battala

View raw message