Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

Updates the version of weave-net to 2.7.0 #279

Merged
merged 4 commits into from
Aug 11, 2020
Merged

Updates the version of weave-net to 2.7.0 #279

merged 4 commits into from
Aug 11, 2020

Conversation

palemtnrider
Copy link
Contributor

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

@palemtnrider palemtnrider self-assigned this Jul 24, 2020
@jrryjcksn
Copy link
Contributor

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

jrryjcksn
jrryjcksn approved these changes Jul 24, 2020
View reviewed changes
Copy link
Contributor

@jrryjcksn jrryjcksn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@palemtnrider
Copy link
Contributor Author

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
Copy link
Contributor

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
Copy link
Contributor

Weave Net 2.7.0 has been released.

@palemtnrider
Copy link
Contributor Author

I'll update the PR to the 2.7.0 release.

@palemtnrider
Copy link
Contributor Author

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.

@foot foot force-pushed the WKP-1012/update-weave-net branch from 08b7f8a to f573178 Compare August 11, 2020 06:51
@foot foot changed the title Updates the version of weave-net to 2.6.5 Updates the version of weave-net to 2.7.9 Aug 11, 2020
@foot foot changed the title Updates the version of weave-net to 2.7.9 Updates the version of weave-net to 2.7.0 Aug 11, 2020
@foot foot merged commit 575b70c into release-0.8 Aug 11, 2020
@foot foot deleted the WKP-1012/update-weave-net branch August 11, 2020 11:35
@bboreham
Copy link
Contributor

Should have been made on master then cherry-picked.

@bboreham bboreham mentioned this pull request Aug 27, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jrryjcksn jrryjcksn jrryjcksn approved these changes

Assignees

@palemtnrider palemtnrider

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@palemtnrider @jrryjcksn @bboreham @foot