)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f31dfd38e59f8cddd3eb9a4a825f76c2efbf2256","unresolved":true,"context_lines":[{"line_number":12,"context_line":"this we\u0027ll make our ironic tests compatible with vscode (and I presume"},{"line_number":13,"context_line":"any other unittest-discover based IDE testing setups)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Generated-By: Cursor"},{"line_number":16,"context_line":"Change-Id: I6c9b92e6b0bee366ff40795c722bd70d16cf0e4f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"35e1e654_50204dd0","line":15,"updated":"2025-04-01 15:17:24.000000000","message":"Is this an actual model, or is it that you did it by CLI?!? Sooo confused!","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"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":"d3249cedb7685cb2d200e7d1a76e8272ad9e45b0","unresolved":true,"context_lines":[{"line_number":12,"context_line":"this we\u0027ll make our ironic tests compatible with vscode (and I presume"},{"line_number":13,"context_line":"any other unittest-discover based IDE testing setups)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Generated-By: Cursor"},{"line_number":16,"context_line":"Change-Id: I6c9b92e6b0bee366ff40795c722bd70d16cf0e4f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"42425531_7113b100","line":15,"in_reply_to":"35e1e654_50204dd0","updated":"2025-04-01 19:06:16.000000000","message":"cursor.com -- an AI agent+IDE combo. I\u0027m not sure I know exactly what specific model was used; it looks like (based solely on asking in the same chat) it\u0027s Claude 3.5 Sonnet; but I\u0027d prefer being able to indicate \"cursor\" or something similar so I don\u0027t have to track what specific model is being used for any given thing.","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"419c1734f7b0233ac98f62b6f68375339645d3df","unresolved":true,"context_lines":[{"line_number":12,"context_line":"this we\u0027ll make our ironic tests compatible with vscode (and I presume"},{"line_number":13,"context_line":"any other unittest-discover based IDE testing setups)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Generated-By: Cursor"},{"line_number":16,"context_line":"Change-Id: I6c9b92e6b0bee366ff40795c722bd70d16cf0e4f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a4d3862e_bae0ea28","line":15,"in_reply_to":"42425531_7113b100","updated":"2025-04-01 23:20:14.000000000","message":"Yeah, Cursor is more or less a wrapper around both ChatGPT and Claude; the model (and version of such model) is selectable manually, or it happens automatically. So, just Cursor is probably fine.","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"419c1734f7b0233ac98f62b6f68375339645d3df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"564718a1_645ee53c","updated":"2025-04-01 23:20:14.000000000","message":"Will a change like this need to be backported?","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"43f6059b765a62312237d6c4a2de7919989fb760","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2a86132c_acd6adf4","updated":"2025-04-01 22:34:27.000000000","message":"two-ish minor revisions, but otherwise good.","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"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":"a07068000024021d9fbd3c097c7f8ba8c463046d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"17078340_0b2ea54a","in_reply_to":"564718a1_645ee53c","updated":"2025-04-01 23:24:03.000000000","message":"https://docs.openstack.org/project-team-guide/stable-branches.html#review-guidelines\n\nIMO, this should not be backported as it is a refactor, and not a bugfix. Although I agree one *could* consider us shadowing a python module name a backport-worthy bug, I don\u0027t think it\u0027s a good idea.","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"8d2bc53e315ac447b3accb24e8b10eb4342f9686","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"555705d8_0d65a78b","updated":"2025-04-02 15:56:58.000000000","message":"Thank you for cleaning up the doc changes by your IDE, and the increased verbosity of the commit message.","commit_id":"3e0f6ed2404e8e48bea77c1dbc0f84ddd1841c2c"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"222a874df2076aa9d28c194caecdff62606ece1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"94edbbe7_7fd688fe","updated":"2025-04-02 10:00:31.000000000","message":"recheck - oomanyrequests: You have reached your unauthenticated pull rate limit.","commit_id":"3e0f6ed2404e8e48bea77c1dbc0f84ddd1841c2c"}],"doc/source/contributor/rolling-upgrades.rst":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"43f6059b765a62312237d6c4a2de7919989fb760","unresolved":true,"context_lines":[{"line_number":374,"context_line":"  ``ironic/common/release_mappings.py``."},{"line_number":375,"context_line":"- New objects must be added to ``ironic/common/release_mappings.py``. Also for"},{"line_number":376,"context_line":"  the first releases they should be excluded from the version check by adding"},{"line_number":377,"context_line":"  their class names to the ``NEW_MODELS`` list in ``ironic.command.dbsync.NEW_MODELS``."},{"line_number":378,"context_line":"- The arguments of remotable methods (methods which are remoted to the"},{"line_number":379,"context_line":"  conductor via RPC) can only be added as optional. They cannot be removed or"},{"line_number":380,"context_line":"  changed in an incompatible way (to the previous release)."}],"source_content_type":"text/x-rst","patch_set":2,"id":"0b02db76_d88ba9e1","line":377,"range":{"start_line":377,"start_character":27,"end_line":377,"end_character":87},"updated":"2025-04-01 22:34:27.000000000","message":"So the list is nested in the list?","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"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":"12cbc90ad55fd9058c0b718d29ab60bb4fe124a2","unresolved":true,"context_lines":[{"line_number":374,"context_line":"  ``ironic/common/release_mappings.py``."},{"line_number":375,"context_line":"- New objects must be added to ``ironic/common/release_mappings.py``. Also for"},{"line_number":376,"context_line":"  the first releases they should be excluded from the version check by adding"},{"line_number":377,"context_line":"  their class names to the ``NEW_MODELS`` list in ``ironic.command.dbsync.NEW_MODELS``."},{"line_number":378,"context_line":"- The arguments of remotable methods (methods which are remoted to the"},{"line_number":379,"context_line":"  conductor via RPC) can only be added as optional. They cannot be removed or"},{"line_number":380,"context_line":"  changed in an incompatible way (to the previous release)."}],"source_content_type":"text/x-rst","patch_set":2,"id":"5525af67_0d9484a0","line":377,"range":{"start_line":377,"start_character":27,"end_line":377,"end_character":87},"in_reply_to":"0b02db76_d88ba9e1","updated":"2025-04-01 23:22:29.000000000","message":"heh, fair, not the best wording, will fix","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"43f6059b765a62312237d6c4a2de7919989fb760","unresolved":true,"context_lines":[{"line_number":479,"context_line":"objects."},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"The method name should be added to the list of ``ONLINE_MIGRATIONS`` in"},{"line_number":482,"context_line":"``ironic.command.dbsync.NEW_MODELS``."},{"line_number":483,"context_line":""},{"line_number":484,"context_line":"The method should be removed in the next named release after this one."},{"line_number":485,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"74bcc6e6_b543695f","line":482,"updated":"2025-04-01 22:34:27.000000000","message":"This seems wrong to me. I guess it took the idea from earlier context and moved it forward to apply elsewhere.","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"},{"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":"12cbc90ad55fd9058c0b718d29ab60bb4fe124a2","unresolved":true,"context_lines":[{"line_number":479,"context_line":"objects."},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"The method name should be added to the list of ``ONLINE_MIGRATIONS`` in"},{"line_number":482,"context_line":"``ironic.command.dbsync.NEW_MODELS``."},{"line_number":483,"context_line":""},{"line_number":484,"context_line":"The method should be removed in the next named release after this one."},{"line_number":485,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"454a15b3_7827f119","line":482,"in_reply_to":"74bcc6e6_b543695f","updated":"2025-04-01 23:22:29.000000000","message":"yeah, I clearly didn\u0027t review the docs change well enough, fixing","commit_id":"d14ca110cf826b79d97e02c2f9599c25d16a7849"}]}