dokku_ports: doesn't check for existing mappings

dokku / ansible-dokku

Ansible modules for installing and configuring Dokku

MIT License

169 stars 44 forks source link

dokku_ports: doesn't check for existing mappings #139

Closed fr3fou closed 1 year ago

fr3fou commented 2 years ago

Running

- name: Configure ports
  dokku_ports:
    app: api
    mappings:
      - http:80:8080
      - http:8080:8080

any number of times, always results in Ansible reporting this as "changed" image (2) I'd expect Ansible to say ok after the first time (given the fact that nothing was changed in between)

ltalirz commented 2 years ago

Agree, this is a bug. Don't have time to fix it currently; contributions welcome!