Skip to content

Remove IPv6 bridge RouteAdd() that always fails#48135

Merged
akerouanton merged 1 commit intomoby:masterfrom
robmry:remove_ipv6_bridge_route_add
Jul 5, 2024
Merged

Remove IPv6 bridge RouteAdd() that always fails#48135
akerouanton merged 1 commit intomoby:masterfrom
robmry:remove_ipv6_bridge_route_add

Conversation

@robmry
Copy link
Copy Markdown
Contributor

@robmry robmry commented Jul 4, 2024

- What I did

The netlink.RouteAdd in setupBridgeIPv6 always fails, because the bridge is always 'down' when it's called.

- How I did it

Deleted it.

- How to verify it

There's no log entry about the failure any more.

A --ipv6 bridge still has an IPv6 route. (Just to check - I deleted the bridge's IPv6 route, bounced dockerd, and checked that the route came back.)

- Description for the changelog

n/a

The netlink.RouteAdd in setupBridgeIPv6 always failed, because
the bridge is always 'down' when it's called.

Signed-off-by: Rob Murray <[email protected]>
@robmry robmry added area/networking Networking kind/refactor PR's that refactor, or clean-up code area/networking/ipv6 Networking labels Jul 4, 2024
@robmry robmry added this to the 28.0.0 milestone Jul 4, 2024
@robmry robmry self-assigned this Jul 4, 2024
@robmry robmry requested review from akerouanton and corhere July 5, 2024 09:41
@robmry robmry marked this pull request as ready for review July 5, 2024 09:41
@akerouanton akerouanton merged commit 1bf2e03 into moby:master Jul 5, 2024
@robmry robmry deleted the remove_ipv6_bridge_route_add branch July 30, 2024 13:27
@robmry robmry mentioned this pull request Nov 4, 2024
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking/ipv6 Networking area/networking Networking kind/refactor PR's that refactor, or clean-up code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants