)]}'
{"playbooks/roles/bifrost-create-vm-nodes/tasks/prepare_libvirt.yml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c71c27c1b30c8780e1c004512bddf335a7420fd0","unresolved":false,"context_lines":[{"line_number":162,"context_line":"    mode: \"0755\""},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"# NOTE(mgoddard): Install to system to ensure SELinux allows Systemd to execute"},{"line_number":165,"context_line":"# the daemon."},{"line_number":166,"context_line":"- name: install virtualbmc"},{"line_number":167,"context_line":"  pip:"},{"line_number":168,"context_line":"    name: virtualbmc"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1fa4df85_053157a3","line":165,"updated":"2020-03-09 11:28:28.000000000","message":"Can we better configure selinux? I\u0027m worried about regressing to requiring \u0027sudo pip install\u0027, it makes bifrost not co-installable with largely anything that uses python.","commit_id":"14a587f79b09593300f86b101099acff36605569"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6bd36902f90b18962d7f24d0b2bacc7c3ab5b40f","unresolved":false,"context_lines":[{"line_number":162,"context_line":"    mode: \"0755\""},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"# NOTE(mgoddard): Install to system to ensure SELinux allows Systemd to execute"},{"line_number":165,"context_line":"# the daemon."},{"line_number":166,"context_line":"- name: install virtualbmc"},{"line_number":167,"context_line":"  pip:"},{"line_number":168,"context_line":"    name: virtualbmc"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1fa4df85_a5e4030e","line":165,"in_reply_to":"1fa4df85_053157a3","updated":"2020-03-09 11:31:35.000000000","message":"We only do this for testing.","commit_id":"14a587f79b09593300f86b101099acff36605569"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"a9329284ea7c5cefcf250e4fd3f71a9c0271f2f7","unresolved":false,"context_lines":[{"line_number":162,"context_line":"    mode: \"0755\""},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"# NOTE(mgoddard): Install to system to ensure SELinux allows Systemd to execute"},{"line_number":165,"context_line":"# the daemon."},{"line_number":166,"context_line":"- name: install virtualbmc"},{"line_number":167,"context_line":"  pip:"},{"line_number":168,"context_line":"    name: virtualbmc"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1fa4df85_e5851b08","line":165,"in_reply_to":"1fa4df85_a5e4030e","updated":"2020-03-09 11:39:20.000000000","message":"Well, testing nodes may also have python packages conflicting with what we\u0027re trying to install.","commit_id":"14a587f79b09593300f86b101099acff36605569"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"b508b91c7b637685ec7096e99a3ec913a9f586f6","unresolved":false,"context_lines":[{"line_number":162,"context_line":"    mode: \"0755\""},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"# NOTE(mgoddard): Install to system to ensure SELinux allows Systemd to execute"},{"line_number":165,"context_line":"# the daemon."},{"line_number":166,"context_line":"- name: install virtualbmc"},{"line_number":167,"context_line":"  pip:"},{"line_number":168,"context_line":"    name: virtualbmc"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1fa4df85_7bdf8868","line":165,"in_reply_to":"1fa4df85_e5851b08","updated":"2020-03-09 13:28:18.000000000","message":"Testing locally, it at least seems to start up ok using the default CI virtualenv of /opt/stack/bifrost. I\u0027ve added it back. The problem I had with Tenks was when using a virtualenv in a home directory. We don\u0027t have CentOS CI working currently, so suggest we merge this to get CI passing again and address SELinux with CentOS 8 if necessary.","commit_id":"14a587f79b09593300f86b101099acff36605569"}]}