)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f9005fa4cf82e69296554cdba7d7f0f49b29abd2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Puneet arora \u003cpuneet.arora@hp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-09-14 23:03:55 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added a Neutron CLI test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Test case added for CLI:-"},{"line_number":10,"context_line":"l3-agent-list-hosting-router"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"baa201ad_5919c520","line":7,"updated":"2014-10-08 02:06:13.000000000","message":"Please be specific in title about the changes, it is same as https://review.openstack.org/#/c/116870/\nsomething like - Add neutron cli test case for `l3-agent-list-hosting-route`","commit_id":"f7451886990cfc267e671ee0e45ce4fc1d9621b7"}],"tempest/cli/simple_read_only/network/test_neutron.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f9005fa4cf82e69296554cdba7d7f0f49b29abd2","unresolved":false,"context_lines":[{"line_number":175,"context_line":"        self.assertFalse(wanted_commands - commands)"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":178,"context_line":"    @test.requires_ext(extension\u003d\u0027l3_agent_scheduler\u0027, service\u003d\u0027network\u0027)"},{"line_number":179,"context_line":"    @testtools.skipIf(not CONF.network.public_router_id,"},{"line_number":180,"context_line":"                      \"router id doesn\u0027t exists\")"},{"line_number":181,"context_line":"    def test_neutron_l3_agent_list_hosting_router(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"baa201ad_d90e95e4","line":178,"updated":"2014-10-08 02:06:13.000000000","message":"service tag not needed anymore - https://review.openstack.org/#/c/111068/","commit_id":"f7451886990cfc267e671ee0e45ce4fc1d9621b7"}],"tempest/cli/simple_read_only/test_neutron.py":[{"author":{"_account_id":6579,"name":"Nir Magnezi","email":"nmagnezi@redhat.com","username":"nmagnezi"},"change_message_id":"e87ef0fe4c4387e1c0347b5f21da10a1cf9d4271","unresolved":false,"context_lines":[{"line_number":174,"context_line":"        self.assertFalse(wanted_commands - commands)"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":177,"context_line":"    @test.requires_ext(extension\u003d\u0027l3_agent_scheduler\u0027, service\u003d\u0027network\u0027)"},{"line_number":178,"context_line":"    def test_neutron_l3_agent_list_hosting_router(self):"},{"line_number":179,"context_line":"        router_name \u003d CONF.network.public_router_id"},{"line_number":180,"context_line":"        if router_name:"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa98f980_99169c54","line":177,"updated":"2014-09-07 12:18:09.000000000","message":"what happens if public_router_id is None?\nlooking at you second patch, I think this also should require: @testtools.skipIf(not CONF.network.public_router_id, \"router id doesn\u0027t exists\")","commit_id":"83ca2db55c048608715fd593a8ecf44861e90005"}]}