search

grafana / alloy

OpenTelemetry Collector distribution with programmable pipelines
https://grafana.com/oss/alloy
Apache License 2.0
1.47k stars 221 forks source link

Updated pod labels to include all labels #2071

Open jimmybchopps opened 2 weeks ago

jimmybchopps commented 2 weeks ago

PR Description

Updates the _pod.yaml file to include the same labels used in the DaemonSet/StatefulSet/Deployment to help improve traceability throughout the application. The new labels include the selectorLabels which it was previously using so this should only be adding additional labels and not removing any.

Which issue(s) this PR fixes

https://github.com/grafana/alloy/issues/2070

#issue_id

Notes to the Reviewer

PR Checklist

jimmybchopps commented 2 weeks ago

@petewall Is there anything specific I need to get this PR to continue its workflow? It's been stuck for a couple of days now at the continuous-integration step and when I click Details it just says to contact the repo admin. :( If there's something I'm missing from my PR, I'm more than happy to address that. Thanks!!!

mattdurham commented 2 weeks ago

I approved and kicked off the build.

jimmybchopps commented 2 weeks ago

Thanks @mattdurham!!! It looks like I needed to run a script to update the tests. I'm guessing that's why it failed at the previous step, but I think it should be good now.

jimmybchopps commented 1 week ago

@petewall Any chance you or someone else can take a look at this PR please?