Skip to content

ci: install Cloud Firewall Controller for LKE cluster #2137

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

Merged
merged 14 commits into from
May 21, 2025
Merged

Conversation

merll
Copy link
Contributor

@merll merll commented May 8, 2025

📌 Summary

The Linode Cloud Firewall Controller has been released for several months now, and there is no known interference with installing or running the platform. We should by default install it on any LKE cluster that we deploy using our GitHub workflow.

This PR also removes the always silently failing Slack notification task.

🔍 Reviewer Notes

HTTP and HTTPS as provided by typical LKE-hosted services should not be impacted. SSH access to the single nodes is no longer possible, which is the desired effect of the firewall.
In the unlikely event that it is required, the firewall can be disabled or edited manually. Also it is possible to deploy custom rules, as provided here: https://github.com/linode/cloud-firewall-controller/tree/main/examples

🧹 Checklist

N/A

@merll merll requested review from j-zimnowoda and Ani1357 May 8, 2025 12:17
Copy link

github-actions bot commented May 8, 2025

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements
41.37% (+0.11% 🔼)
774/1871
🔴 Branches
33.41% (+0.47% 🔼)
287/859
🔴 Functions
41% (+0.33% 🔼)
123/300
🔴 Lines
42.34% (+0.17% 🔼)
732/1729

Test suite run success

93 tests passing in 9 suites.

Report generated by 🧪jest coverage report action from bb82418

@j-zimnowoda j-zimnowoda merged commit 9111015 into main May 21, 2025
14 checks passed
@j-zimnowoda j-zimnowoda deleted the ci-cfw branch May 21, 2025 13:09
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