Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Deprecate image settings in AKODeploymentConfig #1942

Merged
merged 1 commit into from
Oct 11, 2021

Conversation

nicolehanjing
Copy link
Contributor

What this PR does / why we need it

Deprecate image settings in AKODeploymentConfig.
This PR removes image settings in akoo 1.4.0 and 1.5.0.

Details for the Release Notes (PLEASE PROVIDE)

Deprecate image settings in AKODeploymentConfig.

Which issue(s) this PR fixes

Fixes: #

Describe testing done for PR

Special notes for your reviewer

@nicolehanjing nicolehanjing requested a review from a team as a code owner September 23, 2021 19:01
@github-actions github-actions bot added the owner/packages Work executed by a package's maintainer label Sep 23, 2021
@jpmcb jpmcb changed the title [WIP] Deprecate image settings in AKODeploymentConfig Deprecate image settings in AKODeploymentConfig Sep 24, 2021
@jpmcb jpmcb marked this pull request as draft September 24, 2021 01:06
@jpmcb
Copy link
Contributor

jpmcb commented Sep 24, 2021

Hi @nicolehanjing - thanks for this PR; I've converted it to a draft PR since it's still a work in progress. Please feel free to set it to "Ready for review" when it's ready! Thanks much!!

Copy link
Member

@XudongLiuHarold XudongLiuHarold left a comment

Choose a reason for hiding this comment

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

I think we can only desperate in 1.5.0 folder and keep 1.4.0 the same?

@nicolehanjing
Copy link
Contributor Author

I think we can only desperate in 1.5.0 folder and keep 1.4.0 the same?

gotcha, will update my changes

@nicolehanjing nicolehanjing force-pushed the topic/nicoleh/deprecate-imageinfo branch from 0083f31 to 3c06231 Compare October 7, 2021 21:55
@nicolehanjing nicolehanjing force-pushed the topic/nicoleh/deprecate-imageinfo branch from 3c06231 to e9f87d7 Compare October 7, 2021 21:56
@nicolehanjing nicolehanjing marked this pull request as ready for review October 7, 2021 21:57
Copy link
Member

@XudongLiuHarold XudongLiuHarold left a comment

Choose a reason for hiding this comment

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

LGTM

@XudongLiuHarold XudongLiuHarold merged commit 1722209 into main Oct 11, 2021
@stmcginnis stmcginnis deleted the topic/nicoleh/deprecate-imageinfo branch October 25, 2021 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required owner/packages Work executed by a package's maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants