From ae9dde044f0a60c9034874b2912729ed694f5375 Mon Sep 17 00:00:00 2001 From: Christian Breunig Date: Sat, 4 Feb 2023 13:23:00 +0100 Subject: T4975: always sync() filesystem after commit (cherry picked from commit 29a44a73c638cb22839aa32986de367231b6efe9) --- src/etc/commit/post-hooks.d/00vyos-sync | 7 +++++++ 1 file changed, 7 insertions(+) create mode 100755 src/etc/commit/post-hooks.d/00vyos-sync (limited to 'src') diff --git a/src/etc/commit/post-hooks.d/00vyos-sync b/src/etc/commit/post-hooks.d/00vyos-sync new file mode 100755 index 000000000..e3bde3abb --- /dev/null +++ b/src/etc/commit/post-hooks.d/00vyos-sync @@ -0,0 +1,7 @@ +#!/bin/sh +# When power is lost right after a commit modified files, the +# system can be corrupted and e.g. login is no longer possible. +# Always sync files to the backend storage after a commit. +# https://phabricator.vyos.net/T4975 +sync + -- cgit v1.2.3 From f8108e3d713928b8f232535261022047f9c84c66 Mon Sep 17 00:00:00 2001 From: Christian Breunig Date: Sat, 4 Feb 2023 13:26:52 +0100 Subject: Revert "login: T4975: Fixed broken CLI commands" This reverts commit 7b36c363cd5b0168bd83c399f50a0a360ba3ee58. A general solution is implemented in Commit ae9dde04 ("T4975: always sync() filesystem after commit"). --- src/conf_mode/system-login.py | 3 --- 1 file changed, 3 deletions(-) (limited to 'src') diff --git a/src/conf_mode/system-login.py b/src/conf_mode/system-login.py index ceca0a14b..aba10689d 100755 --- a/src/conf_mode/system-login.py +++ b/src/conf_mode/system-login.py @@ -311,9 +311,6 @@ def apply(login): except Exception as e: raise ConfigError(f'RADIUS configuration failed: {e}') - # sync user data files to disk - os.sync() - return None -- cgit v1.2.3