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

Helm chart - Ingress notes #22586

Closed
wants to merge 3 commits into from
Closed

Conversation

jlpedrosa
Copy link
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
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.

None yet

2 participants