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

Added custom resource definition instructions #30

Merged

Conversation

jroper
Copy link
Contributor

@jroper jroper commented Nov 12, 2018

Fixes #29.

Adds the missing instructions for adding an API definition for custom resources.

Note, I have no idea what I'm doing, I'm just trying to follow the getting started guide myself to learn about operators, and worked out that this step appears to be missing. If anything here is wrong, it will likely be far more expedient for someone who actually knows what they are doing to fix it.

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 12, 2018
@openshift-ci-robot
Copy link

Hi @jroper. Thanks for your PR.

I'm waiting for a operator-framework member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

I understand the commands that are listed here.

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.

@openshift-ci-robot openshift-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Nov 12, 2018
@jroper jroper force-pushed the add-custom-resource-instructions branch from 23c23af to 819b169 Compare November 12, 2018 04:19
@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 12, 2018
@jroper
Copy link
Contributor Author

jroper commented Nov 12, 2018

Updated to reflect feedback in #29 (comment).

Copy link
Collaborator

@hasbro17 hasbro17 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 for the fix.

@hasbro17 hasbro17 merged commit b44ec5b into operator-framework:master Nov 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generic error when following guide
3 participants