diff --git a/frontend-modern/src/components/Settings/UnifiedAgents.tsx b/frontend-modern/src/components/Settings/UnifiedAgents.tsx index 343cf61..92440fa 100644 --- a/frontend-modern/src/components/Settings/UnifiedAgents.tsx +++ b/frontend-modern/src/components/Settings/UnifiedAgents.tsx @@ -107,9 +107,6 @@ export const UnifiedAgents: Component = () => { const [lookupResult, setLookupResult] = createSignal(null); const [lookupError, setLookupError] = createSignal(null); const [lookupLoading, setLookupLoading] = createSignal(false); - const [enableDocker, setEnableDocker] = createSignal(false); // Default to false - user must opt-in for Docker monitoring - const [enableKubernetes, setEnableKubernetes] = createSignal(false); // Default to false - user must opt-in for Kubernetes monitoring - const [enableProxmox, setEnableProxmox] = createSignal(false); // For Proxmox VE/PBS nodes - creates API token and auto-registers const [insecureMode, setInsecureMode] = createSignal(false); // For self-signed certificates (issue #806) createEffect(() => { @@ -237,9 +234,6 @@ export const UnifiedAgents: Component = () => { } }; - const getDockerFlag = () => enableDocker() ? ' --enable-docker' : ''; - const getKubernetesFlag = () => enableKubernetes() ? ' --enable-kubernetes' : ''; - const getProxmoxFlag = () => enableProxmox() ? ' --enable-proxmox' : ''; const getInsecureFlag = () => insecureMode() ? ' --insecure' : ''; const getCurlInsecureFlag = () => insecureMode() ? '-k' : ''; @@ -499,56 +493,23 @@ export const UnifiedAgents: Component = () => {

Installation commands

-

The installer auto-detects Docker, Kubernetes, and Proxmox. Use these to force enable/disable:

+

The installer auto-detects Docker, Kubernetes, and Proxmox on the target machine.

-
- - - - -
- -
-

Proxmox force-enable

-

- The agent will create a pulse-monitor user and API token on the Proxmox node, - then register it with Pulse automatically. Includes temperature monitoring. -

+ +
+ TLS verification disabled — commands will skip certificate checks (for self-signed certs).
+
@@ -568,29 +529,8 @@ export const UnifiedAgents: Component = () => { if (insecureMode() && cmd.includes('curl -fsSL')) { cmd = cmd.replace('curl -fsSL', 'curl -kfsSL'); } - // For bash scripts (not PowerShell), append flags directly + // For bash scripts (not PowerShell), append insecure flag const isBashScript = !cmd.includes('$env:') && !cmd.includes('irm'); - // Append docker flag if enabled - if (enableDocker()) { - if (cmd.includes('$env:PULSE_URL')) { - cmd = `$env:PULSE_ENABLE_DOCKER="true"; ` + cmd; - } else if (isBashScript) { - cmd += getDockerFlag(); - } - } - // Append kubernetes flag if enabled - if (enableKubernetes()) { - if (cmd.includes('$env:PULSE_URL')) { - cmd = `$env:PULSE_ENABLE_KUBERNETES="true"; ` + cmd; - } else if (isBashScript) { - cmd += getKubernetesFlag(); - } - } - // Append proxmox flag if enabled (Linux only - Proxmox doesn't run on Windows/macOS) - if (enableProxmox() && isBashScript) { - cmd += getProxmoxFlag(); - } - // Append insecure flag for agent if enabled if (insecureMode() && isBashScript) { cmd += getInsecureFlag(); } @@ -706,31 +646,20 @@ export const UnifiedAgents: Component = () => {
- Advanced options (uninstall & manual install) + Troubleshooting
-

Uninstall

-
- - {getUninstallCommand()} - - -
-

- Stops the agent, removes the binary, the systemd unit, and related files. +

Auto-detection not working?

+

+ If Docker, Kubernetes, or Proxmox isn't detected automatically, add these flags to the install command:

+
    +
  • --enable-docker — Force enable Docker/Podman monitoring
    • +
  • --enable-kubernetes — Force enable Kubernetes monitoring
    • +
  • --enable-proxmox — Force enable Proxmox integration (creates API token)
    • +
  • --disable-docker — Skip Docker even if detected
    • +
diff --git a/frontend-modern/src/components/Settings/__tests__/UnifiedAgents.test.tsx b/frontend-modern/src/components/Settings/__tests__/UnifiedAgents.test.tsx index 38803a2..fac1ec1 100644 --- a/frontend-modern/src/components/Settings/__tests__/UnifiedAgents.test.tsx +++ b/frontend-modern/src/components/Settings/__tests__/UnifiedAgents.test.tsx @@ -362,7 +362,7 @@ describe('UnifiedAgents platform commands', () => { expect(screen.getByText('Install on Windows')).toBeInTheDocument(); }); - it('includes docker flag when docker monitoring is enabled', async () => { + it('includes insecure flag when TLS skip is enabled', async () => { createTokenMock.mockResolvedValue({ token: 'test-token', record: { @@ -382,18 +382,18 @@ describe('UnifiedAgents platform commands', () => { await waitFor(() => expect(createTokenMock).toHaveBeenCalled(), { interval: 0 }); await waitFor(() => { - expect(screen.getByText('Force Docker')).toBeInTheDocument(); + expect(screen.getByText(/Skip TLS certificate verification/i)).toBeInTheDocument(); }); - // Docker is disabled by default (force off) - const checkbox = screen.getByRole('checkbox', { name: /Force Docker/i }); + // TLS skip is disabled by default + const checkbox = screen.getByRole('checkbox', { name: /Skip TLS certificate verification/i }); expect(checkbox).not.toBeChecked(); // Enable it fireEvent.click(checkbox); expect(checkbox).toBeChecked(); - // Find a copy button and check that the command includes the docker flag + // Find a copy button and check that the command includes the insecure flag const copyButtons = screen.getAllByRole('button', { name: /Copy command/i }); expect(copyButtons.length).toBeGreaterThan(0); });