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

add kind cluster create script #791

Merged
merged 33 commits into from
Sep 1, 2019
Merged

Conversation

xiaojingchen
Copy link
Contributor

@xiaojingchen xiaojingchen commented Aug 20, 2019

What problem does this PR solve?

add scripts to create kind cluster env

What is changed and how does it work?

Check List

Tests

  • Manual test (add detailed scripts or steps below)
  • No code

Code changes

  • Has documents change

Related changes

  • Need to update the documentation

Does this PR introduce a user-facing change?:

Support using Kind (Kubernetes IN Docker) to set up a testing environment.

echo "init the mount dirs for kind cluster"
mkdir -p ${data_dir}/worker{,1,2,3,4,5}/vol{1..9}

cd ${env_dir}
Copy link
Contributor

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?

@xiaojingchen xiaojingchen changed the title [WIP] add kind cluster create script add kind cluster create script Aug 22, 2019
hack/kind-cluster-v1.12.8.sh Outdated Show resolved Hide resolved
hack/kind-cluster-v1.12.8.sh Outdated Show resolved Hide resolved
$ ./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)
Copy link
Member

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.

Copy link
Contributor Author

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

Copy link
Member

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.

- role: control-plane
- role: worker
extraPortMappings:
- containerPort: 5001
Copy link
Member

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.

@tennix tennix added the test/ci CI scripts label Aug 27, 2019
@cofyc
Copy link
Contributor

cofyc commented Aug 27, 2019

/run-e2e-in-kind

docs/CONTRIBUTING.md Outdated Show resolved Hide resolved
hack/kind-cluster-v1.12.8.sh Outdated Show resolved Hide resolved
hack/kind-cluster-v1.12.8.sh Outdated Show resolved Hide resolved
hack/kind-cluster-v1.12.8.sh Outdated Show resolved Hide resolved
cofyc
cofyc previously approved these changes Aug 28, 2019
Copy link
Contributor

@cofyc cofyc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

cofyc
cofyc previously approved these changes Aug 28, 2019
cofyc
cofyc previously approved these changes Aug 30, 2019
@xiaojingchen
Copy link
Contributor Author

/run-e2e-in-kind

@xiaojingchen
Copy link
Contributor Author

@cofyc @tennix @onlymellb PTAL again

hack/kind-cluster-build.sh Outdated Show resolved Hide resolved
hack/kind-cluster-build.sh Outdated Show resolved Hide resolved
docs/CONTRIBUTING.md Outdated Show resolved Hide resolved
docs/CONTRIBUTING.md Outdated Show resolved Hide resolved
docs/CONTRIBUTING.md Outdated Show resolved Hide resolved
docs/CONTRIBUTING.md Outdated Show resolved Hide resolved
docs/CONTRIBUTING.md Outdated Show resolved Hide resolved
xiaojingchen and others added 7 commits September 1, 2019 13:02
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>
Copy link
Member

@tennix tennix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tennix tennix merged commit 2dfa097 into pingcap:master Sep 1, 2019
@cofyc cofyc mentioned this pull request Sep 2, 2019
yahonda pushed a commit that referenced this pull request Dec 27, 2021
* 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test/ci CI scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants