Skip to content

feat(elt): Allow ELT task log level to be configured externally#170

Merged
martyngigg merged 1 commit into
mainfrom
elt-default-log-level
Dec 9, 2025
Merged

feat(elt): Allow ELT task log level to be configured externally#170
martyngigg merged 1 commit into
mainfrom
elt-default-log-level

Conversation

@martyngigg

@martyngigg martyngigg commented Dec 9, 2025

Copy link
Copy Markdown
Member

Summary

Lower default log level for ELT tasks to INFO and allow it to be overridden.

Summary by CodeRabbit

  • Chores
    • Updated default log level from DEBUG to INFO. Log verbosity is now configurable via environment variable for operational flexibility.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

The LOG_LEVEL variable in the ELT cron task template was modified from a hardcoded DEBUG value to a configurable parameter using environment variable substitution. The default verbosity level shifted from DEBUG to INFO, whilst maintaining the ability to override via the ELT_LOG_LEVEL environment variable.

Changes

Cohort / File(s) Summary
ELT cron task logging configuration
infra/ansible-docker/playbooks/elt/templates/cron/elt_task.sh.j2
LOG_LEVEL variable updated from hardcoded DEBUG to environment-configurable ${ELT_LOG_LEVEL:-INFO}, enabling runtime control with INFO as the new default level

Poem

🐰 A rabbit's tale of logs set free,
From DEBUG noise to INFO's clarity,
With ELT_LOG_LEVEL now in play,
Configuration finds its way,
Flexibility hops along with glee! 🌟

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: making the ELT task log level externally configurable, which directly matches the changeset modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting

📥 Commits

Reviewing files that changed from the base of the PR and between dfb9c99 and d11069c.

📒 Files selected for processing (1)
  • infra/ansible-docker/playbooks/elt/templates/cron/elt_task.sh.j2 (1 hunks)
🔇 Additional comments (1)
infra/ansible-docker/playbooks/elt/templates/cron/elt_task.sh.j2 (1)

9-9: Excellent improvement to script configurability.

The change correctly implements external configuration of the log level whilst providing a sensible default. The syntax follows the existing pattern used for other configurable variables in the script, and the shift from DEBUG to INFO as the default reduces log verbosity appropriately.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@martyngigg
martyngigg merged commit daa2d10 into main Dec 9, 2025
2 checks passed
@martyngigg
martyngigg deleted the elt-default-log-level branch December 9, 2025 18:29
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.

1 participant