Skip to content

Conversation

@houshengbo
Copy link
Contributor

@houshengbo houshengbo commented Nov 11, 2020

Proposed Changes

  • Knative operator is also able to configure multiple ingresses other than istio. The guide is based on the existing guide in knative. The only change is that Knative operator leverage the serving CR instead of directly working on configMaps.

@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Nov 11, 2020
@knative-prow-robot knative-prow-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 11, 2020
Comment on lines 542 to 543
In the case of the provided `helloworld-go` sample application, the output
should, using the default configuration, be:
Copy link
Contributor

@mpetason mpetason Nov 12, 2020

Choose a reason for hiding this comment

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

Suggested change
In the case of the provided `helloworld-go` sample application, the output
should, using the default configuration, be:
In the case of the provided `helloworld-go` sample application, using the default configuration, the output should be:

Copy link
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

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

/approve

I'll let Mike do the /lgtm

@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evankanderson, houshengbo

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mpetason
Copy link
Contributor

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 16, 2020
@knative-prow-robot knative-prow-robot merged commit 3834c3d into knative:master Nov 16, 2020
RichieEscarez pushed a commit to RichieEscarez/docs that referenced this pull request Mar 6, 2021
…native#3025)

* Add description on how to configure each ingress for knative serving

* Rephrase the sentence in a correct order
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants