Re: [Xen-devel] [PATCH OSSTEST v2 04/11] HostDB: introduce set_property

2017-08-01 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST v2 04/11] HostDB: introduce set_property"): > And provide a helper in TestSupport to use it. This allows osstest to > set host properties from test script themselves (instead of using > the mg-hosts clu). ... > +sub set_property() { > +my ($hd, $ho,

Re: [Xen-devel] [PATCH OSSTEST v2 07/11] ts-freebsd-host-install: add arguments to test memdisk append options

2017-08-01 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST v2 07/11] ts-freebsd-host-install: add arguments to test memdisk append options"): > This is needed in order to figure out which memdisk options should be > used to boot the images on each specific box. > > Note that upon success the script stores the tenta

Re: [Xen-devel] [PATCH OSSTEST v2 01/11] netboot_memdisk: allow each host to have different append values

2017-08-01 Thread Ian Jackson
gt; on a per-host basis. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop

2017-08-01 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop"): > After reading README osstest doesn't seem to have any limitation on > the characters that can be used for host idents, would you be fine > with me modifying it to add that '-' cannot be used in host idents

Re: [Xen-devel] [PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop

2017-08-01 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop"): > On Tue, Aug 01, 2017 at 02:01:35PM +0100, Ian Jackson wrote: > > TBH, since this is only being called in the one > > ts-set-host-properties-from-runvars script (or whatever you&#x

Re: [Xen-devel] [PATCH] libxc: check pointer is not null before printing

2017-08-01 Thread Ian Jackson
Wei Liu writes ("[PATCH] libxc: check pointer is not null before printing"): > Signed-off-by: Wei Liu Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [OSSTEST PATCH] mg-repro-setup: Slightly better document the alloc: syntax

2017-08-01 Thread Ian Jackson
Provide a clearer indication that the is passed to mg-allocate (and therefore, implicitly, that mg-allocate's docs should be consulted). Signed-off-by: Ian Jackson CC: Wei Liu --- mg-repro-setup | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mg-repro-setup b/mg-

Re: [Xen-devel] [PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop

2017-08-01 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop"): > IMHO, I think the right approach is to leave mg-hosts as it is now, Yes. > and implement a set_property in HostDB/{Executive/Static}.pm and > implement a helper in TestSupport that makes use of it > ($mh

Re: [Xen-devel] [PATCH v2 2/2] libxl: do not start dom0 qemu for stubdomain when not needed

2017-07-31 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 2/2] libxl: do not start dom0 qemu for stubdomain when not needed"): > Hmm... I don't think there is requirement in CODING_STYLE for > multiple-line comment, so there are quite a few styles in use. But > looking at libxl code the prevailing style seems to be: > >

Re: [Xen-devel] [PATCH 0/3] Docs: consolidate release related documents

2017-07-31 Thread Ian Jackson
Wei Liu writes ("[PATCH 0/3] Docs: consolidate release related documents"): > Wei Liu (3): > docs: consolidate release related documents > docs: add xen-release-management.pandoc > docs: hook up process/ to build system FWIW, Acked-by: Ian Jackson However, AFAIC

Re: [Xen-devel] [PATCH v2] xen: Implement hypercall for tracing of program counters

2017-07-31 Thread Ian Jackson
Wei Liu writes ("Re: [Xen-devel] [PATCH v2] xen: Implement hypercall for tracing of program counters"): > On Mon, Jul 31, 2017 at 09:22:35AM +0100, Julien Grall wrote: .. > > Should not it be -EOPNOTSUPP to match return error when CONFIG_TRACE_PC is > > not? > > AIUI EOPNOTSUPP means "This is a v

Re: [Xen-devel] [PATCH v2 1/2] libxl: Implement the handler to handle unrecoverable AER errors.

2017-07-28 Thread Ian Jackson
Venu Busireddy writes ("[PATCH v2 1/2] libxl: Implement the handler to handle unrecoverable AER errors."): > Implement the callback function to handle unrecoverable AER errors, and > also the public APIs that can be used to register/unregister the handler. > When an AER error occurs, the handler w

Re: [Xen-devel] [PATCH] libxl: do not start dom0 qemu for stubdomain when not needed

2017-07-28 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH] libxl: do not start dom0 qemu for stubdomain when not needed"): > On Fri, Jul 28, 2017 at 06:05:13PM +0200, Marek Marczykowski-Górecki wrote: > > On Fri, Jul 28, 2017 at 04:17:51PM +0100, Wei Liu wrote: > > > On Thu, Jul 27, 2017 at 12:44:25AM +0200, Marek Marczykowski

Re: [Xen-devel] [PATCH OSSTEST 10/11] make-hosts-flight: set runvars for FreeBSD test

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 10/11] make-hosts-flight: set runvars for FreeBSD test"): > This is needed in order to run the memdisk test. Acked-by: Ian Jackson Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://

