kubernetes-sigs / cluster-api-provider-azure

Cluster API implementation for Microsoft Azure
https://capz.sigs.k8s.io/
Apache License 2.0
293 stars 423 forks source link

Enable Azure Container Storage for ManagedClusters #3573

Closed dtzar closed 2 months ago

dtzar commented 1 year ago

/kind feature

Describe the solution you'd like Enable Azure Container Storage with ManagedCluster. Relies on k8s-extension, so depends on #3513 being complete first

k8s-triage-robot commented 8 months ago

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

You can:

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

jackfrancis commented 7 months ago

@mtougeron is this AKS feature on your roadmap?

mtougeron commented 7 months ago

We'd like to have it but it is not a high priority as I understand it from the teams who want to use it.

dtzar commented 6 months ago

This should work once we have #4678 resolved.

{
    "id": "/subscriptions/###/resourceGroups/###/providers/Microsoft.ContainerService/ManagedClusters/###/providers/Microsoft.KubernetesConfiguration/extensionTypes/microsoft.azurecontainerstorage",
    "name": "microsoft.azurecontainerstorage",
    "properties": {
      "description": "Azure container storage",
      "extensionType": "microsoft.azurecontainerstorage",
      "isManagedIdentityRequired": false,
      "isSystemExtension": false,
      "planInfo": {
        "offerId": null,
        "planId": null,
        "publisherId": null
      },
      "publisher": null,
      "supportedClusterTypes": [
        "managedclusters",
        "Managedclusters",
        "managedclusters"
      ],
      "supportedScopes": {
        "clusterScopeSettings": {
          "allowMultipleInstances": false,
          "defaultReleaseNamespace": "acstor",
          "id": null,
          "name": null,
          "type": null
        },
        "defaultScope": "cluster"
      }
    },
    "resourceGroup": "ascapz",
    "type": "Microsoft.KubernetesConfiguration/extensionTypes"
  },
willie-yao commented 6 months ago

4678 is now resolved. Let me know if the AKS extension does the trick here!

jackfrancis commented 5 months ago

@dtzar to validate we are gtg here

nojnhuh commented 2 months ago

/assign @dtzar

mboersma commented 2 months ago

/close

k8s-ci-robot commented 2 months ago

@mboersma: Closing this issue.

In response to [this](https://github.com/kubernetes-sigs/cluster-api-provider-azure/issues/3573#issuecomment-2223357240): >/close > > Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.