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.

Sign up for GitHub

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

Jump to bottom

fix: add get project perm to destroy inception #147

Merged

krtk6160 merged 2 commits into main from fix--perm-for-vpc-destroy

Nov 9, 2023

Member

krtk6160 commented Nov 9, 2023 •

edited

Loading

I have tested the perm by adding it to a service account, impersonating it and trying to delete the google_service_networking_connection resource.

krtk6160 added 2 commits

November 9, 2023 18:21


          fix: add get project perm to destroy inception

287ed66


          refactor: success check logic

19378ae

krtk6160 requested a review from bodymindarts

November 9, 2023 12:54

bodymindarts approved these changes

View reviewed changes

krtk6160 merged commit aafb9a2 into main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet