-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Add plugin reload docs to the website sidebar #10108
Conversation
Should probably be addressed separately, but while looking at #9347 I noticed that value for the |
@@ -33,3 +33,6 @@ flags](/docs/commands) included on all commands. | |||
|
|||
- `-mounts` `(array: [])` - Array or comma-separated string mount paths of the | |||
plugin backends to reload. | |||
|
|||
- `-scope` `(string: "")` - The scope of the reload, omitted for local, 'global' |
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 we re-phrase "omitted for local", as local could it mean mounts, namespaces, or nodes.
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.
Will do. I just copied what was in the CLI help text. I will adjust that in a later PR after re-wording this one.
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.
@calvn What are your thoughts on the new phrasing?
They weren't in the sidebar, and the
-scope
flag was also missing, so I added that.