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

[Communication - Identity] CommunicationIdentityClient [Playback/Live] [AAD] successfully creates a user failing in nightly runs #18360

Closed
v-jiaodi opened this issue Oct 26, 2021 · 4 comments
Closed

[Communication - Identity] CommunicationIdentityClient [Playback/Live] [AAD] successfully creates a user failing in nightly runs #18360

v-jiaodi opened this issue Oct 26, 2021 · 4 comments
Assignees
@petrsvihlik @martinbarnas-ms
Labels
Client This issue points to a problem in the data-plane of the library. Communication test-reliability Issue that causes tests to be unreliable

Comments

@v-jiaodi
Copy link
Member

Communication - Identity nightly test runs are failing with:

Error message:
EnvironmentCredential authentication failed.(status code 400). More details: Cannot read properties of undefined (reading 'error')

Stack trace:
AuthenticationError: EnvironmentCredential authentication failed.(status code 400).
More details:
Cannot read properties of undefined (reading 'error')
at D:\a_work\1\s\common\temp\node_modules.pnpm@azure+identity@2.0.0-beta.6\node_modules@azure\identity\src\credentials\environmentCredential.ts:153:39
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at trace (D:\a_work\1\s\common\temp\node_modules.pnpm@azure+identity@2.0.0-beta.6\node_modules@azure\identity\src\util\tracing.ts:37:20)
at DefaultAzureCredential.getToken (D:\a_work\1\s\common\temp\node_modules.pnpm@azure+identity@2.0.0-beta.6\node_modules@azure\identity\src\credentials\chainedTokenCredential.ts:67:17)
at tryGetAccessToken (D:\a_work\1\s\sdk\core\core-http\src\policies\bearerTokenAuthenticationPolicy.ts:82:26)
at beginRefresh (D:\a_work\1\s\sdk\core\core-http\src\policies\bearerTokenAuthenticationPolicy.ts:93:35)
at BearerTokenAuthenticationPolicy.sendRequest (D:\a_work\1\s\sdk\core\core-http\src\policies\bearerTokenAuthenticationPolicy.ts:249:25)
at ThrottlingRetryPolicy.sendRequest (D:\a_work\1\s\sdk\core\core-http\src\policies\throttlingRetryPolicy.ts:53:22)
at IdentityRestClient.sendOperationRequest (D:\a_work\1\s\sdk\core\core-http\src\serviceClient.ts:519:23)
at CommunicationIdentityClient.createUser (D:\a_work\1\s\sdk\communication\communication-identity\src\communicationIdentityClient.ts:84:28)
at Context. (D:\a_work\1\s\sdk\communication\communication-identity\test\public\communicationIdentityClient.spec.ts:38:49)

For more details check here:
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Oct 26, 2021
@maorleger
Copy link
Member

@v-jiaodi @v-xuto thank you for your diligence! I know @v-xuto usually creates labels and assignees - looks like this might have been missed or filtered out? I will correct it, just wanted to let you know 😄

@maorleger maorleger added Client This issue points to a problem in the data-plane of the library. Communication test-reliability Issue that causes tests to be unreliable labels Oct 26, 2021
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Oct 26, 2021
@petrsvihlik
Copy link
Contributor

petrsvihlik commented Oct 26, 2021 •
edited
Loading

@martinbarnas-ms can you please take a look? it's a warning so we might have missed it because the builds show up as green. thanks

@v-xuto
Copy link
Member

v-xuto commented Oct 27, 2021

@maorleger - Sorry, I missed it.

@petrsvihlik
Copy link
Contributor

this was a temporary error that hasn't appeared again. currently, INT tests are failing due to a problem in the chat gateway (a service that we're calling). resolution is in progress.

azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue Apr 14, 2022
@bcho @zqingqing1
@yizhang4321 @wenxuan0923 @ZeroMagic @alexeldeib
CodeGen from PR 18633 in Azure/azure-rest-api-specs
c16c82c 
Review request for Microsoft.ContainerService to add version 2022-03-02-preview (Azure#18633)

* Adds base for updating Microsoft.ContainerService from version preview/2022-02-02-preview to version 2022-03-02-preview

* Updates readme

* Updates API version in new specs and examples

* feat: add `/rotateServiceAccountSigningKeys` API (Azure#18359)

* feat: add `/rotateServiceAccountSigningKeys` API

* chore: remove duplicated description

* feat: add `workloadIdentity` settings to `SecurityProfile` (Azure#18360)

* fix: update example name (Azure#18382)

* add creationData to mc data (Azure#18414)

* add creationData to mc data

* fix test

* fix format

* Add API properties and example JSON for Web App Routing of IngressProfile (Azure#18564)

* Add API properties and example JSON for Web App Routing of IngressProfile.

* Add a ending period for description to match the style in all other "descriptions" in the same file.

* Update readmes for the 2022-03-02-preview dev branch of container service (Azure#18358)

* update readme

* update sdk readmes

* Agentpool alias minor version 2022-03-02-preview (Azure#18381)

* Add field currentOrchestratorVersion to support Agentpool alias minor version

* Add new exmaple for alias minor version

* Remove example for PrettierCheck

* Fix typo

* Latest patch version supported is 1.22.7 at the moment

* Address comments - refine descriptions for fields

* feat: add ManagedCluster StorageProfile in 0302preview (Azure#18590)

Signed-off-by: Ji An Liu <jiliu8@microsoft.com>

* Swagger change for ignore-pod-disruption-budget (Azure#18548)

* Swagger change for ignore-pod-disruption-budget

* Change ignorePodDisruptionBudget to string in example file.

* Change ignorePodDisruptionBudget to boolean type.

* add effectiveNoProxy for AKS (Azure#18544)

* add effectiveNoProxy for AKS

* add to correct api

* fix lowercase -> uppercase O

* spelling

* Replace common type definitions with references since 2022-03-02-preview for Microsoft.ContainerService  (Azure#18567)

* replace Resource

* replace SystemData

* replace parameters

* change track2 to python

Co-authored-by: hbc <bcxxxxxx@gmail.com>
Co-authored-by: Qingqing <qizhe@microsoft.com>
Co-authored-by: Yi Zhang <yizhang5@microsoft.com>
Co-authored-by: Thalia Wang <58485997+wenxuan0923@users.noreply.github.com>
Co-authored-by: Ji'an Liu <jiliu8@microsoft.com>
Co-authored-by: Tong Chen <45081443+tonche@users.noreply.github.com>
Co-authored-by: Ace Eldeib <aleldeib@microsoft.com>
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@petrsvihlik petrsvihlik

@martinbarnas-ms martinbarnas-ms

Labels
Client This issue points to a problem in the data-plane of the library. Communication test-reliability Issue that causes tests to be unreliable
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@maorleger @petrsvihlik @v-xuto @martinbarnas-ms @v-jiaodi