theforeman / obal

packaging wrapper using ansible
GNU General Public License v2.0
9 stars 13 forks source link

Add support for rpmlint folder in the root of inventory_dir #377

Closed Odilhao closed 10 months ago

Odilhao commented 10 months ago

This PRs adds support for one folder called rpmlint inside of the inventory_dir.

With Pulp packaging going to Python 3.11, rpmlint is about to break when the Jenkins nodes start to use one ansible version that requires python 3.11 and install python3.11-libs, it will start to complain about the dist-utls deprecation in python 3.12+.

evgeni commented 10 months ago

What's the reason to do it like this vs setting rpmlint_rc_file in the manifest?

I would expect this to break when someone wants to set rc file in the manifest, but there is one in the folder?

Odilhao commented 10 months ago

Sometimes I forget that we can use the manifest as one Ansible Inventory, I need to buy one rubber duck

evgeni commented 10 months ago

E-v6TS_XEAAEvRC