Skip to content

Conversation

aparnams1999
Copy link
Contributor

@aparnams1999 aparnams1999 commented Aug 11, 2025

Description of the change

This change adds optional Ingress support for the Jaeger Query service in the Bitnami Jaeger Helm chart.

Introduces an ingress block in values.yaml to enable configuring hostname, path, ingress class, annotations, and TLS.

Updates the chart templates to conditionally create a Kubernetes Ingress resource targeting the jaeger-query service on port 16686 when ingress is enabled.

Aligns the Jaeger chart with other Bitnami charts that provide native Ingress support, improving deployment automation and ease of use.

Benefits

Simplifies exposing the Jaeger Query UI by allowing users to enable and configure Ingress directly via Helm values.

Eliminates the need for users to manually create and manage separate Ingress resources outside the chart, reducing operational overhead.

Enables consistent and standardized ingress configurations, including support for TLS and annotations for common ingress controllers.

Enhances automation by integrating ingress resource lifecycle with Helm, ensuring Ingress is created, updated, or deleted with the chart.

Brings the Jaeger chart in line with Bitnami’s best practices and other charts that already include native Ingress support.

Possible drawbacks

Not to my knowledge

Applicable issues

Additional information

Tested with NGINX ingress controller on Kubernetes

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

@github-actions github-actions bot added jaeger triage Triage is needed labels Aug 11, 2025
@github-actions github-actions bot requested a review from javsalgar August 11, 2025 16:07
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Aug 12, 2025
@github-actions github-actions bot removed the triage Triage is needed label Aug 12, 2025
@github-actions github-actions bot removed the request for review from javsalgar August 12, 2025 06:43
@github-actions github-actions bot requested a review from gongomgra August 12, 2025 06:43
@aparnams1999
Copy link
Contributor Author

Hi, please let me know if I need to make any changes from my side. I noticed that some pipelines are failing.

aparnams1999 and others added 2 commits August 20, 2025 12:25
Signed-off-by: Bitnami Bot <[email protected]>
@gongomgra
Copy link
Contributor

Hi @aparnams1999. It looks like the deployment is failing with your changes. Can you try to reproduce it?

Not all endpoints are ready after the time out. 1 of 4 failed:
  [name=ing-jaeger-http, ready=false, expectedToBeReady=true, type=INGRESS],

aparnams1999 and others added 2 commits September 3, 2025 13:44
Signed-off-by: Bitnami Bot <[email protected]>
Copy link

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the stale 15 days without activity label Sep 19, 2025
aparnams1999 and others added 2 commits September 19, 2025 11:16
Signed-off-by: Bitnami Bot <[email protected]>
@github-actions github-actions bot removed the stale 15 days without activity label Sep 20, 2025
Copy link

github-actions bot commented Oct 5, 2025

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the stale 15 days without activity label Oct 5, 2025
Copy link

Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Pull Request. Do not hesitate to reopen it later if necessary.

@bitnami-bot bitnami-bot added stale 15 days without activity and removed stale 15 days without activity labels Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jaeger solved stale 15 days without activity verify Execute verification workflow for these changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants