Re: [libvirt PATCH 2/2] ci: integration: Set 'safe.directory' when installing QEMU from git

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 09, 2022 at 06:01:34AM -0400, Andrea Bolognani wrote:
> On Wed, Jun 08, 2022 at 08:03:07AM +0200, Erik Skultety wrote:
> > FWIW we could alternatively update the submodules manually, but we'd have list
> > them explicitly, IOW:
> >     $ git clone qemu ...
> >     $ cd qemu.git
> >     $ scripts/git-submodule.sh ui/keycodemapdb dtc slirp
> 
> We could avoid hardcoding the names of the submodules by using
> something along the lines of
> 
>   $ ./scripts/git-submodule.sh update $(git submodule | awk '{print
> $2}' | grep -Ev '^(meson|roms/.*|tests/.*)$')
> 
> A bit of a mouthful, but should be solid enough.
> 
> >     $ mkdir build && cd build
> >     $ ../configure ... --with-git-submodules=ignore
> 
> Using
> 
>   --with-git-submodules=validate
> 
> would work too, since we'd have updated the submodules beforehand.

'validate' will still cause QEMU to run git commands to check
the submodule state, so I presume it'll still hit the problem
of ownership.

> I think I would prefer this approach to changing the git
> configuration for the root user.

I was going to say the opposite. Updating the root user git config
is harmless since our integration suite is intended to always run
inside a single use throwaway VM. IOW, we already assume the VM is
compromised at the end of every test cycle.

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux