-
Notifications
You must be signed in to change notification settings - Fork 103
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
Try to improve concepts docs #473
Conversation
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 really like to see under Instances the random string creation mentioned, wdyt?
|
||
## OperatorVersion | ||
|
||
Specific version of a deployable application, including configuration and lifecycle hooks for deployments, upgrades, and rollbacks (e.g., [Kafka version 2.4.0](https://github.com/kudobuilder/kudo/blob/master/config/samples/kafka-operatorversion.yaml)). | ||
This is already complete definition of application to be installed (except overridable parameters). By adding OperatorVersion to your cluster, no application is running yet. |
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.
This is already complete definition of application to be installed (except overridable parameters). By adding OperatorVersion to your cluster, no application is running yet. | |
This is already a complete definition of an application to be installed (except overridable parameters). By adding OperatorVersion to your cluster, no application is running yet. |
What do you mean by By adding OperatorVersion to your cluster, no application is running yet.
?
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.
that there are not pods or anything running. You have to create instance to actually run the application (= to actually run kafka)
Resource created to manage an instance of specific OperatorVersion. Instances are pets and have the same name throughout its entire lifecycle. (e.g., [Kafka 2.4.0 cluster with 1 broker](https://github.com/kudobuilder/kudo/blob/master/config/samples/kafka-instance.yaml)). | ||
When you create an instance, you provide missing parameters for installed OperatorVersion. Creating an instance typically causes bunch of services, pods or statefulsets to start running in your cluster. | ||
Instances have the same name throughout its entire lifecycle. (e.g., [Kafka 2.4.0 cluster with 1 broker](https://github.com/kudobuilder/kudo/blob/master/config/samples/kafka-instance.yaml)). | ||
|
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.
Should we also mention that we add random strings #437 ?
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.
do you think this is the right place? 🤔 not sure, but no strong opinion as well.
Co-Authored-By: Fabian Baier <[email protected]>
Co-Authored-By: Fabian Baier <[email protected]>
Co-Authored-By: Fabian Baier <[email protected]>
Co-Authored-By: Fabian Baier <[email protected]>
Co-Authored-By: Fabian Baier <[email protected]>
Co-Authored-By: Fabian Baier <[email protected]>
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.
nice updates... there are suggestions worth considering. Its an improvement regardless!
Co-Authored-By: Ken Sipe <[email protected]>
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.
thanks for driving this! Let's iterate on it!
What type of PR is this?
/kind documentation
What this PR does / why we need it:
Try to improve the concepts docs. Right now it was very much around the CRDs (storage) than concepts itself. I think there is a lot to be done here, this is just a possible first step.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?: