Skip to content

Commit 00f4e18

Browse files
authored
Sync main oct 27 feature branch resource identity (#15430)
2 parents be07d5b + 78c0a3d commit 00f4e18

File tree

877 files changed

+30666
-5591
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

877 files changed

+30666
-5591
lines changed

.ci/containers/build-environment/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM golang:1.23-bullseye AS builder
1+
FROM golang:1.24-bullseye AS builder
22

33
# Set working directory
44
WORKDIR /app

.ci/containers/go-plus/Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# Stage 1: Download go module cache for builds
2-
FROM golang:1.23-bullseye AS builder
2+
FROM golang:1.24-bullseye AS builder
33
ENV GOCACHE=/go/cache
44

55
RUN apt-get update && apt-get install -y unzip
@@ -12,7 +12,7 @@ WORKDIR /app1/magic-modules-main/.ci/magician
1212
RUN go build -o /dev/null .
1313

1414
# Stage 2: Creating the final image
15-
FROM golang:1.23-bullseye
15+
FROM golang:1.24-bullseye
1616
SHELL ["/bin/bash", "-c"]
1717
ENV GOCACHE=/go/cache
1818

.ci/gcb-contributor-membership-checker.yml

Lines changed: 0 additions & 78 deletions
This file was deleted.

.ci/infra/terraform/main.tf

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,18 @@ resource "google_organization_iam_member" "sa_principal_access_boundary_admin" {
171171
member = google_service_account.sa.member
172172
}
173173

174+
resource "google_organization_iam_member" "dlp_admin" {
175+
org_id = data.google_organization.org.org_id
176+
role = "roles/dlp.admin"
177+
member = google_project_service_identity.dlp_sa.member
178+
}
179+
180+
resource "google_organization_iam_member" "dlp_org_driver" {
181+
org_id = data.google_organization.org.org_id
182+
role = "roles/dlp.orgdriver"
183+
member = google_project_service_identity.dlp_sa.member
184+
}
185+
174186
resource "google_billing_account_iam_member" "sa_master_billing_admin" {
175187
billing_account_id = data.google_billing_account.master_acct.id
176188
role = "roles/billing.admin"
@@ -315,6 +327,7 @@ module "project-services" {
315327
"networksecurity.googleapis.com",
316328
"networkservices.googleapis.com",
317329
"notebooks.googleapis.com",
330+
"observability.googleapis.com",
318331
"orgpolicy.googleapis.com",
319332
"osconfig.googleapis.com",
320333
"oslogin.googleapis.com",
@@ -420,6 +433,14 @@ resource "google_project_service_identity" "parametermanager_sa" {
420433
service = "parametermanager.googleapis.com"
421434
}
422435

436+
resource "google_project_service_identity" "dlp_sa" {
437+
provider = google-beta
438+
depends_on = [module.project-services]
439+
440+
project = google_project.proj.project_id
441+
service = "dlp.googleapis.com"
442+
}
443+
423444
# TestAccComposerEnvironment_fixPyPiPackages
424445
# TestAccComposerEnvironmentComposer2_private
425446
# TestAccComposerEnvironment_withEncryptionConfigComposer1

.ci/magician/cmd/membership_checker.go

Lines changed: 0 additions & 89 deletions
This file was deleted.

.ci/magician/cmd/membership_checker_test.go

Lines changed: 0 additions & 125 deletions
This file was deleted.

.ci/magician/cmd/reassign_reviewer.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,8 @@ func execReassignReviewer(prNumber, newPrimaryReviewer string, gh GithubClient)
111111
return err
112112
}
113113

114+
gh.RemoveLabel(prNumber, "assign-reviewer")
115+
114116
return nil
115117
}
116118

.ci/magician/cmd/test_terraform_vcr.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package cmd
22

33
import (
4+
"errors"
45
"fmt"
56
"os"
67
"path/filepath"
@@ -470,9 +471,7 @@ func runReplaying(runFullVCR bool, version provider.Version, services map[string
470471
Version: version,
471472
TestDirs: []string{servicePath},
472473
})
473-
if serviceReplayingErr != nil {
474-
replayingErr = serviceReplayingErr
475-
}
474+
replayingErr = errors.Join(replayingErr, serviceReplayingErr)
476475
result.PassedTests = append(result.PassedTests, serviceResult.PassedTests...)
477476
result.SkippedTests = append(result.SkippedTests, serviceResult.SkippedTests...)
478477
result.FailedTests = append(result.FailedTests, serviceResult.FailedTests...)

0 commit comments

Comments
 (0)