Saramanda9988 commented on issue #3765: URL: https://github.com/apache/hertzbeat/issues/3765#issuecomment-3451758342
Hi community, @tomsun28 I’ve been investigating this issue and found that the host value currently only includes the IP (e.g. 127.0.0.1), not the full IP:Port The MetricsData metadata contains only instancehost and instancename, without any port information.Each monitorTask has a globally unique id and name. <img width="751" height="137" alt="Image" src="https://github.com/user-attachments/assets/b07ab921-cd17-4c08-9529-fb4ae999685e" /> Given this, there seem to be a few possible approaches: 1. Modify host to include ip:port, so that the instance label directly matches Prometheus best practices. Or add a port field in metadata allowing ip:port to be reconstructed later — but these may introduce larger structural changes. 2. Use monitorTask.name as the instance label, which requires minimal modification but doesn’t fully align with Prometheus conventions. Before proceeding with a PR, I’d like to hear your thoughts, Any suggestions from the community would be greatly appreciated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@hertzbeat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@hertzbeat.apache.org For additional commands, e-mail: notifications-h...@hertzbeat.apache.org
