Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

Version 1.2.6 shows "Upgrade to version 1.2.0 of @sentry/profiling-node to enable memory usage collection" in UI #210

Open
3 tasks done
WillSmithTE opened this issue Nov 17, 2023 · 3 comments

Comments

@WillSmithTE
Copy link

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

SDK Version

1.2.6

Link to Sentry event

https://ourlit-4.sentry.io/profiling/profile/marketplace/292b65b916b9426d9f92024f34cf425b/flamegraph

What environment is your node script running in?

Remix, Express, Node 20, ECS

How is your code deployed and bundled?

we build in a dockerfile from a github action with remix build then node ./server.js

Steps to Reproduce

This is how I init sentry

export function initMonitoring() {
	Sentry.init({
		dsn: sentryDsn,
		tracesSampleRate: 1,
		profilesSampleRate: 1,

		integrations: [
			new ProfilingIntegration(),
		],
		environment,
	})
}

Expected Result

Couldn't see it in the docs, but I guess I should see memory and cpu usage in the profiling UI

Actual Result

No memory or cpu usage

Upgrade to version 1.2.0 of @sentry/profiling-node to enable CPU usage collection

Upgrade to version 1.2.0 of @sentry/profiling-node to enable memory usage collection

Screenshot 2023-11-17 at 09 06 59
@JonasBa
Copy link
Member

JonasBa commented Feb 15, 2024

Hey, we just improved the in-app messaging around this today as it was showing a false positive. Can you tell me if you are still seeing this or if there is a different message being shown?

In short, we only collect memory and CPU metrics at a roughly 100ms interval rate, meaning any profiles < 200ms might have less than 2 metrics collected, which is not enough to render a chart

@TJEvans
Copy link

TJEvans commented Feb 15, 2024

The sample, 16ms duration, that was giving me this error before is now saying "Profile duration was too short to collect enough metrics" which aligns with your statement. I checked some longer samples, >200 ms, and am seeing some with "Profile has empty measurements" and other with data. Thank you for the update!

@magnusburton
Copy link

Hi! I'm still seeing this message. The profile I'm currently looking at is 8s. I'm on version 8.2.1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants