Skip to content

log the actual error when failing to add IPv6 route#2444

Merged
selansen merged 1 commit intomoby:masterfrom
kdomanski:verbose-ipv6-cannot-add
Sep 28, 2019
Merged

log the actual error when failing to add IPv6 route#2444
selansen merged 1 commit intomoby:masterfrom
kdomanski:verbose-ipv6-cannot-add

Conversation

@kdomanski
Copy link
Contributor

@kdomanski kdomanski commented Aug 29, 2019

Signed-off-by: Kamil Domański kamil@domanski.co

Currently the error message Could not add route to IPv6 network %s via device %s obscures the cause of the issue by not logging the original error as well.

Signed-off-by: Kamil Domański <kamil@domanski.co>
Copy link
Contributor

@arkodg arkodg left a comment

Choose a reason for hiding this comment

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

LGTM thanks @kdomanski
PTAL @euanh

Copy link
Contributor

@selansen selansen left a comment

Choose a reason for hiding this comment

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

LGTM

@selansen selansen merged commit 10c7fb6 into moby:master Sep 28, 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