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

fix: retire gpu dedicated aks image #5548

Merged
merged 22 commits into from
Jan 11, 2025
Merged

Conversation

zachary-bailey
Copy link
Collaborator

What type of PR is this?

/kind deprecation

What this PR does / why we need it:

This PR deprecates the 1804gpucontainerd images.

Requirements:

@@ -378,6 +378,12 @@ const (
// of support and image builds have stopped.
FrozenCBLMarinerV1SIGImageVersionForDeprecation string = "202308.28.0"

// DO NOT MODIFY: 1804GPUContainerd Gen1 & Gen2 pinned to the last image build as
// GPU Dedicated preview image is being deprecated and image builds have stopped.
Frozen1804Gen1GPUDedicatedVHDContainerdSIGIMageVersionForDeprecation string = "202412.29.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: Frozen1804GPUContainerdSIGImageVersionForDeprecation

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

I would also do

Frozen1804Gen2GPUDedicatedVHDContainerdSIGIMageVersionForDeprecation -> Frozen1804Gen2GPUContainerdSIGImageVersionForDeprecation

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks. Changed it. Not sure why I made those different haha.

E2E is failing for PR gate for all PRs that I see, so Im going to wait until all the other checks pass and then bypass merge for the release Monday.

Copy link
Contributor

@anujmaheshwari1 anujmaheshwari1 left a comment

Choose a reason for hiding this comment

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

lgtm, kill it!

@zachary-bailey zachary-bailey merged commit ee75fdd into dev Jan 11, 2025
11 of 14 checks passed
@zachary-bailey zachary-bailey deleted the zb/retireGPUDedicatedVHD branch January 11, 2025 00:13
zachary-bailey added a commit that referenced this pull request Jan 11, 2025
Co-authored-by: Zachary Bailey <[email protected]>
Co-authored-by: Zachary Bailey <[email protected]>
zachary-bailey added a commit that referenced this pull request Jan 11, 2025
Co-authored-by: Zachary Bailey <[email protected]>
Co-authored-by: Zachary Bailey <[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.

3 participants