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

LuCI configuration-change: timeout of confirmation triggers rollback #832

Closed
SvenRoederer opened this issue Jul 19, 2020 · 3 comments
Closed

Comments

@SvenRoederer
Copy link
Contributor

Bug report

What is the problem?
Changing Network or olsr parameters via LuCI will not apply, as they get reverted.

What is the expected behavior?
A change of these parameters should be applied.
Currently it seems to fail as the new LuCI (OpenWrt-19.07) waits for a confirmation for the client (JS-code) that the changes are woking. There is a 30 seconds timeout for this. A change on some network related parameters causes the restart of the routing, which will very likely take more than 30 seconds to reestablish.

Firmware Version:
I found this with "Dev-Monthly-Master 295051dd"; which is 835bcb9 + #794

@everloop2
Copy link
Contributor

in recent versions of openwrt seen 60-90sec session timeouts

luci-base: extend rollback timeout to 90 seconds
https://git.openwrt.org/?p=project/luci.git;a=commit;h=81cf99a50706a75a2259296e4aab3f7b3bf5d54e

@everloop2
Copy link
Contributor

think we can close due: #802 19.07.4 update.

@SvenRoederer
Copy link
Contributor Author

Thanks for watching this. Indeed 90 sec should be enough to restart olsr and re-establish the routing.

A sidenote: the affected node was installed with an image dated before this rollback happened and upgraded to a more recent version. As the config was kept, the old "30 secs timeout" was kept also.
But as we only take care for such changes between official releases of our firmware, I don't think we need a migration to take care.

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

No branches or pull requests

2 participants