Merge branch 'main' into BG

Author: EnriqueL8

doc-site/docs/gettingstarted/setup_env.md

@@ -84,7 +84,7 @@ Global Flags:
   -d, --database string               Database type to use. Options are: [sqlite3 postgres] (default "sqlite3")
   -e, --external int                  Manage a number of FireFly core processes outside of the docker-compose stack - useful for development and debugging
   -p, --firefly-base-port int         Mapped port base of FireFly core API (1 added for each member) (default 5000)
-      --ipfs-mode string              Set the mode in which IFPS operates. Options are: [private public] (default "private")
+      --ipfs-mode string              Set the mode in which IPFS operates. Options are: [private public] (default "private")
   -m, --manifest string               Path to a manifest.json file containing the versions of each FireFly microservice to use. Overrides the --release flag.
       --multiparty                    Enable or disable multiparty mode (default true)
       --node-name stringArray         Node name

doc-site/docs/reference/config.md

@@ -450,7 +450,7 @@ title: Configuration Reference
 |Key|Description|Type|Default Value|
 |---|-----------|----|-------------|
 |address|The IP address on which the metrics HTTP API should listen|`int`|`127.0.0.1`
-|enabled|Enables the metrics API|`boolean`|`true`
+|enabled|Enables the metrics API|`boolean`|`false`
 |metricsPath|The path from which to serve the Prometheus metrics|`string`|`/metrics`
 |port|The port on which the metrics HTTP API should listen|`int`|`6000`
 |publicURL|The fully qualified public URL for the metrics API. This is used for building URLs in HTTP responses and in OpenAPI Spec generation|URL `string`|`<nil>`

internal/apiserver/metrics_server.go

@@ -32,6 +32,6 @@ func initDeprecatedMetricsConfig(config config.Section) {
 }
 
 func initMonitoringConfig(config config.Section) {
-	config.AddKnownKey(Enabled, true)
+	config.AddKnownKey(Enabled, false)
 	config.AddKnownKey(MetricsPath, "/metrics")
 }

internal/apiserver/server.go

@@ -115,17 +115,18 @@ func (as *apiServer) Serve(ctx context.Context, mgr namespace.Manager) (err erro
 	} else if config.GetBool(coreconfig.LegacyAdminEnabled) {
 		log.L(ctx).Warnf("Your config includes an 'admin' section, which should be renamed to 'spi' - SPI server will not be enabled until this is corrected")
 	}
-	serverName := "metrics"
-	mConfig := deprecatedMetricsConfig
-	if as.monitoringEnabled {
-		serverName = "monitoring"
-		mConfig = monitoringConfig
-	}
-
 	if as.deprecatedMetricsEnabled || as.monitoringEnabled {
-		monitoringServer, err := httpserver.NewHTTPServer(ctx, serverName, as.createMonitoringMuxRouter(), metricsErrChan, mConfig, corsConfig, &httpserver.ServerOptions{
-			MaximumRequestTimeout: as.apiMaxTimeout,
-		})
+		var monitoringServer httpserver.HTTPServer
+		var err error
+		if as.monitoringEnabled {
+			monitoringServer, err = httpserver.NewHTTPServer(ctx, "monitoring", as.createMonitoringMuxRouter(), metricsErrChan, monitoringConfig, corsConfig, &httpserver.ServerOptions{
+				MaximumRequestTimeout: as.apiMaxTimeout,
+			})
+		} else {
+			monitoringServer, err = httpserver.NewHTTPServer(ctx, "metrics", as.createMonitoringMuxRouter(), metricsErrChan, deprecatedMetricsConfig, corsConfig, &httpserver.ServerOptions{
+				MaximumRequestTimeout: as.apiMaxTimeout,
+			})
+		}
 		if err != nil {
 			return err
 		}