Fixed stateful update in image_to_live.

image_to_live was not providing a URL to the stateful_update script executed on the remote machine. Without the URL, stateful_update would default to reading /mnt/stateful_partition/etc/lsb-release for the URL, which is not setup by the script.

I've modified image_to_live to convert the dev server url into the proper form to access the stateful.image.gz and pass it along to stateful_update on the remote system.

Change-Id: I77971c83b7efef5f053fda7fcd2585ca73f3134a

BUG=6705
TEST=Imaged several machines using Dev Server.

[DaleCurtis, 2010-09-14 03:13:21]

PTAL

[ericli, 2010-09-14 03:44:41]

LGTM with nit.

http://codereview.chromium.org/3417001/diff/1/2
File image_to_live.sh (right):

http://codereview.chromium.org/3417001/diff/1/2#newcode73
image_to_live.sh:73: local stateful_url=$(echo $dev_url | sed -e
"s/update/static\/archive/")
${dev_url}

http://codereview.chromium.org/3417001/diff/1/2#newcode79
image_to_live.sh:79: remote_cp_to "$dev_dir/stateful_update" "/tmp"
also update this $dev_dir

[DaleCurtis, 2010-09-14 17:28:46]

http://codereview.chromium.org/3417001/diff/1/2
File image_to_live.sh (right):

http://codereview.chromium.org/3417001/diff/1/2#newcode73
image_to_live.sh:73: local stateful_url=$(echo $dev_url | sed -e
"s/update/static\/archive/")
On 2010/09/14 03:44:41, ericli wrote:
> ${dev_url}

Done.

http://codereview.chromium.org/3417001/diff/1/2#newcode79
image_to_live.sh:79: remote_cp_to "$dev_dir/stateful_update" "/tmp"
On 2010/09/14 03:44:41, ericli wrote:
> also update this $dev_dir

Done.

[ericli, 2010-09-14 17:31:39]

LGTM.

[sosa, 2010-09-14 17:55:07]

http://codereview.chromium.org/3417001/diff/3002/5001
File image_to_live.sh (right):

http://codereview.chromium.org/3417001/diff/3002/5001#newcode73
image_to_live.sh:73: local stateful_url=$(echo ${dev_url} | sed -e
"s/update/static\/archive/")
static/archive isn't used as the default for dev servers.  I'm guess you're
passing special args into the devserver (a custom label) to get that address?

[sosa, 2010-09-14 18:29:30]

http://codereview.chromium.org/3417001/diff/3002/5001
File image_to_live.sh (right):

http://codereview.chromium.org/3417001/diff/3002/5001#newcode73
image_to_live.sh:73: local stateful_url=$(echo ${dev_url} | sed -e
"s/update/static\/archive/")
Talking to seano a little bit, I think the correct way to handle this is

if [ -n "${FLAGS_update_url}" ]; then
  stateful_url=(what you have)
else
  stateful_url= what you have but sed -e "/s/update/static"

On 2010/09/14 17:55:07, sosa wrote:
> static/archive isn't used as the default for dev servers.  I'm guess you're
> passing special args into the devserver (a custom label) to get that address?

[DaleCurtis, 2010-09-14 19:49:40]

PTAL.

http://codereview.chromium.org/3417001/diff/3002/5001
File image_to_live.sh (right):

http://codereview.chromium.org/3417001/diff/3002/5001#newcode73
image_to_live.sh:73: local stateful_url=$(echo ${dev_url} | sed -e
"s/update/static\/archive/")
On 2010/09/14 18:29:30, sosa wrote:
> Talking to seano a little bit, I think the correct way to handle this is
> 
> if [ -n "${FLAGS_update_url}" ]; then
>   stateful_url=(what you have)
> else
>   stateful_url= what you have but sed -e "/s/update/static"
> 
> On 2010/09/14 17:55:07, sosa wrote:
> > static/archive isn't used as the default for dev servers.  I'm guess you're
> > passing special args into the devserver (a custom label) to get that
address?
> 
> 

Done.

[sosa, 2010-09-14 19:56:39]

LGTM, thx