Skip to content

Commit

Permalink
Merge pull request #366 from appuio/install/include-openshift4yml-ear…
Browse files Browse the repository at this point in the history
…lier

Move including `openshift4.yml` to before configuring Cilium
  • Loading branch information
simu authored Nov 19, 2024
2 parents 2018604 + f766ee9 commit 734c9ed
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 9 deletions.
12 changes: 12 additions & 0 deletions docs/modules/ROOT/partials/install/prepare-commodore.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,18 @@ yq eval -i ".parameters.openshift.baseDomain = \"${CLUSTER_DOMAIN}\"" \
git commit -a -m "Configure cluster domain for ${CLUSTER_ID}"
----

. Include `openshift4.yml` in the cluster's config if it exists
+
NOTE: For some tenants, this may already configure some of the settings shown in this how-to.
+
[source,bash]
----
if ls openshift4.y*ml 1>/dev/null 2>&1; then
yq eval -i '.classes += ".openshift4"' ${CLUSTER_ID}.yml;
git commit -a -m "Include openshift4 class for ${CLUSTER_ID}"
fi
----

. Add Cilium to cluster configuration
+
[NOTE]
Expand Down
9 changes: 0 additions & 9 deletions docs/modules/ROOT/partials/install/prepare-syn-config.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,6 @@
pushd "inventory/classes/${TENANT_ID}/"
----

. Include openshift4.yml if it exists
+
[source,bash]
----
if ls openshift4.y*ml 1>/dev/null 2>&1; then
yq eval -i '.classes += ".openshift4"' ${CLUSTER_ID}.yml;
fi
----

. Include no-opsgenie class to prevent monitoring noise during cluster setup
+
[source,bash]
Expand Down

0 comments on commit 734c9ed

Please sign in to comment.