Skip to content

Commit

Permalink
Merge pull request openshift#622 from jmprusi/jmprusi/microshift-fixes
Browse files Browse the repository at this point in the history
OPRUN-3063: Microshift manifests fixes
  • Loading branch information
openshift-merge-bot[bot] authored Nov 29, 2023
2 parents 5c00fde + 13adfcb commit cc54174
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
4 changes: 2 additions & 2 deletions microshift-manifests/0000_50_olm_15-csv-viewer.rbac.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ metadata:
include.release.openshift.io/self-managed-high-availability: "true"
capability.openshift.io/name: "OperatorLifecycleManager"
name: copied-csv-viewer
namespace: openshift
namespace: openshift-operator-lifecycle-manager
rules:
- apiGroups:
- "operators.coreos.com"
Expand All @@ -27,7 +27,7 @@ metadata:
include.release.openshift.io/self-managed-high-availability: "true"
capability.openshift.io/name: "OperatorLifecycleManager"
name: copied-csv-viewers
namespace: openshift
namespace: openshift-operator-lifecycle-manager
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
Expand Down
3 changes: 0 additions & 3 deletions microshift-manifests/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,3 @@ resources:
- 0000_50_olm_09-aggregated.clusterrole.yaml
- 0000_50_olm_13-operatorgroup-default.yaml
- 0000_50_olm_15-csv-viewer.rbac.yaml
- 0000_50_olm_99-operatorstatus.yaml
- 0000_90_olm_00-service-monitor.yaml
- 0000_90_olm_01-prometheus-rule.yaml
4 changes: 3 additions & 1 deletion scripts/generate_crds_manifests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -520,7 +520,7 @@ microshift_manifests_files=$(find "${ROOT_DIR}/microshift-manifests" -type f -na
# Let's sort the files so that we can have a deterministic order
microshift_manifests_files=$(echo "${microshift_manifests_files}" | sort)
# files to ignore, substring match.
files_to_ignore=("ibm-cloud-managed.yaml" "kustomization.yaml" "psm-operator" "removed" "collect-profiles")
files_to_ignore=("ibm-cloud-managed.yaml" "kustomization.yaml" "psm-operator" "removed" "collect-profiles" "prometheus" "service-monitor" "operatorstatus")

# Add all the manifests files to the kustomization file while ignoring the files in the files_to_ignore list
for file in ${microshift_manifests_files}; do
Expand All @@ -541,3 +541,5 @@ done
#
${SED} -i '/- --writeStatusName/,+3d' ${ROOT_DIR}/microshift-manifests/0000_50_olm_07-olm-operator.deployment.yaml

# Replace the namespace openshift, as it doesn't exist on microshift, in the rbac file
${SED} -i 's/ namespace: openshift/ namespace: openshift-operator-lifecycle-manager/g' ${ROOT_DIR}/microshift-manifests/0000_50_olm_15-csv-viewer.rbac.yaml

0 comments on commit cc54174

Please sign in to comment.