Skip to content

Commit

Permalink
revert-me: why is CI not using the correct kafka port
Browse files Browse the repository at this point in the history
  • Loading branch information
endigma committed Feb 11, 2025
1 parent 4f8f17e commit 0769b96
Showing 1 changed file with 3 additions and 9 deletions.
12 changes: 3 additions & 9 deletions router-tests/testenv/testenv.go
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,6 @@ func Bench(b *testing.B, cfg *Config, f func(b *testing.B, xEnv *Environment)) {
assertCacheMetrics(b, env, v, ff)
}
}

}

const letterBytes = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ"
Expand Down Expand Up @@ -359,7 +358,7 @@ func createTestEnv(t testing.TB, cfg *Config) (*Environment, error) {
)

if len(cfg.KafkaSeeds) == 0 {
cfg.KafkaSeeds = []string{"localhost:9092"}
cfg.KafkaSeeds = []string{"localhost:3949"}
}

if cfg.EnableKafka {
Expand Down Expand Up @@ -407,9 +406,7 @@ func createTestEnv(t testing.TB, cfg *Config) (*Environment, error) {

ctx, cancel := context.WithCancelCause(context.Background())

var (
logObserver *observer.ObservedLogs
)
var logObserver *observer.ObservedLogs

if oc := cfg.LogObservation; oc.Enabled {
var zCore zapcore.Core
Expand Down Expand Up @@ -441,9 +438,7 @@ func createTestEnv(t testing.TB, cfg *Config) (*Environment, error) {
Countries: atomic.NewInt64(0),
}

var (
requiredPorts = 2
)
requiredPorts := 2

ports := freeport.GetN(t, requiredPorts)

Expand Down Expand Up @@ -1663,7 +1658,6 @@ func (e *Environment) ReadSSE(ctx context.Context, body io.ReadCloser, handler f
handler(data)
}
}

}
}

Expand Down

0 comments on commit 0769b96

Please sign in to comment.