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

Facing issues in getting the IP address to access Jaeger-UI on Linux platform #1568

Closed
odidev opened this issue Oct 4, 2021 · 3 comments · Fixed by #2466
Closed

Facing issues in getting the IP address to access Jaeger-UI on Linux platform #1568

odidev opened this issue Oct 4, 2021 · 3 comments · Fixed by #2466
Labels
bug Something isn't working

Comments

@odidev
Copy link

odidev commented Oct 4, 2021

Describe the bug

Hi Team,

I am working on jaeger deployment using jaeger-operators on the Ubuntu Linux/AMD64 machine using minikube.

I have followed the docs, addon ingress to minikube, installed operator using given yaml files, and activated extra features. Then created the jaeger instance once jaeger-operator is ready with namespace ‘observability’, following the above linked documentation.

Below is the output of kubectl get all -n observability:

$ kubectl get all -n observability 

NAME                                 READY   STATUS    RESTARTS   AGE 
pod/jaeger-operator-dbf5767c-8dgjf   1/1     Running   0          92m 
pod/simplest-5675c68699-c7wgp        1/1     Running   0          89m   

NAME                                  TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)                                  AGE 
service/jaeger-operator-metrics       ClusterIP   10.97.208.223   <none>        8383/TCP,8686/TCP                        92m 
service/simplest-agent                ClusterIP   None            <none>        5775/UDP,5778/TCP,6831/UDP,6832/UDP      89m 
service/simplest-collector            ClusterIP   10.99.48.59     <none>        9411/TCP,14250/TCP,14267/TCP,14268/TCP   89m 
service/simplest-collector-headless   ClusterIP   None            <none>        9411/TCP,14250/TCP,14267/TCP,14268/TCP   89m 
service/simplest-query                ClusterIP   10.104.172.28   <none>        16686/TCP,16685/TCP                      89m 

NAME                              READY   UP-TO-DATE   AVAILABLE   AGE 
deployment.apps/jaeger-operator   1/1     1            1           92m 
deployment.apps/simplest          1/1     1            1           89m 

NAME                                       DESIRED   CURRENT   READY   AGE 
replicaset.apps/jaeger-operator-dbf5767c   1         1         1       92m 
replicaset.apps/simplest-5675c68699        1         1         1       89m 

However, I am not getting the value of address, where Jaeger-UI is available. Below is the ingress information:

$ kubectl get -n observability ingress 

NAME             CLASS    HOSTS   ADDRESS   PORTS   AGE 
simplest-query   <none>   *                 80      25h 

I have verified that ingress is addon to minikube.

Can you please provide some pointers on what could be the issue here?

To Reproduce

I have exactly followed the getting started document.

Expected behavior

Ingress should serve the Jaeger-UI and IP address must be returned.

Screenshots

NA

Version (please complete the following information):

  • OS: Linux
  • Jaeger version: master branch
  • Deployment: minikube

What troubleshooting steps did you try?

Checked that ingress is addon to minikube.

Additional context

NA

@odidev odidev added the bug Something isn't working label Oct 4, 2021
@odidev
Copy link
Author

odidev commented Dec 17, 2021

Hello Team, can you please review this issue?

@antoniomerlin
Copy link
Contributor

Hi, did your cluster having Ingress controller to satisfy the ingress, merely creating the ingress resource will have no effect.

@antoniomerlin
Copy link
Contributor

Screenshot from 2024-02-05 16-35-53
Refer this one https://kubernetes.github.io/ingress-nginx/deploy/
OR
i think try by adding this in simplest-query ingresses metadata after some time you'll get loadbalancer IP in status of this ingresses.

annotations:
    kubernetes.io/ingress.class: nginx

antoniomerlin pushed a commit to antoniomerlin/jaeger-operator that referenced this issue Feb 5, 2024
if spec.IngressClassName is not specified in CRD, then this will query kubernetes API for getting list of IngressClasses and look for if any default IngressClass is available, if present the it attach that IngressClass to networkingv1.IngressSpec of jaegerIngress and if not available then it check for nginx ingress controller and attach that and if nothing is avalable then it do nothing.

Signed-off-by: Gaurav Singh <[email protected]>
@antoniomerlin antoniomerlin mentioned this issue Feb 5, 2024
4 tasks
antoniomerlin pushed a commit to antoniomerlin/jaeger-operator that referenced this issue Feb 6, 2024
if spec.IngressClassName is not specified in CRD, then this will query kubernetes API for getting list of IngressClasses and look for if any default IngressClass is available, if present the it attach that IngressClass to networkingv1.IngressSpec of jaegerIngress and if not available then it check for nginx ingress controller and attach that and if nothing is avalable then it do nothing.

Signed-off-by: Gaurav Singh <[email protected]>
@antoniomerlin antoniomerlin mentioned this issue Feb 6, 2024
4 tasks
antoniomerlin pushed a commit to antoniomerlin/jaeger-operator that referenced this issue Feb 12, 2024
added function getIngressClass which set default or nginx ingress class to jaeger define ingress for providing loadbalancer IP.

Signed-off-by: Gaurav Singh <[email protected]>
@antoniomerlin antoniomerlin mentioned this issue Feb 12, 2024
4 tasks
iblancasa pushed a commit that referenced this issue Mar 11, 2024
* fixes #1568 #2225

added function getIngressClass which set default or nginx ingress class to jaeger define ingress for providing loadbalancer IP.

Signed-off-by: Gaurav Singh <[email protected]>

* Added test for checking if ingressClass is not set then the default or nginx ingressClass is added to the jaeger ingress.

Signed-off-by: Gaurav Singh <[email protected]>

* Added request chenges mentioned by iblancasa re: test for checking if ingressClass is not set then the default or nginx ingressClass is added to the jaeger ingress.

Signed-off-by: Gaurav Singh <[email protected]>

* changed func name from getInClusterAvailableIngressClasses() to
getInClusterAvailableIngressClass() and updated manifests

Signed-off-by: Gaurav Singh <[email protected]>

* fixed formatting

Signed-off-by: Gaurav Singh <[email protected]>

---------

Signed-off-by: Gaurav Singh <[email protected]>
Co-authored-by: Gaurav Singh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants