allow accessing test vms from fedi201, closes #286 #297

Merged
kiara merged 3 commits from kiara/fediversity:panel-vm-access-test-vms into main 2025-04-09 16:58:52 +02:00
Owner
todo: - [ ] [deploy to test machines](https://git.fediversity.eu/Fediversity/Fediversity/issues/273#issuecomment-5644)
kiara force-pushed panel-vm-access-test-vms from b839159130
All checks were successful
/ check-pre-commit (pull_request) Successful in 24s
/ check-website (pull_request) Successful in 9s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m24s
to 3c0d1478cc
All checks were successful
/ check-pre-commit (pull_request) Successful in 24s
/ check-website (pull_request) Successful in 9s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m22s
2025-04-05 13:50:26 +02:00
Compare
kiara force-pushed panel-vm-access-test-vms from 3c0d1478cc
All checks were successful
/ check-pre-commit (pull_request) Successful in 24s
/ check-website (pull_request) Successful in 9s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m22s
to 94420f112f
All checks were successful
/ check-pre-commit (pull_request) Successful in 25s
/ check-website (pull_request) Successful in 10s
/ check-peertube (pull_request) Successful in 21s
/ check-panel (pull_request) Successful in 1m23s
2025-04-05 13:54:59 +02:00
Compare
kiara changed title from WIP: allow accessing test vms from fedi201's machine ssh key, closes #286 to allow accessing test vms from fedi201's machine ssh key, closes #286 2025-04-05 15:10:03 +02:00
kiara changed title from allow accessing test vms from fedi201's machine ssh key, closes #286 to allow accessing test vms from fedi201, closes #286 2025-04-06 11:26:08 +02:00
kiara force-pushed panel-vm-access-test-vms from 0759d7d8c7
All checks were successful
/ check-pre-commit (pull_request) Successful in 24s
/ check-website (pull_request) Successful in 10s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m16s
to aeb4f0cc8b
All checks were successful
/ check-pre-commit (pull_request) Successful in 24s
/ check-website (pull_request) Successful in 10s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m21s
2025-04-06 11:30:13 +02:00
Compare
also add panel key from nixops
All checks were successful
/ check-pre-commit (pull_request) Successful in 24s
/ check-website (pull_request) Successful in 10s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m20s
fb70db5a67
@ -29,0 +33,4 @@
{
nixos.module.users.users.root.openssh.authorizedKeys.keys = [
# allow our panel vm access to the test machines
(import ../keys).panel
Owner

Isn't this duplicate of your change in infra/common/resource.nix?

Isn't this duplicate of your change in `infra/common/resource.nix`?
Author
Owner

good catch, lemme see if i can ditch this one

good catch, lemme see if i can ditch this one
@ -18,0 +19,4 @@
owner = name;
group = name;
mode = "400";
};
Owner

I guess technically we don't care much about group = name if when then give a mode that only touches the user. - future proofness, I suppose.

I guess technically we don't care much about `group = name` if when then give a mode that only touches the user. - future proofness, I suppose.
Author
Owner

thanks, i removed it now

thanks, i removed it now
kiara marked this conversation as resolved
@ -35,3 +35,4 @@
contributors = collectKeys ./contributors;
systems = collectKeys ./systems;
panel = removeTrailingWhitespace (readFile ./panel-ssh-key.pub);
}
Owner

I'm thinking if we enforce the .pub suffix, we could actually rather easily compute this nested record structure from the filesystem directly. But OK, that's really not a priority, and definitely not the topic of this PR.

I'm thinking if we enforce the `.pub` suffix, we could actually rather easily compute this nested record structure from the filesystem directly. But OK, that's really not a priority, and definitely not the topic of this PR.
Author
Owner

i consider keys to be deployment-specific state which should have no place in a code-base that should be as useful to someone from outside our team as to us - so down the line i do wanna get rid of the hard-coded cruft, yeah.

i consider keys to be deployment-specific state which should have no place in a code-base that should be as useful to someone from outside our team as to us - so down the line i do wanna get rid of the hard-coded cruft, yeah.
kiara marked this conversation as resolved
@ -159,3 +159,3 @@
users.users.${name} = {
isSystemUser = true;
isNormalUser = true;
Owner

What exactly are the impacts that we are looking for here? I suppose:

  • Home Manager
  • SSH

Is that true? Can we not SSH into a system user?

What exactly are the impacts that we are looking for here? I suppose: - Home Manager - SSH Is that true? Can we not SSH into a system user?
Author
Owner

we ssh (and execute orchestration) from that user, not to it.

making panel a normal user grants it its own home folder, with a .cache it can write to, and .ssh stuff.

home-manager i'm using just to grant user panel an ssh config - which tbf does feel overkill, tho i hadn't looked into alternatives as of yet.

we ssh (and execute orchestration) from that user, not to it. making `panel` a normal user grants it its own home folder, with a `.cache` it can write to, and `.ssh` stuff. home-manager i'm using just to grant user `panel` an ssh config - which tbf does feel overkill, tho i hadn't looked into alternatives as of yet.
Owner

It does feel like overkill indeed, but that's really fine. That you for the details about system vs. normal user!

It does feel like overkill indeed, but that's really fine. That you for the details about system vs. normal user!
kiara marked this conversation as resolved
Owner

LGTM. If this has been tested successfully, feel free to merge!

LGTM. If this has been tested successfully, feel free to merge!
ditch panel's unused group as per @niols's suggestion
All checks were successful
/ check-pre-commit (pull_request) Successful in 24s
/ check-website (pull_request) Successful in 9s
/ check-peertube (pull_request) Successful in 20s
/ check-panel (pull_request) Successful in 1m24s
3b35f1ace0
kiara merged commit c69f1f52e0 into main 2025-04-09 16:58:52 +02:00
kiara deleted branch panel-vm-access-test-vms 2025-04-09 16:58: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!297
No description provided.