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

Implement support for custom internal IP mapping via HCLOUD_INSTANCES… #696

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

elohmeier
Copy link
Contributor

This update adds the ability to specify custom internal IP addresses for Hetzner Cloud and Robot Instances through the HCLOUD_INSTANCES_INTERNAL_IP_MAP environment variable.

This is especially useful when mixing cloud servers with dedicated robot servers via vSwitch / private networks. Users will need to manage the IPs of the robot servers manually anyway, this change let users specify them as well for the InternalIp field in K8S.

@elohmeier elohmeier requested a review from a team as a code owner July 9, 2024 15:35
@apricote
Copy link
Member

Hey @elohmeier, thanks for the PR! Our team is a little bit short on time right now with vacations, so the review is going to take some more weeks. I am sorry for this.

@elohmeier
Copy link
Contributor Author

Thanks for you message, no problem. I wonder if there is upstream interest for this addition? If so, please let me know - I would then take a look into the CI issues and rebase.

Copy link
Contributor

This PR has been marked as stale because it has not had recent activity. The bot will close the PR if no further action occurs.

@github-actions github-actions bot added the stale label Oct 21, 2024
@elohmeier elohmeier force-pushed the hcloud-instances-internal-ip-map branch from d72734f to 9198a4f Compare October 29, 2024 14:49
@elohmeier
Copy link
Contributor Author

Rebased onto latest main branch.

…_INTERNAL_IP_MAP

This update adds the ability to specify custom internal IP addresses for Hetzner Cloud and Robot Instances through the `HCLOUD_INSTANCES_INTERNAL_IP_MAP` environment variable.
@pat-s
Copy link

pat-s commented Nov 11, 2024

@elohmeier Can you add a usage example for how to specify the mapping? Tried your PR but neither got the IP added nor an error in hccm. Specified it as

          HCLOUD_INSTANCES_INTERNAL_IP_MAP:
            nodename: 'nodeip'

@mehrdadpfg
Copy link

would highly appreciate and looking forward for this PR to be merged

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.

4 participants