-
Notifications
You must be signed in to change notification settings - Fork 336
Beyla 2.2 #3610
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
💻 Deploy preview deleted. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Sweet!
I think that this PR should be targeted at the Otel upgrade PR no? |
yeah, but i wanted to open this PR to see if the integration tests are passing (and is not the case actually!) |
@@ -262,6 +274,7 @@ the instrumented processes need to match all the selector properties. | |||
| `namespace` | `string` | Regular expression of Kubernetes Namespaces to match. | `""` | no | | |||
| `owner_name` | `string` | Regular expression of Kubernetes owners of running Pods to match. | `""` | no | | |||
| `pod_labels` | `map(string)` | Key-value pairs of labels with keys matching Kubernetes Pods with the provided value as regular expression. | `{}` | no | | |||
| `pod_annotations` | `map(string)` | Key-value pairs of labels with keys matching Kubernetes annotations with the provided value as regular expression. | `{}` | no | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| `pod_annotations` | `map(string)` | Key-value pairs of labels with keys matching Kubernetes annotations with the provided value as regular expression. | `{}` | no | | |
| `pod_annotations` | `map(string)` | Key-value pairs of labels with keys matching Kubernetes annotations with the provided value as regular expression. | `{}` | no | |
One REALLY small spacing suggestion. If it's not included in this PR it's not a big deal :-)
PR Description
PR Checklist