search

SUSE / caasp-salt

A collection of salt states used to provision a kubernetes cluster
Apache License 2.0
64 stars 29 forks source link

Fix bsc#1116005: Dex pods should run only on master nodes #688

Closed emilianolangella closed 5 years ago

emilianolangella commented 5 years ago

Dex pods should run only on master nodes due to firewall/security policies that could be applied to workers nodes.

stefsuse commented 5 years ago

please include bug ref 1116005

MaximilianMeister commented 5 years ago

please include bug ref 1116005

@emilianolangella oh yes, forgot to mention, for bugfixes we need to include a tag in the commit message, so it will land in the package's changelog. this will then be picked up by openbuildservice's issue tracker.

just mention this: bsc#1116005 somewhere in the commit message

MaximilianMeister commented 5 years ago

Just checked the first CI run and no dex was running on any worker. We might need to do some manual testing though, but this can be done by QA or QAM later