Skip to content

Commit a08f541

Browse files
authored
test(operator): fix editStatus to use fresh resource version (kroxylicious#3817)
The deploymentChecksumUpdatesWhenOpenshiftRouteHostAssigned test was using the stale bootstrapRoute variable instead of the fresh resource passed to the editStatus lambda. This prevented the retry mechanism from working correctly, causing 409 Conflict errors. Changes: - Use lambda parameter 'r' instead of stale 'bootstrapRoute' - Use editOrNewStatus() to preserve existing status fields Fixes the CI failure in: https://github.com/kroxylicious/kroxylicious/actions/runs/25044994969/job/73359160775 Assisted-by: Claude Sonnet 4.5 <noreply@anthropic.com> Signed-off-by: Keith Wall <kwall@apache.org>
1 parent 9985c19 commit a08f541

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

kroxylicious-operator/src/test/java/io/kroxylicious/kubernetes/operator/reconciler/kafkaproxy/KafkaProxyReconcilerIT.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -986,8 +986,8 @@ void deploymentChecksumUpdatesWhenOpenshiftRouteHostAssigned() {
986986
// controller will have updated the resource or not yet. We make an update to
987987
// the Route that we are certain will be different to that made by the controller
988988
// in order to be certain an update is made.
989-
testActor.resources(Route.class).resource(bootstrapRoute).editStatus(r -> new RouteBuilder(bootstrapRoute)
990-
.withNewStatus()
989+
testActor.resources(Route.class).resource(bootstrapRoute).editStatus(r -> new RouteBuilder(r)
990+
.editOrNewStatus()
991991
.addNewIngress().withHost("different.invalid").endIngress()
992992
.endStatus()
993993
.build());

0 commit comments

Comments
 (0)