Re: [Xen-devel] [PATCH OSSTEST 11/11] sg-run-job: hook the memdisk test into examine

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 11/11] sg-run-job: hook the memdisk test into examine"): > Hook the memdisk parameter detection and the saving of the host > properties into the examine jobs. Looking at this: > + run-ts . =ts-memdisk-try-append + host > } > run

Re: [Xen-devel] [PATCH OSSTEST 09/11] ts-examine-hostprops-save: introduce a script to save properties

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 09/11] ts-examine-hostprops-save: introduce a script to save properties"): > The introduce script turns the properties stored in the runvars using > the format hostprop_$hotname_$prop=$val into host properties stored in > the database. ... > +exit 0 if inten

Re: [Xen-devel] [PATCH OSSTEST 08/11] ts-memdisk-try-append: introduce a script to test memdisk options

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 08/11] ts-memdisk-try-append: introduce a script to test memdisk options"): > The intended usage is to run this script against every host in order > to record the possible needed memdisk flags. Acked-by: Ian Jackson Except that I think m

Re: [Xen-devel] [PATCH OSSTEST 07/11] ts-freebsd-host-install: add option to test memdisk options

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 07/11] ts-freebsd-host-install: add option to test memdisk options"): ... > +if ($bootonly) { > +hostprop_putative_record($ho, "MemdiskAppend", $memdisk_append) > +if $memdisk_append; > +exit 0; I think this should be a separate option --rec

Re: [Xen-devel] [PATCH OSSTEST 07/11] ts-freebsd-host-install: add option to test memdisk options

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 07/11] ts-freebsd-host-install: add option to test memdisk options"): > This is needed in order to figure out which memdisk options should be > used to boot the images on each specific box. ... > +if ($r{'arch'} !~ m/amd64/g) { > +logm("Arch $r{'arch'} n

Re: [Xen-devel] [PATCH OSSTEST 06/11] TestSupport: introduce hostprop_putative_record

2017-07-28 Thread Ian Jackson
roperties stored in the database. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH OSSTEST 05/11] mfi-common: move set_freebsd_runvars to mfi-common

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 05/11] mfi-common: move set_freebsd_runvars to mfi-common"): > So that it can also be used by make-hosts-flight. No functional change > intended. Acked-by: Ian Jackson ___ Xen-devel mailin

Re: [Xen-devel] [PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 04/11] TestSupport: introduce set_host_prop"): > This is from the code in mg-hosts. Switch cmd_setprops to use the > newly introduced function. I think this needs to be abstracted through jobdb. Certainly these SQL statements operating on the host_properti

[Xen-devel] [OSSTEST PATCH] cs-hosts-list: Document the --arches, --kernels and --suites options

2017-07-28 Thread Ian Jackson
These have a rather counterintuitive behaviour which is nevertheless useful. Document it, and the reasoning. CC: Roger Pau Monne Signed-off-by: Ian Jackson --- cs-hosts-list | 37 + 1 file changed, 37 insertions(+) diff --git a/cs-hosts-list b/cs-hosts

Re: [Xen-devel] [PATCH OSSTEST 02/11] ts-freebsd-host-install: fix image permissions

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 02/11] ts-freebsd-host-install: fix image permissions"): > Make sure images copied to the tftp path have the right permissions, > so use dd instead of cp, which will obviously not preserve the > original permissions. Ac

Re: [Xen-devel] [PATCH OSSTEST 03/11] sg-run-job: fix typo in the examine jobs

2017-07-28 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH OSSTEST 03/11] sg-run-job: fix typo in the examine jobs"): > proc prep-job/host-examine-xen is declared twice, one of them should > be prep-job/host-examine-linux instead. > > Signed-off-by: Roger Pau Monné Thank you.

Re: [Xen-devel] [OSSTEST PATCH v14 21/24] make-flight: Prepare devstack jobs on "openstack*" branches only

2017-07-28 Thread Ian Jackson
gt; overall functional change. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v14 22/24] New branch openstack-ocata

2017-07-28 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v14 22/24] New branch openstack-ocata"): > Testing of the Ocata stable branch of OpenStack against Xen unstable. Thanks for the additonal explanation. Acked-by: Ian Jackson ___ Xen-devel mailing l

Re: [Xen-devel] [OSSTEST PATCH v14 19/24] TestSupport: Implement target_subunit_cmd a subunit stream parser into substeps

