Skip to content

Commit

Permalink
fix: revert mistake
Browse files Browse the repository at this point in the history
  • Loading branch information
VihasMakwana committed Sep 18, 2024
1 parent 8b1ecff commit 21ae1b2
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 13 deletions.
10 changes: 5 additions & 5 deletions testbed/testbed/test_bed.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,11 @@ var GlobalConfig = struct {
// DoTestMain is intended to be run from TestMain somewhere in the test suit.
// This enables the testbed.
func DoTestMain(m *testing.M, resultsSummary TestResultsSummary) {
// testBedConfigFile := os.Getenv(testBedEnableEnvVarName)
// if testBedConfigFile == "" {
// log.Printf(testBedEnableEnvVarName + " is not defined, skipping E2E tests.")
// os.Exit(0)
// }
testBedConfigFile := os.Getenv(testBedEnableEnvVarName)
if testBedConfigFile == "" {
log.Printf(testBedEnableEnvVarName + " is not defined, skipping E2E tests.")
os.Exit(0)
}

// Load the test bed config first.
err := Start(resultsSummary)
Expand Down
16 changes: 8 additions & 8 deletions testbed/tests/trace_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -183,15 +183,15 @@ func TestTraceNoBackend10kSPS(t *testing.T) {
`,
}

// noLimitProcessors := map[string]string{}
noLimitProcessors := map[string]string{}

var processorsConfig = []processorConfig{
// {
// Name: "NoMemoryLimit",
// Processor: noLimitProcessors,
// ExpectedMaxRAM: 100,
// ExpectedMinFinalRAM: 80,
// },
{
Name: "NoMemoryLimit",
Processor: noLimitProcessors,
ExpectedMaxRAM: 100,
ExpectedMinFinalRAM: 80,
},
{
Name: "MemoryLimit",
Processor: limitProcessors,
Expand All @@ -206,7 +206,7 @@ func TestTraceNoBackend10kSPS(t *testing.T) {
t,
testbed.NewOTLPTraceDataSender(testbed.DefaultHost, testutil.GetAvailablePort(t)),
testbed.NewOTLPDataReceiver(testutil.GetAvailablePort(t)),
testbed.ResourceSpec{ExpectedMaxCPU: 99, ExpectedMaxRAM: testConf.ExpectedMaxRAM},
testbed.ResourceSpec{ExpectedMaxCPU: 90, ExpectedMaxRAM: testConf.ExpectedMaxRAM},
performanceResultsSummary,
testConf,
)
Expand Down

0 comments on commit 21ae1b2

Please sign in to comment.