oravirt/ansible-oracle-modules

Migration to Python3 - No cmp support

capitanh opened this issue · 2 comments

capitanh commented

Are you planning a full python3 support in the future? After upgrading our ddbb servers to centos 8 we were forced to use python3, and some of your ansible modules stopped working. Specifically, the use of cmp() in oracle_profile is no longer supported in python3. We found a workaround for that one, but then oracle_role also failed. It looks like some others will fail too... It would be great if you could make this (incredibly useful) set of modules full python3-compatible

Thanks for your work

serpro69 commented

Any updates on this. Facing the same issue :/

ari-stark commented

Try to change if cmp(dict(current_properties), dict(wanted_attributes)) is not 0: by if dict(current_properties) != dict(wanted_attributes):.

There's no real reason to use comparison when you just want to know if it's different.