)]}'
{"tasks/main.yml":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"ff9461b9a102894b0672f719f7cb625ada2873e1","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Install networkd distro packages"},{"line_number":17,"context_line":"  package:"},{"line_number":18,"context_line":"    name: \"{{ systemd_networkd_distro_packages }}\""},{"line_number":19,"context_line":"    state: \"{{ systemd_networkd_package_state }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df7087c5_975f720d","line":16,"updated":"2018-03-20 04:28:37.000000000","message":"I think it\u0027d be best if the role simply assumes networkd is installed and available on the host.","commit_id":"b31267b03fecbab4748f28d39736d3539641f479"},{"author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"change_message_id":"f8b7f30e4fa0144c3e33393fb29942a1f615b883","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Install networkd distro packages"},{"line_number":17,"context_line":"  package:"},{"line_number":18,"context_line":"    name: \"{{ systemd_networkd_distro_packages }}\""},{"line_number":19,"context_line":"    state: \"{{ systemd_networkd_package_state }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df7087c5_7bbba00d","line":16,"in_reply_to":"df7087c5_3bd68851","updated":"2018-03-20 11:23:09.000000000","message":"Also another problem.. in the review above, it still fails centos even with this patch as parent. The centos install task shows state OK so it seems like these packages are already installed but not working?","commit_id":"b31267b03fecbab4748f28d39736d3539641f479"},{"author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"change_message_id":"5918c6f19053b9033d0bd68ec2ab21c0a3f513f3","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Install networkd distro packages"},{"line_number":17,"context_line":"  package:"},{"line_number":18,"context_line":"    name: \"{{ systemd_networkd_distro_packages }}\""},{"line_number":19,"context_line":"    state: \"{{ systemd_networkd_package_state }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df7087c5_dbfe6c6a","line":16,"in_reply_to":"df7087c5_7bbba00d","updated":"2018-03-20 11:37:12.000000000","message":"I added a limit so the install task only runs if networkd is enabled... is that OK?\n\nRegarding the issue with the tests patch still failling.. maybe networkd is installed but masked on nodepool nodes?","commit_id":"b31267b03fecbab4748f28d39736d3539641f479"},{"author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"change_message_id":"32e9b1aba5c74b1a1bf8ad1b03391db3e165d7ca","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Install networkd distro packages"},{"line_number":17,"context_line":"  package:"},{"line_number":18,"context_line":"    name: \"{{ systemd_networkd_distro_packages }}\""},{"line_number":19,"context_line":"    state: \"{{ systemd_networkd_package_state }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df7087c5_3bd68851","line":16,"in_reply_to":"df7087c5_975f720d","updated":"2018-03-20 11:21:45.000000000","message":"In https://review.openstack.org/#/c/554350/ would you suggest we install packages in the test playbook?","commit_id":"b31267b03fecbab4748f28d39736d3539641f479"}]}