)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"ebcbfc51994be641afacbae3d6a5b54c1563d4bc","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes nova-compute failing to start on Fedora 18"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes bug: https://bugs.launchpad.net/devstack/+bug/1086784"},{"line_number":10,"context_line":"Adds a rule to the policy-kit allowing the stack user to manage libvirt"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I6e9c0106c932f5f5f5c5c18ff79ac81a050c4599"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAM3%2F%2Fd20%3D","line":9,"updated":"2013-01-21 14:18:49.000000000","message":"You can make this simply \"Fixes bug #1086784\" and the launchpad bug will be auto-updated","commit_id":"193a57b97aae9007cc1b2d7bcc94365635ebd15d"},{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"dccf5feb1a74c6fdb24923be92657e6d1a02ed16","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes nova-compute failing to start on Fedora 18"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixes bug: https://bugs.launchpad.net/devstack/+bug/1086784"},{"line_number":10,"context_line":"Adds a rule to the policy-kit allowing the stack user to manage libvirt"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I6e9c0106c932f5f5f5c5c18ff79ac81a050c4599"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAM3%2F%2Fd10%3D","line":9,"in_reply_to":"AAAAM3%2F%2Fd20%3D","updated":"2013-01-21 14:29:36.000000000","message":"Thanks for tip! Will do","commit_id":"193a57b97aae9007cc1b2d7bcc94365635ebd15d"}],"stack.sh":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"ebcbfc51994be641afacbae3d6a5b54c1563d4bc","unresolved":false,"context_lines":[{"line_number":666,"context_line":""},{"line_number":667,"context_line":"# On fedora 18 enable stack-user to virsh -c qemu:///system"},{"line_number":668,"context_line":"# by creating a policy-kit rule for stack-user"},{"line_number":669,"context_line":"if is_fedora \u0026\u0026  [[ \"$os_RELEASE\" \u003d\u003d \"18\" ]]; then"},{"line_number":670,"context_line":"    echo_summary \"Creating policy-kit rules\""},{"line_number":671,"context_line":""},{"line_number":672,"context_line":"    cat \u003c\u003cEOF \u003e/tmp/50-libvirt-$STACK_USER.rules"}],"source_content_type":"text/x-sh","patch_set":1,"id":"AAAAM3%2F%2Fd3Q%3D","line":669,"updated":"2013-01-21 14:18:49.000000000","message":"Is it a question of \"fedora\u003d\u003d18\" or \"fedora\u003e\u003d18\" or \"policykit\u003e\u003dX\"\n\ni.e. surely newer versions of the distro have the same problem? How about other distros that use newer PolicyKit?","commit_id":"193a57b97aae9007cc1b2d7bcc94365635ebd15d"},{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"dccf5feb1a74c6fdb24923be92657e6d1a02ed16","unresolved":false,"context_lines":[{"line_number":666,"context_line":""},{"line_number":667,"context_line":"# On fedora 18 enable stack-user to virsh -c qemu:///system"},{"line_number":668,"context_line":"# by creating a policy-kit rule for stack-user"},{"line_number":669,"context_line":"if is_fedora \u0026\u0026  [[ \"$os_RELEASE\" \u003d\u003d \"18\" ]]; then"},{"line_number":670,"context_line":"    echo_summary \"Creating policy-kit rules\""},{"line_number":671,"context_line":""},{"line_number":672,"context_line":"    cat \u003c\u003cEOF \u003e/tmp/50-libvirt-$STACK_USER.rules"}],"source_content_type":"text/x-sh","patch_set":1,"id":"AAAAM3%2F%2Fd14%3D","line":669,"in_reply_to":"AAAAM3%2F%2Fd3Q%3D","updated":"2013-01-21 14:29:36.000000000","message":"I agree with checking the version of policykit but I wasn\u0027t sure about other distros, if they may chose to create a rules file upon installation of say - libvirt. What I am only sure is about F18\n\nI thought we cross the bridge when we come to it since the solution is known","commit_id":"193a57b97aae9007cc1b2d7bcc94365635ebd15d"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"ebcbfc51994be641afacbae3d6a5b54c1563d4bc","unresolved":false,"context_lines":[{"line_number":678,"context_line":"});"},{"line_number":679,"context_line":"EOF"},{"line_number":680,"context_line":"    sudo mkdir -p /etc/polkit-1/rules.d/"},{"line_number":681,"context_line":"    sudo mv -f /tmp/50-libvirt-$STACK_USER.rules /etc/polkit-1/rules.d"},{"line_number":682,"context_line":"fi"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"TRACK_DEPENDS\u003d${TRACK_DEPENDS:-False}"}],"source_content_type":"text/x-sh","patch_set":1,"id":"AAAAM3%2F%2Fd3A%3D","line":681,"updated":"2013-01-21 14:18:49.000000000","message":"any reason to not just write the file directly to the dir, avoiding /tmp?","commit_id":"193a57b97aae9007cc1b2d7bcc94365635ebd15d"},{"author":{"_account_id":6473,"name":"Sunil Thaha","email":"sthaha@redhat.com","username":"sthaha"},"change_message_id":"dccf5feb1a74c6fdb24923be92657e6d1a02ed16","unresolved":false,"context_lines":[{"line_number":678,"context_line":"});"},{"line_number":679,"context_line":"EOF"},{"line_number":680,"context_line":"    sudo mkdir -p /etc/polkit-1/rules.d/"},{"line_number":681,"context_line":"    sudo mv -f /tmp/50-libvirt-$STACK_USER.rules /etc/polkit-1/rules.d"},{"line_number":682,"context_line":"fi"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"TRACK_DEPENDS\u003d${TRACK_DEPENDS:-False}"}],"source_content_type":"text/x-sh","patch_set":1,"id":"AAAAM3%2F%2Fd2I%3D","line":681,"in_reply_to":"AAAAM3%2F%2Fd3A%3D","updated":"2013-01-21 14:29:36.000000000","message":"I wanted to avoid sudo bash -c cat \u003c\u003c \nwhich IMHO looks more complex than this solution","commit_id":"193a57b97aae9007cc1b2d7bcc94365635ebd15d"},{"author":{"_account_id":6717,"name":"Jakub Ruzicka","email":"jruzicka@redhat.com","username":"jruzicka"},"change_message_id":"5ac51fc1816b464556bebfdcff810316335844f1","unresolved":false,"context_lines":[{"line_number":678,"context_line":"});"},{"line_number":679,"context_line":"EOF"},{"line_number":680,"context_line":"    sudo mkdir -p /etc/polkit-1/rules.d/"},{"line_number":681,"context_line":"    sudo mv -f /tmp/50-libvirt-$STACK_USER.rules /etc/polkit-1/rules.d"},{"line_number":682,"context_line":"fi"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"TRACK_DEPENDS\u003d${TRACK_DEPENDS:-False}"}],"source_content_type":"text/x-sh","patch_set":1,"id":"AAAAM3%2F%2Fd18%3D","line":681,"in_reply_to":"AAAAM3%2F%2Fd3A%3D","updated":"2013-01-21 14:36:26.000000000","message":"If there is such reason, wouldn\u0027t it be better to store the filename in a variable  so there isn\u0027t a redundancy?\n\nI. e. when you edit the filename on line 672 but not on 861, it will break.","commit_id":"193a57b97aae9007cc1b2d7bcc94365635ebd15d"}]}