feat ✨: Add new task to stop log2ram before extracting it #42

Jose · 2026-02-16T20:53:09+01:00

Jose commented

2026-02-16 20:53:09 +01:00

Introduce a new step in the extraction process to ensure that log2ram is stopped before proceeding with the extraction, preventing potential data corruption or loss.

Jose added 1 commit 2026-02-16 20:53:09 +01:00

feat ✨: Add new task to stop log2ram before extracting it

ansible-lint / Ansible Lint (push) Successful in 12s

Details

Gitleaks Scan / gitleaks (push) Successful in 5s

Details

Markdown Lint / markdown-lint (push) Successful in 6s

Details

ai-reviews / Review PR (pull_request) Successful in 14s

Details

PR check / Gitleaks (pull_request) Successful in 4s

Details

PR check / lint tests (pull_request) Successful in 15s

Details

PR check / labeler (pull_request) Successful in 2s

Details

PR check / handle_failures (pull_request) Has been skipped

Details

PR check / handle_success (pull_request) Successful in 2s

Details

eb548e8476

Introduce a new step in the extraction process to ensure that log2ram is stopped before proceeding with the extraction, preventing potential data corruption or loss.

gitea-actions bot reviewed 2026-02-16 20:53:24 +01:00

gitea-actions bot left a comment

Review Summary

tasks/ram.yml: Summary: Adding a check to stop log2ram service before extraction and installation, improving the maintainability of the script by preventing potential conflicts. diff File: tasks/ram.yml Context: Line 47: - name: ram | Logic to extract and install log2ram Line 48: when: download_archive.changed # noqa: no-handler Line 49: block: Line 50: + - name: ram | Stop log2ram if present Line 51: + ansible.builtin.systemd: Line 52: + name: log2ram Line 53: + state: stopped Line 54: + become: true Line 55: + failed_when: false Line 56: + Line 57: - name: ram | Extract log2ram Line 58: ansible.builtin.unarchive: Line 59: src: "/tmp/log2ram.tar.gz" Context: Line 50: + - name: ram | Stop log2ram if present StartLine: 50 EndLine: 55 Comment: [Score: 3] Adding a check to stop the `log2ram` service before extraction and installation, preventing potential conflicts.

# Review Summary * **tasks/ram.yml**: **Summary:** Adding a check to stop `log2ram` service before extraction and installation, improving the maintainability of the script by preventing potential conflicts. ```diff File: tasks/ram.yml Context: Line 47: - name: ram | Logic to extract and install log2ram Line 48: when: download_archive.changed # noqa: no-handler Line 49: block: Line 50: + - name: ram | Stop log2ram if present Line 51: + ansible.builtin.systemd: Line 52: + name: log2ram Line 53: + state: stopped Line 54: + become: true Line 55: + failed_when: false Line 56: + Line 57: - name: ram | Extract log2ram Line 58: ansible.builtin.unarchive: Line 59: src: "/tmp/log2ram.tar.gz" Context: Line 50: + - name: ram | Stop log2ram if present StartLine: 50 EndLine: 55 Comment: [Score: 3] Adding a check to stop the `log2ram` service before extraction and installation, preventing potential conflicts. ```

gitea-actions bot added the source ci-pass labels 2026-02-16 20:53:49 +01:00

Jose merged commit 6f27f0c47c into main

2026-02-16 20:54:28 +01:00

Jose referenced this issue from a commit

2026-02-16 20:54:29 +01:00

Merge pull request 'feat ✨: Add new task to stop log2ram before extracting it' (#42) from dev into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Jose/ansible_role_proxmox_provision#42