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.

Sign up for GitHub

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

Jump to bottom

Azure App Configuration Readme Issue #26034

Open
zzhxiaofeng opened this issue May 30, 2023 · 6 comments
Open

Azure App Configuration Readme Issue #26034

zzhxiaofeng opened this issue May 30, 2023 · 6 comments
Assignees
@HarshaNalluru
Labels
App Configuration Azure.ApplicationModel.Configuration Client This issue points to a problem in the data-plane of the library. Docs test-manual-pass

Comments

@zzhxiaofeng
Copy link
Contributor

zzhxiaofeng commented May 30, 2023 •
edited by v-xuto
Loading

Section link
GetImage (8)
Reason:
image

@joheredi, @jsquire, @dw511214992 for notification.
@github-actions github-actions bot added ARM Client This issue points to a problem in the data-plane of the library. needs-team-triage Workflow: This issue needs the team to triage. labels May 30, 2023
@v-xuto v-xuto added Docs App Configuration Azure.ApplicationModel.Configuration test-manual-pass and removed ARM labels May 30, 2023
@xirzec xirzec removed the needs-team-triage Workflow: This issue needs the team to triage. label May 30, 2023
@HarshaNalluru HarshaNalluru mentioned this issue May 30, 2023
Closed
@HarshaNalluru
Copy link
Member

HarshaNalluru commented May 30, 2023 •
edited
Loading

I've tried the command, doesn't seem to work..
I hit.. "Failed to find the App Configuration store '<resource-name>'." error.

@zzhxiaofeng
Copy link
Contributor Author

@HarshaNalluru I have tried the command with the latest version (2.49.0) of az cli and it runs successfully. Are you currently using the latest version? If not, please update to latest and try again.

@HarshaNalluru
Copy link
Member

HarshaNalluru commented May 31, 2023 •
edited
Loading

Oh, realized the problem, a different subscription was set.
Also, upgraded to 2.49.0, though unrelated to the problem.

Both commands are working fine for me.

PS /workspaces/azure-sdk-for-js> az appconfig credential list -g resource-group -n account-name --query "([?name=='Primary'].connectionString)[0]"      
"Endpoint=https://account-name.azconfig.io;Id=<redacted>;Secret=<redacted>"

PS /workspaces/azure-sdk-for-js> az appconfig credential list -g resource-group -n account-name --query "[?name=='Primary'].connectionString"
[
  "Endpoint=https://account-name.azconfig.io;Id=<redacted>;Secret=<redacted>"
]

The difference being array or not..

@zzhxiaofeng
Copy link
Contributor Author

@HarshaNalluru Do you run this command in local on windows system? If not, you can try this command in local on windows. We run this command by this way and the error still exists.

@HarshaNalluru
Copy link
Member

HarshaNalluru commented Jun 1, 2023 •
edited by xirzec
Loading

My above response is from linux, will check windows...

Okay, tested it on Windows, and it fails.

PS C:\path> az appconfig credential list -g <resource-group> -n <account-name> --query "[?name=='Primary'].connectionString"
[
  "Endpoint=https://<account-name>.azconfig.io;Id=Bl85;Secret=<redacted>"
]

PS C:\path> az appconfig credential list -g <resource-group> -n <account-name> --query "([?name=='Primary'].connectionString)[0]"
[0] was unexpected at this time.
C:\path>  "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\wbin\\..\python.exe" -IBm azure.cli appconfig credential list -g <resource-group> -n <account-name> --query ([?name=='Primary'].connectionString)[0]

PS C:\path>

Sounds like a bug in az cli it seems.

@HarshaNalluru HarshaNalluru mentioned this issue Jun 1, 2023
Open
@HarshaNalluru
Copy link
Member

Logged an issue at Azure/azure-cli#26583

azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue Oct 25, 2023
@junjiezhang1997 @zqingqing1
@pineapplethevoyager @matthchr @MaxHorstmann @karataliu @helayoty @ganeshkumarashok
CodeGen from PR 26313 in Azure/azure-rest-api-specs
1d49cdb 
[Hub Generated] Review request for Microsoft.ContainerService/aks to add version preview/2023-09-02-preview (Azure#26313)

* Adds base for updating Microsoft.ContainerService/aks from version preview/2023-08-02-preview to version 2023-09-02-preview

* Updates readme

* Updates API version in new specs and examples

* OSSKU supports WindowsAnnual option (Azure#26120)

* add VirtualMachines agent pool type (Azure#26034)

* GPU Profile and Properties for AKS Windows (and Linux) GPU Feature (Azure#26072)

* switch file and fix typos

* Update specification/containerservice/resource-manager/Microsoft.ContainerService/aks/preview/2023-09-02-preview/managedClusters.json

Co-authored-by: Matthew Christopher <matthchr@users.noreply.github.com>

---------

Co-authored-by: Matthew Christopher <matthchr@users.noreply.github.com>

* add AgentPoolUpgradeSettings.nodeSoakDurationInMinutes (Azure#26046)

* Update format for trustedaccess sourceResourceId (Azure#26123)

* update readmes (Azure#26126)

* Add NodeAutoProvisioning to AKS preview API (Azure#26047)

* Add AIToolchainOperator to AKS preview API (Azure#26151)

* Add AIToolchainOperator to AKS preview API

Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com>

* Remove custom name from the PR

Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com>

* prettier fix

---------

Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com>
Co-authored-by: Heba Elayoty <hebaelayoty@gmail.com>

* Add ArtifactStreamingProfile and Enabled property  (Azure#26144)

* add ArtifactStreamingProfile to AKS API

* reduce repetition

* Update specification/containerservice/resource-manager/Microsoft.ContainerService/aks/preview/2023-09-02-preview/managedClusters.json

---------

Co-authored-by: FumingZhang <81607949+FumingZhang@users.noreply.github.com>

* fix typo (Azure#26152)

* upgrade common types (Azure#26232)

* rename enum to nodeOSUpgradeChannel (Azure#26366)

---------

Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com>
Co-authored-by: Junjie Zhang <109257558+junjiezhang1997@users.noreply.github.com>
Co-authored-by: Qingqing <qizhe@microsoft.com>
Co-authored-by: Chris Lopez <91150577+pineapplethevoyager@users.noreply.github.com>
Co-authored-by: Matthew Christopher <matthchr@users.noreply.github.com>
Co-authored-by: Max Horstmann <MaxHorstmann@users.noreply.github.com>
Co-authored-by: Dong Liu <doliu@microsoft.com>
Co-authored-by: Heba Elayoty <hebaelayoty@gmail.com>
Co-authored-by: Ganeshkumar Ashokavardhanan <35557827+ganeshkumar5699@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HarshaNalluru HarshaNalluru

Labels
App Configuration Azure.ApplicationModel.Configuration Client This issue points to a problem in the data-plane of the library. Docs test-manual-pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[App Config] Update README.md Azure/azure-sdk-for-js
4 participants
@xirzec @HarshaNalluru @zzhxiaofeng @v-xuto