ansible-lint: Groom the .ansible-lint file

- Enable production profile
- Remove excludes no longer necessary
- Try to not skip any tests

Signed-off-by: Felix Matouschek <fmatouschek@redhat.com>

.ansible-lint

@@ -1,37 +1,11 @@
 # .ansible-lint
 exclude_paths:
-  - .cache/
   - .github/
-  - molecule/
-  - .ansible-lint
-  - .yamllint
-  - meta/
-  - playbooks/roles/
-  - changelogs/changelog.yaml
 
 enable_list:
   - fqcn-builtins  # opt-in
   - no-log-password  # opt-in
+  - yaml
 
-warn_list:
+profile: production
-  - experimental
-  - ignore-errors
-  - no-handler
-  - fqcn-builtins
-  - no-log-password
-  - no-empty-data-files
-  - name[template]
-  - fqcn[keyword]
-  - schema[meta]
-  - no-free-form
-  - run-once[task]
-  - var-naming[no-role-prefix]
-  - galaxy[version-incorrect]
-  - yaml[key-duplicates]
-
-skip_list:
-  - jinja[spacing]
-  - yaml[truthy]
-
-use_default_rules: true
 parseable: true

examples/kubesecondarydns.kubevirt.yml

@@ -3,4 +3,4 @@ connections:
   - namespaces:
       - default
     network_name: bridge-network
-    kube_secondary_dns: yes
+    kube_secondary_dns: true

examples/play-create-dv.yml

@@ -39,4 +39,4 @@
                   ssh_authorized_keys:
                     - ssh-ed25519 AAAA...
               name: cloudinit
-        wait: yes
+        wait: true

examples/play-create-min.yml

@@ -6,4 +6,4 @@
         state: present
         name: testvm
         namespace: default
-        running: no
+        running: false

examples/play-create.yml

@@ -37,4 +37,4 @@
                   ssh_authorized_keys:
                     - ssh-ed25519 AAAA...
               name: cloudinit
-        wait: yes
+        wait: true

examples/play-delete-dv.yml

@@ -6,4 +6,4 @@
         name: testvm-with-dv
         namespace: default
         state: absent
-        wait: yes
+        wait: true

examples/play-delete.yml

@@ -6,4 +6,4 @@
         name: testvm
         namespace: default
         state: absent
-        wait: yes
+        wait: true

examples/services.kubevirt.yml

@@ -2,4 +2,4 @@ plugin: kubevirt.core.kubevirt
 connections:
   - namespaces:
       - default
-    use_service: yes
+    use_service: true

plugins/inventory/kubevirt.py

@@ -131,34 +131,32 @@ requirements:
 EXAMPLES = """
 # Filename must end with kubevirt.[yml|yaml]
 
-# Authenticate with token, and return all VirtualMachineInstances for all accessible namespaces
+- name: Authenticate with token and return all VirtualMachineInstances for all accessible namespaces
-plugin: kubevirt.core.kubevirt
+  plugin: kubevirt.core.kubevirt
-connections:
+  connections:
-  - host: https://192.168.64.4:8443
+    - host: https://192.168.64.4:8443
-    api_key: xxxxxxxxxxxxxxxx
+      api_key: xxxxxxxxxxxxxxxx
-    validate_certs: false
+      validate_certs: false
 
-# Use default config (~/.kube/config) file and active context, and return VirtualMachineInstances
+- name: Use default ~/.kube/config and return VirtualMachineInstances from namespace testing with interfaces connected to network bridge-network
-# from namespace testing with interfaces connected to network bridge-network
+  plugin: kubevirt.core.kubevirt
-plugin: kubevirt.core.kubevirt
+  connections:
-connections:
+    - namespaces:
-  - namespaces:
+        - testing
-      - testing
+      network_name: bridge-network
-    network_name: bridge-network
 
-# Use default config (~/.kube/config) file and active context, and return VirtualMachineInstances
+- name: Use default ~/.kube/config and return VirtualMachineInstances from namespace testing with label app=test
-# from namespace testing with label app=test
+  plugin: kubevirt.core.kubevirt
-plugin: kubevirt.core.kubevirt
+  connections:
-connections:
+    - namespaces:
-  - namespaces:
+        - testing
-      - testing
+      label_selector: app=test
-    label_selector: app=test
 
-# Use a custom config file, and a specific context.
+- name: Use a custom config file and a specific context
-plugin: kubevirt.core.kubevirt
+  plugin: kubevirt.core.kubevirt
-connections:
+  connections:
-  - kubeconfig: /path/to/config
+    - kubeconfig: /path/to/config
-    context: 'awx/192-168-64-4:8443/developer'
+      context: 'awx/192-168-64-4:8443/developer'
 """
 
 from dataclasses import dataclass

plugins/modules/kubevirt_vm.py

@@ -188,7 +188,7 @@ EXAMPLES = """
               ssh_authorized_keys:
                 - ssh-ed25519 AAAA...
           name: cloudinit
-    wait: yes
+    wait: true
 
 - name: Delete a VirtualMachine
   kubevirt.core.kubevirt_vm:

plugins/modules/kubevirt_vm_info.py

@@ -97,7 +97,7 @@ EXAMPLES = """
   kubevirt.core.kubevirt_vm_info:
     name: testvm
     namespace: default
-    wait: yes
+    wait: true
 """
 
 RETURN = """

tests/integration/targets/inventory_kubevirt/playbook.yml

@@ -10,7 +10,7 @@
         namespace: default
         labels:
           app: test
-        wait: yes
+        wait: true
         wait_timeout: 600
         spec:
           domain:
@@ -41,7 +41,7 @@
         namespace: default
         labels:
           foo: bar
-        wait: yes
+        wait: true
         wait_timeout: 600
         spec:
           domain:

tests/integration/targets/inventory_kubevirt/test.kubevirt.yml

@@ -2,4 +2,4 @@ plugin: kubevirt.core.kubevirt
 connections:
   - namespaces:
       - default
-    create_groups: yes
+    create_groups: true

tests/integration/targets/inventory_kubevirt/test.label.kubevirt.yml

@@ -2,5 +2,5 @@ plugin: kubevirt.core.kubevirt
 connections:
   - namespaces:
       - default
-    create_groups: yes
+    create_groups: true
     label_selector: app=test

tests/integration/targets/inventory_kubevirt/test.net.kubevirt.yml

@@ -2,5 +2,5 @@ plugin: kubevirt.core.kubevirt
 connections:
   - namespaces:
       - default
-    create_groups: yes
+    create_groups: true
     network_name: bridge-network