)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"8609162f3a0224f6352f6a925ce1abf165184561","unresolved":true,"context_lines":[{"line_number":13,"context_line":" - T117 test is now disabled. There are a lot of lines violating"},{"line_number":14,"context_line":"   this rule and we have to decide if we really want to enforce it."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":" - We probably have to update bump hacking in plugins n case these"},{"line_number":17,"context_line":"   imports hacking extensions from tempest."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I5ee5e152418079f9f2720eb97c3a5361edba2695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7b7180f6_ab5aa59f","line":16,"range":{"start_line":16,"start_character":54,"end_line":16,"end_character":55},"updated":"2024-02-08 21:04:08.000000000","message":"nit: in","commit_id":"aba30bd37d8da8d4a4b74e3713fceba87406b57d"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"949fae13e7f42792265bb0aa878d1e038c1c0a52","unresolved":false,"context_lines":[{"line_number":13,"context_line":" - T117 test is now disabled. There are a lot of lines violating"},{"line_number":14,"context_line":"   this rule and we have to decide if we really want to enforce it."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":" - We probably have to update bump hacking in plugins n case these"},{"line_number":17,"context_line":"   imports hacking extensions from tempest."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I5ee5e152418079f9f2720eb97c3a5361edba2695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"49df588a_1c1781ec","line":16,"range":{"start_line":16,"start_character":54,"end_line":16,"end_character":55},"in_reply_to":"7b7180f6_ab5aa59f","updated":"2024-02-09 03:56:59.000000000","message":"Done","commit_id":"aba30bd37d8da8d4a4b74e3713fceba87406b57d"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"8609162f3a0224f6352f6a925ce1abf165184561","unresolved":true,"context_lines":[{"line_number":13,"context_line":" - T117 test is now disabled. There are a lot of lines violating"},{"line_number":14,"context_line":"   this rule and we have to decide if we really want to enforce it."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":" - We probably have to update bump hacking in plugins n case these"},{"line_number":17,"context_line":"   imports hacking extensions from tempest."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I5ee5e152418079f9f2720eb97c3a5361edba2695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"8a3b47c4_fbd02614","line":17,"range":{"start_line":16,"start_character":61,"end_line":17,"end_character":10},"updated":"2024-02-08 21:04:08.000000000","message":"they import?","commit_id":"aba30bd37d8da8d4a4b74e3713fceba87406b57d"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"949fae13e7f42792265bb0aa878d1e038c1c0a52","unresolved":false,"context_lines":[{"line_number":13,"context_line":" - T117 test is now disabled. There are a lot of lines violating"},{"line_number":14,"context_line":"   this rule and we have to decide if we really want to enforce it."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":" - We probably have to update bump hacking in plugins n case these"},{"line_number":17,"context_line":"   imports hacking extensions from tempest."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I5ee5e152418079f9f2720eb97c3a5361edba2695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"aeb9dc53_398c2b2d","line":17,"range":{"start_line":16,"start_character":61,"end_line":17,"end_character":10},"in_reply_to":"8a3b47c4_fbd02614","updated":"2024-02-09 03:56:59.000000000","message":"Done","commit_id":"aba30bd37d8da8d4a4b74e3713fceba87406b57d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"42d0ad6b5f03dd758b15ec2befbd668c7ecda225","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"741e9151_189f66de","updated":"2024-05-03 11:39:00.000000000","message":"recheck 3 months since the last run","commit_id":"0e10aa9b2cfaf7b63815dcf81a31a513ff0aef7f"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"9d8db2e9fb7580fc5590d41f5a3dd3e6c8cd2c43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"90bf7a60_90058dfe","updated":"2024-02-14 14:16:59.000000000","message":"recheck see the latest ci results","commit_id":"0e10aa9b2cfaf7b63815dcf81a31a513ff0aef7f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"67f3f8d34d03522a0e48312936359de53eca7496","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"58ba1476_543efe30","updated":"2024-11-12 08:35:23.000000000","message":"I\u0027m still struggling to fix the unit test failure. The current test implementation does not really work well with the latest hacking code but I\u0027ve not yet identified the actual change causing the problem.","commit_id":"fdd866462ae342a1b608101b3878856928c01bbc"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1dc6d667bbf241ec237d4e8781213f0404393045","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"794ff105_a52852ec","updated":"2024-11-11 01:20:31.000000000","message":"recheck refresh the test results as logs not present\n\noverall lgtm","commit_id":"fdd866462ae342a1b608101b3878856928c01bbc"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5f945fb87f32e8c15919327ecc716537c33a8d2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6224e9f4_e7088106","updated":"2024-11-11 18:05:00.000000000","message":"seems like unit test failure are valid","commit_id":"fdd866462ae342a1b608101b3878856928c01bbc"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"294cfcb4e09da656ce8959eaaac5bc4a8e75e91b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a80d4742_b2fb98ca","updated":"2024-11-14 05:52:33.000000000","message":"unit tests are passing now. As I changed the tests, I am leaving the approval for Martin.","commit_id":"ab57c03c62c6477cc1880a45ff73e12dab87f7e4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5b7b8024e665847b57441635062a69328784ea09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"b04555dc_e2b461a5","updated":"2024-11-16 02:36:37.000000000","message":"recheck plugin sanity job is non voting now","commit_id":"397f49c2eed9b96a2c3c662154f5b89b53ea20af"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"181e696759b79cdda92c3df03a2d372cc314795b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"bf6d321f_a68c3b69","updated":"2024-11-15 13:13:11.000000000","message":"recheck plugin-sanity check failed (unrelated to this change), let\u0027s see if it\u0027s consistent","commit_id":"397f49c2eed9b96a2c3c662154f5b89b53ea20af"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"006643f09611e08a3b13a68f5b17cbd6649be69d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"119458bc_31faaf2a","updated":"2024-11-15 00:43:28.000000000","message":"recheck timeout in multinode job","commit_id":"397f49c2eed9b96a2c3c662154f5b89b53ea20af"}],"requirements.txt":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"8609162f3a0224f6352f6a925ce1abf165184561","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"pbr!\u003d2.1.0,\u003e\u003d2.0.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"eeb2e7cc_21bf5f9c","side":"PARENT","line":1,"updated":"2024-02-08 21:04:08.000000000","message":"what\u0027s the reason for this change?","commit_id":"1da7a33099e0a06e4cca3b4a8e6c9d7874426df6"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"949fae13e7f42792265bb0aa878d1e038c1c0a52","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"pbr!\u003d2.1.0,\u003e\u003d2.0.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"cc708b26_c0922cc3","side":"PARENT","line":1,"in_reply_to":"eeb2e7cc_21bf5f9c","updated":"2024-02-09 03:56:59.000000000","message":"I\u0027ve added a note to mention this. This behavior was fixed in pip and this description is no longer valid for recent pip.","commit_id":"1da7a33099e0a06e4cca3b4a8e6c9d7874426df6"}],"tempest/common/utils/net_downtime.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"4a3282b47e56cddcecb4ecf1a708b660d73653f3","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        cmd.append(\u0027-i{}\u0027.format(self.interval))"},{"line_number":65,"context_line":"        cmd.append(self.dest_ip)"},{"line_number":66,"context_line":"        LOG.debug(\"Starting background pinger to \u0027%s\u0027 with interval %d\","},{"line_number":67,"context_line":"                  self.dest_ip, self.interval)"},{"line_number":68,"context_line":"        self.ping_process \u003d subprocess.Popen("},{"line_number":69,"context_line":"            cmd, stdout\u003dsubprocess.PIPE, stderr\u003dsubprocess.PIPE)"},{"line_number":70,"context_line":"        self.addCleanup(self.cleanup)"}],"source_content_type":"text/x-python","patch_set":8,"id":"1231742f_bdce4cc0","line":67,"range":{"start_line":67,"start_character":0,"end_line":67,"end_character":46},"updated":"2024-11-14 08:39:26.000000000","message":"TypeError: not all arguments converted during string formatting\n\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_556/906634/8/check/tempest-slow-py3/5567447/testr_results.html\n\nhttps://dd43e0f9601da5e2e650-51b18fcc89837fbadd0245724df9c686.ssl.cf1.rackcdn.com/906634/8/check/tempest-multinode-full-py3/541ba0e/testr_results.html\n\nmaybe we could use %s to simply print the var as string?","commit_id":"ab57c03c62c6477cc1880a45ff73e12dab87f7e4"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b26a442bf2af2799d7ab182be3938c08318a8748","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        cmd.append(\u0027-i{}\u0027.format(self.interval))"},{"line_number":65,"context_line":"        cmd.append(self.dest_ip)"},{"line_number":66,"context_line":"        LOG.debug(\"Starting background pinger to \u0027%s\u0027 with interval %d\","},{"line_number":67,"context_line":"                  self.dest_ip, self.interval)"},{"line_number":68,"context_line":"        self.ping_process \u003d subprocess.Popen("},{"line_number":69,"context_line":"            cmd, stdout\u003dsubprocess.PIPE, stderr\u003dsubprocess.PIPE)"},{"line_number":70,"context_line":"        self.addCleanup(self.cleanup)"}],"source_content_type":"text/x-python","patch_set":8,"id":"ce2dcf24_ca45d8df","line":67,"range":{"start_line":67,"start_character":0,"end_line":67,"end_character":46},"in_reply_to":"1231742f_bdce4cc0","updated":"2024-11-14 08:57:02.000000000","message":"Ugh. I didn\u0027t expect that the interval can be a string...","commit_id":"ab57c03c62c6477cc1880a45ff73e12dab87f7e4"}],"tempest/hacking/checks.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5e3d87915de009b1e2a5616282f6c01b7a12acdc","unresolved":false,"context_lines":[{"line_number":310,"context_line":""},{"line_number":311,"context_line":"        if TEST_DEFINITION.match(logical_line):"},{"line_number":312,"context_line":"            if not _HAVE_NEGATIVE_DECORATOR:"},{"line_number":313,"context_line":"                yield ("},{"line_number":314,"context_line":"                    0, \"T117: Must apply `@decorators.attr(type\u003d[\u0027negative\u0027])`\""},{"line_number":315,"context_line":"                       \" to all negative API tests\""},{"line_number":316,"context_line":"                )"}],"source_content_type":"text/x-python","patch_set":7,"id":"3a88e171_d355d39f","line":313,"range":{"start_line":313,"start_character":15,"end_line":313,"end_character":22},"updated":"2024-11-14 05:42:46.000000000","message":"this change cause the failure and now this function will return the its object instead of simple return value.","commit_id":"69071ceece5411850e60b6f2e5976f902b6a1c95"}],"tempest/tests/test_hacking.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5e3d87915de009b1e2a5616282f6c01b7a12acdc","unresolved":false,"context_lines":[{"line_number":207,"context_line":"            # arbitrarily many decorators. These insert decorators above the"},{"line_number":208,"context_line":"            # @decorators.attr(type\u003d[\u0027negative\u0027]) decorator."},{"line_number":209,"context_line":"            for decorator in other_decorators:"},{"line_number":210,"context_line":"                self.assertIsNone(check(\" %s\" % decorator, filename))"},{"line_number":211,"context_line":"        if with_negative_decorator:"},{"line_number":212,"context_line":"            self.assertIsNone("},{"line_number":213,"context_line":"                check(\"@decorators.attr(type\u003d[\u0027negative\u0027])\", filename))"}],"source_content_type":"text/x-python","patch_set":7,"id":"eebbc769_0417cfeb","line":210,"range":{"start_line":210,"start_character":16,"end_line":210,"end_character":69},"updated":"2024-11-14 05:42:46.000000000","message":"here you need to change to convert the check function (this is yield generator function) to list and then check the value.\n\nlet me fix these.","commit_id":"69071ceece5411850e60b6f2e5976f902b6a1c95"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d6b861cfcf300a2055e93283477e59fa7457fea5","unresolved":false,"context_lines":[{"line_number":207,"context_line":"            # arbitrarily many decorators. These insert decorators above the"},{"line_number":208,"context_line":"            # @decorators.attr(type\u003d[\u0027negative\u0027]) decorator."},{"line_number":209,"context_line":"            for decorator in other_decorators:"},{"line_number":210,"context_line":"                self.assertIsNone(check(\" %s\" % decorator, filename))"},{"line_number":211,"context_line":"        if with_negative_decorator:"},{"line_number":212,"context_line":"            self.assertIsNone("},{"line_number":213,"context_line":"                check(\"@decorators.attr(type\u003d[\u0027negative\u0027])\", filename))"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f9ccd52_826a454b","line":210,"range":{"start_line":210,"start_character":16,"end_line":210,"end_character":69},"in_reply_to":"eebbc769_0417cfeb","updated":"2024-11-14 08:49:54.000000000","message":"Ahh ok. That makes sense.","commit_id":"69071ceece5411850e60b6f2e5976f902b6a1c95"}]}