Skip to content

Commit

Permalink
[chore] [extension/jaegerremotesampling] Avoid port conflict in tests (
Browse files Browse the repository at this point in the history
  • Loading branch information
dmitryax authored Nov 3, 2023
1 parent b39f83d commit 040426f
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ func TestRemote(t *testing.T) {

// make test case defined number of calls
for i := 0; i < tc.performedClientCallCount; i++ {
resp, err := http.Get("http://127.0.0.1:5778/sampling?service=foo")
resp, err := http.Get("http://127.0.0.1:5779/sampling?service=foo")
assert.NoError(t, err)
assert.Equal(t, 200, resp.StatusCode)
}
Expand Down Expand Up @@ -163,7 +163,7 @@ func (s *samplingServer) GetSamplingStrategy(ctx context.Context, params *api_v2

func testConfig() *Config {
cfg := createDefaultConfig().(*Config)
cfg.HTTPServerSettings.Endpoint = "127.0.0.1:5778"
cfg.GRPCServerSettings.NetAddr.Endpoint = "127.0.0.1:14250"
cfg.HTTPServerSettings.Endpoint = "127.0.0.1:5779"
cfg.GRPCServerSettings.NetAddr.Endpoint = "127.0.0.1:14251"
return cfg
}

0 comments on commit 040426f

Please sign in to comment.