Skip to content
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

Revert some backend changes #35

Merged
merged 1 commit into from
Nov 3, 2023
Merged

Conversation

yrodiere
Copy link
Member

@yrodiere yrodiere commented Nov 3, 2023

This reverts commit e4dcd09.

It appears OpenSearch needs to run with UID 1000
because the entrypoint script is only accessible with that user:
opensearch-project/opensearch-devops#97

But our OpenShift cluster only allows containers to run with wildly high UIDs, like 100000000. The security context constraints do not allow anything else.

The entrypoint script is not accessible to groups, either, so OpenShift's fsGroup config
(https://docs.openshift.com/container-platform/3.11/install_config/persistent_storage/pod_security_context.html#fsgroup) is no use.

And finally, user namespace mapping is not supported in OpenShift yet:

https://access.redhat.com/solutions/6977863

This reverts commit e4dcd09.

It appears OpenSearch needs to run with UID 1000
because the entrypoint script is only accessible with that user:
opensearch-project/opensearch-devops#97

But our OpenShift cluster only allows containers to run with wildly high
UIDs, like 100000000. The security context constraints do not allow
anything else.

The entrypoint script is not accessible to groups, either,
so OpenShift's fsGroup config
(https://docs.openshift.com/container-platform/3.11/install_config/persistent_storage/pod_security_context.html#fsgroup)
is no use.

And finally, user namespace mapping is not supported in OpenShift yet:

https://access.redhat.com/solutions/6977863
@yrodiere yrodiere merged commit 365ac40 into quarkusio:main Nov 3, 2023
@yrodiere yrodiere deleted the elasticsearch branch January 29, 2024 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant