-
Notifications
You must be signed in to change notification settings - Fork 551
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
Example YAMLs #2
Conversation
alm-manifests/affordance.crd.yaml
Outdated
@@ -0,0 +1,15 @@ | |||
apiVersion: apiextensions.k8s.io/v1beta1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure we're not putting Affordances in our first proposal, unless you are hereby voting for them and pitching them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, removed.
alm-manifests/mysql.crd.yaml
Outdated
plural: mysqls | ||
singular: mysql | ||
kind: MySQL | ||
schema: "TODO" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
schema properties should be under spec, like so:
spec:
validation:
openAPISpecV3:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please add them back in with a TODO for the actual schema?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
plural: mysqls | ||
singular: mysql | ||
kind: MySQL | ||
schema: TODO |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix this one too.
Write cluster operator status after successful startup (attempt #2)
Write cluster operator status after successful startup (attempt #2)
[doc] Add fake README to the project proposal
Implementations of the concepts in the current proposal. Includes
AppType
,Operator
and app CRDs.Create CRD in running cluster:
kubectl create -f <file>