Fix detection if the collector is running as service on Windows

.chloggen/fix-running-as-service-detection.yaml

@@ -0,0 +1,25 @@
+# Use this changelog template to create an entry for release notes.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: bug_fix
+
+# The name of the component, or a single word describing the area of concern, (e.g. otlpreceiver)
+component: cmd/otelcorecol
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: Fix the code detecting if the collector is running as a service on Windows.
+
+# One or more tracking issues or pull requests related to the change
+issues: [7350]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext: This fix should make setting the `NO_WINDOWS_SERVICE` environment variable unnecessary.
+
+# Optional: The change log or logs in which this entry should be included.
+# e.g. '[user]' or '[user, api]'
+# Include 'user' if the change is relevant to end users.
+# Include 'api' if there is a change to a library API.
+# Default: '[user]'
+change_logs: [user]

cmd/builder/internal/builder/templates/main_windows.go.tmpl

@@ -8,39 +8,30 @@ package main
 import (
 	"fmt"
 	"os"
+	"golang.org/x/sys/windows"
 	"golang.org/x/sys/windows/svc"
 	"go.opentelemetry.io/collector/otelcol"
 )
 
 func run(params otelcol.CollectorSettings) error {
-	if useInteractiveMode, err := checkUseInteractiveMode(); err != nil {
-		return err
-	} else if useInteractiveMode {
-		return runInteractive(params)
-	} else {
-		return runService(params)
-	}
-}
-
-func checkUseInteractiveMode() (bool, error) {
-	// If environment variable NO_WINDOWS_SERVICE is set with any value other
-	// than 0, use interactive mode instead of running as a service. This should
-	// be set in case running as a service is not possible or desired even
-	// though the current session is not detected to be interactive
+	// There shouldn't be any reason to use NO_WINDOWS_SERVICE anymore, but,
+	// keeping it as a forcing mechanism or if someone is concerned about
+	// the cost of attempting to run as a service before falling back to
+	// interactive mode.
 	if value, present := os.LookupEnv("NO_WINDOWS_SERVICE"); present && value != "0" {
-		return true, nil
-	}
-
-	isInteractiveSession, err := svc.IsAnInteractiveSession()
-	if err != nil {
-		return false, fmt.Errorf("failed to determine if we are running in an interactive session: %w", err)
+		return runInteractive(params)
 	}
-	return isInteractiveSession, nil
-}
 
-func runService(params otelcol.CollectorSettings) error {
-	// do not need to supply service name when startup is invoked through Service Control Manager directly
+	// No need to supply service name when startup is invoked through
+	// the Service Control Manager directly.
 	if err := svc.Run("", otelcol.NewSvcHandler(params)); err != nil {
+		errno, ok := err.(syscall.Errno)
+		if ok && errno == windows.ERROR_FAILED_SERVICE_CONTROLLER_CONNECT {
+			// Per https://learn.microsoft.com/en-us/windows/win32/api/winsvc/nf-winsvc-startservicectrldispatchera#return-value
+			// this means that the process is not running as a service, so run interactively.
+			return runInteractive(params)
+		}
+
 		return fmt.Errorf("failed to start collector server: %w", err)
 	}
 

docs/troubleshooting.md

@@ -283,7 +283,7 @@ configuration issue. This could be due to a firewall, DNS, or proxy
 issue. Note that the Collector does have
 [proxy support](https://github.com/open-telemetry/opentelemetry-collector/tree/main/exporter#proxy-support).
 
-### Startup failing in Windows Docker containers
+### Startup failing in Windows Docker containers (v0.90.1 and earlier)
 
 The process may fail to start in a Windows Docker container with the following
 error: `The service process could not connect to the service controller`. In