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

ingress fails to deploy but reports success #163

Closed
dwradcliffe opened this issue Sep 8, 2017 · 4 comments
Closed

ingress fails to deploy but reports success #163

dwradcliffe opened this issue Sep 8, 2017 · 4 comments
Labels
🪲 bug Something isn't working

Comments

@dwradcliffe
Copy link
Member

I have an invalid ingress that I tried to deploy. It reported success but the ingress did not change or show me the error.

Error should have been: error: ingresses "web" could not be patched: cannot convert int64 to string

What I saw:

[INFO][2017-09-08 19:28:45 +0000]	Deploying resources:
[INFO][2017-09-08 19:28:45 +0000]	- Ingress/web (timeout: 30s)
Successfully deployed in 8.8s: Ingress/web
@dwradcliffe dwradcliffe added the 🪲 bug Something isn't working label Sep 8, 2017
@KnVerey
Copy link
Contributor

KnVerey commented Sep 13, 2017

That's very odd. Do you have the relevant part of the old/new specs I could use to try to reproduce this?

@dwradcliffe
Copy link
Member Author

dwradcliffe commented Sep 13, 2017

here's a sample:

apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: web
  annotations:
    kubernetes.io/ingress.class: nginx
    ingress.kubernetes.io/limit-rps: 30
  labels:
    name: web
    app: foobar
    environment: production
spec:
  rules:
  - host: foobar.example.com
    http:
      paths:
      - backend:
          serviceName: web
          servicePort: 80

Notice the ingress.kubernetes.io/limit-rps is an int where it should be a string. This deploy was going from no annotation to adding this annotation.

@dturn
Copy link
Contributor

dturn commented Jun 11, 2019

Stale, this is likely issue with kubectl itself given the issue was with the resource patching.

@dturn dturn closed this as completed Jun 11, 2019
@KnVerey
Copy link
Contributor

KnVerey commented Jul 23, 2019

Breadcrumb: this is the upstream bug: kubernetes/kubernetes#59113

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

No branches or pull requests

3 participants