Adrian Cole
Adrian Cole
@TAOXUY this might be interesting to you. I'm happy to continue and do metrics after this once merged
Thanks for the tips and attention folks. I will pare down per guidance tomorrow utc+8
One note is that the minimum viable implementation must include a service name from somewhere. If the primary aggregation field is missing, we still require an intermediate collector to put...
ahh I better understand now. What I will do is just use resource_attributes here because we can, and indeed don't need env support yet. So there's no real conflict. sorry...
ok I think I got all the feedback, also updated the PR desc with new screenshots which are the same, just proving still works
sorry took me a while to figure out how to adjust for coverage etc 96% is a tough bar until found the exceptions file ;)
rando joining in.. the x/sys decision was to avoid lockups but this angle wasn't a part of the discussion, and a pragmatic way forward!
I'd suggest this path, which seems to work for me alright. If folks are keen maybe we can help close out this issue so folks can reserve energy? ```yaml -...
@alexhancock I tried to match exactly what was before so that the diff is just about API updates. Lemme know if we need anything else on this.
@alexhancock fyi I'm gonna drop all your historical comments per instructions so we can move forward.