Skip to content
This repository was archived by the owner on Aug 2, 2019. It is now read-only.

Commit 020023f

Browse files
committed
Using cat is bad without proper yaml separators
A tad slower, but at least we shouldn't miss any.
1 parent 3231d3e commit 020023f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

etc/scripts/install-functions.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ function install_olm {
3030
rm -rf "$OLM_DIR"
3131
git clone https://github.com/operator-framework/operator-lifecycle-manager "$OLM_DIR"
3232
for i in "$OLM_DIR"/deploy/okd/manifests/latest/*.crd.yaml; do oc apply -f $i; done
33-
find "$OLM_DIR/deploy/okd/manifests/latest/" -type f ! -name "*crd.yaml" | sort | xargs cat | oc create -f -
33+
for i in $(find "$OLM_DIR/deploy/okd/manifests/latest/" -type f ! -name "*crd.yaml" | sort); do oc create -f $i; done
3434
wait_for_all_pods openshift-operator-lifecycle-manager
3535
# perms required by the OLM console: $OLM_DIR/scripts/run_console_local.sh
3636
oc adm policy add-cluster-role-to-user cluster-admin system:serviceaccount:kube-system:default

0 commit comments

Comments
 (0)