Azure / azure-cli-extensions

Public Repository for Extensions of Azure CLI.
https://docs.microsoft.com/en-us/cli/azure
MIT License
382 stars 1.21k forks source link

[Containerapp] `az containerapp auth update`: Fix AuthConfigSecretRefNotFound #7679

Closed Greedygre closed 4 months ago

Greedygre commented 4 months ago

This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

az containerapp auth update

When set_secrets during update settings, need to wait until finish, otherwise it will throw error about secret not found in containerapp , for example:


src\azure-cli\azure\cli\command_modules\containerapp\custom.py:4214: in update_aad_settings
   handle_raw_exception(e)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _  

e = HTTPError('Bad Request({"error":{"code":"AuthConfigSecretRefNotFound","message":"Secret with name \'microsoft-provider-authentication-secret\' is not found in configuration of container app \'containerapp-auth4chpvat\'."}})')

General Guidelines

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

azure-client-tools-bot-prd[bot] commented 4 months ago
️✔️Azure CLI Extensions Breaking Change Test >
> ️✔️Non Breaking Changes > > >
azure-client-tools-bot-prd[bot] commented 4 months ago

Hi @Greedygre, Please write the description of changes which can be perceived by customers into HISTORY.rst. If you want to release a new extension version, please update the version in setup.py as well.

yonzhan commented 4 months ago

Containerapp

github-actions[bot] commented 4 months ago

:warning: Release Suggestions

Module: containerapp