ZFS isn't supported in debian, it's always been a community side-load
situation (think EPEL for CentOS). This is the only containers repo.
that makes use of it in CI. Worse, since we're testing with SID,
installing (i.e. compiling) the ZFS kernel module is an even more
fragile and failure prone process. Because of those issues I've
stripped it out of these CI VM images. I no-longer wish to support
its use in this CI setup. If somebody else wants to support it, they're
more than welcome and free to do so.
Signed-off-by: Chris Evich <cevich@redhat.com>
Updated Debian CI VM Image.
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Chris Evich <cevich@redhat.com>
Cirrus CI emits a bunch of warnings like this one:
> task "success" depends on task "fedora-38 vfs", but their only_if conditions are different
This was probably copied from another repo, which uses CI:DOCS magic to
skip some tests. In this repo, it is not used, so remove the condition.
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Refactor Ubuntu and Fedora tests to depend on a common base that isn't a
task, so that we don't run the Ubuntu test matrix-of-scenarios on top of
the Fedora test matrix-of-scenarios, where they would multiply.
Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
Make a file and mount it over loopback to ensure that we can test btrfs,
assuming we were built with the libraries and the kernel supports it.
Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
The new Reexp structure, will handle the initialization of regexp
and then will do the MustCompile within a sync.Once field.
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
If STORAGE_TRANSIENT is set to 1 then transient mode will be enabled
in the tests.
Also adds overlay-transient to the cirrus CI.
Signed-off-by: Alexander Larsson <alexl@redhat.com>
Make sure all PRs are formated with `go fmt` correctly.
I added it to the existing lint job as this seems to make the most
sense to me.
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
We can reenable this once we have fedora 37 images available and PRIOR
becomes f36
Revert this patch and switch to f37/f36 when ready
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
For consistency with CI in other repositories, update the variable name.
Also, remove some variable definitions which are not included in the
common automation library. Finally, strip off the special Fedora
partition setup, as it's no-longer supported in the VM images.
Signed-off-by: Chris Evich <cevich@redhat.com>
Also modernize main library to resemble what's used in other
containers-org repositories. Lastly, update hack/get_ci_vm.sh to use
the new shared container image.
Signed-off-by: Chris Evich <cevich@redhat.com>
Removed by request, due to two long-term persistent problems:
* The first test fails due to `level=error msg="File descriptor 3
(pipe:[60185]) leaked on pvdisplay invocation. Parent PID 29275:
/var/tmp/go/src/github.com/cont\nFile descriptor 10
(/dev/mapper/control) leaked on pvdisplay invocation. Parent PID 29275:
/var/tmp/go/src/github.com/cont\n Failed to find physical volume
\"/dev/sda2\".\n" error="exit status 5"`
* All tests then seem to fail due to a non-pristine state: `error
initializing: /dev/sda2 is already part of a volume group "storage":
must remove this device from any volume group or provide a different
device`
Signed-off-by: Chris Evich <cevich@redhat.com>
renovate[bot]