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

ImageOverrider Implementation #370

Merged
merged 2 commits into from
May 26, 2021

Conversation

XiShanYongYe-Chang
Copy link
Member

@XiShanYongYe-Chang XiShanYongYe-Chang commented May 24, 2021

What type of PR is this?

/kind feature

What this PR does / why we need it:

ImageOverrider implementation

Which issue(s) this PR fixes:
Fixes #313

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


@XiShanYongYe-Chang XiShanYongYe-Chang force-pushed the imageoverride branch 3 times, most recently from 9321764 to a0a1060 Compare May 24, 2021 08:34
@RainbowMango
Copy link
Member

cc me when ready for review.

@XiShanYongYe-Chang XiShanYongYe-Chang force-pushed the imageoverride branch 4 times, most recently from e17c3b3 to 3d47f62 Compare May 25, 2021 02:26
@XiShanYongYe-Chang XiShanYongYe-Chang force-pushed the imageoverride branch 2 times, most recently from 9dd83ed to 25ad37d Compare May 25, 2021 04:21
@XiShanYongYe-Chang
Copy link
Member Author

I've changed a lot, can you have a look, thanks. @RainbowMango

@XiShanYongYe-Chang XiShanYongYe-Chang force-pushed the imageoverride branch 3 times, most recently from d3fc4a4 to ae29c06 Compare May 25, 2021 13:38
@RainbowMango RainbowMango self-assigned this May 26, 2021
@RainbowMango RainbowMango requested a review from mrlihanbo May 26, 2021 03:54
pkg/util/overridemanager/overridemanager.go Outdated Show resolved Hide resolved
pkg/util/overridemanager/overridemanager.go Outdated Show resolved Hide resolved
pkg/util/overridemanager/overridemanager.go Outdated Show resolved Hide resolved
pkg/util/overridemanager/imageoverrider/imageoverride.go Outdated Show resolved Hide resolved
pkg/util/overridemanager/imageoverrider/imageoverride.go Outdated Show resolved Hide resolved
@XiShanYongYe-Chang XiShanYongYe-Chang force-pushed the imageoverride branch 2 times, most recently from 2ba2a61 to 50fc9af Compare May 26, 2021 09:17
Signed-off-by: changzhen <changzhen5@huawei.com>
Signed-off-by: changzhen <changzhen5@huawei.com>
@RainbowMango RainbowMango merged commit 4ae750b into karmada-io:master May 26, 2021
@XiShanYongYe-Chang XiShanYongYe-Chang deleted the imageoverride branch May 27, 2021 03:06
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.

ImageOverrider Implementation
2 participants