Skip to content

.zuul: Enable testing on Fedora 39#1385

Merged
debarshiray merged 1 commit intocontainers:mainfrom
debarshiray:wip/rishi/test-f39
Oct 6, 2023
Merged

.zuul: Enable testing on Fedora 39#1385
debarshiray merged 1 commit intocontainers:mainfrom
debarshiray:wip/rishi/test-f39

Conversation

@debarshiray
Copy link
Copy Markdown
Member

@debarshiray debarshiray commented Oct 6, 2023

No description provided.

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul bot commented Oct 6, 2023

Build failed.
https://softwarefactory-project.io/zuul/t/local/buildset/fb8530145ec042ef92b5c6d554fe3b09

✔️ unit-test SUCCESS in 9m 00s
✔️ unit-test-migration-path-for-coreos-toolbox SUCCESS in 3m 30s
✔️ unit-test-restricted SUCCESS in 7m 51s
❌ system-test-fedora-rawhide FAILURE in 38m 49s
❌ system-test-fedora-39 FAILURE in 36m 15s
✔️ system-test-fedora-38 SUCCESS in 30m 34s
✔️ system-test-fedora-37 SUCCESS in 29m 18s

@debarshiray
Copy link
Copy Markdown
Member Author

debarshiray commented Oct 6, 2023

There are still some test failures on Fedora Rawhide and Fedora 39.

For example, these that seem to be because of changes in various other components in Fedora:

fedora-rawhide | not ok 3 help: Run command 'help' in 145ms
fedora-rawhide | # (from function `assert_line' in file test/system/libs/bats-assert/src/assert.bash, line 479,
fedora-rawhide | #  in test file test/system/002-help.bats, line 45)
fedora-rawhide | #   `assert_line --index 0 --partial "toolbox(1)"' failed
fedora-rawhide | # /usr/bin/man
fedora-rawhide | #
fedora-rawhide | # -- line does not contain substring --
fedora-rawhide | # index     : 0
fedora-rawhide | # substring : toolbox(1)
fedora-rawhide | # line      : troff:<standard input>:33: warning: cannot select font 'C'
fedora-rawhide | # --

However, at least these two failures are a fallout from the Fedora 39 Change that made the fedora-toolbox:39 and above images to be generated as part of the nightly composes:

fedora-rawhide | not ok 82 list: Default image in 39783ms
fedora-rawhide | # (from function `assert_line' in file test/system/libs/bats-assert/src/assert.bash, line 479,
fedora-rawhide | #  in test file test/system/102-list.bats, line 99)
fedora-rawhide | #   `assert_line --index 1 --partial "$default_image"' failed
fedora-rawhide | #
fedora-rawhide | # -- line does not contain substring --
fedora-rawhide | # index     : 1
fedora-rawhide | # substring : registry.fedoraproject.org/fedora-toolbox:40
fedora-rawhide | # line      :
fedora-rawhide | # --
fedora-rawhide | #
fedora-rawhide | not ok 83 list: Default image (using --images) in 39062ms
fedora-rawhide | # (from function `assert_line' in file test/system/libs/bats-assert/src/assert.bash, line 479,
fedora-rawhide | #  in test file test/system/102-list.bats, line 117)
fedora-rawhide | #   `assert_line --index 1 --partial "$default_image"' failed
fedora-rawhide | #
fedora-rawhide | # -- line does not contain substring --
fedora-rawhide | # index     : 1
fedora-rawhide | # substring : registry.fedoraproject.org/fedora-toolbox:40
fedora-rawhide | # line      :
fedora-rawhide | # --
fedora-rawhide | #

The images are now built with Image Factory with Kickstarts and Pungi instead of with OpenShift Build Service cluster from Dockerfiles. During that migration we lost the com.github.containers.toolbox LABEL from the images and they no longer show up as Toolbx images.

I submitted some pull requests to address that:

@debarshiray debarshiray merged commit 53a0316 into containers:main Oct 6, 2023
@debarshiray
Copy link
Copy Markdown
Member Author

debarshiray commented Oct 6, 2023

In the mean time, while we track down and fix these failures, I am going to temporarily override these failures.

@debarshiray debarshiray deleted the wip/rishi/test-f39 branch October 6, 2023 13:25
debarshiray added a commit to debarshiray/toolbox that referenced this pull request Jan 27, 2026
@debarshiray
.zuul: Enable testing on Fedora 39
72e6344 
containers#1385
containers#1744
(cherry picked from commit 53a0316)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@martymichal martymichal Awaiting requested review from martymichal

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@debarshiray