-
Notifications
You must be signed in to change notification settings - Fork 500
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
add kind cluster create script #791
Conversation
echo "init the mount dirs for kind cluster" | ||
mkdir -p ${data_dir}/worker{,1,2,3,4,5}/vol{1..9} | ||
|
||
cd ${env_dir} |
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.
Where is this variable defined?
docs/CONTRIBUTING.md
Outdated
$ ./hack/kind-cluster-v1.12.8.sh | ||
``` | ||
|
||
Then you can build and push Docker images to the public Docker registry (There have not docker registry in Kind Kubernetes environment currently) |
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.
Docker images can be loaded with kind load docker-image
or kind load image-archive
command.
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.
this can push images to all nodes, but if ImagePullPolicy
is Always
, the pod still can not get image from local. it can not resolve the image problem completely
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 think kind load
also load image to all nodes. For image always pull issue, can we use IfNotPresent
to replace Always
.
hack/kind-cluster-v1.12.8.sh
Outdated
- role: control-plane | ||
- role: worker | ||
extraPortMappings: | ||
- containerPort: 5001 |
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.
This port mapping is unused.
/run-e2e-in-kind |
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.
LGTM
/run-e2e-in-kind |
@cofyc @tennix @onlymellb PTAL again |
Co-Authored-By: Tennix <tennix@users.noreply.github.com>
Co-Authored-By: Tennix <tennix@users.noreply.github.com>
Co-Authored-By: Tennix <tennix@users.noreply.github.com>
Co-Authored-By: Tennix <tennix@users.noreply.github.com>
Co-Authored-By: Tennix <tennix@users.noreply.github.com>
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.
LGTM
* zh: add 1.1-ga chinese release notes Signed-off-by: Ran <huangran@pingcap.com> * Apply suggestions from code review Co-authored-by: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com> Co-authored-by: DanielZhangQD <36026334+DanielZhangQD@users.noreply.github.com>
What problem does this PR solve?
add scripts to create kind cluster env
What is changed and how does it work?
Check List
Tests
Code changes
Related changes
Does this PR introduce a user-facing change?: