Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve tests on almalinux OSes #584

Merged
merged 5 commits into from
Aug 9, 2022

Conversation

roussidis
Copy link
Contributor

  • Add necessary package glibc-static on almalinux 8 and almalinux 9
  • Created a new function to get the kernel headers based on the linux distro.
  • Moved the Start sysbox block after the initialization of the docker because it is dependend on the docker itself to find out the host filesystem
  • Add an extra check regarding the host filesystem. If it is xfs idmapped_mounts is not supported.
  • Add the correct kernel header paths in the tests.

Signed-off-by: Christos Roussidis <xristos.roussidis@gmail.com>
- Check the filesystem as well because xfs is not supported from idmapped
  mounts.
  If the system has xfs filesystem the tests will skip idmapped mount
  tests.
- Add lifecycle tests last because they were not respecting the
  `--disable-idmapped-mount` flag that was being set from the previous
  checks.

Signed-off-by: Christos Roussidis <xristos.roussidis@gmail.com>
Improve kernel header path based on the linux distro that the tests are
running.

- Drop lsb_release use, use `cat /etc/os-release` instead.
- Create a function which defines the kernel headers path based on the
  host distro.
- Use that function in the sysmgr tests.

Signed-off-by: Christos Roussidis <xristos.roussidis@gmail.com>
Signed-off-by: Christos Roussidis <xristos.roussidis@gmail.com>
Copy link
Member

@ctalledo ctalledo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great in general, thanks @roussidis; just one comment / question.

Copy link
Member

@rodnymolina rodnymolina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question below.

Signed-off-by: Christos Roussidis <xristos.roussidis@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants