opentelemetry-erlang-contrib icon indicating copy to clipboard operation
opentelemetry-erlang-contrib copied to clipboard

fix: setting oban attributes

Open yordis opened this issue 1 year ago • 9 comments

  • [ ] Should I prefix everything with "oban.*" for these attributes?

yordis avatar Dec 12 '23 04:12 yordis

Should I prefix everything with "oban.*" for these attributes thou? I prefer to do

yordis avatar Dec 12 '23 05:12 yordis

Should I prefix everything with "oban.*" for these attributes thou? I prefer to do

You can add that to this. Might as well get it done since we're touching it.

bryannaegele avatar Dec 13 '23 17:12 bryannaegele

@bryannaegele done!

yordis avatar Dec 13 '23 19:12 yordis

@tsloughter @bryannaegele is there anything I need to do to get this to the finish line?

yordis avatar Dec 26 '23 15:12 yordis

@bryannaegele done, hopefully I understood the assignment

yordis avatar Dec 27 '23 19:12 yordis

@joaothallis anything else?

I am wondering how to get this to the finish line

yordis avatar Feb 03 '24 09:02 yordis

Should I prefix everything with "oban.*" for these attributes?

Reading the spec it is not specifying that other attributes should start with messasing. But the attributes that are in the spec they can not have the prefix oban.

joaothallis avatar Feb 04 '24 19:02 joaothallis

Added Unit Tests, let me know!

yordis avatar Feb 05 '24 07:02 yordis

@bryannaegele ready to go

yordis avatar Feb 06 '24 20:02 yordis

You're going to kill me but would you mind poking at this test that started failing with newer Oban after I bumped it? I think the test just needs to be updated but I'm not an Oban expert

https://github.com/open-telemetry/opentelemetry-erlang-contrib/issues/287

bryannaegele avatar Feb 14 '24 05:02 bryannaegele

@bryannaegele https://github.com/sorentwo/oban/pull/1035 it needs to be fixed in the Oban side, it was removed by mistake

yordis avatar Feb 14 '24 15:02 yordis

Awesome! Thanks for following up.

bryannaegele avatar Feb 14 '24 16:02 bryannaegele