)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Failed to discover the volumes on ppc64le - Part 2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Code refactoring for Volume discovery on ppcle systems."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"In continuation with review"},{"line_number":12,"context_line":"https://review.opendev.org/c/openstack/os-brick/+/737446"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"9e81a0d3_4f6e3e37","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":55},"updated":"2021-01-13 17:59:16.000000000","message":"-1: This goes beyond code refactoring, this seems to change how the code behaves.  This needs a better explanation on what this tries to fix (since the argument issue is fixed in the other patch) and how does it fix it.","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[{"line_number":11,"context_line":"In continuation with review"},{"line_number":12,"context_line":"https://review.opendev.org/c/openstack/os-brick/+/737446"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1884707"},{"line_number":15,"context_line":"Change-Id: I3e8188e596ca8711d231f9f44b0de3ec00a30e36"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"e4ba2c9e_ed80493e","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":20},"updated":"2021-01-13 17:59:16.000000000","message":"-1: Patch https://review.opendev.org/c/openstack/os-brick/+/737446 already closes this bug.","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"da0b49bef7c3e80f11252f3300ca88d9b988ca66","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Code refactoring for Volume discovery on ppcle systems."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"In continuation with review"},{"line_number":12,"context_line":"https://review.opendev.org/c/openstack/os-brick/+/737446"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1884707"},{"line_number":15,"context_line":"Change-Id: I3e8188e596ca8711d231f9f44b0de3ec00a30e36"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"ab370c62_12681cdb","line":12,"updated":"2021-06-03 23:21:21.000000000","message":"Should this be:\n`Depends-On: https://review...` ?","commit_id":"ad9030f4a204748b048262196f643d07321375df"}],"doc/requirements.txt":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[{"line_number":4,"context_line":"openstackdocstheme\u003e\u003d2.2.1 # Apache-2.0"},{"line_number":5,"context_line":"reno\u003e\u003d3.1.0 # Apache-2.0"},{"line_number":6,"context_line":"sphinx\u003e\u003d2.0.0,!\u003d2.1.0 # BSD"},{"line_number":7,"context_line":"os-api-ref\u003e\u003d1.5.0 # Apache-2.0"},{"line_number":8,"context_line":"sphinxcontrib-apidoc\u003e\u003d0.2.0  # BSD"},{"line_number":9,"context_line":"sphinx-feature-classification\u003e\u003d1.1.0 # Apache 2.0"}],"source_content_type":"text/plain","patch_set":12,"id":"a9a71e79_1d117389","line":9,"range":{"start_line":7,"start_character":0,"end_line":9,"end_character":49},"updated":"2021-01-13 17:59:16.000000000","message":"-1: Unrelated changes","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"}],"lower-constraints.txt":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"b49d6ccb_12a0f02f","line":75,"range":{"start_line":1,"start_character":0,"end_line":75,"end_character":0},"updated":"2021-01-13 17:59:16.000000000","message":"-1: Unrelated changes","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"}],"os_brick/initiator/connectors/fibre_channel_ppc64.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[{"line_number":90,"context_line":"            realdev \u003d \u0027/dev/\u0027 + devname"},{"line_number":91,"context_line":"            try:"},{"line_number":92,"context_line":"                (out, _err) \u003d self._linuxscsi._execute("},{"line_number":93,"context_line":"                    \u0027/usr/lib/udev/scsi_id\u0027, \u0027--page\u003d0x83\u0027,"},{"line_number":94,"context_line":"                    \u0027--whitelisted\u0027, \u0027--device\u003d%s\u0027 % realdev,"},{"line_number":95,"context_line":"                    run_as_root\u003dTrue, attempts\u003d2, timeout\u003d100, interval\u003d10,"},{"line_number":96,"context_line":"                    root_helper\u003dself._linuxscsi._root_helper)"}],"source_content_type":"text/x-python","patch_set":12,"id":"8d04ef53_7f96af77","line":93,"range":{"start_line":93,"start_character":0,"end_line":93,"end_character":59},"updated":"2021-01-13 17:59:16.000000000","message":"-1: We don\u0027t want to add new code that uses scsi_id to query volumes, as the call sometimes hangs forever.  Existing code that uses it is part of a technical debt we don\u0027t want to increase.  We should be using sysfs to get that information.","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d4bd31298b21d91b5df0cadc9f4515bbbbbdb09d","unresolved":true,"context_lines":[{"line_number":90,"context_line":"            realdev \u003d \u0027/dev/\u0027 + devname"},{"line_number":91,"context_line":"            try:"},{"line_number":92,"context_line":"                (out, _err) \u003d self._linuxscsi._execute("},{"line_number":93,"context_line":"                    \u0027/usr/lib/udev/scsi_id\u0027, \u0027--page\u003d0x83\u0027,"},{"line_number":94,"context_line":"                    \u0027--whitelisted\u0027, \u0027--device\u003d%s\u0027 % realdev,"},{"line_number":95,"context_line":"                    run_as_root\u003dTrue, attempts\u003d2, timeout\u003d100, interval\u003d10,"},{"line_number":96,"context_line":"                    root_helper\u003dself._linuxscsi._root_helper)"}],"source_content_type":"text/x-python","patch_set":13,"id":"75ec4ce0_8be06ad7","line":93,"updated":"2021-06-07 13:59:21.000000000","message":"-1: We don\u0027t want to add new code that uses scsi_id to query volumes, as the call sometimes hangs forever.  Existing code that uses it is part of a technical debt we don\u0027t want to increase.  We should be using sysfs to get that information.","commit_id":"ad9030f4a204748b048262196f643d07321375df"}],"releasenotes/notes/bug_1884707_failure_to_discover_ppcle_volumes_2.yaml":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"dfbd7c85_5c91eb62","line":7,"range":{"start_line":2,"start_character":0,"end_line":7,"end_character":0},"updated":"2021-01-13 17:59:16.000000000","message":"-1: The bug was fixed already on the other patch","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"}],"requirements.txt":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[{"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":""},{"line_number":5,"context_line":"pbr\u003e\u003d5.5.0 # Apache-2.0"},{"line_number":6,"context_line":"eventlet\u003e\u003d0.26.1 # MIT"},{"line_number":7,"context_line":"oslo.concurrency\u003e\u003d4.3.0 # Apache-2.0"},{"line_number":8,"context_line":"oslo.context\u003e\u003d3.1.1 # Apache-2.0"},{"line_number":9,"context_line":"oslo.log\u003e\u003d4.4.0 # Apache-2.0"},{"line_number":10,"context_line":"oslo.i18n\u003e\u003d5.0.1 # Apache-2.0"},{"line_number":11,"context_line":"oslo.privsep\u003e\u003d2.4.0 # Apache-2.0"},{"line_number":12,"context_line":"oslo.serialization\u003e\u003d4.0.1 # Apache-2.0"},{"line_number":13,"context_line":"oslo.service\u003e\u003d2.4.0 # Apache-2.0"},{"line_number":14,"context_line":"oslo.utils\u003e\u003d4.7.0 # Apache-2.0"},{"line_number":15,"context_line":"requests\u003e\u003d2.23.0 # Apache-2.0"},{"line_number":16,"context_line":"six\u003e\u003d1.15.0 # MIT"},{"line_number":17,"context_line":"tenacity\u003e\u003d6.2.0 # Apache-2.0"},{"line_number":18,"context_line":"os-win\u003e\u003d5.1.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":12,"id":"56114045_7bb1d6b4","line":18,"range":{"start_line":5,"start_character":0,"end_line":18,"end_character":26},"updated":"2021-01-13 17:59:16.000000000","message":"-1: Unrelated changes","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"}],"test-requirements.txt":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"361a68f438c95e2712d7557b92e20453592704a7","unresolved":true,"context_lines":[{"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":""},{"line_number":5,"context_line":"hacking\u003e\u003d3.1.0,\u003c3.2.0 # Apache-2.0"},{"line_number":6,"context_line":"coverage\u003e\u003d5.2.1 # Apache-2.0"},{"line_number":7,"context_line":"ddt\u003e\u003d1.4.1 # MIT"},{"line_number":8,"context_line":"oslotest\u003e\u003d4.4.1 # Apache-2.0"},{"line_number":9,"context_line":"testscenarios\u003e\u003d0.5.0 # Apache-2.0/BSD"},{"line_number":10,"context_line":"testtools\u003e\u003d2.4.0 # MIT"},{"line_number":11,"context_line":"stestr\u003e\u003d3.0.1 # Apache-2.0"},{"line_number":12,"context_line":"oslo.vmware\u003e\u003d3.7.0 # Apache-2.0"},{"line_number":13,"context_line":"castellan\u003e\u003d3.6.0 # Apache-2.0"},{"line_number":14,"context_line":"pycodestyle\u003d\u003d2.6.0 # MIT"},{"line_number":15,"context_line":"doc8\u003e\u003d0.8.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":12,"id":"20b36737_c75a39e5","line":15,"range":{"start_line":5,"start_character":0,"end_line":15,"end_character":24},"updated":"2021-01-13 17:59:16.000000000","message":"-1: Unrelated changes","commit_id":"55a512fbeef333baf93750e1eb6a220bb6d3f0ec"}]}