-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add configmap for clusteringress #2486
Add configmap for clusteringress #2486
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lichuqiang: 0 warnings.
In response to this:
First step to address #1969
Part of #2434
Split this into separate PR for review.Proposed Changes
- add ingress gateway configmap
Release Note
allow config ingress gateway
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.
config/config-ingressgateway.yaml
Outdated
namespace: knative-serving | ||
data: | ||
# Gateway used for knative traffic. Default to istio-ingressgateway. | ||
ingress-gateway: "istio-ingressgateway.istio-system.svc.cluster.local" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: newline
config/config-ingressgateway.yaml
Outdated
apiVersion: v1 | ||
kind: ConfigMap | ||
metadata: | ||
name: config-ingressgateway |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm worried that this name is overly specific, perhaps we could call this config-istio
and have it hold the istio-specific concepts that our networking
controllers back by istio need?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to having one config map for Istio related settings. I think there are a few Istio items in config-networking
that could move here as well (in a later PR).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
02af078
to
bf87455
Compare
cfc1f28
to
3b2818a
Compare
The following is the coverage report on pkg/.
|
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lichuqiang, mattmoor, tcnghia 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 |
First step to address #1969
Part of #2434
Split this into separate PR for review.
Proposed Changes
Release Note