)]}'
{"elements/ironic-agent/post-install.d/80-ironic-agent":[{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"c7733f36ce547b25988f6b5eb64dd5673d617a92","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"install-packages -e python-dev git cloud-init"},{"line_number":10,"context_line":"# dnf removes dmidecode even though it\u0027s listed in package-installs"},{"line_number":11,"context_line":"install-packages dmidecode"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"rm -rf /tmp/ironic-python-agent"},{"line_number":14,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"1a4dcd0f_ba75c698","line":11,"updated":"2015-08-11 22:20:35.000000000","message":"This explains \"What?\" but not \"Why?\" Do we know why dnf is doing that?","commit_id":"4ed26fc06134c7f82149adf90fc4da48948f1662"},{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"24fef99ee58d7603aa024fda8c0a7a4b2f31608d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"install-packages -e python-dev git cloud-init"},{"line_number":10,"context_line":"# dnf removes dmidecode even though it\u0027s listed in package-installs"},{"line_number":11,"context_line":"install-packages dmidecode"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"rm -rf /tmp/ironic-python-agent"},{"line_number":14,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"fa1b9901_c4587f8d","line":11,"in_reply_to":"1a4dcd0f_4e0dda3c","updated":"2015-08-21 19:12:21.000000000","message":"Where do we remove cloud-init? I really think we should try and find another way around this issue...","commit_id":"4ed26fc06134c7f82149adf90fc4da48948f1662"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d53e4f7789932621cd61c76c643f8a64c433ca90","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"install-packages -e python-dev git cloud-init"},{"line_number":10,"context_line":"# dnf removes dmidecode even though it\u0027s listed in package-installs"},{"line_number":11,"context_line":"install-packages dmidecode"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"rm -rf /tmp/ironic-python-agent"},{"line_number":14,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"1a4dcd0f_4e0dda3c","line":11,"in_reply_to":"1a4dcd0f_ba75c698","updated":"2015-08-12 08:05:05.000000000","message":"My guess is that dmidecode is installed as dependency for cloud-init. Then on removing cloud-init dnf considers this an unneeded dependency and removes it. Which is usually a good thing to do, but not this time.","commit_id":"4ed26fc06134c7f82149adf90fc4da48948f1662"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"858b93785db417deb8f036be1f2a27508ee57dfe","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"install-packages -e python-dev git cloud-init"},{"line_number":10,"context_line":"# dnf removes dmidecode even though it\u0027s listed in package-installs"},{"line_number":11,"context_line":"install-packages dmidecode"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"rm -rf /tmp/ironic-python-agent"},{"line_number":14,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"fa1b9901_7e5490c9","line":11,"in_reply_to":"fa1b9901_17a54bb4","updated":"2015-08-22 15:14:50.000000000","message":"In this use case, there is absolutely no reason for cloud-init to be involved as this element is building a purpose built ramdisk image to write out user operating systems to machine nodes.","commit_id":"4ed26fc06134c7f82149adf90fc4da48948f1662"},{"author":{"_account_id":10035,"name":"greghaynes","email":"greg@greghaynes.net","username":"greghaynes"},"change_message_id":"cc2d9e37264bf192d32eac6fcf7ceda9c07bc5e1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"install-packages -e python-dev git cloud-init"},{"line_number":10,"context_line":"# dnf removes dmidecode even though it\u0027s listed in package-installs"},{"line_number":11,"context_line":"install-packages dmidecode"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"rm -rf /tmp/ironic-python-agent"},{"line_number":14,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"da20952f_d3ea8cac","line":11,"in_reply_to":"fa1b9901_7e5490c9","updated":"2015-09-01 15:33:48.000000000","message":"Chatted in IRC: What is going on is that if you install x that depends on y, then install y, and then remove x dnf will still attempt to remove y. Can we update the comment above to explain this, and also add a comment above the dmidecode in package-installs.yaml to warn that the package-installs isnt respected on dnf, see comment in 80-ironic-agent.\n\nSorry for being a pain on comments, but I can see this biting us in the future really easily.","commit_id":"4ed26fc06134c7f82149adf90fc4da48948f1662"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"51d0ce432bc020a4adeffc03c07f9150bd68dcfc","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"install-packages -e python-dev git cloud-init"},{"line_number":10,"context_line":"# dnf removes dmidecode even though it\u0027s listed in package-installs"},{"line_number":11,"context_line":"install-packages dmidecode"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"rm -rf /tmp/ironic-python-agent"},{"line_number":14,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"fa1b9901_17a54bb4","line":11,"in_reply_to":"fa1b9901_c4587f8d","updated":"2015-08-21 19:32:55.000000000","message":"We remove cloud-init on line 9. Any ideas? I have only this...","commit_id":"4ed26fc06134c7f82149adf90fc4da48948f1662"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"37555412b93267bba8b9931a53bdb96391caea71","unresolved":false,"context_lines":[{"line_number":23,"context_line":"        ;;"},{"line_number":24,"context_line":"    systemd)"},{"line_number":25,"context_line":"        systemctl disable iptables.service"},{"line_number":26,"context_line":"        systemctl enable openstack-ironic-python-agent.service"},{"line_number":27,"context_line":"        ;;"},{"line_number":28,"context_line":"    sysv)"},{"line_number":29,"context_line":"        update-rc.d iptables disable"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"da20952f_0ce43039","line":26,"updated":"2015-09-02 12:33:24.000000000","message":"Service name is different with a source install, there is another patch in flight remedy this.  These steps were primarily intended to disable the firewall configuration the host upon reboot.  Trown\u0027s patch at https://review.openstack.org/#/c/218879/ addresses this.","commit_id":"5f68750b4b152a927d7f3ab4dc21b73d24733081"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"37555412b93267bba8b9931a53bdb96391caea71","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        ;;"},{"line_number":28,"context_line":"    sysv)"},{"line_number":29,"context_line":"        update-rc.d iptables disable"},{"line_number":30,"context_line":"        update-rc.d openstack-ironic-python-agent enable"},{"line_number":31,"context_line":"        ;;"},{"line_number":32,"context_line":"    *)"},{"line_number":33,"context_line":"        echo \"Unsupported init system\""}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"da20952f_accf44b2","line":30,"updated":"2015-09-02 12:33:24.000000000","message":"Ditto","commit_id":"5f68750b4b152a927d7f3ab4dc21b73d24733081"}]}