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

tmt-1.31 breaks testing against alpine container image #2694

Closed
thrix opened this issue Feb 21, 2024 · 4 comments · Fixed by #2788
Closed

tmt-1.31 breaks testing against alpine container image #2694

thrix opened this issue Feb 21, 2024 · 4 comments · Fixed by #2788
Assignees
Labels
bug Something isn't working containers
Milestone

Comments

@thrix
Copy link
Collaborator

thrix commented Feb 21, 2024

See https://artifacts.dev.testing-farm.io/1b4cf023-dc40-4e28-885d-d4504ed89064/ for details. The new feature for package manager broke it.

@thrix thrix added this to the 1.32 milestone Feb 21, 2024
@thrix thrix self-assigned this Feb 21, 2024
@thrix thrix added bug Something isn't working containers labels Feb 21, 2024
@happz
Copy link
Collaborator

happz commented Feb 21, 2024

For the record, what broke it is the new explicit requirement of flock being available on the guest. Without it, the tests won't run correctly, as was discovered with CentOS Stream container images, hence the flock being a requirement now. Which results in tmt trying to check for it & maybe even install it.

Hopefully, the package manager PR would allow for adding a package manager implementation for apk..

@thrix
Copy link
Collaborator Author

thrix commented Feb 21, 2024

thx for correction

@psss
Copy link
Collaborator

psss commented Mar 25, 2024

The package manager pull request #2557 has been merged. @thrix, do you plan to write the apk plugin?

thrix added a commit that referenced this issue Mar 25, 2024
Add support for `apk` package manager from Alpine Linux.

Unsupported debuginfo installation, seems not have a nice
`debuginfo-install`-like solution.

Installing by path is limited to certain recognized paths.
The existing `apk-file` solution looks flakey, it was randomly
hanging and does not have a good machine readable output.

For testing we need to use a locally build alpine image
which contains required packages of tmt and the unit tests.

Resolves #2694

Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
@thrix
Copy link
Collaborator Author

thrix commented Mar 25, 2024

@psss it is done \o/

@psss psss closed this as completed in 32f43b7 Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working containers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants