Skip to content
This repository has been archived by the owner on Jul 15, 2021. It is now read-only.

updated readme.md #173

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

pbisen
Copy link

@pbisen pbisen commented Oct 1, 2020

replaced referenced to heptio contour by Project Contour
also replaced link to heptio contour github page by link to Project Contour.

replaced referenced to heptio contour by Project Contour
also replaced link to helption contour github page by link to Project Contour.
Copy link
Contributor

@stevesloka stevesloka left a comment

Choose a reason for hiding this comment

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

Just one small nit. Also you need to sign the commit to make the DCO robot happy.

docs/README.md Outdated
@@ -6,7 +6,7 @@ How you provide access to the service is going to be dependent on your specific

A good starting point for yaml manifests for deploying to Kubernetes is in the [yaml](./yaml) directory.
This creates a namespace, configmap, deployment and service.
There is also an example ingress config that is set up to work with [Heptio Contour](https://github.com/heptio/contour), [JetStack cert-manager](https://github.com/jetstack/cert-manager) and [Let's Encrypt](https://letsencrypt.org/).
There is also an example ingress config that is set up to work with [Project Contour](https://github.com/projectcontour/contour), [JetStack cert-manager](https://github.com/jetstack/cert-manager) and [Let's Encrypt](https://letsencrypt.org/).
See [Dave Cheney's blog post](https://blog.heptio.com/how-to-deploy-web-applications-on-kubernetes-with-heptio-contour-and-lets-encrypt-d58efbad9f56) for an example of getting all of that configured.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe swap out Dave's link for this guide which is kept more up to date: https://projectcontour.io/guides/cert-manager/

Copy link
Contributor

@stevesloka stevesloka left a comment

Choose a reason for hiding this comment

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

/lgtm

Can you sign the 2nd commit @pbisen to make the DCO bot happy?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants