Preserve existing annotations in submitted pod (#3)

asaintsever · Nov 11, 2021 · 830a4a9 · 830a4a9
1 parent a33ebf9
commit 830a4a9
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 9 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@
 
 - [OVAI #2](https://github.com/asaintsever/open-vault-agent-injector/pull/2) - Add issuer on [vault k8s config](https://www.vaultproject.io/docs/auth/kubernetes#discovering-the-service-account-issuer) for Kubernetes 1.21+
 
+**Fixed**
+
+- [OVAI #3](https://github.com/asaintsever/open-vault-agent-injector/pull/3) - Preserve existing annotations in submitted pod
+
 ## Release v1.0.0 - 2021-09-21
 
 **Changed**

diff --git a/VERSION_CHART b/VERSION_CHART
@@ -1 +1 @@
-1.0.0
+1.0.1
diff --git a/VERSION_OVAI b/VERSION_OVAI
@@ -1 +1 @@
-1.0.0
+1.0.1
diff --git a/pkg/webhook/utils.go b/pkg/webhook/utils.go
@@ -119,18 +119,15 @@ Loop:
 func updateAnnotation(target map[string]string, added map[string]string) (patch []ctx.PatchOperation) {
 	for key, value := range added {
 		if target == nil || target[key] == "" {
-			target = map[string]string{}
 			patch = append(patch, ctx.PatchOperation{
-				Op:   ctx.JsonPatchOpAdd,
-				Path: ctx.JsonPathAnnotations,
-				Value: map[string]string{
-					key: value,
-				},
+				Op:    ctx.JsonPatchOpAdd,
+				Path:  ctx.JsonPathAnnotations + "/" + strings.Replace(key, "/", "~1", -1), // as per http://jsonpatch.com/#json-pointer
+				Value: value,
 			})
 		} else {
 			patch = append(patch, ctx.PatchOperation{
 				Op:    ctx.JsonPatchOpReplace,
-				Path:  ctx.JsonPathAnnotations + "/" + key,
+				Path:  ctx.JsonPathAnnotations + "/" + strings.Replace(key, "/", "~1", -1), // as per http://jsonpatch.com/#json-pointer
 				Value: value,
 			})
 		}