Nikola Grcevski comments

Results 173 comments of


                                            Nikola Grcevski

Include generated object files in commit history and published packages

> The tooling option could also be based on go workspaces Oh that's cool, we haven't looked at that!

Wrong client_address/server_address in application client/server metrics(http_server_request_duration_seconds_bucket)

Thanks for reporting this @sudhi1991, I think we need to split those attributes, address and name.

Wrong client_address/server_address in application client/server metrics(http_server_request_duration_seconds_bucket)

According to the OpenTelemetry semantic convention the name is preferred, which is why this change was made. We weren't able to determine the name until recently and this is simply...

Invalid metric reported on the transmit side and missing metric on the receiving service side

Thanks so much for reporting this @corneliu-calciu ! I believe we fixed number of these issues in our main branch, 1.7.0 is imminent I think, so I expect it will...

Invalid metric reported on the transmit side and missing metric on the receiving service side

Thanks again @corneliu-calciu ! I think the issue is that we are not instrumenting the server application correctly. It's marked in the DEMO as a Go service, which makes me...

Invalid metric reported on the transmit side and missing metric on the receiving service side

Hi @corneliu-calciu, I wasn't able to find the line in your log which mentions us instrumenting the catalog service, but I was able to run their example myself. I didn't...

Invalid metric reported on the transmit side and missing metric on the receiving service side

Hi @corneliu-calciu, the nodejs application will not have these limitations.

OTLP: 401 error sending trace to consumer

Thanks for all of this feedback! I think we need to improve our documentation, if you would like to contribute a PR we'll gladly take it, if not I think...

Beyla: inform that CAP_NET_ADMIN is required

Ah good catch @rafaelroquetto, I think we should add a separate issue to make sure we don't miss this part.

Add service name template

Hi @thoro, do you mind merging with main and moving this PR to ready for review so we can kick off the tests and continue with the process of merging...