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 tests for deploy key specifically #727

Closed
wants to merge 1 commit into from

Conversation

cdrage
Copy link
Member

@cdrage cdrage commented Jul 26, 2017

Due to:
9dcb2bf
tests for "replicas" was disabled due to generating "pod-only"
kubernetes artifacts as per "restart_policy" being added.

This PR adds "deploy" tests so we can effectively test replicas

@kompose-bot
Copy link
Collaborator

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

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 26, 2017
@cdrage
Copy link
Member Author

cdrage commented Aug 2, 2017

I'm going to wait until #738 is merged in before merging this one in since this will create a conflict.

@cdrage
Copy link
Member Author

cdrage commented Aug 14, 2017

#738 is merged in. Let's wait for these tests to pass and then merge this in.

Due to:
kubernetes@9dcb2bf
tests for "replicas" was disabled due to generating "pod-only"
kubernetes artifacts as per "restart_policy" being added.

This PR adds "deploy" tests so we can effectively test `replicas`
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