Skip to content
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

add MapServer configuration file for MapServer 8 #102

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

Dukestep
Copy link
Contributor

This PR adds the generation of the mapserver.conf file via the geomet-climate mapfile generate comman. The MS_MAP_PATTERN path uses the GEOMET_CLIMATE_BASEDIR environment variable to determine the file's filepath.

I've also updated the Dockerfile to add a MAPSERVER_CONFIG_FILE environment variable.

@Dukestep Dukestep force-pushed the add-mapserver-config-file branch from 56083dc to e425e3a Compare December 30, 2024 18:32
@Dukestep Dukestep force-pushed the add-mapserver-config-file branch from e425e3a to 9b35b7e Compare January 8, 2025 18:24
@tomkralidis tomkralidis merged commit 6002a6d into ECCC-CCCS:jammy Jan 8, 2025
1 check passed
kngai pushed a commit to kngai/geomet-climate that referenced this pull request Jan 16, 2025
tomkralidis added a commit that referenced this pull request Jan 28, 2025
* Docker changes for dev-31: (#98)

- Testing with wmo-staging PPA
- Check to ensure required ENVs are set
- Add SSL ceritficate volume mounts for dev-31

Co-authored-by: Geomet user for nightly docker builds <[email protected]>

* ensure MAP.WEB.METADATA wms_allow_getmap_without_styles set to true (#101)

* add MapServer configuration file for MapServer 8 (#102)

* Docker changes for dev-31 and MapServer 8: (#98) (#100)

- Testing with wmo-staging PPA
- Check to ensure required ENVs are set
- Add SSL ceritficate volume mounts for dev-31

Co-authored-by: Etienne Pelletier <[email protected]>

* Update deploy-nightly-docker.sh

---------

Co-authored-by: Geomet user for nightly docker builds <[email protected]>
Co-authored-by: Etienne Pelletier <[email protected]>
Co-authored-by: Tom Kralidis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants