Skip to content

Some changes for GKE#3

Merged
imjohnbo merged 2 commits intogithub-developer:masterfrom
bharathkkb:gke-improvements
May 13, 2020
Merged

Some changes for GKE#3
imjohnbo merged 2 commits intogithub-developer:masterfrom
bharathkkb:gke-improvements

Conversation

@bharathkkb
Copy link
Copy Markdown
Contributor

fixes #2
Changelist:

@bharathkkb
Copy link
Copy Markdown
Contributor Author

@imjohnbo ci error seems to due to GCP_PROJECT env var? PTAL

Copy link
Copy Markdown
Contributor

@imjohnbo imjohnbo left a comment

Choose a reason for hiding this comment

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

Thanks so much for the pull request, @bharathkkb! Small request to keep TOKEN as-is or use a different name than GITHUB_TOKEN.

I think the CI won't pass just yet because it uses secrets and it came from a fork.

Comment thread deployment.yml Outdated
Comment thread deployment.yml Outdated
Comment thread startup.sh
@bharathkkb
Copy link
Copy Markdown
Contributor Author

@imjohnbo thanks for the review, I will fix these up today :)

docs: update README for Anthos usage

fix: clean up workflow and use kustomize

remove env check for now

fix: docker auth

fix: deploy

CI: bump

CI: bump

chore: scale down replicas to 1

CI: bump

chore: add echo statements for debugging

fix: ubuntu 18

fix: let bin/installdependencies.sh take care of updates
Copy link
Copy Markdown
Contributor

@imjohnbo imjohnbo left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@imjohnbo imjohnbo merged commit 967f832 into github-developer:master May 13, 2020
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.

Using Container Lifecycle Hooks vs capturing exit signals for runner removal

2 participants