Skip to content

Commit 9b14ed0

Browse files
committed
Pivot to option 2 for pubsub | Update cleanup trigger to use PR branch for config file | Update filter for ops-e2e-testing tag
1 parent 42ffa25 commit 9b14ed0

7 files changed

Lines changed: 28 additions & 3 deletions

cloudbuild-e2e-cloud-functions-gen2.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,3 +72,6 @@ logsBucket: gs://opentelemetry-ops-e2e-cloud-build-logs
7272
substitutions:
7373
_TEST_RUNNER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-e2e-testing:${SHORT_SHA}
7474
_TEST_SERVER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-go-e2e-test-server:${SHORT_SHA}
75+
76+
options:
77+
pubsubTopic: projects/${PROJECT_ID}/topics/e2e-cleanup

cloudbuild-e2e-cloud-run.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,3 +48,6 @@ logsBucket: gs://opentelemetry-ops-e2e-cloud-build-logs
4848
substitutions:
4949
_TEST_RUNNER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-e2e-testing:${SHORT_SHA}
5050
_TEST_SERVER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-go-e2e-test-server:${SHORT_SHA}
51+
52+
options:
53+
pubsubTopic: projects/${PROJECT_ID}/topics/e2e-cleanup

cloudbuild-e2e-gae-standard.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,3 +71,6 @@ logsBucket: gs://opentelemetry-ops-e2e-cloud-build-logs
7171
substitutions:
7272
_TEST_RUNNER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-e2e-testing:${SHORT_SHA}
7373
_TEST_SERVER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-go-e2e-test-server:${SHORT_SHA}
74+
75+
options:
76+
pubsubTopic: projects/${PROJECT_ID}/topics/e2e-cleanup

cloudbuild-e2e-gae.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,3 +49,6 @@ logsBucket: gs://opentelemetry-ops-e2e-cloud-build-logs
4949
substitutions:
5050
_TEST_RUNNER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-e2e-testing:${SHORT_SHA}
5151
_TEST_SERVER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-go-e2e-test-server:${SHORT_SHA}
52+
53+
options:
54+
pubsubTopic: projects/${PROJECT_ID}/topics/e2e-cleanup

cloudbuild-e2e-gce.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,3 +48,6 @@ logsBucket: gs://opentelemetry-ops-e2e-cloud-build-logs
4848
substitutions:
4949
_TEST_RUNNER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-e2e-testing:${SHORT_SHA}
5050
_TEST_SERVER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-go-e2e-test-server:${SHORT_SHA}
51+
52+
options:
53+
pubsubTopic: projects/${PROJECT_ID}/topics/e2e-cleanup

cloudbuild-e2e-gke.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,3 +48,6 @@ logsBucket: gs://opentelemetry-ops-e2e-cloud-build-logs
4848
substitutions:
4949
_TEST_RUNNER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-e2e-testing:${SHORT_SHA}
5050
_TEST_SERVER_IMAGE: us-central1-docker.pkg.dev/${PROJECT_ID}/e2e-testing/opentelemetry-operations-go-e2e-test-server:${SHORT_SHA}
51+
52+
options:
53+
pubsubTopic: projects/${PROJECT_ID}/topics/e2e-cleanup

tf/persistent/repo-ci-triggers.tf

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,26 +40,33 @@ module "go" {
4040
run_on = ["local", "gce", "gke", "gae", "gae-standard", "cloud-run", "cloud-functions-gen2"]
4141
}
4242

43+
resource "google_pubsub_topic" "e2e_cleanup" {
44+
name = "e2e-cleanup"
45+
project = var.project_id
46+
}
47+
4348
resource "google_cloudbuild_trigger" "global_cleanup" {
4449
name = "global-e2e-cleanup"
4550
description = "Global cleanup for E2E tests triggered by Pub/Sub"
4651

4752
pubsub_config {
48-
topic = "projects/${var.project_id}/topics/cloud-builds"
53+
topic = google_pubsub_topic.e2e_cleanup.id
4954
}
5055

51-
filter = "\"terraform-resources\" in body.message.data.tags && (body.message.data.status == \"SUCCESS\" || body.message.data.status == \"FAILURE\")"
56+
filter = "(_BUILD_TAGS.contains(\"terraform-resources\") || _BUILD_TAGS.contains(\"ops-e2e-testing\")) && (_BUILD_STATUS == \"SUCCESS\" || _BUILD_STATUS == \"FAILURE\")"
5257

5358
git_file_source {
5459
path = "cloudbuild-cleanup.yaml"
5560
uri = "https://github.com/GoogleCloudPlatform/opentelemetry-operations-e2e-testing"
56-
revision = "refs/heads/main"
61+
revision = "refs/heads/durable-async-cleanup"
5762
repo_type = "GITHUB"
5863
}
5964

6065
substitutions = {
6166
_TEST_RUN_ID = "$(body.message.data.id)"
6267
_E2E_ENVIRONMENT = "$(body.message.data.substitutions._E2E_ENVIRONMENT)"
6368
_TEST_RUNNER_IMAGE = "$(body.message.data.substitutions._TEST_RUNNER_IMAGE)"
69+
_BUILD_TAGS = "$(body.message.data.tags)"
70+
_BUILD_STATUS = "$(body.message.data.status)"
6471
}
6572
}

0 commit comments

Comments
 (0)