-
Notifications
You must be signed in to change notification settings - Fork 19
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 Dockerfile.ngen to add NGEN_WITH_SQLITE: ON #73
Conversation
Related to #63 For Testing gpkg support in NGIAB
@ZacharyWills |
I tested this PR with the image created i.e. :latest-PR tag, and it seems like not working. NGEN_WITH_SQLITE: OFF bash-5.1# /dmod/bin/ngen --info |
According to Zach, the image works fine when built locally. Therefore, it seems that further investigation is required. |
One thing I can think of is no-cache option during build. |
It doesn't, when built locally with --no-cache it builds fine. It's only an issue when built by the runner. |
I have removed all caches from Runner. |
Ben, please let us know when you rerunun the CI against this PR. Also, please share the image name so Zach could test and make sure this PR is working. |
* Run actions when PR is created and updated. (#74) * refactor to support to run actions on pr * add debug * add docker token * add input * add env * define env * define env * define env * define env * add secrets * add secrets * secrets inherit * fix tag name * fix tag name * fix tag name * set input required * fix * use vars * fix variable * remove debug * Update HelloNGEN.sh (#75) * Use image tag based on branch (#76) * set tag name based on branch * remove unused inputs --------- Co-authored-by: ZacharyWills <[email protected]>
It is building now. |
github is not very good at diff. |
Build failed.
https://github.com/CIROH-UA/NGIAB-CloudInfra/actions/runs/7479625300/job/20357925663 |
@arpita0911patel @ZacharyWills |
Opend #79 |
Closing this one as #79 is opened for the same. |
Related to #63
For Testing gpkg support in NGIAB