[travis skip] idempotence

OsgiliathEnterprise · May 6, 2021 · 324e817 · 324e817
1 parent 74c3f76
commit 324e817
Show file tree

Hide file tree

Showing 6 changed files with 4 additions and 32 deletions.
diff --git a/.github/workflows/molecule.yml b/.github/workflows/molecule.yml
@@ -26,7 +26,7 @@ jobs:
       - name: install lint prerequisite
         run: |
           sudo apt-get update
-          sudo apt -y install python3-setuptools ansible-lint vagrant
+          sudo apt -y install python3-setuptools vagrant
 
       - name: Install molecule
         run: |

diff --git a/molecule/default/molecule.yml b/molecule/default/molecule.yml
@@ -37,19 +37,3 @@ verifier:
     PYTHONWARNINGS: "ignore:.*U.*mode is deprecated:DeprecationWarning"
   options:
     v: 1
-scenario:
-  name: default
-  test_sequence:
-    - lint
-    - dependency
-    - cleanup
-    - destroy
-    - syntax
-    - create
-    - prepare
-    - converge
- #   - idempotence
-    - side_effect
-    - verify
-    - cleanup
-    - destroy
diff --git a/molecule/kvm/molecule.yml b/molecule/kvm/molecule.yml
@@ -31,17 +31,3 @@ verifier:
     v: 1
 scenario:
   name: kvm
-  test_sequence:
-    - lint
-    - dependency
-    - cleanup
-    - destroy
-    - syntax
-    - create
-    - prepare
-    - converge
- #   - idempotence
-    - side_effect
-    - verify
-    - cleanup
-    - destroy
diff --git a/tasks/networks.yml b/tasks/networks.yml
@@ -22,4 +22,5 @@
     state: active
     uri: "{{ libvirt_host_uri | default(omit, true) }}"
   with_items: "{{ libvirt_host_networks }}"
+  changed_when: false
   become: True
diff --git a/tasks/pools.yml b/tasks/pools.yml
@@ -61,6 +61,7 @@
     state: active
     uri: "{{ libvirt_host_uri | default(omit, true) }}"
   loop: "{{ libvirt_host_pools | flatten(levels=1) }}"
+  changed_when: false
   become: True
 
 - name: Ensure libvirt storage pools are started on boot

diff --git a/templates/network.xml.j2 b/templates/network.xml.j2
@@ -1,7 +1,7 @@
 <network connections='1'>
   <name>{{ item.name }}</name>
   <forward mode='{{ item.mode }}'/>
-  <bridge name='{{ item.bridge }}'{% if item.mode == 'route' or item.mode == 'nat' %} zone='{{item.zone | default("public")}}'{% endif %}/>
+  <bridge name='{{ item.bridge }}'{% if item.mode == 'route' or item.mode == 'nat' %} zone='{{ item.zone | default("public") }}'{% endif %}/>
   {% if item.mode == 'route' or item.mode == 'nat' %}
   <domain name='{{ item.domain|default(item.name) }}'/>
   <ip address='{{ item.ip }}' netmask='{{ item.netmask }}'>