-
Notifications
You must be signed in to change notification settings - Fork 13.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
netman: update module description #21664
Conversation
clarifies a few key aspects from the module
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.
see inline
Co-authored-by: David Sidrane <David.Sidrane@Nscdg.com>
Co-authored-by: David Sidrane <David.Sidrane@Nscdg.com>
Co-authored-by: David Sidrane <David.Sidrane@Nscdg.com>
Co-authored-by: David Sidrane <David.Sidrane@Nscdg.com>
Hey @davids5, which startup script is responsible for running netman update on boot? |
@mrpollo Thanks for updating all this doc. I answered this as "some part of the startup" by providing a link to the start up docs. @junwoo091400 Thanks for tagging some of these with documentation. Historically no one has bothered with that, and so I have not been monitoring. I will do so now that I know you are watching that. I don't have time to look at everything, but very happy to look at things that are relevant. |
it is in rcS line 167 |
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
Co-authored-by: David Sidrane <David.Sidrane@Nscdg.com> Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
Co-authored-by: David Sidrane <David.Sidrane@Nscdg.com> Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
clarifies a few key aspects of the module for our docs here https://docs.px4.io/main/en/modules/modules_system.html#netman
Hey @davids5 it would be good to get your eyes on this one I want to make sure I captured the details correctly, I think the added info will help users.