Skip to content

WIP Add operators.coreos.com/v2alpha1#4

Closed
njhale wants to merge 7 commits intomasterfrom
operators-v2alpha1
Closed

WIP Add operators.coreos.com/v2alpha1#4
njhale wants to merge 7 commits intomasterfrom
operators-v2alpha1

Conversation

@njhale
Copy link
Copy Markdown
Member

@njhale njhale commented Sep 24, 2019

Add the Operator resource under operators.coreos.com/v2alpha1 as the first part in a series of PRs to implement the enhancement to simplify OLM's APIs.

This PR also makes some decisions about the layout of this repo, mainly that directories containing versioned and unversioned types for API groups live directly under the repo root. This structure is taken from the Kubernetes API staging repo.

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Sep 24, 2019
@tkashem
Copy link
Copy Markdown

tkashem commented Sep 25, 2019

Any thoughts about using operatorframework based domain - operators.operatorframework.io? openshift/enhancements#27 (comment)

@njhale
Copy link
Copy Markdown
Member Author

njhale commented Sep 25, 2019

Any thoughts about using operatorframework based domain - operators.operatorframework.io? openshift/enhancements#27 (comment)

Just that we've had pushback when trying to change the group domain before. (If not for that, I would be partial to lifecycle.operatorframework.io to leave room for other components in the org)

@njhale njhale force-pushed the operators-v2alpha1 branch from 6063f32 to 66b7e46 Compare October 1, 2019 11:30
@njhale
Copy link
Copy Markdown
Member Author

njhale commented Oct 1, 2019

Closing in favor of #5

@njhale njhale closed this Oct 1, 2019
@njhale njhale deleted the operators-v2alpha1 branch October 1, 2019 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants