-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Failing test(s): TestAccCloudbuildv2Connection* (beta) #14390
Failing test(s): TestAccCloudbuildv2Connection* (beta) #14390
Comments
@roaks3 Could this be due to the test project split? It looks like the Cloud Build Service Agent from the test projects need to be given permissions on this secret from a separate project : https://github.com/hashicorp/terraform-provider-google-beta/blob/f7997b603c082fdd6680e4ab1573d034c1559bf4/google-beta/iam_cloudbuildv2_connection_generated_test.go#L126 Edit: plus other permission issues from the tests |
Note: A googler likely needs to grant permissions to these internally |
b/280100395 |
Forwarding this as it's an issue with test infrastructure owned by the service team. |
@SarahFrench Yes, access to these secrets was granted for our original project, so we did the same for the new GA/Beta projects, but it appears that the process has become more restrictive since it was originally done, and we're in-progress with granting permission using the new process. The service team is owning this: b/278293277 and b/278547809. I'm also updating the description with 2 more tests affected by this: |
I spoke with the service team on this yesterday, and it looks like this could take a bit longer to resolve (and the team may need to use an alternate approach). So for now, we can skip these tests with GoogleCloudPlatform/magic-modules#7867, and revert that change once we get things working. cc @vicpadilla |
I'm seeing 100% failure of
Same for |
@vicpadilla I thought I saw some follow-up changes on your end, but I don't see them referenced here. Pinging you to make sure you're aware, error is the same as before. We can of course skip again if this requires some time. |
Could you retry those tests? We were missing an IAM permission (for networking) for the P4SA of the nightly-beta and nightly-ga projects. I've just added those permissions. |
👍 it already failed from last night's run, so probably easiest to wait for the next run tonight |
Permission issue appears resolved from last night's run. Thanks for resolving @vicpadilla ! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Failure rates
Impacted tests
Affected Resource(s)
Nightly build test history
Message(s)
Error about not having permissions to access a secret from a different project:
A different missing permission (Permission 'servicedirectory.services.resolve' denied on resource 'projects/gcb-terraform-creds/locations/us-central1/namespaces/myns/services/serv'.):
The text was updated successfully, but these errors were encountered: