Uploaded image for project: 'RHEL'
  1. RHEL
  2. RHEL-147454

virtiofs: Improve CI

Linking RHIVOS CVEs to...Migration: Automation ...Sync from "Extern...XMLWordPrintable

    • Icon: Story Story
    • Resolution: Unresolved
    • Icon: Normal Normal
    • None
    • None
    • virtiofsd
    • None
    • None
    • False
    • False
    • Hide

      None

      Show
      None
    • None
    • VirtStorage Planning backlog
    • None
    • None
    • Unspecified
    • Unspecified
    • Unspecified
    • None

      Background

      Our CI currently only does a fmt+clippy+test check, and compiles a static version and the documentation.

      That is, beyond checking code style, clippy hints, and testing that everything compiles, we only have the cargo test cases, which are very limited in scope. We have more comprehensive tests elsewhere (e.g. xfstests, pjdfstest, migration tests), but those are in no capacity part of CI, and it’s cumbersome to run them.

      Goal

      As someone who’s interested in the easy testability and maintainability of virtiofsd, the comprehensive testing we have elsewhere must be made more accessible, ideally including making as much of it part of CI as possible and feasible.

      Ideally we can also increase the coverage of cargo test.

      Note on Virtualization

      xfstests and pjdfstest test the mounted filesystem, so currently must be run in a VM guest operating system. This probably complicates running them in a CI environment. If it helps, we can patch virtiofsd to allow mounting the filesystem on the host as a “normal” FUSE filesystem to remove the virtualization requirement.

      Acceptance criteria

      • Existing comprehensive test suites are more easily accessible (runnable)
      • CI is extended by to include at least some more comprehensive tests
      • Optional: Increase coverage of cargo test

              rhn-support-qcheng Qinghua Cheng
              hreitz@redhat.com Hanna Czenczek
              virt-maint virt-maint
              virt-bugs virt-bugs
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated: