Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Flaking Test] [Conformance] [sig-apps] Job should apply changes to a job status #123799

Closed
dims opened this issue Mar 7, 2024 · 6 comments · Fixed by #123815
Closed

[Flaking Test] [Conformance] [sig-apps] Job should apply changes to a job status #123799

dims opened this issue Mar 7, 2024 · 6 comments · Fixed by #123815
Assignees
@mimowo
Labels
kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/apps Categorizes an issue or PR as relevant to SIG Apps.

Comments

@dims
Copy link
Member

dims commented Mar 7, 2024

image

Failure cluster f6abcbfa2725f7826845

Error text:
[FAILED] Job.batch "suspend-false-to-true" is invalid: status.startTime: Required value: startTime cannot be removed for unsuspended job
In [It] at: k8s.io/kubernetes/test/e2e/apps/job.go:867 @ 03/05/24 21:35:31.286

Recent failures:

3/7/2024, 5:46:18 AM ci-kubernetes-e2e-gci-gce-alpha-enabled-default
3/7/2024, 5:02:18 AM ci-kubernetes-e2e-gci-gce-alpha-enabled-default
3/7/2024, 4:20:19 AM ci-kubernetes-e2e-ec2-alpha-enabled-default
3/7/2024, 4:19:18 AM ci-kubernetes-e2e-gci-gce-alpha-enabled-default
3/7/2024, 3:37:18 AM ci-kubernetes-e2e-gci-gce-alpha-enabled-default

/kind failing-test

/sig apps
@k8s-ci-robot k8s-ci-robot added kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. sig/apps Categorizes an issue or PR as relevant to SIG Apps. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Mar 7, 2024
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

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/test-infra repository.

@dims
Copy link
Member Author

dims commented Mar 7, 2024

/assign @mimowo

suspecting #123273 as cause

@mimowo
Copy link
Contributor

mimowo commented Mar 7, 2024 •
edited

Ack, indeed, because this is the suite for alpha features enabled.

We have this fix ready for the test itself: #123751.

/cc @alculquicondor @soltysh @atiratree

@tenzen-y tenzen-y mentioned this issue Mar 7, 2024
Merged
3 tasks
@pacoxu
Copy link
Member

pacoxu commented Mar 8, 2024 •
edited

/cc @kubernetes/release-team-release-signal

It flakes in

@pacoxu
Copy link
Member

pacoxu commented Mar 8, 2024

/retitle [Flaking Test] [Conformance] [sig-apps] Job should apply changes to a job status

@k8s-ci-robot k8s-ci-robot changed the title Failure cluster [f6abcbfa...] Job should apply changes to a job status [Flaking Test] [Conformance] [sig-apps] Job should apply changes to a job status Mar 8, 2024
@mimowo mimowo mentioned this issue Mar 8, 2024
Merged
@mimowo
Copy link
Contributor

mimowo commented Mar 8, 2024

It used to fail with errors like

[FAILED] Job.batch "suspend-false-to-true" is invalid: status.startTime: Required value: startTime cannot be removed for unsuspended job

After #123751 it fails with :

[FAILED] patched object should have the applied condition with LastTransitionTime time.Date(2024, time.March, 7, 17, 58, 34, 0, time.UTC), got time.Date(2024, time.March, 7, 17, 58, 33, 0, time.Local) instead In [It] at: k8s.io/kubernetes/test/e2e/apps/job.go:901 @ 03/07/24 17:58:34.109

Here I open the follow up fix: #123815. PTAL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mimowo mimowo

Labels
kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. sig/apps Categorizes an issue or PR as relevant to SIG Apps.
Projects
CI Signal (SIG Release / Release Team)
Status: RESOLVED
SIG Apps
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Follow up fix to the job status update test mimowo/kubernetes
4 participants
@dims @pacoxu @mimowo @k8s-ci-robot