-
Notifications
You must be signed in to change notification settings - Fork 328
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
feat(installer): support upgrade through installer #939
Conversation
5bd6655
to
60139c2
Compare
@@ -142,7 +142,7 @@ function clean_old_data() { | |||
function start_installer() { | |||
echo "Step.5 start tke-installer [doing]" | |||
|
|||
docker run $OPTIONS "tkestack/tke-installer-${ARCH}:$VERSION" | |||
docker run $OPTIONS "tkestack/tke-installer-${ARCH}:$VERSION" /app/bin/tke-installer $@ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think need duplicate /app/bin/tke-installer
again here
if len(depl.Spec.Template.Spec.Containers) == 0 { | ||
return fmt.Errorf("%s has no containers", com) | ||
} | ||
depl.Spec.Template.Spec.Containers[0].Image = images.Get().TKEPlatformController.FullName() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
depl.Spec.Template.Spec.InitContainers[0].Image
e909833
to
607a0ea
Compare
@leoryu please handle the tke.json already exist case and retest |
* feat(installer): support upgrade through installer * fix(installer): fix installer registry nil pointer * feat(platform): remove save third party registry * feat(platform): load tke data before upgrade * feat(platform): move load func to installer
* feat(installer): support upgrade through installer * fix(installer): fix installer registry nil pointer * feat(platform): remove save third party registry * feat(platform): load tke data before upgrade * feat(platform): move load func to installer
)" This reverts commit 2e1e884.
)" This reverts commit 2e1e884.
What type of PR is this?
/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: