[PATCH][XM-TEST] Add support for pulling initrd images from a repo

[PATCH][XM-TEST] Add support for pulling initrd images from a repo

[Paul Larson]

[-- Attachment #1: Type: text/plain, Size: 357 bytes --]

XM-Test takes a really long time to build the initrd it uses for 
testing.  This patch to the Makefile allows you to specify INITRD="url" 
when running make existing to make it download a prebuilt copy of the 
initrd from a repository.  This also updates the README to document the 
change.

Signed-off-by: Paul Larson <pl@us.ibm.com>

Thanks,
Paul Larson


[-- Attachment #2: xmtest-wget-initrd.patch --]
[-- Type: text/x-patch, Size: 1613 bytes --]

# HG changeset patch
# User pl@us.ibm.com
# Node ID 1f91359510f31c52f48ca5ab85a6ee44c41212e5
# Parent  1c7145a5bb434413204a3ce4114de7f58fbb9029
XM-Test takes a really long time to build the initrd it uses for testing.  This patch to the Makefile allows you to specify INITRD="url" when running make existing to make it download a prebuilt copy of the initrd from a repository.

Signed-off-by: Paul Larson <pl@us.ibm.com>

diff -r 1c7145a5bb43 -r 1f91359510f3 tools/xm-test/README
--- a/tools/xm-test/README	Tue Feb 21 11:19:13 2006
+++ b/tools/xm-test/README	Tue Feb 21 19:12:04 2006
@@ -48,6 +48,15 @@
 Simply copy the initrd-X.Y.img file into ramdisk/ and then run:
 
    # make existing
+
+Or, you can run:
+   # INITRD="http://url.of.initrd.repo/" make existing
+
+You do not need to include the name of the image itself in the url, 
+however, an initrd with the right name (initrd.X.Y.img) and version 
+number must exist at that location.  The script will determine which 
+version of the initrd it needs and try to download the right file from 
+that location.
 
 This will set up the link so that xm-test will use the existing
 ramdisk.  Next, just run "runtest.sh" normally.  Note that in general,
diff -r 1c7145a5bb43 -r 1f91359510f3 tools/xm-test/ramdisk/Makefile.am
--- a/tools/xm-test/ramdisk/Makefile.am	Tue Feb 21 11:19:13 2006
+++ b/tools/xm-test/ramdisk/Makefile.am	Tue Feb 21 19:12:04 2006
@@ -57,6 +57,9 @@
 	fi
 
 existing:
+	@if test -n "$(INITRD)"; then \
+		wget $(INITRD)/$(XMTEST_VER_IMG); \
+	fi
 	@if [ -f $(XMTEST_VER_IMG) ] ; then \
 		ln -sf $(XMTEST_VER_IMG) initrd.img; \
 	else \

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Re: [PATCH][XM-TEST] Add support for pulling initrd images from a repo

[Dan Smith]

PL> This patch to the Makefile allows you to specify INITRD="url" when
PL> running make existing to make it download a prebuilt copy of the
PL> initrd from a repository.

Yay.  Really.  Yay.

It was previously discussed at the summit that xensource might be able
to host an initrd for the community.  Now that this functionality is
in xm-test, perhaps a URL could be posted to be used as a default for
the INITRD variable?

-- 
Dan Smith
IBM Linux Technology Center
Open Hypervisor Team
email: danms@us.ibm.com

Re: [PATCH][XM-TEST] Add support for pulling initrd images from a repo

[Ewan Mellor]

On Tue, Feb 21, 2006 at 11:34:29AM -0800, Dan Smith wrote:

> 
> PL> This patch to the Makefile allows you to specify INITRD="url" when
> PL> running make existing to make it download a prebuilt copy of the
> PL> initrd from a repository.
> 
> Yay.  Really.  Yay.
> 
> It was previously discussed at the summit that xensource might be able
> to host an initrd for the community.  Now that this functionality is
> in xm-test, perhaps a URL could be posted to be used as a default for
> the INITRD variable?

Yes, we will do this.  Dan, if you (or someone else at IBM) could provide a
'known good' initrd, then we'll host that, and I'll synchronise with you so
that the patch to refer to that initrd goes in only once the initrd is in
place.

They will be placed into

http://xm-test.xensource.com/ramdisks/

Please make sure that xm-test will only fetch the initrd if it can't find one
locally, and will resume interrupted downloads rather than start again from
scratch.  This will hopefully avoid our machines melting under the load!

Thanks,

Ewan.

Re: [PATCH][XM-TEST] Add support for pulling initrd images from a repo

[Paul Larson]

Ewan Mellor wrote:

>On Tue, Feb 21, 2006 at 11:34:29AM -0800, Dan Smith wrote:
>
>  
>
>>PL> This patch to the Makefile allows you to specify INITRD="url" when
>>PL> running make existing to make it download a prebuilt copy of the
>>PL> initrd from a repository.
>>
>>Yay.  Really.  Yay.
>>
>>It was previously discussed at the summit that xensource might be able
>>to host an initrd for the community.  Now that this functionality is
>>in xm-test, perhaps a URL could be posted to be used as a default for
>>the INITRD variable?
>>    
>>
>
>Yes, we will do this.  Dan, if you (or someone else at IBM) could provide a
>'known good' initrd, then we'll host that, and I'll synchronise with you so
>that the patch to refer to that initrd goes in only once the initrd is in
>place.
>
>They will be placed into
>
>http://xm-test.xensource.com/ramdisks/
>
>Please make sure that xm-test will only fetch the initrd if it can't find one
>locally, and will resume interrupted downloads rather than start again from
>scratch.  This will hopefully avoid our machines melting under the load!
>  
>
Hi Ewan, I'm still not seeing a xm-test.xensource.com.  Did this ever 
get created?

Thanks,
Paul Larson

Re: [PATCH][XM-TEST] Add support for pulling initrd images from a repo

[Ewan Mellor]

On Tue, Feb 21, 2006 at 01:22:14PM -0600, Paul Larson wrote:

> XM-Test takes a really long time to build the initrd it uses for 
> testing.  This patch to the Makefile allows you to specify INITRD="url" 
> when running make existing to make it download a prebuilt copy of the 
> initrd from a repository.  This also updates the README to document the 
> change.
> 
> Signed-off-by: Paul Larson <pl@us.ibm.com>
> 
> Thanks,
> Paul Larson

Applied, thanks.

Ewan.