Eero Tamminen

Results 723 comments of Eero Tamminen

Full Perl version gets added as `git` package dependency (minimal Python image already included few MB minimal Perl, as that's POSIX requirement). There are so many dependencies between "GenAIComps" and...

Dropping `libgl1-mesa-glx` and replacing `vim` with `nano` in `Dockerfile`, reduces `chatqna` container size by 253MB i.e. 35%: ``` $ docker images|grep chatqna opea/chatqna latest 4b71cbea8ab6 About an hour ago 727MB...

> Another alternative would be having separate "fetch" phase in the Dockerfile which would install Git, do `git pull` (using `-depth 1` option to speed it), and remove `.git` dir...

All common dependencies should be on a shared base layer, see: https://github.com/opea-project/GenAIComps/issues/265 That way these optimizations need to be done only once.

Once the base images have been cleaned of extra content, it's easy to generate additional, separate "devel" images where those (Vim, Perl, Git etc) tools are added back. All it...

> > Another alternative would be having separate "fetch" phase in the Dockerfile which would install Git, do `git pull` (using `-depth 1` option to speed it), and remove `.git`...

> please submit pr Ok, here's an example of doing that for GenAIExamples repo containers: https://github.com/opea-project/GenAIExamples/pull/1031 @kevinintel Do you want me to write example PR also for GenAIComps repo containers?

> Implementation of the Conversational UI requires root privilege to run Why? Node / JS in general does not require that...

I don't think this is specific to ChatQnA, I think it concerns all UIs containers? @xiguiw please tag also related ticket in "GenAIInfra": https://github.com/opea-project/GenAIInfra/issues/500

Ok, first issue seems to be wrong read access to config file, and second one is temp files being written to wrong location. If you make those 2 locations writable...