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 does not get updated when name contains period(s) #2012

Closed
1 task done
jeroen-s opened this issue Nov 15, 2021 · 1 comment · Fixed by #2013
Closed
1 task done

Ingress does not get updated when name contains period(s) #2012

jeroen-s opened this issue Nov 15, 2021 · 1 comment · Fixed by #2013
Assignees
Labels
bug Something isn't working priority/medium waiting-on-pr-review This work is on-hold, specifically because no further work can be done until review is complete

Comments

@jeroen-s
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Due to:

routeInf := strings.Split(*((*route).Name), ".")

I end up here:

log.V(util.DebugLevel).Info("failed to retrieve v1/Ingress: the object is gone, quitting status updates", "namespace", namespace, "name", name)

with name being the part of the Ingress name before the first period.

Expected Behavior

Normal Ingress update.

Steps To Reproduce

No response

Kong Ingress Controller version

v2.0.5

Kubernetes version

Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.3", GitCommit:"c92036820499fedefec0f847e2054d824aea6cd1", GitTreeState:"clean", BuildDate:"2021-10-27T18:34:20Z", GoVersion:"go1.16.10", Compiler:"gc", Platform:"darwin/arm64"}
Server Version: version.Info{Major:"1", Minor:"21+", GitVersion:"v1.21.2-eks-0389ca3", GitCommit:"8a4e27b9d88142bbdd21b997b532eb6d493df6d2", GitTreeState:"clean", BuildDate:"2021-07-31T01:34:46Z", GoVersion:"go1.16.5", Compiler:"gc", Platform:"linux/amd64"}

Anything else?

No response

@jeroen-s jeroen-s added the bug Something isn't working label Nov 15, 2021
@shaneutt shaneutt self-assigned this Nov 15, 2021
@shaneutt shaneutt added the waiting-on-pr-review This work is on-hold, specifically because no further work can be done until review is complete label Nov 15, 2021
@shaneutt
Copy link
Contributor

Hi @jeroen-s thanks for reporting this 👍

I have a PR up which addresses this as well as one other issue: #2013

Ideally if review goes swiftly the intention will be to release this in the next patch version v2.0.6 in the next few days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority/medium waiting-on-pr-review This work is on-hold, specifically because no further work can be done until review is complete
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants