Skip to content

Conversation

@hiasr
Copy link
Contributor

@hiasr hiasr commented Oct 10, 2025

Hi! This PR fixes two things to improve the infrastructure context panel.

  1. Change the CPU field used from utilization to usage, since utilization has been deprecated
  2. Make the infrastructure panel work with JSON-based schemas

Feel free to make suggestions about a different approach, I wasn't sure about the current idiomatic way to handle the JSON<>Map distinction.

@changeset-bot
Copy link

changeset-bot bot commented Oct 10, 2025

🦋 Changeset detected

Latest commit: 8a1f61e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@hyperdx/app Patch
@hyperdx/api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Oct 10, 2025

Someone is attempting to deploy a commit to the HyperDX Team on Vercel.

A member of the Team first needs to authorize it.

@hiasr hiasr requested a review from wrn14897 October 27, 2025 08:15
@vercel
Copy link

vercel bot commented Nov 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
hyperdx-v2-oss-app Ready Ready Preview Comment Nov 9, 2025 5:41am

},
);

// Normalize resource and event attributes to always use flat keys for both JSON and Map columns
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fix makes sense to me. We normalize the attributes structure upstream so that the downstream chart component can render properly.

Copy link
Member

@wrn14897 wrn14897 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR looks good. We just need to fix the linting error @hiasr

@hiasr
Copy link
Contributor Author

hiasr commented Nov 8, 2025

Can't run CI, but all linting errors related to the files I changed look fixed locally.

Copy link
Member

@wrn14897 wrn14897 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution!

@wrn14897 wrn14897 merged commit c5cb1d4 into hyperdxio:main Nov 9, 2025
8 of 10 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.

2 participants