Skip to content

Conversation

@steinwaywhw
Copy link
Contributor

The GOOGLE_CLOUD_TESTS_VPCSC_OUTSIDE_PERIMETER_PROJECT should be a project outside of the VPCSC perimeter. PROJECT_ID is a project inside the perimeter. Do not uncomment until we have a project for the second environment variable. Otherwise this wrong value is breaking the system tests of the VPCSC team.

The `GOOGLE_CLOUD_TESTS_VPCSC_OUTSIDE_PERIMETER_PROJECT` should be a project outside of the VPCSC perimeter. `PROJECT_ID` is a project inside the perimeter. Do not uncomment until we have a project for the second environment variable. Otherwise this wrong value is breaking the system tests of the VPCSC team.
@steinwaywhw steinwaywhw requested a review from busunkim96 as a code owner June 12, 2019 19:07
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 12, 2019
@busunkim96
Copy link
Contributor

Hi @steinwaywhw, I think this was my mistake when I set the environment variables.

Would a setup like this result in the tests running correctly?

PROJECT_OUTISDE -> the default project we use for tests in this repo (outside the perimeter)
PROJECT_INSIDE -> secure-gcp-test-project

@busunkim96 busunkim96 added api: monitoring Issues related to the Cloud Monitoring API. testing labels Jun 13, 2019
@steinwaywhw
Copy link
Contributor Author

Hi @busunkim96 no worries. Yes, your suggested value for the environment variables would work.

@tseaver tseaver changed the title Update noxfile.py Monitoring: Update noxfile.py Jun 19, 2019
@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Jun 19, 2019
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Jul 3, 2019
@tseaver
Copy link
Contributor

tseaver commented Jul 30, 2019

Superseded by #8302.

@tseaver tseaver closed this Jul 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: monitoring Issues related to the Cloud Monitoring API. cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants