Fix linter

Author: evgeniy-scherbina

Makefile

@@ -62,7 +62,7 @@ e2e-test:
 		echo "E2E tests require Linux platform. Current platform: $$(uname)"; \
 		exit 1; \
 	fi
-	sudo $(shell which go) test -v -race ./e2e_tests
+	sudo $(shell which go) test -v -race ./e2e_tests -count=1
 	@echo "✓ E2E tests passed!"
 
 # Run tests with coverage (needs sudo for E2E tests)

cli/cli.go

@@ -110,7 +110,6 @@ func Run(ctx context.Context, config Config, args []string) error {
 		// Program to run
 		bin := args[0]
 		args = args[1:]
-		env := os.Environ()
 
 		cmd := exec.Command(bin, args...)
 		cmd.Stdin = os.Stdin
@@ -122,20 +121,7 @@ func Run(ctx context.Context, config Config, args []string) error {
 			return err
 		}
 		log.Printf("successfully run %s: %s", bin, "output")
-		
-		return nil
-
-		//log.Printf("bin: %v, args: %v\n", bin, args)
-		//log.Printf("env: %v\n", os.Environ())
-		// syscall.Exec replaces the current process image
-		// with the new program, so nothing after this call runs.
-		if err := syscall.Exec(bin, args, env); err != nil {
-			log.Printf("failed to exec child process: %v", err)
-			return fmt.Errorf("failed to exec child process: %v", err)
-		}
 
-		// This line is never reached if Exec succeeds.
-		log.Println("done")
 		return nil
 	}
 

e2e_tests/boundary_integration_test.go

@@ -38,70 +38,6 @@ func findProjectRoot(t *testing.T) string {
 	}
 }
 
-// getNamespaceName gets the single network namespace name
-// Fails if there are 0 or multiple namespaces
-func getNamespaceName(t *testing.T) string {
-	cmd := exec.Command("ip", "netns", "list")
-	output, err := cmd.Output()
-	require.NoError(t, err, "Failed to list network namespaces")
-
-	lines := strings.Split(string(output), "\n")
-	var namespaces []string
-
-	for _, line := range lines {
-		line = strings.TrimSpace(line)
-		if line != "" {
-			// Extract namespace name (first field)
-			parts := strings.Fields(line)
-			if len(parts) > 0 {
-				namespaces = append(namespaces, parts[0])
-			}
-		}
-	}
-
-	require.Len(t, namespaces, 1, "Expected exactly one network namespace, found %d: %v", len(namespaces), namespaces)
-	return namespaces[0]
-}
-
-//func getChildProcessPID(t *testing.T) int {
-//	// Option 1: Look for processes with CHILD=true
-//	cmd := exec.Command("pgrep", "-f", "CHILD=true")
-//	output, err := cmd.CombinedOutput()
-//	require.NoError(t, err, "output: %v", output)
-//
-//	pidStr := strings.TrimSpace(string(output))
-//	pid, err := strconv.Atoi(pidStr)
-//	require.NoError(t, err)
-//	return pid
-//
-//	// Option 2: Use the boundary process's child PID
-//	// This would require modifying boundary to expose the child PID
-//}
-
-//func getBoundaryProcessPID(t *testing.T) int {
-//	cmd := exec.Command("pgrep", "-f", "boundary-test")
-//	output, err := cmd.Output()
-//	require.NoError(t, err)
-//
-//	pidStr := strings.TrimSpace(string(output))
-//	pid, err := strconv.Atoi(pidStr)
-//	require.NoError(t, err)
-//	return pid
-//}
-//
-//func getChildProcessPID(t *testing.T) int {
-//	boundaryPID := getBoundaryProcessPID(t)
-//
-//	cmd := exec.Command("pgrep", "-P", fmt.Sprintf("%d", boundaryPID))
-//	output, err := cmd.Output()
-//	require.NoError(t, err)
-//
-//	pidStr := strings.TrimSpace(string(output))
-//	pid, err := strconv.Atoi(pidStr)
-//	require.NoError(t, err)
-//	return pid
-//}
-
 func getChildProcessPID(t *testing.T) int {
 	cmd := exec.Command("pgrep", "-f", "boundary-test", "-n")
 	output, err := cmd.Output()

jail/linux.go

@@ -248,46 +248,6 @@ func (l *LinuxJail) Close() error {
 	return nil
 }
 
-// createNamespace creates a new network namespace
-func (l *LinuxJail) createNamespace() error {
-	cmd := exec.Command("ip", "netns", "add", l.namespace)
-	err := cmd.Run()
-	if err != nil {
-		return fmt.Errorf("failed to create namespace: %v", err)
-	}
-	return nil
-}
-
-// setupDNS configures DNS resolution for the namespace
-// This ensures reliable DNS resolution by using public DNS servers
-// instead of relying on the host's potentially complex DNS configuration
-func (l *LinuxJail) setupDNS() error {
-	// Always create namespace-specific resolv.conf with reliable public DNS servers
-	// This avoids issues with systemd-resolved, Docker DNS, and other complex setups
-	netnsEtc := fmt.Sprintf("/etc/netns/%s", l.namespace)
-	err := os.MkdirAll(netnsEtc, 0755)
-	if err != nil {
-		return fmt.Errorf("failed to create /etc/netns directory: %v", err)
-	}
-
-	// Write custom resolv.conf with multiple reliable public DNS servers
-	resolvConfPath := fmt.Sprintf("%s/resolv.conf", netnsEtc)
-	dnsConfig := `# Custom DNS for network namespace
-nameserver 8.8.8.8
-nameserver 8.8.4.4
-nameserver 1.1.1.1
-nameserver 9.9.9.9
-options timeout:2 attempts:2
-`
-	err = os.WriteFile(resolvConfPath, []byte(dnsConfig), 0644)
-	if err != nil {
-		return fmt.Errorf("failed to write namespace-specific resolv.conf: %v", err)
-	}
-
-	l.logger.Debug("DNS setup completed")
-	return nil
-}
-
 // setupIptables configures iptables rules for comprehensive TCP traffic interception
 func (l *LinuxJail) configureIptables() error {
 	// Enable IP forwarding