fix: k8schaos: automatically include metadata.resourceVersion when up…

…dating existing resources.
form3tech-oss · Nov 21, 2023 · bb64338 · bb64338
1 parent 9c1b15f
commit bb64338
Showing 1 changed file with 15 additions and 3 deletions.
diff --git a/controllers/chaosimpl/k8schaos/impl.go b/controllers/chaosimpl/k8schaos/impl.go
@@ -91,9 +91,21 @@ func (impl *Impl) Apply(ctx context.Context, index int, records []*v1alpha1.Reco
 			return v1alpha1.NotInjected, err
 		}
 
-		unstructured.RemoveNestedField(impl.initialValue.Object, "metadata", "creationTimestamp")
-		unstructured.RemoveNestedField(impl.initialValue.Object, "metadata", "resourceVersion")
-		unstructured.RemoveNestedField(impl.initialValue.Object, "metadata", "uid")
+		if impl.initialValue != nil {
+			var resourceVersion string
+			var found bool
+			resourceVersion, found, err = unstructured.NestedString(impl.initialValue.Object, "metadata", "resourceVersion")
+			if err != nil {
+				return v1alpha1.NotInjected, err
+			}
+			if found {
+				resource.SetResourceVersion(resourceVersion)
+			}
+
+			unstructured.RemoveNestedField(impl.initialValue.Object, "metadata", "creationTimestamp")
+			unstructured.RemoveNestedField(impl.initialValue.Object, "metadata", "resourceVersion")
+			unstructured.RemoveNestedField(impl.initialValue.Object, "metadata", "uid")
+		}
 
 		_, err = client.Resource(mapping.Resource).Namespace(resource.GetNamespace()).Update(ctx, resource, v1.UpdateOptions{})
 	} else {