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

Add/update CSVs for RH and partners #440

Merged
merged 19 commits into from
Sep 12, 2018

Conversation

robszumski
Copy link
Collaborator

@robszumski robszumski commented Aug 31, 2018

Updates CSV catalogs for Red Hat and partner Operators for 3.11

  • Add AMQ Streams
  • Add MongoDB
  • Add Dynatrace
  • Add Couchbase
  • Add CRD permissions to aggregated roles

@robszumski robszumski requested a review from ecordell August 31, 2018 00:16
@ecordell ecordell added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Aug 31, 2018
@ecordell
Copy link
Member

/test

@ecordell ecordell closed this Aug 31, 2018
@ecordell ecordell reopened this Aug 31, 2018
@ecordell ecordell closed this Aug 31, 2018
@ecordell ecordell reopened this Aug 31, 2018
@robszumski
Copy link
Collaborator Author

Prometheus image locations, although I haven't had time to test them in a CSV:

registry.reg-aws.openshift.com/openshift3/ose-configmap-reloader:v3.11.0-0.25.0.0
args:
                - -namespace=$(K8S_NAMESPACE)
                - -manage-crds=false
                - -logtostderr=true
                - --config-reloader-image=registry.reg-aws.openshift.com/openshift3/ose-configmap-reloader:v3.11.0-0.25.0.0
                - --prometheus-config-reloader=registry.reg-aws.openshift.com/openshift3/ose-prometheus-config-reloader:v3.11.0-0.25.0

@ecordell
Copy link
Member

ecordell commented Sep 4, 2018

@robszumski I can't pull registry.reg-aws.openshift.com/openshift3/ose-configmap-reloader:v3.11.0-0.25.0.0 without logging in, which means OLM will to include a pull secret for it (and I'm not sure what to include). I think we need to figure out a better solution for operator images.

@robszumski robszumski changed the title [WIP] Add/update CSVs for RH and partners Add/update CSVs for RH and partners Sep 5, 2018
@robszumski
Copy link
Collaborator Author

This is ready for a review

@ecordell
Copy link
Member

ecordell commented Sep 7, 2018

@robszumski I think we should update the instructions for these operators so that they describe the CR types and how to use them, at least.

If it helps you should be able to log in here: https://console.apps.olmci.origin-gce.dev.openshift.com/k8s/ns/ci-alm-pr-440-update-csv/clusterserviceversions with github and test out this branch in the cluster

@ecordell ecordell closed this Sep 11, 2018
@ecordell ecordell reopened this Sep 11, 2018
ecordell
ecordell previously approved these changes Sep 11, 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.

We can go ahead and merge, but I would like to discuss criteria for an operator being listed in the catalog at some point.

ecordell
ecordell previously approved these changes Sep 11, 2018
@ecordell ecordell merged commit 9429e23 into operator-framework:master Sep 12, 2018
ecordell added a commit to ecordell/operator-lifecycle-manager that referenced this pull request Mar 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Indicates a non-member PR verified by an org member that is safe to test.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants