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

Please support "Services without selectors" feature #17320

Closed
qingkunl opened this issue Sep 6, 2021 · 6 comments
Closed

Please support "Services without selectors" feature #17320

qingkunl opened this issue Sep 6, 2021 · 6 comments
Labels
kind/community-report This was reported by a user in the Cilium community, eg via Slack. kind/feature This introduces new functionality. need-more-info More information is required to further debug or fix the issue. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.

Comments

@qingkunl
Copy link

qingkunl commented Sep 6, 2021

The K8s official document has Services without selectors feature. However, when I experiment it with Cilium (kubeProxyReplacement=strict), it doesn't work. It seems like Cilium doesn't support this feature now. Many other CNIs already support it.

@qingkunl qingkunl added the kind/feature This introduces new functionality. label Sep 6, 2021
@brb
Copy link
Member

brb commented Sep 6, 2021

@qingkunl Thanks for the issue. Are you sure that it doesn't work? Have you seen #12513 and #12438?

@qingkunl
Copy link
Author

qingkunl commented Sep 6, 2021

@qingkunl Thanks for the issue. Are you sure that it doesn't work? Have you seen #12513 and #12438?

@brb Thanks for the reference. I'm using K8s v1.20.2 + Cilium v1.9.9 and still see this issue.

Following is my service config. Please advice.

apiVersion: v1
kind: Service
metadata:
  name: service-qingkun
spec:
  type: NodePort
  ports:
    - protocol: TCP
      port: 80
      nodePort: 31081

---

apiVersion: v1
kind: Endpoints
metadata:
  name: service-qingkun
subsets:
  - addresses:
      - ip: 10.62.32.11
      - ip: 10.62.32.12
    ports:
      - port: 32080

I can confirm endpointslice has been created:

$ kubectl get svc | grep service-qingkun
service-qingkun   NodePort    172.18.117.9    <none>            80:31081/TCP   5m49s

$ kubectl get endpoints | grep service-qingkun
service-qingkun   10.62.32.11:32080,10.62.32.12:32080                          5m56s

$ kubectl get endpointslice | grep service-qingkun
service-qingkun-mm9g4   IPv4          32080   10.62.32.11,10.62.32.12                            6m

@joestringer
Copy link
Member

Maybe you can debug a bit on the Cilium side to understand how Cilium is interpreting those services?

Eg, cilium service list, cilium bpf lb list inside the pod.

@pchaigno pchaigno added kind/community-report This was reported by a user in the Cilium community, eg via Slack. need-more-info More information is required to further debug or fix the issue. labels Sep 13, 2021
@aanm aanm added the sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. label Jan 6, 2022
@github-actions
Copy link

github-actions bot commented Jul 9, 2022

This issue has been automatically marked as stale because it has not
had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. and removed stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. labels Jul 9, 2022
@github-actions
Copy link

This issue has been automatically marked as stale because it has not
had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Sep 14, 2022
@github-actions
Copy link

github-actions bot commented Oct 1, 2022

This issue has not seen any activity since it was marked stale.
Closing.

@github-actions github-actions bot closed this as completed Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/community-report This was reported by a user in the Cilium community, eg via Slack. kind/feature This introduces new functionality. need-more-info More information is required to further debug or fix the issue. sig/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Projects
None yet
Development

No branches or pull requests

5 participants