-
Notifications
You must be signed in to change notification settings - Fork 358
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(plugin-meetings): numberOfCores stats #3643
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
5443c87
to
eb182a6
Compare
eb182a6
to
0f87ce3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work Anna, thank you for handling this!
62730cc
to
0b3729c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Left an optional comment.
6dff40b
to
26a0781
Compare
packages/@webex/plugin-meetings/src/mediaQualityMetrics/config.ts
Outdated
Show resolved
Hide resolved
packages/@webex/plugin-meetings/test/unit/spec/stats-analyzer/index.js
Outdated
Show resolved
Hide resolved
341367b
to
343f687
Compare
COMPLETES # WEBEX-379500
This pull request addresses
Missing MQE stats
cpuInfo.numberOfCores
by making the following changes
Adds
cpuInfo.numberOfCores
to MQE to expose the number of logical processorsChange Type
The following scenarios where tested
I certified that
I have read and followed contributing guidelines
I discussed changes with code owners prior to submitting this pull request
I have not skipped any automated checks
All existing and new tests passed
I have updated the documentation accordingly
Make sure to have followed the contributing guidelines before submitting.