Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: install OLM with 'kubectl create -f' instead of 'kubectl apply -f' #449

Merged
merged 1 commit into from
Sep 10, 2018

Conversation

djwhatle
Copy link
Contributor

@djwhatle djwhatle commented Sep 7, 2018

Changes:

  • kubectl apply -f to kubectl create -f
  • OLM manifest from 0.4.0 to 0.6.0 to reflect latest release

Rationale:

Trying to install OLM release 0.6.0 onto Origin v3.11 with oc apply -f leads to the error shown below:

$ oc apply -f deploy/upstream/manifests/0.6.0/
serviceaccount/olm-operator-serviceaccount created
clusterrolebinding.rbac.authorization.k8s.io/olm-operator-binding-kube-system created
customresourcedefinition.apiextensions.k8s.io/clusterserviceversions.operators.coreos.com created
customresourcedefinition.apiextensions.k8s.io/catalogsources.operators.coreos.com created
customresourcedefinition.apiextensions.k8s.io/installplans.operators.coreos.com created
customresourcedefinition.apiextensions.k8s.io/subscriptions.operators.coreos.com created
catalogsource.operators.coreos.com/ocs created
deployment.apps/alm-operator created
deployment.apps/catalog-operator created
clusterrole.rbac.authorization.k8s.io/aggregate-olm-edit created
clusterrole.rbac.authorization.k8s.io/aggregate-olm-view created
The ConfigMap "ocs" is invalid: metadata.annotations: Too long: must have at most 262144 characters

Using oc create -f instead allows for successful creation of resources:

$ oc create -f deploy/upstream/manifests/0.6.0/
serviceaccount/olm-operator-serviceaccount created
clusterrolebinding.rbac.authorization.k8s.io/olm-operator-binding-kube-system created
customresourcedefinition.apiextensions.k8s.io/clusterserviceversions.operators.coreos.com created
customresourcedefinition.apiextensions.k8s.io/catalogsources.operators.coreos.com created
customresourcedefinition.apiextensions.k8s.io/installplans.operators.coreos.com created
customresourcedefinition.apiextensions.k8s.io/subscriptions.operators.coreos.com created
configmap/ocs created
catalogsource.operators.coreos.com/ocs created
deployment.apps/alm-operator created
deployment.apps/catalog-operator created
clusterrole.rbac.authorization.k8s.io/aggregate-olm-edit created
clusterrole.rbac.authorization.k8s.io/aggregate-olm-view created

@djwhatle djwhatle changed the title docs: install OLM with kubectl create instead of kubectl apply docs: install OLM with 'kubectl create -f' instead of 'kubectl apply -f' Sep 7, 2018
Copy link
Member

@ecordell ecordell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks!

@ecordell ecordell merged commit 15ae1ae into operator-framework:master Sep 10, 2018
ecordell added a commit to ecordell/operator-lifecycle-manager that referenced this pull request Mar 8, 2019
docs: install OLM with 'kubectl create -f' instead of 'kubectl apply -f'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants