Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Add ssh key name via hetzner secret #1289

Merged
merged 1 commit into from
May 8, 2024
Merged

✨ Add ssh key name via hetzner secret #1289

merged 1 commit into from
May 8, 2024

Conversation

janiskemper
Copy link
Contributor

What this PR does / why we need it:
Adding the ssh key name via hetzner secret facilitates use cases where a lot of clusters are created with the same ssh key. Instead of having to specify it every time in the HetznerClusterTemplate object, it can be specified in the secret ones.

On top, this commit introduces default values for the HetznerSecret to allow use cases where all is hard-coded and doesn't have to be specified e.g. in a ClusterClass or in the templates of the cluster object anymore.

TODOs:

  • squash commits
  • include documentation
  • add unit tests

Copy link
Collaborator

@guettli guettli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added two comments. Overall it looks good.

Copy link
Collaborator

@guettli guettli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Adding the ssh key name via hetzner secret facilitates use cases where a
lot of clusters are created with the same ssh key. Instead of having to
specify it every time in the HetznerClusterTemplate object, it can be
specified in the secret ones.

On top, this commit introduces default values for the HetznerSecret to
allow use cases where all is hard-coded and doesn't have to be specified
e.g. in a ClusterClass or in the templates of the cluster object
anymore.
@janiskemper janiskemper marked this pull request as ready for review May 7, 2024 07:15
@syself-bot syself-bot bot added area/code Changes made in the code directory area/api Changes made in the api directory labels May 7, 2024
@janiskemper janiskemper merged commit 9eaf507 into main May 8, 2024
13 of 14 checks passed
@janiskemper janiskemper deleted the sshkey-secret branch May 8, 2024 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api Changes made in the api directory area/code Changes made in the code directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants