-
Notifications
You must be signed in to change notification settings - Fork 883
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
doc(autoinstall): Remove incorrect statements, be more direct #5545
Conversation
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.
LGTM, thanks! I left a minor nit comment.
doc/rtd/reference/faq.rst
Outdated
``cloud-init``'s cloud-config with additional keys. | ||
|
||
If you are an autoinstall user, please direct questions to their IRC channel | ||
(``#ubuntu-server`` on Libera) or Discourse. |
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 include a link to their Discourse?
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.
Apart from the mentioned comments, LGTM :)
Commit message
Additional Context
Top level keys are no longer ignored, so drop this incorrect statement.
Tailor this section more directly toward subiquity / autoinstall, since
that is where the frequently asked questions come from.