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

Telemetry tables are not requesting the correct sort order for limited mode #7636

Closed
2 of 7 tasks
davetsay opened this issue Mar 21, 2024 · 2 comments · Fixed by #7645
Closed
2 of 7 tasks

Telemetry tables are not requesting the correct sort order for limited mode #7636

davetsay opened this issue Mar 21, 2024 · 2 comments · Fixed by #7645
Assignees
Labels
severity:critical type:bug verified Tested or intentionally closed
Milestone

Comments

@davetsay
Copy link
Contributor

davetsay commented Mar 21, 2024

Summary

In descending mode limited the x rows, the earliest x rows are being returned from the historical request.

Expected vs Current Behavior

When sorted descending and limiting to x rows, I would expect the x rows requested to be the latest. Instead I am getting the earliest in the time range.

Workaround

  1. Use unlimited mode

Steps to Reproduce

  1. Navigate to a telemetry table that has telemetry
  2. Ensure it is in limited mode (toggle at bottom right of telemetry table)
  3. Set time conductor to a large range (1 hr or more)
  4. Hover mouse over pause button
  5. Refresh the browser
  6. Pause as soon as telemetry request is finished (pretty much when pause button becomes visible)
  7. Observe the timestamps for the data is from the beginning of the time range, not the end

Environment

  • Open MCT Version:
  • Deployment Type:
  • OS:
  • Browser:

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

@jvigliotta
Copy link
Contributor

Testing

  • Follow the repro steps, make sure that the results that come in do not have a large gap
  • make sure the request that goes out has order=desc url param

@rukmini-bose
Copy link
Contributor

Verified Testathon 4/2/2024

@ozyx ozyx added the verified Tested or intentionally closed label Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity:critical type:bug verified Tested or intentionally closed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants