Azure / bicep-registry-modules

Bicep registry modules
MIT License
522 stars 368 forks source link

[Bug Report]: 'aks-run-helm' test case are failed. additional paramater needed #776

Closed apc-kamezaki closed 8 months ago

apc-kamezaki commented 11 months ago

Module path

/modules/deployment-scripts/aks-run-helm

Bicep version

0.24.24

Describe the bug

test code did not succeed 'helm install' because of some reasons

To reproduce

Execute deployment-scripts/aks-run-helm/main.test.bicep and check the output of each test templates

Code snippet

$ az deployment group create --resource-group ${RESOURCE_GROUP} --template-file main.test.bicep

### Relevant log output

```Shell
Result of 'sampleHelmChart'

"logs": "\"azure-marketplace\" has been added to your repositories\nHang tight while we grab the latest from your chart repositories...\n...Successfully got an update from the \"azure-marketplace\" chart repository\nUpdate Complete. ⎈Happy Helming!⎈\nError: query: failed to query with labels: secrets is forbidden: User \"6a2d0f68-1eb8-41e0-93d9-1754298c077d\" cannot list resource \"secrets\" in API group \"\" in the namespace \"default\": User does not have access to the resource in Azure. Update role assignment to allow access.\n",

Result of helmContour

"logs":"\"bitnami\" has been added to your repositories\nHang tight while we grab the latest from your chart repositories...\n...Successfully got an update from the \"bitnami\" chart repository\nUpdate Complete. ⎈Happy Helming!⎈\nError: unknown flag: --create_namespace\n",

Result of milvus

"logs":"\"milvus\" has been added to your repositories\nHang tight while we grab the latest from your chart repositories...\n...Successfully got an update from the \"milvus\" chart repository\nUpdate Complete. ⎈Happy Helming!⎈\nError: query: failed to query with labels: secrets is forbidden: User \"cbb0247f-5534-4f4a-884f-69e968eb0b00\" cannot list resource \"secrets\" in API group \"\" in the namespace \"default\": User does not have access to the resource in Azure. Update role assignment to allow access.\n",
ChrisSidebotham commented 8 months ago

Dear @apc-kamezaki , Azure Verified Modules (AVM) will become the single Microsoft standard for Bicep modules. Transitioning to AVM, requests for new features or bug fixes will need to be submitted against the appropriate AVM module via AVM Module Issues.

For bugs or new features with existing modules in the /modules directory of this repository, we encourage you to first try the AVM equivalent module if available and if the module is lacking any features, you can file an AVM Module Issue. If the module is not available yet, you can file a new AVM Module Proposal.

See this informational notice for more details on the upcoming changes.

Thank you for your understanding!

microsoft-github-policy-service[bot] commented 8 months ago

[!NOTE] The "Type: AVM :a: :v: :m:" label was added as per ITA08BCP.