-
Notifications
You must be signed in to change notification settings - Fork 584
feat(platform): introduce Platform API at v1alpha1 level #1234
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
feat(platform): introduce Platform API at v1alpha1 level #1234
Conversation
|
Skipping CI for Draft Pull Request. |
|
Hello @tylerslaton! Some important instructions when contributing to openshift/api: For merging purposes, this repository follows the no-Feature-Freeze process which means that in addition to the standard
OR
Who should apply these qe/docs/px labels?
|
|
/test all |
|
/test all |
|
@tylerslaton Once we get the CRD generation working, we need to ensure that the |
|
/retest |
3d07a64 to
898e2e6
Compare
|
/retest |
c34bc08 to
849179b
Compare
4d43e0e to
5739b51
Compare
|
cc @joelanford |
|
In case it isn't clear, it should be noted that the EP for platform operators dictates that the That value will be interpreted as the package name that the controller should lookup and attempt to source bundles from. |
e569ce6 to
0786302
Compare
ebc72d8 to
3d200ea
Compare
|
/lgtm |
|
Wow crazy timing. /lgtm cancel |
3d200ea to
c664f25
Compare
c664f25 to
0e0775b
Compare
Signed-off-by: Tyler Slaton <tyslaton@redhat.com>
0e0775b to
6642585
Compare
|
@tylerslaton: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
/lgtm |
|
@jianzhangbjz @michaelryanpeter Would you mind taking a look and giving the QE/Doc tags here? |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, JoelSpeed, tylerslaton The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/label qe-approved |
|
/label docs-approved |
|
/label px-approved |
Summary
Adding the necassary API's for
PlatformOperatorsas outlined by this EP.This is a work in progress PR that is meant to burn down on review while the EP finishes up and is thus subject to change as progress is made.This PR is now ready for review.