Skip to content

Add Module for managing lists #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from
Closed

Conversation

muller2002
Copy link
Contributor

This is an implementation for an module, with an interface similar to manage_lists.

This Module is faster, as it doesn't spawn tasks for every list and is thus suitable for large collections of block lists.

This closes #3

Run ansible-lint against this repo, fixing everything except:
- galaxy[no-changelog]
- name[template]
This Commit adds a module, which can be used to manage blocklist of the pihole
Allows handling of allow and block lists with an interface influenced by the manage list role
This is due to a eror in the PiHole API, see: pi-hole/FTL#2377
This will lead to a lot faster updates, fixes sbarbett#3
@sbarbett
Copy link
Owner

Getting things to pass ansible-lint has introduced some conflicts with this PR that can't be automatically resolved. Can you rebase from the main repository and then plug your module back in. The main repo is now compliant with Ansible's linting requirements.

@sbarbett
Copy link
Owner

Please try v1.1.0. I have refactored the lists modules.

@sbarbett sbarbett closed this Apr 7, 2025
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.

using sbarbett.pihole.manage_lists with large blocklists takes really long
2 participants