astefanutti commented on a change in pull request #406: The example 
RestWithRestlet does not work calling its service via curl or webbrowser
URL: https://github.com/apache/camel-k/pull/406#discussion_r252989853
 
 

 ##########
 File path: pkg/trait/service.go
 ##########
 @@ -79,7 +79,7 @@ func (t *serviceTrait) Apply(e *Environment) (err error) {
        }
        port := corev1.ServicePort{
                Name:       httpPortName,
-               Port:       80,
+               Port:       int32(t.Port),
 
 Review comment:
   @lburgazzoli I think that it should work without this change. I think the 
original issue is due to the host bound to `localhost`. I've tested the sample 
with service port 80 and it works on my local setup (Minishift). Some Linux 
capabilities exist for port binding on pods but I do not know of any 
restrictions applicable to services.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to