Skip to content

Make CLP_LOGS_INPUT_DIR mounts readonly in docker-compose-all.yaml #1689

@coderabbitai

Description

@coderabbitai

Description

All mount points for CLP_LOGS_INPUT_DIR_HOST in tools/deployment/package/docker-compose-all.yaml should be made readonly by adding the :ro suffix.

Affected Services

Currently, the following services mount the logs input directory without the readonly flag:

  • compression-scheduler
  • compression-worker
  • webui

Expected Behavior

The mount should be:

"${CLP_LOGS_INPUT_DIR_HOST:-empty}:${CLP_LOGS_INPUT_DIR_CONTAINER:-/mnt/logs}:ro"

Context

This issue was identified in PR #1688: #1688 (comment)

Requested by: @junhaoliao

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions