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
Migrate docker image off geobase to osgeo/gdal #1233
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1233 +/- ##
========================================
Coverage 93.79% 93.79%
========================================
Files 103 103
Lines 10597 10597
========================================
Hits 9939 9939
Misses 658 658 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only change needed really is to pin the FROM docker image version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Reason for this pull request
Docker image used for testing was built on
datacube/geobase
which is now deprecated.Proposed changes
New Docker build based on
osgeo/gdal
. Simple single-stage docker build, could be further optimised but seems sufficient for CI testing and dev purposes.docs/about/whats_new.rst
for all changes