-
Notifications
You must be signed in to change notification settings - Fork 235
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
Check if Service has Endpoints for dangling service template #747
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #747 +/- ##
==========================================
- Coverage 72.37% 72.28% -0.10%
==========================================
Files 45 45
Lines 1937 1945 +8
==========================================
+ Hits 1402 1406 +4
- Misses 421 423 +2
- Partials 114 116 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
56ef9d3
to
6eafd64
Compare
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.
Could you add sample from
- [BUG] - service has no selector specified raised when using Endpoint #746
to
kube-linter/e2etests/bats-tests.sh
Lines 131 to 134 in 49aa5bb
@test "dangling-service" { tmp="tests/checks/dangling-service.yml" cmd="${KUBE_LINTER_BIN} lint --include dangling-service --do-not-auto-add-defaults --format json ${tmp}" run ${cmd}
Hi @janisz, |
Is there a reason this has been closed? Would also be cool, if this could be extended for |
Motivation
Fix #746
Changed
This PR update dangling-service template.
When no selector is specified on the Service, check if Endpoints exists in the context with the same name and namespace.