Open Telemetry Uncategorized, 13 Dec 2022

Previous Meeting Next Meeting

⏯

youtube image

►

From YouTube: 2022-12-13 meeting

Description

cncf-opentelemetry@cncf.io's Personal Meeting Room

A

B

B

C

I'm going to say, uh we should probably give people another minute or two to join.

B

C

Yes, uh but remember to put your name in the attendees list: okay,.

D

Are you waiting for anyone.

C

I was hoping that um Tyler might join us, but if not it's fine, um we are about three minutes past the hour. uh I can kick us off. We give me a second to share.

C

That great um hi everybody uh welcome to the auto instrumentation for go Sig. um Sorry, it's uh such lightly attended today. But let's get this going, um it's kind of uh near the end of the year. uh I will put a note on that. At the end, um it looks like uh Eden sorry if I mispronounce, your name yeah.

E

Yeah, uh that's all the first two items on me, uh so I'm working on Pixar issue. If it's number 17 that.

E

um I think it's like trying to reproduce it on my machine so far, that's very without a lot of black, but getting there I think it's uh it's something in this specific Channel version that I'm trying to test it on this specific version that you are working on um and I also have an idea how to fix it.

E

So, hopefully I'll be able to to open a pull requests to fix this specific issue uh in the next few days and uh the second action item, which is us something that I'm working on is uh this launching code with uh just click the simple application?

E

If you launch the application uh with this Launcher app, it allocates a memory and then the instrumentation agent can write to this memory block, and this is how we basically do conduct propagation.

E

For this part of the code and yeah also hoping to finish it uh by the end of the week, yeah.

A

D

Don't know is there a way we can actually pair or like, uh because you are the only one who has actually working knowledge of this whole code base like somehow. If we can distribute the knowledge like either find how to do the code fix or like a pair? Well, you are doing a code fix or like something like that. I, don't know any thoughts.

E

um Yeah sure so uh we're going to the about the fury reporting on the pink. um If you can open this issue,.

E

Yeah, so uh you can I can tell you like. What's the process that I'm currently trying to do in order to understand? What's the problem so uh like the the arrow itself is, uh is in one of those lights here, if you will.

C

Scrolling me: would it be easier for you to share your screen. Yeah yeah.

A

C

B

E

Yeah, so that's basically the same log lines.

E

Yeah, so basically, that's the error output. The distribution agent gave us uh failing to instrument the grpc library and basically, uh the interesting parties. Actually this this line, which is uh every time the cannot fails to load an evpf program, I print, this ugly uh sort of assembly code of where what it expects so I opened it in a visual studio code and try to uh get a better understanding of what's going on here and basically, you can see that it's currently failing on this line.

E

Exactly uh so, that's the part of the job instrumentation that um as responsible for reading the incoming headers of the requests and.

E

It creates the.

E

But the the interesting the relevant lines, failing is this one is basically trying to read whatever this uh RBX registry into everything. It sounds like the the land of the slice. It means how many uh headers we have in the request and then we can iterate it on it later and basically read it head overhead and find a development handle, but uh it fails because uh well, it's good theory right now, but I think like the casting to to sign the integers anger and- and you can actually see it.

E

Right here, basically, it's expected, the type is context and the expected type is FB and both of them are like characters from Linux structures, but basically it means that the casting is made here. So Magnus is basically maybe some additional enforcement that they added in this recent Linux kernel that maybe we need to do some additional checks before you can cast to to an integer. So maybe that's the reason. This avpf program failed.

E

But again it's still testing all those assumptions, I'm not really sure, but I think like uh like, if you think about it in a more generic way. uh Basically, the way to debug uh nvps program errors is it like, so basically can be two types of Errors: either we can't uh we can't load the evpf program into the channel, and in this case it will look like uh uh the output. The initial posted here, which is basically all those assembly lines and not very clear messaging about uh White, fails to load it into the camera.

E

Oh, it can actually be some runtime error or something bad and the easiest way to uh to debug. Those issues is basically used. An EPF helper, which called BPF print K, it's some slack printer on the other languages, where you basically can add any print statement and print all the different variables and in which you're working on and then you can go to.

E

There is a specific file in Linux a lot of places, and here uh you can see all the BPF print case that you're doing from a pro BPF program. So let's take a way to the bug, runtime stuff, using this print statement, and then you can read it uh from this specific file or if it turns earlier it comes to load it into the kernel, uh it's harder to debug it. If you need to try to understand this uh weird error messages, so that's regarded.

E

How I think about this specification um regarding, like a better transfer of knowledge, I, I, agree and I I think I will take the next meeting that we have uh to maybe do a little bit of a deep dive of the repository and how the different structure I wanted to do it for a long time. But uh uh I don't know if it doesn't happen, so I will do it on the next meeting.

E

I think it will be much easier for other to for us to contribute to this, especially in EPF, because gold and just think most of the people on the school know.

B

E

B

D

C

Sense, what would also could be helpful um like for the specific is if YouTube could, connect and um schedule some time just to pair and working on this, because Dinesh shared the the uh reporter of this problem and and I didn't hear you're the one trying to solve it right so that that might also be helpful in in transferring the knowledge in this case.

B

um Yeah I'm not saying.

C

You have to but.

E

Yeah sure, yes, uh let me try to reproduce it on my machine and I will contact you and try to see how we can work with it together sounds good. Yeah sounds.

D

Good yeah and a huge plus one like I, think we should do a repository deep. There I think our next meeting is going to be in the next year, so start of the start of the meeting next next year. The repository should be awesome.

C

Way to start the year right, yep.

C

Excuse me, sorry, all right, um president.

D

C

Sorry I'm, how do you pronounce your name? Should I make? Yes, uh it looks like you have the next two issues. If you would want to take them away.

F

Last week, I opened it very small, pull requests it's about adding uh this new uh directory into the country and also I, think uh readme file and I had I have already approval from Tyler, but I need at least one more to to merge it, uh and uh that that's currently blocks me, because uh I would like to open another PR with the implementation of this. This tool.

C

um This actually does have that um I will I will endeavor to take a look after this meeting. Okay.

F

Thank you, and uh one more question is about, because uh is my understanding correct that I will become or I am already owner of this subdirectory.

F

And also a maintainer- or there is some kind of process to becoming maintainer of this subdirectory, how it look likes.

C

um So maintainer permissions are at a repository level: uh okay, okay, if you are not a user or an approver um which I think that might be what uh this is selling me, um then uh we should have you go through the user and or user and approver uh steps to become user. uh You I think it's a member member of hotel and then an approval. Approver of the hotel go contrib.

C

At least um I, don't know if we have something if we've split the goken trip versus the go approvers status, but uh that should give you enough permissions to have approval status on on this for PRS in this directory. Okay,.

B

C

um So I can point you, let's see um actually a community.

C

um Community membership uh are you, do you know if you're a member already or not, of the hotel, Community yeah.

F

I'm already a member, but so the only thing I'm, not a proverb of this specific directory, I, think and Country, because I I haven't contributed to this country. Yet that's.

C

Okay um yeah, so that would mean we would need you to take to take steps to uh add yourself as a contributor um I'm spacing I believe it's under the contributing uh become a contributor.

C

um You've probably already signed the CLA. Yes, but essentially we open up pull requests to that says you want to be uh added to the contributors. uh I believe we do a public requests against the community uh me Tyler other contributors to go, um approve it, and then that will get you into the proper group to become uh approver like code approver per se for GitHub, like okay.

B

C

You the right permissions. Okay,.

B

C

C

C

um Responsibilities and privileges.

F

Okay, so I will open a request for that.

B

C

um Apparently we're missing you becoming an appropriate step, but it's I think it's the same as becoming a maintainer.

C

um You know open a a PR or not a PR, but a uh a issue in the community. To saying, please add me here: got approval from me and Tyler. Okay,.

B

F

And the last thing I wanted to mention that I'm currently cleaning out a little bit to to open this new PR with uh with implementation, and that should happen, probably next to uh in next few few days, I think but first it would be good, of course, to to uh to merge the the first one with this directory and readme file.

F

So basically that's it for from my set.

C

Great um so then the last thing is: uh does anybody here want to have a a meeting on December 27th.

E

B

F

Notification, I'm not sure if I will be available.

C

uh I know personally I'm not available I believe Tyler is not available, um he might be I'm, not I'm, not.

D

Sure yeah I'm not yeah I'm not available I'm on vacation, so.

C

I think most likely I will go ahead and cancel it after this meeting.

A

C

And with that I think, that's probably the end of this meeting. Nice short one to round out the year um in other cigs. We typically like to ask questions alike. Has anybody used this? Does anybody have any stories from uh using any of the auto instrumentations.

D

No I guess I think we are still waiting on like building the first version that yeah.

C

But uh you know there might be somebody you might be using it internally.

E

Yeah I actually know about a couple of years are using it uh yeah.

C

So uh we asked those stories because that usually helps invigorate the people who are contributing. So if you can get any of them, that would be really appreciated, just any kind of small feedback, even if it's just a minute of like yeah so-and-so's using it and they're, giving the success yeah.

E

Yeah so I know also some folks from honeycomb are also using it. uh uh Trying to integrate into now. um Yeah I would try to collect the question. Thank.

C

You I appreciate it and then um is there anything else? Anybody else wants to talk about uh foreign.

C

Well, thank you all for coming um I'm going to give you a 27 minutes back or 37 minutes back have a wonderful holidays. Thank.

D