elb_network_lb - change default state from 'absent' to 'present' (ans…

…ible-collections#1167) elb_network_lb - change default state from 'absent' to 'present' SUMMARY change default state from 'absent' to 'present' to match our usual behaviour. ISSUE TYPE Feature Pull Request COMPONENT NAME elb_network_lb ADDITIONAL INFORMATION See also: ansible/ansible#65468 Reviewed-by: Markus Bergholz <[email protected]>
abikouo · Sep 18, 2023 · a51e68c · a51e68c
1 parent 73a51e9
commit a51e68c
Showing 1 changed file with 8 additions and 13 deletions.
diff --git a/elb_network_lb.py b/elb_network_lb.py
@@ -108,11 +108,10 @@
   state:
     description:
       - Create or destroy the load balancer.
-      - The current default is C(absent).  However, this behavior is inconsistent with other modules
-        and as such the default will change to C(present) in a release after 2022-06-01.
-        To maintain the existing behavior explicitly set I(state=absent).
+      - The default changed from C('absent') to C('present') in release 4.0.0.
     choices: [ 'present', 'absent' ]
     type: str
+    default: 'present'
   tags:
     description:
       - A dictionary of one or more tags to assign to the load balancer.
@@ -448,28 +447,24 @@ def main():
             subnets=dict(type='list', elements='str'),
             subnet_mappings=dict(type='list', elements='dict'),
             scheme=dict(default='internet-facing', choices=['internet-facing', 'internal']),
-            state=dict(choices=['present', 'absent'], type='str'),
+            state=dict(choices=['present', 'absent'], type='str', default='present'),
             tags=dict(type='dict'),
             wait_timeout=dict(type='int'),
             wait=dict(type='bool'),
             ip_address_type=dict(type='str', choices=['ipv4', 'dualstack'])
         )
     )
 
+    required_if = [
+        ('state', 'present', ('subnets', 'subnet_mappings',), True)
+    ]
+
     module = AnsibleAWSModule(argument_spec=argument_spec,
+                              required_if=required_if,
                               mutually_exclusive=[['subnets', 'subnet_mappings']])
 
     # Check for subnets or subnet_mappings if state is present
     state = module.params.get("state")
-    if state == 'present':
-        if module.params.get("subnets") is None and module.params.get("subnet_mappings") is None:
-            module.fail_json(msg="'subnets' or 'subnet_mappings' is required when state=present")
-
-    if state is None:
-        # See below, unless state==present we delete.  Ouch.
-        module.deprecate('State currently defaults to absent.  This is inconsistent with other modules'
-                         ' and the default will be changed to `present` in a release after 2022-06-01',
-                         date='2022-06-01', collection_name='community.aws')
 
     # Quick check of listeners parameters
     listeners = module.params.get("listeners")