[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib |
Date: |
Wed, 30 Aug 2017 21:14:41 +0800 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Wed, 08/30 18:16, Fam Zheng wrote:
> On Tue, 08/29 10:35, Philippe Mathieu-Daudé wrote:
> > On 08/29/2017 10:22 AM, Kamil Rytarowski wrote:
> > > On 29.08.2017 15:10, Philippe Mathieu-Daudé wrote:
> > > > Hi Fam, Kamil,
> > > >
> > > > > On 08/28/2017 02:47 PM, Fam Zheng wrote:
> > > > [...]
> > > > > > + subprocess.check_call([tar_cmd,
> > > > > > + "--exclude-vcs",
> > > > > > + "--exclude=tests/vm/*.img",
> > > > > > + "--exclude=tests/vm/*.img.*",
> > > > > > + "--exclude=*.d",
> > > > > > + "--exclude=*.o",
> > > > > > + "--exclude=docker-src.*",
> > > > > > + "-cf", tarfile, '.'], cwd=data_dir,
> > > > >
> > > > > I'm not happy with this command :/
> > > > > My distrib uses tmpfs for /tmp and suddently the whole X window became
> > > > > irresponsive until this script failing after filling 8G of /tmp and
> > > > > swap:
> > > > >
> > > > > ...
> > > > > DEBUG:root:Creating archive /tmp/qemu-vm-F7CY9O/data-3a52c.tar for
> > > > > data dir: .
> > > > > tar: /tmp/qemu-vm-F7CY9O/data-3a52c.tar: Wrote only 4096 of 10240
> > > > > bytes
> > > > > tar: Error is not recoverable: exiting now
> > > > > Failed to prepare guest environment
> > > > >
> > > > > Then I figured out my workdir is full of testing stuff, debug images,
> > > > > firmwares, coredumps, etc.
> > > > >
> > > > > I'll think of another way.
> > > >
> > > > Using:
> > > >
> > > > (git ls-files; git submodule foreach --recursive "git ls-files | sed
> > > > s_^_\$sm_path/_" | sed /^Entering/d) > files.txt
> > > >
> > > > and:
> > > >
> > > > subprocess.check_call([tar_cmd,
> > > > "-cf", tarfile,
> > > > "-v" if self.debug else "",
> > > > "-T", "files.txt"], cwd=data_dir,
> > > > stdin=self._devnull, stdout=self._stdout)
> > > >
> > > > Current /master and submodules generated tarball is ~305MB uncompressed.
> > > >
> > > > Kamil do you mind testing this command on NetBSD?
> > > >
> > > > (obviously 'files.txt' goes in tmpdir).
> > > >
> > >
> > > Result:
> > >
> > > http://www.netbsd.org/~kamil/qemu/files.txt
> >
> > \o/
> >
> > > I've tested this command line on a checkout from Jul 17th rev.
> > > 77031ee1ce4c
> >
> > Thanks Kamil :)
> >
> > Fam are you OK using this command instead?
>
> Looks good, will use it! Thanks.
Hmm, there is a complication. "git ls-files" output includes submodule names,
which are directory names from the "tar -T" PoV, and the whole directory is
archived.
I'm going to stop fighting on this and refactor the docker tests's "git archive"
rules, to reuse in this script.
Fam
- [Qemu-devel] [PATCH v4 02/10] qemu.py: Add "wait()" method, (continued)
- [Qemu-devel] [PATCH v4 02/10] qemu.py: Add "wait()" method, Fam Zheng, 2017/08/28
- [Qemu-devel] [PATCH v4 04/10] tests: Add ubuntu.i386 image, Fam Zheng, 2017/08/28
- [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Fam Zheng, 2017/08/28
- Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Philippe Mathieu-Daudé, 2017/08/29
- Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Philippe Mathieu-Daudé, 2017/08/29
- Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Kamil Rytarowski, 2017/08/29
- Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Philippe Mathieu-Daudé, 2017/08/29
- Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Fam Zheng, 2017/08/30
- Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib,
Fam Zheng <=
Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Philippe Mathieu-Daudé, 2017/08/29
Re: [Qemu-devel] [PATCH v4 03/10] tests: Add vm test lib, Philippe Mathieu-Daudé, 2017/08/29
[Qemu-devel] [PATCH v4 05/10] tests: Add FreeBSD image, Fam Zheng, 2017/08/28
[Qemu-devel] [PATCH v4 06/10] tests: Add NetBSD image, Fam Zheng, 2017/08/28