Bug 1697869
Summary: | ansible should own /usr/share/ansible and subdirectories, /usr/share/ansible/roles | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Till Maas <till> | ||||
Component: | ansible | Assignee: | Kevin Fenzi <kevin> | ||||
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | unspecified | Docs Contact: | |||||
Priority: | unspecified | ||||||
Version: | rawhide | CC: | a.badger, athmanem, kevin, maxim, mhroncok, pcahyna | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | ansible-2.8.2-1.el7 ansible-2.8.2-1.fc30 ansible-2.8.2-1.fc29 | Doc Type: | If docs needed, set a value | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2019-07-23 01:49:12 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Till Maas
2019-04-09 08:31:23 UTC
My RHEL package (when Ansible was still in RHEL 7 Extras) used to have this in %files: %dir /usr/share/ansible %dir /usr/share/ansible/roles %dir /usr/share/ansible/plugins As do the Ansible Engine builds (I looked at ansible-2.7.9-1.el7ae) PR applied. Do you need it now? Or can we wait for the next release? Or would you like rawhide now? Hold on.... what's the rationale for owning those directories? The Ansible Engine 2.8.0 build was going to drop those directories from the spec file as we don't place any files in there and to be more inline with the Fedora package. If they're going to be added to the Fedora package, I should probably re-add them to the engine package as well. It would be good if we had a common understanding of the criteria for adding the directories, though. Perhaps something like "every /usr/share/ansible directory listed in https://github.com/ansible/ansible/blob/devel/lib/ansible/config/base.yml " ? Toshio, the current need comes from the linux-system-roles package that puts content there. For RHEL there is rhel-system-roles with the same problem. To ensure proper directory ownership the ansible package should own the directories. Kevin, thank you! The next release is fine with me. I do not think it rewards a separate update. Okay, but what I'm saying is that we need to coordinate this change otherwise linux-system-roles won't work with the packages provided by Ansible Engine 2.8. Can we agree on what criteria to use to decide what directories need to be owned by the packages? Created attachment 1555378 [details]
Potential change to own all defaults system paths in ansible config
Okay, this would implement owning all of the system paths that ansible lists in its base config.
Yes, it makes sense to add all directories to the spec where other RPMs could drop files/directories. Unfortunately your patch seems to be wrong: It creates everything below %{_datadir}/ansible/plugins/ but the roles directory needs to be created in %{_datadir}/ansible/ https://github.com/ansible/ansible/blob/devel/lib/ansible/config/base.yml#L989 https://github.com/ansible/ansible/blob/devel/lib/ansible/config/base.yml#L380 Also it would be nice to sort the plugin type entries to make it easier to check if a certain entry is in there. <nod> Yeah, and it looks like the same fix is needed for collections as well. I'm not sure what the best sort order is. The current order matches what's in the base.yml file which makes a simple grep work well but doesn't work so well when upstream changes the order in their file. Till, PR here: https://src.fedoraproject.org/rpms/ansible/pull-request/9 Can this be fixed in Fedora 30 as well please? FEDORA-2019-7301f6afd1 has been submitted as an update to Fedora 30. https://bodhi.fedoraproject.org/updates/FEDORA-2019-7301f6afd1 FEDORA-2019-4523b5e0c2 has been submitted as an update to Fedora 29. https://bodhi.fedoraproject.org/updates/FEDORA-2019-4523b5e0c2 FEDORA-EPEL-2019-e96deab9a3 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2019-e96deab9a3 ansible-2.8.2-1.fc30, python-sphinx-notfound-page-0.4-2.fc30 has been pushed to the Fedora 30 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-7301f6afd1 ansible-2.8.2-1.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2019-e96deab9a3 ansible-2.8.2-1.fc29, python-sphinx-notfound-page-0.4-2.fc29 has been pushed to the Fedora 29 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-4523b5e0c2 ansible-2.8.2-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report. ansible-2.8.2-1.fc30, python-sphinx-notfound-page-0.4-2.fc30 has been pushed to the Fedora 30 stable repository. If problems still persist, please make note of it in this bug report. ansible-2.8.2-1.fc29, python-sphinx-notfound-page-0.4-2.fc29 has been pushed to the Fedora 29 stable repository. If problems still persist, please make note of it in this bug report. |