cdi-camel adds falsely jolokia via hawt-app setenv.sh

fabric8io / ipaas-quickstarts

quickstarts for the fabric8 project

Apache License 2.0

66 stars 86 forks source link

cdi-camel adds falsely jolokia via hawt-app setenv.sh #1156

Open rhuss opened 8 years ago

rhuss commented 8 years ago

In this setenv.sh jolokia is added to the startup options for haw-app which is also used in a s2i context with the base image bringing in jolokia. This obviously doesn't work, also because the jar referenced in this setenv.sh doesn't exist.

Also since in an OpenShift context Jolokia needs to be configured for using the correct certs it makes more sense to leave the Jolokia configuration to the base image in this case.

@chirino , what was the reason for adding it here ?

davsclaus commented 8 years ago

I disabled it today as 2 weeks later no response from @chirino https://github.com/fabric8io/ipaas-quickstarts/commit/fe05af4dc49b0c0ef3aeaf20039314c5de0ee6dc