Skip to content

Commit

Permalink
demote kubelet events test from Conformance
Browse files Browse the repository at this point in the history
This test has been part of the Conformance suite since at least
Kubernetes 1.2 (2015-10-xx). Some years later, around 2018-10-xx, we
drafted a rigorous set of rules for tests to follow in order to be
eligible for promotion to Conformance. We explicitly disallowed any
tests that check for specific Events, since they are not an API, and we
make no guarantees about their contents nor their delivery.

Unfortunately, we neglected to go through the existing corpus of
Conformance tests with a fine-toothed comb after drafting these rules.
The very nature of what this test is attempting to exercise and verify
is specific Events, and their delivery, thus making it ineligible for
Conformance. We should have caught and demoted this test back then.

Better late than never?
  • Loading branch information
spiffxp committed Nov 17, 2021
1 parent 7f75641 commit 357ad91
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 16 deletions.
10 changes: 0 additions & 10 deletions test/conformance/testdata/conformance.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1788,16 +1788,6 @@
visible at runtime in the container.
release: v1.9
file: test/e2e/common/node/downwardapi.go
- testname: Pod events, verify event from Scheduler and Kubelet
codename: '[sig-node] Events should be sent by kubelets and the scheduler about
pods scheduling and running [Conformance]'
description: Create a Pod, make sure that the Pod can be queried. Create a event
selector for the kind=Pod and the source is the Scheduler. List of the events
MUST be at least one. Create a event selector for kind=Pod and the source is the
Kubelet. List of the events MUST be at least one. Both Scheduler and Kubelet MUST
send events when scheduling and running a Pod.
release: v1.9
file: test/e2e/node/events.go
- testname: init-container-starts-app-restartalways-pod
codename: '[sig-node] InitContainer [NodeConformance] should invoke init containers
on a RestartAlways pod [Conformance]'
Expand Down
7 changes: 1 addition & 6 deletions test/e2e/node/events.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,7 @@ import (
var _ = SIGDescribe("Events", func() {
f := framework.NewDefaultFramework("events")

/*
Release: v1.9
Testname: Pod events, verify event from Scheduler and Kubelet
Description: Create a Pod, make sure that the Pod can be queried. Create a event selector for the kind=Pod and the source is the Scheduler. List of the events MUST be at least one. Create a event selector for kind=Pod and the source is the Kubelet. List of the events MUST be at least one. Both Scheduler and Kubelet MUST send events when scheduling and running a Pod.
*/
framework.ConformanceIt("should be sent by kubelets and the scheduler about pods scheduling and running ", func() {
ginkgo.It("should be sent by kubelets and the scheduler about pods scheduling and running ", func() {

podClient := f.ClientSet.CoreV1().Pods(f.Namespace.Name)

Expand Down

0 comments on commit 357ad91

Please sign in to comment.