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.

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

docs: docs: adjust a description error in NOTES.txt file #1395

Merged
merged 3 commits into from
Nov 1, 2024

Conversation

dartagnanli
Copy link
Contributor

previous:

kubectl --namespace {{ .Release.Namespace }} get pods -l "release={{ $.Release.Name }}"

now:

kubectl --namespace {{ .Release.Namespace }} get pods -l "app.kubernetes.io/name={{ $.Release.Name }}"

this will correctly list pods like blew:

$ kubectl --namespace opentelemetry-operator-system get pods -l "app.kubernetes.io/name=opentelemetry-operator"
NAME                                     READY   STATUS    RESTARTS   AGE
opentelemetry-operator-556cf4f79-8bqwd   2/2     Running   0          8m47s

@dartagnanli dartagnanli requested review from Allex1 and a team as code owners October 22, 2024 12:46
Copy link

linux-foundation-easycla bot commented Oct 22, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@povilasv
Copy link
Contributor

@dartagnanli please sign the CLA

@dartagnanli
Copy link
Contributor Author

@dartagnanli please sign the CLA

done

@TylerHelmuth TylerHelmuth merged commit 362a82a into open-telemetry:main Nov 1, 2024
4 checks passed
codeboten pushed a commit to codeboten/opentelemetry-helm-charts that referenced this pull request Nov 14, 2024
…try#1395)

* docs: docs: adjust a description error in NOTES.txt file

* fix: bump the patch version of the operator Chart.yaml
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.

4 participants