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

Adding initial Hetzner Cloud provider #7

Merged
merged 6 commits into from
Mar 11, 2021
Merged

Adding initial Hetzner Cloud provider #7

merged 6 commits into from
Mar 11, 2021

Conversation

arnarg
Copy link
Contributor

@arnarg arnarg commented Mar 7, 2021

This adds initial Hetzner Cloud provider. It's mostly referenced by the AWS EC2 provider but adapted to HCloud SDK.

In Hetzner Cloud a server name is required and unique (at least per project, which is the scope of the API token). The server gets a random numeric ID but it's not used anywhere in UI to identify the server, so I thought instead it would be better to choose a name prefix and then a random string appended to it for the ID.

@arnarg
Copy link
Contributor Author

arnarg commented Mar 7, 2021

It occurred to me that I already had access to the target name so it made no sense to ask for a name attribute in the config. Oopsie.

Copy link
Owner

@stephank stephank left a comment

Choose a reason for hiding this comment

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

Really great stuff, thanks for contributing! I'll see if I can roll a release in a bit. 😃

@stephank stephank merged commit d148fc2 into stephank:main Mar 11, 2021
@arnarg arnarg deleted the add-hcloud-provider branch March 11, 2021 19:46
@arnarg
Copy link
Contributor Author

arnarg commented Mar 11, 2021

You're welcome. This is a really neat project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants