-
Notifications
You must be signed in to change notification settings - Fork 45
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
DOCS-3434: Document Micro-RDK OTA updates #3917
Conversation
✅ Deploy Preview for viam-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Requesting changes, see comment.
Also: What are your thoughts on placing the OTA info in the Deploy & Manage section under Update packages?
To remotely update the firmware on your microcontroller without a physical connection to the device, add the OTA (over-the-air) service to your microcontroller's configuration in the [Viam app](https://app.viam.com). | ||
Use **JSON** mode to add the service as in the template below, then configure the URL from which to fetch new firmware, and the version name. | ||
|
||
When the `version` field is modified, the Micro-RDK will automatically download and install the firmware at the URL onto your device. |
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.
According to the scope, I think this would be more accurate as:
Your device checks for configuration updates periodically. If the devices receives a configuration with the OTA service and a modified version in it, the device downloads the new firmware to an inactive partition and restarts. When the device boots it will load the new firmware.
The main point being that the Micro-RDK does not download anything. The Micro-RDK isn't an executable, it's what the device firmwares are built from. This needs SME review though.
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.
@npentrel yes that correction is more accurate; however I think in your comment you may have meant 'does not install anything' instead of 'does not download anything'.
It's an all-or-nothing process since it's similar to trying to swap engines while driving. At least currently, there is no direct way to roll-back to a previous firmware if you do a bad upgrade. It's very possible to update to broken firmware that can leave the device in a bad state where it needs to be reflashed entirely
|
I'm ok with that while it's as long as you add a notice on the update package page. We probably also need one on the deploy page. Speaking in broader terms though, both the Deploy package and update package pages could be changed to have sections for viam-server and viam-micro-server. It depends on how people think about deploying firmware. We could even add a subpage that is "Deploy firmware" and "Update firmware (OTA)".
At least a note, probably.
I don't actually know if fragments are supported for Micro-RDK, I'll test it tomorrow. If it is, it may be worth having a conversation whether we suggest using fragments for the ota service |
@npentrel They are and are actually how we are configuring (both components and OTA service) the in-office temperature sensors |
Also, @JessamyT, I'm not sure where, but we should put a large disclaimer about how any custom firmware is hosted As of now, there are two hard requirements for properly hosting firmware:
|
@npentrel okay this section has gotten a bit longer with the added tips so it's almost big enough to be a sub-page at this point. I would be fine with this moving over to there, and having a link from the micro page to where this content lands in /manage/. I'd err on the side of merging this sooner rather than later though since it's an exciting new update, and then if you decide it makes more sense in your section, editing those intros, descriptions etc. and moving it can happen in a subsequent PR? In the meantime, I've added notes to deploy and to update, and I think I've addressed all the points you brought up Matt though if more details are needed, happy to add. |
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.
I need to think some more about whether I think this should or should not move to the manage page, but good to merge for now and I'll make a follow up ticket for myself
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.
LGreatTM!
🔎💬 Inkeep AI search and chat service is syncing content for source 'Viam Docs (https://docs.viam.com)' |
No description provided.