Test k8s with node-os-distro #2265
Conversation
Adding SharedCounters broke the test and wasn't detected in the presubmit because the test is slow and didn't run. We could add a "SharedCounters is empty" clause, but probably it is safer to be less strict in the test and tolerate unknown fields. This will also minimize the work which needs to be done in the v1beta2 PR.
Fix the logic on TestGetStaticPodToMirrorPodMap Add back the length check logic
Serve LISTs with exact RV and continuations from cache
following user namespaces Signed-off-by: Peter Hunt <[email protected]>
…isn't supported and the pod requests a user namespace Signed-off-by: Peter Hunt <[email protected]>
KEP-3094: Promote NodeInclusionPolicyInPodTopologySpread to Stable
…ceslices-shared-counters DRA E2E: fix "cluster must manage ResourceSlices" test
Split out a helper to assert correct EndpointSlice hints (and extend that helper to deal with node hints as well, including making sure they *aren't* present when the traffic distribution mode doesn't call for them).
…on_filter_NodeResources chore(scheduler): add filter integration tests for missing part plugins: NodeResources plugin
…nhost [e2e/node] update base image from busybox to agnhost
…fault [KEP-4265]: features: bump proc mount to on by default beta
There was one error path that led to a "controller has shut down" log message. Other errors caused different log entries or are so unlikely (event handler registration failure!) that they weren't checked at all. It's clearer to let Run return an error in all cases and then log the "controller has shut down" error at the call site. This also enables tests to mark themselves as failed, should that ever happen.
Events get recorded in the apiserver asynchronously, so even if the test knows that the event has been evicted because the pod is deleted, it still has to also check for the event to be recorded. This caused a flake in the "Consistently" check of events.
openshift-ci-robot
added
the
backports/unvalidated-commits
|
/test k8s-e2e-gcp-ovn |
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: jsafrane The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/test k8s-e2e-gcp-ovn |
|
/test k8s-e2e-gcp-ovn |
|
/test k8s-e2e-gcp-ovn |
|
/test k8s-e2e-gcp-ovn |
|
@jsafrane: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Add
--node-os-distroto Kubernetese2e.test.Based on #2148