Merge pull request openshift#2243 from spadgett/tags-op

Fix error adding first annotation
dperaza4dustbit · Apr 19, 2018 · 727af9e · 727af9e
2 parents ad18608 + 5f12be1
commit 727af9e
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/frontend/public/components/modals/tags.jsx b/frontend/public/components/modals/tags.jsx
@@ -41,7 +41,9 @@ class TagsModal extends PromiseComponent {
     // Convert any blank values to null
     _.each(tags, t => t[1] = _.isEmpty(t[1]) ? null : t[1]);
 
-    const patch = [{path: this.props.path, op: 'replace', value: _.fromPairs(tags)}];
+    // Make sure to 'add' if the path does not already exist, otherwise the patch request will fail
+    const op = this.props.tags ? 'replace' : 'add';
+    const patch = [{path: this.props.path, op, value: _.fromPairs(tags)}];
     const promise = k8sPatch(this.props.kind, this.props.resource, patch);
     this.handlePromise(promise).then(this.props.close);
   }