Fix eip association when both instance id and private ip address are …

…passed (ansible-collections#328)

* Fix ec2_eip with both instance_id and private_ip_address
* Add changelog fragment for the ec2_eip fix

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@4d1aa98

plugins/modules/ec2_eip.py

@@ -241,7 +241,7 @@ def associate_ip_and_device(ec2, module, address, private_ip_address, device_id,
                     AllowReassociation=allow_reassociation,
                 )
                 if private_ip_address:
-                    params['PrivateIPAddress'] = private_ip_address
+                    params['PrivateIpAddress'] = private_ip_address
                 if address['Domain'] == 'vpc':
                     params['AllocationId'] = address['AllocationId']
                 else:

tests/integration/targets/ec2_eip/tasks/main.yml

@@ -571,6 +571,22 @@
         - instance_eip is success
         - eip_info.addresses[0].allocation_id is defined
         - eip_info.addresses[0].instance_id == '{{ instance_info.instances[0].instance_id }}'
+  - name: Attach eip to an EC2 instance with private Ip specified
+    ec2_eip:
+      device_id: '{{ instance_info.instances[0].instance_id }}'
+      private_ip_address: '{{ instance_info.instances[0].private_ip_address }}'
+      state: present
+      release_on_disassociation: yes
+    register: instance_eip
+  - ec2_eip_info:
+      filters:
+        public-ip: '{{ instance_eip.public_ip }}'
+    register: eip_info
+  - assert:
+      that:
+        - instance_eip is success
+        - eip_info.addresses[0].allocation_id is defined
+        - eip_info.addresses[0].instance_id == '{{ instance_info.instances[0].instance_id }}'
   # =====================================================
   - name: Cleanup instance
     ec2_instance: