fix TestHardlinksExistsFiles for 25.8, add chmod 0640 for --hardlink-exists-files during download and restore_remote, fix #1164

Slach · Slach · commit 33e2bfd45364 · 2025-09-04T18:08:41.000+04:00
Signed-off-by: Slach &lt;bloodjazman@gmail.com&gt;
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -11,6 +11,7 @@ IMPROVEMENTS
 BUG FIXES
 - fix restore schema on cluster for VIEW, fix [1199](https://github.com/Altinity/clickhouse-backup/issues/1199).
 - disable free space check when using `--hardlink-exists-files`, fix [1198](https://github.com/Altinity/clickhouse-backup/issues/1198)
+- add chmod 0640 for `--hardlink-exists-files` during `download` and `restore_remote`, fix [1164](https://github.com/Altinity/clickhouse-backup/issues/1164)
 
 # v2.6.33
 BUG FIXES
diff --git a/pkg/backup/download.go b/pkg/backup/download.go
@@ -1288,6 +1288,9 @@ func (b *Backuper) makePartHardlinks(exists, new string) error {
 				return err
 			}
 		}
+		if err = os.Chmod(newF, 0640); err != nil {
+			return err
+		}
 		return nil
 	}); walkErr != nil {
 		log.Warn().Msgf("Link recursively %s -> %s return error: %v", new, exists, walkErr)

Original file line number	Diff line number	Diff line change
`@@ -1288,6 +1288,9 @@ func (b *Backuper) makePartHardlinks(exists, new string) error {`
`1288`	`1288`	`return err`
`1289`	`1289`	`}`
`1290`	`1290`	`}`
	`1291`	`+ if err = os.Chmod(newF, 0640); err != nil {`
	`1292`	`+ return err`
	`1293`	`+ }`
`1291`	`1294`	`return nil`
`1292`	`1295`	`}); walkErr != nil {`
`1293`	`1296`	`log.Warn().Msgf("Link recursively %s -> %s return error: %v", new, exists, walkErr)`