Skip to content

ci(deploy): use runtime-versioned image tags - #12359

Closed
sttts wants to merge 43 commits into
tmonty12/dyn-940-runtime-versionfrom
sttts-ci-runtime-version-tags
Closed

ci(deploy): use runtime-versioned image tags#12359
sttts wants to merge 43 commits into
tmonty12/dyn-940-runtime-versionfrom
sttts-ci-runtime-version-tags

Conversation

@sttts

@sttts sttts commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • tag CI deployment images as <dynamo-version>-ci-<sha>-<image> when copying them to ACR
  • keep the immutable SHA tag as the ECR build artifact
  • pass the resulting ACR tag through deploy, checkpoint, and GAIE workflows
  • remove test-side runtimeVersionOverride injection and version parsing

Dependency

Prefactor: #12362 introduces the reusable CI image-tagging changes against main. This stacked PR currently carries those workflow changes for end-to-end validation. After #12362 merges, this PR will be rebased onto it and retain only the #10494-specific test cleanup.

Why

#10494 validates runtime compatibility from the main container image tag. CI currently replaces those images with SHA-tagged artifacts, forcing generic test utilities to add an override. Giving the CI artifacts an accurate SemVer-compatible tag lets admission use the normal production path.

Validation

  • pre-commit hooks
  • git diff --check

Stacked on #10494.

@github-actions github-actions Bot added ci Issues/PRs that reference CI build/test actions labels Jul 29, 2026
@datadog-official

This comment has been minimized.

@github-actions github-actions Bot added the deployment::k8s Relates to dynamo deployment in kubernetes label Jul 29, 2026
@sttts
sttts force-pushed the sttts-ci-runtime-version-tags branch from 6d02f59 to 720205e Compare July 29, 2026 16:07
tmonty12 added 24 commits July 29, 2026 18:19
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
tmonty12 and others added 18 commits July 29, 2026 18:19
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Thomas Montfort <tjmontfort12@gmail.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
@sttts
sttts force-pushed the tmonty12/dyn-940-runtime-version branch from 6c603f8 to c1c2e19 Compare July 29, 2026 16:21
Signed-off-by: Dr. Stefan Schimanski <sschimanski@nvidia.com>
@sttts
sttts force-pushed the sttts-ci-runtime-version-tags branch from 720205e to 0ac59bb Compare July 29, 2026 16:22
@sttts
sttts force-pushed the tmonty12/dyn-940-runtime-version branch from c1c2e19 to 23f0120 Compare July 29, 2026 18:36
@sttts

sttts commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

Closing this follow-up because its final changes are now folded directly into #10494 after rebasing #10494 onto the prefactor in #12362. The workflow portion is provided by #12362, and the remaining test-helper cleanup is already present on current main.

@sttts sttts closed this Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

actions ci Issues/PRs that reference CI build/test deployment::k8s Relates to dynamo deployment in kubernetes size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants