github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-20 02:11:09 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #115030 from kannon92/remove-pod-error-job-tracking

Browse Source 
Update SyncJob with PodControllerError updates in job unit tests
This commit is contained in:
Kubernetes Prow Robot 2023-01-13 12:08:14 -08:00 committed by GitHub
commit 9af5ae0365
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
pkg/controller/job/job_controller_test.go
View File

@ -211,6 +211,10 @@ func TestControllerSyncJob(t *testing.T) {
initialStatus *jobInitialStatus
// pod setup
// If a podControllerError is set, finalizers are not able to be removed.
// This means that there is no status update so the counters for
// failedPods and succeededPods cannot be incremented.
podControllerError error
jobKeyForget bool
pendingPods int
@ -318,12 +322,13 @@ func TestControllerSyncJob(t *testing.T) {
completions: 5,
backoffLimit: 6,
podControllerError: fmt.Errorf("fake error"),
jobKeyForget: true,
jobKeyForget: false,
activePods: 1,
succeededPods: 1,
expectedCreations: 1,
expectedActive: 1,
expectedSucceeded: 1,
expectedSucceeded: 0,
expectedPodPatches: 1,
},
"too many active pods": {
parallelism: 2,
@ -342,7 +347,8 @@ func TestControllerSyncJob(t *testing.T) {
podControllerError: fmt.Errorf("fake error"),
jobKeyForget: false,
activePods: 3,
expectedDeletions: 1,
expectedDeletions: 0,
expectedPodPatches: 1,
expectedActive: 3,
},
"failed + succeed pods: reset backoff delay": {
@ -391,7 +397,8 @@ func TestControllerSyncJob(t *testing.T) {
failedPods: 1,
expectedCreations: 1,
expectedActive: 1,
expectedFailed: 1,
expectedFailed: 0,
expectedPodPatches: 1,
},
"job finish": {
parallelism: 2,
@ -737,9 +744,6 @@ func TestControllerSyncJob(t *testing.T) {
for name, tc := range testCases {
t.Run(name, func(t *testing.T) {
if tc.podControllerError != nil {
t.Skip("Can't track status if finalizers can't be removed")
}
defer featuregatetesting.SetFeatureGateDuringTest(t, feature.DefaultFeatureGate, features.JobReadyPods, tc.jobReadyPodsEnabled)()
// job manager setup