Skip to content
This repository was archived by the owner on Jul 30, 2021. It is now read-only.

Conversation

@sohankunkerkar
Copy link
Contributor

@sohankunkerkar sohankunkerkar commented Aug 13, 2019

Thanks submitting your Operator. Please check below list before you create your Pull Request.

New Submissions

  • Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
  • Are you familiar with our contribution guidelines?
  • Have you packaged and deployed your Operator for Operator Framework?
  • Have you tested your Operator with all Custom Resource Definitions?
  • Have you tested your Operator in all supported installation modes?

Updates to existing Operators

  • Is your new CSV pointing to the previous version with the replaces property?
  • Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
  • Have you tested an update to your Operator when deployed via OLM?

Your submission should not

  • Modify more than one operator
  • Submit operators to both upstream-community-operators and community-operators at once
  • Modify any files outside the above mentioned folders
  • Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

  1. Description about the managed Application and where to find more information
  2. Features and capabilities of your Operator and how to use it
  3. Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

  • Human readable name and 1-liner description about your Operator
  • Valid category name1
  • One of the pre-defined capability levels2
  • Links to the maintainer, source code and documentation
  • Example templates for all Custom Resource Definitions intended to be used
  • A quadratic logo

Remember that you can preview your CSV here.

--

1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need

2 For more information see here

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 13, 2019
@openshift-ci-robot openshift-ci-robot added openshift-operator An Operator targeting OpenShift / OKD size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 13, 2019
@sohankunkerkar sohankunkerkar force-pushed the update-csv branch 3 times, most recently from 6d48295 to 65d3887 Compare August 13, 2019 15:27
@sohankunkerkar sohankunkerkar changed the title WIP Updating the install process description for KubeFed operator WIP Updating the install process description for KubeFed operator (downstream) Aug 13, 2019
@sohankunkerkar sohankunkerkar changed the title WIP Updating the install process description for KubeFed operator (downstream) WIP Updating the install process description for KubeFed operator (OKD) Aug 13, 2019
@openshift-ci-robot openshift-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 13, 2019
@sohankunkerkar sohankunkerkar force-pushed the update-csv branch 3 times, most recently from 3d9d3ea to 0d60f13 Compare August 14, 2019 15:11
@sohankunkerkar
Copy link
Contributor Author

/retest

@sohankunkerkar sohankunkerkar force-pushed the update-csv branch 3 times, most recently from 2dd387e to dfa3706 Compare August 21, 2019 18:48
@sohankunkerkar
Copy link
Contributor Author

/retest

@abhat
Copy link
Contributor

abhat commented Aug 23, 2019

/test scorecard

@abhat
Copy link
Contributor

abhat commented Aug 23, 2019

/retest

1 similar comment
@sohankunkerkar
Copy link
Contributor Author

/retest

@sohankunkerkar sohankunkerkar force-pushed the update-csv branch 2 times, most recently from 53b0ade to f0ad8a3 Compare August 23, 2019 22:14
anyOf:
- type: "nullable"
- type: object
status:
Copy link
Contributor

Choose a reason for hiding this comment

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

This needs a version property. Look at my changes in the operator repo: https://github.com/openshift/kubefed-operator/pull/70/files#diff-74e9cd8b8a8ad1f4e69931290d33e0e8R35

@abhat
Copy link
Contributor

abhat commented Aug 24, 2019

/test scorecard

@abhat
Copy link
Contributor

abhat commented Aug 24, 2019

/retest

@sohankunkerkar sohankunkerkar changed the title WIP Updating the install process description for KubeFed operator (OKD) Updating the install process description for KubeFed operator (OKD) Aug 26, 2019
@sohankunkerkar
Copy link
Contributor Author

/assign @dmesser PTAL

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 26, 2019
@openshift-ci-robot
Copy link
Collaborator

@sohankunkerkar: GitHub didn't allow me to assign the following users: PTAL.

Note that only operator-framework members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign @dmesser PTAL

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@dmesser dmesser merged commit 9ad80c5 into operator-framework:master Aug 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

openshift-operator An Operator targeting OpenShift / OKD size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants