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
introduce --private-image-registry
flag to karmadactl init
#2655
introduce --private-image-registry
flag to karmadactl init
#2655
Conversation
e81a163
to
aa600c3
Compare
Currently testing |
I will develop |
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 |
pkg/karmadactl/cmdinit/cmdinit.go
Outdated
releaseVer = &version.ReleaseVersion{} // initialize to avoid panic | ||
} | ||
// image registry | ||
flags.StringVarP(&opts.ImageRegistry, "image-registry", "", "", "Image registry.") |
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.
We could add more detail description for image-registry
.
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.
Image registry. Registry containing karmada all image
Whether such a description is readable ?
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.
Perhaps, you can look up and refer other projects. Then we can compare and determine a good description.
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.
Choose a container registry to pull control plane images from
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.
It looks good.
aa600c3
to
06fa84b
Compare
06fa84b
to
004374a
Compare
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 |
@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. |
06b0287
to
5e25398
Compare
Generally looks good to me. But I didn't test it on my side. |
It works fine in my env. :) Thanks @helen-frank /lgtm And please update |
/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. |
Signed-off-by: helen <haitao.zhang@daocloud.io>
5e25398
to
d4f2104
Compare
[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
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/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?: