Skip to content

Resolve "bridge fdb show" hang issue#2456

Merged
euanh merged 1 commit intomoby:masterfrom
suwang48404:master
Sep 27, 2019
Merged

Resolve "bridge fdb show" hang issue#2456
euanh merged 1 commit intomoby:masterfrom
suwang48404:master

Conversation

@suwang48404
Copy link
Contributor

The output of "bridge fdb show" command invoked under a network
namespace is unpredicable. Sometime it returns empty, and sometime
non-stop rolling output. This perhaps is a bug in kernel
and/or iproute2 implementation. To work around, display fdb for
each bridge.

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "master" git@github.com:suwang48404/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@suwang48404
Copy link
Contributor Author

The output of "bridge fdb show" command invoked under a network
namespace is unpredicable. Sometime it returns empty, and sometime
non-stop rolling output. This perhaps is a bug in kernel
and/or iproute2 implementation. To work around, display fdb  for
 each bridge.

Signed-off-by: Su Wang <su.wang@docker.com>
@euanh euanh merged commit 953ec5e into moby:master Sep 27, 2019
espensuenson added a commit to espensuenson/moby that referenced this pull request Oct 1, 2019
Issues that are fixed :
log the actual error when failing to add IPv6 route moby/libnetwork#2444
Shorten controller ID in exec-root to not hit UNIX_PATH_MAX moby/libnetwork#2443
Fixed getNetworkFromStore, which returned an incorrect struct moby/libnetwork#2449
weighted scheduling methods constants for ipvs moby/libnetwork#2453
Resolve "bridge fdb show" hang issue moby/libnetwork#2456

Signed-off-by: Espen Suenson <mail@espensuenson.dk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants