)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"f8106f5b875e419a9aca12a8bfed8e772cc51f11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0190c6c0_7b6e0a3d","updated":"2025-03-11 19:16:25.000000000","message":"That\u0027s a great improvement for mistral config, thanks!\nOne comment about the option to be visible in doc / config ref","commit_id":"35c38d2b8936558086098444e72d3af9a4ac7f70"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"6c415914ee7b365bc2c9e897488f053f561579a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0752d8ac_f63f5be4","updated":"2025-04-23 21:03:27.000000000","message":"I reviewed again and that sounds good, sorry for my late review","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"797494e71935a22439df03ea742c8e48969d6bfa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fc8aec31_34b68d81","updated":"2025-03-12 09:37:29.000000000","message":"That sounds good to me.\nAs we are in feature freeze, do you mind if we wait for the 2025.1 branch to exist before merging?","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"21eb363be4e3ffb3edb56a2e38756656c8b2dbb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0d0058ab_6566e1e4","in_reply_to":"24ec6f31_c563da16","updated":"2025-03-17 17:57:23.000000000","message":"yes i believe it would be a nice addition","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"f6b60d155c5cb7d5a10f6837361e300664644e37","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"24ec6f31_c563da16","in_reply_to":"fc8aec31_34b68d81","updated":"2025-03-12 12:47:17.000000000","message":"Not a problem, would you deem this (and the other patches) eligible for backport to stable branches? Since this doesn\u0027t change the default/original behaviour.","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"}],"mistral/actions/legacy.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"63656982d002f5fc700405fd7450fb247246ccef","unresolved":true,"context_lines":[{"line_number":122,"context_line":"                namespace\u003d\u0027\u0027"},{"line_number":123,"context_line":"            )"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"            self._action_descs[action_name] \u003d action_desc"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"            LOG.debug(\u0027Registered action: %s\u0027, action_desc)"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"b9e7db13_82b37770","line":125,"updated":"2025-03-26 22:16:12.000000000","message":"we are going to miss adding the action in self._action_descs if it\u0027s allowed","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"4fd2792cb25113cf3daaa7e45f07810ee0c9d099","unresolved":true,"context_lines":[{"line_number":122,"context_line":"                namespace\u003d\u0027\u0027"},{"line_number":123,"context_line":"            )"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"            self._action_descs[action_name] \u003d action_desc"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"            LOG.debug(\u0027Registered action: %s\u0027, action_desc)"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f577c1cc_06324202","line":125,"in_reply_to":"b9e7db13_82b37770","updated":"2025-03-31 12:25:23.000000000","message":"Can you elaborate what you mean I don\u0027t understand","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"6c415914ee7b365bc2c9e897488f053f561579a4","unresolved":false,"context_lines":[{"line_number":122,"context_line":"                namespace\u003d\u0027\u0027"},{"line_number":123,"context_line":"            )"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"            self._action_descs[action_name] \u003d action_desc"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"            LOG.debug(\u0027Registered action: %s\u0027, action_desc)"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"0e0d7530_74bfbeb1","line":125,"in_reply_to":"f577c1cc_06324202","updated":"2025-04-23 21:03:27.000000000","message":"Done","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"63656982d002f5fc700405fd7450fb247246ccef","unresolved":true,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"            self._action_descs[action_name] \u003d action_desc"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"            LOG.debug(\u0027Registered action: %s\u0027, action_desc)"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    def _load_action_generators(self):"},{"line_number":130,"context_line":"        if not CONF.legacy_action_provider.load_action_generators:"}],"source_content_type":"text/x-python","patch_set":3,"id":"323a63c4_dd6fa815","line":127,"updated":"2025-03-26 22:16:12.000000000","message":"I believe we are going to miss this debug line if some actions are in allowlist?","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"6c415914ee7b365bc2c9e897488f053f561579a4","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"            self._action_descs[action_name] \u003d action_desc"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"            LOG.debug(\u0027Registered action: %s\u0027, action_desc)"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    def _load_action_generators(self):"},{"line_number":130,"context_line":"        if not CONF.legacy_action_provider.load_action_generators:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f3616be_fa81e0dd","line":127,"in_reply_to":"323a63c4_dd6fa815","updated":"2025-04-23 21:03:27.000000000","message":"Done","commit_id":"11785d1e31611ebf0b1f11fcb10fcd4dea3efe9e"}],"mistral/config.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"f8106f5b875e419a9aca12a8bfed8e772cc51f11","unresolved":true,"context_lines":[{"line_number":884,"context_line":"]"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"def list_opts():"},{"line_number":888,"context_line":"    return ["},{"line_number":889,"context_line":"        (API_GROUP, api_opts),"},{"line_number":890,"context_line":"        (ENGINE_GROUP, engine_opts),"}],"source_content_type":"text/x-python","patch_set":2,"id":"e9e8759c_8c568c43","line":887,"updated":"2025-03-11 19:16:25.000000000","message":"We are missing both the legacy_provider and provider options from here.\nThis prevent the documentation to be built correctly.\nWould you mind adding this? At the for the new group, we may want to create an extra patchset for the legacy one","commit_id":"35c38d2b8936558086098444e72d3af9a4ac7f70"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"a6840095e80dab4ed9d878ef7c2c111513dc510a","unresolved":true,"context_lines":[{"line_number":884,"context_line":"]"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"def list_opts():"},{"line_number":888,"context_line":"    return ["},{"line_number":889,"context_line":"        (API_GROUP, api_opts),"},{"line_number":890,"context_line":"        (ENGINE_GROUP, engine_opts),"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf796974_aa961d8c","line":887,"in_reply_to":"e9e8759c_8c568c43","updated":"2025-03-12 07:38:17.000000000","message":"Thanks! I missed that. I\u0027ve updated the patch and also pushed a separate patch to fix the same for legacy provider config opts.","commit_id":"35c38d2b8936558086098444e72d3af9a4ac7f70"}]}