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

style: default values for name parameter in Cloud Run samples #164

Open
rogerthatdev opened this issue Sep 15, 2022 · 2 comments · May be fixed by #739
Open

style: default values for name parameter in Cloud Run samples #164

rogerthatdev opened this issue Sep 15, 2022 · 2 comments · May be fixed by #739
Labels
type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@rogerthatdev
Copy link
Contributor

rogerthatdev commented Sep 15, 2022

TL;DR

Currently, in Cloud Run samples, the google_cloud_run_service resource uses various generic values as the name, such ascloudrun-srv, cloud-run-service-name. For the sake of consistency, these names should be either meaningful to their intended function (example) or have a default generic value (something like my-cloud-run-service).

Terraform Resources

google_cloud_run_service

Detailed design

No response

Additional information

No response

@rsamborski rsamborski added the type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. label Sep 4, 2023
glasnt added a commit that referenced this issue Jan 16, 2024
@glasnt
Copy link
Contributor

glasnt commented Jan 16, 2024

Since this issue was logged, the only samples that use generic names appear to be:

msampathkumar pushed a commit that referenced this issue Jan 16, 2024
* fix: apply style guide to run/tasks

* fix: update service name (#164)
@msampathkumar
Copy link
Contributor

Quick update: I was checking to know if this issues is still valid, and I found few areas to improve. So not closing this request.

noauth/main.tf:  name     = "cloudrun-srv"
..
remove_tag/main.tf:  name     = "cloudrun-srv"
secret_manager/main.tf:  name     = "cloudrun-srv-env-var-secret"
..
traffic_gradual_rollout/main.tf:  name     = "cloudrun-srv"
traffic_latest_revision/main.tf:  name     = "cloudrun-srv"
traffic_rollback/main.tf:  name     = "cloudrun-srv"
traffic_split/main.tf:  name     = "cloudrun-srv"
traffic_split_tag/main.tf:  name     = "cloudrun-srv"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
5 participants
@glasnt @rsamborski @msampathkumar @rogerthatdev and others