Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update int test to receive access context manager policy rather than create + destroy #103

Open
bharathkkb opened this issue Jan 6, 2023 · 2 comments
Labels
bug Something isn't working triaged Scoped and ready for work

Comments

@bharathkkb
Copy link
Member

TL;DR

We do a few clean ups between tests like https://github.com/terraform-google-modules/terraform-google-vpc-service-controls/blob/0d94f375b9e4aa98d830b02e53b34cc83c79075b/build/int.cloudbuild.yaml#L26that can result in the access context manager policy not existing. It would be better to skip testing the main root module as it is a single resource without any logic and instead receive a static policy as a var for testing other more complex submodules.

Expected behavior

No response

Observed behavior

No response

Terraform Configuration

N/A

Terraform Version

N/A

Additional information

No response

@github-actions
Copy link

github-actions bot commented Mar 7, 2023

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the Stale label Mar 7, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 15, 2023
@bharathkkb bharathkkb reopened this Apr 5, 2023
@bharathkkb bharathkkb added triaged Scoped and ready for work and removed Stale labels Apr 5, 2023
@apeabody
Copy link
Contributor

Looks like this is also breaking the terraform-example-foundation test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triaged Scoped and ready for work
Projects
None yet
Development

No branches or pull requests

2 participants