Skip to content

Kubernetes deployment#1286

Merged
jb3 merged 5 commits into
masterfrom
kubernetes-deploy
Nov 19, 2020
Merged

Kubernetes deployment#1286
jb3 merged 5 commits into
masterfrom
kubernetes-deploy

Conversation

@jb3
Copy link
Copy Markdown
Member

@jb3 jb3 commented Nov 15, 2020

This PR adds a Kubernetes manifest for deploying our bot to our Kubernetes cluster.

It adds the necessary steps to the deployment workflow to auto-deploy on pushes to our default branch.

It also changes some constants to connect to the right services with the new Kubernetes DNS.

@jb3 jb3 requested a review from a team as a code owner November 15, 2020 20:02
@jb3 jb3 requested review from jerbob and mbaruh and removed request for a team November 15, 2020 20:02
@jb3 jb3 added a: CI Related to continuous intergration and deployment p: 0 - critical Needs to be addressed ASAP t: feature New feature or request labels Nov 15, 2020
@ghost ghost added the needs 2 approvals label Nov 15, 2020
@jb3
Copy link
Copy Markdown
Member Author

jb3 commented Nov 15, 2020

Actually, this cannot be merged until snekbox is ready.

@jb3 jb3 marked this pull request as draft November 15, 2020 20:04
@jb3 jb3 marked this pull request as ready for review November 19, 2020 00:27
@coveralls
Copy link
Copy Markdown

coveralls commented Nov 19, 2020

Coverage Status

Coverage remained the same at 56.519% when pulling 5400642 on kubernetes-deploy into 1da131a on master.

Copy link
Copy Markdown
Contributor

@lemonsaurus lemonsaurus left a comment

Choose a reason for hiding this comment

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

Looks bunny! 🐰

@jb3 jb3 merged commit 506909e into master Nov 19, 2020
@jb3 jb3 deleted the kubernetes-deploy branch November 19, 2020 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: CI Related to continuous intergration and deployment p: 0 - critical Needs to be addressed ASAP t: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants