elastic / helm-charts

You know, for Kubernetes
Apache License 2.0
1.89k stars 1.93k forks source link

[kibana/elasticsearch] ingress defined with hosts breaks other ingress with nginx #536

Open aitorhh opened 4 years ago

aitorhh commented 4 years ago

What I would like to suggest is to have support through values configuration for non-hosts definition when deploying kibana (or elasticsearch).

Chart version:

version: 5.0.11
appVersion: 7.6.1

Kubernetes version:

Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"clean", BuildDate:"2020-02-11T18:14:22Z", GoVersion:"go1.13.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"clean", BuildDate:"2020-02-11T18:07:13Z", GoVersion:"go1.13.6", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes provider: E.g. GKE (Google Kubernetes Engine) Local deployment

Helm Version:

version.BuildInfo{Version:"v3.1.1", GitCommit:"afe70585407b420d0097d07b21c47dc511525ac8", GitTreeState:"clean", GoVersion:"go1.13.8"}

Describe the bug:

When creating an ingress with hosts, it makes all others ingress with default "*" to be not accessible. I have seen this behaviour with nginx-ingress (stable/nginx-ingress from https://helm.nginx.co/stable appVersion: 1.6.3)

Steps to reproduce:

  1. helm install nginx-ingress stable/nginx-ingress
  2. helm install kibana elastic/kibana --set ingress.enabled=true --set ingress.hosts[0]=test.com

Expected behavior:

I expect that all the ingress should be working regardless

Any additional context: I understand that this issue comes from nginx, but I also believe that it has less impact by updating the helm chart here.

botelastic[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

aitorhh commented 4 years ago

Unless it has been solved in another issues or PR, I believe it's something quite relevant that shouldn't be closed.

botelastic[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

jmlrt commented 4 years ago

still valid

botelastic[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

jmlrt commented 3 years ago

still valid

botelastic[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

jmlrt commented 3 years ago

still valid

botelastic[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

jmlrt commented 3 years ago

still valid

ghost commented 3 years ago

anyone facing issue with kibana ingress? {"statusCode":400,"error":"Bad Request","message":"Invalid cookie header"}

linode kubernetes cluster

botelastic[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

jmlrt commented 2 years ago

still valid

botelastic[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

botelastic[bot] commented 2 years ago

This issue has been automatically closed because it has not had recent activity since being marked as stale.

botelastic[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

tormachris commented 2 years ago

still valid

DmitryZakharov commented 2 years ago

Please make it work. We now deploy our own ingress object instead and set ingress.enabled: false

DmitryZakharov commented 2 years ago

One could take an idea from grafana, which does exactly what is needed https://artifacthub.io/packages/helm/grafana/grafana/6.1.10?modal=template&template=ingress.yaml

botelastic[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

DmitryZakharov commented 1 year ago

Please don't close it. Is there any plan to have it fixed ?

botelastic[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.