Skip to content

Commit

Permalink
Correct ansible-lint issues
Browse files Browse the repository at this point in the history
It seems ansible-lint didn't bother warning when the new
community.general.dig lookup was introduced in
415c143 while it does now complain with
a more recent patch.

Let's fix it in a dedicated PR, making things easier to check.
  • Loading branch information
cjeanner authored and openshift-merge-bot[bot] committed Jul 22, 2024
1 parent 76de17c commit 8a39582
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ repos:
- id: black

- repo: https://github.com/ansible/ansible-lint
rev: v6.22.1
rev: v6.22.2
hooks:
- id: ansible-lint
additional_dependencies:
Expand Down
2 changes: 1 addition & 1 deletion meta/runtime.yml
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
---
requires_ansible: '>=2.13.0'
requires_ansible: '>=2.16.0'
2 changes: 1 addition & 1 deletion roles/libvirt_manager/molecule/check_dns/verify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
- name: Verify DNS
hosts: instance
tasks:
- name: Verify DNS entries
- name: Verify DNS entries # noqa: jinja[invalid]
vars:
_lookup: "{{ lookup('community.general.dig', item.rec) }}"
ansible.builtin.assert:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
- not _run_fail | bool
- scenario.check_dns is defined
block:
- name: Ensure we have expected records
- name: Ensure we have expected records # noqa: jinja[invalid]
vars:
_lookup: "{{ lookup('community.general.dig', dns.rec) }}"
ansible.builtin.assert:
Expand Down

0 comments on commit 8a39582

Please sign in to comment.