Skip to content

Commit

Permalink
don't explicitly set dstport
Browse files Browse the repository at this point in the history
  • Loading branch information
moshelitvin-MS committed Oct 25, 2020
1 parent 3f9ca60 commit d697367
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions link_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -633,12 +633,10 @@ func TestGeneveCompareToIP(t *testing.T) {
ns, tearDown := setUpNamedNetlinkTest(t)
defer tearDown()

t.Log(ns)

expected := &Geneve{
ID: 0x764332, // 23 bits
Remote: net.ParseIP("1.2.3.4"),
Dport: 0x1234,
Dport: 6081,
}

// Create interface
Expand All @@ -647,7 +645,9 @@ func TestGeneveCompareToIP(t *testing.T) {
"type", "geneve",
"vni", fmt.Sprint(expected.ID),
"remote", expected.Remote.String(),
"dstport", fmt.Sprint(expected.Dport),
// TODO: unit tests are currently done on ubuntu 16, and the version of iproute2 there doesn't support dstport
// We can still do most of the testing by verifying that we do read the default port
// "dstport", fmt.Sprint(expected.Dport),
)
out := &bytes.Buffer{}
cmd.Stdout = out
Expand Down

0 comments on commit d697367

Please sign in to comment.