25125400821
44%

Ran 29 Apr 2026 06:03PM UTC

Jobs 1

Files 20

Run time 1min

Badge

Embed ▾

Committed 29 Apr 2026 06:01PM UTC coverage: 44.419%. Remained the same

Build # 25125400821

Build Type

push

github

Committed by

web-flow

Commit Message

fix: avoid 409 Conflict races in integration test status updates (#832)

The 'Test default progressing state' integration test misuses
gomega.Eventually by passing the result of k8sClient.Status().Update
directly, so Update is only invoked once and Eventually keeps
re-checking the same already-resolved error. When the reconciler
updates the LWS status concurrently, the single attempt hits a 409
Conflict and the test times out after 120s.

The SetStatefulsetToUnReady test helper has the same class of bug:
the caller does a Get first and then this helper issues a single
Status().Update on the (potentially already-stale) object, so any
concurrent reconciler write produces a 409.

Wrap both updates in an Eventually polling closure that re-Gets the
latest object on every retry and re-issues Status().Update, matching
the canonical pattern used in kueue and jobset integration tests.

Signed-off-by: Kay Yan <kay.yan@daocloud.io>

Coverage Stats

1182 of 2661 relevant lines covered (44.42%)

2.99 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	25125400821.1	29 Apr 2026 06:03PM UTC	20	44.42	GitHub Action Run

kubernetes-sigs / lws / 25125400821
44%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 25125400821

kubernetes-sigs / lws / 25125400821 44%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 25125400821

kubernetes-sigs / lws / 25125400821
44%

README BADGES
x