)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"ac68b510f33c305ab7e2296846898bcbe95bb8fa","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Use \u0027pip_upper_constraints_file\u0027 for openstacksdk installation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use \u0027pip_upper_constraints_file\u0027 for \u0027openstacksdk\u0027 installation"},{"line_number":10,"context_line":"instead of the hard-coded value in the inventory. This is very useful"},{"line_number":11,"context_line":"to leverage the need to override the"},{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"69530f7c_57a6566c","line":13,"range":{"start_line":10,"start_character":49,"end_line":13,"end_character":38},"updated":"2025-04-23 07:17:28.000000000","message":"This part of the commit message is hard to read, maybe just drop it?","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"94cad883e23d9a0119cefcdda43fd0912869d3de","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Use \u0027pip_upper_constraints_file\u0027 for openstacksdk installation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use \u0027pip_upper_constraints_file\u0027 for \u0027openstacksdk\u0027 installation"},{"line_number":10,"context_line":"instead of the hard-coded value in the inventory. This is very useful"},{"line_number":11,"context_line":"to leverage the need to override the"},{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"f3c41546_51cda169","line":13,"range":{"start_line":10,"start_character":49,"end_line":13,"end_character":38},"in_reply_to":"69530f7c_57a6566c","updated":"2025-04-23 07:49:12.000000000","message":"sure)","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"b9ef8fef3250ac1793486e79f9e4694539b44082","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Use \u0027pip_upper_constraints_file\u0027 for openstacksdk installation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use \u0027pip_upper_constraints_file\u0027 for \u0027openstacksdk\u0027 installation"},{"line_number":10,"context_line":"instead of the hard-coded value in the inventory. This is very useful"},{"line_number":11,"context_line":"to leverage the need to override the"},{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"19c47139_8fd05d6f","line":13,"range":{"start_line":10,"start_character":49,"end_line":13,"end_character":38},"in_reply_to":"f3c41546_51cda169","updated":"2025-04-23 07:49:41.000000000","message":"Done","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"ac68b510f33c305ab7e2296846898bcbe95bb8fa","unresolved":true,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"f09c5dbd_8b2d9ee5","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"updated":"2025-04-23 07:17:28.000000000","message":"Not so trivial since it could impact people who already customise pip_upper_constraints_file. May even need an upgrade release note.","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"379038f10aee2c56e36b3c9243bf06bcb1bffeb0","unresolved":false,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d0006e47_810badf0","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"12191021_7d33ed49","updated":"2025-04-23 14:47:55.000000000","message":"good news. I\u0027m ok with stable releases only, so 2023.2 is the last. so let\u0027s merge it.","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"ad1b47cf9c12cb6d939b38f88de39efe34b9582d","unresolved":true,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa865ea8_88c7741d","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"21219756_459692c7","updated":"2025-04-23 13:58:23.000000000","message":"I\u0027m confused a bit. You agree to merge this, but not backports? or agree with backports too but there need a reno with upgrade section? why?","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"89e52b1aaad66ca138636ed93e20d6dbeed77ed5","unresolved":false,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5bec92c4_01cb8d9a","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"575b87af_0e3ec02a","updated":"2025-04-23 09:09:27.000000000","message":"Done","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"fb7f08e172ea5af3d5715203a3cbdcfc5f9239ef","unresolved":true,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"21219756_459692c7","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"5bec92c4_01cb8d9a","updated":"2025-04-23 13:20:44.000000000","message":"Agree, this is a better default. I\u0027d also prefer this to be called out in a upgrade note and probably wouldn\u0027t back port it due to the change in behavior.","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"0ab3a4e982a253c73c2a1b1e9e77d7a456e5f081","unresolved":true,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"575b87af_0e3ec02a","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"906e0020_3885eb00","updated":"2025-04-23 08:47:10.000000000","message":"At least fix the typo: it\u0027s TrivialFix, not TrifialFix.","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"94cad883e23d9a0119cefcdda43fd0912869d3de","unresolved":false,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"906e0020_3885eb00","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"f09c5dbd_8b2d9ee5","updated":"2025-04-23 07:49:12.000000000","message":"It\u0027s not, because this change is the default inventory, so even someone override these settings downstream - all would be fine.\nOn the other hand, if someone doesn\u0027t know about \u0027openstacksdk_upper_constraint_file\u0027 and override only \u0027pip_upper_constraint_file\u0027 - they can face a problems.","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"51959bc21e88867d722de7cd025ef94af0662731","unresolved":true,"context_lines":[{"line_number":12,"context_line":"\u0027openstacksdk_upper_constraint_file\u0027, which is not described anywhere"},{"line_number":13,"context_line":"in the \u0027etc/kayobe\u0027 nor documentation."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TrifialFix"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8ea60974e2ace6876514271be4156e6c889d8776"},{"line_number":18,"context_line":"Signed-off-by: Maksim Malchuk \u003cmaksim.malchuk@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"12191021_7d33ed49","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":10},"in_reply_to":"fa865ea8_88c7741d","updated":"2025-04-23 14:26:43.000000000","message":"I agree this is a good change. I could be persuaded to backport with a suitable release note and bug report. It can\u0027t go beyond 2023.2 since we used to pin to yoga deliberately in 2023.1. Which release did you want it in?","commit_id":"dcd3e4f07619ee5920ffb0b2db55bde1180891fe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"da4aac621f5614ec3cadb2f1b701eb0198cff1e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"74398931_98133a6d","updated":"2025-05-12 12:16:46.000000000","message":"@mattc@stackhpc.com, @will@stackhpc.com could you please review and merge this?","commit_id":"580437dfbf938594ef1ddd87bce5ce30cc918724"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"39f644383ab86e21edd133cea76588765ff4a64e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a1bc0cdd_d8c43486","updated":"2025-05-12 12:50:37.000000000","message":"Could you please add the upgrade release note we discussed?","commit_id":"580437dfbf938594ef1ddd87bce5ce30cc918724"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"9ebc630647318f8e2d2ba41a16909e51681fdaad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e46196ed_15e0f769","updated":"2025-05-08 20:31:37.000000000","message":"up","commit_id":"580437dfbf938594ef1ddd87bce5ce30cc918724"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"948bdbef2f6e8f006230448e6ae0f8633cfe3ab9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"da036d86_943f3edb","in_reply_to":"a1bc0cdd_d8c43486","updated":"2025-05-12 13:42:17.000000000","message":"I think we agree that at least the typo should be fixed.\nWhat do you think about including in the upgrade notes?\nIMHO, it is not necessary, since if the user overrides `openstacksdk_upper_constraints_file`, the deploy will not be affected. If it is not this change solves the issue, when only `pip_upper_constraints_file` overridden.","commit_id":"580437dfbf938594ef1ddd87bce5ce30cc918724"},{"author":{"_account_id":28048,"name":"Will Szumski","email":"will@stackhpc.com","username":"jovial"},"change_message_id":"6442253513ba959a2125663414f942d919f1dee5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e691e3a2_94c9a5a4","in_reply_to":"da036d86_943f3edb","updated":"2025-05-13 13:56:24.000000000","message":"Just flag that ``openstacksdk_upper_constraints_file`` now defaults to ``pip_upper_constraints_file``. If you were relying on them operating independently you will need to explicitly set pip_upper_constraints_file.","commit_id":"580437dfbf938594ef1ddd87bce5ce30cc918724"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"68050c460ffb8cae71cf3fced3d9da16c6bbc990","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7a7c8a3c_c1e5fe17","in_reply_to":"e691e3a2_94c9a5a4","updated":"2025-05-13 14:36:51.000000000","message":"Done","commit_id":"580437dfbf938594ef1ddd87bce5ce30cc918724"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"809791f41da2e176d5d3778b66b67ef01af5edbf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"06aa8c5c_362fe9b6","updated":"2025-05-15 09:32:53.000000000","message":"for air-gapped environments its really the fix. anyway lets merge it.","commit_id":"f31549da7dec0e457f2b622e07177341d0b81260"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"52f978e0fc10edc467e654bfb74b1cd16cdc34e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"17dd3804_31b7ea55","updated":"2025-05-14 08:57:39.000000000","message":"imo this isn\u0027t really a bugfix, just a sensible change to defaults. So just an upgrade note seem fine when backporting.","commit_id":"f31549da7dec0e457f2b622e07177341d0b81260"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"3fa29f4f4c4de82877d8e2fb57af221d70799262","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e86b1dd3_56a5d08f","updated":"2025-05-19 13:39:35.000000000","message":"@mattc@stackhpc.com please merge","commit_id":"81affd272f65624e04ffcbe812437087cb5255fb"}]}
