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

Duplicate records repeatedly added to Ingress.status.loadBalancer.ingress #88

Closed
giovannimatina opened this issue May 31, 2024 · 2 comments · Fixed by #89
Closed

Duplicate records repeatedly added to Ingress.status.loadBalancer.ingress #88

giovannimatina opened this issue May 31, 2024 · 2 comments · Fixed by #89

Comments

@giovannimatina
Copy link

Since I updated the controller to version 0.0.10, I've noticed an anomalous behavior. Every time the controller iterates, new duplicate records are added to the ingress list within the loadBalancer field of the status. This behavior is leading to a continuous accumulation of duplicates, compromising the integrity and usefulness of the status.

Example of the issue:

status:
  loadBalancer:
    ingress:
    - hostname: example-uuid.cfargotunnel.com
      ports:
      - port: 443
        protocol: TCP
    - hostname: example-uuid.cfargotunnel.com
      ports:
      - port: 443
        protocol: TCP
    - hostname: example-uuid.cfargotunnel.com
      ports:
      - port: 443
        protocol: TCP
...
@UnstoppableMango
Copy link
Contributor

I think I may have introduced this recently 😬 hopefully #89 should resolve it! The implementation I went with will only prevent future duplicates, I have an open question on whether its worthwhile to check and remove duplicates.

@STRRL
Copy link
Owner

STRRL commented Jun 1, 2024

Hi @giovannimatina @UnstoppableMango , thanks for your reporting and contribution! We are working on this!

@STRRL STRRL closed this as completed in #89 Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants