Skip to content

Conversation

siddharthab
Copy link
Contributor

@siddharthab siddharthab commented Mar 5, 2023

The gzip content encoding is not right for a zip archive and does not
actually work. When working with this module, I had to remove the
content-encoding to make things work. Otherwise, I was getting a
"End-of-central-directory signature not found" error in Cloud Build
during function deployment.

When I tried to download the archive using my browser, the browser
errored, but gsutil succeeded. The file uploaded to GCS was complete so
that was not the reason for the error. Removing the content-encoding
field fixed the deployment.

@siddharthab siddharthab requested a review from a team as a code owner March 5, 2023 18:08
@siddharthab siddharthab changed the title Remove gzip content encoding on function archive fix: remove gzip content-encoding metadata on function source archive Mar 5, 2023
siddharthab and others added 2 commits March 5, 2023 18:22
The gzip content encoding is not right for a zip archive and does not
actually work. When working with this module, I had to remove the
content-encoding to make things work. Otherwise, I was getting a
"End-of-central-directory signature not found" error in Cloud Build
during function deployment.

When I tried to download the archive using my browser, the browser
errored, but gsutil succeeded. The file uploaded to GCS was complete so
that was not the reason for the error. Removing the content-encoding
field fixed the deployment.
@apeabody
Copy link
Contributor

apeabody commented Mar 6, 2023

/gcbrun

@apeabody
Copy link
Contributor

apeabody commented Mar 6, 2023

/gcbrun

Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution @siddharthab

@apeabody apeabody merged commit 4995870 into terraform-google-modules:master Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants