Coder Social home page Coder Social logo

Freshclam on K8S about helm-charts HOT 6 OPEN

mailu avatar mailu commented on May 5, 2024 1
Freshclam on K8S

from helm-charts.

Comments (6)

stale avatar stale commented on May 5, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

from helm-charts.

ghostwheel42 avatar ghostwheel42 commented on May 5, 2024

Hi @kadomino - sorry for the late reply.

mailu (that is Mailu/Mailu) is developed for a setup via docker-compose on a single node.
I think we should make sure this is clear in the Issue/PR templates and documentation.
Everything k8s-related should go to the Mailu/helm-charts repository (maintainers needed).
There are many things to be thought of when using mailu via k8s or docker-swarm on multiple nodes (encrypted inter-container communication for example).

That said: freshclam is running as daemon (not via cron). I can implement a flag to disable freshclam via the environment.
The helm chart then needs to set this flag and also create a matching k8s CronJob object.
I'm sure we can work something out if there's another "flavor" of the antivirus container needed for this to work.

from helm-charts.

kadomino avatar kadomino commented on May 5, 2024

Hi @ghostwheel42 - thanks for looking into this :-) No problem for the delay - there is no urgency.

Indeed, I do understand that Mailu's main platform is docker-compose. Nevertheless, in my experience it works fine on K8S. I have been using it in production for 6 months now and the only issue that I have encountered is this freshclam DB curruption. This statement should be taken with a grain of salt though, because my use case is quite basic. Also, I only noticed the problem because on my particular K8S cluster the nodes were rebooting frequently (for unrelated reasons), so there was a high probability of the a pod being killed in the middle of a freshclam download.

Sorry for missing the point that freshclam runs as a daemon and not as a cron in the clamav containter. In any case this still violates the "one service = one container" principle which is more strictly needed under K8S than under docker-compose.

Indeed, it would be great if you could allow somehow for running freshclam and clamav in 2 different containers (they would need to operate on a common ReadWriteMany storage, of course). This would eliminate this corruption issue on K8S, I think.

Concerning the Helm Chart, I noticed it recently and made a fork with additions for my needs, which I would be happy to contribute back. I can also write the K8S CronJob part if/when the separate freshclam image becomes available.

from helm-charts.

ghostwheel42 avatar ghostwheel42 commented on May 5, 2024

Hi @kadomino - I know the "one service = one container" principle, but I think we'll have to live with what we have for now.
The team is very small and I only recently joined to implement a configuration import/export I needed for my setup.

I think the Dockerfile/entrypoint of the sntispam container could be changed to run in 3 modes (vie env):

  • mode 1: as it is now: two daemons (freshclam and clamav) running
  • mode 2: run only clamav as daemon
  • mode 3: run freshclam only once (maybe with some script around to activate the freshened database)

The helm-chart could be updated to make use of mode 2 and 3. What do you think?

from helm-charts.

kadomino avatar kadomino commented on May 5, 2024

Hi @ghostwheel42 - thanks again, this seems like a good idea to me :-) Indeed, it's not that important what's in the image, as long as it can be used for running different types of containers.

In terms of help from me, I'm probably not the right person to touch the images, but I can help with Helm & K8S. Maybe you could get the maintainer of the helm chart involved in this discussion at the right time.

from helm-charts.

stale avatar stale commented on May 5, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

from helm-charts.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.