Skip to content

Commit

Permalink
chore(deploy): remove packageserver subscription and catalog from chart
Browse files Browse the repository at this point in the history
this has several advantages:
 - deploy artifacts are simpler
 - faster install/startup time
 - simpler integration with CVO
 - unblocking using kustomize for our release scripts
  • Loading branch information
ecordell committed Jul 3, 2019
1 parent e974cc6 commit a61428c
Show file tree
Hide file tree
Showing 14 changed files with 4 additions and 424 deletions.
11 changes: 0 additions & 11 deletions deploy/chart/templates/0000_50_olm_11-olm-operators.configmap.yaml

This file was deleted.

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,3 +1 @@
{{ if and .Values.installType (eq .Values.installType "ocp") }}
{{- include "packageserver.clusterserviceversion" . }}
{{- end }}

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ spec:
strategy: deployment
spec:
clusterPermissions:
- serviceAccountName: packageserver
- serviceAccountName: olm-operator-serviceaccount
rules:
- apiGroups:
- authorization.k8s.io
Expand Down
2 changes: 1 addition & 1 deletion deploy/chart/templates/_packageserver.deployment-spec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ spec:
labels:
app: packageserver
spec:
serviceAccountName: packageserver
serviceAccountName: olm-operator-serviceaccount
{{- if and .Values.installType (eq .Values.installType "ocp") }}
priorityClassName: "system-cluster-critical"
{{- end }}
Expand Down
269 changes: 0 additions & 269 deletions manifests/0000_50_olm_11-olm-operators.configmap.yaml

This file was deleted.

10 changes: 0 additions & 10 deletions manifests/0000_50_olm_12-olm-operators.catalogsource.yaml

This file was deleted.

10 changes: 0 additions & 10 deletions manifests/0000_50_olm_14-packageserver.subscription.yaml

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ spec:
displayName: Package Server
description: Represents an Operator package that is available from a given CatalogSource which will resolve to a ClusterServiceVersion.
minKubeVersion: 1.11.0
replaces: packageserver.v0.9.0
keywords: ['packagemanifests', 'olm', 'packages']
maintainers:
- name: Red Hat
Expand All @@ -32,7 +31,7 @@ spec:
strategy: deployment
spec:
clusterPermissions:
- serviceAccountName: packageserver
- serviceAccountName: olm-operator-serviceaccount
rules:
- apiGroups:
- authorization.k8s.io
Expand Down Expand Up @@ -78,7 +77,7 @@ spec:
labels:
app: packageserver
spec:
serviceAccountName: packageserver
serviceAccountName: olm-operator-serviceaccount
priorityClassName: "system-cluster-critical"
nodeSelector:
beta.kubernetes.io/os: linux
Expand Down
Loading

0 comments on commit a61428c

Please sign in to comment.