Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[chore][CI/CD] Remove integration test naming restriction #32529

Merged
merged 22 commits into from
May 2, 2024
Merged
Show file tree
Hide file tree
Changes from 15 commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
6efea0a
[chore][CI/CD] Remove integration test naming restriction
crobert-1 Apr 18, 2024
efb65d9
Skip integration test panic: Issue 32530
crobert-1 Apr 18, 2024
990aa64
Skip more clickhouse exporter integration tests
crobert-1 Apr 18, 2024
82ec656
Skip failing tests, fix lint error
crobert-1 Apr 18, 2024
d8570ec
Skip hostmetrics integration test
crobert-1 Apr 18, 2024
4f0e38a
Skip another host metrics integration test
crobert-1 Apr 18, 2024
0673f11
Skip failing hostmetrics integration test
crobert-1 Apr 18, 2024
d7f15c7
Skip mongodbatlas receiver integration tests that are failing
crobert-1 Apr 18, 2024
de1d0bb
Update receiver/cloudflarereceiver/logs_integration_test.go
crobert-1 Apr 22, 2024
c68cb58
Update receiver/hostmetricsreceiver/integration_test.go
crobert-1 Apr 22, 2024
aeb4e9e
Merge branch 'main' into fix_integration_testing
crobert-1 Apr 22, 2024
64daa11
Re-add skip test for hostmetrics receiver integration test
crobert-1 Apr 22, 2024
720dae0
Merge branch 'main' into fix_integration_testing
crobert-1 Apr 24, 2024
82702f4
Merge branch 'main' into fix_integration_testing
crobert-1 Apr 25, 2024
3aca5c2
Merge branch 'main' into fix_integration_testing
crobert-1 Apr 26, 2024
c32f2e1
Merge branch 'main' into fix_integration_testing
crobert-1 May 1, 2024
73be625
Update exporter/splunkhecexporter/integration_test.go
crobert-1 May 1, 2024
fc15a78
Update receiver/mongodbatlasreceiver/alerts_integration_test.go
crobert-1 May 1, 2024
53a1984
Update receiver/mongodbatlasreceiver/alerts_integration_test.go
crobert-1 May 1, 2024
7a657b3
Update receiver/mongodbatlasreceiver/alerts_integration_test.go
crobert-1 May 1, 2024
d3dc557
Merge branch 'main' into fix_integration_testing
TylerHelmuth May 2, 2024
e1d1b1d
Merge branch 'main' into fix_integration_testing
codeboten May 2, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Makefile.Common
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ GOTEST_TIMEOUT?= 600s
GOTEST_OPT?= -race -timeout $(GOTEST_TIMEOUT) -parallel 4 --tags=$(GO_BUILD_TAGS)
GOTEST_INTEGRATION_OPT?= -race -timeout 360s -parallel 4
GOTEST_OPT_WITH_COVERAGE = $(GOTEST_OPT) -coverprofile=coverage.txt -covermode=atomic
GOTEST_OPT_WITH_INTEGRATION=$(GOTEST_INTEGRATION_OPT) -tags=integration,$(GO_BUILD_TAGS) -run=Integration
GOTEST_OPT_WITH_INTEGRATION=$(GOTEST_INTEGRATION_OPT) -tags=integration,$(GO_BUILD_TAGS)
GOTEST_OPT_WITH_INTEGRATION_COVERAGE=$(GOTEST_OPT_WITH_INTEGRATION) -coverprofile=integration-coverage.txt -covermode=atomic
GOCMD?= go
GOOS=$(shell $(GOCMD) env GOOS)
Expand Down
25 changes: 13 additions & 12 deletions exporter/clickhouseexporter/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,18 +26,19 @@ func TestIntegration(t *testing.T) {
name string
image string
}{
{
name: "test clickhouse 24-alpine",
image: "clickhouse/clickhouse-server:24-alpine",
},
{
name: "test clickhouse 23-alpine",
image: "clickhouse/clickhouse-server:23-alpine",
},
{
name: "test clickhouse 22-alpine",
image: "clickhouse/clickhouse-server:22-alpine",
},
// TODO: Skipping due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32530
crobert-1 marked this conversation as resolved.
Show resolved Hide resolved
// {
// name: "test clickhouse 24-alpine",
// image: "clickhouse/clickhouse-server:24-alpine",
// },
// {
// name: "test clickhouse 23-alpine",
// image: "clickhouse/clickhouse-server:23-alpine",
// },
// {
// name: "test clickhouse 22-alpine",
// image: "clickhouse/clickhouse-server:22-alpine",
// },
}

for _, c := range testCase {
Expand Down
2 changes: 2 additions & 0 deletions exporter/datasetexporter/logs_exporter_stress_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,8 @@ import (
)

func TestConsumeLogsManyLogsShouldSucceed(t *testing.T) {
t.Skip("TODO: Skipping due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32533")

const maxDelay = 200 * time.Millisecond
createSettings := exportertest.NewNopCreateSettings()

Expand Down
2 changes: 2 additions & 0 deletions exporter/splunkhecexporter/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -298,6 +298,8 @@ func tracesTest(t *testing.T, config *Config, url *url.URL, test testCfg) {
}

func TestSplunkHecExporter(t *testing.T) {
t.Skip("TODO: Skipping due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32534")

crobert-1 marked this conversation as resolved.
Show resolved Hide resolved
splunkContCfg := setup()
defer teardown(splunkContCfg)

Expand Down
6 changes: 6 additions & 0 deletions receiver/hostmetricsreceiver/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ import (
)

func Test_ProcessScrape(t *testing.T) {
t.Skip("TODO: Skipping for now due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32536")

crobert-1 marked this conversation as resolved.
Show resolved Hide resolved
expectedFile := filepath.Join("testdata", "e2e", "expected_process.yaml")
cmd := exec.Command("/bin/sleep", "300")
require.NoError(t, cmd.Start())
Expand Down Expand Up @@ -57,6 +59,8 @@ func Test_ProcessScrape(t *testing.T) {
}

func Test_ProcessScrapeWithCustomRootPath(t *testing.T) {
t.Skip("TODO: Skipping for now due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32536")

expectedFile := filepath.Join("testdata", "e2e", "expected_process_separate_proc.yaml")

scraperinttest.NewIntegrationTest(
Expand Down Expand Up @@ -88,6 +92,8 @@ func Test_ProcessScrapeWithCustomRootPath(t *testing.T) {
}

func Test_ProcessScrapeWithBadRootPathAndEnvVar(t *testing.T) {
t.Skip("TODO: Skipping for now due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32536")

expectedFile := filepath.Join("testdata", "e2e", "expected_process_separate_proc.yaml")
t.Setenv("HOST_PROC", filepath.Join("testdata", "e2e", "proc"))
scraperinttest.NewIntegrationTest(
Expand Down
6 changes: 6 additions & 0 deletions receiver/mongodbatlasreceiver/alerts_integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,8 @@ const (
)

func TestAlertsReceiver(t *testing.T) {
t.Skip("TODO: Skipping due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32543")

crobert-1 marked this conversation as resolved.
Show resolved Hide resolved
for _, payloadName := range testPayloads {
t.Run(payloadName, func(t *testing.T) {
testAddr := testutil.GetAvailableLocalAddress(t)
Expand Down Expand Up @@ -110,6 +112,8 @@ func TestAlertsReceiver(t *testing.T) {
}

func TestAlertsReceiverTLS(t *testing.T) {
t.Skip("TODO: Skipping due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32543")

for _, payloadName := range testPayloads {
t.Run(payloadName, func(t *testing.T) {
testAddr := testutil.GetAvailableLocalAddress(t)
Expand Down Expand Up @@ -183,6 +187,8 @@ func TestAlertsReceiverTLS(t *testing.T) {
}

func TestAtlasPoll(t *testing.T) {
t.Skip("TODO: Skipping due to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32543")

crobert-1 marked this conversation as resolved.
Show resolved Hide resolved
mockClient := mockAlertsClient{}

alerts := []mongodbatlas.Alert{}
Expand Down
Loading