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

Service with selector not recognized #6404

Open
bittermandel opened this issue Jun 24, 2024 · 2 comments
Open

Service with selector not recognized #6404

bittermandel opened this issue Jun 24, 2024 · 2 comments
Labels
bug Something isn't working needs repro case

Comments

@bittermandel
Copy link

bittermandel commented Jun 24, 2024

Expected Behavior

I expect the k8s service to be created and managed by Tilt as intended.

Current Behavior

Unless I change either the key or the value of the selector, Tilt ignores the file.

Steps to Reproduce

This is the Service I'm targetting:

---
apiVersion: v1
kind: Service
metadata:
  name: webhook-service
  namespace: temporal-system
spec:
  ports:
  - port: 443
    protocol: TCP
    targetPort: 9443
  selector:
    control-plane: controller-manager

Context

tilt doctor Output

$ tilt doctor
Tilt: v0.33.17, built 2024-06-12
System: linux-amd64
---
Docker
- Host: unix:///var/run/docker.sock
- Server Version: 24.0.7
- API Version: 1.43
- Builder: 2
- Compose Version: v2.27.0
---
Kubernetes
- Env: kind
- Context: kind-molnett
- Cluster Name: kind-molnett
- Namespace: default
- Container Runtime: containerd
- Version: v1.30.0
- Cluster Local Registry: none
---
Thanks for seeing the Tilt Doctor!
Please send the info above when filing bug reports. 💗

The info below helps us understand how you're using Tilt so we can improve,
but is not required to ask for help.
---
Analytics Settings
--> (These results reflect your personal opt in/out status and may be overridden by an `analytics_settings` call in your Tiltfile)
- User Mode: opt-in
- Machine: 0f229b09995c2721819417b9cf1b51bc
- Repo: rqGPZYxzrdJ/vT+5WBItoQ==
...

About Your Use Case

This blocks us from using Tilt to fully replicate our local environment.

@bittermandel bittermandel added the bug Something isn't working label Jun 24, 2024
@nicks
Copy link
Member

nicks commented Jun 24, 2024

hmmm...sorry to hear you're having trouble. most teams who use Tilt use it deploy Services like the one in your report. I do it every day.

i tried to repro your example and tilt deployed the service successfully. can you add additional repro steps?

@bittermandel
Copy link
Author

So I think the Service should be grouped together with the deployment. I'm not sure why, but I made the labels slightly smaller (control-plane: controller) and it works. I wonder if there's some cache somewhere that's messing with me?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs repro case
Projects
None yet
Development

No branches or pull requests

2 participants