-
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
Clean up some of the reconcilation testing infrastructure #1977
Clean up some of the reconcilation testing infrastructure #1977
Conversation
dprotaso
commented
Sep 5, 2018
- extract out a generic object sorter from a specific lister
- move the API typed lister to a separate subpackage under v1alpha1/testing
- clean up readability of the API for extracting actions
- use go aliasing help with readability now that we have two packages
5accfc5
to
584d94a
Compare
- extract out a generic object sorter from a specific lister - move the API typed lister to a separate subpackage under v1alpha1/testing - clean up readability of the API for extracting actions - use go aliasing help with readability now that we have two packages
584d94a
to
4d62c45
Compare
/test all |
/test pull-knative-serving-integration-tests |
cc @jonjohnsonjr Who was thinking of a similar split of some logic into /lgtm Thanks, @dprotaso I definitely think this cleans things up. I look forward to more :) |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dprotaso, mattmoor 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 |
/test pull-knative-serving-integration-tests |
1 similar comment
/test pull-knative-serving-integration-tests |
bumping knative.dev/hack 4ed1ecb...ea9c9a0: > ea9c9a0 Add hermetic go mod aware handling to run_go_tool (knative#38) bumping knative.dev/pkg 51c72e2...93874f0: > 93874f0 Drainer supports customized kubelet probe check (knative#1977) bumping knative.dev/networking ecb89ce...c30dc94: > c30dc94 Use symlink for testdata version of config-network.yaml (knative#328) Signed-off-by: Knative Automation <automation@knative.team>
``` bumping knative.dev/hack 4ed1ecb...ea9c9a0: > ea9c9a0 Add hermetic go mod aware handling to run_go_tool (knative#38) bumping knative.dev/networking ecb89ce...c30dc94: > c30dc94 Use symlink for testdata version of config-network.yaml (knative#328) bumping knative.dev/pkg 51c72e2...93874f0: > 93874f0 Drainer supports customized kubelet probe check (knative#1977) ``` Signed-off-by: Knative Automation <automation@knative.team>
bumping knative.dev/hack 4ed1ecb...ea9c9a0: > ea9c9a0 Add hermetic go mod aware handling to run_go_tool (knative#38) bumping knative.dev/networking ecb89ce...3a89509: > 3a89509 As per discussion with @mattmoor keep drainer in pkg (knative#329) > c30dc94 Use symlink for testdata version of config-network.yaml (knative#328) bumping knative.dev/pkg 51c72e2...93874f0: > 93874f0 Drainer supports customized kubelet probe check (knative#1977) Signed-off-by: Knative Automation <automation@knative.team>