chore: do not throw error on chmod failure in embedded mode

flanksource · Oct 11, 2023 · 3a88324 · 3a88324
1 parent 83a32a0
commit 3a88324
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/pkg/db/init.go b/pkg/db/init.go
@@ -56,10 +56,7 @@ func IsConnected() bool {
 
 func embeddedDB() error {
 	embeddedPath := strings.TrimSuffix(strings.TrimPrefix(ConnectionString, "embedded://"), "/")
-	err := os.Chmod(embeddedPath, 0750)
-	if err != nil {
-		logger.Warnf("Error changing permission of dataPath: %v, Error: %v", embeddedPath, err)
-	}
+	_ = os.Chmod(embeddedPath, 0750)
 
 	logger.Infof("Starting embedded postgres server at %s", embeddedPath)
 
@@ -72,8 +69,7 @@ func embeddedDB() error {
 		Username("postgres").Password("postgres").
 		Database("canary"))
 	ConnectionString = "postgres://postgres:postgres@localhost:6432/canary?sslmode=disable"
-	err = PostgresServer.Start()
-	if err != nil {
+	if err := PostgresServer.Start(); err != nil {
 		return fmt.Errorf("error starting embedded postgres: %v", err)
 	}
 	return nil