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

Cleanup for the automatically generated config options #13229

Merged
merged 5 commits into from
Mar 28, 2024

Conversation

ru-fu
Copy link
Contributor

@ru-fu ru-fu commented Mar 27, 2024

No description provided.

ru-fu added 5 commits March 27, 2024 15:10
While these settings aren't technically config options, I think it can
be beneficial for users to find them in the config option index, and
have a direct link to reference them.

They aren't extracted from the code, so they are included in the
documentation page directly. They will also not be part of the metadata
file.

Signed-off-by: Ruth Fuchss <[email protected]>
- Remove the note about not all config options being included.
- Explain that sysctl options are not LXD options.
- Make the headings and links to headings all uppercase (they are
  generated from the code).

Signed-off-by: Ruth Fuchss <[email protected]>
I didn't go through the full file yet, but some of the links are
better than none. ;-)

Signed-off-by: Ruth Fuchss <[email protected]>
@ru-fu ru-fu requested a review from tomponline as a code owner March 27, 2024 17:06
@github-actions github-actions bot added Documentation Documentation needs updating API Changes to the REST API labels Mar 27, 2024
Copy link

Heads up @ru-fu - the "Documentation" label was applied to this issue.

@tomponline tomponline merged commit cbf3911 into canonical:main Mar 28, 2024
29 checks passed
@ru-fu ru-fu deleted the LXD-894-config-options branch April 8, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Changes to the REST API Documentation Documentation needs updating
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants