)]}'
{"neutron/tests/functional/test_server.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"7324c3796add715c40ae524b32bee4c965aa43f9","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        # Wait for temp file to be created and its size reaching the expected"},{"line_number":159,"context_line":"        # value"},{"line_number":160,"context_line":"        expected_size \u003d len(expected_msg)"},{"line_number":161,"context_line":"        def is_temp_file_ok():"},{"line_number":162,"context_line":"            LOG.debug(\"Checking file %s\", self.temp_file)"},{"line_number":163,"context_line":"            if not os.path.isfile(self.temp_file):"},{"line_number":164,"context_line":"                LOG.debug(\"File %s not exists.\", self.temp_file)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_57a271fd","line":161,"updated":"2020-07-09 21:00:24.000000000","message":"pep8: E306 expected 1 blank line before a nested definition, found 0","commit_id":"a73abe2654b2e6ee84c3ba51d796c6106cdd9e99"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a62dca52bfe9f635f9e30de3ac6f77b4ad8f9ab5","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        # Wait for temp file to be created and its size reaching the expected"},{"line_number":159,"context_line":"        # value"},{"line_number":160,"context_line":"        expected_size \u003d len(expected_msg)"},{"line_number":161,"context_line":"        def is_temp_file_ok():"},{"line_number":162,"context_line":"            LOG.debug(\"Checking file %s\", self.temp_file)"},{"line_number":163,"context_line":"            if not os.path.isfile(self.temp_file):"},{"line_number":164,"context_line":"                LOG.debug(\"File %s not exists.\", self.temp_file)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_06ea890d","line":161,"in_reply_to":"bf51134e_57a271fd","updated":"2020-07-10 08:07:05.000000000","message":"Done","commit_id":"a73abe2654b2e6ee84c3ba51d796c6106cdd9e99"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"bde633b7d6962d43b24d8eb51b37eaf1f0adf403","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            if not os.path.isfile(self.temp_file):"},{"line_number":164,"context_line":"                LOG.debug(\"File %s not exists.\", self.temp_file)"},{"line_number":165,"context_line":"                return False"},{"line_number":166,"context_line":"            LOG.debug(\"File %s exists. Checking its size now.\", self.temp_file)"},{"line_number":167,"context_line":"            temp_file_size \u003d os.stat(self.temp_file).st_size"},{"line_number":168,"context_line":"            if temp_file_size !\u003d expected_size:"},{"line_number":169,"context_line":"                LOG.debug(\"Size of file %s is %s. Expected size: %s\","}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_291fab5b","line":166,"range":{"start_line":166,"start_character":12,"end_line":166,"end_character":79},"updated":"2020-07-09 15:19:50.000000000","message":"nit: seems redundant, LOG at #169 implies that file exists.","commit_id":"a73abe2654b2e6ee84c3ba51d796c6106cdd9e99"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a62dca52bfe9f635f9e30de3ac6f77b4ad8f9ab5","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            if not os.path.isfile(self.temp_file):"},{"line_number":164,"context_line":"                LOG.debug(\"File %s not exists.\", self.temp_file)"},{"line_number":165,"context_line":"                return False"},{"line_number":166,"context_line":"            LOG.debug(\"File %s exists. Checking its size now.\", self.temp_file)"},{"line_number":167,"context_line":"            temp_file_size \u003d os.stat(self.temp_file).st_size"},{"line_number":168,"context_line":"            if temp_file_size !\u003d expected_size:"},{"line_number":169,"context_line":"                LOG.debug(\"Size of file %s is %s. Expected size: %s\","}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_86ddf971","line":166,"range":{"start_line":166,"start_character":12,"end_line":166,"end_character":79},"in_reply_to":"bf51134e_291fab5b","updated":"2020-07-10 08:07:05.000000000","message":"Done","commit_id":"a73abe2654b2e6ee84c3ba51d796c6106cdd9e99"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"bde633b7d6962d43b24d8eb51b37eaf1f0adf403","unresolved":false,"context_lines":[{"line_number":169,"context_line":"                LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":170,"context_line":"                          self.temp_file, temp_file_size, expected_size)"},{"line_number":171,"context_line":"                return False"},{"line_number":172,"context_line":"            LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":173,"context_line":"                      self.temp_file, temp_file_size, expected_size)"},{"line_number":174,"context_line":"            return True"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_092467ae","line":173,"range":{"start_line":172,"start_character":12,"end_line":173,"end_character":68},"updated":"2020-07-09 15:19:50.000000000","message":"No need to duplicate, could be logged once before line #168","commit_id":"a73abe2654b2e6ee84c3ba51d796c6106cdd9e99"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a62dca52bfe9f635f9e30de3ac6f77b4ad8f9ab5","unresolved":false,"context_lines":[{"line_number":169,"context_line":"                LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":170,"context_line":"                          self.temp_file, temp_file_size, expected_size)"},{"line_number":171,"context_line":"                return False"},{"line_number":172,"context_line":"            LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":173,"context_line":"                      self.temp_file, temp_file_size, expected_size)"},{"line_number":174,"context_line":"            return True"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_26f50dee","line":173,"range":{"start_line":172,"start_character":12,"end_line":173,"end_character":68},"in_reply_to":"bf51134e_092467ae","updated":"2020-07-10 08:07:05.000000000","message":"Done","commit_id":"a73abe2654b2e6ee84c3ba51d796c6106cdd9e99"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0da3350a534424b8ffd19963878583a8e02dab2b","unresolved":false,"context_lines":[{"line_number":167,"context_line":"            temp_file_size \u003d os.stat(self.temp_file).st_size"},{"line_number":168,"context_line":"            LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":169,"context_line":"                      self.temp_file, temp_file_size, expected_size)"},{"line_number":170,"context_line":"            return temp_file_size !\u003d expected_size:"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        try:"},{"line_number":173,"context_line":"            utils.wait_until_true(is_temp_file_ok, timeout\u003d5, sleep\u003d1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_472bdce9","line":170,"updated":"2020-07-10 10:10:40.000000000","message":"pep8: E999 SyntaxError: invalid syntax","commit_id":"c4680365504274ebd528771775b278e814e7aac6"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"0da3350a534424b8ffd19963878583a8e02dab2b","unresolved":false,"context_lines":[{"line_number":169,"context_line":"                      self.temp_file, temp_file_size, expected_size)"},{"line_number":170,"context_line":"            return temp_file_size !\u003d expected_size:"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        try:"},{"line_number":173,"context_line":"            utils.wait_until_true(is_temp_file_ok, timeout\u003d5, sleep\u003d1)"},{"line_number":174,"context_line":"        except utils.TimerTimeout:"},{"line_number":175,"context_line":"            if not os.path.isfile(self.temp_file):"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_27306856","line":172,"updated":"2020-07-10 10:10:40.000000000","message":"pep8: E112 expected an indented block","commit_id":"c4680365504274ebd528771775b278e814e7aac6"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"bc1d568a22ccf80a4843c3cab9fd2c279cd846a7","unresolved":false,"context_lines":[{"line_number":167,"context_line":"            temp_file_size \u003d os.stat(self.temp_file).st_size"},{"line_number":168,"context_line":"            LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":169,"context_line":"                      self.temp_file, temp_file_size, expected_size)"},{"line_number":170,"context_line":"            return temp_file_size !\u003d expected_size"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        try:"},{"line_number":173,"context_line":"            utils.wait_until_true(is_temp_file_ok, timeout\u003d5, sleep\u003d1)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_b51389b8","line":170,"range":{"start_line":170,"start_character":34,"end_line":170,"end_character":36},"updated":"2020-07-27 14:56:22.000000000","message":"seems should be \u0027\u003d\u003d\u0027","commit_id":"a284d48993baa1ee1141c4e896d3b14d151d499b"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b295f290cd95d833d70f54c4f492e76db650cb3b","unresolved":false,"context_lines":[{"line_number":167,"context_line":"            temp_file_size \u003d os.stat(self.temp_file).st_size"},{"line_number":168,"context_line":"            LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":169,"context_line":"                      self.temp_file, temp_file_size, expected_size)"},{"line_number":170,"context_line":"            return temp_file_size !\u003d expected_size"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        try:"},{"line_number":173,"context_line":"            utils.wait_until_true(is_temp_file_ok, timeout\u003d5, sleep\u003d1)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_e6d0b9cc","line":170,"range":{"start_line":170,"start_character":34,"end_line":170,"end_character":36},"in_reply_to":"9f560f44_b51389b8","updated":"2020-07-27 17:40:39.000000000","message":"Agree","commit_id":"a284d48993baa1ee1141c4e896d3b14d151d499b"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"51927a3ca0577f6463029a741fe2ef09ab5d50d5","unresolved":false,"context_lines":[{"line_number":167,"context_line":"            temp_file_size \u003d os.stat(self.temp_file).st_size"},{"line_number":168,"context_line":"            LOG.debug(\"Size of file %s is %s. Expected size: %s\","},{"line_number":169,"context_line":"                      self.temp_file, temp_file_size, expected_size)"},{"line_number":170,"context_line":"            return temp_file_size !\u003d expected_size"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"        try:"},{"line_number":173,"context_line":"            utils.wait_until_true(is_temp_file_ok, timeout\u003d5, sleep\u003d1)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_df3b93c5","line":170,"range":{"start_line":170,"start_character":34,"end_line":170,"end_character":36},"in_reply_to":"9f560f44_e6d0b9cc","updated":"2020-07-29 09:39:41.000000000","message":"hmm, You are right.\nBut the condition works as expected as I see:\nhttps://1db75cf0b12d832dc11e-0da77774ba1e870a646f19d85e727a87.ssl.cf5.rackcdn.com/740283/3/check/neutron-functional/752ff6c/controller/logs/dsvm-functional-logs/neutron.tests.functional.test_server.TestPluginWorker.test_start.txt\n\nthe actual log line:\nSize of file /tmp/tmpyjf5521v/tmpdit4ury3/test_server.tmp is 5. Expected size: 15 is_temp_file_ok","commit_id":"a284d48993baa1ee1141c4e896d3b14d151d499b"}]}