)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"73bffa26287916c8ba3a080b6e67ffe78b77a597","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Added neutron cli test case"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Test case added for below CLI:-"},{"line_number":10,"context_line":"neutron-net-list-on-dhcp-agent"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ibadb2c55f40e0bd7a7787f93fd7c3f8da706303e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1abeadc6_4653e4e1","line":10,"updated":"2014-08-20 13:33:31.000000000","message":"This is a quite specific test case. I am not totally convinced that this fits into a \"simple_read_only\" cli test case. Could you describe in your commit message why do you think this test case need\u0027s to get checked in CLI and not as an API check.","commit_id":"b153640116b6799e07ab94fb715137627dbe2c30"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b91b7f4963a92d194c6c92b87223fbc3af14a6bb","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:15:56 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added neutron cli test case"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Test case added for below CLI:-"},{"line_number":10,"context_line":"neutron-net-list-on-dhcp-agent"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"baa201ad_d95775c8","line":7,"updated":"2014-10-08 01:59:18.000000000","message":"Same as https://review.openstack.org/#/c/115621/2//COMMIT_MSG","commit_id":"df1fff3897fed148dcb1b0b379ad3d9d443ef906"}],"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":"b91b7f4963a92d194c6c92b87223fbc3af14a6bb","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\u0027dhcp_agent_scheduler\u0027, service\u003d\u0027network\u0027)"},{"line_number":178,"context_line":"    def test_neutron_net_list_on_dhcp_agent(self):"},{"line_number":179,"context_line":"        agent_list \u003d self.parser.listing(self.neutron(\u0027agent-list\u0027))"},{"line_number":180,"context_line":"        for agent in agent_list:"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_b93d5191","line":177,"updated":"2014-10-08 01:59:18.000000000","message":"service tag not needed anymore. https://review.openstack.org/#/c/111068/7","commit_id":"df1fff3897fed148dcb1b0b379ad3d9d443ef906"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"13cb810e6affa779db7e3a134e5b0f9cbb47391a","unresolved":false,"context_lines":[{"line_number":178,"context_line":"    def test_neutron_net_list_on_dhcp_agent(self):"},{"line_number":179,"context_line":"        agent_list \u003d self.parser.listing(self.neutron(\u0027agent-list\u0027))"},{"line_number":180,"context_line":"        for agent in agent_list:"},{"line_number":181,"context_line":"            if agent[\u0027agent_type\u0027] \u003d\u003d \"DHCP agent\":"},{"line_number":182,"context_line":"                net_list \u003d self.parser.listing(self.neutron"},{"line_number":183,"context_line":"                                               (\u0027net-list-on-dhcp-agent\u0027,"},{"line_number":184,"context_line":"                                                params\u003dagent[\u0027id\u0027]))"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa98f980_d992f59d","line":181,"updated":"2014-09-16 00:56:20.000000000","message":"If there is not \"DHCP agent\" in the list, this test passes even if loading dhcp_agent_scheduler extension.\nIs that an expected result?\n\nI feel that we need to check \"DHCP agent\" existence also in this test. Is that right?","commit_id":"df1fff3897fed148dcb1b0b379ad3d9d443ef906"}],"tempest/cli/simple_read_only/test_neutron.py":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"73bffa26287916c8ba3a080b6e67ffe78b77a597","unresolved":false,"context_lines":[{"line_number":178,"context_line":"    def test_neutron_net_list_on_dhcp_agent(self):"},{"line_number":179,"context_line":"        agent_list \u003d self.parser.listing(self.neutron(\u0027agent-list\u0027))"},{"line_number":180,"context_line":"        for agent in agent_list:"},{"line_number":181,"context_line":"            if agent[\u0027agent_type\u0027] \u003d\u003d \"DHCP agent\":"},{"line_number":182,"context_line":"                net_list \u003d self.parser.listing(self.neutron"},{"line_number":183,"context_line":"                                               (\u0027net-list-on-dhcp-agent\u0027,"},{"line_number":184,"context_line":"                                                params\u003dagent[\u0027id\u0027]))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_06b1ec46","line":181,"updated":"2014-08-20 13:33:31.000000000","message":"if you don\u0027t have any DHCP agent in the list you won\u0027t check anything. Please also assert that a agent_type of \"DHCP agent\" exist.","commit_id":"b153640116b6799e07ab94fb715137627dbe2c30"}]}