-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Test runner not working within recent release-3_22
docker image
#50729
Comments
Same regression with 3.26 and/or 3.28 ? |
No, neither on 3.24/3.26/3.28 nor on master. Only on the 3.22 branch. The I also noticed that on the |
I started to further investigate on this issue. It turns out that within the
This is on 3.22 container, the failing one:
This is the output on one of the working images:
|
It seems to me that the build process does not fully succeed, but the image still gets pushed to docker hub. The build exit value is
vs. on working images
This should get picked up by the following lines in the workflow: QGIS/.github/workflows/build-docker.yml Lines 130 to 133 in bf44964
I don't quite understand why that does not work |
after searching through the action logs it seems to me that Linking CXX shared library output/lib/libqgis_3d is the step in the build process that always fails with following error, e.g. here
This happens in the Finalize image step of the workflow where there is no guarding check against failing build. Unfortunately I don't understand much about compiling C++ and the internals of QGIS docker builds but |
Same problem on qgis from docker image deployment. It seems image is badly formed or some ref libs not respond correctly |
@CodeBardian Backport PR has been opened #50850 Could you check after it has been merged and update this issue ? |
Setting feedback label as PRs have been merged. |
Everything working again 🚀 |
qgis/QGIS#50729 is now fixed. So tests should work without this deticated requirements.txt.
qgis/QGIS#50729 is now fixed. So tests should work without this deticated requirements.txt.
qgis/QGIS#50729 is now fixed. So tests should work without this deticated requirements.txt.
EDIT:
all recent docker images of the 3.22 branch are useless because the QGIS executable is not present. This is due to failing build when creating docker images. See my latest comments on this thread for details. The issue description below can be ignored, it will probably fix itself once the build is working again.
What is the bug or the crash?
I am using the
qgis_testrunner.sh
script to run some unit tests for a QGIS-Plugin, which always worked fine. However, since today something is off within therelease-3_22
container.The test runner always returns with
ERROR: no output from the test runner! (exit code: 0)
The previous
release-3_22
image with digest sha256:a05df35259317e3457d4cdc73f0aabf0623f1e7a8257c56bff10b041ded3dce3 which was the latest available until yesterday runs fine.I put a minimal sample within the following repository where I ran the test runner script within GitHub Actions: https://github.com/CodeBardian/qgis-docker-test
You can see in the Actions tab that the exact same test passes on almost all of the other containers: https://github.com/CodeBardian/qgis-docker-test/actions/runs/3344611088. The first failing one is introduced with
final-3_22_10
.Appreciate if anyone has some insights on this issue.
Steps to reproduce the issue
docker pull qgis/qgis:release-3_22
docker run -d --name qgis22 -e DISPLAY=:99 qgis/qgis:release-3_22
docker start qgis22
docker exec -it qgis22 sh -c "qgis_testrunner.sh"
output:
expected (result from previous image):
Versions
3.22
Supported QGIS version
New profile
Additional context
No response
The text was updated successfully, but these errors were encountered: