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

Storage Bucket environment variable name not consistent #62

Closed
frankyn opened this issue Sep 22, 2016 · 1 comment
Closed

Storage Bucket environment variable name not consistent #62

frankyn opened this issue Sep 22, 2016 · 1 comment
Labels
code health 🚨 This issue needs some love. triage me I really want to be triaged.
Milestone

Comments

@frankyn
Copy link
Member

frankyn commented Sep 22, 2016

Story

Trying to run RSPEC on samples which use a bucket id isn't straight forward. It always documented either. It's hidden in the SPEC.

Issue
No consistent ENV name was used. A quick grep reveals the following.

bigquery/spec/bigquery_sample_spec.rb:
appengine/storage/app.rb:bucket = storage.bucket ENV["GCLOUD_STORAGE_BUCKET"]
appengine/storage/README.md:export GCLOUD_STORAGE_BUCKET=
appengine/storage/spec/storage_spec.rb: .sub("", ENV["GCLOUD_STORAGE_BUCKET"])
appengine/storage/app.yaml: GCLOUD_STORAGE_BUCKET:
language/spec/language_samples_spec.rb: @bucket_name = ENV["STORAGE_BUCKET"]
logging/spec/sample_spec.rb: # BUCKET Name of Google Cloud Storage bucket to use for log sink
logging/spec/sample_spec.rb: # ALT_BUCKET Name of an alternative bucket to also use for log sink
logging/spec/sample_spec.rb: @bucket = @storage.bucket ENV["BUCKET"]
logging/spec/sample_spec.rb: @alt_bucket = @storage.bucket ENV["ALT_BUCKET"]
storage/spec/list_buckets_spec.rb: output(/#{BUCKET_NAME}/).to_stdout)
storage/spec/spec_helper.rb:BUCKET_NAME=ENV["GOOGLE_PROJECT_ID"]

Possible-Solution

Use a consistent name , maybe, GOOGLE_STORAGE_BUCKET_ID ?

@frankyn
Copy link
Member Author

frankyn commented Dec 11, 2017

Resolved.

@frankyn frankyn closed this as completed Dec 11, 2017
@yoshi-automation yoshi-automation added triage me I really want to be triaged. 🚨 This issue needs some love. labels Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code health 🚨 This issue needs some love. triage me I really want to be triaged.
Projects
None yet
Development

No branches or pull requests

3 participants