-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: remove cert-manager as dependency & support HTTP/HTTPs endpoints #88
Conversation
18136a4
to
8a600d6
Compare
seccompProfile: | ||
type: RuntimeDefault | ||
serviceAccountName: {{ template "dash0-operator.serviceAccountName" . }} | ||
automountServiceAccountToken: true |
Check warning
Code scanning / SonarCloud
Service account tokens should not be mounted in pods
images/collector/Dockerfile
Outdated
|
||
RUN apk add --update make git && apk cache clean | ||
|
||
ADD ./src /src |
Check notice
Code scanning / SonarCloud
Prefer COPY over ADD for copying local resources
a5c5115
to
ae2c19d
Compare
67227e7
to
9f94209
Compare
helm-chart/dash0-operator/tests/operator/deployment-and-webhook_test.yaml
Outdated
Show resolved
Hide resolved
helm-chart/dash0-operator/templates/operator/pre-delete-hook.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice stuff! 🎉
internal/backendconnection/otelcolresources/otelcol_resources.go
Outdated
Show resolved
Hide resolved
214c46b
to
a679edd
Compare
Also: * remove e2e custom logic in operator * introduce otlp-sink to validate telemetry in e2e
a679edd
to
992d136
Compare
Quality Gate passedIssues Measures |
Use a self-signed certificate for the injector webhook, removing the need of cert-manager as dependency.