)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"40e442f96d5ef14bfc5a4d28845dfece4ca98ad1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vasyl Saienko \u003cvsaienko@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-18 17:42:34 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Unblock broken gates"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Neutron no longer provide access to private network."},{"line_number":10,"context_line":"We need to add route manually to have ability to contact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a77a97e_91cd6068","line":7,"updated":"2016-11-18 15:45:36.000000000","message":"This should probably actually describe what\u0027s being fixed rather than just saying it\u0027s unblocking broken gates (i.e. this commit message needs to make sense out of context).","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"041c8acea66dde460bb5e53498acdced08b773e2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vasyl Saienko \u003cvsaienko@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-18 17:42:34 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Unblock broken gates"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Neutron no longer provide access to private network."},{"line_number":10,"context_line":"We need to add route manually to have ability to contact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a77a97e_4634e9fa","line":7,"in_reply_to":"7a77a97e_91cd6068","updated":"2016-11-18 16:36:59.000000000","message":"Done","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"40e442f96d5ef14bfc5a4d28845dfece4ca98ad1","unresolved":false,"context_lines":[{"line_number":10,"context_line":"We need to add route manually to have ability to contact"},{"line_number":11,"context_line":"with VM during provisioning."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"https://github.com/openstack-dev/devstack/commit/df5e69114fa2af663c50e734b4a842c727e6a7e9"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes-Bug: #1642986"},{"line_number":16,"context_line":"Change-Id: If281a1d6bc0fdaf48c4cfb8d923f7b1761f8162c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a77a97e_51e668e8","line":13,"updated":"2016-11-18 15:45:36.000000000","message":"Usually GitHub links to OpenStack stuff is :(. This should maybe be a git.openstack.org link?","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"041c8acea66dde460bb5e53498acdced08b773e2","unresolved":false,"context_lines":[{"line_number":10,"context_line":"We need to add route manually to have ability to contact"},{"line_number":11,"context_line":"with VM during provisioning."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"https://github.com/openstack-dev/devstack/commit/df5e69114fa2af663c50e734b4a842c727e6a7e9"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes-Bug: #1642986"},{"line_number":16,"context_line":"Change-Id: If281a1d6bc0fdaf48c4cfb8d923f7b1761f8162c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a77a97e_6669e5f8","line":13,"in_reply_to":"7a77a97e_51e668e8","updated":"2016-11-18 16:36:59.000000000","message":"Done","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"}],"devstack/lib/ironic":[{"author":{"_account_id":18653,"name":"Milan Kováčik","email":"mkovacik@redhat.com","username":"dparalen"},"change_message_id":"d3ddfbb1c1d79032e109a9613e9a615bf118749f","unresolved":false,"context_lines":[{"line_number":991,"context_line":"    # NOTE(vsaienko) Neutron no longer setup routing to private netwrok."},{"line_number":992,"context_line":"    # https://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df4017"},{"line_number":993,"context_line":"    # Add route here to have connection to VMs during provisioning."},{"line_number":994,"context_line":"    ROUTER_GW_IP\u003d$(openstack --os-cloud devstack-admin --os-region \"$REGION_NAME\" port list -c \u0027Fixed IP Addresses\u0027 --device-owner network:router_gateway | awk -F\u0027ip_address\u0027  \u0027{ print $2 }\u0027 | cut -f2 -d\\\u0027 | tr \u0027\\n\u0027 \u0027 \u0027)"},{"line_number":995,"context_line":"    local replace_range\u003d${SUBNETPOOL_PREFIX_V4}"},{"line_number":996,"context_line":"    if [[ -z \"${SUBNETPOOL_V4_ID}\" ]]; then"},{"line_number":997,"context_line":"        replace_range\u003d${FIXED_RANGE}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7a77a97e_b624120d","line":994,"updated":"2016-11-18 15:25:30.000000000","message":"just wondering this check is missing[1] from the original devstack thing\n\n[1] https://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df4017#diff-553937b39616c8a41282299a514dfdfcL373","commit_id":"2f7c9805d63b58812e304ccab608e708d32e591f"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"fa05cdc86b6d9c083dbd07f578122da4779edd36","unresolved":false,"context_lines":[{"line_number":988,"context_line":"    # to serve TFTP to a single network namespace via the tap device created above."},{"line_number":989,"context_line":"    openstack network set $ironic_net_id --share"},{"line_number":990,"context_line":""},{"line_number":991,"context_line":"    # NOTE(vsaienko) Neutron no longer setup routing to private netwrok."},{"line_number":992,"context_line":"    # https://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df4017"},{"line_number":993,"context_line":"    # Add route here to have connection to VMs during provisioning."},{"line_number":994,"context_line":"    local pub_router_id"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7a77a97e_86aa8183","line":991,"range":{"start_line":991,"start_character":64,"end_line":991,"end_character":71},"updated":"2016-11-18 16:23:50.000000000","message":"network","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"041c8acea66dde460bb5e53498acdced08b773e2","unresolved":false,"context_lines":[{"line_number":988,"context_line":"    # to serve TFTP to a single network namespace via the tap device created above."},{"line_number":989,"context_line":"    openstack network set $ironic_net_id --share"},{"line_number":990,"context_line":""},{"line_number":991,"context_line":"    # NOTE(vsaienko) Neutron no longer setup routing to private netwrok."},{"line_number":992,"context_line":"    # https://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df4017"},{"line_number":993,"context_line":"    # Add route here to have connection to VMs during provisioning."},{"line_number":994,"context_line":"    local pub_router_id"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7a77a97e_b916c249","line":991,"range":{"start_line":991,"start_character":64,"end_line":991,"end_character":71},"in_reply_to":"7a77a97e_86aa8183","updated":"2016-11-18 16:36:59.000000000","message":"Done","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"3ce1b0eaad65910be9f1fd7defc9ac4e876eda78","unresolved":false,"context_lines":[{"line_number":994,"context_line":"    local pub_router_id"},{"line_number":995,"context_line":"    local r_net_gateway"},{"line_number":996,"context_line":"    pub_router_id\u003d$(openstack router show $Q_ROUTER_NAME -f value -c id)"},{"line_number":997,"context_line":"    r_net_gateway\u003d$(sudo ip netns exec qrouter-$pub_router_id ip -4 route get 8.8.8.8 |grep dev | awk \u0027{print $7}\u0027)"},{"line_number":998,"context_line":"    local replace_range\u003d${SUBNETPOOL_PREFIX_V4}"},{"line_number":999,"context_line":"    if [[ -z \"${SUBNETPOOL_V4_ID}\" ]]; then"},{"line_number":1000,"context_line":"        replace_range\u003d${FIXED_RANGE}"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7a77a97e_51fd886c","line":997,"updated":"2016-11-18 15:46:48.000000000","message":"This seems like it wouldn\u0027t work on multi-node devstack, but I\u0027m not sure we care about that here.","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"041c8acea66dde460bb5e53498acdced08b773e2","unresolved":false,"context_lines":[{"line_number":994,"context_line":"    local pub_router_id"},{"line_number":995,"context_line":"    local r_net_gateway"},{"line_number":996,"context_line":"    pub_router_id\u003d$(openstack router show $Q_ROUTER_NAME -f value -c id)"},{"line_number":997,"context_line":"    r_net_gateway\u003d$(sudo ip netns exec qrouter-$pub_router_id ip -4 route get 8.8.8.8 |grep dev | awk \u0027{print $7}\u0027)"},{"line_number":998,"context_line":"    local replace_range\u003d${SUBNETPOOL_PREFIX_V4}"},{"line_number":999,"context_line":"    if [[ -z \"${SUBNETPOOL_V4_ID}\" ]]; then"},{"line_number":1000,"context_line":"        replace_range\u003d${FIXED_RANGE}"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7a77a97e_d93ffeb1","line":997,"in_reply_to":"7a77a97e_51fd886c","updated":"2016-11-18 16:36:59.000000000","message":"Ironic devstack doesn\u0027t work with multinode setup yet, we need to merge 3 patches https://review.openstack.org/#/q/topic:bug/1622653","commit_id":"ade5e1245940d816c1c3ec290e514cf8945ecad7"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"b2d8303fc162bea362c52375070c7c74d17347cc","unresolved":false,"context_lines":[{"line_number":1045,"context_line":"        ironic_net_id\u003d$(openstack network show \"$PRIVATE_NETWORK_NAME\" -c id -f value)"},{"line_number":1046,"context_line":"        create_ovs_taps $ironic_net_id"},{"line_number":1047,"context_line":""},{"line_number":1048,"context_line":"        # NOTE(vsaienko) Neutron no longer setup routing to private network."},{"line_number":1049,"context_line":"        # https://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df4017"},{"line_number":1050,"context_line":"        # Add route here to have connection to VMs during provisioning."},{"line_number":1051,"context_line":"        local pub_router_id"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7a77a97e_d9ad3eea","line":1048,"updated":"2016-11-18 16:37:04.000000000","message":"Move logic outside of create_ovs_taps, because we calling create_ovs_taps() during grenade and Neutron variables are not visible there.","commit_id":"ff4c11e71e0174a580a164f9098421b5c45631f6"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"9adfb3110007defcc01b271730095bbe95bef58b","unresolved":false,"context_lines":[{"line_number":1046,"context_line":"        create_ovs_taps $ironic_net_id"},{"line_number":1047,"context_line":""},{"line_number":1048,"context_line":"        # NOTE(vsaienko) Neutron no longer setup routing to private network."},{"line_number":1049,"context_line":"        # https://github.com/openstack-dev/devstack/commit/1493bdeba24674f6634160d51b8081c571df4017"},{"line_number":1050,"context_line":"        # Add route here to have connection to VMs during provisioning."},{"line_number":1051,"context_line":"        local pub_router_id"},{"line_number":1052,"context_line":"        local r_net_gateway"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7a77a97e_f24dafc4","line":1049,"updated":"2016-11-18 17:44:31.000000000","message":"still github link.. anyway, can follow up.","commit_id":"ff4c11e71e0174a580a164f9098421b5c45631f6"}]}