Skip to content

Commit 717a001

Browse files
committed
docs: Fix remaining :rw mount references to :ro
Updates all remaining references to read-write socket mounts in TEMPERATURE_MONITORING.md to use read-only (:ro) mounts for security. Changes: - Manual installation section - Docker-only responsibilities section - Ansible playbook example All socket mounts should be :ro to prevent container tampering.
1 parent d787d33 commit 717a001

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

docs/TEMPERATURE_MONITORING.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ When you need to provision the proxy yourself (for example via your own automati
219219

220220
7. **Expose the socket to Pulse**
221221
- **Proxmox LXC:** append `lxc.mount.entry: /run/pulse-sensor-proxy run/pulse-sensor-proxy none bind,create=dir 0 0` to `/etc/pve/lxc/<CTID>.conf` and restart the container.
222-
- **Docker:** bind mount `/run/pulse-sensor-proxy` into the container (`- /run/pulse-sensor-proxy:/run/pulse-sensor-proxy:rw`).
222+
- **Docker:** bind mount `/run/pulse-sensor-proxy` into the container (`- /run/pulse-sensor-proxy:/run/pulse-sensor-proxy:ro`).
223223

224224
After the container restarts, the backend will automatically use the proxy. To refresh SSH keys on cluster nodes (e.g., after adding a new node), SSH to your Proxmox host and re-run the setup script: `curl -fsSL https://get.pulsenode.com/install-proxy.sh | bash -s -- --ctid <your-container-id>`
225225

@@ -294,7 +294,7 @@ If the node is part of a Proxmox cluster, the script will detect other members a
294294
> **Note:** For LXC deployments, the setup script handles all of this automatically. This section applies to **Docker deployments only**.
295295

296296
- Run the host installer (`install-sensor-proxy.sh --standalone`) on the Proxmox machine that hosts Pulse to install and maintain the `pulse-sensor-proxy` service
297-
- Add the bind mount to your docker-compose.yml: `- /run/pulse-sensor-proxy:/run/pulse-sensor-proxy:rw`
297+
- Add the bind mount to your docker-compose.yml: `- /run/pulse-sensor-proxy:/run/pulse-sensor-proxy:ro`
298298
- Re-run the host installer if the service or socket disappears after a host upgrade or configuration cleanup; the installer is idempotent
299299
- The installer ships a self-heal timer (`pulse-sensor-proxy-selfheal.timer`) that restarts or reinstalls the proxy if it ever goes missing; leave it enabled for automatic recovery
300300
- Hot dev builds warn when only a container-local proxy socket is present, signaling that the host proxy needs to be reinstalled before temperatures will flow back into Pulse
@@ -611,7 +611,7 @@ install-sensor-proxy.sh [OPTIONS]
611611
marker: "# {mark} ANSIBLE MANAGED - Sensor Proxy"
612612
insertafter: "volumes:"
613613
block: |
614-
- /run/pulse-sensor-proxy:/run/pulse-sensor-proxy:rw
614+
- /run/pulse-sensor-proxy:/run/pulse-sensor-proxy:ro
615615
notify: restart pulse container
616616

617617
handlers:

0 commit comments

Comments
 (0)