Fix Update/Delete Usage of Firewall Association

linode / linode-cloud-controller-manager

Kubernetes Cloud Controller Manager for Linode

Apache License 2.0

81 stars 61 forks source link

Fix Update/Delete Usage of Firewall Association #146

Closed luthermonson closed 8 months ago

luthermonson commented 9 months ago

The APIs do not currently exist to update or delete a firewall from a nodebalancer. When things like TPT-2535 are completed we can update the logic in CCM to catch annotation changes and update/delete the firewall association.

luthermonson commented 9 months ago

@rammanoj

rammanoj commented 9 months ago

With the merge of https://github.com/linode/linodego/pull/436, the feature has been implemented. But I guess this might need to wait till the latest release of linodego.