From d505e4bec69ded896ea5963c308e5bed8c581d89 Mon Sep 17 00:00:00 2001 From: iiiceoo Date: Wed, 31 May 2023 18:11:30 +0800 Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20fakeClient.Status().Update(...)?= =?UTF-8?q?=20cannot=20recognize=20resource=20version=20conflicts?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The fake client of subresource is unable to correctly handle the case of resource version conflict when updating. The phenomenon is that it did not return a 409 status error. Close #2362 Signed-off-by: iiiceoo --- pkg/client/fake/client.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/client/fake/client.go b/pkg/client/fake/client.go index 7167c5505e..aaedac8440 100644 --- a/pkg/client/fake/client.go +++ b/pkg/client/fake/client.go @@ -992,11 +992,11 @@ func copyNonStatusFrom(old, new runtime.Object) error { } } - newClientObject.SetResourceVersion(rv) - if err := fromMapStringAny(newMapStringAny, new); err != nil { return fmt.Errorf("failed to convert back from map[string]any: %w", err) } + newClientObject.SetResourceVersion(rv) + return nil }