Fix race condition in obtaining the public IP for ec2 (#43)

[danielpodwysocki]

Fixes #43

This prevents a race condition where we attempt to look up the public IP before an interface is assigned to the instance.

An instance in a "running" state will always have this completed - a "started" one not necessarily.

This would then go on and fail when this module is called (

molecule-plugins/src/molecule_plugins/ec2/cookiecutter/{{cookiecutter.molecule_directory}}/{{cookiecutter.scenario_name}}/create.yml

Line 264 in 7fe6acc

- name: Collect instance configs

)

        - name: Collect instance configs
          set_fact:
            instance_config:
              instance: "{{ item.name }}"
              address: "{{ item.assign_public_ip | ternary(instance.public_ip_address, instance.private_ip_address) }}"
              user: "{{ item.ssh_user }}"
              port: "{{ item.ssh_port }}"
              identity_file: "{{ item.private_key_path }}"
              instance_ids:
                - "{{ instance.instance_id }}"
          vars:
            instance: "{{ ec2_instances.results[index].instances[0] }}"
          loop: "{{ platforms }}"
          loop_control:
            index_var: index
            label: "{{ item.name }}"
          register: instance_configs

[ssbarnea]

Please raise this towards the new code location at https://github.com/ansible-community/molecule-plugins

[danielpodwysocki]

@ssbarnea This had already been opened at the new repo. Can we get this PR reopened?

[danielpodwysocki]

Any news on this PR?

[apatard]

@danielpodwysocki damn. forgot that gh rebase is messing with the commit authorship, sorry. Feel free to push force a commit with proper authorship if you care. If you don't mind it, say it too and I'll merge the PR.

[apatard]

after checking seems it's fine. Sorry for the noise, I've yet to learn how to deal with github interface. I'm going to merge the PR then.