fix: update Dockerfile for post-PR#59 package layout#69
Merged
Conversation
data/datasets/, data/extents.yaml, and config/pygeoapi/base.yml no longer exist as repo-level files — they were moved into src/climate_api/data/ as bundled package data in PR #59. Remove the stale COPY instructions and delete config/pygeoapi/base.yml (superseded by src/climate_api/data/pygeoapi/base.yml). Set CACHE_OVERRIDE=/app/data so all runtime paths (downloads, artifacts, pygeoapi) resolve consistently under /app/data in the container.
abyot
reviewed
May 7, 2026
Member
abyot
left a comment
abyot
left a comment
There was a problem hiding this comment.
Update the README pygeoapi section so it reflects the new bundled-base-config and runtime-generated paths.
## pygeoapi
The OGC API is served by pygeoapi, mounted at `/ogcapi`. Its configuration is generated dynamically from published artifacts and written to the resolved runtime data directory (for Docker: `/app/data/pygeoapi/pygeoapi-config.yml`).
turban
commented
May 7, 2026
Contributor
Author
turban
left a comment
turban
left a comment
There was a problem hiding this comment.
Done — updated the pygeoapi section in README.md to reflect the bundled base config and runtime-resolved paths. The reference to config/pygeoapi/base.yml has also been removed from the regeneration note since that file no longer exists.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Docker build was broken after PR #59 moved several files into the package. The Dockerfile still tried to
COPYpaths that no longer exist:data/datasets/— moved tosrc/climate_api/data/datasets/as bundled package datadata/extents.yaml— removed entirely (replaced byextent:inCLIMATE_API_CONFIG)config/pygeoapi/base.yml— moved tosrc/climate_api/data/pygeoapi/base.ymlas bundled package dataChanges:
COPYinstructionsconfig/pygeoapi/base.yml(superseded by the bundled copy)CACHE_OVERRIDE=/app/dataso all runtime paths (downloads, artifacts, pygeoapi) resolve consistently under/app/datain the container via the XDG resolution logic added in PR Installable package foundation, Python client, and User-facing docs #59