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

[BUG] Changing name of node pool cannot be upper case, but the wizard uses upper case by default. #344

Open
The-TeQnician opened this issue May 11, 2023 · 0 comments
Assignees

Comments

@The-TeQnician
Copy link

Describe the bug
When deploying an hybrid aks cluster via the portal the wizard creates a node pool name based on the clustername which is my case was partly uppercase. But I also wanted to deploy a additional node pool so I tried changing the name to linuxnodepool1. Partly uppercase from beginning (first screenshot) no error. But got the error (see second screenshot) when i changed the name. Al lower case no issue's (third screenshot). Only first character lower case and rest upper also works (4th screenshot).

To Reproduce
Steps to reproduce the behavior:

  1. Create a hybrid AKS cluster from the Azure portal.
  2. Provide necessary details and info and go to the node pools option.
  3. See screenshots for more info

Expected behavior
When case sensitivity matters make sure it is consistent and uppercase is not allowed at all. Or leave it as is and let the code in the background handle the changing of characters to all lower case. This is odd behavior.

Screenshots
image
image
image
image

Environment (please complete the following information):

  • OS: Azure Stack HCI OS 22H2
  • Browser: Edge
  • Version: 112.0.1722.64
  • AKS-HCI Version: Resource bridge version 1.0.11
  • Kubernetes Version: (v1.22.11)
  • AZ CLI Version: "azure-cli": "2.46.0",
    "azure-cli-core": "2.46.0",
    "azure-cli-telemetry": "1.0.8",
    "extensions": {
    "arcappliance": "0.2.30",
    "azurestackhci": "0.2.6",
    "customlocation": "0.1.3",
    "hybridaks": "0.2.1",
    "k8s-extension": "1.4.0"
    }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants