Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Name length rule for NewKubernetesCluster is more strict than in azure #3030

Open
alex92ch opened this issue Feb 19, 2025 · 1 comment · May be fixed by #3031
Open

Name length rule for NewKubernetesCluster is more strict than in azure #3030

alex92ch opened this issue Feb 19, 2025 · 1 comment · May be fixed by #3031
Labels
kind/bug Some behavior is incorrect or out of spec

Comments

@alex92ch
Copy link

Describe what happened

Defining the cluster name using the unique name of the Pulumi resource is restricted to 24 characters, even though Azure allows up to 63 characters. Defining the name using the KubernetesClusterArgs.Name works, but I would like to use the unique suffixes as well.

Image

Sample program

	aks, err := containerservice.NewKubernetesCluster(ctx, "aks-aviato-abt-container-cluster-", &containerservice.KubernetesClusterArgs{
		ResourceGroupName:       args.ResourceGroupName,
		WorkloadIdentityEnabled: pulumi.Bool(true),
		AzureRbacEnabled: pulumi.Bool(false),
		TenantId:         args.ClusterAadTenantID,
	        ...
                ...
         },

Log output

No response

Affected Resource(s)

NewKubernetesCluster

Output of pulumi about

CLI
Version 3.150.0
Go Version go1.23.6
Go Compiler gc

Plugins
KIND NAME VERSION
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azure-native 2.85.0
resource azuread 5.53.5
resource datadog 4.42.0
language go 3.150.0
resource random 4.17.0

Host
OS darwin
Version 15.3.1
Arch arm64

This project is written in go: executable='/opt/homebrew/bin/go' version='go version go1.24.0 darwin/arm64'

Additional context

No response

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@alex92ch alex92ch added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Feb 19, 2025
@rquitales
Copy link
Member

Thanks for reporting this issue. The default autonaming rule for this provider sets a maximum character limit of 24. We'll need to add a custom override for this resource to support a max length of 64.

@rquitales rquitales removed the needs-triage Needs attention from the triage team label Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Some behavior is incorrect or out of spec
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants