Some changes - Githubissues

camunda-community-hub / camunda-7-community-helm

Camunda public Kubernetes Helm repo and charts

Apache License 2.0

39 stars 38 forks source link

Some changes #5

Closed seluard closed 3 years ago

seluard commented 3 years ago

Change the default img to "run-latest", the "latest" or "tomcat-latest" doesn't start very well.
Change the ingress default path, for a easy installation by first time.
Change the path of readinessProbe, It seems to not work.

CLAassistant commented 3 years ago

All committers have signed the CLA.

aabouzaid commented 3 years ago

@seluard Thanks for your PR :+1:

I've a question about replacing tomcat-latest, what kind of error did you have? Also could you add more details about the readiness? What is the issue there?

aabouzaid commented 3 years ago

@seluard please take a look at chart v1.0.0, new features have been added and bugs fixed. I believe that this PR could be fixed now.

seluard commented 3 years ago

Hi @aabouzaid !

I don't remember exacly, but probably was something wrong on my side, like no hit the /camunda/ path or something.

All is working now, I see that you give more time to the readiness, probably some problem with that before.. idk...

Sorry for late reply ! :(

This can be close ! :)