build(chart): update statsd-exporter to 0.28.0

[Lee-W]

Why

https://github.com/apache/airflow/actions/runs/11828396054/job/32960017155

What

build(chart): update statsd-exporter to 0.28.0 #44009

---

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

[jedcunningham]

Need a newsfragment for it, otherwise LGTM.

[Lee-W]

Need a newsfragment for it, otherwise LGTM.

Just added! Thanks!

[jedcunningham]

Oops, actually, it needs to go in chart/newsfragments instead.

[jedcunningham]

(@Lee-W I'll give it a green so you can merge once you've fixed it)

[Lee-W]

Oops, actually, it needs to go in chart/newsfragments instead.

ah got it, didn't notice there's this folder. let me fix it now

[eladkal]

Why this PR targets v2-10-test branch?
Helm chart is released from main branch

[Lee-W]

Why this PR targets v2-10-test branch? Helm chart is released from main branch

Hmmm... this is just for fixing the CI failure on v2-10-test

[eladkal]

Why this PR targets v2-10-test branch? Helm chart is released from main branch

Hmmm... this is just for fixing the CI failure on v2-10-test

Then why the newsfragment? I think me and Jed see this as feature not an internal fix to the CI.
Is this already avaliable in main branch and this is just the "chery pick" to fix the CI?

[Lee-W]

Why this PR targets v2-10-test branch? Helm chart is released from main branch

Hmmm... this is just for fixing the CI failure on v2-10-test

Then why the newsfragment? I think me and Jed see this as feature not an internal fix to the CI. Is this already avaliable in main branch and this is just the "chery pick" to fix the CI?

Just notice there's indeed a PR for this on the main branch #43393, but it was not backported

[eladkal]

Why this PR targets v2-10-test branch? Helm chart is released from main branch

Hmmm... this is just for fixing the CI failure on v2-10-test

Then why the newsfragment? I think me and Jed see this as feature not an internal fix to the CI. Is this already avaliable in main branch and this is just the "chery pick" to fix the CI?

Just notice there's indeed a PR for this on the main branch #43393, but it was not backported

Ok then we need this PR against v2-10-test to be only backport and new pr against main with the newsfragment

[Lee-W]

Why this PR targets v2-10-test branch? Helm chart is released from main branch

Hmmm... this is just for fixing the CI failure on v2-10-test

Then why the newsfragment? I think me and Jed see this as feature not an internal fix to the CI. Is this already avaliable in main branch and this is just the "chery pick" to fix the CI?

Just notice there's indeed a PR for this on the main branch #43393, but it was not backported

Ok then we need this PR against v2-10-test to be only backport and new pr against main with the newsfragment

Sounds good. Then I think I don't need to do anything for this one, but will need to create an new one to main (for newsfragment)

[eladkal]

No need for newsfragment on this PR after the fix in main

[Lee-W]

it seems to be fixed already. close this one