)]}'
{"/PATCHSET_LEVEL":[{"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":"4ae6f7cdd57c28b2291d58a436bacb7ab25459d2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d9db6991_54d206de","updated":"2025-02-13 19:26:44.000000000","message":"This needs a release note.","commit_id":"4c6df7fe2168b35b960fe5f5239e8866425a0ebc"},{"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":"2e9f1453fed3ac96214015c644258861a692e77e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e96964e7_b9f22563","updated":"2025-02-13 19:35:57.000000000","message":"https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/agent_client.py#L282\n\nThe conductor is not currently sending an agent token to get command results; that\u0027s why CI is failing.","commit_id":"4c6df7fe2168b35b960fe5f5239e8866425a0ebc"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"182890f18addf7fbed9ec8c2ba4a347fc8af0810","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aefbe393_4d3a06c5","in_reply_to":"e96964e7_b9f22563","updated":"2025-02-13 19:47:56.000000000","message":"I was just thinking about this as well. We need to teach ironic to send it as well.","commit_id":"4c6df7fe2168b35b960fe5f5239e8866425a0ebc"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"2d2e2b523d86247f7b5c6e48d8204b79a5b996a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"25f507d4_7b7405eb","updated":"2025-02-14 14:19:54.000000000","message":"Looks good, but we\u0027ll want Ironic\u0027s patch to merge first.","commit_id":"6f860995c64d5143d8684a175e3b043bc3bf1f54"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"7c8ac6da3d9b38f3fce014fbcb4921383657887a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"eac6a671_c6b16b2d","updated":"2025-02-14 17:00:17.000000000","message":"recheck hopefully CI is happier today","commit_id":"6f860995c64d5143d8684a175e3b043bc3bf1f54"}],"ironic_python_agent/api/app.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"1b5c210bd276e37b5e4fd661dae7832fc2d3ff36","unresolved":true,"context_lines":[{"line_number":207,"context_line":"            status \u003d self.agent.get_status()"},{"line_number":208,"context_line":"            return jsonify(status)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    @require_agent_token"},{"line_number":211,"context_line":"    def api_list_commands(self, request):"},{"line_number":212,"context_line":"        with metrics_utils.get_metrics_logger(__name__).timer(\u0027list_commands\u0027):"},{"line_number":213,"context_line":"            results \u003d self.agent.list_command_results()"}],"source_content_type":"text/x-python","patch_set":4,"id":"679123ec_d581b401","line":210,"updated":"2025-02-14 14:19:36.000000000","message":"Is there a corresponding ironic change for this?\n\n...\n\nI\u0027m wondering if this is orphaned code at the moment.\n\nSo, this *does* appear to be orphaned code. I think we should remove it in a separate patch.","commit_id":"6f860995c64d5143d8684a175e3b043bc3bf1f54"},{"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":"feff96d1a44029c0968cab3787fdfd64f24ec3d7","unresolved":true,"context_lines":[{"line_number":207,"context_line":"            status \u003d self.agent.get_status()"},{"line_number":208,"context_line":"            return jsonify(status)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    @require_agent_token"},{"line_number":211,"context_line":"    def api_list_commands(self, request):"},{"line_number":212,"context_line":"        with metrics_utils.get_metrics_logger(__name__).timer(\u0027list_commands\u0027):"},{"line_number":213,"context_line":"            results \u003d self.agent.list_command_results()"}],"source_content_type":"text/x-python","patch_set":4,"id":"02ba948c_d03dd70c","line":210,"in_reply_to":"679123ec_d581b401","updated":"2025-02-14 21:06:14.000000000","message":"Makes sense. Was something we used very early on in teeth-agent days for testing. I thought we had gotten all that out :D","commit_id":"6f860995c64d5143d8684a175e3b043bc3bf1f54"}],"releasenotes/notes/prevent-restart-in-rescue-state-1c5d2ecf174ece63.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"7de52d218b434c389e0a2f719d16389b8d50b2f1","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    In rescued state, prevent IPA from restarting to avoid exposing its API"},{"line_number":5,"context_line":"    on tenant networks."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"6cfcae2c_98fb0fb7","line":5,"updated":"2025-02-17 13:35:27.000000000","message":"Is this release note from a different patch? What I\u0027d really like to see here is a huge upgrade warning that we\u0027re breaking compatibility with all unpatched versions of Ironic.. which we arguably should not have done.","commit_id":"6f860995c64d5143d8684a175e3b043bc3bf1f54"}]}