Merge pull request #331 from PatchMon/release/1-3-5

fixing host route of version checking for other architectures

agents/proxmox_auto_enroll.sh

@@ -197,40 +197,6 @@ while IFS= read -r line; do
         continue
     fi
 
-    # Check if agent is already installed and working BEFORE enrollment
-    info "  Checking if agent is already configured..."
-    config_check=$(timeout 10 pct exec "$vmid" -- bash -c "
-        if [[ -f /etc/patchmon/config.yml ]] && [[ -f /etc/patchmon/credentials.yml ]]; then
-            if [[ -f /usr/local/bin/patchmon-agent ]]; then
-                # Try to ping using existing configuration
-                if /usr/local/bin/patchmon-agent ping >/dev/null 2>&1; then
-                    echo 'ping_success'
-                else
-                    echo 'ping_failed'
-                fi
-            else
-                echo 'binary_missing'
-            fi
-        else
-            echo 'not_configured'
-        fi
-    " 2>/dev/null </dev/null || echo "error")
-
-    if [[ "$config_check" == "ping_success" ]]; then
-        info "  ✓ Host already enrolled and agent ping successful - skipping enrollment"
-        ((skipped_count++)) || true
-        echo ""
-        continue
-    elif [[ "$config_check" == "ping_failed" ]]; then
-        warn "  ⚠ Agent configuration exists but ping failed - will re-enroll and reinstall"
-    elif [[ "$config_check" == "binary_missing" ]]; then
-        warn "  ⚠ Config exists but agent binary missing - will re-enroll and reinstall"
-    elif [[ "$config_check" == "not_configured" ]]; then
-        info "  ℹ Agent not yet configured - proceeding with enrollment"
-    else
-        warn "  ⚠ Could not check agent status - proceeding with enrollment"
-    fi
-
     # Call PatchMon auto-enrollment API
     info "  Enrolling $friendly_name in PatchMon..."
     
@@ -264,6 +230,40 @@ while IFS= read -r line; do
 
         info "  ✓ Host enrolled successfully: $api_id"
 
+        # Check if agent is already installed and working
+        info "  Checking if agent is already configured..."
+        config_check=$(timeout 10 pct exec "$vmid" -- bash -c "
+            if [[ -f /etc/patchmon/config.yml ]] && [[ -f /etc/patchmon/credentials.yml ]]; then
+                if [[ -f /usr/local/bin/patchmon-agent ]]; then
+                    # Try to ping using existing configuration
+                    if /usr/local/bin/patchmon-agent ping >/dev/null 2>&1; then
+                        echo 'ping_success'
+                    else
+                        echo 'ping_failed'
+                    fi
+                else
+                    echo 'binary_missing'
+                fi
+            else
+                echo 'not_configured'
+            fi
+        " 2>/dev/null </dev/null || echo "error")
+
+        if [[ "$config_check" == "ping_success" ]]; then
+            info "  ✓ Host already enrolled and agent ping successful - skipping"
+            ((skipped_count++)) || true
+            echo ""
+            continue
+        elif [[ "$config_check" == "ping_failed" ]]; then
+            warn "  ⚠ Agent configuration exists but ping failed - will reinstall"
+        elif [[ "$config_check" == "binary_missing" ]]; then
+            warn "  ⚠ Config exists but agent binary missing - will reinstall"
+        elif [[ "$config_check" == "not_configured" ]]; then
+            info "  ℹ Agent not yet configured - proceeding with installation"
+        else
+            warn "  ⚠ Could not check agent status - proceeding with installation"
+        fi
+
         # Ensure curl is installed in the container
         info "  Checking for curl in container..."
         curl_check=$(timeout 10 pct exec "$vmid" -- bash -c "command -v curl >/dev/null 2>&1 && echo 'installed' || echo 'missing'" 2>/dev/null </dev/null || echo "error")

backend/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "patchmon-backend",
-	"version": "1.3.6",
+	"version": "1.3.5",
 	"description": "Backend API for Linux Patch Monitoring System",
 	"license": "AGPL-3.0",
 	"main": "src/server.js",

backend/prisma/schema.prisma

@@ -103,7 +103,6 @@ model hosts {
   gateway_ip              String?
   hostname                String?
   kernel_version          String?
-  installed_kernel_version String?
   load_average             Json?
   network_interfaces      Json?
   ram_installed           Int?

backend/src/routes/hostRoutes.js

@@ -12,6 +12,7 @@ const {
 } = require("../middleware/permissions");
 const { queueManager, QUEUE_NAMES } = require("../services/automation");
 const { pushIntegrationToggle, isConnected } = require("../services/agentWs");
+const agentVersionService = require("../services/agentVersionService");
 const { compareVersions } = require("../services/automation/shared/utils");
 
 const router = express.Router();
@@ -506,10 +507,6 @@ router.post(
 			.optional()
 			.isString()
 			.withMessage("Kernel version must be a string"),
-		body("installedKernelVersion")
-			.optional()
-			.isString()
-			.withMessage("Installed kernel version must be a string"),
 		body("selinuxStatus")
 			.optional()
 			.isIn(["enabled", "disabled", "permissive"])
@@ -591,8 +588,6 @@ router.post(
 			// System Information
 			if (req.body.kernelVersion)
 				updateData.kernel_version = req.body.kernelVersion;
-			if (req.body.installedKernelVersion)
-				updateData.installed_kernel_version = req.body.installedKernelVersion;
 			if (req.body.selinuxStatus)
 				updateData.selinux_status = req.body.selinuxStatus;
 			if (req.body.systemUptime)

frontend/env.example

@@ -6,5 +6,5 @@ VITE_API_URL=http://localhost:3001/api/v1
 
 # Application Metadata
 VITE_APP_NAME=PatchMon
-VITE_APP_VERSION=1.3.6
+VITE_APP_VERSION=1.3.5
 

frontend/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "patchmon-frontend",
 	"private": true,
-	"version": "1.3.6",
+	"version": "1.3.5",
 	"license": "AGPL-3.0",
 	"type": "module",
 	"scripts": {

frontend/src/pages/HostDetail.jsx

@@ -1013,25 +1013,29 @@ const HostDetail = () => {
 												</div>
 											)}
 
-											{host.kernel_version && (
-												<div>
-													<p className="text-xs text-secondary-500 dark:text-secondary-300">
-														Running Kernel
-													</p>
-													<p className="font-medium text-secondary-900 dark:text-white font-mono text-sm break-all">
-														{host.kernel_version}
-													</p>
-												</div>
-											)}
-
-											{host.installed_kernel_version && (
-												<div>
-													<p className="text-xs text-secondary-500 dark:text-secondary-300">
-														Installed Kernel
-													</p>
-													<p className="font-medium text-secondary-900 dark:text-white font-mono text-sm break-all">
-														{host.installed_kernel_version}
-													</p>
+											{(host.kernel_version ||
+												host.installed_kernel_version) && (
+												<div className="flex flex-col gap-2">
+													{host.kernel_version && (
+														<div>
+															<p className="text-xs text-secondary-500 dark:text-secondary-300 mb-1">
+																Running Kernel
+															</p>
+															<p className="font-medium text-secondary-900 dark:text-white font-mono text-sm break-all">
+																{host.kernel_version}
+															</p>
+														</div>
+													)}
+													{host.installed_kernel_version && (
+														<div>
+															<p className="text-xs text-secondary-500 dark:text-secondary-300 mb-1">
+																Installed Kernel
+															</p>
+															<p className="font-medium text-secondary-900 dark:text-white font-mono text-sm break-all">
+																{host.installed_kernel_version}
+															</p>
+														</div>
+													)}
 												</div>
 											)}
 

frontend/src/pages/Hosts.jsx

@@ -248,6 +248,7 @@ const Hosts = () => {
 		const showFiltersParam = searchParams.get("showFilters");
 		const osFilterParam = searchParams.get("osFilter");
 		const groupParam = searchParams.get("group");
+		const rebootParam = searchParams.get("reboot");
 
 		if (filter === "needsUpdates") {
 			setShowFilters(true);
@@ -649,27 +650,11 @@ const Hosts = () => {
 		);
 	};
 
-	const handleSelectAll = (hostsToSelect) => {
-		const hostIdsToSelect = hostsToSelect.map((host) => host.id);
-		const allSelected = hostIdsToSelect.every((id) =>
-			selectedHosts.includes(id),
-		);
-		if (allSelected) {
-			// Deselect all hosts in this group
-			setSelectedHosts((prev) =>
-				prev.filter((id) => !hostIdsToSelect.includes(id)),
-			);
+	const handleSelectAll = () => {
+		if (selectedHosts.length === hosts.length) {
+			setSelectedHosts([]);
 		} else {
-			// Select all hosts in this group (merge with existing selections)
-			setSelectedHosts((prev) => {
-				const newSelection = [...prev];
-				hostIdsToSelect.forEach((id) => {
-					if (!newSelection.includes(id)) {
-						newSelection.push(id);
-					}
-				});
-				return newSelection;
-			});
+			setSelectedHosts(hosts.map((host) => host.id));
 		}
 	};
 
@@ -1234,7 +1219,7 @@ const Hosts = () => {
 		navigate("/hosts", { replace: true });
 	};
 
-	const _handleUpToDateClick = () => {
+	const handleUpToDateClick = () => {
 		// Filter to show only up-to-date hosts
 		setStatusFilter("active");
 		setShowFilters(true);
@@ -1671,14 +1656,11 @@ const Hosts = () => {
 																		{column.id === "select" ? (
 																			<button
 																				type="button"
-																				onClick={() =>
-																					handleSelectAll(groupHosts)
-																				}
+																				onClick={handleSelectAll}
 																				className="flex items-center gap-2 hover:text-secondary-700"
 																			>
-																				{groupHosts.every((host) =>
-																					selectedHosts.includes(host.id),
-																				) ? (
+																				{selectedHosts.length ===
+																				groupHosts.length ? (
 																					<CheckSquare className="h-4 w-4" />
 																				) : (
 																					<Square className="h-4 w-4" />

frontend/src/pages/settings/Integrations.jsx

@@ -1586,7 +1586,7 @@ const Integrations = () => {
 										<div className="flex items-center gap-2">
 											<input
 												type="text"
-												value={`curl ${curl_flags} "${getEnrollmentUrl()}" | ${selected_script_type === "proxmox-lxc" ? "bash" : "sh"}`}
+												value={`curl ${curl_flags} "${getEnrollmentUrl()}" | sh`}
 												readOnly
 												className="flex-1 px-3 py-2 border border-secondary-300 dark:border-secondary-600 rounded-md bg-secondary-50 dark:bg-secondary-900 text-secondary-900 dark:text-white font-mono text-xs"
 											/>
@@ -1594,7 +1594,7 @@ const Integrations = () => {
 												type="button"
 												onClick={() =>
 													copy_to_clipboard(
-														`curl ${curl_flags} "${getEnrollmentUrl()}" | ${selected_script_type === "proxmox-lxc" ? "bash" : "sh"}`,
+														`curl ${curl_flags} "${getEnrollmentUrl()}" | sh`,
 														"enrollment-command",
 													)
 												}

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "patchmon",
-	"version": "1.3.6",
+	"version": "1.3.5",
 	"description": "Linux Patch Monitoring System",
 	"license": "AGPL-3.0",
 	"private": true,