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

add http4k #113

Merged
merged 4 commits into from Jan 22, 2021
Merged

add http4k #113

merged 4 commits into from Jan 22, 2021

Conversation

jamesward
Copy link
Contributor

No description provided.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Dec 11, 2020
Copy link
Collaborator

@bshaffer bshaffer left a comment

Choose a reason for hiding this comment

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

@jamesward for starters can we add this to the github actions tests.yaml file?

@jamesward
Copy link
Contributor Author

Done.

with:
arguments: test --console=plain --info
build-root-directory: run/grpc-hello-world-bidi-streaming
wrapper-directory: run/grpc-hello-world-bidi-streaming
Copy link
Collaborator

Choose a reason for hiding this comment

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

🥂 I was hoping you'd add this one!!

1. Build the container image on Cloud Build using Buildpacks (currently an alpha feature), storing the image on Google Container Registry:
```
export PROJECT_ID=YOUR_GCP_PROJECT
gcloud alpha builds submit --pack=image=gcr.io/$PROJECT_ID/http4k-hello-world
Copy link
Collaborator

Choose a reason for hiding this comment

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

is alpha intentional here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just a copy & paste thing. Will resolve with #117

Copy link
Collaborator

@bshaffer bshaffer left a comment

Choose a reason for hiding this comment

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

Tested local, docker local, and CLI deploy on Cloud Run. Worked like a charm. Nice job!

... one question, otherwise approved 👍

@jamesward jamesward mentioned this pull request Jan 22, 2021
@jamesward jamesward merged commit 02ffd1d into master Jan 22, 2021
@jamesward jamesward deleted the http4k branch January 22, 2021 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants