RedHatSatellite / satellite-clone

Utility to assist cloning an existing Satellite 6 server to a new host
GNU General Public License v3.0
37 stars 31 forks source link

callback warnings/deprecations with ansible-core 2.12+ #420

Closed evgeni closed 1 year ago

evgeni commented 2 years ago

With the ansible packaging re-shuffling, we now have a plain ansible-core in EL 8.6+ and should try to be compatible with that.

While running in such a setup, I've seen the following warnings:

The deprecation warning is harmless for now.

The profile_tasks callback has been moved to the ansible.posix collection which isn't available by default to core users. We don't strictly need it, but it also doesn't hurt as Ansible just ignores the setting.

The human_log callback is the one we ship in callback_plugins and that's just plain incompatible with Python 3. Again, no functional problem as Ansible is smart enough to ignore it. The normal guideline is to just replace human_log with debug as that has all the same output and was part of Ansible since 2.2, but that's also only in the ansible.posix collection and thus not available if you have core only. Annoyingly, we can't just configure the debug callback for users who have it, as there is no fallback for the stdout_callback setting and Ansible errors out: ERROR! Invalid callback for stdout specified: debug

evgeni commented 1 year ago

@ehelms @ekohl thoughts?

evgeni commented 1 year ago

The later is now fixed via https://github.com/RedHatSatellite/satellite-clone/pull/437

I consider this issue fixed