Skip to content

Commit 80fca66

Browse files
authored
Merge pull request #5146 from KOREAN139/refactor/set-machinepool-true-as-default
refactor(manager): set machinepool true as default
2 parents 553cd0f + aef33ac commit 80fca66

File tree

5 files changed

+14
-7
lines changed

5 files changed

+14
-7
lines changed

config/manager/manager.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ spec:
2424
- --leader-elect
2525
- "--diagnostics-address=${CAPZ_DIAGNOSTICS_ADDRESS:=:8443}"
2626
- "--insecure-diagnostics=${CAPZ_INSECURE_DIAGNOSTICS:=false}"
27-
- "--feature-gates=MachinePool=${EXP_MACHINE_POOL:=false},AKSResourceHealth=${EXP_AKS_RESOURCE_HEALTH:=false},EdgeZone=${EXP_EDGEZONE:=false},ASOAPI=${EXP_ASO_API:=true}"
27+
- "--feature-gates=MachinePool=${EXP_MACHINE_POOL:=true},AKSResourceHealth=${EXP_AKS_RESOURCE_HEALTH:=false},EdgeZone=${EXP_EDGEZONE:=false},ASOAPI=${EXP_ASO_API:=true}"
2828
- "--v=0"
2929
image: controller:latest
3030
imagePullPolicy: Always

docs/book/src/developers/development.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -571,7 +571,7 @@ You can optionally set the following variables:
571571
| `KUBERNETES_VERSION` | Desired Kubernetes version to test. You can pass in a definitive released version, e.g., "v1.24.0". If you want to use pre-released CI bits of a particular release you may use the "latest-" prefix, e.g., "latest-1.24"; you may use the very latest built CI bits from the kubernetes/kubernetes master branch by passing in "latest". If you provide a `KUBERNETES_VERSION` environment variable, you may not also use `CI_VERSION` (below). Use only one configuration variable to declare the version of Kubernetes to test. |
572572
| `CI_VERSION` | Provide a custom CI version of Kubernetes (e.g., `v1.25.0-alpha.0.597+aa49dffc7f24dc`). If not specified, this will be determined from the KUBERNETES_VERSION above if it is an unreleased version. If you provide a `CI_VERSION` environment variable, you may not also use `KUBERNETES_VERSION` (above). |
573573
| `TEST_CCM` | Build a cluster that uses custom versions of the Azure cloud-provider cloud-controller-manager and node-controller-manager images |
574-
| `EXP_MACHINE_POOL` | Use [Machine Pool](../self-managed/machinepools.md) for worker machines. |
574+
| `EXP_MACHINE_POOL` | Use [Machine Pool](../self-managed/machinepools.md) for worker machines. Defaults to true. |
575575
| `TEST_WINDOWS` | Build a cluster that has Windows worker nodes. |
576576
| `REGISTRY` | Registry to push any custom k8s images or cloud provider images built. |
577577
| `CLUSTER_TEMPLATE` | Use a custom cluster template. It can be a path to a template under templates/, a path on the host or a link. If the value is not set, the script will choose the appropriate cluster template based on existing environment variables. |

docs/book/src/managed/managedcluster.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,12 +68,13 @@ export AZURE_CLUSTER_IDENTITY_SECRET_NAMESPACE="default"
6868
export CLUSTER_IDENTITY_NAME="cluster-identity"
6969
```
7070

71-
Managed clusters require the Cluster API "MachinePool" feature flag enabled. You can do that via an environment variable thusly:
71+
Managed clusters require the Cluster API "MachinePool" feature flag enabled. The feature flag is enabled by default, but you can configure it using the following environment variable:
7272

7373
```bash
74-
export EXP_MACHINE_POOL=true
74+
export EXP_MACHINE_POOL=true
7575
```
7676

77+
7778
Create a local kind cluster to run the management cluster components:
7879

7980
```bash
@@ -416,4 +417,4 @@ To learn more about OIDC and AKS refer [AKS Docs on OIDC issuer](https://learn.m
416417
CAPZ no longer supports passing custom headers to AKS APIs with `infrastructure.cluster.x-k8s.io/custom-header-` annotations.
417418
Custom headers are deprecated in AKS in favor of new features first landing in preview API versions:
418419

419-
https://github.com/Azure/azure-rest-api-specs/pull/18232
420+
https://github.com/Azure/azure-rest-api-specs/pull/18232

docs/book/src/self-managed/machinepools.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# MachinePools
2-
- **Feature status:** Experimental
2+
- **Feature status:** Experimental (Beta)
33
- **Feature gate:** MachinePool=true
4+
- **Default value:** true (enabled)
45

56
> In Cluster API (CAPI) v1alpha2, users can create MachineDeployment, MachineSet or Machine custom
67
> resources. When you create a MachineDeployment or MachineSet, Cluster API components react and
@@ -31,6 +32,11 @@
3132
3233
*Source: [MachinePool API Proposal](https://github.com/kubernetes-sigs/cluster-api/blob/bf51a2502f9007b531f6a9a2c1a4eae1586fb8ca/docs/proposals/20190919-machinepool-api.md)*
3334

35+
> In Cluster API (CAPI) v1.7.0, MachinePool feature flag's default settings become true and promoted
36+
> to Beta from Alpha. To disable this feature, you can use `EXP_MACHINE_POOL=false`.
37+
38+
*Source: [Cluster API v1.7.0 Release Note](https://github.com/kubernetes-sigs/cluster-api/releases/tag/v1.7.0)*
39+
3440
## AzureMachinePool
3541
Cluster API Provider Azure (CAPZ) has experimental support for `MachinePool` through the infrastructure
3642
types `AzureMachinePool` and `AzureMachinePoolMachine`. An `AzureMachinePool` corresponds to a

hack/observability/opentelemetry/controller-manager-patch.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,5 @@ spec:
1212
- "--diagnostics-address=:8080"
1313
- "--insecure-diagnostics"
1414
- "--leader-elect"
15-
- "--feature-gates=MachinePool=${EXP_MACHINE_POOL:=false},AKSResourceHealth=${EXP_AKS_RESOURCE_HEALTH:=false},EdgeZone=${EXP_EDGEZONE:=false},ASOAPI=${EXP_ASO_API:=true}"
15+
- "--feature-gates=MachinePool=${EXP_MACHINE_POOL:=true},AKSResourceHealth=${EXP_AKS_RESOURCE_HEALTH:=false},EdgeZone=${EXP_EDGEZONE:=false},ASOAPI=${EXP_ASO_API:=true}"
1616
- "--enable-tracing"

0 commit comments

Comments
 (0)