-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
⚠ Bring in testing framework #749
⚠ Bring in testing framework #749
Conversation
This is bringing in https://github.com/kubernetes-sigs/testing_frameworks/tree/694245ac7b60bff90cc8b159fbd8ddb7efe3f218. The package is fully integrated into the local tooling, e.g.: - updated import path - uses the same binary assets for testing - runs its tests as part of `make test` and on prow - updated OWNERS In the original repo the CLA bot was enabled, thus all commits that have been squashed here should be fine regarding the CLA. All authors and committers of the squashed commits have been added here as co-authors. Co-authored-by: Aaron Crickenberger <spiffxp@gmail.com> Co-authored-by: Gareth Smith <gsmith@pivotal.io> Co-authored-by: Hannes Hörl <hhorl@pivotal.io> Co-authored-by: Josh Hill <jhill@pivotal.io> Co-authored-by: Lantao Liu <lantaol@google.com> Co-authored-by: Maru Newby <marun@redhat.com> Co-authored-by: Mengqi Yu <mengqiy@google.com> Co-authored-by: Steven Chung <steve.chung7@gmail.com>
/lgtm I think we'll want to play around with this structure a bit eventually, but this is fine for now cc @mengqiy |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: DirectXMan12, hoegaarden The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
xref: #676 |
🏃 remove Dep manifests from KB repo
This is bringing in https://github.com/kubernetes-sigs/testing_frameworks/tree/694245ac7b60bff90cc8b159fbd8ddb7efe3f218.
The package is fully integrated into the local tooling, e.g.:
make test
and on prowIn the original repo the CLA bot was enabled, thus all commits that have been
squashed here should be fine regarding the CLA.
All authors and committers of the squashed commits have been added here as
co-authors.
Superseds #714 (see: #714 (comment))
/assign @DirectXMan12 @gerred