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

improve ota #238

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

improve ota #238

wants to merge 5 commits into from

Conversation

the0ne
Copy link
Collaborator

@the0ne the0ne commented Mar 9, 2019

beautified ota output, try to look like the flash tool output
added option to provide server from lua

added option to provide a project name from lua so that one ota server
can serve several different projects

added option to verify the written image => if verification fails then the system will not be rebooted.

added option to not reboot after the ota was done successfully

Thomas E. Horner added 2 commits March 9, 2019 15:17
beautified ota output, try to look like the flash tool output
added option to provide server from lua

added option to provide a project name from lua so that one ota server
can serve several different projects

added option to verify the written image.
if verification fails then the system will not be rebooted.

added option to not reboot after a successful ota
@the0ne
Copy link
Collaborator Author

the0ne commented Mar 12, 2019

@jolivepetrus added to the wiki at the bottom of https://github.com/the0ne/Lua-RTOS-ESP32/wiki/OTA-updates

@jolivepetrus
Copy link
Contributor

@the0ne,

This PR have collateral efects with the Whitecat Ecosystem services. I think that your idea about "project" is covered by the current idea of type / subtype defined in KConfig.

@the0ne
Copy link
Collaborator Author

the0ne commented Mar 18, 2019

@jolivepetrus,

having the lua part define the project opens the possibility to share binaries between projects, yet still being able to "split" them later - without having to touch the device.

I didn't realize the the whitecat ide does read+analyze the output of the ota update.
the idea of the changes was to make the terminal output nice, easy and informative.
please let me know the Whitecat Ecosystem services requirements so I can possibly find a way to achieve both targets.

@the0ne
Copy link
Collaborator Author

the0ne commented Jul 16, 2019

I think that there's a lot of benefit from the added information and the much more structured output.

/ > net.ota(nil, nil, false, true)
Connecting to https://sdkconfig-ota-server.tld ...
Current firmware commit is 09d7592cf183fdc138428c756e0ec124ea043a01
Running from ota_0 at offset 0x000a0000
Writing to   ota_1 at offset 0x00250000 - capacity is 1.69 MB
Starting OTA update, downloading partition image with 1.14 MB
Writing chunk at 0x00373d50... (100 %)
OTA update finished
Successfully changed boot partition
/ > 

@jolivepetrus if possible please let me know the Whitecat Ecosystem services requirements so I can try to find a way to achieve compatibility.

@the0ne
Copy link
Collaborator Author

the0ne commented May 16, 2022

@the0ne,

This PR have collateral effects with the Whitecat Ecosystem services. I think that your idea about "project" is covered by the current idea of type / subtype defined in KConfig.

Please note that those parameters are completely optional.
So there shouldn't be a negative effect to the Whitecat Ecosystem services.

@progerstar
Copy link

I just want to point out that branches are still not merged, and the wiki has already been updated and does not correspond to the current project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants