-
Notifications
You must be signed in to change notification settings - Fork 148
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
Avoid downloading image from GCS when generating website #38
Comments
bobcatfish
added a commit
to bobcatfish/website
that referenced
this issue
Mar 18, 2020
The DEVELOPMENT.md was very out of date! Also uncovered some difficult to work with things along the way such as tektoncd#38
2 tasks
bobcatfish
added a commit
to bobcatfish/website
that referenced
this issue
Mar 18, 2020
The DEVELOPMENT.md was very out of date! Also uncovered some difficult to work with things along the way such as tektoncd#38 which made it so that you need credentials to access the GCP bucket with the background image in order to be able to run the sync script. I've added a very hacky env var which lets one skip this until we tackle tektoncd#38
tekton-robot
pushed a commit
that referenced
this issue
Mar 19, 2020
The DEVELOPMENT.md was very out of date! Also uncovered some difficult to work with things along the way such as #38 which made it so that you need credentials to access the GCP bucket with the background image in order to be able to run the sync script. I've added a very hacky env var which lets one skip this until we tackle #38
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Expected Behavior
It should be possible to build and preview the website without requiring access to a GCS bucket for the background image.
Actual Behavior
The makefile commands used to build and run the website run sync.py which requires GCP credentials:
website/sync/sync.py
Lines 26 to 31 in 43e149b
It seems like the reason is to grab the background image:
website/sync/sync.py
Lines 105 to 107 in 43e149b
Additional Info
Maybe we can commit the background image to the repo instead? I have a feeling I'm missing something here
The text was updated successfully, but these errors were encountered: