)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"b8c4ca7e766156c1a7e9351e84ae30d4f3cdd935","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Update Ironic VM network connection"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When VM is dirrectly connected to OVS (current setup) there is no port"},{"line_number":10,"context_line":"in OVS when VM is in powered off state. Since Ironic plug port to"},{"line_number":11,"context_line":"network when VM is in powered-off state. Port should be present in OVS"},{"line_number":12,"context_line":"even when VM is powered-off."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":69,"id":"7aa08908_0392729f","line":10,"range":{"start_line":10,"start_character":52,"end_line":10,"end_character":58},"updated":"2016-06-15 23:31:44.000000000","message":"don\u0027t you mean uplug?","commit_id":"1a68f1cf17d24baef14f88d900f9b43693daf376"}],"devstack/tools/ironic/scripts/configure-vm.py":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"95bc90b4200a2d135e4b6057db12e26713014d7a","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    parser.add_argument(\u0027--libvirt-nic-driver\u0027, default\u003d\u0027virtio\u0027,"},{"line_number":72,"context_line":"                        help\u003d\u0027The libvirt network driver to use\u0027)"},{"line_number":73,"context_line":"    parser.add_argument(\u0027--bridge\u0027, default\u003d\"br-seed\","},{"line_number":74,"context_line":"                        help\u003d\u0027The linux bridge name to use for seeding \\"},{"line_number":75,"context_line":"                                the baremetal pseudo-node\\\u0027s OS image\u0027)"},{"line_number":76,"context_line":"    parser.add_argument(\u0027--console-log\u0027,"},{"line_number":77,"context_line":"                        help\u003d\u0027File to log console\u0027)"}],"source_content_type":"text/x-python","patch_set":68,"id":"9abb7d3a_09956a3f","line":74,"range":{"start_line":74,"start_character":71,"end_line":74,"end_character":72},"updated":"2016-06-03 01:26:25.000000000","message":"nit: OpenStack Style Guidelines says:\n\nIt is preferred to wrap long lines in parentheses and not a backslash for line continuation.\n\nhttp://docs.openstack.org/developer/hacking/","commit_id":"767bda535ec95966f096c7d9b8f5b4dc0ca33503"}],"devstack/tools/ironic/scripts/create-node.sh":[{"author":{"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"},"change_message_id":"0c26af132da325950e4479e6266289ae7e9c6328","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"sudo brctl addbr br-$NAME"},{"line_number":66,"context_line":"sudo ifconfig br-$NAME up"},{"line_number":67,"context_line":"sudo ovs-vsctl add-port $BRIDGE ovs-$NAME -- set Interface ovs-$NAME type\u003dinternal"},{"line_number":68,"context_line":"sudo brctl addif br-$NAME ovs-$NAME"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"if ! virsh list --all | grep -q $NAME; then"}],"source_content_type":"text/x-sh","patch_set":10,"id":"da6ed579_401c3077","line":67,"updated":"2016-01-13 00:43:34.000000000","message":"This command fails the devstack with an error that the port is already in use","commit_id":"2543205c519a19f3ee0d1752b8768424b03bfa30"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"b397b91812d2253d5f2b3e48dead0299eccf22b7","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"sudo brctl addbr br-$NAME"},{"line_number":66,"context_line":"sudo ifconfig br-$NAME up"},{"line_number":67,"context_line":"sudo ovs-vsctl add-port $BRIDGE ovs-$NAME -- set Interface ovs-$NAME type\u003dinternal"},{"line_number":68,"context_line":"sudo brctl addif br-$NAME ovs-$NAME"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"if ! virsh list --all | grep -q $NAME; then"}],"source_content_type":"text/x-sh","patch_set":10,"id":"da6ed579_689d5bbb","line":67,"in_reply_to":"da6ed579_401c3077","updated":"2016-01-13 23:24:51.000000000","message":"cleanup part was missing. I\u0027ve added it. Thank you for review.","commit_id":"2543205c519a19f3ee0d1752b8768424b03bfa30"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"b8c4ca7e766156c1a7e9351e84ae30d4f3cdd935","unresolved":false,"context_lines":[{"line_number":56,"context_line":"fi"},{"line_number":57,"context_line":"VOL_NAME\u003d\"${NAME}.${DISK_FORMAT}\""},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"# Create bridge and add VM interface to it."},{"line_number":60,"context_line":"# Additional interface will be added to this bridge and"},{"line_number":61,"context_line":"# it will be plugged to OVS."},{"line_number":62,"context_line":"# This is needed in order to have interface in OVS even"}],"source_content_type":"text/x-sh","patch_set":69,"id":"7aa08908_c3cd0ab9","line":59,"range":{"start_line":59,"start_character":20,"end_line":59,"end_character":43},"updated":"2016-06-15 23:31:44.000000000","message":"where do you add the vm interface?","commit_id":"1a68f1cf17d24baef14f88d900f9b43693daf376"}]}