skupperproject / skupper

Skupper is an implementation of a Virtual Application Network, enabling rich hybrid cloud communication.
http://skupper.io
Apache License 2.0
595 stars 74 forks source link

Fix SecuredAccessManager updatePorts #1723

Closed c-kruse closed 1 month ago

c-kruse commented 1 month ago

When comparing desired and actual state of services, ignore the value of NodePort as this is assigned dynamically by an external controller and is not managed by us.

Fixes #1720

grs commented 1 month ago

@c-kruse I think https://github.com/skupperproject/skupper/pull/1730 is a better way to fix the issue