Mohammed Naser

Results 85 comments of Mohammed Naser

> My suggestion is to turn the Flavor field, which is a string, into a struct, and add some fields in it like `name`, `ID`, `memory`, etc. I'm trying to...

> We reached > 350 clusters with > 1990 machines in one of our OpenStack regions. This is a number where we have to start analyzing as our provisioning takes...

``` VERSION 0.7 test: FROM busybox RUN echo "docker.io/timberio/vector:1.23.4@sha256:29f23dab76fa306b67b10eac3e9decdb01c906f8aa3b00a2f5b2e8ae088b84e0" ``` => ``` Build 🔧 ———————————————————————————————————————————————————————————————————————————————— busybox | --> Load metadata busybox linux/amd64 +test | --> FROM busybox +test | [----------]...

👋 @sebastianriese -- have you had a chance to follow up on the rest of this?

> We're still working on getting the tests right--as this will depend on Keystone for integration tests, much of the recent work has been on integrating that somehow sensible. >...

https://github.com/terraform-provider-openstack/terraform-provider-openstack/blob/main/openstack/resource_openstack_containerinfra_cluster_v1.go#L165 it shouldn't force a new if node count changes, is Terraform perhaps giving you some other reason that's forcing the change? If you can share the diff, that would...

I created a PR that updates the documentation for this, it seems that it's not possible to actually update the labels using the API via Magnum, but I do think...

We need to get rid of the `ForceNew` here: https://github.com/terraform-provider-openstack/terraform-provider-openstack/blob/main/openstack/resource_openstack_containerinfra_cluster_v1.go#L86-L91

> There is a special method in Magnum API to upgrade cluster providing a new cluster template - this is used to upgrade Kubernetes version. > > > > >...