NimsForest Issues

Replace Python in cloud-init with jq or pure shell
proposed bug Priority: high Project: hetznertreehouse Reporter: 21 Mar 2026 16:04

Description

The hetznertreehouse cloud-init script uses `python3 -c "import sys, json..."` to parse the JSON response from landconfigregistry and write config files. This violates the no-Python rule.

Location: `hetznertreehouse/internal/provisioner/cloudinit.go` lines 49-61

The script fetches config from landconfigregistry (`/api/v1/roles/<role>/render?org_slug=...`) and uses Python to extract fields from the JSON response. Replace with `jq` (add to cloud-init apt install list) or a pure shell approach.

This is on the critical path for organization provisioning — every new land server runs this cloud-init on first boot.

Comments (1)

nebula 21 Mar 2026 16:16
Grooming: no field changes needed — category (bug), priority (high), and project (hetznertreehouse) are all correct

Nebula's reasoning: Well-written issue with clear problem statement, exact file location, and rationale. Category bug is correct — using Python violates a firm project constraint, making this a defect not just a style preference. Priority high is warranted since this runs on the critical path for every new land server provisioning. Description already includes actionable remediation (jq or pure shell).