epiphany icon indicating copy to clipboard operation
epiphany copied to clipboard

[FEATURE REQUEST] Review warnings from Ansible output

Open to-bar opened this issue 5 years ago • 2 comments

Is your feature request related to a problem? Please describe. We have some warnings in Ansible output log.

Describe the solution you'd like Get rid of warnings. Check and try to improve our code.

Additional context Examples: [DEPRECATION WARNING]: evaluating {'evictionHard': {'memory.available': '200Mi'}, 'kubeReserved': {'cpu': '50m', 'memory': '256Mi'}, 'systemReserved': {'cpu': '50m', 'memory': '768Mi'}} as a bare variable, this behaviour will go away and you might need to add |bool to the expression in the future. Also see CONDITIONAL_BARE_VARS configuration toggle. This feature will be removed in version 2.12. Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.

[DEPRECATION WARNING]: ALLOW_WORLD_READABLE_TMPFILES option, moved to a per plugin approach that is more flexible, use mostly the same config will work, but now controlled from the plugin itself and not using the general constant.


DoD checklist

  • Changelog
    • [ ] updated
    • [ ] not needed
  • COMPONENTS.md
    • [ ] updated
    • [ ] not needed
  • Schema
    • [ ] updated
    • [ ] not needed
  • Backport tasks
    • [ ] created
    • [ ] not needed
  • Documentation
    • [ ] added
    • [ ] updated
    • [ ] not needed
  • [ ] Feature has automated tests
  • [ ] Automated tests passed (QA pipelines)
    • [ ] apply
    • [ ] upgrade
    • [ ] backup/restore
  • [ ] Idempotency tested
  • [ ] All conversations in PR resolved
  • [ ] Solution meets requirements and is done according to design doc
  • [ ] Usage compliant with license

to-bar avatar Dec 30 '19 15:12 to-bar

Is this issue still valid @to-bar ?

rafzei avatar Mar 11 '21 08:03 rafzei

Yes, there is at least:

[DEPRECATION WARNING]: ALLOW_WORLD_READABLE_TMPFILES option, moved to a per plugin approach that is more flexible, use mostly the same config will work, but now controlled from the plugin itself and not using the general constant.

to-bar avatar Mar 11 '21 14:03 to-bar

This was already done with several Ansible upgrades so closing for now. Can be recreated as they pop up.

seriva avatar Nov 16 '22 11:11 seriva