-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
test/integration: improve TestTransform #108495
test/integration: improve TestTransform #108495
Conversation
/triage accepted |
/test pull-kubernetes-conformance-kind-ga-only-parallel |
Unknown CLA label state. Rechecking for CLA labels. Send feedback to sig-contributor-experience at kubernetes/community. /check-cla |
@sttts added a GoDoc comment |
44f0496
to
f4833c9
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: stevekuznetsov, sttts 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 |
/hold |
f4833c9
to
33fdbcf
Compare
33fdbcf
to
bdb8de3
Compare
Here's an investigation into the implicit logic going on with the parsing ... https://go.dev/play/p/0AEDExX7zwV |
ResourceVersion values are opaque and math should not be done on them. The intent of this test was to watch from a resourceVersion before the moment where the objects were created, and we can find such a version by listing from the API server before the tests begin. Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
Without such a context, this test would hang without data for 10 minutes, then panic. Signed-off-by: Steve Kuznetsov <skuznets@redhat.com>
bdb8de3
to
ff8fbc2
Compare
/hold cancel |
I mixed my |
/lgtm |
test/integration: use a known previous resourceVersion
ResourceVersion values are opaque and math should not be done on them.
The intent of this test was to watch from a resourceVersion before the
moment where the objects were created, and we can find such a version by
listing from the API server before the tests begin.
Signed-off-by: Steve Kuznetsov skuznets@redhat.com
test/integration: use a context with a deadline for streaming
Without such a context, this test would hang without data for 10
minutes, then panic.
Signed-off-by: Steve Kuznetsov skuznets@redhat.com
/kind cleanup
/cc @smarterclayton @liggitt @deads2k
/sig api-machinery