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

Adds Kubernetes cluster tests #731

Merged
merged 1 commit into from
Aug 4, 2017
Merged

Conversation

cdrage
Copy link
Member

@cdrage cdrage commented Jul 28, 2017

This adds cluster tests for "kompose up" and "kompose down" for
Kubernetes. At the moment this checks that they are deployable to a
single-node Kubernetes cluster.

More proficient tests such as testing if the pods are actually up will
be added in the future.

You can test this by running: make test-ci on your local-machine.

Furthermore, we'll eventually have this enabled on Fabric8 / CentOS CI /
Semaphore (whatever comes first) so we can have a full end-to-end
testing environment.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 28, 2017
@kompose-bot
Copy link
Collaborator

@cdrage, thank you for the pull request! We'll ping some people to review your PR. @kadel and @ashetty1, please review this.

This adds cluster tests for "kompose up" and "kompose down" for
Kubernetes. At the moment this checks that they are deployable to a
single-node Kubernetes cluster.

More proficient tests such as testing if the pods are actually up will
be added in the future.

You can test this by running: `make test-ci` on your local-machine.

Furthermore, we'll eventually have this enabled on Fabric8 / CentOS CI /
Semaphore (whatever comes first) so we can have a full end-to-end
testing environment.
@cdrage
Copy link
Member Author

cdrage commented Aug 2, 2017

@surajnarwade @surajssd @ashetty1 @kadel

Please review

@ashetty1
Copy link
Contributor

ashetty1 commented Aug 3, 2017

@cdrage when I run examples/docker-gitlab.yaml, it throws up volume mount related warnings; the pods are not brought up successfully. How about we add --emptyvols for kompose up command?

Also, you aren't checking if the pods are actually running post kompose up. Is that how you want it to be?

@cdrage
Copy link
Member Author

cdrage commented Aug 3, 2017

@ashetty1 That kind of sucks that our current gitlab example doesn't work... I don't like how we have to pass in --emptyvols.. I think we should update our examples.

Yup, we are currently not checking if each pod are actually running post kompose up. I'll be adding that implementaiton later. Right now, it's getting make test-ci into Semaphore / some sort of CI.

Can you LGTM?

Copy link
Contributor

@ashetty1 ashetty1 left a comment

Choose a reason for hiding this comment

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

LGTM

@cdrage cdrage merged commit 3b59d9b into kubernetes:master Aug 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants