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 - Opentelemetry Exporter] Test OpenTelemetry Versions - Monitor failing in nightly runs #22181

Closed
v-jiaodi opened this issue Jun 10, 2022 · 2 comments · Fixed by #22297
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Monitor - Exporter Monitor OpenTelemetry Exporter Monitor Monitor, Monitor Ingestion, Monitor Query test-reliability Issue that causes tests to be unreliable

Comments

@v-jiaodi
Copy link
Member

Monitor - Opentelemetry Exporter nightly test runs are failing with:

Error message:
require() of D:\a_work\1\s\common\temp\node_modules.pnpm\execa@6.1.0\node_modules\execa\index.js from D:\a_work\1\s\sdk\monitor\monitor-opentelemetry-exporter\test-opentelemetry-versions.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from D:\a_work\1\s\common\temp\node_modules.pnpm\execa@6.1.0\node_modules\execa\package.json.
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1015:13)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Module.require (internal/modules/cjs/loader.js:887:19)
at require (internal/modules/cjs/helpers.js:74:18)
at Object. (D:\a_work\1\s\sdk\monitor\monitor-opentelemetry-exporter\test-opentelemetry-versions.js:1:15)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14) {
code: 'ERR_REQUIRE_ESM'
}
The script failed with exit code 1
##[error]Bash exited with code '1'.

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 Jun 10, 2022
@azure-sdk
Copy link
Collaborator

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.41735408,Azure.Core:0.19096695,Monitor - Exporter:0.16029927'

@v-xuto v-xuto added the Monitor Monitor, Monitor Ingestion, Monitor Query label Jun 10, 2022
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jun 10, 2022
@v-xuto v-xuto added Client This issue points to a problem in the data-plane of the library. needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. test-reliability Issue that causes tests to be unreliable labels Jun 10, 2022
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jun 10, 2022
@azure-sdk
Copy link
Collaborator

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.41735408,Azure.Core:0.19096695,Monitor - Exporter:0.16029927'

@scottaddie scottaddie added the Monitor - Exporter Monitor OpenTelemetry Exporter label Jun 15, 2022
@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.
Labels
Client This issue points to a problem in the data-plane of the library. Monitor - Exporter Monitor OpenTelemetry Exporter Monitor Monitor, Monitor Ingestion, Monitor Query test-reliability Issue that causes tests to be unreliable
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants