)]}'
{"tempest/api/baremetal/test_ports.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4d437a9006eb8c49cc6c0c58e3f3d3a5d8dd319f","unresolved":false,"context_lines":[{"line_number":180,"context_line":"                                    port[\u0027links\u0027][0][\u0027href\u0027])"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    def test_list_ports_details_with_adress(self):"},{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_770c5d43","line":183,"updated":"2014-05-09 09:41:22.000000000","message":"typo -\ntest_list_ports_details_with_adress \u003e test_list_ports_details_with_address","commit_id":"22eb85c5ce35cc80a2751203cfaac5c5ca12f149"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"55df25c62c20f6d89f050b848c15240c7f269da9","unresolved":false,"context_lines":[{"line_number":180,"context_line":"                                    port[\u0027links\u0027][0][\u0027href\u0027])"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"    def test_list_ports_details_with_adress(self):"},{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_5a74e898","line":183,"in_reply_to":"7adec928_770c5d43","updated":"2014-05-09 09:54:11.000000000","message":"Wao! Thank you! I\u0027ll modify it.","commit_id":"22eb85c5ce35cc80a2751203cfaac5c5ca12f149"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"9a4c74411dcff291f0d3891dd812abc64d5a93a5","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    def test_list_ports_details_with_address(self):"},{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"},{"line_number":187,"context_line":"            self.create_port(node_id\u003dnode_id,"},{"line_number":188,"context_line":"                             address\u003ddata_utils.rand_mac_address())"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_8ba9abe1","line":185,"updated":"2014-06-02 09:53:27.000000000","message":"Why can\u0027t this line be included in the for loop ?","commit_id":"3083fcad701aff89c36d93f3c45003f4bb4624cf"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"17d671ed7f1caa832152a5659bfd64a535b16024","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    def test_list_ports_details_with_address(self):"},{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"},{"line_number":187,"context_line":"            self.create_port(node_id\u003dnode_id,"},{"line_number":188,"context_line":"                             address\u003ddata_utils.rand_mac_address())"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_112b8f82","line":185,"in_reply_to":"1ae5cdf2_7f2415af","updated":"2014-06-20 04:47:43.000000000","message":"I guess this separated create_port is for specifying particular address to use the following list_ports_detail() later.\n\nThe ports which are created in the loop are dummy, and this test checks it is possible to the particular port from many ports.\n\nSo current implementation seems not so bad.","commit_id":"3083fcad701aff89c36d93f3c45003f4bb4624cf"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"48488a9103fbb002b28a428e8c821c8d50b92f28","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    def test_list_ports_details_with_address(self):"},{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"},{"line_number":187,"context_line":"            self.create_port(node_id\u003dnode_id,"},{"line_number":188,"context_line":"                             address\u003ddata_utils.rand_mac_address())"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_7f2415af","line":185,"in_reply_to":"1ae5cdf2_8ba9abe1","updated":"2014-06-02 11:47:04.000000000","message":"Yes, Jordan is right. I think you can remove L184-185.","commit_id":"3083fcad701aff89c36d93f3c45003f4bb4624cf"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"09dd2bacc2e58ba5c57b094eb05de977a192dc54","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"},{"line_number":187,"context_line":"            self.create_port(node_id\u003dnode_id,"},{"line_number":188,"context_line":"                             address\u003ddata_utils.rand_mac_address())"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_09d2f15b","line":186,"updated":"2014-05-13 10:43:05.000000000","message":"Why do you create 5 ports? I think that 2 ones will be enough.","commit_id":"3083fcad701aff89c36d93f3c45003f4bb4624cf"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"b822bcf9914ae4305ea6fae7bb6c24c0d2424995","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"},{"line_number":187,"context_line":"            self.create_port(node_id\u003dnode_id,"},{"line_number":188,"context_line":"                             address\u003ddata_utils.rand_mac_address())"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3ae8d1ca_264657c0","line":186,"in_reply_to":"3ae8d1ca_31d05253","updated":"2014-05-27 08:32:52.000000000","message":"OK. Let\u0027s leave 5 ports for consistency.","commit_id":"3083fcad701aff89c36d93f3c45003f4bb4624cf"},{"author":{"_account_id":7882,"name":"Yuiko Takada Mori","email":"yuiko-mori@nec.com","username":"YuikoTakada"},"change_message_id":"e13c43cd3ed7dd6885e3d7c64d8a44ac76e517cb","unresolved":false,"context_lines":[{"line_number":183,"context_line":"        node_id \u003d self.node[\u0027uuid\u0027]"},{"line_number":184,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":185,"context_line":"        self.create_port(node_id\u003dnode_id, address\u003daddress)"},{"line_number":186,"context_line":"        for i in range(0, 5):"},{"line_number":187,"context_line":"            self.create_port(node_id\u003dnode_id,"},{"line_number":188,"context_line":"                             address\u003ddata_utils.rand_mac_address())"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3ae8d1ca_31d05253","line":186,"in_reply_to":"7adec928_09d2f15b","updated":"2014-05-27 06:38:10.000000000","message":"Just because test_list_ports_details(L157-180) creates 5 ports. Of course there is no problem creating just 2 ports.","commit_id":"3083fcad701aff89c36d93f3c45003f4bb4624cf"}]}