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

make export attribute optional in monitoring resource, fall back to operator configuration's export #140

Merged
merged 9 commits into from
Sep 16, 2024

Conversation

basti1302
Copy link
Member

No description provided.

@basti1302 basti1302 force-pushed the eng-2323-operator-default-backend-connection branch from de608af to 56a347d Compare September 13, 2024 18:25
Use the export setting from the operator configuration resource as
a fallback if the monitoring resource does not define an export.
@basti1302 basti1302 force-pushed the eng-2323-operator-default-backend-connection branch from 56a347d to 52a8b31 Compare September 13, 2024 21:23
* webhook package -> webhooks
* dash0_webhook.go -> instrumentation_webhook.go
@basti1302 basti1302 changed the title chore(crd): make export attribute optional in monitoring resource make export attribute optional in monitoring resource, fall back to operator configuration's export Sep 15, 2024
Reject monitoring resources without export if no operator configuration
resource with an export is available.
Reject operator configuration resources with self-telemetry enabled that
have no export configuration.
@basti1302 basti1302 force-pushed the eng-2323-operator-default-backend-connection branch from f332c2b to d0d5fc7 Compare September 16, 2024 10:40
@basti1302 basti1302 force-pushed the eng-2323-operator-default-backend-connection branch from c4a827e to cb76606 Compare September 16, 2024 15:13
Copy link

sonarcloud bot commented Sep 16, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@basti1302 basti1302 marked this pull request as ready for review September 16, 2024 20:43
@basti1302 basti1302 merged commit 1799e97 into main Sep 16, 2024
9 checks passed
@basti1302 basti1302 deleted the eng-2323-operator-default-backend-connection branch September 16, 2024 20:44
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.

1 participant