Skip to content

feat(ardac1dmo): pin image tags to 2026.03 for gen3 services#13

Merged
alan-walsh merged 4 commits intomainfrom
feat/ardac1dmo-pin-image-tags
Mar 11, 2026
Merged

feat(ardac1dmo): pin image tags to 2026.03 for gen3 services#13
alan-walsh merged 4 commits intomainfrom
feat/ardac1dmo-pin-image-tags

Conversation

@alan-walsh
Copy link
Copy Markdown
Contributor

Summary

Pin explicit image.tag: 2026.03 for all enabled gen3 services in the ardac1dmo/demo.ardac.org environment whose upstream chart defaults are "" (empty) or master.

Services Updated

  • arborist — previous default: ""
  • audit — previous default: master
  • aws-es-proxy — previous default: ""
  • fence — previous default: master
  • hatchery (main image + fence sidecar) — previous defaults: "" / master
  • indexd — previous default: ""
  • manifestservice — previous default: ""
  • peregrine — previous default: master
  • revproxy — previous default: master
  • sheepdog — previous default: master
  • wts — previous default: master

Services Left Unchanged

  • ambassador — upstream default is 1.4.2 (specific version)
  • metadata — upstream default is feat_es-7 (specific version)
  • portal — already uses custom image leftsweety/ardac-portal:1_11_23

alan-walsh and others added 2 commits March 9, 2026 17:18
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add explicit image.tag: 2026.03 for all enabled services whose
upstream chart defaults are empty or master. Services with specific
version defaults (ambassador: 1.4.2, metadata: feat_es-7) and the
custom portal image are left unchanged.

Services updated: arborist, audit, aws-es-proxy, fence, hatchery
(including fence sidecar), indexd, manifestservice, peregrine,
revproxy, sheepdog, wts

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 9, 2026 21:26
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Pins explicit image.tag: "2026.03" values in Gen3 Helm values files to avoid relying on upstream chart defaults (e.g., empty tag or master) for enabled services.

Changes:

  • Add image.tag: "2026.03" for multiple enabled Gen3 services in ardac1dmo/demo.ardac.org/values.yaml (including Hatchery main image and its Fence sidecar override).
  • Add image.tag: "2026.03" for multiple services in ardac1prd/portal.ardac.org/values.yaml (including Guppy).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ardac1prd/portal.ardac.org/values.yaml Pins multiple service image tags to 2026.03 in the production values file.
ardac1dmo/demo.ardac.org/values.yaml Pins multiple service image tags to 2026.03 in the demo environment values file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 37 to 41
arborist:
enabled: true
image:
tag: "2026.03"
externalSecrets:
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

PR title/description indicate this change is scoped to the ardac1dmo demo environment, but this PR also pins image tags in the ardac1prd production values file. If production pinning is intended, please update the PR title/description (and the list of updated services) to reflect the broader scope; otherwise, revert the ardac1prd/portal.ardac.org changes to avoid an unintended production rollout.

Copilot uses AI. Check for mistakes.
Comment on lines 109 to 113
guppy:
enabled: true
image:
tag: "2026.03"
indices:
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

This adds a pinned image tag for guppy in ardac1prd, but guppy is not listed under “Services Updated” in the PR description. Please either add guppy to the documented service list/scope (and confirm its upstream default warranted pinning), or drop this change to keep the PR consistent with its stated intent.

Copilot uses AI. Check for mistakes.
alan-walsh and others added 2 commits March 9, 2026 17:34
Accidentally modified this file in previous commit. Only ardac1dmo
changes were intended.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add image.repository alongside image.tag for all 11 services updated
with the 2026.03 tag, making the full image reference visible and
easily editable in values.yaml.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@alan-walsh alan-walsh merged commit a37754a into main Mar 11, 2026
1 check passed
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.

3 participants