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.

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

[Monitor - Query] test failing in nightly runs #16795

Closed
v-jiaodi opened this issue Aug 6, 2021 · 1 comment
Closed

[Monitor - Query] test failing in nightly runs #16795

v-jiaodi opened this issue Aug 6, 2021 · 1 comment
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Monitor Monitor, Monitor Ingestion, Monitor Query test-reliability Issue that causes tests to be unreliable

Comments

@v-jiaodi
Copy link
Member

v-jiaodi commented Aug 6, 2021

Monitor - Query nightly test runs are failing with:

Error message:
Expected values to be loosely deep-equal: { body: { query: 'the kusto query', timespan: 'PT5M' }, headers: { Prefer: 'wait=100,include-statistics=true' }, id: '1', workspace: 'the primary workspace id' } should loosely deep-equal { body: { additionalWorkspaces: [ 'additionalWorkspace', 'resourceId1' ], query: 'the kusto query', timespan: 'PT5M' }, headers: { Prefer: 'wait=100,include-statistics=true' }, id: '1', workspace: 'the primary workspace id' }

Stack trace:
AssertionError [ERR_ASSERTION]: Expected values to be loosely deep-equal:
{
body: {
query: 'the kusto query',
timespan: 'PT5M'
},
headers: {
Prefer: 'wait=100,include-statistics=true'
},
id: '1',
workspace: 'the primary workspace id'
}
should loosely deep-equal
{
body: {
additionalWorkspaces: [
'additionalWorkspace',
'resourceId1'
],
query: 'the kusto query',
timespan: 'PT5M'
},
headers: {
Prefer: 'wait=100,include-statistics=true'
},
id: '1',
workspace: 'the primary workspace id'
}
at Context. (/Users/runner/work/1/s/sdk/monitor/monitor-query/test/internal/unit/modelConverters.unittest.spec.ts:89:14)
at callFn (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runnable.js:374:21)
at Test.Runnable.run (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runnable.js:361:7)
at Runner.runTest (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runner.js:619:10)
at /Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runner.js:745:12
at next (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runner.js:536:14)
at /Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runner.js:546:7
at next (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runner.js:448:14)
at Immediate._onImmediate (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/mocha@7.2.0/node_modules/mocha/lib/runner.js:514:5)
at processImmediate (internal/timers.js:464:21)

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 Aug 6, 2021
@v-xuto v-xuto added Client This issue points to a problem in the data-plane of the library. Monitor - Query test-reliability Issue that causes tests to be unreliable and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Aug 6, 2021
@maorleger maorleger assigned KarishmaGhiya and unassigned maorleger Aug 9, 2021
@KarishmaGhiya
Copy link
Contributor

Fixed this with the last release

@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
@scottaddie scottaddie added Monitor Monitor, Monitor Ingestion, Monitor Query and removed Monitor - Query labels Sep 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Monitor Monitor, Monitor Ingestion, Monitor Query test-reliability Issue that causes tests to be unreliable
Projects
None yet
Development

No branches or pull requests

5 participants