jboss-openshift / cct_module

CEKit modules shared by OpenShift container images
Apache License 2.0
45 stars 79 forks source link

[rhpam-3056] Business Central remains "Container Creating" if missing known_hosts file in the githooks configuration #386

Closed desmax74 closed 3 years ago

desmax74 commented 3 years ago

https://issues.redhat.com/browse/RHPAM-3056

@spolti @tchughesiv

spolti commented 3 years ago

we should not edit it. it should be done only on our repositories.

desmax74 commented 3 years ago

@spolti we can't add or edit under /home/jboss/.ssh because is mounted as a secret by k8s and the secret volumes are red-only (we can't change to readonly false), the other options are add under /etc/ssh (this pr) or change the /etc/ssh/sshconfig to use a specific path for known_hosts, or use a option with path of known_hosts when ssh is used by the client code, do you have ideas other than these ?

spolti commented 3 years ago

I didn't say you to do not do this, just do not use this repo to apply this change, use ours instead :)

desmax74 commented 3 years ago

@spolti sorry for the misunderstanding :) I've used this because here the user is root there isn't and the dir isn't writable by the other user, is it okay change the user there ?

spolti commented 3 years ago

any configuration script processed by cekit will use root.

spolti commented 3 years ago

we could, for example add this on the kieserver's configuration script and set the right permissions.

desmax74 commented 3 years ago

replaced with https://github.com/jboss-container-images/jboss-kie-modules/pull/389/