banzaicloud / banzai-charts

Curated list of Banzai Cloud Helm charts used by the Pipeline Platform
Apache License 2.0
368 stars 284 forks source link

openshift bug #1079

Open shakenfr opened 4 years ago

shakenfr commented 4 years ago

Describe the bug </usr/local/bin/docker-entrypoint.sh: line 35: /etc/cassandra/cassandra.yaml: Permission denied>

Steps to reproduce the issue: <helm install banzaicloud-stable/cadence>

Expected behavior

**Screenshots** <--> **Additional context**
shakenfr commented 4 years ago

I think related to cassandra bug https://bugs.launchpad.net/trove/+bug/1415862 is it possible to update the requirement version of cassandra ?

sagikazarmark commented 4 years ago

@shakenfr generally OpenShift is not supported by our software/charts because it's not standard Kubernetes and it's not compatible with a lot of Kubernetes features.

PRs are always welcome though! So if simply upgrading the cassandra chart is enough for OpenShift, I guess that should be OK.

shakenfr commented 4 years ago

well, my skills are limited. I see the version of cassandra inside https://github.com/banzaicloud/banzai-charts/blob/master/cadence/requirements.yaml. I try to fork your repo, than changing the cassandra tag to a most recent one but after I don't know to to add the helm repo with my fork to install cadence with cassandra version updated...

sagikazarmark commented 4 years ago

@shakenfr unfortunately we cannot accept untested PRs.

I would suggest fixing it in your environment then.

The chart will receive an (Cadence) upgrade soon we will try to upgrade the Cassandra chart as well.

shakenfr commented 4 years ago

hi, we succeeded to run your it after forking your repo and add following workaround to the docker file

FROM ubercadence/server:0.11.0 RUN chmod 775 /etc/cadence/config

it seems to be a problem with security restriction in openshift. So if you add this chmod it's better in openshift and not very problematic in kubernetes to add this.

regards bruno

sagikazarmark commented 4 years ago

I think you should send a PR to Cadence then.

shakenfr commented 4 years ago

yes I think you're right, It will do that

thanks