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
Feature/scheme service #275
Conversation
func (p *Provider) getScheme(annotations map[string]string) string { | ||
scheme := annotations[k8s.AnnotationScheme] | ||
|
||
if scheme == "" { |
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.
we should validate that the scheme is an acceptable scheme here.
@@ -180,6 +181,16 @@ func (p *Provider) getServiceMode(annotations map[string]string) string { | |||
return mode | |||
} | |||
|
|||
func (p *Provider) getScheme(annotations map[string]string) string { |
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.
since this doesn't actually use the provider at all, did we want to have this method moved to the base
package so that we can use it with the SMI provider?
Did we want to add this functionality to the SMI provider in this PR? or did we want to add it in another? |
9343679
to
4468ee0
Compare
4468ee0
to
f91a898
Compare
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.
LGTM. Thanks!
Description
Allow user to customize scheme.