On Fri, 2011-06-03 at 15:25 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [PATCH 9/9] xl: new
> "check-xl-disk-parse" test script for disk parser"):
> > Does this work if there is no dom1? It looks like the
> > domain_qualifier_to_domid will fail before you get to produce any
> > output.
>
> Fixed.
Except the commit message, anyway:
Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
>
> commit d76bea783e4c4ee97a7c3f2f5862589cea134523
> Author: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> Date: Fri May 27 16:50:41 2011 +0100
>
> xl: new "check-xl-disk-parse" test script for disk parser
>
> This runs "xl block-attach -N 1 <some strings>" for various sets of
> strings and checks that the output is as expected.
>
> Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
>
> diff --git a/tools/libxl/check-xl-disk-parse b/tools/libxl/check-xl-disk-parse
> new file mode 100755
> index 0000000..8f59485
> --- /dev/null
> +++ b/tools/libxl/check-xl-disk-parse
> @@ -0,0 +1,82 @@
> +#!/bin/bash
> +
> +set -e
> +
> +fprefix=tmp.check-xl-disk-parse
> +
> +expected () {
> + cat >$fprefix.expected
> +}
> +
> +failures=0
> +
> +one () {
> + expected_rc=$1; shift
> + printf "test case %s...\n" "$*"
> + set +e
> + LD_LIBRARY_PATH=.:../libxc:../xenstore \
> + ./xl block-attach -N 0 "$@" </dev/null >$fprefix.actual 2>/dev/null
> + actual_rc=$?
> + diff -u $fprefix.expected $fprefix.actual
> + diff_rc=$?
> + set -e
> + if [ $actual_rc != $expected_rc ] || [ $diff_rc != 0 ]; then
> + echo >&2 "test case \`$*' failed ($actual_rc $diff_rc)"
> + failures=$(( $failures + 1 ))
> + fi
> +}
> +
> +complete () {
> + if [ "$failures" = 0 ]; then
> + echo all ok.; exit 0
> + else
> + echo "$failures tests failed."; exit 1
> + fi
> +}
> +
> +e=255
> +
> +
> +#---------- test data ----------
> +#
> +# culled from docs/misc/xl-disk-configuration.txt
> +
> +expected </dev/null
> +one $e foo
> +
> +expected <<END
> +disk.backend_domid = 0
> +disk.pdev_path = /dev/vg/guest-volume
> +disk.vdev = hda
> +disk.backend = 0
> +disk.format = 4
> +disk.script = (null)
> +disk.removable = 0
> +disk.readwrite = 1
> +disk.is_cdrom = 0
> +END
> +one 0 /dev/vg/guest-volume,,hda
> +one 0 /dev/vg/guest-volume,raw,hda,rw
> +one 0 "format=raw, vdev=hda, access=rw, target=/dev/vg/guest-volume"
> +one 0 format=raw vdev=hda access=rw target=/dev/vg/guest-volume
> +one 0 raw:/dev/vg/guest-volume,hda,w
> +
> +expected <<END
> +disk.backend_domid = 0
> +disk.pdev_path = /root/image.iso
> +disk.vdev = hdc
> +disk.backend = 0
> +disk.format = 4
> +disk.script = (null)
> +disk.removable = 1
> +disk.readwrite = 0
> +disk.is_cdrom = 1
> +END
> +one 0 /root/image.iso,,hdc,cdrom
> +one 0 /root/image.iso,,hdc,,cdrom
> +one 0 /root/image.iso,raw,hdc,devtype=cdrom
> +one 0 "format=raw, vdev=hdc, access=ro, devtype=cdrom,
> target=/root/image.iso"
> +one 0 format=raw vdev=hdc access=ro devtype=cdrom target=/root/image.iso
> +one 0 raw:/root/image.iso,hdc:cdrom,ro
> +
> +complete
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|