feat(helm): add languagetool support

[genofire]

Summary

@Rash419 for you later - still in developing

TODO

• [ ] add initContainer to download ngram correctly
• [ ] Testing if it works

[genofire]

@Rash419 do you have an tipp - where the Containerfile/Dockerfile for your languagetool container are?

i like to review and take an look inside the setup / start.sh it correct.

In past i use this files: https://github.com/someone-stole-my-name/docker-languagetool/blob/master/misc/init.sh https://github.com/someone-stole-my-name/docker-languagetool/blob/master/misc/ngram.sh

i believe i have to use for the ngram part an init container (and reimplement that script there).

[Rash419]

@Rash419 do you have an tipp - where the Containerfile/Dockerfile for your languagetool container are?

i like to review and take an look inside the setup / start.sh it correct.

In past i use this files: https://github.com/someone-stole-my-name/docker-languagetool/blob/master/misc/init.sh https://github.com/someone-stole-my-name/docker-languagetool/blob/master/misc/ngram.sh

i believe i have to use for the ngram part an init container (and reimplement that script there).

@genofire tbh I have no idea about languagetool , maybe @timar can help us here

[genofire]

@timar

I found /start.sh with:

#!/bin/bash

EXTRAOPTIONS=""
[ -d "/ngrams" ] && EXTRAOPTIONS=" --languageModel /ngrams "

java -cp languagetool-server.jar  org.languagetool.server.HTTPServer --port 8010 --public --allow-origin '*' ${EXTRAOPTIONS}

Could we update it to:

#!/usr/bin/env bash

EXTRAOPTIONS=${EXTRAOPTIONS:-}
JAVAOPTIONS=${JAVAOPTIONS:-}

if [ -d "/ngrams" ]; then
    EXTRAOPTIONS="${EXTRAOPTIONS} --languageModel /ngrams"
fi

for var in ${!LT_*}; do
  EXTRA_LT=true
  echo "${var#'LT_'}="${!var} >> /tmp/config.properties
done

echo JAVAOPTIONS=$JAVAOPTIONS
if [ "$EXTRA_LT" = true ]; then
  EXTRAOPTIONS="${EXTRAOPTIONS} --config /tmp/config.properties"
    echo config.properties:
    echo "$(cat /tmp/config.properties)"
fi
echo EXTRAOPTIONS=$EXTRAOPTIONS

java ${JAVAOPTIONS} -cp languagetool-server.jar org.languagetool.server.HTTPServer --port 8010 --public --allow-origin '*' ${EXTRAOPTIONS}

For:

• make EXTRAOPTION modifyable
• add JAVAOPTIONS
• add LT_ for https://languagetool.org/development/api/org/languagetool/server/HTTPServerConfig.html for e.g. enable PrometheusMetrics

Otherwise I need to reimplement it inside this helmchart again

[mmeeks]

Rash - can we get this merged & closed or actioned in some useful way ? =)

[genofire]

We wait feedback from @timar for your image of languagetools, so i could also create monitoring and ngram loading in this helm-chart.

That does not depends on @Rash419

[timar]

We wait feedback from @timar for your image of languagetools, so i could also create monitoring and ngram loading in this helm-chart.

That does not depends on @Rash419

@genofire I pushed languagetools image with the startup scirpt you suggested.

[Darshan-upadhyay1110]

Hi @genofire! Great news—timar has added the language tool, so you can go ahead. Assigning task back to you 😉

[genofire]

okay, i will take a look in this again ;)

[eszkadev]

@Rash419 what should we do with this PR?

[Rash419]

@genofire are you still working on this ? Do you need help to finish this PR ?

[mmeeks]

@Rash419 I think we can safely say @genofire is not working on it. Can we come to a resolution here - finish & merge or close for now - either is fine for me =)