fix(gc-hosts): use pct/qm list output directly as hostnames #106

2026-07-29T04:02:47Z

beatzaplenty commented

2026-07-29 04:02:47 +00:00

The flake hostname map approach was wrong: create-proxmox-resource.sh names
every guest after its NixOS hostname (pct create --hostname ,
qm create --name ), so pct/qm list already output the final
hostname directly. Looking those names up in a map keyed by flake target names
(lxc-docker, proxmox-server, ...) found nothing and the dynamic list was
always empty.

Drop the flake eval entirely; use the pct/qm list output as-is, filtering
only nix-cache and already-seen entries.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com

The flake hostname map approach was wrong: create-proxmox-resource.sh names every guest after its NixOS hostname (pct create --hostname <nixos-hostname>, qm create --name <nixos-hostname>), so pct/qm list already output the final hostname directly. Looking those names up in a map keyed by flake target names (lxc-docker, proxmox-server, ...) found nothing and the dynamic list was always empty. Drop the flake eval entirely; use the pct/qm list output as-is, filtering only nix-cache and already-seen entries. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

beatzaplenty added 1 commit 2026-07-29 04:02:47 +00:00

fix(gc-hosts): use pct/qm list output directly as hostnames

Check NixOS configurations / eval-hosts (pull_request) Successful in 10m32s

Details

a5308a7ee5

The flake hostname map approach was wrong: create-proxmox-resource.sh names
every guest after its NixOS hostname (pct create --hostname <nixos-hostname>,
qm create --name <nixos-hostname>), so pct/qm list already output the final
hostname directly. Looking those names up in a map keyed by flake target names
(lxc-docker, proxmox-server, ...) found nothing and the dynamic list was
always empty.

Drop the flake eval entirely; use the pct/qm list output as-is, filtering
only nix-cache and already-seen entries.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

beatzaplenty merged commit f3c2965f78 into main

2026-07-29 04:02:51 +00:00

This repo is archived. You cannot comment on pull requests.

1 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: beatzaplenty/nixos#106