Skip to content

Add optional flags for DOKS control plane firewall feature#1540

Merged
danaelhe merged 7 commits intodigitalocean:mainfrom
llDrLove:olove/add-control-plane-firewall
Jun 20, 2024
Merged

Add optional flags for DOKS control plane firewall feature#1540
danaelhe merged 7 commits intodigitalocean:mainfrom
llDrLove:olove/add-control-plane-firewall

Conversation

@llDrLove
Copy link
Contributor

Add optional flags for the control plane firewall feature.
This is still a per/user feature and can't be used by every DOKS users.
Also, I needed to update to godo 0.117.0 which has the new request parameters.

@llDrLove llDrLove force-pushed the olove/add-control-plane-firewall branch 4 times, most recently from 48a3391 to c316163 Compare June 18, 2024 02:17
@llDrLove llDrLove force-pushed the olove/add-control-plane-firewall branch from c316163 to 0c61f59 Compare June 18, 2024 02:20
@llDrLove llDrLove marked this pull request as ready for review June 18, 2024 16:00
@llDrLove
Copy link
Contributor Author

Hi @bbassingthwaite , I needed to adjust some of the LB tests when introducing the godo bump to v0.118.0 for your ProxyProtocol LoadBalancer HealthCheck changes introduced in that version. The changes I made relates to the LBs integration tests so it is minimal but could you give it a quick review ? Thanks !

@bbassingthwaite
Copy link

LGTM thanks for fixing!

Copy link
Member

@danaelhe danaelhe left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@danaelhe danaelhe merged commit 5b24b23 into digitalocean:main Jun 20, 2024
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.

3 participants