diff options
author | John Estabrook <jestabro@vyos.io> | 2023-12-31 20:45:58 -0600 |
---|---|---|
committer | Mergify <37929162+mergify[bot]@users.noreply.github.com> | 2024-01-01 16:58:57 +0000 |
commit | a5cdb6f4053428c94604b48adda1ec11081ab542 (patch) | |
tree | db1808bcc1dd2aa2c99d1102f04cb8c43122ef58 | |
parent | 13fddcfef2f9c13dd6e789fa9e8050011241e2b5 (diff) | |
download | vyos-1x-a5cdb6f4053428c94604b48adda1ec11081ab542.tar.gz vyos-1x-a5cdb6f4053428c94604b48adda1ec11081ab542.zip |
image-tools: T5883: preserve file owner in /config on add system update
(cherry picked from commit 9f66b9ccfa25f56c209d90a0ad5ad779f3963bee)
-rwxr-xr-x | src/op_mode/image_installer.py | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/op_mode/image_installer.py b/src/op_mode/image_installer.py index 6a8797aec..5e2af2d97 100755 --- a/src/op_mode/image_installer.py +++ b/src/op_mode/image_installer.py @@ -257,6 +257,15 @@ def search_previous_installation(disks: list[str]) -> None: disk.partition_umount(image_drive) +def copy_preserve_owner(src: str, dst: str, *, follow_symlinks=True): + if not Path(src).is_file(): + return + if Path(dst).is_dir(): + dst = Path(dst).joinpath(Path(src).name) + st = Path(src).stat() + copy(src, dst, follow_symlinks=follow_symlinks) + chown(dst, user=st.st_uid) + def copy_previous_installation_data(target_dir: str) -> None: if Path('/mnt/config').exists(): @@ -814,7 +823,7 @@ def add_image(image_path: str, vrf: str = None, username: str = '', chown(target_config_dir, group='vyattacfg') chmod_2775(target_config_dir) copytree('/opt/vyatta/etc/config/', target_config_dir, - dirs_exist_ok=True) + copy_function=copy_preserve_owner, dirs_exist_ok=True) else: Path(target_config_dir).mkdir(parents=True) chown(target_config_dir, group='vyattacfg') |