-
Notifications
You must be signed in to change notification settings - Fork 327
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
kuma-cp: generate HTTP-specific inbound listeners for services tagged with protocol: http
#574
kuma-cp: generate HTTP-specific inbound listeners for services tagged with protocol: http
#574
Conversation
@@ -84,3 +105,14 @@ func (d *DataplaneResource) GetIP() string { | |||
} | |||
return ifaces[0].DataplaneIP | |||
} | |||
|
|||
func (d *DataplaneResource) GetProtocol(idx int) Protocol { |
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.
How about idx
-> inboundIdx
? I had to read the func body to know what is it about.
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
} | ||
|
||
pbst, err := ptypes.MarshalAny(config) | ||
util_error.MustNot(err) |
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.
Isn't it better to rethrow this error just in case? Or are we 100% sure this won't be an error?
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.
MustNot(err)
has been replaced with propagation
… with `protocol: http`
0a93f21
to
7480f09
Compare
Summary
protocol: http