From 7c0057bb4cf660779ad69d69306f4503e3d69747 Mon Sep 17 00:00:00 2001 From: Yifan Gu Date: Mon, 14 May 2018 16:40:49 -0700 Subject: [PATCH] tectonic-alm-operator: Use command instead of args. Since TCO is using 2-way merge for the containers in the manifest, using `args` alone will not remove the old deployment's `command`, which will causes error. --- .../updater/operators/tectonic-alm-operator.yaml | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/modules/tectonic/resources/manifests/updater/operators/tectonic-alm-operator.yaml b/modules/tectonic/resources/manifests/updater/operators/tectonic-alm-operator.yaml index 64dd459b20..6ecc5fa8ac 100644 --- a/modules/tectonic/resources/manifests/updater/operators/tectonic-alm-operator.yaml +++ b/modules/tectonic/resources/manifests/updater/operators/tectonic-alm-operator.yaml @@ -27,8 +27,9 @@ spec: containers: - name: tectonic-alm-operator image: ${tectonic_alm_operator_image} - args: - - --manifest-dir=/manifests - - --operator-name=tectonic-alm-operator - - --appversion-name=tectonic-alm-operator - - --v=2 + command: + - /app/x-operator/cmd/xoperator/xoperator + - '--operator-name=tectonic-alm-operator' + - '--appversion-name=tectonic-alm-operator' + - '--v=2' + - '-manifest-dir=/manifests'