[linkerd-jaeger] Adds support for collector config merge and Deployment env#11283
Merged
adleong merged 10 commits intolinkerd:mainfrom Sep 14, 2023
Merged
[linkerd-jaeger] Adds support for collector config merge and Deployment env#11283adleong merged 10 commits intolinkerd:mainfrom
adleong merged 10 commits intolinkerd:mainfrom
Conversation
…t env Signed-off-by: Cameron Boulton <[email protected]>
Signed-off-by: Cameron Boulton <[email protected]>
20aee8c to
b8adf8f
Compare
added 3 commits
August 22, 2023 14:28
… Kubernetes components Signed-off-by: Cameron Boulton <[email protected]>
… Kubernetes components Signed-off-by: Cameron Boulton <[email protected]>
Signed-off-by: Cameron Boulton <[email protected]>
ceb65db to
51ea3c2
Compare
Signed-off-by: Cameron Boulton <[email protected]>
c089bef to
1e62567
Compare
Member
|
Thank you @iAnomaly! |
Signed-off-by: Cameron Boulton <[email protected]>
5e04241 to
c279129
Compare
added 3 commits
August 30, 2023 14:46
Signed-off-by: Cameron Boulton <[email protected]>
Signed-off-by: Cameron Boulton <[email protected]>
…collector-config-env
Member
|
Sorry for the delay. The relevant maintainers were pulled into some high-priority customer escalations. This is still in the queue to review. |
mateiidavid
approved these changes
Sep 14, 2023
Member
mateiidavid
left a comment
There was a problem hiding this comment.
🚢 📦! Thanks for the PR :)
adamshawvipps
pushed a commit
to adamshawvipps/linkerd2
that referenced
this pull request
Sep 18, 2023
…nt env (linkerd#11283) This PR changes a variety of things: 1. Changes value `collector.config` from a string to an object to support field level merging/changing without having to copy/paste the entire unchanged portion of the config. 2. Changes the default `collector.config` for sensible out of the box Linkerd tracing use-case: 1. Only `opencensus` receiver (the format `linkerd-proxy` emits when configured for tracing) 2. `resource` processor that maps the Pod name and namespace labels `linkerd-proxy` sets on injected Pods to the fields the `k8sattributes` processor expects for looking up the trace's source Pod. 3. `k8sattributes` processor for decorating all traces with K8s metadata. 3. Standardizes `image.pullPolicy` behavior across all components and defaults to absent/null `imagePullPolicy` such that the K8s cluster configuration is used. 4. Adds collector RBAC needed for `k8sattributes` to lookup trace source K8s workloads 5. Adds collector ConfigMap checksum to rollout new Pod(s) when config is changed. 6. Adds/parameterizes the collector container command with value `collector.command`. 7. Changes collector image from [opentelemetry-collector](https://github.com/open-telemetry/opentelemetry-collector) to [opentelemetry-collector-contrib](https://github.com/open-telemetry/opentelemetry-collector-contrib) which includes additional connectors, exporters, extensions, processors, receivers. 8. Adds value `collector.env` for additional collector Deployment environment variables. Signed-off-by: Cameron Boulton <[email protected]>
adamshawvipps
pushed a commit
to adamshawvipps/linkerd2
that referenced
this pull request
Sep 18, 2023
…nt env (linkerd#11283) This PR changes a variety of things: 1. Changes value `collector.config` from a string to an object to support field level merging/changing without having to copy/paste the entire unchanged portion of the config. 2. Changes the default `collector.config` for sensible out of the box Linkerd tracing use-case: 1. Only `opencensus` receiver (the format `linkerd-proxy` emits when configured for tracing) 2. `resource` processor that maps the Pod name and namespace labels `linkerd-proxy` sets on injected Pods to the fields the `k8sattributes` processor expects for looking up the trace's source Pod. 3. `k8sattributes` processor for decorating all traces with K8s metadata. 3. Standardizes `image.pullPolicy` behavior across all components and defaults to absent/null `imagePullPolicy` such that the K8s cluster configuration is used. 4. Adds collector RBAC needed for `k8sattributes` to lookup trace source K8s workloads 5. Adds collector ConfigMap checksum to rollout new Pod(s) when config is changed. 6. Adds/parameterizes the collector container command with value `collector.command`. 7. Changes collector image from [opentelemetry-collector](https://github.com/open-telemetry/opentelemetry-collector) to [opentelemetry-collector-contrib](https://github.com/open-telemetry/opentelemetry-collector-contrib) which includes additional connectors, exporters, extensions, processors, receivers. 8. Adds value `collector.env` for additional collector Deployment environment variables. Signed-off-by: Cameron Boulton <[email protected]> Signed-off-by: Adam Shaw <[email protected]>
mateiidavid
added a commit
that referenced
this pull request
Sep 22, 2023
This edge release updates the proxy's dependency on the `rustls` library to patch security vulnerability [RUSTSEC-2023-0052] (GHSA-8qv2-5vq6-g2g7), a potential CPU usage denial-of-service attack when acceting a TLS handshake from an untrusted peer with a maliciously-crafted certificate. Furthermore, this edge release contains a few improvements to the control plane and jaeger extension Helm charts. * Addressed security vulnerability [RUSTSEC-2023-0052] in the proxy by updating its dependency on the `rustls` library * Added a `prometheusUrl` field for the heartbeat job in the control plane Helm chart (thanks @david972!) ([#11343]; fixes [#11342]) * Introduced support for arbitrary labels in the `podMonitors` field in the control plane Helm chart (thanks @jseiser!) ([#11222]; fixes [#11175]) * Added support for config merge and Deployment environment to `opentelemetry-collector` in the jaeger extension (thanks @iAnomaly!) ([#11283]) [#11283]: #11283 [#11222]: #11222 [#11175]: #11175 [#11343]: #11343 [#11342]: #11342 Signed-off-by: Matei David <[email protected]>
Merged
mateiidavid
added a commit
that referenced
this pull request
Sep 22, 2023
* edge-29.9.3 This edge release updates the proxy's dependency on the `rustls` library to patch security vulnerability [RUSTSEC-2023-0052] (GHSA-8qv2-5vq6-g2g7), a potential CPU usage denial-of-service attack when acceting a TLS handshake from an untrusted peer with a maliciously-crafted certificate. Furthermore, this edge release contains a few improvements to the control plane and jaeger extension Helm charts. * Addressed security vulnerability [RUSTSEC-2023-0052] in the proxy by updating its dependency on the `rustls` library * Added a `prometheusUrl` field for the heartbeat job in the control plane Helm chart (thanks @david972!) ([#11343]; fixes [#11342]) * Introduced support for arbitrary labels in the `podMonitors` field in the control plane Helm chart (thanks @jseiser!) ([#11222]; fixes [#11175]) * Added support for config merge and Deployment environment to `opentelemetry-collector` in the jaeger extension (thanks @iAnomaly!) ([#11283]) [#11283]: #11283 [#11222]: #11222 [#11175]: #11175 [#11343]: #11343 [#11342]: #11342 Signed-off-by: Matei David <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Signed-off-by: Cameron Boulton [email protected]
This PR changes a variety of things:
collector.configfrom a string to an object to support field level merging/changing without having to copy/paste the entire unchanged portion of the config.collector.configfor sensible out of the box Linkerd tracing use-case:opencensusreceiver (the formatlinkerd-proxyemits when configured for tracing)resourceprocessor that maps the Pod name and namespace labelslinkerd-proxysets on injected Pods to the fields thek8sattributesprocessor expects for looking up the trace's source Pod.k8sattributesprocessor for decorating all traces with K8s metadata.image.pullPolicybehavior across all components and defaults to absent/nullimagePullPolicysuch that the K8s cluster configuration is used.k8sattributesto lookup trace source K8s workloadscollector.command.collector.envfor additional collector Deployment environment variables.