Jose/ansible_proxmox_WOL
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

update readme #3

Open
Jose wants to merge 0 commits from test into main
pull from: test
merge into: Jose:main
Conversation 1 Commits - Files Changed -
Jose commented 2026-01-25 11:40:17 +01:00
Owner
Copy Link
No description provided.
Jose added 16 commits 2026-01-25 11:40:17 +01:00
Fix corrupted .ansible-lint git object
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 10s
Details
e8e8f6d51b
fix 🐛: Fix issue with multiple interfaces and MAC addresses
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 10s
Details
a0db41cb81 
This fix targets the first element in the list to resolve issues related to multiple interfaces and MAC addresses.
feat ✨: Add support for starting tcpdump with 'become: true' in wakeonlan module
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 9s
Details
21ec247569 
This commit introduces the functionality to start tcpdump using elevated privileges by setting 'become: true' in the wakeonlan module. This enhancement ensures that tcpdump can capture network traffic even when run as a non-root user.
refactor ♻️: Refactored WOL packet capture and validation logic
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 10s
Details
d0774ff2b3 
Improved the efficiency and readability of the WOL packet handling by refactoring the existing code. This includes restructuring the validation logic and optimizing the capture process.
feat ✨: Add pause before sending Wake-on-LAN packet
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Details
af4c561b3d 
This feature introduces a delay before sending a Wake-on-LAN (WoL) packet to ensure the network is ready, improving reliability and reducing potential issues during power-up.
feat ✨: Add task to install wakeonlan and send WOL packets
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 9s
Details
f2ead2ad47 
This commit introduces a new feature that includes adding a task to install the `wakeonlan` package and sending Wake-on-LAN (WOL) packets. This allows for remote control of devices by waking them up from a powered-off state.
refactor ♻️: Remove unnecessary delegate_to: localhost from WOL tasks
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Details
d509a4946e 
Refactored WOL task configuration by removing redundant 'delegate_to: localhost' directive to simplify and improve readability.
refactor ♻️: Refactor WOL task to use localhost
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Details
44f9c8ffa9 
This commit refactors the existing WOL task by commenting out the original implementation and adding a new one that sends a Wake-on-LAN packet from localhost.
refactor ♻️: Refactor task management by removing unused task and adjusting pause duration.
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 10s
Details
a82bd5bac5 
This commit refactors the task management system by removing an unused task and adjusting the pause duration to ensure consistent behavior across different environments.
chore 📦: Add new markdown lint configuration, Ansible best practices, README guidelines, AI PR review workflow, and security checks.
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 11s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Failing after 1m17s
Details
74474c263d 
This commit includes the addition of a new markdown lint configuration to disable MD041 rule. It also introduces an Ansible best practices file, README guidelines for comprehensive project files, an AI PR review workflow, and new security checks using Gitleaks and markdown-lint.
style 💎: Fix minor formatting and punctuation
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 11s
Details
Gitleaks Scan / gitleaks (push) Successful in 9s
Details
Markdown Lint / markdown-lint (push) Failing after 1m13s
Details
9d35c16d2c 
Updated README.md to improve readability by fixing minor formatting and punctuation issues.
docs 📝: Reformatted README.md for better readability and added table of contents.
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 10s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Failing after 1m20s
Details
9a6ac67e20 
Improved the structure and accessibility of the README file by reformatting it and adding a table of contents to make it easier for users to navigate.
chore 📦: Add markdown linting config for line length and code block formatting
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 9s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Failing after 1m8s
Details
36401dc8f4 
This commit adds a new markdown linting configuration to enforce line length limits and consistent formatting of code blocks across the project.
docs 📝: Refine README formatting and slightly rephrase some sentences for clarity.
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 10s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Failing after 1m18s
Details
b9358eb043 
Updated the README file to improve its readability and clarity by refining the formatting and rephrasing certain sentences.
docs 📝: Fix typo and update bridge layout
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 10s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Failing after 1m8s
Details
8be72bc931 
Corrected a minor typo in the README.md file and improved the visual clarity of the bridge layout.
style 💎: Remove extra spaces in 'Multiple Bridges' table row
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 10s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Successful in 1m15s
Details
ai-reviews / Review PR (pull_request) Successful in 9s
Details
ansible-lint / Ansible Lint (pull_request) Successful in 10s
Details
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Details
Markdown Lint / markdown-lint (pull_request) Successful in 1m9s
Details
6d03aacc85 
Cleaned up formatting by removing unnecessary spaces within a specific table row to improve readability.
gitea-actionsbot reviewed 2026-01-25 11:40:26 +01:00
gitea-actionsbot left a comment
Copy Link

Review Summary

  • README.md: Summary: The changes in this diff remove an example of multiple bridges with incorrect formatting and replace it with the correct format, while keeping the overall description intact.
# Review Summary * **README.md**: Summary: The changes in this diff remove an example of multiple bridges with incorrect formatting and replace it with the correct format, while keeping the overall description intact.
README.md
@@ -145,3 +155,3 @@
|----------|---------------------|--------------------|
|----------|-------------------|------------------|
| **Standard vmbr0** | `eno1` → `vmbr0` | Enables WOL on `eno1`. |
| **Bonded NICs** | `eno1`, `eno2` → `bond0` → `vmbr0` | Detects `bond0` and sets WOL on *both* slaves. |
gitea-actionsbot commented 2026-01-25 11:40:26 +01:00
Copy Link

[Lines 155-157] [Score: 2] The formatting is incorrect and hard to read. Please remove the backslashes () and use vertical bars (|).

[Lines 155-157] [Score: 2] The formatting is incorrect and hard to read. Please remove the backslashes (\) and use vertical bars (|). ---
README.md
@@ -146,3 +156,3 @@
| **Standard vmbr0** | `eno1` → `vmbr0` | Enables WOL on `eno1`. |
| **Bonded NICs** | `eno1`, `eno2` → `bond0` → `vmbr0` | Detects `bond0` and sets WOL on *both* slaves. |
| **Multiple Bridges** | `eno1` → `vmbr0` <br> `eno2` → `vmbr1` <br> `eno3`, `eno4` → `bond0` → `vmbr2` | One role run configures all three bridges automatically. |
| **Multiple Bridges** | - `eno1` → `vmbr0` - `eno2` → `vmbr1` - `eno3`, `eno4` → `bond0` → `vmbr2` | One role run configures all three bridges automatically. |
gitea-actionsbot commented 2026-01-25 11:40:26 +01:00
Copy Link

[Score: 1] Good formatting and clear description. Well done!

[Score: 1] Good formatting and clear description. Well done!
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 10s
Details
Gitleaks Scan / gitleaks (push) Successful in 4s
Details
Markdown Lint / markdown-lint (push) Successful in 1m15s
Details
ai-reviews / Review PR (pull_request) Successful in 9s
Details
ansible-lint / Ansible Lint (pull_request) Successful in 10s
Details
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Details
Markdown Lint / markdown-lint (pull_request) Successful in 1m9s
Details
This branch is already included in the target branch. There is nothing to merge.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin test:test
git checkout test
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Jose
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Jose/ansible_proxmox_WOL#3
Delete Branch "test"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?