Skip to content
This repository has been archived by the owner on Feb 5, 2020. It is now read-only.

Commit

Permalink
tectonic-alm-operator: Use command instead of args.
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Yifan Gu committed May 15, 2018
1 parent 9cdb755 commit 7c0057b
Showing 1 changed file with 6 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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'

0 comments on commit 7c0057b

Please sign in to comment.