)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e4b01699e1313f0ef56d3f6688f4438906d8b4e3","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adapt unit tests to keystoneauth 5.9.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since keystoneauth 5.0.0, LegacyJSONAdapter no longer inherits"},{"line_number":10,"context_line":"the base Adapter class. Update mocks according to that change."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I8495db0ca59681bb7888bb8b7b353f8b16aa42fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"c3764e7d_b7d28ee3","line":9,"range":{"start_line":9,"start_character":19,"end_line":9,"end_character":24},"updated":"2025-01-13 14:58:08.000000000","message":"Correct me if I\u0027m wrong, but I think this happens in 5.9.0 with change https://review.opendev.org/#/q/I4512aa19fd30 ?","commit_id":"898b5cecbfaa7cafe2c93619adc4cd71cb520699"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4e65f9ca1bf0753ad60edc013b6af47d48b85fd5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adapt unit tests to keystoneauth 5.9.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since keystoneauth 5.0.0, LegacyJSONAdapter no longer inherits"},{"line_number":10,"context_line":"the base Adapter class. Update mocks according to that change."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I8495db0ca59681bb7888bb8b7b353f8b16aa42fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3409e03f_e134e97d","line":9,"range":{"start_line":9,"start_character":19,"end_line":9,"end_character":24},"in_reply_to":"a1379a59_226c6e22","updated":"2025-01-13 15:34:46.000000000","message":"Done","commit_id":"898b5cecbfaa7cafe2c93619adc4cd71cb520699"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"04d99893d1b5b50524606d87b76473a83e3d8186","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adapt unit tests to keystoneauth 5.9.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since keystoneauth 5.0.0, LegacyJSONAdapter no longer inherits"},{"line_number":10,"context_line":"the base Adapter class. Update mocks according to that change."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I8495db0ca59681bb7888bb8b7b353f8b16aa42fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a1379a59_226c6e22","line":9,"range":{"start_line":9,"start_character":19,"end_line":9,"end_character":24},"in_reply_to":"c3764e7d_b7d28ee3","updated":"2025-01-13 15:31:07.000000000","message":"Yes you are correct. Let me fix the comment.","commit_id":"898b5cecbfaa7cafe2c93619adc4cd71cb520699"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e4b01699e1313f0ef56d3f6688f4438906d8b4e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e54d9c79_2a60c05b","updated":"2025-01-13 14:58:08.000000000","message":"Thanks for posting this Takashi.  Some questions inline.","commit_id":"898b5cecbfaa7cafe2c93619adc4cd71cb520699"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c4bcd53b63a394bd2b3b09d43eb15051c98950bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0ccdb7af_dcd4d879","updated":"2025-01-13 20:36:33.000000000","message":"LGTM.","commit_id":"a0aaed8c010bda0aff485bcd375ec811ef76f772"}],"requirements.txt":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e4b01699e1313f0ef56d3f6688f4438906d8b4e3","unresolved":true,"context_lines":[{"line_number":3,"context_line":"# you find any incorrect lower bounds, let us know or propose a fix."},{"line_number":4,"context_line":"pbr\u003e\u003d5.5.0 # Apache-2.0"},{"line_number":5,"context_line":"PrettyTable\u003e\u003d0.7.2 # BSD"},{"line_number":6,"context_line":"keystoneauth1\u003e\u003d5.0.0 # Apache-2.0"},{"line_number":7,"context_line":"oslo.i18n\u003e\u003d5.0.1 # Apache-2.0"},{"line_number":8,"context_line":"oslo.utils\u003e\u003d4.8.0 # Apache-2.0"},{"line_number":9,"context_line":"requests\u003e\u003d2.25.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"695ab2b0_874de9c3","line":6,"range":{"start_line":6,"start_character":15,"end_line":6,"end_character":20},"updated":"2025-01-13 14:58:08.000000000","message":"I think this should be bumped to 5.9.0?  If I\u0027m reading git history correctly, the change in base class for LegacyJsonAdapter is currently only in 5.9.0+. so won\u0027t your patch fail if tested with 5.0.0?","commit_id":"898b5cecbfaa7cafe2c93619adc4cd71cb520699"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"04d99893d1b5b50524606d87b76473a83e3d8186","unresolved":true,"context_lines":[{"line_number":3,"context_line":"# you find any incorrect lower bounds, let us know or propose a fix."},{"line_number":4,"context_line":"pbr\u003e\u003d5.5.0 # Apache-2.0"},{"line_number":5,"context_line":"PrettyTable\u003e\u003d0.7.2 # BSD"},{"line_number":6,"context_line":"keystoneauth1\u003e\u003d5.0.0 # Apache-2.0"},{"line_number":7,"context_line":"oslo.i18n\u003e\u003d5.0.1 # Apache-2.0"},{"line_number":8,"context_line":"oslo.utils\u003e\u003d4.8.0 # Apache-2.0"},{"line_number":9,"context_line":"requests\u003e\u003d2.25.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"8e1a0e2d_ae23efc6","line":6,"range":{"start_line":6,"start_character":15,"end_line":6,"end_character":20},"in_reply_to":"695ab2b0_874de9c3","updated":"2025-01-13 15:31:07.000000000","message":"I didn\u0027t bump the min because the requirement is specific to unit tests and is not relevant to actual functionality, but as we have no good mechanism to separately handle requirements for runtime and tests we should probably bump this here.","commit_id":"898b5cecbfaa7cafe2c93619adc4cd71cb520699"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4e65f9ca1bf0753ad60edc013b6af47d48b85fd5","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# you find any incorrect lower bounds, let us know or propose a fix."},{"line_number":4,"context_line":"pbr\u003e\u003d5.5.0 # Apache-2.0"},{"line_number":5,"context_line":"PrettyTable\u003e\u003d0.7.2 # BSD"},{"line_number":6,"context_line":"keystoneauth1\u003e\u003d5.0.0 # Apache-2.0"},{"line_number":7,"context_line":"oslo.i18n\u003e\u003d5.0.1 # Apache-2.0"},{"line_number":8,"context_line":"oslo.utils\u003e\u003d4.8.0 # Apache-2.0"},{"line_number":9,"context_line":"requests\u003e\u003d2.25.1 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"e9fb16b8_e9b46e98","line":6,"range":{"start_line":6,"start_character":15,"end_line":6,"end_character":20},"in_reply_to":"8e1a0e2d_ae23efc6","updated":"2025-01-13 15:34:46.000000000","message":"Done","commit_id":"898b5cecbfaa7cafe2c93619adc4cd71cb520699"}]}