Closed helen-frank closed 1 year ago
swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-controller-manager:v1.3.0
swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-scheduler:v1.3.0
swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-webhook:v1.3.0
swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-aggregated-apiserver:v1.3.0
docker.io/alpine:3.15.1
registry.cn-hangzhou.aliyuncs.com/google_containers/etcd:3.5.3-0
registry.cn-hangzhou.aliyuncs.com/google_containers/kube-apiserver:v1.25.2
registry.cn-hangzhou.aliyuncs.com/google_containers/kube-controller-manager:v1.25.2
Currently testing image-registry
needs to push these images into the image repository.
swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-controller-manager:v1.3.0 swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-scheduler:v1.3.0 swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-webhook:v1.3.0 swr.ap-southeast-1.myhuaweicloud.com/karmada/karmada-aggregated-apiserver:v1.3.0 docker.io/alpine:3.15.1 registry.cn-hangzhou.aliyuncs.com/google_containers/etcd:3.5.3-0 registry.cn-hangzhou.aliyuncs.com/google_containers/kube-apiserver:v1.25.2 registry.cn-hangzhou.aliyuncs.com/google_containers/kube-controller-manager:v1.25.2
Currently testing
image-registry
needs to push these images into the image repository.
I will develop image-tag
after this pr (tentative name)
For quick testing, I have provided a mirror warehouse here.
./karmadactl init --image-registry=daocloud.io/atsctoo
Thanks for your contribute. I will review it!
/assign
export karmada_registry="harbor.xxx.xx/xxx"
karmadactl init --etcd-image=$karmada_registry/etcd:3.5.3-0 \
--etcd-init-image=$karmada_registry/alpine:3.15.1 \
--karmada-aggregated-apiserver-image=$karmada_registry/karmada-aggregated-apiserver:v1.3.1 \
--karmada-apiserver-image=$karmada_registry/kube-apiserver:v1.18.20 \
--karmada-controller-manager-image=$karmada_registry/karmada-controller-manager:v1.3.1 \
--karmada-kube-controller-manager-image=$karmada_registry/kube-controller-manager:v1.18.20 \
--karmada-scheduler-image=$karmada_registry/karmada-scheduler:v1.3.1 \
--karmada-webhook-image=$karmada_registry/karmada-webhook:v1.3.1 \
--crds=./crds.tar.gz
In yesterday's document, I need to give the command of karmada offline installation, which makes me a little painful.🥲 /cc @RainbowMango
/cc @RainbowMango
@lonelyCZ Do you have any other comments?
Looks good, I will test it in my env today.
Hi, @helen-frank could you please help to resolve the conflicts? I will test it ASAP.
resolve conflicts and test @lonelyCZ
./karmadactl init --image-registry=daocloud.io/atsctoo @lonelyCZ
Thanks @helen-frank , it works fine in my env.
/lgtm
/assign
By the way, I see the words from PR description that seems this PR is part of #2541, and the comment at https://github.com/karmada-io/karmada/pull/2655#issuecomment-1282710770.
What's another flag look like?
virtual-image-tag
or private-image-tag
? @RainbowMango
By the way, I see the words from PR description that seems this PR is part of #2541, and the comment at #2655 (comment).
What's another flag look like?
I suddenly realized that users may not care about the versions of karmada and etcd. I think the scope of image-tag can be reduced to kube. More compatibility issues are on kube.
What do you think of kube-image-tag
? @RainbowMango
Private image registry. Choose a container registry to pull control plane images from. Including all images required to karmada init, recommended for use when deploying karmada in privatized environments. This is a minimum priority parameter. For example, you can use kube-image-registry=registry.cn-hangzh to change the image related to kube
The description has some long ... @RainbowMango
Generally looks good to me. But I didn't test it on my side. Leave approval to @lonelyCZ
It works fine in my env. :)
Thanks @helen-frank
/lgtm /approve
And please update Does this PR introduce a user-facing change?
in this PR.
/retitle introduce --private-image-registry
flag to karmadactl init
In addition to the release notes that @lonelyCZ asked for above, please also update the PR descriptions as per the latest content.
OK, I rested earlier yesterday and updated it immediately.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: lonelyCZ
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
What type of PR is this? /kind feature
What this PR does / why we need it: At present, it is mainly used for offline installation karmada to simplify installation commands. Which issue(s) this PR fixes: Fixes # Port of #2541 Special notes for your reviewer: Test
Does this PR introduce a user-facing change?: