-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[nginx-ingress-controller] cleanup before update updating nginx controller to 0.7 #1104
Conversation
ccb1e2f
to
154746e
Compare
cf637fa
to
4371273
Compare
) | ||
|
||
const ( | ||
secureUpstream = "ingress.kubernetes.io/secure-upstream" |
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.
please add a comment with explanantion, or link to docs about how the encryption is performed to the upstream (which certs are used etc).
ingLister StoreToIngressLister | ||
svcLister cache.StoreToServiceLister | ||
endpLister cache.StoreToEndpointsLister | ||
mapLister StoreToMapLister |
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.
can you rename to storetoconfigmaplister? since this makes it sounds like you're listing to a go map
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
LGTM, guess we'll need followups for examples and readme anyway so you can fix the nits in that |
@@ -260,239 +258,6 @@ In case of an error in a request the body of the response is obtained from the ` | |||
Using this two headers is possible to use a custom backend service like [this one](https://github.com/aledbf/contrib/tree/nginx-debug-server/Ingress/images/nginx-error-server) that inspect each request and returns a custom error page with the format expected by the client. Please check the example [custom-errors](examples/custom-errors/README.md) | |||
|
|||
|
|||
### Annotations |
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.
Was this deletion of documentation by accident? The README is now missing vital information about the available configuration options.
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.
@databus23 this is being added here #1130
fixes #930
fixes #884