AdWerx / charts

helm charts maintained by Adwerx Engineering
MIT License
25 stars 34 forks source link

[faktory] Fix ingress reference in Faktory's NOTES.txt and README.md #9

Closed Envek closed 4 years ago

Envek commented 4 years ago

What this PR does / why we need it:

helm template is failing if I try to enable Faktory UI:

faktory:
  ui:
    ingress:
      enabled: yes
      hosts:
        - host: example.com
          path: /admin/faktory

Fix README along the way.

Workaround

Duplicate ingress in values:

faktory:
  ui:
    ingress: &faktory-ingress
      hosts:
        - host: example.com
          path: /admin/faktory

  ingress:
    <<: *faktory-ingress

Special notes for your reviewer:

Checklist

jbielick commented 4 years ago

Thank you!

jbielick commented 4 years ago

0.10.2 is on it's way out and will be available in the repo shortly.