)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14394,"name":"Dale Smith","email":"dale@catalystcloud.nz","username":"dalees"},"change_message_id":"a3b45431bff8ff0892a26c70975aec4bf0bf231e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aab0402e_95e69edd","updated":"2024-08-02 01:03:46.000000000","message":"Needs release notes added, and the CIDRs made configurable.","commit_id":"ea624128ca587ad58e29301a90ad3e4659748e4b"}],"magnum_capi_helm/driver.py":[{"author":{"_account_id":14394,"name":"Dale Smith","email":"dale@catalystcloud.nz","username":"dalees"},"change_message_id":"a3b45431bff8ff0892a26c70975aec4bf0bf231e","unresolved":true,"context_lines":[{"line_number":838,"context_line":""},{"line_number":839,"context_line":"        network_id \u003d self._get_fixed_network_id(context, cluster)"},{"line_number":840,"context_line":"        subnet_id \u003d neutron.get_fixed_subnet_id(context, cluster.fixed_subnet)"},{"line_number":841,"context_line":"        # NOTE(dalees): These are defaults from the Helm charts, but moved"},{"line_number":842,"context_line":"        # here to ensure consistency with allowed address pairs."},{"line_number":843,"context_line":"        # They should move to a configurable location before merging upstream"},{"line_number":844,"context_line":"        # as part of https://github.com/stackhpc/magnum-capi-helm/pull/30"}],"source_content_type":"text/x-python","patch_set":1,"id":"2d94d2f2_686ba756","line":841,"updated":"2024-08-02 01:03:46.000000000","message":"As noted; this part needs to be re-worked to be configurable prior to merging.\n\nIt does need to be known in the Magnum driver however, due to the allowed address pairs CIDRs needing to be the same.","commit_id":"ea624128ca587ad58e29301a90ad3e4659748e4b"}]}