)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"53fa64397d0f331d62ee36be0fac6487ed46e223","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"82b99a28_07e47a97","updated":"2024-08-01 16:23:54.000000000","message":"Please do add a release note for this one, its quite a major change in the defaults. It should probably note that its part of submitting Azimuth code into the CNCF sandbox.","commit_id":"eba5e2b38b2cfa9c49dfa8f8ec0a49d3f09f20bb"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"b598648cedf7b9b572fe259142c2195ec15f8547","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"04278a9a_5a064894","updated":"2024-10-02 14:43:38.000000000","message":"I think this looks OK apart from one small typo. Please also address Michal\u0027s comments.","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a0a9771e64f14c19a8a6bed4aca7fa80f3979955","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"439f379d_7db514ec","updated":"2024-10-01 03:48:50.000000000","message":"please raise a bug for users looking why everything stopped working and close it with this patch","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"744003c843d1b040d2c0659c30704a29ee2a70ee","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b672f4f1_c4050297","in_reply_to":"439f379d_7db514ec","updated":"2024-10-04 11:08:19.000000000","message":"I think this perhaps was a transient error as Matt mentioned. Is this still required?","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a6df77462419a9537d81f1f2be3a83779baf4f36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"42bbe4bb_797b6e18","in_reply_to":"a547998d_ac0c2492","updated":"2024-10-04 11:11:12.000000000","message":"Done","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"27955a0eb91f1c3cac8a6a8bde6a84aec411464a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a547998d_ac0c2492","in_reply_to":"b672f4f1_c4050297","updated":"2024-10-04 11:11:00.000000000","message":"nah, if it was transient then it\u0027s fine.","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"}],"doc/source/index.rst":[{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"b598648cedf7b9b572fe259142c2195ec15f8547","unresolved":true,"context_lines":[{"line_number":25,"context_line":"required to provision and manage a k8s cluster using Cluster API,"},{"line_number":26,"context_line":"including various useful add-ons like a CNI and a monitoring stack."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"These Helm charts are currently live at https://github.com/azimuth-cloud/capi-helm-charts."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The Helm charts are intended to be a way to share a reference method to"},{"line_number":31,"context_line":"create k8s clusters on OpenStack. The charts are not expected or intended to"}],"source_content_type":"text/x-rst","patch_set":3,"id":"e88a7a59_451e90e0","line":28,"updated":"2024-10-02 14:43:38.000000000","message":"Typo here - \"are\" is not required.","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"192efa210a802d36cb45393c780e88e2a500136e","unresolved":false,"context_lines":[{"line_number":25,"context_line":"required to provision and manage a k8s cluster using Cluster API,"},{"line_number":26,"context_line":"including various useful add-ons like a CNI and a monitoring stack."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"These Helm charts are currently live at https://github.com/azimuth-cloud/capi-helm-charts."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The Helm charts are intended to be a way to share a reference method to"},{"line_number":31,"context_line":"create k8s clusters on OpenStack. The charts are not expected or intended to"}],"source_content_type":"text/x-rst","patch_set":3,"id":"0c8c55c0_40f236c5","line":28,"in_reply_to":"7dedd1db_eae87c53","updated":"2024-10-04 14:50:06.000000000","message":"Done","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"744003c843d1b040d2c0659c30704a29ee2a70ee","unresolved":true,"context_lines":[{"line_number":25,"context_line":"required to provision and manage a k8s cluster using Cluster API,"},{"line_number":26,"context_line":"including various useful add-ons like a CNI and a monitoring stack."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"These Helm charts are currently live at https://github.com/azimuth-cloud/capi-helm-charts."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The Helm charts are intended to be a way to share a reference method to"},{"line_number":31,"context_line":"create k8s clusters on OpenStack. The charts are not expected or intended to"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7dedd1db_eae87c53","line":28,"in_reply_to":"e88a7a59_451e90e0","updated":"2024-10-04 11:08:19.000000000","message":"ah, I meant \u0027live\u0027 as in active. currently \u0027live\u0027 as in \u0027living\u0027 also works :)","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"}],"magnum_capi_helm/conf.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"53fa64397d0f331d62ee36be0fac6487ed46e223","unresolved":true,"context_lines":[{"line_number":58,"context_line":"    ),"},{"line_number":59,"context_line":"    cfg.StrOpt("},{"line_number":60,"context_line":"        \"default_helm_chart_version\","},{"line_number":61,"context_line":"        default\u003d\"0.4.0\","},{"line_number":62,"context_line":"        help\u003d("},{"line_number":63,"context_line":"            \"Version of the helm chart specified \""},{"line_number":64,"context_line":"            \"by the config: capi_driver.helm_chart_repo \""}],"source_content_type":"text/x-python","patch_set":1,"id":"79cde22a_9481480d","line":61,"updated":"2024-08-01 16:23:54.000000000","message":"This is quite old now, its probably worth bumping this default?\n\nI wonder if we should default to \"latest\" or similar, with a note that operators should set this to a version they have tested with their images? Unsure.","commit_id":"eba5e2b38b2cfa9c49dfa8f8ec0a49d3f09f20bb"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"e23954e8ddacfa38dcc769f9e3e70381da920a2d","unresolved":false,"context_lines":[{"line_number":58,"context_line":"    ),"},{"line_number":59,"context_line":"    cfg.StrOpt("},{"line_number":60,"context_line":"        \"default_helm_chart_version\","},{"line_number":61,"context_line":"        default\u003d\"0.4.0\","},{"line_number":62,"context_line":"        help\u003d("},{"line_number":63,"context_line":"            \"Version of the helm chart specified \""},{"line_number":64,"context_line":"            \"by the config: capi_driver.helm_chart_repo \""}],"source_content_type":"text/x-python","patch_set":1,"id":"f23b4cc3_ae1f005f","line":61,"in_reply_to":"79cde22a_9481480d","updated":"2024-09-30 20:26:42.000000000","message":"updated to 0.10.1. I guess the question of \u0027latest\u0027 or not is kind of unrelated to this change, and may require some further thought","commit_id":"eba5e2b38b2cfa9c49dfa8f8ec0a49d3f09f20bb"}],"magnum_capi_helm/helm.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"53fa64397d0f331d62ee36be0fac6487ed46e223","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"# This code is loosely based on:"},{"line_number":28,"context_line":"#  https://github.com/stackhpc/pyhelm3"},{"line_number":29,"context_line":"#  Ideally we can share this code in the future."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def mergeconcat(defaults, *overrides):"}],"source_content_type":"text/x-python","patch_set":1,"id":"d8cde2de_9ed7db8e","side":"PARENT","line":29,"updated":"2024-08-01 16:23:54.000000000","message":"I think ideally we would still share this code.","commit_id":"957ce61b3cc601b6425b1b161b5ca0978ceb6d3b"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"e23954e8ddacfa38dcc769f9e3e70381da920a2d","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"# This code is loosely based on:"},{"line_number":28,"context_line":"#  https://github.com/stackhpc/pyhelm3"},{"line_number":29,"context_line":"#  Ideally we can share this code in the future."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def mergeconcat(defaults, *overrides):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7e488716_28cadbc6","side":"PARENT","line":29,"in_reply_to":"d8cde2de_9ed7db8e","updated":"2024-09-30 20:26:42.000000000","message":"Done","commit_id":"957ce61b3cc601b6425b1b161b5ca0978ceb6d3b"}],"releasenotes/notes/change-helm-repo-55869552914bb9ed.yaml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a0a9771e64f14c19a8a6bed4aca7fa80f3979955","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"c65f5b55_fec7e889","line":2,"updated":"2024-10-01 03:48:50.000000000","message":"shouldn’t that be rather upgrade and fixes?","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"744003c843d1b040d2c0659c30704a29ee2a70ee","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"654a334e_4093fec8","line":2,"in_reply_to":"c65f5b55_fec7e889","updated":"2024-10-04 11:08:19.000000000","message":"Done","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a0a9771e64f14c19a8a6bed4aca7fa80f3979955","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"},{"line_number":6,"context_line":"  - |"},{"line_number":7,"context_line":"    Helm repository \u0027stackhpc.github.io/capi-helm-charts\u0027 is deprecated."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bdbabc08_03058200","line":5,"updated":"2024-10-01 03:48:50.000000000","message":"it’s not deprecated, it’s not existent (and it broke all users)","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"744003c843d1b040d2c0659c30704a29ee2a70ee","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"},{"line_number":6,"context_line":"  - |"},{"line_number":7,"context_line":"    Helm repository \u0027stackhpc.github.io/capi-helm-charts\u0027 is deprecated."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"983bdf9f_8835587d","line":5,"in_reply_to":"7b987176_8bdfadc0","updated":"2024-10-04 11:08:19.000000000","message":"I\u0027m going to side with Matt here and say it is a deprecation as the charts up to the 0.9.0 version tag are still accessible at the old URL. I can change \u0027features\u0027 to \u0027upgrade\u0027 though.","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"d1e3445f9607e94f4ad08fbbfaeaf3d86ce512ce","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"},{"line_number":6,"context_line":"  - |"},{"line_number":7,"context_line":"    Helm repository \u0027stackhpc.github.io/capi-helm-charts\u0027 is deprecated."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"8f958557_f38cd919","line":5,"in_reply_to":"8e4e2106_a8e0ab83","updated":"2024-10-04 12:08:37.000000000","message":"Done","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"27955a0eb91f1c3cac8a6a8bde6a84aec411464a","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"},{"line_number":6,"context_line":"  - |"},{"line_number":7,"context_line":"    Helm repository \u0027stackhpc.github.io/capi-helm-charts\u0027 is deprecated."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"99cc7e3b_bf331171","line":5,"in_reply_to":"983bdf9f_8835587d","updated":"2024-10-04 11:11:00.000000000","message":"Should we note what is the maximum version supported by the old repo?","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"10530d01e876542724191f81de758a5e01788ae2","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"},{"line_number":6,"context_line":"  - |"},{"line_number":7,"context_line":"    Helm repository \u0027stackhpc.github.io/capi-helm-charts\u0027 is deprecated."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"8e4e2106_a8e0ab83","line":5,"in_reply_to":"99cc7e3b_bf331171","updated":"2024-10-04 12:08:20.000000000","message":"good idea","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"b598648cedf7b9b572fe259142c2195ec15f8547","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"},{"line_number":6,"context_line":"  - |"},{"line_number":7,"context_line":"    Helm repository \u0027stackhpc.github.io/capi-helm-charts\u0027 is deprecated."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7b987176_8bdfadc0","line":5,"in_reply_to":"bdbabc08_03058200","updated":"2024-10-02 14:43:38.000000000","message":"It shouldn\u0027t have broken all users - the old repository still exists with all the tags up to that point. Perhaps this was a transient error that only occurred during the migration from stackhpc to azimuth-cloud.","commit_id":"40f37288ba78abdcb3b314a0f085bf7d34da5dbc"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"0b9c3ffffa5a3fc3aeec9ba207fe97ba218f2843","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrades:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"a4940e4b_fe15445c","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":8},"updated":"2024-10-09 15:05:40.000000000","message":"nit: this should be \u0027upgrade\u0027 otherwise reno does not understand it [1].\n\n[1] WARNING: The upgrades section of releasenotes/notes/change-helm-repo-55869552914bb9ed.yaml is not a recognized section. It should be one of: upgrade, fixes, deprecations, issues, features, critical, other, security. This will be an error in a future release.","commit_id":"fd6c6be84840fff504fd54f5fdfca7296abe9b7d"},{"author":{"_account_id":35260,"name":"Tyler","email":"tyler@stackhpc.com","username":"scrungus"},"change_message_id":"cdf7760cffdbee28f39863c6ef2da149977d9a01","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrades:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Default helm repository is updated to \u0027azimuth-cloud.github.io/capi-helm-charts\u0027."},{"line_number":5,"context_line":"deprecations:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"4bfaaaf2_c034b49e","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":8},"in_reply_to":"a4940e4b_fe15445c","updated":"2024-10-10 10:47:50.000000000","message":"apologies, I just put a patch up to correct this: https://review.opendev.org/c/openstack/magnum-capi-helm/+/932050. I will bump the hash in the release when this merges","commit_id":"fd6c6be84840fff504fd54f5fdfca7296abe9b7d"}]}