Nosmoht / ansible-module-foreman

Ansible module to manage configuration of Foreman
49 stars 32 forks source link

library: Allow to update compute resources #47

Closed agx closed 7 years ago

agx commented 7 years ago

This is not perfect. We always display changes since the Foreman API doesn't return the password but we have one and so the comparison fails. But it's far better than issuing "no changes required" as we do right now.

Nosmoht commented 7 years ago

I would prefer to remove the password from the compared dictionaries so the comparison does not return changed,

agx commented 7 years ago

Since #48 is merged this is no longer needed. Thanks!