Skip to content
This repository was archived by the owner on Sep 21, 2020. It is now read-only.

Update apiVersion to operators.coreos.com/v1alpha1 #22

Merged
merged 1 commit into from
Sep 19, 2018

Conversation

darkowlzz
Copy link
Contributor

csv is defined under operators.coreos.com group in the latest version
of OLM.

Fixes #21

@@ -1,7 +1,7 @@
# This file defines the ClusterServiceVersion (CSV) to tell the catalog how to display, create and
# manage the application as a whole. If changes are made to the CRD for this application kind,
# make sure to replace those references below as well.
apiVersion: app.coreos.com/v1alpha1
apiVersion: operators.coreos.com/v1alpha1
kind: ClusterServiceVersion-v1
Copy link
Member

Choose a reason for hiding this comment

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

The -v1 also needs to be removed.

@@ -1,7 +1,7 @@
# This file defines the ClusterServiceVersion (CSV) to tell the catalog how to display, create and
# manage the application as a whole. If changes are made to the CRD for this application kind,
# make sure to replace those references below as well.
apiVersion: app.coreos.com/v1alpha1
apiVersion: operators.coreos.com/v1alpha1
kind: ClusterServiceVersion-v1
Copy link
Member

Choose a reason for hiding this comment

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

The -v1 also needs to be removed.

@lilic
Copy link
Member

lilic commented Sep 19, 2018

Also I noticed in the README there is a query that should be removed that includes the v1s.

$ kubectl get ClusterServiceVersion memcachedoperator.v0.0.1 -o json | jq '.status'

csv is defined under operators.coreos.com group in the latest version
of OLM.
@darkowlzz
Copy link
Contributor Author

oops! 😅 I thought I did remove -v1 from there.
Added the suggested changes in the same commit.
Thanks!

@robszumski
Copy link
Contributor

Thanks for putting this PR together!

Copy link
Contributor

@rithujohn191 rithujohn191 left a comment

Choose a reason for hiding this comment

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

👍 LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants