Skip to content

Commit c8b0d4e

Browse files
Fixes issue #21773 for Eventarc test failures due to beta-only resources (#13478)
[upstream:b745f17d629a385adc6602164f8e21cf854adfc8] Signed-off-by: Modular Magician <[email protected]>
1 parent dad0901 commit c8b0d4e

File tree

2 files changed

+174
-14
lines changed

2 files changed

+174
-14
lines changed

google-beta/services/eventarc/resource_eventarc_google_api_source_test.go

+52-7
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@ import (
1616
transport_tpg "github.com/hashicorp/terraform-provider-google-beta/google-beta/transport"
1717
)
1818

19+
// This test is beta-only due to the dependency on google_project_service_identity,
20+
// but enables testing updates to the message_bus field in GoogleApiSource.
1921
func TestAccEventarcGoogleApiSource_update(t *testing.T) {
2022
t.Parallel()
2123

@@ -29,7 +31,7 @@ func TestAccEventarcGoogleApiSource_update(t *testing.T) {
2931

3032
acctest.VcrTest(t, resource.TestCase{
3133
PreCheck: func() { acctest.AccTestPreCheck(t) },
32-
ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories(t),
34+
ProtoV5ProviderFactories: acctest.ProtoV5ProviderBetaFactories(t),
3335
CheckDestroy: testAccCheckEventarcGoogleApiSourceDestroyProducer(t),
3436
ExternalProviders: map[string]resource.ExternalProvider{
3537
"time": {},
@@ -81,6 +83,7 @@ func TestAccEventarcGoogleApiSource_update(t *testing.T) {
8183
func testAccEventarcGoogleApiSource_full(context map[string]interface{}) string {
8284
return acctest.Nprintf(`
8385
resource "google_project" "project" {
86+
provider = google-beta
8487
project_id = "tf-test%{random_suffix}"
8588
name = "tf-test%{random_suffix}"
8689
org_id = "%{org_id}"
@@ -94,24 +97,28 @@ resource "time_sleep" "wait_create_project" {
9497
}
9598
9699
resource "google_project_service" "compute" {
100+
provider = google-beta
97101
project = google_project.project.project_id
98102
service = "compute.googleapis.com"
99103
depends_on = [time_sleep.wait_create_project]
100104
}
101105
102106
resource "google_project_service" "servicenetworking" {
103-
project = google_project.project.project_id
104-
service = "servicenetworking.googleapis.com"
107+
provider = google-beta
108+
project = google_project.project.project_id
109+
service = "servicenetworking.googleapis.com"
105110
depends_on = [google_project_service.compute]
106111
}
107112
108113
resource "google_project_service" "kms" {
114+
provider = google-beta
109115
project = google_project.project.project_id
110116
service = "cloudkms.googleapis.com"
111117
depends_on = [google_project_service.servicenetworking]
112118
}
113119
114120
resource "google_project_service" "eventarc" {
121+
provider = google-beta
115122
project = google_project.project.project_id
116123
service = "eventarc.googleapis.com"
117124
depends_on = [google_project_service.kms]
@@ -123,18 +130,21 @@ resource "time_sleep" "wait_enable_service" {
123130
}
124131
125132
resource "google_kms_key_ring" "keyring" {
133+
provider = google-beta
126134
name = "keyring"
127135
location = "%{region}"
128136
project = google_project.project.project_id
129137
depends_on = [google_project_service.kms]
130138
}
131139
132140
resource "google_kms_crypto_key" "key" {
141+
provider = google-beta
133142
name = "key1"
134143
key_ring = google_kms_key_ring.keyring.id
135144
}
136145
137146
resource "google_project_service_identity" "eventarc_sa" {
147+
provider = google-beta
138148
service = "eventarc.googleapis.com"
139149
project = google_project.project.project_id
140150
depends_on = [time_sleep.wait_enable_service]
@@ -146,6 +156,7 @@ resource "time_sleep" "wait_create_sa" {
146156
}
147157
148158
resource "google_kms_crypto_key_iam_member" "eventarc_sa_keyuser" {
159+
provider = google-beta
149160
crypto_key_id = google_kms_crypto_key.key.id
150161
role = "roles/cloudkms.cryptoKeyEncrypterDecrypter"
151162
member = google_project_service_identity.eventarc_sa.member
@@ -158,13 +169,15 @@ resource "time_sleep" "wait_propagate_iam" {
158169
}
159170
160171
resource "google_eventarc_message_bus" "message_bus" {
172+
provider = google-beta
161173
location = "%{region}"
162174
message_bus_id = "tf-test-messagebus%{random_suffix}"
163175
project = google_project.project.project_id
164176
depends_on = [time_sleep.wait_propagate_iam]
165177
}
166178
167179
resource "google_eventarc_google_api_source" "primary" {
180+
provider = google-beta
168181
location = "%{region}"
169182
google_api_source_id = "tf-test-googleapisource%{random_suffix}"
170183
project = google_project.project.project_id
@@ -189,6 +202,7 @@ resource "google_eventarc_google_api_source" "primary" {
189202
func testAccEventarcGoogleApiSource_update(context map[string]interface{}) string {
190203
return acctest.Nprintf(`
191204
resource "google_project" "project" {
205+
provider = google-beta
192206
project_id = "tf-test%{random_suffix}"
193207
name = "tf-test%{random_suffix}"
194208
org_id = "%{org_id}"
@@ -197,58 +211,68 @@ resource "google_project" "project" {
197211
}
198212
199213
resource "google_project_service" "compute" {
214+
provider = google-beta
200215
project = google_project.project.project_id
201216
service = "compute.googleapis.com"
202217
}
203218
204219
resource "google_project_service" "servicenetworking" {
205-
project = google_project.project.project_id
206-
service = "servicenetworking.googleapis.com"
220+
provider = google-beta
221+
project = google_project.project.project_id
222+
service = "servicenetworking.googleapis.com"
207223
depends_on = [google_project_service.compute]
208224
}
209225
210226
resource "google_project_service" "kms" {
227+
provider = google-beta
211228
project = google_project.project.project_id
212229
service = "cloudkms.googleapis.com"
213230
depends_on = [google_project_service.servicenetworking]
214231
}
215232
216233
resource "google_project_service" "eventarc" {
234+
provider = google-beta
217235
project = google_project.project.project_id
218236
service = "eventarc.googleapis.com"
219237
depends_on = [google_project_service.kms]
220238
}
221239
222240
resource "google_kms_key_ring" "keyring" {
241+
provider = google-beta
223242
name = "keyring"
224243
location = "%{region}"
225244
project = google_project.project.project_id
226245
depends_on = [google_project_service.kms]
227246
}
228247
229248
resource "google_kms_crypto_key" "key" {
249+
provider = google-beta
230250
name = "key1"
231251
key_ring = google_kms_key_ring.keyring.id
232252
}
233253
234254
resource "google_kms_crypto_key" "key_update" {
255+
provider = google-beta
235256
name = "key2"
236257
key_ring = google_kms_key_ring.keyring.id
237258
}
238259
239260
resource "google_project_service_identity" "eventarc_sa" {
261+
provider = google-beta
240262
service = "eventarc.googleapis.com"
241263
project = google_project.project.project_id
242264
depends_on = [google_project_service.eventarc]
243265
}
244266
245267
resource "google_kms_crypto_key_iam_member" "eventarc_sa_keyuser" {
268+
provider = google-beta
246269
crypto_key_id = google_kms_crypto_key.key.id
247270
role = "roles/cloudkms.cryptoKeyEncrypterDecrypter"
248271
member = google_project_service_identity.eventarc_sa.member
249272
}
250273
251274
resource "google_kms_crypto_key_iam_member" "eventarc_sa_keyuser_update" {
275+
provider = google-beta
252276
crypto_key_id = google_kms_crypto_key.key_update.id
253277
role = "roles/cloudkms.cryptoKeyEncrypterDecrypter"
254278
member = google_project_service_identity.eventarc_sa.member
@@ -261,6 +285,7 @@ resource "time_sleep" "wait_propagate_iam_update" {
261285
262286
# Create a separate project to contain another MessageBus.
263287
resource "google_project" "project_update" {
288+
provider = google-beta
264289
project_id = "tf-test2%{random_suffix}"
265290
name = "tf-test2%{random_suffix}"
266291
org_id = "%{org_id}"
@@ -274,6 +299,7 @@ resource "time_sleep" "wait_create_project_update" {
274299
}
275300
276301
resource "google_project_service" "eventarc_update" {
302+
provider = google-beta
277303
project = google_project.project_update.project_id
278304
service = "eventarc.googleapis.com"
279305
depends_on = [time_sleep.wait_create_project_update]
@@ -285,6 +311,7 @@ resource "time_sleep" "wait_enable_service_update" {
285311
}
286312
287313
resource "google_project_service_identity" "eventarc_sa_update" {
314+
provider = google-beta
288315
project = google_project.project_update.project_id
289316
service = "eventarc.googleapis.com"
290317
depends_on = [time_sleep.wait_enable_service_update]
@@ -296,13 +323,15 @@ resource "time_sleep" "wait_create_sa_update" {
296323
}
297324
298325
resource "google_eventarc_message_bus" "message_bus_update" {
326+
provider = google-beta
299327
location = "%{region}"
300328
message_bus_id = "tf-test-messagebus2%{random_suffix}"
301329
project = google_project.project_update.project_id
302330
depends_on = [time_sleep.wait_create_sa_update]
303331
}
304332
305333
resource "google_eventarc_google_api_source" "primary" {
334+
provider = google-beta
306335
location = "%{region}"
307336
google_api_source_id = "tf-test-googleapisource%{random_suffix}"
308337
project = google_project.project.project_id
@@ -327,6 +356,7 @@ resource "google_eventarc_google_api_source" "primary" {
327356
func testAccEventarcGoogleApiSource_unset(context map[string]interface{}) string {
328357
return acctest.Nprintf(`
329358
resource "google_project" "project" {
359+
provider = google-beta
330360
project_id = "tf-test%{random_suffix}"
331361
name = "tf-test%{random_suffix}"
332362
org_id = "%{org_id}"
@@ -335,64 +365,75 @@ resource "google_project" "project" {
335365
}
336366
337367
resource "google_project_service" "compute" {
368+
provider = google-beta
338369
project = google_project.project.project_id
339370
service = "compute.googleapis.com"
340371
}
341372
342373
resource "google_project_service" "servicenetworking" {
343-
project = google_project.project.project_id
344-
service = "servicenetworking.googleapis.com"
374+
provider = google-beta
375+
project = google_project.project.project_id
376+
service = "servicenetworking.googleapis.com"
345377
depends_on = [google_project_service.compute]
346378
}
347379
348380
resource "google_project_service" "kms" {
381+
provider = google-beta
349382
project = google_project.project.project_id
350383
service = "cloudkms.googleapis.com"
351384
depends_on = [google_project_service.servicenetworking]
352385
}
353386
354387
resource "google_project_service" "eventarc" {
388+
provider = google-beta
355389
project = google_project.project.project_id
356390
service = "eventarc.googleapis.com"
357391
depends_on = [google_project_service.kms]
358392
}
359393
360394
resource "google_kms_key_ring" "keyring" {
395+
provider = google-beta
361396
name = "keyring"
362397
location = "%{region}"
363398
project = google_project.project.project_id
364399
depends_on = [google_project_service.kms]
365400
}
366401
367402
resource "google_kms_crypto_key" "key" {
403+
provider = google-beta
368404
name = "key1"
369405
key_ring = google_kms_key_ring.keyring.id
370406
}
371407
372408
resource "google_kms_crypto_key" "key_update" {
409+
provider = google-beta
373410
name = "key2"
374411
key_ring = google_kms_key_ring.keyring.id
375412
}
376413
377414
resource "google_project_service_identity" "eventarc_sa" {
415+
provider = google-beta
378416
service = "eventarc.googleapis.com"
379417
project = google_project.project.project_id
380418
depends_on = [google_project_service.eventarc]
381419
}
382420
383421
resource "google_kms_crypto_key_iam_member" "eventarc_sa_keyuser" {
422+
provider = google-beta
384423
crypto_key_id = google_kms_crypto_key.key.id
385424
role = "roles/cloudkms.cryptoKeyEncrypterDecrypter"
386425
member = google_project_service_identity.eventarc_sa.member
387426
}
388427
389428
resource "google_kms_crypto_key_iam_member" "eventarc_sa_keyuser_update" {
429+
provider = google-beta
390430
crypto_key_id = google_kms_crypto_key.key_update.id
391431
role = "roles/cloudkms.cryptoKeyEncrypterDecrypter"
392432
member = google_project_service_identity.eventarc_sa.member
393433
}
394434
395435
resource "google_project" "project_update" {
436+
provider = google-beta
396437
project_id = "tf-test2%{random_suffix}"
397438
name = "tf-test2%{random_suffix}"
398439
org_id = "%{org_id}"
@@ -401,24 +442,28 @@ resource "google_project" "project_update" {
401442
}
402443
403444
resource "google_project_service" "eventarc_update" {
445+
provider = google-beta
404446
project = google_project.project_update.project_id
405447
service = "eventarc.googleapis.com"
406448
}
407449
408450
resource "google_project_service_identity" "eventarc_sa_update" {
451+
provider = google-beta
409452
project = google_project.project_update.project_id
410453
service = "eventarc.googleapis.com"
411454
depends_on = [google_project_service.eventarc_update]
412455
}
413456
414457
resource "google_eventarc_message_bus" "message_bus_update" {
458+
provider = google-beta
415459
location = "%{region}"
416460
message_bus_id = "tf-test-messagebus2%{random_suffix}"
417461
project = google_project.project_update.project_id
418462
depends_on = [google_project_service_identity.eventarc_sa_update]
419463
}
420464
421465
resource "google_eventarc_google_api_source" "primary" {
466+
provider = google-beta
422467
location = "%{region}"
423468
google_api_source_id = "tf-test-googleapisource%{random_suffix}"
424469
project = google_project.project.project_id

0 commit comments

Comments
 (0)