EDPM deployment should not do network cleanup by default

With nmstate-provider set as default for os-net-config,
default cleanup during update/adoption is not desired.

Signed-off-by: <[email protected]>
Jira: https://issues.redhat.com/browse/OSPRH-16537
      https://issues.redhat.com/browse/OSPRH-17036

Author: vcandapp

docs/source/roles/role-edpm_network_config.rst

@@ -19,10 +19,14 @@ This Ansible role does the following tasks:
   - Checks for the presence of required RPMS
   - Uses "provider" ifcfg/nmstate based on flag "edpm_network_config_nmstate"
 
-Note: By default this role will cleanup devices/interfaces not in
-"edpm_network_config_template". If there is requirement to keep them
-for pre-provisioned nodes, "edpm_network_config_nonconfigured_cleanup"
-ansible var can be set to "false".
+Note: * With nmstate-provider as the default for os-net-config,
+        "os-net-config --cleanup" should be used only for very
+        specific use case, eg
+        - SSH provisioned over VLAN-tagged single interface
+        - os-net-config needs to override ctlplane over bond+VLAN
+
+      * "os-net-config --cleanup" SHOULD NOT be set for update/adoption
+        use case
 
 Here is an example playbook to run os-net-config tool:
 

roles/edpm_network_config/defaults/main.yml

@@ -54,4 +54,4 @@ edpm_network_config_safe_defaults: false
 edpm_network_config_template: ""
 edpm_bond_interface_ovs_options: "bond_mode=active-backup"
 edpm_dns_search_domains: []
-edpm_network_config_nonconfigured_cleanup: true
+edpm_network_config_nonconfigured_cleanup: false

roles/edpm_network_config/meta/argument_specs.yml

@@ -81,5 +81,5 @@ argument_specs:
         default: nic1
       edpm_network_config_nonconfigured_cleanup:
         type: bool
-        description: "Cleanup network interfaces not in network config"
-        default: true
+        description: "Cleanup network interfaces before network config"
+        default: false

roles/edpm_network_config/molecule/nmstate/converge.yml

@@ -14,7 +14,6 @@
 # License for the specific language governing permissions and limitations
 # under the License.
 
-
 - name: Converge
   hosts: all
   gather_facts: false
@@ -30,9 +29,11 @@
           state: up
           ipv4:
             address:
-            - ip: 192.168.180.2
-              prefix-length: 24
+              - ip: 192.168.180.2
+                prefix-length: 24
             dhcp: false
             enabled: true
+          ipv6:
+            enabled: false
   roles:
     - role: osp.edpm.edpm_network_config