-
Notifications
You must be signed in to change notification settings - Fork 11
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
"Getting started" section across all operators #237
Labels
epic
release-note
Denotes a PR that will be considered when it comes time to generate release notes.
Comments
Closed
As part of this, we can also tackle this ticket: stackabletech/issues#239 |
This was referenced Jul 25, 2022
bors bot
pushed a commit
to stackabletech/druid-operator
that referenced
this issue
Aug 8, 2022
# Description stackabletech/documentation#237 This is a first "getting started" section for an operator, which we hopefully can bring to other operators afterwards. This replaces the old install page, and takes parts of the usage page and the examples to build it all into a "Getting started" guide/tutorial. Also, all the script snippets are in one big script which can be executed and functions as a test. Also, templating was introduced to be able to template operator versions into the script. I wanted to avoid this, but didn't see an alterantive way to get Helm and stackablectl to install the correct versions. I think with the templating there's just one file that needs to be adapted, which we can also automate in the future. I've removed the simple-druid-cluster example, because that's now in the docs (and with the script it is also testable in a semi-automatic way: we can fully automate this in the future). If this PR gets accepted, i'd like to put the `docs_templating.sh` script into operator-templating. Co-authored-by: Felix Hennig <fhennig@users.noreply.github.com>
Update: The Druid docs were merged.
|
This was referenced Aug 8, 2022
bors bot
pushed a commit
to stackabletech/superset-operator
that referenced
this issue
Aug 10, 2022
# Description stackabletech/documentation#237 *Please add a description here. This will become the commit message of the merge request later.* Co-authored-by: Felix Hennig <fhennig@users.noreply.github.com>
This was referenced Aug 11, 2022
This was referenced Aug 16, 2022
fhennig
added
the
release-note
Denotes a PR that will be considered when it comes time to generate release notes.
label
Aug 23, 2022
All done! |
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
epic
release-note
Denotes a PR that will be considered when it comes time to generate release notes.
We've decided that it would be good for onboarding new users to have a dedicated "Getting started" section in all operators.
It should contain
This also relates to #204 - we should mention stackablectl in the install page (not done currently)
All the script and yaml files in there should be put into a shell script that can actually be executed, for testing purposes. Have a look at the druid docs, where this has been implemented already: /~https://github.com/stackabletech/druid-operator/tree/main/docs/modules/getting_started
Secret and Commons Operator don't need a Getting Started guide.
Acceptance criteria per operator:
getting_started
exists, with anindex.adoc
, ainstallation.adoc
andfirst_steps.adoc
installation.adoc
is removed.yaml
and shell snippets are in theexamples/code
directory and can be executed as a script, to test the documentationtemplate_docs.sh
script and thetemplating_vars.yaml
file.The text was updated successfully, but these errors were encountered: