Ansible-lint: Refactor ansible-lint rules (#127)

Partly osism/issues#496

Signed-off-by: Ramona Beermann <ramona.beermann@osism.tech>

.ansible-lint

@@ -5,11 +5,7 @@ use_default_rules: true
 rulesdir:
   - ./.ansible-lint-rules/
 skip_list:
-  - parser-error  # AnsibleParserError.
-  - var-spacing  # Variables should have spaces before and after: {{ var_name }}.
-  - fqcn-builtins
-  - yaml
-  - args[module]
+  - yaml  # We use yamllint
 # DO NOT DELETE THE WARNLIST! It is required for our custom rules!
 # If this isn't there our custom rules will only through a warning and wont generate a failure!:
 warn_list:

files/ansible/playbooks/import-netbox.yml

@@ -33,7 +33,7 @@
       when: "item != 'localhost'"
       ignore_errors: true
 
-    - name: Create local config context
+    - name: Create local config context  # noqa: args[module]
       netbox.netbox.netbox_device:
         netbox_url: "{{ netbox_url }}"
         netbox_token: "{{ netbox_token }}"
@@ -43,6 +43,8 @@
           local_context_data: "{{ lookup('file', item.stat.path) | from_yaml }}"
       loop: "{{ results.results }}"
       when: item.stat.exists
+      tags:
+        - skip_ansible_lint
 
     - name: Check inventory host_vars file with short hostnames
       ansible.builtin.stat:
@@ -62,3 +64,5 @@
           local_context_data: "{{ lookup('file', item.stat.path) | from_yaml }}"
       loop: "{{ results.results }}"
       when: item.stat.exists
+      tags:
+        - skip_ansible_lint

requirements.yml

@@ -0,0 +1,4 @@
+---
+collections:
+  - name: netbox.netbox
+    source: https://galaxy.ansible.com