)]}'
{"octavia/tests/contrib/gate_hook.sh":[{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"51214202e06bc3c699fbd058a1c0243503fa49b5","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        ;;"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    \"tempest\")"},{"line_number":72,"context_line":"        # These are not needed with either v1 or v2"},{"line_number":73,"context_line":"        ENABLED_SERVICES+\u003d\"-c-api,-c-bak,-c-sch,-c-vol,-cinder,\""},{"line_number":74,"context_line":"        ENABLED_SERVICES+\u003d\"-s-account,-s-container,-s-object,-s-proxy,\""},{"line_number":75,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_24475d78","line":72,"range":{"start_line":72,"start_character":42,"end_line":72,"end_character":51},"updated":"2016-09-20 18:28:57.000000000","message":"Nit: Copy-paste error. Either we\u0027re testing the Octavia v1 API, or Neutron LBaaS v2 API here. It doesn\u0027t make sense to refer to \"v1 or v2\" here.","commit_id":"225da2eceba2f5b4aa47ca2bb7faa1ce7a9e1a6e"},{"author":{"_account_id":11302,"name":"Phillip Toohill","email":"phillip.toohill@rackspace.com","username":"ptoohill1"},"change_message_id":"aff91433a6cfe874435059261fdde20d24504565","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        if [ \"$lbaasdriver\" \u003d \"namespace\" ]; then"},{"line_number":88,"context_line":"            export DEVSTACK_LOCAL_CONFIG+\u003d\""},{"line_number":89,"context_line":"        NEUTRON_LBAAS_SERVICE_PROVIDERV2\u003dLOADBALANCERV2:Haproxy:neutron_lbaas.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default"},{"line_number":90,"context_line":"    \""},{"line_number":91,"context_line":"        fi"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_9b68e489","line":89,"range":{"start_line":89,"start_character":8,"end_line":89,"end_character":40},"updated":"2016-09-19 14:35:58.000000000","message":"We could probably pull this block out.","commit_id":"225da2eceba2f5b4aa47ca2bb7faa1ce7a9e1a6e"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"5eb7e90ffbd3359f54cb4c688781699f5776fd51","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        if [ \"$lbaasdriver\" \u003d \"namespace\" ]; then"},{"line_number":88,"context_line":"            export DEVSTACK_LOCAL_CONFIG+\u003d\""},{"line_number":89,"context_line":"        NEUTRON_LBAAS_SERVICE_PROVIDERV2\u003dLOADBALANCERV2:Haproxy:neutron_lbaas.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default"},{"line_number":90,"context_line":"    \""},{"line_number":91,"context_line":"        fi"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_ef34befc","line":89,"range":{"start_line":89,"start_character":8,"end_line":89,"end_character":40},"in_reply_to":"7a8ec9b2_9b68e489","updated":"2016-09-19 15:38:32.000000000","message":"I left it because with the merge we will likely still need to test the namespace driver.\nIf you want to re-add this later we can, but I felt we might as well leave it now.","commit_id":"225da2eceba2f5b4aa47ca2bb7faa1ce7a9e1a6e"},{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"51214202e06bc3c699fbd058a1c0243503fa49b5","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        if [ \"$lbaasdriver\" \u003d \"namespace\" ]; then"},{"line_number":88,"context_line":"            export DEVSTACK_LOCAL_CONFIG+\u003d\""},{"line_number":89,"context_line":"        NEUTRON_LBAAS_SERVICE_PROVIDERV2\u003dLOADBALANCERV2:Haproxy:neutron_lbaas.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default"},{"line_number":90,"context_line":"    \""},{"line_number":91,"context_line":"        fi"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_9f2f9e87","line":89,"range":{"start_line":89,"start_character":8,"end_line":89,"end_character":40},"in_reply_to":"7a8ec9b2_ef34befc","updated":"2016-09-20 18:28:57.000000000","message":"Hopefully such a test ends up being pretty minimal. :P","commit_id":"225da2eceba2f5b4aa47ca2bb7faa1ce7a9e1a6e"}],"octavia/tests/contrib/post_test_hook.sh":[{"author":{"_account_id":11302,"name":"Phillip Toohill","email":"phillip.toohill@rackspace.com","username":"ptoohill1"},"change_message_id":"aff91433a6cfe874435059261fdde20d24504565","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    case \"$lbaasenv\" in"},{"line_number":14,"context_line":"        minimal)"},{"line_number":15,"context_line":"            # Temporarily just do the happy path"},{"line_number":16,"context_line":"            test_subset\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_create_load_balancer(?!_) \""},{"line_number":17,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_get_load_balancer_stats(?!_) \""},{"line_number":18,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_get_load_balancer_status_tree(?!_) \""},{"line_number":19,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_listeners_non_admin.ListenersTestJSON.test_create_listener(?!_) \""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_7bb8782b","line":16,"range":{"start_line":16,"start_character":47,"end_line":16,"end_character":49},"updated":"2016-09-19 14:35:58.000000000","message":"These links are no longer valid. Octavia is testing against its v1.","commit_id":"225da2eceba2f5b4aa47ca2bb7faa1ce7a9e1a6e"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"5eb7e90ffbd3359f54cb4c688781699f5776fd51","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    case \"$lbaasenv\" in"},{"line_number":14,"context_line":"        minimal)"},{"line_number":15,"context_line":"            # Temporarily just do the happy path"},{"line_number":16,"context_line":"            test_subset\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_create_load_balancer(?!_) \""},{"line_number":17,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_get_load_balancer_stats(?!_) \""},{"line_number":18,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_get_load_balancer_status_tree(?!_) \""},{"line_number":19,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_listeners_non_admin.ListenersTestJSON.test_create_listener(?!_) \""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_8f05fa15","line":16,"range":{"start_line":16,"start_character":47,"end_line":16,"end_character":49},"in_reply_to":"7a8ec9b2_7bb8782b","updated":"2016-09-19 15:38:32.000000000","message":"Again, I left these due to the merge related needs.  We can remove them if you want.","commit_id":"225da2eceba2f5b4aa47ca2bb7faa1ce7a9e1a6e"},{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"51214202e06bc3c699fbd058a1c0243503fa49b5","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    case \"$lbaasenv\" in"},{"line_number":14,"context_line":"        minimal)"},{"line_number":15,"context_line":"            # Temporarily just do the happy path"},{"line_number":16,"context_line":"            test_subset\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_create_load_balancer(?!_) \""},{"line_number":17,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_get_load_balancer_stats(?!_) \""},{"line_number":18,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_get_load_balancer_status_tree(?!_) \""},{"line_number":19,"context_line":"            test_subset+\u003d\"octavia.tests.tempest.v2.api.test_listeners_non_admin.ListenersTestJSON.test_create_listener(?!_) \""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_3f4bea2c","line":16,"range":{"start_line":16,"start_character":47,"end_line":16,"end_character":49},"in_reply_to":"7a8ec9b2_8f05fa15","updated":"2016-09-20 18:28:57.000000000","message":"Are we simply pulling in the existing tests initially, then? (I assume we\u0027ll want to rename that path to reflect reality eventually.)","commit_id":"225da2eceba2f5b4aa47ca2bb7faa1ce7a9e1a6e"}]}