Skip to content

[BUG] Fix es_logstash_setup permission error by explicit container user#713

Open
rishavtarway wants to merge 1 commit intoaiondemand:developfrom
rishavtarway:fix/224-logstash-permission
Open

[BUG] Fix es_logstash_setup permission error by explicit container user#713
rishavtarway wants to merge 1 commit intoaiondemand:developfrom
rishavtarway:fix/224-logstash-permission

Conversation

@rishavtarway
Copy link

Change(s)

Change Type: Fixed
Change Category: Internal
Changelog Entry: Fix permission failure in es_logstash_setup by setting an explicit container user in compose.

How to Test

  1. Start from a clean checkout.
  2. Run docker compose up -d.
  3. Verify es_logstash_setup does not fail with permission denied on logstash config generation paths.

Checklist

  • Tests have been added or updated to reflect the changes, or their absence is explicitly explained.
  • Documentation has been added or updated to reflect the changes, or their absence is explicitly explained.
  • A self-review has been conducted checking:
    • No unintended changes have been committed.
    • The changes in isolation seem reasonable.
    • Anything that may be odd or unintuitive is provided with a GitHub comment explaining it.
  • All CI checks pass before pinging a reviewer, or provide an explanation if they do not.
  • The PR title matches the changelog entry's one-line description.

Explanation for tests/docs: this is a compose runtime permission fix; no Python logic changed. Reproduction/validation steps are provided above.

Related Issues

Closes #224

@rishavtarway
Copy link
Author

This PR intentionally changes only docker-compose.yaml (es_logstash_setup user) to avoid broad chmod/chown host workarounds discussed in #224. If preferred, I can also test and include the same explicit user in docker-compose.dev.yaml.

@rishavtarway
Copy link
Author

Validation update: I built aiod_metadata_catalogue, created a local src/config.override.toml for the bind mount expected by compose, and verified targeted startup path (elasticsearch + es_logstash_setup) succeeds with this change. PR scope remains intentionally limited to the es_logstash_setup user configuration for #224.

@rishavtarway rishavtarway marked this pull request as ready for review March 3, 2026 11:05
@rishavtarway
Copy link
Author

rishavtarway commented Mar 11, 2026

Quick status update on this PR:

  • Scope remains intentionally limited to docker-compose.yaml (es_logstash_setup user).
  • Local validation completed for the targeted path:
    • docker compose up -d
    • Verified es_logstash_setup no longer fails with permission-denied during logstash setup.
  • No unrelated files are included in this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docker compose permission error

1 participant