feat(anolisa): add logrotate config for ops telemetry#1094
Merged
Conversation
Add /etc/logrotate.d/anolisa to bound disk usage of ops telemetry JSONL files under /var/log/anolisa/sls/ops/. The policy rotates at 30M per file and keeps one historical copy, using create mode so ilogtail inode offsets survive rename-based rotation. logrotate setup runs during register and is intentionally preserved during unregister: the .jsonl files remain writable by components, so size control is still required. Remove the daily trigger from the policy; size-based rotation is sufficient for these best-effort telemetry buffers. Known limitations: - Requires host logrotate and cron/timer to actually perform rotation. - Lightweight containers without cron/logrotate will not benefit. Signed-off-by: KaiLong Zhou <zhoukailong.zkl@alibaba-inc.com> Assisted-by: Qoder:latest
KaiLongZhou
force-pushed
the
anolisa-telemetry
branch
from
7cf3bc0 to
a6cf758
Compare
Ubuntu and many Debian-based distributions symlink /bin/sh to dash, which does not support the bash syntax used by the ilogtail installation script. Switch to explicit bash to prevent install failures on these systems. Assisted-by: Qoder:latest Signed-off-by: KaiLong Zhou <zhoukailong.zkl@alibaba-inc.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add /etc/logrotate.d/anolisa to bound disk usage of ops telemetry JSONL files under /var/log/anolisa/sls/ops/. The policy rotates at 30M per file and keeps one historical copy, using create mode so ilogtail inode offsets survive rename-based rotation.
logrotate setup runs during register and is intentionally preserved during unregister: the .jsonl files remain writable by components, so size control is still required.
Remove the daily trigger from the policy; size-based rotation is sufficient for these best-effort telemetry buffers.
Known limitations:
Assisted-by: Qoder:latest
Description
Related Issue
closes #
Type of Change
Scope
cosh(copilot-shell)sec-core(agent-sec-core)skill(os-skills)sight(agentsight)tokenless(tokenless)ckpt(ws-ckpt)memory(agent-memory)anolisa(anolisa-cli)skillfs(SkillFS)Checklist
cosh: Lint passes, type check passes, and tests passsec-core(Rust):cargo clippy -- -D warningsandcargo fmt --checkpasssec-core(Python): Ruff format and pytest passskill: Skill directory structure is valid and shell scripts pass syntax checksight:cargo clippy -- -D warningsandcargo fmt --checkpasstokenless:cargo clippy -- -D warningsandcargo fmt --checkpassmemory(Linux only):cargo clippy --all-targets -- -D warnings,cargo fmt --check, andcargo testpassanolisa:cargo clippy --all-targets --locked -- -D warnings,cargo fmt --all --check, andcargo test --lockedpassskillfs:cargo fmt --all --check,cargo clippy --workspace --all-targets -- -D warnings, andcargo test --workspacepasspackage-lock.json/Cargo.lock)Testing
Additional Notes