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

Explicitly detail the recommendations for the rock-ons root share (#340) #341

Merged

Conversation

FroggyFlox
Copy link
Member

@FroggyFlox FroggyFlox commented Dec 11, 2021

Fixes #340
@phillxnet, ready for review.

This pull request's proposal

This pull request proposes to slightly improve the recommendations on the creation of the Rock-ons root share to improve clarity. It essentially:

  • explicitly state that the rock-ons root can have any name
  • recommend to create a dedicated share for it (and provides a brief explanation)
  • explicitly state that the rock-ons root share can be part of any pool

Checklist

  • With the proposed changes no Sphinx errors or warnings are generated.
  • I have added my name to the AUTHORS file, if required (descending alphabetical order).

Copy link
Member

@phillxnet phillxnet left a comment

Choose a reason for hiding this comment

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

@FroggyFlox Well done. And nice re-arrangement / enhancement.

I've only got the two word subtraction suggestions indicated. See what you think?

Either way let me know and we can get this out toot sweet.

interface/overview.rst Outdated Show resolved Hide resolved
interface/overview.rst Outdated Show resolved Hide resolved
FroggyFlox and others added 2 commits December 12, 2021 13:02
Co-authored-by: Philip Guyton <[email protected]>
Co-authored-by: Philip Guyton <[email protected]>
@FroggyFlox FroggyFlox requested a review from phillxnet December 12, 2021 18:04
Copy link
Member

@phillxnet phillxnet left a comment

Choose a reason for hiding this comment

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

@FroggyFlox Thanks. That's super. I'll merge and get this in production.

@phillxnet phillxnet merged commit fc2d659 into rockstor:master Dec 12, 2021
@phillxnet
Copy link
Member

PR product published.

@FroggyFlox FroggyFlox deleted the 340_Improve_rockons-root_description branch June 23, 2022 14:02
@phillxnet
Copy link
Member

Linking back to a pending doc issue re the rock-ons-root #166

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.

Clarify use of rockons-root share
2 participants