Update OpenTelemetry Collector #7053
Conversation
Signed-off-by: anastasia.malysheva <[email protected]>
anastasia-malysheva
changed the title
… into fixes_for_opa_update � Conflicts: � examples/features/jaeger/README.md
| @@ -2,5 +2,7 @@ | |||
| apiVersion: kustomize.config.k8s.io/v1beta1 | |||
| kind: Kustomization | |||
|
|
|||
| namespace: nsm-system | |||
There was a problem hiding this comment.
Do we need this change?
There was a problem hiding this comment.
actually no, I'll remove
There was a problem hiding this comment.
Im not sure that we need folder
examples/features/jaeger/nsm-system.
Because this example is using by users that already setup NSM.
Other things are looking fine.
@NikitaSkrynnik Could you also have a look?
|
@denis-tingaikin , I think local nsm configs needed for this example because we have to be sure that tracing is enabled for forwarder and manager, even though it is enabled by default now. Plus this setup is not installing nsm as in other examples in the folder, but it applies it as any other resources. |
Signed-off-by: anastasia.malysheva <[email protected]>
|
@anastasia-malysheva Could you please resolve conflict? |
… into fixes_for_opa_update � Conflicts: � examples/features/jaeger/README.md
|
@denis-tingaikin conflicts resolved |
… into fixes_for_opa_update � Conflicts: � examples/features/jaeger/README.md
… into fixes_for_opa_update � Conflicts: � examples/features/jaeger/README.md � examples/observability/jaeger-and-prometheus/README.md � examples/observability/jaeger-and-prometheus/nsm-system/kustomization.yaml
Add fixes for OPA update to the latest version
Signed-off-by: anastasia.malysheva [email protected]