-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
security context root for modifing opensearch filesystem dependency #659
Comments
@Siegfriedk FYI |
I unassigned myself as i expect a collegue/team member of bpdm to drive this. General speaking: if this is a dependency were you do not expect someome else to use an external setup or managed, it become a component of your product and need to be able to run as non root. It should also not be hard to fix it |
We recently decided to not longer use Opensearch for our future BPDM solutions and are currently in the process of removing it from our applications and Helm Charts. Therefore, no problems for future releases 👍 |
@nicoprow out of curiosity: what did you use it for and how are you replacing it? |
In the past we had a functionality to search business partners by different text fields such as names, descriptions such like that. At that time it seemed that the requirements would lead to more complex search functionality over the business partners. However, we never actually experienced that demand and with a drastic data model change the existing search functionality was also scaled back. |
Hi BPDM-Team,
found an opensearch dependency in our helm chart that is a against our rule with https://eclipse-tractusx.github.io/docs/release/trg-4/trg-4-03
After a little Chat and research with our System Team Member this should be clarified till the next release.
Problem:
runAsUser: 0
bpdm/charts/bpdm/values.yaml
Lines 53 to 73 in 700af9a
And guess there are also some opensearch values you can play arround with
https://github.com/opensearch-project/helm-charts/blob/main/charts/opensearch/values.yaml
But also other users seems to have the problem:
opensearch-project/helm-charts#369
opensearch-project/helm-charts#480
Bitnami approach:
https://github.com/bitnami/charts/blob/2ba572d7ec51d5ff9ecd47f5748aab308aef693e/bitnami/elasticsearch/templates/data-statefulset.yaml#L72
Clear announcement that we have to discuss this and you as a bpdm-team have to push the topic.
The text was updated successfully, but these errors were encountered: