)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"39a1d10600495fe282e3b80626f09f3008f130f2","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Make the rootwrap filters more portable by removing"},{"line_number":10,"context_line":"the explicit exec_dirs prefix from them. This allows"},{"line_number":11,"context_line":"a binariy in all exec_dirs to be used by rootwrap"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ieccb8c480e0e116a976d0920ab900305a5615015"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAOn%2F%2FzLQ%3D","line":11,"updated":"2013-04-22 21:58:04.000000000","message":"Typo - binary :) - also missing full stops on your sentences.","commit_id":"3bfbd618a5a76a5477440f88867c40eed16c1970"},{"author":{"_account_id":6593,"name":"Dirk Mueller","email":"dirk@dmllr.de","username":"dmllr"},"change_message_id":"3498b266396425a88722d7887b5a796204e81ab3","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Make the rootwrap filters more portable by removing"},{"line_number":10,"context_line":"the explicit exec_dirs prefix from them. This allows"},{"line_number":11,"context_line":"a binariy in all exec_dirs to be used by rootwrap"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ieccb8c480e0e116a976d0920ab900305a5615015"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAOn%2F%2FxZo%3D","line":11,"in_reply_to":"AAAAOn%2F%2FzLQ%3D","updated":"2013-04-23 21:11:13.000000000","message":"Thanks for the review. I\u0027ve changed the commit message accordingly and will reupload the change once I figured out why devstack is failing on it.","commit_id":"3bfbd618a5a76a5477440f88867c40eed16c1970"}],"etc/nova/rootwrap.d/baremetal-compute-ipmi.filters":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"930413fa58de3f7fb342867609cd75b078181a87","unresolved":false,"context_lines":[{"line_number":6,"context_line":"ipmitool: CommandFilter, ipmitool, root"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"# nova/virt/baremetal/ipmi.py: \u0027kill\u0027, \u0027-TERM\u0027, str(console_pid)"},{"line_number":9,"context_line":"kill_shellinaboxd: KillFilter, root, /usr/local/bin/shellinaboxd, -15, -TERM"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAAPH%2F%2F8pE%3D","line":9,"updated":"2013-05-18 01:54:08.000000000","message":"I wonder if we should fix shellinaboxd too?","commit_id":"04283565ec32c023d373547b77df8f5b7dadaf92"},{"author":{"_account_id":6593,"name":"Dirk Mueller","email":"dirk@dmllr.de","username":"dmllr"},"change_message_id":"336a8060b33680cfea8d3a7dc93ab2ef4074d885","unresolved":false,"context_lines":[{"line_number":6,"context_line":"ipmitool: CommandFilter, ipmitool, root"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"# nova/virt/baremetal/ipmi.py: \u0027kill\u0027, \u0027-TERM\u0027, str(console_pid)"},{"line_number":9,"context_line":"kill_shellinaboxd: KillFilter, root, /usr/local/bin/shellinaboxd, -15, -TERM"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAAPH%2F%2F8fs%3D","line":9,"in_reply_to":"AAAAPH%2F%2F8pE%3D","updated":"2013-05-18 06:26:39.000000000","message":"Yes, we should but I first need to update KillFilter to handle exec_dirs - that was the reason the previous attempts failed, because so far only CommandFilter is handling exec_dirs. I\u0027ll submit a followup patch once the issues is solved in oslo-incubator.","commit_id":"04283565ec32c023d373547b77df8f5b7dadaf92"}]}