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

Update santiy test to use ginkgo. #8

Merged
merged 1 commit into from
Sep 26, 2018

Conversation

leakingtapan
Copy link
Contributor

@leakingtapan leakingtapan commented Sep 17, 2018

Fix #9

This have several benefits that:

  1. Using consistent testing framework as e2e test
  2. Able to define multiple test configurations per:
    https://github.com/kubernetes-csi/csi-test/tree/master/pkg/sanity
  3. Test results will be print to stdout as test runs as opposes to stuck
    for a while and dump all results at once in the old way

Tested using make test-sanity and tests are passing.

This have several benefits that:
1. Using consistent testing framework as e2e test
2. Able to define multiple test configurations per:
https://github.com/kubernetes-csi/csi-test/tree/master/pkg/sanity
3. Test results will be print to stdout as test runs as opposes to stuck
for a while and dump all results at once in the old way
@leakingtapan
Copy link
Contributor Author

/assign @bertinatto

@d-nishi
Copy link
Contributor

d-nishi commented Sep 25, 2018

@jsafrane @gnufied --- can you please review this?

@jsafrane
Copy link
Contributor

/lgtm

@jsafrane
Copy link
Contributor

Do we have bots running here? Merging manually.

@jsafrane jsafrane merged commit 43efd7c into kubernetes-sigs:master Sep 26, 2018
@leakingtapan leakingtapan deleted the ginkgo-test-sanity branch September 26, 2018 15:56
@leakingtapan leakingtapan mentioned this pull request Sep 26, 2018
@d-nishi
Copy link
Contributor

d-nishi commented Sep 27, 2018

@leakingtapan - this will take effect when we move to kubernetes-sigs - I should send the email tomorrow. Closing on Fossabadge issue on licensing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants