Skip to content

Conversation

pvriel
Copy link
Contributor

@pvriel pvriel commented Sep 11, 2025

https://xenitsupport.jira.com/browse/DSNYHD-84

  • Allow Solr to auto-detect the memory in the Pods by slightly updating the JVM version from 11.0.13 to 11.0.16;
  • Dropped our own custom Xenit OpenJDK images;
  • Dropped hard-coded XMS and XMX properties to support configuration e.g. -XX:InitialRAMPercentage=50 -XX:MaxRAMPercentage=50

value=$(echo $i | cut -d '=' -f 2-)
setOption $key $value "$file"
if [[ -z "$key" ]]; then
echo "Skipping env. variable ($i), since the resulting key is empty."
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Example: 2025-09-11T13:42:23.976221557Z Skipping env. variable (LC_ALL=en_US.UTF-8), since the resulting key is empty.

Because this initialization script thinks that LC is a property that should be applied to ALL cores.

ALFRESCO_SOLR_SUGGESTER_ENABLED=${ALFRESCO_SOLR_SUGGESTER_ENABLED:-'true'}
ALFRESCO_SOLR_FACETABLE_CATEGORIES_ENABLED=${ALFRESCO_SOLR_FACETABLE_CATEGORIES_ENABLED:-'false'}

JAVA_XMS=${JAVA_XMS:-'512M'}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This property does not seem to be used?

ALFRESCO_SOLR_FACETABLE_CATEGORIES_ENABLED=${ALFRESCO_SOLR_FACETABLE_CATEGORIES_ENABLED:-'false'}

JAVA_XMS=${JAVA_XMS:-'512M'}
JAVA_XMX=${JAVA_XMX:-'2048M'}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait to low for Solr anyway. Drop it + give a warning.


DIR=/docker-entrypoint.d

BOLD="\033[1m"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might not work in K8S, but this gives a nice representation of what the container is actually doing locally.

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