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

Fix Monitor Query tests #17353

Closed
joshfree opened this issue Aug 30, 2021 · 2 comments
Closed

Fix Monitor Query tests #17353

joshfree opened this issue Aug 30, 2021 · 2 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Monitor Monitor, Monitor Ingestion, Monitor Query

Comments

@joshfree
Copy link
Member

tracking issue to investigate and fix the 2 unique consistently failing tests in js - monitorquery - tests

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1071008&view=ms.vss-test-web.build-test-results-tab

https://dev.azure.com/azure-sdk/internal/_test/analytics?definitionId=2766&contextType=build

  • LogsQueryClient live tests Ingested data tests (can be slow due to loading times) "before all" hook for "queryLogs (last day)"
  • LogsQueryClient live tests queryLogsBatch with types
@joshfree joshfree added Client This issue points to a problem in the data-plane of the library. Monitor - Query labels Aug 30, 2021
@joshfree joshfree added this to the [2021] September milestone Aug 30, 2021
@KarishmaGhiya
Copy link
Member

LogsQueryClient live tests queryLogsBatch with types - the failures are happening due to delay in permissions getting propogated. Two ways to fix this -

  • Use some delay so that the tests start after the permissions have propogated or
  • Run some other tests in the meanwhile till the permissions have been propogated

@KarishmaGhiya
Copy link
Member

KarishmaGhiya commented Sep 10, 2021

LogsQueryClient live tests Ingested data tests (can be slow due to loading times) "before all" hook for "queryLogs (last day)" - failure due to @azure/monitor-open-telemetry-exporter - opening an issue for them to track this - #17556

@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
Projects
None yet
Development

No branches or pull requests

4 participants