Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add functional end-to-end tests for the chart #40

Open
tigrannajaryan opened this issue Mar 30, 2021 · 1 comment
Open

Add functional end-to-end tests for the chart #40

tigrannajaryan opened this issue Mar 30, 2021 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@tigrannajaryan
Copy link
Member

We need to test the Helm chart's functionality end-to-end:

  • Install the chart on a test Kubernetes cluster.
  • Run a test mock backend (can possibly reuse mock backends from Collector testbed).
  • Configure the chart to send collected traces, metrics and logs to the mock backend.
  • Run some test workload on the cluster. The workload should generate telemetry: application traces, metrics and logs and cluster metrics and logs.
  • Let mock backend to receiver generated telemetry and compare against expected data. It would be best that expected data is defined in a set of golden files.

Make sure end-to-end test is run on Github action as part of every PR and main branch build.
@tigrannajaryan
Copy link
Member Author

Looks like we are already using https://github.com/helm/chart-testing-action which supposedly is what we need to install the chart, so probably step 1 above is already covered.

@tigrannajaryan tigrannajaryan added the help wanted Extra attention is needed label Mar 30, 2021
@github-actions github-actions bot added the Stale label Mar 31, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale May 1, 2023
@TylerHelmuth TylerHelmuth reopened this May 1, 2023
@TylerHelmuth TylerHelmuth removed the Stale label May 1, 2023
matej-g added a commit to matej-g/opentelemetry-helm-charts that referenced this issue Oct 30, 2023
@povilasv @matej-g
fix: kubelstatsreceiver change insecure_skip_verify to true (open-tel…
0f6aced 
…emetry#40)

* kubelstatsreceiver change insecure_skip_verify to true

* Update charts/opentelemetry-collector/CHANGELOG.md

Co-authored-by: Matej Gera <[email protected]>

---------

Co-authored-by: Matej Gera <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tigrannajaryan @TylerHelmuth