Currently, when I add or update annotations to my Ingress object, none of the changes

Fixed in <a class="issue-link js-issue-link" data-error-text="Failed to load title" da

If you retry does it work? <a class="issue-link js-issue-link" data-

We have merged <a class="issue-link js-issue-link" data-error-text="Failed to load tit

Wow great, excited to try it out. <span class="email-hidden-toggle

Updating Ingress annotations are not picked up by controller about voyager HOT 6 CLOSED

voyagermesh commented on June 1, 2024

Updating Ingress annotations are not picked up by controller

from voyager.

Comments (6)

tamalsaha commented on June 1, 2024

Fixed in #134

from voyager.

julianvmodesto commented on June 1, 2024

Unfortunately, I think i'm running into this issue again on v5.0.0-rc.11.

My change is adding a new annotation.

# Old
    ingress.appscode.com/annotations-service: |
      {
        "service.beta.kubernetes.io/aws-load-balancer-ssl-cert": "REDACTED",
        "service.beta.kubernetes.io/aws-load-balancer-ssl-ports": "443",
        "service.beta.kubernetes.io/aws-load-balancer-backend-protocol": "http"
      }

# New
    ingress.appscode.com/annotations-service: |
      {
        "service.beta.kubernetes.io/aws-load-balancer-ssl-cert": "REDACTED",
        "service.beta.kubernetes.io/aws-load-balancer-ssl-ports": "443",
        "service.beta.kubernetes.io/aws-load-balancer-backend-protocol": "http",
        "external-dns.alpha.kubernetes.io/hostname": "REDACTED"
      }

from voyager.

tamalsaha commented on June 1, 2024

If you retry does it work?

#803 will permanently solve this type of issues, hopefully.

from voyager.

julianvmodesto commented on June 1, 2024

Ohh hmm sorry yes after retrying it did pick up, weird!

Interesting, never implemented a controller before, hope that the workqueue is helpful.

from voyager.

tamalsaha commented on June 1, 2024

We have merged #803.

from voyager.

julianvmodesto commented on June 1, 2024

Wow great, excited to try it out.

…

On Mon, Feb 5, 2018 at 1:03 PM Tamal Saha ***@***.***> wrote: We have merged #803 <#803>. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#115 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABmsloZNiEMXpC5XkTdq4wHurqm-BZcmks5tR0KBgaJpZM4NiyJq> .

from voyager.

Recommend Projects