Skip to content
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

Added proposal for handbook. #12

Merged
merged 3 commits into from
Jun 7, 2021
Merged

Added proposal for handbook. #12

merged 3 commits into from
Jun 7, 2021

Conversation

bwplotka
Copy link
Contributor

@bwplotka bwplotka commented Jun 2, 2021

@bwplotka
Copy link
Contributor Author

bwplotka commented Jun 2, 2021

NOTE: Proposal process will be proposed in another PR

Copy link
Contributor

@jan--f jan--f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it!

One thing that could be mentioned imho is our user documentation (i.e https://docs.openshift.com/container-platform/4.7/monitoring/understanding-the-monitoring-stack.html). I think we should either explicitly mention it as a non-goal (since I suppose we have processes for that) or mention it as a future item.

Copy link

@dgrisonnet dgrisonnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an awesome initiative @bwplotka, thank you for looking into that 🙂

I have a couple of questions that you might be able to ask in regards to that initiative:

  • How I see one part of the project is as a central place that leverages upstream documentation or any other learning resources and supplements it. One thing that might be hard to understand today is the actual boundary between what should be contributed in the handbook and what should be contributed upstream. Do you think it would be meaningful to document that?
  • What's the process for upstream enhancements? Should we start drafts in this repository before submitting them to the upstream communities?

content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved

### Flow of Adding/Consuming Documentation to Handbook

![flow](../../assets/handbook-proposal.png)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

README.md Show resolved Hide resolved
This handbook was designed to be used in two ways:

* Through our website `https://rhobs-handbook.netlify.app/`.
* Directly from GitHub UI by navigating to [`content`](https://github.com/rhobs/handbook/tree/main/content) directory.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about Github pages?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean instead of hugo + netlify? We have some amazing pre-processors which makes GitHub UI and website working smoothly (you write docs like for GH UI) (: GH pages would lack of that.

Also we have rdy tooling as other website use it, so we can maintain one way of generating website.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have some amazing pre-processors which makes GitHub UI and website working smoothly

Interesting, could you send me some links?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
content/Proposals/Accepted/202106-handbook.md Show resolved Hide resolved
content/contributing.md Show resolved Hide resolved
Copy link

@kakkoyun kakkoyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️ LGTM

@paulfantom
Copy link

What's the process for upstream enhancements? Should we start drafts in this repository before submitting them to the upstream communities?

I would vote "upstream first" as docs have this nasty tendency of being stale and not updated or propagated. We can always use links and reference issues/PRs/other pages, no?

## Non-Goals

* Support other formats than `Markdown,` e.g. Asciidoc
* Precise design proposal process (it will come in a separate proposal).
Copy link

@paulfantom paulfantom Jun 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we align it with Kubernetes Enhancement Proposal process? This way we would have a sort of test bed for writing KEPs and OEPs :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that's the plan, just wanted to keep website and proposals idea separate (although the process will be similar)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh, super useful, will use in next proposal

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm... although it "only" generates TOC, does not verify any format,but cool idea.

Copy link
Contributor

@bill3tt bill3tt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really like that we are aspiring to follow GitLab's examples of open-first documentation ❤️

A shiny new website alone is not going to solve problems we experience when working with documentation.

Let's try it out 💪

README.md Show resolved Hide resolved
content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, everyone for the quick review!

One thing that could be mentioned imho is our user documentation (i.e https://docs.openshift.com/container-platform/4.7/monitoring/understanding-the-monitoring-stack.html). I think we should either explicitly mention it as a non-goal (since I suppose we have processes for that) or mention it as a future item.

Yes good point @jan--f

How I see one part of the project is as a central place that leverages upstream documentation or any other learning resources and supplements it. One thing that might be hard to understand today is the actual boundary between what should be contributed in the handbook and what should be contributed upstream. Do you think it would be meaningful to document that?

I added the handbook-proposal.png with the algorithm to decide. Are there any other rules we can add to make sure it's clearer? (:

What's the process for upstream enhancements? Should we start drafts in this repository before submitting them to the upstream communities?
would vote "upstream first" as docs have this nasty tendency of being stale and not updated or propagated. We can always use links and reference issues/PRs/other pages, no?

Design doc to be added soon in next PR, but the plan is similar to this proposal really. (similar to KEP)

@dgrisonnet

This handbook was designed to be used in two ways:

* Through our website `https://rhobs-handbook.netlify.app/`.
* Directly from GitHub UI by navigating to [`content`](https://github.com/rhobs/handbook/tree/main/content) directory.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean instead of hugo + netlify? We have some amazing pre-processors which makes GitHub UI and website working smoothly (you write docs like for GH UI) (: GH pages would lack of that.

Also we have rdy tooling as other website use it, so we can maintain one way of generating website.

content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
content/Proposals/Accepted/202106-handbook.md Outdated Show resolved Hide resolved
## Non-Goals

* Support other formats than `Markdown,` e.g. Asciidoc
* Precise design proposal process (it will come in a separate proposal).
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that's the plan, just wanted to keep website and proposals idea separate (although the process will be similar)

content/Proposals/Accepted/202106-handbook.md Show resolved Hide resolved
content/contributing.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@bwplotka
Copy link
Contributor Author

bwplotka commented Jun 2, 2021

Thanks I think I addressed all comments, PTAL

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>
Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>
@bwplotka
Copy link
Contributor Author

bwplotka commented Jun 3, 2021

👋🏽 I changed:

  • Moved naming of this handbook to Monitoring Group to reflect that we have only monitoring team in the process nowhere.
  • Articulated clearly, that it's not to replace any product documentation - it's mainly for our own documentation to not get lost, have smooth guides and index for further resources.

@bwplotka
Copy link
Contributor Author

bwplotka commented Jun 4, 2021

If no objection, will merge in couple of hours/Monday. Anyone want to take a look still?

@sthaha
Copy link

sthaha commented Jun 7, 2021

/approve

@kyoto
Copy link
Contributor

kyoto commented Jun 7, 2021

LGTM

Co-authored-by: Andrew Pickering <andy.nihon@gmail.com>
@bwplotka bwplotka merged commit b3a6f65 into main Jun 7, 2021
@bwplotka bwplotka deleted the proposal2 branch June 7, 2021 11:11
bwplotka added a commit that referenced this pull request Sep 7, 2022
* Added proposal for handbook.

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

* Addressed feedback, moved naming to Monitoring Group.

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

* Update content/Proposals/Accepted/202106-handbook.md

Co-authored-by: Andrew Pickering <andy.nihon@gmail.com>

Co-authored-by: Andrew Pickering <andy.nihon@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants