No logs with App Configuration export when 'yes' flag is setted

[Philippe85]

Hello,

When we export App Configuration to App Service without the 'yes' flag as this :
az appconfig kv export ...

we can get this information message :

Exporting feature flags to properties file or appservice is currently not supported.
---------------- Key Values Preview ----------------
Target configuration already contains all key-values in source. No changes will be made.

But if we use the flag '--yes' as this :
az appconfig kv export ... --yes

we don't see the information message, which is very useful.

Could you show the same logs when '--yes' flag is setted or not ?

Thanks.

[zhenlan]

@avanigupta what's the convention in CLI about whether warning messages should be showed when --yes is provided?

[avanigupta]

Warning messages are always shown by default in CLI, unless turned off by specifying --only-show-errors option.
The purpose of --yes is to disable user confirmation prompts, but we can still log print preview.

[zhenlan]

So sounds like we should show the warning message even with --yes?

[avanigupta]

We don't show it today, but yes, we can surely add it.

[zhenlan]

Sounds good. Let's add it then as it doesn't sound intentional to be hidden with --yes.

[Philippe85]

Very good. Thanks.

[brental]

This issue also seems to affect other az appconfig commands such as az appconfig kv import. Does the fix applied in Azure/azure-cli#25560 fix the issue for all az appconfig commands or is it just for the az appconfig kv export command?

It looks like it applies to all commands but just wanted to confirm.

EDIT: I now see in the description of the fix it says it is for the az appconfig kv import command as well, but is it just for those two commands?

[albertofori]

This issue also seems to affect other az appconfig commands such as az appconfig kv import. Does the fix applied in Azure/azure-cli#25560 fix the issue for all az appconfig commands or is it just for the az appconfig kv export command?

It looks like it applies to all commands but just wanted to confirm.

EDIT: I now see in the description of the fix it says it is for the az appconfig kv import command as well, but is it just for those two commands?

Hello @brental As can be seen in the description, this fix updates the appconfig kv import and export commands only as these are the commands for which the logging update is applicable.

[brental]

Okay, thanks for getting back to me @albertofori . And thanks for the fix as well, it will be useful to have that logging on import.

[zhenlan]

The Fix has been merged.