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.

Sign up for GitHub

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

Jump to bottom

Decide how to integrate firewalling #39

Open
robertvolkmann opened this issue Jan 22, 2025 · 1 comment
Open

Decide how to integrate firewalling #39

robertvolkmann opened this issue Jan 22, 2025 · 1 comment

Comments

@robertvolkmann
Copy link
Contributor

robertvolkmann commented Jan 22, 2025 •
edited
Loading

The first iteration was removed by #40 to help third parties become familiar with metal-stack.
vknabel added a commit that referenced this issue Jan 27, 2025
@vknabel
feat: make nodeNetworkID mandatory
abe034d 
Currently the cluster controller will never be able to delete the node network - even if all machines have been deleted, because the firewall is unmanaged. This requires the node network to also be unmanaged. See #39 for more context
@vknabel vknabel mentioned this issue Jan 27, 2025
Merged
@vknabel
Copy link
Contributor

vknabel commented Jan 27, 2025

Due to the fact that firewalls are currently unmanaged and that they make use of the node network, we are unable to freely reconcile the node network and decided to make it part of this issue.

#55 makes the node network mandatory and might need to be reverted in the scope of this issue in the future..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vknabel @robertvolkmann