Skip to content

Commit 952223c

Browse files
authored
chore(metadata): multiple args now supported by dev-tools (#2430)
1 parent b4b12f1 commit 952223c

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

Makefile

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,14 +76,13 @@ docker_test_lint:
7676
/usr/local/bin/test_lint.sh
7777

7878
# Generate documentation
79-
# Removed `display` for https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/issues/3140
8079
.PHONY: docker_generate_docs
8180
docker_generate_docs:
8281
$(DOCKER_BIN) run --rm -it \
8382
-e ENABLE_BPMETADATA=1 \
8483
-v "$(CURDIR)":/workspace \
8584
$(REGISTRY_URL)/${DOCKER_IMAGE_DEVELOPER_TOOLS}:${DOCKER_TAG_VERSION_DEVELOPER_TOOLS} \
86-
/bin/bash -c 'source /usr/local/bin/task_helper_functions.sh && generate_docs --per-module-requirements'
85+
/bin/bash -c 'source /usr/local/bin/task_helper_functions.sh && generate_docs -d --per-module-requirements'
8786

8887
# Generate files from autogen
8988
.PHONY: docker_generate_modules

modules/gke-node-pool/metadata.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,9 +403,9 @@ spec:
403403
roles:
404404
- level: Project
405405
roles:
406+
- roles/compute.admin
406407
- roles/container.admin
407408
- roles/iam.serviceAccountUser
408-
- roles/compute.admin
409409
services:
410410
- compute.googleapis.com
411411
- container.googleapis.com

modules/gke-standard-cluster/metadata.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1008,9 +1008,9 @@ spec:
10081008
roles:
10091009
- level: Project
10101010
roles:
1011+
- roles/iam.serviceAccountUser
10111012
- roles/compute.admin
10121013
- roles/container.admin
1013-
- roles/iam.serviceAccountUser
10141014
services:
10151015
- compute.googleapis.com
10161016
- container.googleapis.com

0 commit comments

Comments
 (0)