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

Fix multi-node networking by using Flannel #645

Merged
merged 7 commits into from
Jul 20, 2020

Conversation

twelho
Copy link
Contributor

@twelho twelho commented Jul 16, 2020

The Weave Net based approach we had in our docs didn't work at all, there's no inter-host network connectivity nor access to the internet. This PR adds a new helper script to set up a Flannel-based overlay and overhauls the networking docs accordingly, with this I'm finally successful in cross-pinging VMs from different hosts.

Hopefully fixes #642.

cc @stealthybox @darkowlzz

Signed-off-by: Dennis Marttinen <dennis@weave.works>
Signed-off-by: Dennis Marttinen <dennis@weave.works>
@twelho twelho requested a review from luxas July 16, 2020 16:12
Signed-off-by: Dennis Marttinen <dennis@weave.works>
Copy link
Contributor

@luxas luxas left a comment

Choose a reason for hiding this comment

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

Impressive work 💪!!
Great to see this coming together in a bit nicer and more stable form.
Only one docs nit, and then we're good to go 😄

/usr/local/bin/etcd \
--name s1 \
--data-dir /etcd-data \
--listen-client-urls http://0.0.0.0:2379 \
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should point out that this getting started guide does NOT secure the etcd traffic 😅

@twelho twelho force-pushed the flannel-cni branch 3 times, most recently from 3865897 to 0897b6c Compare July 17, 2020 10:41
Copy link
Contributor

@luxas luxas left a comment

Choose a reason for hiding this comment

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

LGTM

@luxas
Copy link
Contributor

luxas commented Jul 17, 2020

However, link checking error:

site/networking/index.html
	ERROR	../tools/ignite-flannel.sh
		Stat site/tools/ignite-flannel.sh: no such file or directory

Signed-off-by: Dennis Marttinen <dennis@weave.works>
Signed-off-by: Dennis Marttinen <dennis@weave.works>
Signed-off-by: Dennis Marttinen <dennis@weave.works>
docs/networking.md Outdated Show resolved Hide resolved
Copy link
Contributor

@darkowlzz darkowlzz left a comment

Choose a reason for hiding this comment

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

LGTM!
Very informative. Thanks.

Co-authored-by: Sunny <darkowlzz@protonmail.com>
@twelho twelho merged commit f9c5c9d into weaveworks:master Jul 20, 2020
@twelho twelho deleted the flannel-cni branch July 20, 2020 15:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multi host ignite VMs networking based on WeaveNet not working as expected (reopening issue #628)
3 participants