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

Have deleteMetrics() use esJsonArrRequest() #125

Open
atheurer opened this issue Nov 19, 2024 · 2 comments
Open

Have deleteMetrics() use esJsonArrRequest() #125

atheurer opened this issue Nov 19, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@atheurer
Copy link
Contributor

I believe this is the one area of code where we still generate really large header/body for a http request. If we can switch to esJsonArrRequest() that will chop it up for us.

@atheurer atheurer self-assigned this Nov 19, 2024
@atheurer atheurer added the bug Something isn't working label Nov 19, 2024
@atheurer
Copy link
Contributor Author

example error:

  Running: crucible rm --run 326098b9-4ff7-462f-813d-2110ca473b31
  
  Checking for OpenSearch...appears to be running
  /opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/node_modules/sync-rpc/lib/index.js:167
      throw error;
      ^
  
  Error: Parse Error: Header overflow
      at extractValue (/opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/node_modules/sync-rpc/lib/index.js:165:19)
      at /opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/node_modules/sync-rpc/lib/index.js:175:12
      at request (/opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/node_modules/sync-request/lib/index.js:28:15)
      at esRequest (/opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/cdm.js:121:14)
      at /opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/cdm.js:683:16
      at Array.forEach (<anonymous>)
      at deleteDocs (/opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/cdm.js:681:12)
      at Object.deleteMetrics (/opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/cdm.js:707:5)
      at Object.<anonymous> (/opt/crucible/repos/[https:github.com](https://app.slack.com/client/E030G10V24F/github.com):perftool-incubator/CommonDataModel/queries/cdmq/delete-run.js:41:5)
      at Module._compile (node:internal/modules/cjs/loader:1369:14) {
    code: 'HPE_HEADER_OVERFLOW'
  }
  
  Node.js v20.12.2
  Error: rc=1
[11:50](https://redhat-internal.slack.com/archives/C047URV6M0R/p1732038622552259)
trying to re-run failed jobs... (edited) 

@atheurer
Copy link
Contributor Author

This is going to be addressed with #127

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Queued
Development

No branches or pull requests

1 participant