cloudfoundry-incubator / kubecf

Cloud Foundry on Kubernetes
Apache License 2.0
115 stars 62 forks source link

Implement HA settings for Eirini #905

Open jandubois opened 4 years ago

jandubois commented 4 years ago

sizing.bits.instances and sizing.eirini.instances are defined in values.yaml, but not used for anything. The subcharts don't have any mechanism to set a replica count.

Acceptance Criteria

Relates

https://github.com/cloudfoundry-incubator/kubecf/issues/1277

mudler commented 4 years ago

With regards to EiriniX - eirini-loggregator-bridge needs to be adapted to be HA. Currently it's not supported. It might be as easy as setting nodeselector for each replica, but needs to be handled in the bridge first.

Edit: opened https://github.com/cloudfoundry-incubator/eirini-loggregator-bridge/issues/23 with a raw idea on how to implement it - any feedback is welcome

jandubois commented 4 years ago

After discussion with @troytop in standup we decided to remove "suse-cap" label.

I suspect that most components will not be ready for HA yet (will Eirinix create separate webhooks for each instance of an extension? Or will the webhook itself always be single-instance?).

Either way, there is not sufficient time left for the current "suse-cap" timeline to actually test this properly, even if it looks like it might be working.