-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Update test base image and move CI to Github workflow #39815
Conversation
22a9cee
to
42181f4
Compare
When I took a look to the build, I see
Is there any sort of disk limitation on GitHub workflow. It doesn't look to an Oracle related issue |
@3nids looks like the results pushed to cdash are missing the PR identifier too and show up as just |
Well... at a preliminary glance it looks like there's ~15 of these which just need new reference images. The rest look... complicated 😱 |
ok with QML |
@troopa81 I increased the Docker base image to 50gb (9365c55) and it went through in the push build (https://github.com/qgis/QGIS/pull/39815/checks?check_run_id=1359335494#step:11:33) but not the pull request one |
The timeout is too low (5 minutes here). It takes approximately 10 minutes on my machine for the Oracle database to be ready to accept connections... And this is without the image pulling. I assume there is a Docker image cache but since we switch to GH workflow maybe the image is not in the cache. |
@troopa81, thanks I increased the timeout to 17 minutes |
@troopa81 just wondering, should we start the DBs before building QGIS? that would let them time to start while we compile? |
It could be good idea indeed. I'm currently trying to make an already started oracle database image. The resulting image would be huge (around 8 GB I think) but we wouldn't have to wait for 10 minutes! Not completely sure I manage to do it though |
I would propose to run everything in the compose (i.e. also the compilation):
The backward is that we fire the DBs (a few minutes to pull the images) even if the compilation fails. |
Still a timeout. Is it possible to have some logs about what happen inside the Docker Oracle instance? The equivalent of |
I finally managed to reduce the oracle start-up time from 16 minutes to 3 minutes. It's here. You could rebase or just pick the modification. |
…ometry() This takes the first top level window and use it to get the screen. The "perfect" approach would be to use a widget as argument in the method. @PeterPetrik thoughts?
error: QQmlComponent: Component is not ready
@troopa81, rebased, thanks a lot! |
@3nids you may have to increase disk space. The new oracle image is around 2 Go bigger than the previous one |
These are fixed by #40039 |
@3nids would it be possible to commit the parts of this which add the new workflow only for now? Then we could at least avoid people introducing new test failures |
Manually marking as stale because stalebot has left for christmas break |
The QGIS project highly values your contribution and would love to see this work merged! Unfortunately this PR has not had any activity in the last 14 days and is being automatically marked as "stale". If you think this pull request should be merged, please check
|
While we hate to see this happen, this PR has been automatically closed because it has not had any activity in the last 21 days. If this pull request should be reconsidered, please follow the guidelines in the previous comment and reopen this pull request. Or, if you have any further questions, just ask! We love to help, and if there's anything the QGIS project can do to help push this PR forward please let us know how we can assist. |
Dependency image has been moved to ubuntu focal (this means Qt 5.12)
CI is moved away from Travis to Github workflow
Caching is working (ccache)
issues:
List of failing tests:
Link to current raw log
Open question: how to deal with failing tests? is there any volunteer?