2017-07-28 Thread Ian Jackson
a description of the subunit v1 protocol, taken from > python-subunit README, or https://pypi.python.org/pypi/python-subunit Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v14 18/24] TestSupport: Introduce target_cmd_stashed

2017-07-28 Thread Ian Jackson
Anthony PERARD Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [OSSTEST PATCH] arm64: tolerate host allocation failures

2017-07-28 Thread Ian Jackson
l CC: Stefano Stabellini Signed-off-by: Ian Jackson --- allow.all | 1 + 1 file changed, 1 insertion(+) diff --git a/allow.all b/allow.all index 1e3e378..22a4d18 100644 --- a/allow.all +++ b/allow.all @@ -4,3 +4,4 @@ test-@@-pcipt@@ test-@@-qemuu-@@ guest-localmigrate test-@@

Re: [Xen-devel] Notes Design Session: Making Releases Lessons Learned: Improving Our Release Process and Tooling

2017-07-27 Thread Ian Jackson
Jan Beulich writes ("Re: Notes Design Session: Making Releases Lessons Learned: Improving Our Release Process and Tooling"): > [Lars:] > >Ian: These are most likely software problems, most likely in Xen. > >ISSUE: nobody wants to debug Windows Heisenbugs > > Hmm, I think we believe to have unders

[Xen-devel] No ARM64 testing in Xen Project Massachusetts Colo

2017-07-27 Thread Ian Jackson
Our remaining arm64 box, laxton1, seems to have died. The symptoms are similar to those of laxton0. I have asked our contractors to investigate on-site but I expect they will tell me it's dead. There are therefore currently no working arm64 test resources in the Xen Project test facility. Statu

Re: [Xen-devel] [OSSTEST PATCH v13 21/24] Create a flight to test OpenStack with xen-unstable and libvirt

2017-07-26 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v13 21/24] Create a flight to test OpenStack with xen-unstable and libvirt"): > On Tue, Jul 25, 2017 at 07:09:06PM +0100, Ian Jackson wrote: > > Anthony PERARD writes ("[OSSTEST PATCH v13 21/24] Create a flight to test > &g

Re: [Xen-devel] [OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*

2017-07-26 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*"): > On Tue, Jul 25, 2017 at 07:18:21PM +0100, Ian Jackson wrote: > > Anthony PERARD writes ("[OSSTEST PATCH v13 24/24] openstack tests: Don't > > run them on arm

Re: [Xen-devel] [OSSTEST PATCH v13 19/24] TestSupport: Implement target_subunit_cmd a subunit stream parser into substeps

2017-07-26 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v13 19/24] TestSupport: Implement target_subunit_cmd a subunit stream parser into substeps"): > I think I start by looking at what kind of characters could be part of > type and sub-type, and just start writing a more complicated regex. > > So is the fol

Re: [Xen-devel] [PATCH] scripts/get_maintainers.pl: Don't blindly drop "THE REST" maintainers

2017-07-26 Thread Ian Jackson
component maintainer. > > However, the script get_maintainers.pl will remove "THE REST" > maintainers as soon as one maintainer of a specific component will be > present. > > Fix the script once for all. Acked-by: Ian Jackson Although: > +# By default "THE REST&q

[Xen-devel] [OSSTEST PATCH 2/2] coverity: Set CoverityToolsStripComponents to 2

2017-07-26 Thread Ian Jackson
This needs to be 2 for cov-analysis-linux64-2017.07.tar.gz because that contains "./" at the start of its paths, which tar counts as a component it's stripping. Signed-off-by: Ian Jackson --- production-config | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pr

[Xen-devel] [OSSTEST PATCH 1/2] coverity: Parameterise CoverityToolsStripComponents

2017-07-26 Thread Ian Jackson
No functional change with current production-config. (In fact, this needs to be 2 for cov-analysis-linux64-2017.07.tar.gz) Reported-by: Wei Liu Signed-off-by: Ian Jackson --- production-config | 1 + ts-coverity-build | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a

Re: [Xen-devel] [OSSTEST PATCH v13 00/24] Have OpenStack tested on top of xen's master and libvirt's master.

2017-07-25 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v13 00/24] Have OpenStack tested on top of xen's master and libvirt's master."): > Now powered with subunit-to-substep engine. > > The Tempest test names reported via subunit are in the form: > tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.tes

Re: [Xen-devel] [OSSTEST PATCH v13 18/24] TestSupport: Introduce target_cmd_stashed

2017-07-25 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v13 18/24] TestSupport: Introduce target_cmd_stashed"): > This works like target_cmd, but takes a ref to a filename as argument > and stash the output of the command then return a path to the stashed > output. ... > +# Like target_cmd, but stash cmd stdout an

Re: [Xen-devel] [OSSTEST PATCH v13 22/24] New branch openstack-ocata

2017-07-25 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v13 22/24] New branch openstack-ocata"): > Testing of the Ocata stable branch of OpenStack against Xen unstable. > > OpenStack have many different repo which should be in sync, so we should > attempd to grab the revisions of the stable branch of every OpenSta

Re: [Xen-devel] [OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*

2017-07-25 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*"): > Signed-off-by: Anthony PERARD Acked-by: Ian Jackson You should probably mention the existence of this patch in the other one's

Re: [Xen-devel] [OSSTEST PATCH v13 21/24] Create a flight to test OpenStack with xen-unstable and libvirt

2017-07-25 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v13 21/24] Create a flight to test OpenStack with xen-unstable and libvirt"): > This patch creates a flight "openstack*", with those jobs: Do you mean it creates a "branch" ? But I don't think it does. I predict no changes to the output of mg-list-all-bran

Re: [Xen-devel] [OSSTEST PATCH v13 19/24] TestSupport: Implement target_subunit_cmd a subunit stream parser into substeps

2017-07-25 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v13 19/24] TestSupport: Implement target_subunit_cmd a subunit stream parser into substeps"): > target_subunit_cmd can be used like target_cmd, but the command would > needs to output a subunit v1 stream, which will be parsed and turned > into osstest substep

Re: [Xen-devel] [OSSTEST PATCH v13 10/24] ts-openstack-deploy: Increase open fd limit for RabbitMQ

2017-07-25 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v13 10/24] ts-openstack-deploy: Increase open fd limit for RabbitMQ"): > Signed-off-by: Anthony PERARD Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v7 20/20] osstest: save/retrieve the last successfully tested FreeBSD build

2017-07-25 Thread Ian Jackson
#* } > +freebsd_envvar="FREEBSD_${freebsd_arch^^}_BUILDJOB" > +if [ "x${!freebsd_envvar}" = "x" ]; then > +flight_job=`./mg-anoint retrieve "$anointed"` > + export ${freebsd_envvar}=${flight_job/ /.} > +fi > +done &g

Re: [Xen-devel] [PATCH v7 18/20] osstest: introduce a script to create a FreeBSD flight

2017-07-25 Thread Ian Jackson
uild--freebsd job. > > 2. Env variables FREEBSD_DIST and FREEBSD_VERSION: set before calling > into make-flight, provide the path to the installer image and sets > folder and the version being installed. > > 3. Config file FreeBSDDist and FreeBSDVersion: same as 2. except that

Re: [Xen-devel] [PATCH v7 19/20] osstest: hook FreeBSD flight into cr-daily-branch

2017-07-25 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v7 19/20] osstest: hook FreeBSD flight into cr-daily-branch"): > Add the missing pieces so that cr-daily-branch knows how to create a > FreeBSD flight. This has been tested by calling `./cr-daily-branch > freebsd-master`. Ac

Re: [Xen-devel] [PATCH v7 11/20] osstest: introduce an OS $ho field

2017-07-25 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v7 11/20] osstest: introduce an OS $ho field"): > In order to differentiate between FreeBSD and Debian hosts. ... > +$ho->{Os} = target_var($ho, "os") // "debian"; I will change this to "OS" when commi

Re: [Xen-devel] [PATCH v6 19/20] osstest: save/retrieve the last successfully tested FreeBSD build

2017-07-25 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v6 19/20] osstest: save/retrieve the last successfully tested FreeBSD build"): > OK, forcing a new flight if the anointed revision doesn't match > OLD_REVISION (regardless of whether OLD_REVISION == NEW_REVISION) > seems like the best option. Jolly good. > I d

Re: [Xen-devel] [PATCH v6 10/20] osstest: add prototypes to target_install_packages{_norec}

2017-07-25 Thread Ian Jackson
ce here. With that changed, Acked-by: Ian Jackson Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 1/3] docs: add pod variant of xen-pv-channel.7

2017-07-25 Thread Ian Jackson
Olaf Hering writes ("[PATCH v2 1/3] docs: add pod variant of xen-pv-channel.7"): > Convert source for xen-pv-channel.7 from markdown to pod. > This removes the buildtime requirement for pandoc, and subsequently the > need for ghc, in the chain for BuildRequires of xen.rpm. Thanks. Thanks also for

Re: [Xen-devel] [PATCH v6 19/20] osstest: save/retrieve the last successfully tested FreeBSD build

2017-07-25 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v6 19/20] osstest: save/retrieve the last successfully tested FreeBSD build"): > On Mon, Jul 24, 2017 at 04:56:25PM +0100, Ian Jackson wrote: > > Also I would like you to discuss explicitly (in a comment or commit > > message) abo

Re: [Xen-devel] [PATCH 2/3] tools: nuke superpage parameters in code

2017-07-25 Thread Ian Jackson
Wei Liu writes ("[PATCH 2/3] tools: nuke superpage parameters in code"): > Also fix manpage because there is no superpages options in xl.cfg. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 0/6] docs: convert manpages to pod

2017-07-25 Thread Ian Jackson
Olaf Hering writes ("Re: [PATCH 0/6] docs: convert manpages to pod"): > On Mon, Jul 24, Ian Jackson wrote: > > * There are a lot of other documents in docs/misc/ which are in > > markdown format. Some of them are internal. I'm pretty sure we don't > > w

[Xen-devel] [OSSTEST PATCH] anointments: Provide ./mg-anoint list-prepared

2017-07-24 Thread Ian Jackson
CC: Roger Pau Monne Signed-off-by: Ian Jackson --- mg-anoint | 29 + 1 file changed, 29 insertions(+) diff --git a/mg-anoint b/mg-anoint index 4bb8ab0..837e608 100755 --- a/mg-anoint +++ b/mg-anoint @@ -20,6 +20,9 @@ # if nothing anointed yet, prints

Re: [Xen-devel] [PATCH 0/6] docs: convert manpages to pod

2017-07-24 Thread Ian Jackson
Olaf Hering writes ("Re: [PATCH 0/6] docs: convert manpages to pod"): > On Mon, Jul 24, Ian Jackson wrote: > > Olaf Hering writes ("[PATCH 0/6] docs: convert manpages to pod"): > > > To remove the buildtime dependency to pandoc/ghc some manpages are >

Re: [Xen-devel] [PATCH v6 18/20] osstest: hook FreeBSD flight into cr-daily-branch

2017-07-24 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v6 18/20] osstest: hook FreeBSD flight into cr-daily-branch"): > On Mon, Jul 24, 2017 at 04:44:09PM +0100, Ian Jackson wrote: > > Roger Pau Monne writes ("[PATCH v6 18/20] osstest: hook FreeBSD flight into > > cr-daily-branch"

Re: [Xen-devel] [PATCH 0/6] docs: convert manpages to pod

2017-07-24 Thread Ian Jackson
Olaf Hering writes ("[PATCH 0/6] docs: convert manpages to pod"): > To remove the buildtime dependency to pandoc/ghc some manpages are > converted from markdown to pod format. This will provide more manpages > which are referenced in xl(1) and xl.cfg(5). Sorry to ask this at this stage, but: did I

Re: [Xen-devel] [PATCH v6 19/20] osstest: save/retrieve the last successfully tested FreeBSD build

2017-07-24 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v6 19/20] osstest: save/retrieve the last successfully tested FreeBSD build"): > And use it in order to install the hosts for the next FreeBSD flight. Oh. Right, here is this. > Signed-off-by: Roger Pau Monné > --- > Changes since v5: > - New in this version. >

Re: [Xen-devel] [PATCH v6 18/20] osstest: hook FreeBSD flight into cr-daily-branch

2017-07-24 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v6 18/20] osstest: hook FreeBSD flight into cr-daily-branch"): > Add the missing pieces so that cr-daily-branch knows how to create a > FreeBSD flight. This has been tested by calling `./cr-daily-branch > freebsd-master`. ... > +TREE_FREEBSD=$PUSH_TREE_FREEBSD PUSH

Re: [Xen-devel] [PATCH v6 17/20] osstest: introduce a script to create a FreeBSD flight

2017-07-24 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v6 17/20] osstest: introduce a script to create a FreeBSD flight"): > Changes since v5: > - Append the "freebsd" hostflag, the specific version will be set >dynamically by the ts-freebsd-set-hostflags script. I think we discussed using PropMinVer ? I'm not sur

Re: [Xen-devel] [PATCH v6 16/20] osstest: add support for FreeBSD buildjobs to sg-run-job

2017-07-24 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v6 16/20] osstest: add support for FreeBSD buildjobs to sg-run-job"): > Add support and introduce a FreeBSD build job to sg-run-job. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.

Re: [Xen-devel] [PATCH v6 15/20] osstest: change the meaning of need_build_host

2017-07-24 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v6 15/20] osstest: change the meaning of need_build_host"): > Make need_build_host store a string instead of a boolean. This is > later going to be expanded to handle the FreeBSD build jobs. Acked

[Xen-devel] [OSSTEST PATCH 3/3] anointments: Introduce anointment scheme, and mg-anoint script

2017-07-24 Thread Ian Jackson
CC: Roger Pau Monne Signed-off-by: Ian Jackson --- README.planner | 4 + mg-anoint | 347 + 2 files changed, 351 insertions(+) create mode 100755 mg-anoint diff --git a/README.planner b/README.planner index b3b41a9..255460b 100644

[Xen-devel] [OSSTEST PATCH 1/3] README.planner: Improve internals documentation a bit

2017-07-24 Thread Ian Jackson
* share-flight resources may end up owned by a different task to their shareix, perhaps as a result of test database operations or perhaps as a result of donation with mg-allocate. This should not be a problem. * Document the xdbref task type. Signed-off-by: Ian Jackson

[Xen-devel] [OSSTEST PATCH 0/3] Introduce anointments

2017-07-24 Thread Ian Jackson
00...)"): > On Thu, Jul 20, 2017 at 04:57:50PM +0100, Ian Jackson wrote: > > How about > > ./mg-anoint retrieve "freebsd build $bsdbranch $arch" > > which prints "$flight $job". > > For consumption it might be easier to simply return $flight.$

[Xen-devel] [OSSTEST PATCH 2/3] schema updates: mg-schema-test-database: Cope with breaking updates

2017-07-24 Thread Ian Jackson
We need to pass -ff, since we want to replicate the existing database, regardless of its compatibility properties. Signed-off-by: Ian Jackson --- mg-schema-test-database | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mg-schema-test-database b/mg-schema-test-database index

Re: [Xen-devel] [BUG] libxl_ctx_free calls close(0)

2017-07-24 Thread Ian Jackson
Victor Kirhenshtein writes ("Re: [Xen-devel] [BUG] libxl_ctx_free calls close(0)"): > It is 4.4.1 (4.4.1-9+deb8u9 from Debian 8 repository). Looks like Debian just > have too old version in repository. Problem is not reproduces on 4.6.5. Debian jessie is not likely to get significant updates to i

Re: [Xen-devel] Missing bits for production usage (was: [PATCH v5 00...)

2017-07-20 Thread Ian Jackson
Roger Pau Monne writes ("Missing bits for production usage (was: [PATCH v5 00...)"): > A script to mark/unmark a $flight.$job as blessed. Marking a > $flight.$job as blessed means that it's build output is never deleted > from osstest storage, a proposed command line would be: We can't call it "b

Re: [Xen-devel] [PATCH v3] Add option to xenconsole to always forward console input

2017-07-20 Thread Ian Jackson
e this, causing the > client to forward input regardless. > > Signed-off-by: Felix Schmoll Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v5 15/17] osstest: add support for FreeBSD buildjobs to sg-run-job

2017-07-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v5 15/17] osstest: add support for FreeBSD buildjobs to sg-run-job"): > Add support and introduce a FreeBSD build job to sg-run-job. ... > +switch -exact $ostype { > +FREEBSD { run-ts broken = ts-freebsd-set-hostflags --share }

Re: [Xen-devel] [PATCH v5 15/17] osstest: add support for FreeBSD buildjobs to sg-run-job

2017-07-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v5 15/17] osstest: add support for FreeBSD buildjobs to sg-run-job"): > Add support and introduce a FreeBSD build job to sg-run-job. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.

Re: [Xen-devel] [PATCH v5 14/17] osstest: change the meaning of need_build_host

2017-07-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v5 14/17] osstest: change the meaning of need_build_host"): > Make need_build_host store a string instead of a boolean. This is > later going to be expanded to handle the FreeBSD build jobs. This is all fine, but I have two style comments: > +if {[string match

Re: [Xen-devel] [PATCH v5 13/17] osstest: introduce a script to set the runtime hostflags runvar for FreeBSD jobs

2017-07-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v5 13/17] osstest: introduce a script to set the runtime hostflags runvar for FreeBSD jobs"): > Due to the nature of the FreeBSD install media, which is > self-generated from the ts-freebsd-build script, the hostflags runvar > set to FreeBSD jobs are related to the

Re: [Xen-devel] [PATCH v5 12/17] osstest: add support for runtime_IDENT_hostflags

2017-07-19 Thread Ian Jackson
ion time. > > Also introduce a helper to set the runtime hostflags. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v5 11/17] osstest: introduce a FreeBSD build script

2017-07-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v5 11/17] osstest: introduce a FreeBSD build script"): > In order to generate the FreeBSD installer image and the install > media. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lis

Re: [Xen-devel] [PATCH v5 10/17] osstest: add support for the FreeBSD package manager

2017-07-19 Thread Ian Jackson
age manager. > > Signed-off-by: Roger Pau Monné ... > +sub package_install_cmd { ^ (;$) Should have a prototype. Sorry for not spotting this before. With that fixed, Acked-by: Ian Jackson Thanks, Ian.

Re: [Xen-devel] [PATCH v5 08/17] osstest: add a FreeBSD host install script

2017-07-19 Thread Ian Jackson
t, which allows the test > harness to just ssh into the box, create the install config file and > launch the scripted install. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v5 07/17] osstest: introduce rename_shared_mark_ready

2017-07-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v5 07/17] osstest: introduce rename_shared_mark_ready"): > That allows marking a host as ready to be shared. Replace the current > caller that open-codes it. You got the Subject wrong. "rename" for "resource". :-) Wit

Re: [Xen-devel] [PATCH v5 06/17] osstest: add executive prefix to resource_shared_mark_ready

2017-07-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v5 06/17] osstest: add executive prefix to resource_shared_mark_ready"): > This is a non-functional change in preparation for introducing a > resource_shared_mark_ready in TestSupport. > > Signed-off-by: Roger Pau Monné

Re: [Xen-devel] [OSSTEST PATCH v12 10/21] ts-openstack-deploy: Increase open fd limit for RabbitMQ

2017-07-19 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v12 10/21] ts-openstack-deploy: Increase open fd limit for RabbitMQ"): > I've created a bug report for openstack. > https://bugs.launchpad.net/devstack/+bug/1703651 Thanks, Ian. ___ Xen-devel mailing list Xen-

Re: [Xen-devel] [PATCH v2] xenconsole: Add pipe option

2017-07-19 Thread Ian Jackson
Felix Schmoll writes ("Re: [PATCH v2] xenconsole: Add pipe option"): > As there is already an interactive variable in the code, it seems > like a rather strange overloading to call the option interactive > that directly affects a different variable (currently pipe). The > name seems to make sense h

Re: [Xen-devel] [OSSTEST PATCH v12 10/21] ts-openstack-deploy: Increase open fd limit for RabbitMQ

2017-07-19 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v12 10/21] ts-openstack-deploy: Increase open fd limit for RabbitMQ"): > On Wed, Jul 19, 2017 at 11:28:29AM +0100, Ian Jackson wrote: > > Anthony PERARD writes ("[OSSTEST PATCH v12 10/21] ts-openstack-deploy: > > Incre

Re: [Xen-devel] [OSSTEST PATCH v12 20/21] Create a flight to test OpenStack with xen-unstable and libvirt

2017-07-19 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v12 20/21] Create a flight to test OpenStack with xen-unstable and libvirt"): > This patch creates a flight "openstack-ocata", with those jobs: ... I think it would help if you split apart the changes to make-flight and mfi-* from the ones to cr-daily-* and

Re: [Xen-devel] [OSSTEST PATCH v12 14/21] ts-openstack-tempest: Update list of skipped tests

2017-07-19 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v12 14/21] ts-openstack-tempest: Update list of skipped tests"): > Signed-off-by: Anthony PERARD Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v12 10/21] ts-openstack-deploy: Increase open fd limit for RabbitMQ

2017-07-19 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v12 10/21] ts-openstack-deploy: Increase open fd limit for RabbitMQ"): > +target_putfilecontents_root_stash($ho, 100, > +< +ulimit -n 65536 Is the lack of this not an upstream bug of some kind ? And, for osstest, why 65536 and not, say, "unlimit

Re: [Xen-devel] A document for Xen release management

2017-07-18 Thread Ian Jackson
Wei Liu writes ("A document for Xen release management"): > When you think all pending issues are fixed and Xen is ready to be released > from the last RC: Mostly LGTM. Thanks! [Items numbered in quoted text so I can refer to them.] > 1. Send out commit moratorium emails to committers@. > > 2.

Re: [Xen-devel] [OSSTEST PATCH v12 18/21] TestSupport: Implement target_cmd_subunit a subunit stream parser into substeps

2017-07-17 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v12 18/21] TestSupport: Implement target_cmd_subunit a subunit stream parser into substeps"): > On Thu, Jul 13, 2017 at 03:43:55PM +0100, Anthony PERARD wrote: > > Will do. And yes, this will duplicate most of the output. But it can > > help debug osstest

Re: [Xen-devel] [OSSTEST PATCH v12 18/21] TestSupport: Implement target_cmd_subunit a subunit stream parser into substeps

2017-07-17 Thread Ian Jackson
Anthony PERARD writes ("Re: [OSSTEST PATCH v12 18/21] TestSupport: Implement target_cmd_subunit a subunit stream parser into substeps"): > On Thu, Jul 13, 2017 at 02:28:11PM +0100, Ian Jackson wrote: > > I think this is not a critical problem, but fixing it would be nic

Re: [Xen-devel] [PATCH v2] xenconsole: Add pipe option

2017-07-17 Thread Ian Jackson
Felix Schmoll writes ("[PATCH v2] xenconsole: Add pipe option"): > Add pipe option to xenconsole that forwards console input. Thanks. IMO the commit message could do with better explanation. It should mention that xenconsole has a strange behaviour where it doesn't forward stdin unless stdin and

Re: [Xen-devel] [XenSummit 2017] Notes from the PVH toolstack interface session

2017-07-17 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [XenSummit 2017] Notes from the PVH toolstack interface session"): > On 17/07/17 10:36, Roger Pau Monné wrote: > > kernel = "" > > ramdisk = "" > > cmdline = "" > > > > : relative or full path in the filesystem. > > Please can xl or libxl's (not

Re: [Xen-devel] [OSSTEST PATCH v12 19/21] ts-openstack-tempest: Use target_cmd_subunit

2017-07-13 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v12 19/21] ts-openstack-tempest: Use target_cmd_subunit"): > Signed-off-by: Anthony PERARD Acked-by: Ian Jackson Subject to my comments aboout the name of the function, of course. Ian. ___ Xen-

Re: [Xen-devel] [OSSTEST PATCH v12 18/21] TestSupport: Implement target_cmd_subunit a subunit stream parser into substeps

2017-07-13 Thread Ian Jackson
Anthony PERARD writes ("[OSSTEST PATCH v12 18/21] TestSupport: Implement target_cmd_subunit a subunit stream parser into substeps"): > target_cmd_subunit can be used like target_cmd, but the command would > needs to output a subunit v1 stream, which will be parsed and turned > into osstest substep

Re: [Xen-devel] [PATCH v4 10/16] osstest: add support for runtime_IDENT_hostflags

2017-07-07 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v4 10/16] osstest: add support for runtime_IDENT_hostflags"): > On Thu, Jul 06, 2017 at 04:28:09PM +0100, Ian Jackson wrote: > > Should set_runtime_hostflag take a $ho, instead ? Can you ever see us > > using it without a $ho ? If no

Re: [Xen-devel] [PATCH v4 13/16] osstest: change the meaning of need_build_host

2017-07-07 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v4 13/16] osstest: change the meaning of need_build_host"): > On Thu, Jul 06, 2017 at 04:37:10PM +0100, Ian Jackson wrote: > > Roger Pau Monne writes ("[PATCH v4 13/16] osstest: change the meaning of > > need_build_host&q

Re: [Xen-devel] [PATCH v4 09/16] osstest: introduce a FreeBSD build script

2017-07-06 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v4 09/16] osstest: introduce a FreeBSD build script"): > On Thu, Jul 06, 2017 at 06:31:02PM +0100, Ian Jackson wrote: > > Although I have to say, evaling the output of make seems brave. > > Does BSD make only ever print blather to std

Re: [Xen-devel] [PATCH v4 10/16] osstest: add support for runtime_IDENT_hostflags

2017-07-06 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v4 10/16] osstest: add support for runtime_IDENT_hostflags"): > On Thu, Jul 06, 2017 at 04:28:09PM +0100, Ian Jackson wrote: > > I like the implied interface better than the one which implicitly > > overwrites all previous runt

Re: [Xen-devel] [PATCH v4 09/16] osstest: introduce a FreeBSD build script

2017-07-06 Thread Ian Jackson
Roger Pau Monne writes ("Re: [PATCH v4 09/16] osstest: introduce a FreeBSD build script"): > On Thu, Jul 06, 2017 at 04:25:24PM +0100, Ian Jackson wrote: > > Roger Pau Monne writes ("[PATCH v4 09/16] osstest: introduce a FreeBSD > > build script"): > >

Re: [Xen-devel] [PATCH v4 16/16] osstest: hook FreeBSD flight into cr-daily-branch

2017-07-06 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v4 16/16] osstest: hook FreeBSD flight into cr-daily-branch"): > Add the missing pieces so that cr-daily-branch knows how to create a > FreeBSD flight. This has been tested by calling `./cr-daily-branch > freebsd`. > > Note that there are two missing pieces. The fi

<    1   2   3   4   5   6   7   8   9   10   >