Update instructions for using temporary DNS in installation section #5318
Conversation
Signed-off-by: Paul S. Schweigert <paul@paulschweigert.com>
✅ Deploy Preview for knative ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
knative-prow
bot
added
approved
|
/cherry-pick release-1.8 |
|
@psschwei: once the present PR merges, I will cherry-pick it on top of release-1.8 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dprotaso, psschwei The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@psschwei: new pull request created: #5322 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Signed-off-by: Paul S. Schweigert paul@paulschweigert.com
Fixes #5316
We updated the default domain in knative/serving#13259 to
svc.cluster.local, which is not reachable outside of the cluster. As a result, users who want to use the temporary DNS option will first need to patch in a domain (for example,example.com) before they'll be able to reach the service.