datadog parsing error for dagrun.schedule_delay since it is not passed in float type

[coinzerge]

Apache Airflow version

2.1.2

Operating System

Gentoo/Linux

Versions of Apache Airflow Providers

No response

Deployment

Docker-Compose

Deployment details

No response

What happened

In datadog-agent logs, got parsing error

[ AGENT ] 2021-09-29 03:20:01 UTC | CORE | ERROR | (pkg/dogstatsd/server.go:411 in errLog) | Dogstatsd: error parsing metric message '"airflow.dagrun.schedule_delay.skew:<Period [2021-09-29T03:20:00+00:00 -> 2021-09-29T03:20:00.968404+00:00]>|ms"': could not parse dogstatsd metric value: strconv.ParseFloat: parsing "<Period [2021-09-29T03:20:00+00:00 -> 2021-09-29T03:20:00.968404+00:00]>": invalid syntax

What you expected to happen

since datadog agent expects a float, see https://github.com/DataDog/datadog-agent/blob/6830beaeb182faadac40368d9d781b796b4b2c6f/pkg/dogstatsd/parse.go#L119, the schedule_delay should be a float instead of timedelta.

How to reproduce

No response

Anything else

No response

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[boring-cyborg]

Thanks for opening your first issue here! Be sure to follow the issue template!

[potiuk]

Maybe you would like to take a stab and fix it @coinzerge ? Seems like really simple fix and good first issue?

[lostkamp]

Hello,
I'm willing to work on this. Would be my first contribution though, so I might need some guidance.

[potiuk]

Feel free!

[lostkamp]

thanks @potiuk
Sorry, I didn't realize that there is already a PR open for this issue. Should I fork the repo from @coinzerge and try to fix the tests?

[potiuk]

why not - but try talk to @coinzerge first maybe :)

[lostkamp]

Sorry, I'm a Github noob, but it seems like you cannot send DMs on Github. So I'm not sure how I am supposed to contact @coinzerge . I have submitted a PR now that includes the changes from the previos PR and fixes the test (hopefully).

[potiuk]

That's enough if you make a comment, if @coinzerge does not respond in a day or two - just take it over.