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

CI: Add cloudrun deploy script #74

Merged
merged 2 commits into from
Aug 11, 2023
Merged

Conversation

spowelljr
Copy link
Collaborator

No description provided.

@spowelljr spowelljr changed the title CI: Add cloudbuild deploy script CI: Add cloudrun deploy script Aug 10, 2023
COPY pkg ./pkg
RUN CGO_ENABLED=0 GOARCH=amd64 GOOS=linux go build -o /dbbrowser

FROM alpine:3
Copy link
Owner

@medyagh medyagh Aug 11, 2023

Choose a reason for hiding this comment

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

this could probably be from Scratch instead of alpine, no ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have to do RUN apk add --no-cache ca-certificates, I don't think scratch supports that, I'll double check though

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, it fails without alpine, and if you don't run that command you get an error when the apps starting

tls: failed to verify certificate: x509: certificate signed by unknown authority cloudrun

@@ -0,0 +1,14 @@
FROM golang:1.21 AS builder
Copy link
Owner

Choose a reason for hiding this comment

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

Copy link
Owner

@medyagh medyagh left a comment

Choose a reason for hiding this comment

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

changes could be done in follow up PR too

@spowelljr spowelljr merged commit 7511214 into medyagh:master Aug 11, 2023
2 checks passed
@spowelljr spowelljr deleted the cloudbuild branch August 11, 2023 22:08
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