Merged
Conversation
* [chore][istio] Add functional integration test for traces * make gofmt * fix spacing in CRD downlaod command * remote crd install from file * teardown httpbin httproute in test * remove telemetry resource on teardown in tests * Delete more httpbin resources in teardown * Fix delete resource address, ignore peer address span attribute * updated expected testdataa * save golden files from test run * comment out t.setenv calls * use new method for specifying ports for OTLP trace sink * add missing return statement * Only compare first received resource span for test * Cleanup unrelated/unnecessary changes * Remove debugging, cleanup gofmt issues * fix broken rebase * update expected istiod metrics New metrics: - provider_lookup_cluster_failures - sidecar_injection_skip_total * Re-work teardown to address flakiness * add wait for traces to potentially remove require.Eventually * fix namespace typo, copy err check to allow not found * remove wait for traces, resulted in no traces coming through * workaround the t.Context being cancelled * fix reversed if statement * fix wording * Move test to wait for a minute (instead of 30 seconds) Saw some tests that were successful take 14 seconds, give a little extra buffer just in case. * remove unnecessary custom header
* Use splunk-otel-python for python auto-instrumentation * use latest image * add changelog * update docs * resolve conflicts * ignore telemetry.auto.version * fix attr ignore * Update .chloggen/splunk-otel-python.yaml Co-authored-by: Curtis Robert <crobert@splunk.com> * Update .chloggen/splunk-otel-python.yaml Co-authored-by: Curtis Robert <crobert@splunk.com> * Update functional_tests/functional/testdata/python/deployment.yaml --------- Co-authored-by: Curtis Robert <crobert@splunk.com>
* Bind the OTLP HTTP port 4318 to the host network Signed-off-by: Dani Louca <dlouca@splunk.com> * Update .chloggen/OTLPPortWindows.yaml Co-authored-by: Jina Jain <jjain@splunk.com> --------- Signed-off-by: Dani Louca <dlouca@splunk.com> Co-authored-by: Jina Jain <jjain@splunk.com>
Signed-off-by: Dani Louca <dlouca@splunk.com>
* [chore] Fix typos, wording, and capitalizations * Update helm-charts/splunk-otel-collector/values.yaml --------- Co-authored-by: Jina Jain <jjain@splunk.com>
* [agent] Fix config when O11y metrics are disabled * Update .chloggen/fixo11ymetricsdisabled.yaml * make pre-commit * nit from copilot: fix whitespace * only include sync_host_metadata when metrics enabled
* Move istio sidecar injection annotation to label * add issue to chlog * remove sidecar_injection_skip_total from expected metrics
…plication features in Splunk Observability Cloud (#2106) * Changes in collector configuration in agent mode to support Secure Application features in Splunk Observability Cloud * replace helper template with direct value as per review comment * keep secureApp changes disabled by default * resolve conflict * added example for SecureApp enabled along with profiling
* kubelet.version * manual test and data update * remove manual testdata changes * updateExpected before error catching * remove attr * rebase (#2115) * [agent] Fix config when O11y metrics are disabled (#2083) * [agent] Fix config when O11y metrics are disabled * Update .chloggen/fixo11ymetricsdisabled.yaml * make pre-commit * nit from copilot: fix whitespace * only include sync_host_metadata when metrics enabled * Fix typo in comment about dynamic receivers (#2113) * [chart] Move istio sidecar injection annotation to label (#2108) * Move istio sidecar injection annotation to label * add issue to chlog * remove sidecar_injection_skip_total from expected metrics --------- Co-authored-by: Curtis Robert <crobert@splunk.com> * add attr * pre-commit * add test debugging * manually add value to testdata * update values * debug missing values * revert attribute * remove all add attr * retest * testing * dynamic metrics in test * dynamic metric lis * hardcoded metric list * remove debugging and dynamic metric list * changelog * kubelet.version * test metrics * revert * validate changes * Update functional_tests/functional/functional_test.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * add kubelet.version * clean up comments * remove additional test optimization * test static metric names * revert test changes * Changes in collector configuration in agent mode to support Secure Application features in Splunk Observability Cloud (#2106) (#2124) * Changes in collector configuration in agent mode to support Secure Application features in Splunk Observability Cloud * replace helper template with direct value as per review comment * keep secureApp changes disabled by default * resolve conflict * added example for SecureApp enabled along with profiling Co-authored-by: tarora-splunk <tarora@splunk.com> * Revert "Changes in collector configuration in agent mode to support Secure Application features in Splunk Observability Cloud (#2106) (#2124)" This reverts commit 77a6e52. --------- Co-authored-by: Curtis Robert <crobert@splunk.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: tarora-splunk <tarora@splunk.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
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.
Description:
Link to Splunk idea: <Link to Splunk idea, see https://ideas.splunk.com>
Testing:
Documentation: