Skip to content

Update ARGS for to streamline customer delivery - #24

Merged
cmaynard-ngwpc merged 1 commit into
developmentfrom
cmaynard_update_docker_org_args
Apr 27, 2026
Merged

Update ARGS for to streamline customer delivery#24
cmaynard-ngwpc merged 1 commit into
developmentfrom
cmaynard_update_docker_org_args

Conversation

@cmaynard-ngwpc

@cmaynard-ngwpc cmaynard-ngwpc commented Apr 20, 2026

Copy link
Copy Markdown

In order to facilitate easier merging of NGWPC code to the parent OWP repositories, several Dockerfile ARG entries have been streamlined and moved to the top.

Additions

  • Added ARG IMAGE_NAMESPACE
  • Instructions in a comment at the top specifying which ARG's to update when this Dockerfile is adopted by OWP

Changes

  • ARG ORG changed to ARG GHCR_ORG
  • Use GH_ORG when referencing the nwm-msw-mgr GitHub repository.
  • ARG EWTS_ORG changed to ARG GH_ORG and moved to the top of the Dockerfile
  • Moved ARG EWTS_REF to the top of the Dockerfile
  • ARG MSW_MGR_VERSION changed to ARG MSW_MGR_REF and moved to the top of the Dockerfile
  • Use IMAGE_NAMESPACE in place of ngwpc in the Dockerfile LABEL declaration

@cmaynard-ngwpc
cmaynard-ngwpc force-pushed the cmaynard_update_docker_org_args branch 3 times, most recently from 5b0b010 to f35ce76 Compare April 22, 2026 21:34
@cmaynard-ngwpc
cmaynard-ngwpc force-pushed the cmaynard_update_docker_org_args branch from 0a35f1e to dd7c14e Compare April 22, 2026 21:43

@miguelp1986 miguelp1986 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pipeline passes with changes. Approved.

@cmaynard-ngwpc
cmaynard-ngwpc merged commit 67d1dc2 into development Apr 27, 2026
7 checks passed
@cmaynard-ngwpc
cmaynard-ngwpc deleted the cmaynard_update_docker_org_args branch April 27, 2026 16:28
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