terraform-google-iam icon indicating copy to clipboard operation
terraform-google-iam copied to clipboard

feat(cloud run service iam): :sparkles: cloud run services iam submodule

Open rs1986x opened this issue 3 years ago • 2 comments

Hello! this is a IAM module for cloud run services, hope you find it useful

rs1986x avatar Oct 03 '22 13:10 rs1986x

Thanks for the PR! 🚀
✅ Lint checks have passed.

comment-bot-dev avatar Oct 03 '22 13:10 comment-bot-dev

i can't seem to pass the cicd build, can somebody share the error output so i can address it? thanks

rs1986x avatar Oct 04 '22 06:10 rs1986x

Hello! would it be possible to get a feedback on this please? many thanks 😄

rs1986x avatar Nov 08 '22 19:11 rs1986x

Hi @rs1986x, thanks for the contribution! It appears there is an error in the Custom Role integration tests:

  ×  GCP Custom Role: Custom Role (1 failed)
     ×  Command: `gcloud iam roles describe customDatastoreViewer_525f --organization 943740911108 --format=json` project_unsupported_custom_role does not have permissions
     expected ["gkehub.features.get", "gkehub.features.getIamPolicy", "gkehub.features.list", "gkehub.locations.get...ons.get", "gkehub.operations.list", "resourcemanager.projects.get", "resourcemanager.projects.list"] not to include "gkehub.features.list"

apeabody avatar Nov 08 '22 20:11 apeabody

Heads up that there are some copy/paste typos in the two README.md files that refer to PubSub Topic.

andrewmackett avatar Nov 23 '22 14:11 andrewmackett

Hi @rs1986x, thanks for the contribution! It appears there is an error in the Custom Role integration tests:

  ×  GCP Custom Role: Custom Role (1 failed)
     ×  Command: `gcloud iam roles describe customDatastoreViewer_525f --organization 943740911108 --format=json` project_unsupported_custom_role does not have permissions
     expected ["gkehub.features.get", "gkehub.features.getIamPolicy", "gkehub.features.list", "gkehub.locations.get...ons.get", "gkehub.operations.list", "resourcemanager.projects.get", "resourcemanager.projects.list"] not to include "gkehub.features.list"

Hi @apeabody thanks for your answer. Could you help me understand where the error is? when i did create the PR, I applied the example in a dev project and worked out of the box, i don't know how to run all your suite of tests now why is the pipeline failing but i would like to understand. I am not sure i know how to fix the integration tests.

@andrewmackett thanks for the comment i will fix that right away 😄

rs1986x avatar Dec 06 '22 10:12 rs1986x

Thanks for the contribution @rs1986x - Looks like the error might have been transitory and all the checks have now passed. :)

apeabody avatar Dec 06 '22 18:12 apeabody

Thanks @apeabody! my pleasure

rs1986x avatar Dec 06 '22 20:12 rs1986x