Skip to content

Conversation

@akira25
Copy link
Member

@akira25 akira25 commented May 23, 2024

Kompiliert Ja/Nein: (Bitte gib Prozessorarchitektur, Modell und Falter/OpenWrtversion an)
Läuft live Ja/Nein: (Bitte gib Prozessorarchitektur, Modell und Falter/OpenWrtversion an und wie du deine Änderungen getestet hast)

Beschreibung deiner Änderungen:


Compile tested: no
Run tested: yes, on falter 1.2.3

When updating nodes automatically, we want to detect custom changes made by the user. As nodes may brake when using non-standard config and updating, we want to avoid updating on these nodes.

akira25 added 2 commits May 22, 2024 15:53
When updating nodes automatically, we want to detect custom changes made by
the user. As nodes may brake when using non-standard config and updating,
we want to avoid updating on these nodes.

Signed-off-by: Martin Hübner <[email protected]>
Pretty unsure, how this was able to work on the last update. :/

Signed-off-by: Martin Hübner <[email protected]>
@pmelange
Copy link
Collaborator

Is the uci-defaults script designed to always run when the router boots? Then it would be best to have an explicit "exit 1" at the end of the code. An "exit 0" causes the file to be deleted and anything else makes the uci-defaults script stay around for the next boot.

@akira25
Copy link
Member Author

akira25 commented May 28, 2024

Is the uci-defaults script designed to always run when the router boots? Then it would be best to have an explicit "exit 1" at the end of the code. An "exit 0" causes the file to be deleted and anything else makes the uci-defaults script stay around for the next boot.

Not quite: The script is intended, to only run once after the wizard finished. Thus we check, wether it ran already and exit , when it hasn't.

So we make checksums of a freshly configured router and check them on autoupdate against the current ones, to detect changes in the files.

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.

3 participants