OpenLiberty / open-liberty-operator

Eclipse Public License 2.0
28 stars 36 forks source link

Support for disabling service links #487

Closed leochr closed 3 weeks ago

leochr commented 10 months ago

Kubernetes by default injects env variables with service links into pods (as env variables). It's only for the services in the same namespace. Add support to a config option to disable service links injection.

From Kubernetes:

EnableServiceLinks indicates whether information about services should be injected into pod's environment variables, matching the syntax of Docker links. Optional: Defaults to true.

Using this config option, users can disable env variables with information about other services being injected into the application pod

leochr commented 6 months ago

Ian and I discussed this today and agreed that changing the field name to disableServiceLinks would avoid any confusion on the out of the box behaviour and the purpose of the field.

leochr commented 2 months ago

@idlewis Merged the PR for RCO. So we can proceed with OLO and WLO changes. Added a comment for the test PR. Let's merge the test PR once OLO and WLO are updated. Thank you.