Skip to content

Commit b19490f

Browse files
authored
Merge branch 'GoogleCloudPlatform:main' into main
2 parents 4c48b33 + 2e72ae2 commit b19490f

113 files changed

Lines changed: 5827 additions & 1154 deletions

File tree

Some content is hidden

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

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

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

.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.

.github/workflows/unit-test-mmv1.yml

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,33 @@ jobs:
4242
if [[ -n "$tmpls" ]]; then
4343
go run main.go version-guard --file-list $tmpls
4444
fi
45+
unused-template-check:
46+
runs-on: ubuntu-22.04
47+
steps:
48+
- name: Checkout Repository
49+
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.2
50+
with:
51+
path: repo
52+
fetch-depth: 0
53+
- name: Merge base branch
54+
if: github.event_name == 'pull_request'
55+
id: pull_request
56+
run: |
57+
cd repo
58+
git config user.name "modular-magician"
59+
git config user.email "magic-modules@google.com"
60+
git fetch origin ${{ github.base_ref }} # Fetch the base branch
61+
git merge --no-ff origin/${{ github.base_ref }} # Merge with the base branch
62+
- name: Check new product templates are linked
63+
id: new_templates
64+
run: |
65+
cd repo/tools/template-check
66+
newtmplfiles=$(git diff --name-only --diff-filter=A origin/${{ github.base_ref }} HEAD -- ../../mmv1 | grep .tmpl | sed 's=^=../../=g')
67+
if [ ! -z "$newtmplfiles" ]; then
68+
newtmplfiles=${newtmplfiles//$'\n'/,}
69+
echo $newtmplfiles
70+
go run main.go unused-tmpl --file-list $newtmplfiles
71+
fi
4572
lint-yaml:
4673
runs-on: ubuntu-22.04
4774
steps:

mmv1/api/type.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -851,7 +851,7 @@ func (t Type) GetFWType() string {
851851
case "ResourceRef":
852852
return "String"
853853
case "NestedObject":
854-
return "Nested"
854+
return "Object"
855855
case "Array":
856856
return "List"
857857
case "KeyValuePairs":

mmv1/products/apihub/Plugin.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,7 @@ properties:
130130
- name: configTemplate
131131
type: NestedObject
132132
description: ConfigTemplate represents the configuration template for a plugin.
133+
default_from_api: true
133134
properties:
134135
- name: authConfigTemplate
135136
type: NestedObject

mmv1/products/certificatemanager/Certificate.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,28 +66,33 @@ examples:
6666
dns_auth_name2: 'dns-auth2'
6767
dns_auth_subdomain2: 'subdomain2'
6868
cert_name: 'dns-cert'
69+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
6970
- name: 'certificate_manager_google_managed_certificate_issuance_config'
7071
primary_resource_id: 'default'
7172
vars:
7273
cert_name: 'issuance-config-cert'
7374
ca_name: 'ca-authority'
7475
pool_name: 'ca-pool'
7576
issuance_config_name: 'issuance-config'
77+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
7678
- name: 'certificate_manager_self_managed_certificate'
7779
primary_resource_id: 'default'
7880
vars:
7981
cert_name: 'self-managed-cert'
82+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
8083
- name: 'certificate_manager_self_managed_certificate_regional'
8184
primary_resource_id: 'default'
8285
vars:
8386
cert_name: 'self-managed-cert'
87+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
8488
- name: 'certificate_manager_google_managed_certificate_issuance_config_all_regions'
8589
primary_resource_id: 'default'
8690
vars:
8791
cert_name: 'issuance-config-cert'
8892
ca_name: 'ca-authority'
8993
pool_name: 'ca-pool'
9094
issuance_config_name: 'issuance-config'
95+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
9196
- name: 'certificate_manager_google_managed_certificate_dns_all_regions'
9297
primary_resource_id: 'default'
9398
vars:
@@ -96,16 +101,19 @@ examples:
96101
dns_auth_name2: 'dns-auth2'
97102
dns_auth_subdomain2: 'subdomain2'
98103
cert_name: 'dns-cert'
104+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
99105
- name: 'certificate_manager_google_managed_regional_certificate_dns_auth'
100106
primary_resource_id: 'default'
101107
vars:
102108
dns_auth_name: 'dns-auth'
103109
dns_auth_subdomain: 'subdomain'
104110
cert_name: 'dns-cert'
111+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
105112
- name: 'certificate_manager_client_auth_certificate'
106113
primary_resource_id: 'default'
107114
vars:
108115
cert_name: 'client-auth-cert'
116+
tgc_skip_test: 'Need to figure out to get the desired cai asset state.'
109117
parameters:
110118
- name: 'name'
111119
type: String

0 commit comments

Comments
 (0)