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

Log all loads + updates #24

Merged
merged 19 commits into from
Dec 6, 2024
Merged

Log all loads + updates #24

merged 19 commits into from
Dec 6, 2024

Conversation

bmacnaughton
Copy link
Contributor

@bmacnaughton bmacnaughton commented Nov 24, 2024

  • added a log-all-loads option, CSI_RM_LOG_ALL_LOADS=true, checked as 'true'
    • does not have tests for this
  • update tests to work with node ESM loader changes (invoke with --import unless node version < 18.19.0.

Note that the updates to work with ESM just lets the tests pass; it does not work quite right for node versions 18.19.0 up to 20.0.0 (really 20.6.0 because node's ESM loader was fubar from 20.0.0 to 20.6.0). And it will miss modules loaded with import statements or the import function until ESM support is added.

testing changes

  • random ports when spawning servers
  • move checks to class-based implementation
  • add ability to specify required entries for routes
  • remove any specific ordering requirements (this is to make tests pass when used with --import)

N.B.:

this fixes the testing ordering problems and makes tests much more maintainable. I am adding ESM support, based on this branch, and will rebase it on main if this is merged.

@bmacnaughton bmacnaughton marked this pull request as draft November 24, 2024 17:28
@bmacnaughton bmacnaughton marked this pull request as ready for review December 3, 2024 00:18
- no longer support --require (only supports node 18.19.0+)
- add primordials for a couple methods that are used in the patcher
- add cpuUser and cpuSystem (monotonically increasing values) to averages
- add route-metric's version number to the header
- fix first/last times
- add express.mjs to the test matrix
- pass relevant env to loader thread
- remove artifact "metrics" usage (replaced by "routes")
@bmacnaughton bmacnaughton merged commit b3842a5 into main Dec 6, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant