Bugfix/2307 change error message in lakefs setup (#2435)

* Added verification of whether the system has already been set up and changed error message when trying to initialize lakefs more than once in the UI to be descriptive

cmd/lakefs/cmd/setup.go

@@ -57,6 +57,16 @@ var setupCmd = &cobra.Command{
 		cloudMetadataProvider := stats.BuildMetadataProvider(logging.Default(), cfg)
 		metadata := stats.NewMetadata(ctx, logging.Default(), cfg.GetBlockstoreType(), metadataManager, cloudMetadataProvider)
 
+		initialized, err := metadataManager.IsInitialized(ctx)
+		if err != nil {
+			fmt.Printf("Setup failed: %s\n", err)
+			os.Exit(1)
+		}
+		if initialized {
+			fmt.Printf("Setup is already complete.\n")
+			os.Exit(1)
+		}
+
 		credentials, err := auth.CreateInitialAdminUserWithKeys(ctx, authService, metadataManager, userName, &accessKeyID, &secretAccessKey)
 		if err != nil {
 			fmt.Printf("Failed to setup admin user: %s\n", err)

webui/src/lib/api/index.js

@@ -651,7 +651,7 @@ class Setup {
             case 200:
                 return response.json();
             case 409:
-                throw new Error('Conflict');
+                throw new Error('Setup is already complete.');
             default:
                 throw new Error('Unknown');
         }