Follow up fix to the job status update test

This commit is contained in:
Michal Wozniak 2024-03-08 06:44:21 +01:00
parent 364ef335db
commit acf6b500ee
2 changed files with 8 additions and 9 deletions

View File

@ -1049,10 +1049,9 @@
- testname: Jobs, apply changes to status - testname: Jobs, apply changes to status
codename: '[sig-apps] Job should apply changes to a job status [Conformance]' codename: '[sig-apps] Job should apply changes to a job status [Conformance]'
description: Attempt to create a running Job which MUST succeed. Attempt to patch description: Attempt to create a running Job which MUST succeed. Attempt to patch
the Job status to include a new start time which MUST succeed. An annotation for the Job status which MUST succeed. An annotation for the job that was patched
the job that was patched MUST be found. Attempt to replace the job status with MUST be found. Attempt to replace the job status with update which MUST succeed.
a new start time which MUST succeed. Attempt to read its status sub-resource which Attempt to read its status sub-resource which MUST succeed
MUST succeed
release: v1.24 release: v1.24
file: test/e2e/apps/job.go file: test/e2e/apps/job.go
- testname: Ensure Pods of an Indexed Job get a unique index. - testname: Ensure Pods of an Indexed Job get a unique index.

View File

@ -825,10 +825,10 @@ done`}
Release: v1.24 Release: v1.24
Testname: Jobs, apply changes to status Testname: Jobs, apply changes to status
Description: Attempt to create a running Job which MUST succeed. Description: Attempt to create a running Job which MUST succeed.
Attempt to patch the Job status to include a new start time which Attempt to patch the Job status which MUST succeed.
MUST succeed. An annotation for the job that was patched MUST be found. An annotation for the job that was patched MUST be found.
Attempt to replace the job status with a new start time which MUST Attempt to replace the job status with update which MUST succeed.
succeed. Attempt to read its status sub-resource which MUST succeed Attempt to read its status sub-resource which MUST succeed
*/ */
framework.ConformanceIt("should apply changes to a job status", func(ctx context.Context) { framework.ConformanceIt("should apply changes to a job status", func(ctx context.Context) {
@ -887,7 +887,7 @@ done`}
if err != nil { if err != nil {
return err return err
} }
if condition := findConditionByType(patchedStatus.Status.Conditions, customConditionType); condition != nil { if condition := findConditionByType(statusToUpdate.Status.Conditions, customConditionType); condition != nil {
condition.LastTransitionTime = now2 condition.LastTransitionTime = now2
} else { } else {
framework.Failf("patched object does not have the required condition %v", customConditionType) framework.Failf("patched object does not have the required condition %v", customConditionType)