Skip to content

Helm chart - Ingress notes#22586

Closed
jlpedrosa wants to merge 3 commits intoKoenkk:devfrom
jlpedrosa:ingress_notes
Closed

Helm chart - Ingress notes#22586
jlpedrosa wants to merge 3 commits intoKoenkk:devfrom
jlpedrosa:ingress_notes

Conversation

@jlpedrosa
Copy link
Copy Markdown
Contributor

@jlpedrosa jlpedrosa commented May 10, 2024

Add notes about ingress + websockets:

# -- Ingress configuration. Zigbee2mqtt does use webssockets, which is not part of the Ingress standart settings.
# most of the popular ingresses supports them through annotations. Please check https://www.zigbee2mqtt.io/guide/installation/08_kubernetes.html
# for examples.

@jlpedrosa jlpedrosa changed the base branch from master to dev May 10, 2024 21:26
@jlpedrosa
Copy link
Copy Markdown
Contributor Author

@Koenkk I have been banging my head against the wall with this for a long time. helm-releaser does not support it append the chart to an existing release. I created a patch to fix it upstream. They don't seem to be reviewing PR at all: helm/chart-releaser#407
Options:

  • Have the two tags in this repo.
  • Wait for upstream to fix it (we can grow old waiting IMO)
  • Move the chart to another repo.
  • Do a lot of code to release here in the same repo.
  • Stop software engineering and open a bar in the bermudas.

@jlpedrosa jlpedrosa closed this May 10, 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

Development

Successfully merging this pull request may close these issues.

2 participants