Skip to content

Conversation

@jmoggr
Copy link
Collaborator

@jmoggr jmoggr commented Jan 29, 2026

Previously we frequently misunderstood and made mistakes about which public keys needed to be defined in which places. This new declarative structure for the public keys should make it much easier to understand what keys are needed where and make adding/editing keys/hosts less error prone.

Stacked on top of #970 due to skill issues with git.

@github-actions github-actions bot temporarily deployed to netlify-preview January 29, 2026 15:37 Destroyed
@jmoggr jmoggr force-pushed the consolidate-ssh-keys branch from 23c8252 to 558471d Compare January 29, 2026 15:39
@github-actions github-actions bot temporarily deployed to netlify-preview January 29, 2026 15:40 Destroyed
@jmoggr jmoggr force-pushed the consolidate-ssh-keys branch from 558471d to 9125a0c Compare January 29, 2026 15:40
@github-actions github-actions bot temporarily deployed to netlify-preview January 29, 2026 15:41 Destroyed
@jmoggr jmoggr changed the base branch from sudo-password to main January 29, 2026 15:41
@jmoggr jmoggr force-pushed the consolidate-ssh-keys branch from 9125a0c to aaa5bd8 Compare January 29, 2026 15:44
@jmoggr jmoggr changed the base branch from main to sudo-password January 29, 2026 15:44
@jmoggr jmoggr marked this pull request as ready for review January 29, 2026 15:45
@jmoggr jmoggr requested a review from kasbah as a code owner January 29, 2026 15:45
@jmoggr jmoggr added the build CI/CD, linting, deployments, and anything Nix label Jan 29, 2026
@epatters
Copy link
Member

epatters commented Jan 29, 2026

Thanks! I'll let Kaspar review but I have indeed previously been tripped up by this.

@kasbah kasbah merged commit 69f0f71 into sudo-password Jan 29, 2026
19 checks passed
@epatters epatters deleted the consolidate-ssh-keys branch January 29, 2026 17:47
@kasbah
Copy link
Member

kasbah commented Jan 29, 2026

Oops maybe shouldn't have merged due to base not being main but we can still merge #970 all together. Just makes it more annoying to review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build CI/CD, linting, deployments, and anything Nix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants