)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"eb023e785bb7fdc63dd74be918d84da5d502f507","unresolved":true,"context_lines":[{"line_number":17,"context_line":"code interaction is, but the docs also indicate to use only whole disk"},{"line_number":18,"context_line":"images in that case."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"rhbz# 1954096"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ie3bd49add9a57fabbcdcbae4b73309066b620d02"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fbd4abbb_99c37824","line":20,"updated":"2021-04-27 18:42:43.000000000","message":"I\u0027d strongly prefer we not include metadata from external systems in our commit messages.","commit_id":"cfb99061aff4e1a8c1ec25a69487ea5c38a7d9a3"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b8d9cd4af7f5c25b1ec14251cf4d798b7fc50e1a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"code interaction is, but the docs also indicate to use only whole disk"},{"line_number":18,"context_line":"images in that case."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"rhbz# 1954096"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ie3bd49add9a57fabbcdcbae4b73309066b620d02"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"31c82a9a_e804f4d2","line":20,"updated":"2021-04-30 12:58:23.000000000","message":"If it\u0027s public, let\u0027s at least use a whole link..","commit_id":"cfb99061aff4e1a8c1ec25a69487ea5c38a7d9a3"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"e61f8df18bb6349353ac2d40c9adb77e841b9b3b","unresolved":true,"context_lines":[{"line_number":17,"context_line":"code interaction is, but the docs also indicate to use only whole disk"},{"line_number":18,"context_line":"images in that case."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"rhbz# 1954096"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ie3bd49add9a57fabbcdcbae4b73309066b620d02"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b57710cc_eea4747a","line":20,"in_reply_to":"301b7873_935e8d52","updated":"2021-05-04 15:26:08.000000000","message":"I do agree with the storyboard bug, I was just slammed for time when I threw this patch up. For what it is worth, we need the bz number in order to get the patch downstream cleanly, so it is less work to have it already as it is just a cherry-pick and push as opposed to cherry pick, edit the commit to have the additional field, and then roll from there.  Its just a pita that doubles the effort to get the cherry-pick in place.","commit_id":"cfb99061aff4e1a8c1ec25a69487ea5c38a7d9a3"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"2b68b0a1301b7c15c7650b3edbc47bc3125ae234","unresolved":true,"context_lines":[{"line_number":17,"context_line":"code interaction is, but the docs also indicate to use only whole disk"},{"line_number":18,"context_line":"images in that case."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"rhbz# 1954096"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ie3bd49add9a57fabbcdcbae4b73309066b620d02"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"301b7873_935e8d52","line":20,"in_reply_to":"31c82a9a_e804f4d2","updated":"2021-04-30 15:31:11.000000000","message":"I mean, I\u0027m even -1 to that. If we need to associate it with a bug, we should use a storyboard bug (and link the RH bug there), as we generally do in Ironic. I do not think it\u0027s a good thing for the community to merge patches with metadata that points to a single OpenStack vendor bugzilla inside the commit.","commit_id":"cfb99061aff4e1a8c1ec25a69487ea5c38a7d9a3"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"e61f8df18bb6349353ac2d40c9adb77e841b9b3b","unresolved":true,"context_lines":[{"line_number":17,"context_line":"code interaction is, but the docs also indicate to use only whole disk"},{"line_number":18,"context_line":"images in that case."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"rhbz# 1954096"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Ie3bd49add9a57fabbcdcbae4b73309066b620d02"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c6cf7968_466db048","line":20,"in_reply_to":"fbd4abbb_99c37824","updated":"2021-05-04 15:26:08.000000000","message":"It actually saves us headaches to just have it. Not having it creates more work for us downstream even though it is a fair informational tag to have and is performed in other public repositories.","commit_id":"cfb99061aff4e1a8c1ec25a69487ea5c38a7d9a3"}],"ironic_python_agent/extensions/image.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b8d9cd4af7f5c25b1ec14251cf4d798b7fc50e1a","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            # partition number"},{"line_number":302,"context_line":"            partition \u003d _get_partition(device, uuid\u003defi_system_part_uuid)"},{"line_number":303,"context_line":"            try:"},{"line_number":304,"context_line":"                efi_partition \u003d int(partition.replace(device, \"\"))"},{"line_number":305,"context_line":"            except ValueError:"},{"line_number":306,"context_line":"                # NVMe Devices get a partitioning scheme that is different from"},{"line_number":307,"context_line":"                # traditional block devices like SCSI/SATA"}],"source_content_type":"text/x-python","patch_set":1,"id":"78d6c703_ce36dce7","line":304,"updated":"2021-04-30 12:58:23.000000000","message":"I wonder if it\u0027s more reliable and clear to use re.match(\u0027.*([0-9]+)$\u0027)","commit_id":"cfb99061aff4e1a8c1ec25a69487ea5c38a7d9a3"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"e61f8df18bb6349353ac2d40c9adb77e841b9b3b","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            # partition number"},{"line_number":302,"context_line":"            partition \u003d _get_partition(device, uuid\u003defi_system_part_uuid)"},{"line_number":303,"context_line":"            try:"},{"line_number":304,"context_line":"                efi_partition \u003d int(partition.replace(device, \"\"))"},{"line_number":305,"context_line":"            except ValueError:"},{"line_number":306,"context_line":"                # NVMe Devices get a partitioning scheme that is different from"},{"line_number":307,"context_line":"                # traditional block devices like SCSI/SATA"}],"source_content_type":"text/x-python","patch_set":1,"id":"9c0af0b9_b01424fd","line":304,"in_reply_to":"78d6c703_ce36dce7","updated":"2021-05-04 15:26:08.000000000","message":"two sets of numbers in the device name, so I think we would need to replace it and then match it which seems... a little weird. I dunno.","commit_id":"cfb99061aff4e1a8c1ec25a69487ea5c38a7d9a3"}]}