Skip to content

update opentelemetry-collector image to the latest version and fix e2e tests.#960

Merged
bpjain2004 merged 6 commits intomainfrom
usr/bjain/update-otel-collector
Apr 28, 2025
Merged

update opentelemetry-collector image to the latest version and fix e2e tests.#960
bpjain2004 merged 6 commits intomainfrom
usr/bjain/update-otel-collector

Conversation

@bpjain2004
Copy link
Copy Markdown
Contributor

Description

Updated the opentelemetry-collector image to the latest version(for the sample files, config files, and test files related CSM 1.14 ). The exporter "logging" was deprecated, and the new exporter "debug" is used as per the guidelines.
Made fixes to some of the e2e tests.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
https://github.com/dell/csm/issues/1861

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have verified that new and existing unit tests pass locally with my changes
  • I have not allowed coverage numbers to degenerate
  • I have maintained at least 90% code coverage
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have maintained backward compatibility
  • I have executed the relevant end-to-end test scenarios

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

  • Run the sanity tests and observability module e2e tests for PowerScale
  • Run the sanity tests and observability module e2e tests for PowerFlex
  • Run the sanity tests and observability module e2e tests for the PowerMax
  • Run the operator e2e tests for PowerScale
  • Run the operator e2e tests for PowerFlex
  • Run the operator e2e tests for PowerMax
  • Verified the changes against the full sample files and minimal sample files.
  • Verified for backward compatibility using the released version(1.13) of the full sample files and minimal sample files.
    All the test logs along with the screen shots are attached in the jira ticket.

@github-actions
Copy link
Copy Markdown
Contributor

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ 🤖
github.com/dell/csm-operator/pkg/modules 90.38% (+0.01%) 👍

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/dell/csm-operator/pkg/modules/observability.go 92.23% (+0.05%) 502 (+3) 463 (+3) 39 👍

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/dell/csm-operator/pkg/modules/observability_test.go

@bpjain2004 bpjain2004 merged commit 376248b into main Apr 28, 2025
6 checks passed
@bpjain2004 bpjain2004 deleted the usr/bjain/update-otel-collector branch April 28, 2025 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants