Git Product home page Git Product logo

Comments (7)

WhiteBahamut avatar WhiteBahamut commented on July 18, 2024 1

I am not the maintainer here ;) Just try to help out!
Wondering why this happens at all, afaik it should not. How have you definded nextcloud_clamav? Is it used by another service as well?
On the other hand I would just ditch the VOLUME declaration in the Dockerfile and add documentation for the sample compose which directory should be mapped. Once my current multi-arch PR is approved I could issue a new PR to remove the Dockerfile VOLUME.

from docker-clamav.

WhiteBahamut avatar WhiteBahamut commented on July 18, 2024 1

I would treat the docker-compose provided in here just a very basic example. Will try to work in the volume issue this week and update the sample compose and readme then.

  • remove VOLUME from Dockerfiles
  • update README with persistency instruction
  • update sample compose to reflect volume usage

from docker-clamav.

WhiteBahamut avatar WhiteBahamut commented on July 18, 2024

I think it is valid to declare the VOLUME within the Dockerfile. You still can specify where to mount it (docker run -v /my/path/:/var/lib/clamav).
Can you post your compose file?

from docker-clamav.

Ryu945 avatar Ryu945 commented on July 18, 2024

I tried that and it just makes two volumes. Both the one you specified and the one I have listed in my compose file. The AV part of my compose file looks like this. Tell me if you need the entire file.

av:
  image: mkodockx/docker-clamav:alpine
  container_name: nextcloud-clamav
  networks: 
    - network1
  depends_on:
    - reverse_proxy
  volumes:
    - nextcloud_clamav:/var/lib/clamav
  environment:
    - NETWORK_ACCESS=internal
  restart: unless-stopped

from docker-clamav.

Ryu945 avatar Ryu945 commented on July 18, 2024

I have defined nextcloud_clamav like a volume. The volume declaration being removed from the docker file should fix this problem. This declaration should be in the docker compose file or in examples of terminal commands.

Summing it up, in addition to removing the volume declaration line in the docker file, the volume declaration line needs to be added to docker compose. Also, the example terminal lines needs to be updated to include the creation of the volume. I would also recommend adding:

restart: unless-stopped

For the purpose of this repository the changes would look like this.

av:
  debian: image: mkodockx/docker-clamav
  image: mkodockx/docker-clamav:alpine
  container_name: nextcloud-clamav
  networks: 
    - network1
  depends_on:
    - reverse_proxy
  volumes:
    - clamav:/var/lib/clamav
  restart: unless-stopped

The examples for terminal would be changed to:

docker run -d -p 3310:3310 -v clamav:/var/lib/clamav mkodockx/docker-clamav:alpine
docker run -v clamav:/var/lib/clamav mkodockx/docker-clamav:alpine-edge

from docker-clamav.

mko-x avatar mko-x commented on July 18, 2024

thanks to @WhiteBahamut closed by #72

from docker-clamav.

WhiteBahamut avatar WhiteBahamut commented on July 18, 2024

more PRs might come as I start to use the images more myself ;-)

from docker-clamav.

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.