search

weaveworks / wksctl

Open Source Weaveworks Kubernetes System
Apache License 2.0
395 stars 55 forks source link

Updates the version of weave-net to 2.7.0 #279

Closed palemtnrider closed 4 years ago

palemtnrider commented 4 years ago

WKP-1012. Update to the latest released version of weave-net

jrryjcksn commented 4 years ago

So... I'm guessing the cpu and readonly changes were from the release notes?

palemtnrider commented 4 years ago

So... I'm guessing the cpu and readonly changes were from the release notes?

I pulled the latest version of weave-net from weave cloud and those changes were included in it.

bboreham commented 4 years ago

The config from Weave Cloud is awaiting a fix for those requests Suggest you use https://github.com/weaveworks/weave/releases/download/latest_release/weave-daemonset-k8s-1.11.yaml

The fixed numbers are a gamble because you need more as your cluster grows; reducing them is a big risk.

bboreham commented 4 years ago

Weave Net 2.7.0 has been released.

palemtnrider commented 4 years ago

I'll update the PR to the 2.7.0 release.

palemtnrider commented 4 years ago

The fixed numbers are a gamble because you need more as your cluster grows; reducing them is a big risk.

I'll revert this part of the change as well.

bboreham commented 4 years ago

Should have been made on master then cherry-picked.