Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

build: update system-test and sample-test config#1118

Merged
BenWhitehead merged 1 commit intogoogleapis:masterfrom
BenWhitehead:integration-test-config-change
Jun 9, 2020
Merged

build: update system-test and sample-test config#1118
BenWhitehead merged 1 commit intogoogleapis:masterfrom
BenWhitehead:integration-test-config-change

Conversation

@BenWhitehead
Copy link
Copy Markdown
Contributor

@BenWhitehead BenWhitehead commented Jun 9, 2020

Add new script .kokoro/populate-secrets.sh which supports the new
SECRET_MANAGER_KEYS configured for each build.

Update kokoro trampoline to call populate-secrets.sh

Update system-test config to use a new project java-review and a new
secret for its credentials.

Update samples-test config to use a new project nodejs-firestore-ci
and a new secret for its credentials.

@BenWhitehead BenWhitehead requested a review from bcoe June 9, 2020 19:40
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 9, 2020
Copy link
Copy Markdown

@bcoe bcoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me @BenWhitehead, but since we haven't been picking up updates to .kokoro/, we'll need to add the populate-secrets.sh to .kokoro/, as is done here:

https://github.com/googleapis/nodejs-secret-manager/pull/110/files#diff-a74d0823026a747b911d37d9a24d8c89

and also update the trampoline.sh to call it.

After we do this, I think we might also want to add .kokoro to the exclude rules in synth.py, since this using another service account than our other libraries (I think this is fine, and gets us on our feet).

Comment thread .kokoro/populate-secrets.sh Outdated
Comment thread .kokoro/populate-secrets.sh Outdated
Comment thread .kokoro/populate-secrets.sh Outdated
Add new script .kokoro/populate-secrets.sh which supports the new
SECRET_MANAGER_KEYS configured for each build.

Update kokoro trampoline to call populate-secrets.sh

Update system-test config to use a new project `java-review` and a new
secret for its credentials.

Update samples-test config to use a new project `nodejs-firestore-ci`
and a new secret for its credentials.
@BenWhitehead BenWhitehead changed the title build: update system-test config to use secret manager build: update system-test and sample-test config Jun 9, 2020
@BenWhitehead BenWhitehead merged commit 325e7ff into googleapis:master Jun 9, 2020
@BenWhitehead BenWhitehead deleted the integration-test-config-change branch June 9, 2020 23:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants