add proxmox repro test #345

Merged
kiara merged 1 commit from kiara/fediversity:proxmox-test into main 2025-09-01 14:31:52 +02:00
Owner

closes #325

closes #325
kiara force-pushed proxmox-test from 01486ec07a to 3ccdf55054 2025-06-08 13:40:18 +02:00 Compare
kiara force-pushed proxmox-test from 3ccdf55054 to d0eed167a0 2025-06-08 13:46:17 +02:00 Compare
kiara force-pushed proxmox-test from d0eed167a0 to f8c2b08e19 2025-06-08 14:09:33 +02:00 Compare
kiara force-pushed proxmox-test from f8c2b08e19 to 1865ca3c08 2025-06-08 17:49:57 +02:00 Compare
kiara force-pushed proxmox-test from 1865ca3c08 to f52f3cb50c 2025-06-08 20:39:55 +02:00 Compare
kiara force-pushed proxmox-test from f52f3cb50c to b03452866f 2025-06-08 20:40:20 +02:00 Compare
kiara force-pushed proxmox-test from b03452866f to 2055fa9b21 2025-06-08 20:59:08 +02:00 Compare
kiara force-pushed proxmox-test from 2055fa9b21 to 123f25e228 2025-06-08 20:59:29 +02:00 Compare
kiara force-pushed proxmox-test from 123f25e228 to f68a87c856 2025-06-09 09:47:10 +02:00 Compare
kiara force-pushed proxmox-test from f68a87c856 to f88ca6e646 2025-08-02 22:32:41 +02:00 Compare
kiara force-pushed proxmox-test from f88ca6e646 to 062110bc93 2025-08-02 23:15:44 +02:00 Compare
kiara changed title from wip: add proxmox repro test to add proxmox repro test 2025-08-02 23:18:13 +02:00
kiara force-pushed proxmox-test from 062110bc93 to 8e538d2a5e 2025-08-03 08:29:42 +02:00 Compare
kiara force-pushed proxmox-test from 8e538d2a5e to 1a35a5b2f0 2025-08-03 08:32:53 +02:00 Compare
kiara force-pushed proxmox-test from 1a35a5b2f0 to 5db609f0dd 2025-08-03 10:24:04 +02:00 Compare
kiara force-pushed proxmox-test from 5db609f0dd to 6d0ad47a25 2025-08-03 10:26:27 +02:00 Compare
Niols approved these changes 2025-08-11 12:45:17 +02:00
Dismissed
Niols left a comment
Owner

This looks like a pretty good first shot, tbh, and CI seems happy. It's been a while, so I don't remember: why haven't we merged this?

This looks like a pretty good first shot, tbh, and CI seems happy. It's been a while, so I don't remember: why haven't we merged this?
.envrc Outdated
@ -11,0 +12,4 @@
# TODO: how to make this work for a test?
export NIX_CONFIG="
extra-substituters = https://cache.saumon.network/proxmox-nixos
trusted-public-keys = proxmox-nixos:D9RYSWpQQC/msZUWphOY2I5RLH5Dd6yQcaHIuug7dWM=
Owner

Flakes provide this functionality, and we rely on flakes for interface and CI, so I'd say we should just add it there and have both the CI and direnv rely on flakes (or even better, give the user a choice, and not have the .envrc in the repo).

Flakes provide this functionality, and we rely on flakes for interface and CI, so I'd say we should just add it there and have both the CI and direnv rely on flakes (or even better, give the user a choice, and not have the `.envrc` in the repo).
kiara marked this conversation as resolved
@ -0,0 +5,4 @@
}:
let
pkgs = import sources.nixpkgs-stable {
Owner

Hum, why do we now have nixpkgs and nixpkgs-stable?

Hum, why do we now have nixpkgs and nixpkgs-stable?
Author
Owner

this i documented 8 lines down

this i documented 8 lines down
@ -0,0 +8,4 @@
minimalIso = pkgs.fetchurl {
url = "https://releases.nixos.org/nixos/24.05/nixos-24.05.7139.bcba2fbf6963/nixos-minimal-24.05.7139.bcba2fbf6963-x86_64-linux.iso";
hash = "sha256-plre/mIHdIgU4xWU+9xErP+L4i460ZbcKq8iy2n4HT8=";
};
Owner

This looks horrendous; can it not go into npins?

This looks horrendous; can it not go into npins?
Author
Owner

this seems not supported yet - i now added a comment to mention that.

this seems [not supported yet](https://github.com/andir/npins/issues/163) - i now added a comment to mention that.
kiara force-pushed proxmox-test from 6d0ad47a25 to 7ba1812083 2025-08-13 13:13:57 +02:00 Compare
kiara dismissed Niols's review 2025-08-13 13:13:57 +02:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

kiara force-pushed proxmox-test from 7ba1812083 to 0a96282607 2025-08-13 13:26:05 +02:00 Compare
Author
Owner

why haven't we merged this?

i only recently got this to actually work, so before it'd just been a WIP branch still

> why haven't we merged this? i only recently got this to actually work, so before it'd just been a WIP branch still
kiara force-pushed proxmox-test from 0a96282607 to c419a76507 2025-09-01 14:07:55 +02:00 Compare
kiara merged commit f954ed5389 into main 2025-09-01 14:31:52 +02:00
kiara referenced this pull request from a commit 2025-09-01 14:31:53 +02:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: fediversity/fediversity#345
No description provided.