)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10910,"name":"Stig Telfer","email":"stig.openstack@telfer.org","username":"stigtelfer"},"change_message_id":"c778d2914ca8259735d36aa6a977cf0fbfec4335","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6b235d1b_6fc618be","updated":"2022-01-12 21:51:58.000000000","message":"Light on detail but hopefully enough to get us started.","commit_id":"6659b2a0f819b54c2871c4dac98398df2a219391"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fa7d956facd03016cf1d8cae49658db01aaa466e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"24c84b50_e7b21ac9","in_reply_to":"6b235d1b_6fc618be","updated":"2022-10-17 16:53:57.000000000","message":"Ack","commit_id":"6659b2a0f819b54c2871c4dac98398df2a219391"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"11434ea957b11d9c9f56cd51dc566f234b1789d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"01f9231c_692f98c4","updated":"2022-02-16 09:05:52.000000000","message":"Basic question that crosses my mind - do we expect users to have a pre existing Kubernetes cluster that will serve as Cluster API server?","commit_id":"1f3877159e5531ba3c2bffcd59f04dfca6609d8a"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ec179296df0b34104290da0c123c7f51e68cb581","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1cb472a9_2ae1e656","updated":"2022-07-21 05:38:35.000000000","message":"needs to be moved to specs/zed/","commit_id":"1f3877159e5531ba3c2bffcd59f04dfca6609d8a"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ec179296df0b34104290da0c123c7f51e68cb581","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"46adc8b1_14f8c7fa","in_reply_to":"01f9231c_692f98c4","updated":"2022-07-21 05:38:35.000000000","message":"I guess we do - at least for now.","commit_id":"1f3877159e5531ba3c2bffcd59f04dfca6609d8a"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"ab7df2ac525ef184187802d47df8384b852ae33c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0bd6798b_86d66e0e","in_reply_to":"1cb472a9_2ae1e656","updated":"2022-10-17 16:47:36.000000000","message":"and now antelope, done.","commit_id":"1f3877159e5531ba3c2bffcd59f04dfca6609d8a"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"ab7df2ac525ef184187802d47df8384b852ae33c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"56371516_3dea4d33","in_reply_to":"46adc8b1_14f8c7fa","updated":"2022-10-17 16:47:36.000000000","message":"Yes, it mentioned cluster-api management cluster being setup outside of magnum, but I have tried to make that clearer.","commit_id":"1f3877159e5531ba3c2bffcd59f04dfca6609d8a"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"d4c42b2b6035cf76a22a2064222a3fc2400f7aeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5fb63156_c937412c","updated":"2022-10-18 14:38:08.000000000","message":"Just a few small tweaks. Mostly looks good though!","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f8f6a37b_342324b5","updated":"2022-10-31 11:28:04.000000000","message":"Thanks for putting in so much effort in creating this spec. Michal has a lot of good points, I think fixing up those will tighten this a bit more. I also hope that we can have more clarity on the \"Initial POC\" section.\n\nI think we are on a good track. Looking forward to the updated POC!","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"a98fd6d0682e8485e4ecf6167432052009166a42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1deef83f_8a15eabd","updated":"2023-01-30 12:24:38.000000000","message":"Yes I agreed we should get this merged. Since Antelope is near release, I\u0027ve updated the spec target to Bobcat.","commit_id":"042d55a322c52b3495d534d13f8b800a5c274bf8"}],"specs/antelope/clusterapi-driver.rst":[{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"d4c42b2b6035cf76a22a2064222a3fc2400f7aeb","unresolved":true,"context_lines":[{"line_number":100,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":101,"context_line":"| Prometheus monitoring    | Installed by default.| Installed via Helm charts.|"},{"line_number":102,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":103,"context_line":"| Ingress controllers      | Octavia, Traefik,    | Nginx installed via Helm. |"},{"line_number":104,"context_line":"|                          | Nginx.               |                           |"},{"line_number":105,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":106,"context_line":"| Horizon dashboard        | Supported.           | Not supported.            |"}],"source_content_type":"text/x-rst","patch_set":5,"id":"b1b59f87_3fc116f1","line":103,"updated":"2022-10-18 14:38:08.000000000","message":"No reason why we can\u0027t support additional ingress controllers in Cluster API, if required.","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"f44be22f3b08ac3899a641b640a28f840c6824d0","unresolved":false,"context_lines":[{"line_number":100,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":101,"context_line":"| Prometheus monitoring    | Installed by default.| Installed via Helm charts.|"},{"line_number":102,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":103,"context_line":"| Ingress controllers      | Octavia, Traefik,    | Nginx installed via Helm. |"},{"line_number":104,"context_line":"|                          | Nginx.               |                           |"},{"line_number":105,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":106,"context_line":"| Horizon dashboard        | Supported.           | Not supported.            |"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7cd002d2_bb59abc9","line":103,"in_reply_to":"b1b59f87_3fc116f1","updated":"2022-10-18 15:06:53.000000000","message":"Done","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"d4c42b2b6035cf76a22a2064222a3fc2400f7aeb","unresolved":true,"context_lines":[{"line_number":103,"context_line":"| Ingress controllers      | Octavia, Traefik,    | Nginx installed via Helm. |"},{"line_number":104,"context_line":"|                          | Nginx.               |                           |"},{"line_number":105,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":106,"context_line":"| Horizon dashboard        | Supported.           | Not supported.            |"},{"line_number":107,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":108,"context_line":"| Delegated authorisation  | Keystone trust.      | Application credential.   |"},{"line_number":109,"context_line":"+--------------------------+----------------------+---------------------------+"}],"source_content_type":"text/x-rst","patch_set":5,"id":"0c641200_d13c63dd","line":106,"updated":"2022-10-18 14:38:08.000000000","message":"Is this referring to the Magnum tab in Horizon? If so, doesn\u0027t implementing the driver get us this?","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"f44be22f3b08ac3899a641b640a28f840c6824d0","unresolved":false,"context_lines":[{"line_number":103,"context_line":"| Ingress controllers      | Octavia, Traefik,    | Nginx installed via Helm. |"},{"line_number":104,"context_line":"|                          | Nginx.               |                           |"},{"line_number":105,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":106,"context_line":"| Horizon dashboard        | Supported.           | Not supported.            |"},{"line_number":107,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":108,"context_line":"| Delegated authorisation  | Keystone trust.      | Application credential.   |"},{"line_number":109,"context_line":"+--------------------------+----------------------+---------------------------+"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5dda4c40_59ad428d","line":106,"in_reply_to":"0c641200_d13c63dd","updated":"2022-10-18 15:06:53.000000000","message":"Done","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"d4c42b2b6035cf76a22a2064222a3fc2400f7aeb","unresolved":true,"context_lines":[{"line_number":124,"context_line":"|                          | default node group.  | infrastructure agnositc   |"},{"line_number":125,"context_line":"|                          |                      | code.                     |"},{"line_number":126,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":127,"context_line":"| Multiple node groups     | Supported.           | Supported, with no default|"},{"line_number":128,"context_line":"|                          |                      | group.                    |"},{"line_number":129,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":130,"context_line":"| Additional networks      | Supported (review    | Supported                 |"}],"source_content_type":"text/x-rst","patch_set":5,"id":"80f0bff5_04acc6df","line":127,"updated":"2022-10-18 14:38:08.000000000","message":"Scaling to zero is not currently supported in the Cluster API OpenStack provider. It is now supported in Cluster API itself, but CAPO does not implement the required interfaces yet.","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"f44be22f3b08ac3899a641b640a28f840c6824d0","unresolved":false,"context_lines":[{"line_number":124,"context_line":"|                          | default node group.  | infrastructure agnositc   |"},{"line_number":125,"context_line":"|                          |                      | code.                     |"},{"line_number":126,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":127,"context_line":"| Multiple node groups     | Supported.           | Supported, with no default|"},{"line_number":128,"context_line":"|                          |                      | group.                    |"},{"line_number":129,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":130,"context_line":"| Additional networks      | Supported (review    | Supported                 |"}],"source_content_type":"text/x-rst","patch_set":5,"id":"b6e9b623_27304d25","line":127,"in_reply_to":"80f0bff5_04acc6df","updated":"2022-10-18 15:06:53.000000000","message":"Done","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":33467,"name":"Matt Pryor","email":"matt@stackhpc.com","username":"mkjpryor"},"change_message_id":"d4c42b2b6035cf76a22a2064222a3fc2400f7aeb","unresolved":true,"context_lines":[{"line_number":177,"context_line":""},{"line_number":178,"context_line":"For clusters we found:"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"* uuid used for the name in k8s, as with templates"},{"line_number":181,"context_line":"* default node group users: cluster.flavor_id, cluster.node_count"},{"line_number":182,"context_line":"* Other node groups map in a similar way, using uuid as the name"},{"line_number":183,"context_line":"* control plane size can some from cluster.master_flavor_id,"}],"source_content_type":"text/x-rst","patch_set":5,"id":"b2a24e65_bccf66df","line":180,"updated":"2022-10-18 14:38:08.000000000","message":"This will result in the UUID being in machine names, which is not very friendly. I think we are better sanitising the Magnum resource name, then adding the UUID as a label for easy fetching.","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0d887cb572a665f29c352da99616cec6d20d94db","unresolved":true,"context_lines":[{"line_number":177,"context_line":""},{"line_number":178,"context_line":"For clusters we found:"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"* uuid used for the name in k8s, as with templates"},{"line_number":181,"context_line":"* default node group users: cluster.flavor_id, cluster.node_count"},{"line_number":182,"context_line":"* Other node groups map in a similar way, using uuid as the name"},{"line_number":183,"context_line":"* control plane size can some from cluster.master_flavor_id,"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5e4ec9d1_190f5d92","line":180,"in_reply_to":"b2a24e65_bccf66df","updated":"2022-10-18 15:07:14.000000000","message":"It would be nice to have a better solution long term, but shouldn\u0027t affect the first cut.","commit_id":"d4f010eee2de055f9fe4baaf26b803a513daaf2c"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Problem description"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The existing Magnum support for kubernetes has proven hard to"},{"line_number":26,"context_line":"maintain, particular around: upgrade, auto healing, auto scaling,"},{"line_number":27,"context_line":"and keeping track of host operating system changes."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"5acc64a7_34db5756","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":42},"updated":"2022-10-19 09:15:05.000000000","message":"Kubernetes?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Problem description"},{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The existing Magnum support for kubernetes has proven hard to"},{"line_number":26,"context_line":"maintain, particular around: upgrade, auto healing, auto scaling,"},{"line_number":27,"context_line":"and keeping track of host operating system changes."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"b4128d73_d3291df3","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":42},"in_reply_to":"5acc64a7_34db5756","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The existing Magnum support for kubernetes has proven hard to"},{"line_number":26,"context_line":"maintain, particular around: upgrade, auto healing, auto scaling,"},{"line_number":27,"context_line":"and keeping track of host operating system changes."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"At the same time, Kubernetes Cluster API has gained a lot of"}],"source_content_type":"text/x-rst","patch_set":6,"id":"8ca1e3dd_45309fee","line":26,"range":{"start_line":26,"start_character":10,"end_line":26,"end_character":20},"updated":"2022-10-31 11:28:04.000000000","message":"particularly","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6f863fe2248c69e1a0861696070ae6d27a33da9c","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"The existing Magnum support for kubernetes has proven hard to"},{"line_number":26,"context_line":"maintain, particular around: upgrade, auto healing, auto scaling,"},{"line_number":27,"context_line":"and keeping track of host operating system changes."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"At the same time, Kubernetes Cluster API has gained a lot of"}],"source_content_type":"text/x-rst","patch_set":6,"id":"92c36a19_766da54c","line":26,"range":{"start_line":26,"start_character":10,"end_line":26,"end_character":20},"in_reply_to":"8ca1e3dd_45309fee","updated":"2022-11-14 09:45:39.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":54,"context_line":"For details on Cluster API terminology please see:"},{"line_number":55,"context_line":"https://cluster-api.sigs.k8s.io/user/concepts.html"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"The new Cluster API driver will not make use of heat, instead it"},{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"c58a901c_a36067a9","line":57,"range":{"start_line":57,"start_character":48,"end_line":57,"end_character":52},"updated":"2022-10-19 09:15:05.000000000","message":"Heat?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":54,"context_line":"For details on Cluster API terminology please see:"},{"line_number":55,"context_line":"https://cluster-api.sigs.k8s.io/user/concepts.html"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"The new Cluster API driver will not make use of heat, instead it"},{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"66782d28_6a05c1b6","line":57,"range":{"start_line":57,"start_character":48,"end_line":57,"end_character":52},"in_reply_to":"c58a901c_a36067a9","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For CI jobs, there is an expection of a tempest magnum plugin doing"},{"line_number":62,"context_line":"something like creating a minikube will all the required Cluster API"},{"line_number":63,"context_line":"components installed. Then configuring the new Magnum driver to have"},{"line_number":64,"context_line":"access to that cluster."}],"source_content_type":"text/x-rst","patch_set":6,"id":"b08c218b_e6c7431c","line":61,"range":{"start_line":61,"start_character":48,"end_line":61,"end_character":54},"updated":"2022-10-19 09:15:05.000000000","message":"Magnum?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For CI jobs, there is an expection of a tempest magnum plugin doing"},{"line_number":62,"context_line":"something like creating a minikube will all the required Cluster API"},{"line_number":63,"context_line":"components installed. Then configuring the new Magnum driver to have"},{"line_number":64,"context_line":"access to that cluster."}],"source_content_type":"text/x-rst","patch_set":6,"id":"80f330d6_6fef3a57","line":61,"range":{"start_line":61,"start_character":40,"end_line":61,"end_character":47},"updated":"2022-10-19 09:15:05.000000000","message":"Tempest?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For CI jobs, there is an expection of a tempest magnum plugin doing"},{"line_number":62,"context_line":"something like creating a minikube will all the required Cluster API"},{"line_number":63,"context_line":"components installed. Then configuring the new Magnum driver to have"},{"line_number":64,"context_line":"access to that cluster."}],"source_content_type":"text/x-rst","patch_set":6,"id":"2dac185b_9f6666d3","line":61,"range":{"start_line":61,"start_character":25,"end_line":61,"end_character":34},"updated":"2022-10-19 09:15:05.000000000","message":"expectation?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For CI jobs, there is an expection of a tempest magnum plugin doing"},{"line_number":62,"context_line":"something like creating a minikube will all the required Cluster API"},{"line_number":63,"context_line":"components installed. Then configuring the new Magnum driver to have"},{"line_number":64,"context_line":"access to that cluster."}],"source_content_type":"text/x-rst","patch_set":6,"id":"fb7e5ba9_4b67cf2e","line":61,"range":{"start_line":61,"start_character":25,"end_line":61,"end_character":34},"in_reply_to":"2dac185b_9f6666d3","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For CI jobs, there is an expection of a tempest magnum plugin doing"},{"line_number":62,"context_line":"something like creating a minikube will all the required Cluster API"},{"line_number":63,"context_line":"components installed. Then configuring the new Magnum driver to have"},{"line_number":64,"context_line":"access to that cluster."}],"source_content_type":"text/x-rst","patch_set":6,"id":"6aaea367_2aebb9f4","line":61,"range":{"start_line":61,"start_character":40,"end_line":61,"end_character":47},"in_reply_to":"80f330d6_6fef3a57","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":58,"context_line":"will create clusters by interacting with a Kubernetes Cluster that"},{"line_number":59,"context_line":"has been configured as a Cluster API Management Cluster."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For CI jobs, there is an expection of a tempest magnum plugin doing"},{"line_number":62,"context_line":"something like creating a minikube will all the required Cluster API"},{"line_number":63,"context_line":"components installed. Then configuring the new Magnum driver to have"},{"line_number":64,"context_line":"access to that cluster."}],"source_content_type":"text/x-rst","patch_set":6,"id":"62781895_0db0e2e3","line":61,"range":{"start_line":61,"start_character":48,"end_line":61,"end_character":54},"in_reply_to":"b08c218b_e6c7431c","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":64,"context_line":"access to that cluster."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"The intention for the new driver is to make it compatible with the"},{"line_number":67,"context_line":"Magnum REST API.  No changes to the REST API are anticipated."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Bootstrapping and managing the external management cluster and the"},{"line_number":70,"context_line":"Cluster API service is out of scope.  This driver would depend on"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fc927b31_ad7b5ce8","line":67,"range":{"start_line":67,"start_character":17,"end_line":67,"end_character":18},"updated":"2022-10-19 09:15:05.000000000","message":"extra whitespace","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":64,"context_line":"access to that cluster."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"The intention for the new driver is to make it compatible with the"},{"line_number":67,"context_line":"Magnum REST API.  No changes to the REST API are anticipated."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Bootstrapping and managing the external management cluster and the"},{"line_number":70,"context_line":"Cluster API service is out of scope.  This driver would depend on"}],"source_content_type":"text/x-rst","patch_set":6,"id":"316caa7a_3c062983","line":67,"range":{"start_line":67,"start_character":17,"end_line":67,"end_character":18},"in_reply_to":"fc927b31_ad7b5ce8","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":66,"context_line":"The intention for the new driver is to make it compatible with the"},{"line_number":67,"context_line":"Magnum REST API.  No changes to the REST API are anticipated."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Bootstrapping and managing the external management cluster and the"},{"line_number":70,"context_line":"Cluster API service is out of scope.  This driver would depend on"},{"line_number":71,"context_line":"a Cluster API service being installed in a similar way to expecting"},{"line_number":72,"context_line":"Heat API to be installed."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Feature comparison"},{"line_number":75,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"bb61a452_e7b84ed9","line":72,"range":{"start_line":69,"start_character":0,"end_line":72,"end_character":25},"updated":"2022-10-19 09:15:05.000000000","message":"I think that assumption is at least bad. Long term we should be able to use the existing Heat driver to spawn a CAPI service cluster?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6f863fe2248c69e1a0861696070ae6d27a33da9c","unresolved":true,"context_lines":[{"line_number":66,"context_line":"The intention for the new driver is to make it compatible with the"},{"line_number":67,"context_line":"Magnum REST API.  No changes to the REST API are anticipated."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Bootstrapping and managing the external management cluster and the"},{"line_number":70,"context_line":"Cluster API service is out of scope.  This driver would depend on"},{"line_number":71,"context_line":"a Cluster API service being installed in a similar way to expecting"},{"line_number":72,"context_line":"Heat API to be installed."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Feature comparison"},{"line_number":75,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"87582541_c9aacb03","line":72,"range":{"start_line":69,"start_character":0,"end_line":72,"end_character":25},"in_reply_to":"6da5efca_901e8116","updated":"2022-11-14 09:45:39.000000000","message":"I don\u0027t think we should be bootstrapping the management cluster here, there are so many different options. Nova doesn\u0027t say you have to install the OS on hypervisors using Ironic, for example.\n\nNow, we do need devstack to bootstrap the system so its testable. My preference is to reuse the approach we are doing during testing, using K3s.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":66,"context_line":"The intention for the new driver is to make it compatible with the"},{"line_number":67,"context_line":"Magnum REST API.  No changes to the REST API are anticipated."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Bootstrapping and managing the external management cluster and the"},{"line_number":70,"context_line":"Cluster API service is out of scope.  This driver would depend on"},{"line_number":71,"context_line":"a Cluster API service being installed in a similar way to expecting"},{"line_number":72,"context_line":"Heat API to be installed."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Feature comparison"},{"line_number":75,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"6da5efca_901e8116","line":72,"range":{"start_line":69,"start_character":0,"end_line":72,"end_character":25},"in_reply_to":"bb61a452_e7b84ed9","updated":"2022-10-31 11:28:04.000000000","message":"Long term wise I think it will be fantastic to be able to use an admin level API to bootstrap a management cluster. Whether Heat is used to do it will be a discussion for future, I agree that it is out of scope for first pass.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":84,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":85,"context_line":"| Host OS support          | FCOS 33 supported,   | Typically Ubuntu 20.04    |"},{"line_number":86,"context_line":"|                          | 34 and beyond WIP    | LTS, various choices      |"},{"line_number":87,"context_line":"|                          | (due to cgroups v2)  | supported by the image    |"},{"line_number":88,"context_line":"|                          |                      | builder [#]_.             |"},{"line_number":89,"context_line":"|                          | *NOTE: no security   |                           |"},{"line_number":90,"context_line":"|                          | updates for FCOS 33  |                           |"}],"source_content_type":"text/x-rst","patch_set":6,"id":"68c407a7_f65223ed","line":87,"range":{"start_line":87,"start_character":29,"end_line":87,"end_character":48},"updated":"2022-10-19 09:15:05.000000000","message":"I don\u0027t think that\u0027s current statement.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":84,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":85,"context_line":"| Host OS support          | FCOS 33 supported,   | Typically Ubuntu 20.04    |"},{"line_number":86,"context_line":"|                          | 34 and beyond WIP    | LTS, various choices      |"},{"line_number":87,"context_line":"|                          | (due to cgroups v2)  | supported by the image    |"},{"line_number":88,"context_line":"|                          |                      | builder [#]_.             |"},{"line_number":89,"context_line":"|                          | *NOTE: no security   |                           |"},{"line_number":90,"context_line":"|                          | updates for FCOS 33  |                           |"}],"source_content_type":"text/x-rst","patch_set":6,"id":"c225b44d_82fefb9c","line":87,"range":{"start_line":87,"start_character":29,"end_line":87,"end_character":48},"in_reply_to":"68c407a7_f65223ed","updated":"2022-10-31 11:28:04.000000000","message":"Currently we do support FCOS35","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6f863fe2248c69e1a0861696070ae6d27a33da9c","unresolved":true,"context_lines":[{"line_number":84,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":85,"context_line":"| Host OS support          | FCOS 33 supported,   | Typically Ubuntu 20.04    |"},{"line_number":86,"context_line":"|                          | 34 and beyond WIP    | LTS, various choices      |"},{"line_number":87,"context_line":"|                          | (due to cgroups v2)  | supported by the image    |"},{"line_number":88,"context_line":"|                          |                      | builder [#]_.             |"},{"line_number":89,"context_line":"|                          | *NOTE: no security   |                           |"},{"line_number":90,"context_line":"|                          | updates for FCOS 33  |                           |"}],"source_content_type":"text/x-rst","patch_set":6,"id":"e3cc7d56_68aef278","line":87,"range":{"start_line":87,"start_character":29,"end_line":87,"end_character":48},"in_reply_to":"c225b44d_82fefb9c","updated":"2022-11-14 09:45:39.000000000","message":"Right, this is miles out of date, from when this was first written.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":84,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":85,"context_line":"| Host OS support          | FCOS 33 supported,   | Typically Ubuntu 20.04    |"},{"line_number":86,"context_line":"|                          | 34 and beyond WIP    | LTS, various choices      |"},{"line_number":87,"context_line":"|                          | (due to cgroups v2)  | supported by the image    |"},{"line_number":88,"context_line":"|                          |                      | builder [#]_.             |"},{"line_number":89,"context_line":"|                          | *NOTE: no security   |                           |"},{"line_number":90,"context_line":"|                          | updates for FCOS 33  |                           |"}],"source_content_type":"text/x-rst","patch_set":6,"id":"0edb70c7_4dc3cf9a","line":87,"range":{"start_line":87,"start_character":29,"end_line":87,"end_character":48},"in_reply_to":"e3cc7d56_68aef278","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":96,"context_line":"| Supported CNIs           | Flannel, Calico.     | Further options available,|"},{"line_number":97,"context_line":"|                          |                      | eg Cilium.                |"},{"line_number":98,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":99,"context_line":"| Cinder CSI               | Default from Victoria| Installed via Helm charts.|"},{"line_number":100,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":101,"context_line":"| Prometheus monitoring    | Installed by default.| Installed via Helm charts.|"},{"line_number":102,"context_line":"+--------------------------+----------------------+---------------------------+"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3bd21e9c_d7106f40","line":99,"range":{"start_line":99,"start_character":29,"end_line":99,"end_character":50},"updated":"2022-10-31 11:28:04.000000000","message":"This is not really true as it has been bumped recently.\nFor simplicity sake, shall we state this as \u0027Managed in Magnum code\u0027 as that is one of the pain points?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":125,"context_line":"|                          |                      | Auto-scaling to zero      |"},{"line_number":126,"context_line":"|                          |                      | supported in CAPI, but    |"},{"line_number":127,"context_line":"|                          |                      | CAPO only supports scaling|"},{"line_number":128,"context_line":"|                          |                      | scaling to one node per   |"},{"line_number":129,"context_line":"|                          |                      | group currently.          |"},{"line_number":130,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":131,"context_line":"| Multiple node groups     | Supported.           | Supported, with no default|"}],"source_content_type":"text/x-rst","patch_set":6,"id":"bdbe59f8_43c03698","line":128,"range":{"start_line":128,"start_character":52,"end_line":128,"end_character":59},"updated":"2022-10-19 09:15:05.000000000","message":"double scaling","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":125,"context_line":"|                          |                      | Auto-scaling to zero      |"},{"line_number":126,"context_line":"|                          |                      | supported in CAPI, but    |"},{"line_number":127,"context_line":"|                          |                      | CAPO only supports scaling|"},{"line_number":128,"context_line":"|                          |                      | scaling to one node per   |"},{"line_number":129,"context_line":"|                          |                      | group currently.          |"},{"line_number":130,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":131,"context_line":"| Multiple node groups     | Supported.           | Supported, with no default|"}],"source_content_type":"text/x-rst","patch_set":6,"id":"daeaf9bf_31956605","line":128,"range":{"start_line":128,"start_character":52,"end_line":128,"end_character":59},"in_reply_to":"bdbe59f8_43c03698","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":128,"context_line":"|                          |                      | scaling to one node per   |"},{"line_number":129,"context_line":"|                          |                      | group currently.          |"},{"line_number":130,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":131,"context_line":"| Multiple node groups     | Supported.           | Supported, with no default|"},{"line_number":132,"context_line":"|                          |                      | group.                    |"},{"line_number":133,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":134,"context_line":"| Additional networks      | Supported (review    | Supported                 |"}],"source_content_type":"text/x-rst","patch_set":6,"id":"d26bd97a_39e332b6","line":131,"range":{"start_line":131,"start_character":77,"end_line":131,"end_character":78},"updated":"2022-10-19 09:15:05.000000000","message":"whitespace before end of column missing","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":128,"context_line":"|                          |                      | scaling to one node per   |"},{"line_number":129,"context_line":"|                          |                      | group currently.          |"},{"line_number":130,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":131,"context_line":"| Multiple node groups     | Supported.           | Supported, with no default|"},{"line_number":132,"context_line":"|                          |                      | group.                    |"},{"line_number":133,"context_line":"+--------------------------+----------------------+---------------------------+"},{"line_number":134,"context_line":"| Additional networks      | Supported (review    | Supported                 |"}],"source_content_type":"text/x-rst","patch_set":6,"id":"cfdfb68c_d7610c16","line":131,"range":{"start_line":131,"start_character":77,"end_line":131,"end_character":78},"in_reply_to":"d26bd97a_39e332b6","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"f2b6be99861fc05201f0ebc1d6bef232797bfcd9","unresolved":true,"context_lines":[{"line_number":158,"context_line":"  (flavor_id and node counts from default node group)"},{"line_number":159,"context_line":"* coe credentials to help users create a valid kubeconfig"},{"line_number":160,"context_line":"* Devstack install (manually) passing sonoboy conformance tests"},{"line_number":161,"context_line":"* support for resizing the default node group"},{"line_number":162,"context_line":"* upgrade by moving to a newer template (with updated image)"},{"line_number":163,"context_line":"* add/remove/resize node groups"},{"line_number":164,"context_line":"* customize internal and external network uuids"}],"source_content_type":"text/x-rst","patch_set":6,"id":"b7b88d94_be6fac57","line":161,"updated":"2022-10-19 14:22:36.000000000","message":"Add in generate app creds somewhere in here.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"b26cadfac36c5b08b7f797b92de89543683112a0","unresolved":true,"context_lines":[{"line_number":158,"context_line":"  (flavor_id and node counts from default node group)"},{"line_number":159,"context_line":"* coe credentials to help users create a valid kubeconfig"},{"line_number":160,"context_line":"* Devstack install (manually) passing sonoboy conformance tests"},{"line_number":161,"context_line":"* support for resizing the default node group"},{"line_number":162,"context_line":"* upgrade by moving to a newer template (with updated image)"},{"line_number":163,"context_line":"* add/remove/resize node groups"},{"line_number":164,"context_line":"* customize internal and external network uuids"}],"source_content_type":"text/x-rst","patch_set":6,"id":"b8fa8fe0_3df982dc","line":161,"in_reply_to":"b7b88d94_be6fac57","updated":"2023-01-18 13:51:03.000000000","message":"Let\u0027s get this merged - seems like it needs an update here?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":162,"context_line":"* upgrade by moving to a newer template (with updated image)"},{"line_number":163,"context_line":"* add/remove/resize node groups"},{"line_number":164,"context_line":"* customize internal and external network uuids"},{"line_number":165,"context_line":"* Sonoboy conformance tests passing in Zuul CI"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"Initial POC"},{"line_number":168,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1b6cf350_078275a1","line":165,"range":{"start_line":165,"start_character":0,"end_line":165,"end_character":46},"updated":"2022-10-31 11:28:04.000000000","message":"I feel like this should be higher up, maybe before supporting resizing? It will give us a good base to work and increase velocity of patches when we have good tests. Is that too difficult?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6f863fe2248c69e1a0861696070ae6d27a33da9c","unresolved":true,"context_lines":[{"line_number":162,"context_line":"* upgrade by moving to a newer template (with updated image)"},{"line_number":163,"context_line":"* add/remove/resize node groups"},{"line_number":164,"context_line":"* customize internal and external network uuids"},{"line_number":165,"context_line":"* Sonoboy conformance tests passing in Zuul CI"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"Initial POC"},{"line_number":168,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"264384c4_a1ce3b97","line":165,"range":{"start_line":165,"start_character":0,"end_line":165,"end_character":46},"in_reply_to":"1b6cf350_078275a1","updated":"2022-11-14 09:45:39.000000000","message":"Yeah, good idea, lets pull that earlier. I forget why I put it there. It somewhat depends on the existing tempest tests getting fixed, that will make this step a lot easier.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":171,"context_line":"https://review.opendev.org/c/openstack/magnum/+/851076"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"The POC established a few ground rules for cluster templates:"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"* image_id in the template the key part of the template"},{"line_number":176,"context_line":"* kube_tag i.e. the k8s version, is fixed for the image_id,"},{"line_number":177,"context_line":"  ideally we should validate that link"},{"line_number":178,"context_line":"* using uuid for the name of templates and clusters in k8s"},{"line_number":179,"context_line":"  as some magnum names are not valid k8s resource names,"},{"line_number":180,"context_line":"  and reduces the need to sanitise user inputs"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"For clusters we found:"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"* uuid used for the name in k8s, as with templates"}],"source_content_type":"text/x-rst","patch_set":6,"id":"bfd5b99d_43d83122","line":181,"range":{"start_line":174,"start_character":0,"end_line":181,"end_character":0},"updated":"2022-10-31 11:28:04.000000000","message":"I find this part a bit confusing. Will this be updated with the Helm implementation? Can we have an example here?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":184,"context_line":"* uuid used for the name in k8s, as with templates"},{"line_number":185,"context_line":"* default node group users: cluster.flavor_id, cluster.node_count"},{"line_number":186,"context_line":"* Other node groups map in a similar way, using uuid as the name"},{"line_number":187,"context_line":"* control plane size can some from cluster.master_flavor_id,"},{"line_number":188,"context_line":"  (although, there might be good reason to ignore this and"},{"line_number":189,"context_line":"  have it defined only in the template)"},{"line_number":190,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"b844e165_06423a3e","line":187,"range":{"start_line":187,"start_character":2,"end_line":187,"end_character":59},"updated":"2022-10-19 09:15:05.000000000","message":"don\u0027t understand this statement","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":true,"context_lines":[{"line_number":184,"context_line":"* uuid used for the name in k8s, as with templates"},{"line_number":185,"context_line":"* default node group users: cluster.flavor_id, cluster.node_count"},{"line_number":186,"context_line":"* Other node groups map in a similar way, using uuid as the name"},{"line_number":187,"context_line":"* control plane size can some from cluster.master_flavor_id,"},{"line_number":188,"context_line":"  (although, there might be good reason to ignore this and"},{"line_number":189,"context_line":"  have it defined only in the template)"},{"line_number":190,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"d1a159df_f0d88d10","line":187,"range":{"start_line":187,"start_character":2,"end_line":187,"end_character":59},"in_reply_to":"b844e165_06423a3e","updated":"2022-11-14 10:58:08.000000000","message":"attempted to fix that up.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":193,"context_line":"added to both the template and the cluster, but they are unlikely"},{"line_number":194,"context_line":"to be a similar format to the existing drivers."},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"Note: all network specifications would be ignored in this first"},{"line_number":197,"context_line":"version, as the external network can be specificed by the default"},{"line_number":198,"context_line":"helm chart variables."},{"line_number":199,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"fb516f91_014e9db4","line":196,"range":{"start_line":196,"start_character":0,"end_line":196,"end_character":4},"updated":"2022-10-19 09:15:05.000000000","message":"uppercase for visibility","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":193,"context_line":"added to both the template and the cluster, but they are unlikely"},{"line_number":194,"context_line":"to be a similar format to the existing drivers."},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"Note: all network specifications would be ignored in this first"},{"line_number":197,"context_line":"version, as the external network can be specificed by the default"},{"line_number":198,"context_line":"helm chart variables."},{"line_number":199,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"f3e04cbc_2b8326f6","line":196,"range":{"start_line":196,"start_character":0,"end_line":196,"end_character":4},"in_reply_to":"fb516f91_014e9db4","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":200,"context_line":"Communicating with K8s"},{"line_number":201,"context_line":"----------------------"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."}],"source_content_type":"text/x-rst","patch_set":6,"id":"b4abfe7a_a5663f72","line":203,"range":{"start_line":203,"start_character":26,"end_line":203,"end_character":31},"updated":"2022-10-19 09:15:05.000000000","message":"Jinja","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":200,"context_line":"Communicating with K8s"},{"line_number":201,"context_line":"----------------------"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."}],"source_content_type":"text/x-rst","patch_set":6,"id":"23ab3f19_87964086","line":203,"range":{"start_line":203,"start_character":26,"end_line":203,"end_character":31},"in_reply_to":"b4abfe7a_a5663f72","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"}],"source_content_type":"text/x-rst","patch_set":6,"id":"0a5d30bc_11682bd3","line":205,"range":{"start_line":205,"start_character":16,"end_line":205,"end_character":23},"updated":"2022-10-19 09:15:05.000000000","message":"Ansible","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"}],"source_content_type":"text/x-rst","patch_set":6,"id":"d93aede3_f36ff711","line":205,"range":{"start_line":205,"start_character":46,"end_line":205,"end_character":56},"updated":"2022-10-19 09:15:05.000000000","message":"considers","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"}],"source_content_type":"text/x-rst","patch_set":6,"id":"e0353f50_7d77917c","line":205,"range":{"start_line":205,"start_character":16,"end_line":205,"end_character":23},"in_reply_to":"0a5d30bc_11682bd3","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"}],"source_content_type":"text/x-rst","patch_set":6,"id":"47c7bc4e_8573d136","line":205,"range":{"start_line":205,"start_character":46,"end_line":205,"end_character":56},"in_reply_to":"d93aede3_f36ff711","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"},{"line_number":209,"context_line":"will map to a specific helm chart and release."}],"source_content_type":"text/x-rst","patch_set":6,"id":"45a79e67_d0d18fbb","line":206,"range":{"start_line":206,"start_character":5,"end_line":206,"end_character":12},"updated":"2022-10-19 09:15:05.000000000","message":"consider rewording to less negative term","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":203,"context_line":"The current template used jinja templates in the driver to"},{"line_number":204,"context_line":"template out the required K8s CRDs. While that is great if"},{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"},{"line_number":209,"context_line":"will map to a specific helm chart and release."}],"source_content_type":"text/x-rst","patch_set":6,"id":"870cc5e4_b00595e4","line":206,"range":{"start_line":206,"start_character":5,"end_line":206,"end_character":12},"in_reply_to":"45a79e67_d0d18fbb","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"},{"line_number":209,"context_line":"will map to a specific helm chart and release."},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"A cluster will take the image_uuid from the template and"}],"source_content_type":"text/x-rst","patch_set":6,"id":"0dfc758f_ff4cdea0","line":208,"range":{"start_line":208,"start_character":33,"end_line":208,"end_character":37},"updated":"2022-10-19 09:15:05.000000000","message":"Helm","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"},{"line_number":209,"context_line":"will map to a specific helm chart and release."},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"A cluster will take the image_uuid from the template and"}],"source_content_type":"text/x-rst","patch_set":6,"id":"0c048de5_b7f28b88","line":208,"range":{"start_line":208,"start_character":0,"end_line":208,"end_character":58},"updated":"2022-10-31 11:28:04.000000000","message":"I am a bit confused by this. \n\nShould the spec only state what is the final design? Discussion about the POC may not be necessary?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6f863fe2248c69e1a0861696070ae6d27a33da9c","unresolved":true,"context_lines":[{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"},{"line_number":209,"context_line":"will map to a specific helm chart and release."},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"A cluster will take the image_uuid from the template and"}],"source_content_type":"text/x-rst","patch_set":6,"id":"5eec64f8_0ece4559","line":208,"range":{"start_line":208,"start_character":0,"end_line":208,"end_character":58},"in_reply_to":"0c048de5_b7f28b88","updated":"2022-11-14 09:45:39.000000000","message":"Yeah, the spec should focus on the final design. I guess I am not 100% certain what it is yet at this level, as its not implemented yet. Likely we too much waffle in this section, I will have a look.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":205,"context_line":"you are used to ansbile, likely everyone else consideres"},{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"},{"line_number":209,"context_line":"will map to a specific helm chart and release."},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"A cluster will take the image_uuid from the template and"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7cfc634b_4e84c630","line":208,"range":{"start_line":208,"start_character":33,"end_line":208,"end_character":37},"in_reply_to":"0dfc758f_ff4cdea0","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":206,"context_line":"that strange."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"We propose we move towards using helm. Where by a template"},{"line_number":209,"context_line":"will map to a specific helm chart and release."},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"A cluster will take the image_uuid from the template and"},{"line_number":212,"context_line":"flavor_ids and node counts from the cluster, to create a"}],"source_content_type":"text/x-rst","patch_set":6,"id":"4c9e7c80_411f3eb5","line":209,"range":{"start_line":209,"start_character":23,"end_line":209,"end_character":27},"updated":"2022-10-19 09:15:05.000000000","message":"Helm","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":210,"context_line":""},{"line_number":211,"context_line":"A cluster will take the image_uuid from the template and"},{"line_number":212,"context_line":"flavor_ids and node counts from the cluster, to create a"},{"line_number":213,"context_line":"set of helm values to be used with the above chart."},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"Operators can then customise a set of \"standard\" helm"},{"line_number":216,"context_line":"charts as needed for their specific Cloud."}],"source_content_type":"text/x-rst","patch_set":6,"id":"206688c0_239d6d7a","line":213,"range":{"start_line":213,"start_character":7,"end_line":213,"end_character":11},"updated":"2022-10-19 09:15:05.000000000","message":"Helm","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":210,"context_line":""},{"line_number":211,"context_line":"A cluster will take the image_uuid from the template and"},{"line_number":212,"context_line":"flavor_ids and node counts from the cluster, to create a"},{"line_number":213,"context_line":"set of helm values to be used with the above chart."},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"Operators can then customise a set of \"standard\" helm"},{"line_number":216,"context_line":"charts as needed for their specific Cloud."}],"source_content_type":"text/x-rst","patch_set":6,"id":"a6fcc571_bbb7714c","line":213,"range":{"start_line":213,"start_character":7,"end_line":213,"end_character":11},"in_reply_to":"206688c0_239d6d7a","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":212,"context_line":"flavor_ids and node counts from the cluster, to create a"},{"line_number":213,"context_line":"set of helm values to be used with the above chart."},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"Operators can then customise a set of \"standard\" helm"},{"line_number":216,"context_line":"charts as needed for their specific Cloud."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"For an example of what we might use as a starting point,"}],"source_content_type":"text/x-rst","patch_set":6,"id":"f1d295a6_d3f1a9d9","line":215,"range":{"start_line":215,"start_character":49,"end_line":215,"end_character":53},"updated":"2022-10-19 09:15:05.000000000","message":"Helm","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":8064,"name":"Jake Yip","email":"jake.yip@ardc.edu.au","username":"jake"},"change_message_id":"fef0777a1b3a00afcc4c5c47add59e544df746d0","unresolved":true,"context_lines":[{"line_number":221,"context_line":""},{"line_number":222,"context_line":"OpenStack creds"},{"line_number":223,"context_line":"---------------"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"The current POC found it hard to use the existing cluster"},{"line_number":226,"context_line":"trusts with cluster api provider openstack, largely"},{"line_number":227,"context_line":"due to limitations in gophercloud\u0027s parsing of clouds.yaml"},{"line_number":228,"context_line":"files. It can work with OpenStack cloud provider, so"},{"line_number":229,"context_line":"prehaps this might be fixable upstream."},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"Instead we considered creating a per cluster application"},{"line_number":232,"context_line":"credentials that are registered with each cluster on creation,"}],"source_content_type":"text/x-rst","patch_set":6,"id":"75580638_92c971aa","line":229,"range":{"start_line":224,"start_character":0,"end_line":229,"end_character":39},"updated":"2022-10-31 11:28:04.000000000","message":"I recommend this to say \"Our preference will be to support Application Credentials as a first choice, with the fallback to Trusts if that is not possible.\"","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6f863fe2248c69e1a0861696070ae6d27a33da9c","unresolved":true,"context_lines":[{"line_number":221,"context_line":""},{"line_number":222,"context_line":"OpenStack creds"},{"line_number":223,"context_line":"---------------"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"The current POC found it hard to use the existing cluster"},{"line_number":226,"context_line":"trusts with cluster api provider openstack, largely"},{"line_number":227,"context_line":"due to limitations in gophercloud\u0027s parsing of clouds.yaml"},{"line_number":228,"context_line":"files. It can work with OpenStack cloud provider, so"},{"line_number":229,"context_line":"prehaps this might be fixable upstream."},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"Instead we considered creating a per cluster application"},{"line_number":232,"context_line":"credentials that are registered with each cluster on creation,"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3319057e_6b8e6941","line":229,"range":{"start_line":224,"start_character":0,"end_line":229,"end_character":39},"in_reply_to":"75580638_92c971aa","updated":"2022-11-14 09:45:39.000000000","message":"Yeah, needs to be simpler here, will have a look at that.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":231,"context_line":"Instead we considered creating a per cluster application"},{"line_number":232,"context_line":"credentials that are registered with each cluster on creation,"},{"line_number":233,"context_line":"and can be deleted when the cluster is removed. Although the"},{"line_number":234,"context_line":"current POC code assumes pre-created per project credentails,"},{"line_number":235,"context_line":"to make it simpler."},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"We need to check if these credentials can be rotated when"}],"source_content_type":"text/x-rst","patch_set":6,"id":"134f81ef_67445ed1","line":234,"range":{"start_line":234,"start_character":49,"end_line":234,"end_character":60},"updated":"2022-10-19 09:15:05.000000000","message":"credentials","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":false,"context_lines":[{"line_number":231,"context_line":"Instead we considered creating a per cluster application"},{"line_number":232,"context_line":"credentials that are registered with each cluster on creation,"},{"line_number":233,"context_line":"and can be deleted when the cluster is removed. Although the"},{"line_number":234,"context_line":"current POC code assumes pre-created per project credentails,"},{"line_number":235,"context_line":"to make it simpler."},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"We need to check if these credentials can be rotated when"}],"source_content_type":"text/x-rst","patch_set":6,"id":"bfdcb067_be17e294","line":234,"range":{"start_line":234,"start_character":49,"end_line":234,"end_character":60},"in_reply_to":"134f81ef_67445ed1","updated":"2022-11-14 10:58:08.000000000","message":"Done","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":234,"context_line":"current POC code assumes pre-created per project credentails,"},{"line_number":235,"context_line":"to make it simpler."},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"We need to check if these credentials can be rotated when"},{"line_number":238,"context_line":"different users manage the cluster, without re-creating all"},{"line_number":239,"context_line":"nodes in the cluster. In parituclar, when a user has their"},{"line_number":240,"context_line":"role assignments changes, this can invalidate all"},{"line_number":241,"context_line":"application credentails, and cause problems similar to"},{"line_number":242,"context_line":"when users are deleted and they created the mangum cluster."},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"More work is needed to determine the best way forward, likely"},{"line_number":245,"context_line":"it will involve the driver creating application credentials"},{"line_number":246,"context_line":"for each cluster, and storing those in kubernetes."},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"Assignee(s)"},{"line_number":249,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1eaad5e9_93b4bf99","line":246,"range":{"start_line":237,"start_character":0,"end_line":246,"end_character":50},"updated":"2022-10-19 09:15:05.000000000","message":"Once that\u0027s done, the spec will be updated, right?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":true,"context_lines":[{"line_number":234,"context_line":"current POC code assumes pre-created per project credentails,"},{"line_number":235,"context_line":"to make it simpler."},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"We need to check if these credentials can be rotated when"},{"line_number":238,"context_line":"different users manage the cluster, without re-creating all"},{"line_number":239,"context_line":"nodes in the cluster. In parituclar, when a user has their"},{"line_number":240,"context_line":"role assignments changes, this can invalidate all"},{"line_number":241,"context_line":"application credentails, and cause problems similar to"},{"line_number":242,"context_line":"when users are deleted and they created the mangum cluster."},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"More work is needed to determine the best way forward, likely"},{"line_number":245,"context_line":"it will involve the driver creating application credentials"},{"line_number":246,"context_line":"for each cluster, and storing those in kubernetes."},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"Assignee(s)"},{"line_number":249,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"a28252a8_f953cd2a","line":246,"range":{"start_line":237,"start_character":0,"end_line":246,"end_character":50},"in_reply_to":"1eaad5e9_93b4bf99","updated":"2022-11-14 10:58:08.000000000","message":"re-worded this.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":260,"context_line":"----------"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"* initial driver with create and delete"},{"line_number":263,"context_line":"* full support for all expected"},{"line_number":264,"context_line":"* CI functional tests passing"},{"line_number":265,"context_line":"* Conformance tests passing on helm chart repo"},{"line_number":266,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"8a344cd6_5bd2c620","line":263,"range":{"start_line":263,"start_character":2,"end_line":263,"end_character":31},"updated":"2022-10-19 09:15:05.000000000","message":"expected what?","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":true,"context_lines":[{"line_number":260,"context_line":"----------"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"* initial driver with create and delete"},{"line_number":263,"context_line":"* full support for all expected"},{"line_number":264,"context_line":"* CI functional tests passing"},{"line_number":265,"context_line":"* Conformance tests passing on helm chart repo"},{"line_number":266,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"9ad3c543_0a8da5f2","line":263,"range":{"start_line":263,"start_character":2,"end_line":263,"end_character":31},"in_reply_to":"8a344cd6_5bd2c620","updated":"2022-11-14 10:58:08.000000000","message":"reworded this.","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fecb1214375d694afce660ff0db14f132b21bc29","unresolved":true,"context_lines":[{"line_number":262,"context_line":"* initial driver with create and delete"},{"line_number":263,"context_line":"* full support for all expected"},{"line_number":264,"context_line":"* CI functional tests passing"},{"line_number":265,"context_line":"* Conformance tests passing on helm chart repo"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"Work Items"},{"line_number":268,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1b520e46_02b6b50c","line":265,"range":{"start_line":265,"start_character":31,"end_line":265,"end_character":35},"updated":"2022-10-19 09:15:05.000000000","message":"Helm","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"6ba715fa21c173b36368988d6cf3dfa8d876027a","unresolved":true,"context_lines":[{"line_number":262,"context_line":"* initial driver with create and delete"},{"line_number":263,"context_line":"* full support for all expected"},{"line_number":264,"context_line":"* CI functional tests passing"},{"line_number":265,"context_line":"* Conformance tests passing on helm chart repo"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"Work Items"},{"line_number":268,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"5734e8b8_771215ce","line":265,"range":{"start_line":265,"start_character":31,"end_line":265,"end_character":35},"in_reply_to":"1b520e46_02b6b50c","updated":"2022-11-14 10:58:08.000000000","message":"reworded this","commit_id":"3a6acd08771d3fa5889b54ed7a25978a85cc85e4"}]}