Skip to content

Multiple minor changes in Jaeger demo deployment in OKE #7427

Merged
yurishkuro merged 3 commits intojaegertracing:mainfrom
chahatsagarmain:fix-image-version
Aug 9, 2025
Merged

Multiple minor changes in Jaeger demo deployment in OKE #7427
yurishkuro merged 3 commits intojaegertracing:mainfrom
chahatsagarmain:fix-image-version

Conversation

@chahatsagarmain
Copy link
Copy Markdown
Contributor

Which problem is this PR solving?

  • Part of Jaeger demo on Oracle Cloud #7115
  • Fix the issue where hotrod redirected to jaeger-ui to localhost
  • Deleted unused YAML in one of the previous PRs but that was indeed needed to help jaeger access the prometheus database when created with helm .
  • add --force flag to ensure upgrade are not done inplace but by recreating the pods

How was this change tested?

  • locally on a kind cluster

Checklist

Signed-off-by: chahat sagar <109112505+chahatsagarmain@users.noreply.github.com>
Signed-off-by: chahat sagar <109112505+chahatsagarmain@users.noreply.github.com>
@chahatsagarmain chahatsagarmain requested a review from a team as a code owner August 9, 2025 17:27
@chahatsagarmain chahatsagarmain requested a review from jkowall August 9, 2025 17:27
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.48%. Comparing base (b3de96a) to head (7bcdd93).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7427      +/-   ##
==========================================
- Coverage   96.49%   96.48%   -0.02%     
==========================================
  Files         377      377              
  Lines       23071    23071              
==========================================
- Hits        22263    22260       -3     
- Misses        611      613       +2     
- Partials      197      198       +1     
Flag Coverage Δ
badger_v1 9.00% <ø> (ø)
badger_v2 1.70% <ø> (ø)
cassandra-4.x-v1-manual 11.69% <ø> (ø)
cassandra-4.x-v2-auto 1.69% <ø> (ø)
cassandra-4.x-v2-manual 1.69% <ø> (ø)
cassandra-5.x-v1-manual 11.69% <ø> (ø)
cassandra-5.x-v2-auto 1.69% <ø> (ø)
cassandra-5.x-v2-manual 1.69% <ø> (ø)
elasticsearch-6.x-v1 16.60% <ø> (ø)
elasticsearch-7.x-v1 16.64% <ø> (ø)
elasticsearch-8.x-v1 16.79% <ø> (ø)
elasticsearch-8.x-v2 1.70% <ø> (ø)
elasticsearch-9.x-v2 1.70% <ø> (ø)
grpc_v1 10.22% <ø> (ø)
grpc_v2 1.70% <ø> (ø)
kafka-3.x-v1 9.69% <ø> (ø)
kafka-3.x-v2 1.70% <ø> (ø)
memory_v2 1.70% <ø> (ø)
opensearch-1.x-v1 16.69% <ø> (ø)
opensearch-2.x-v1 16.69% <ø> (ø)
opensearch-2.x-v2 1.70% <ø> (ø)
opensearch-3.x-v2 1.70% <ø> (ø)
query 1.70% <ø> (ø)
tailsampling-processor 0.47% <ø> (ø)
unittests 95.47% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yurishkuro yurishkuro added the changelog:experimental Change to an experimental part of the code label Aug 9, 2025
@yurishkuro yurishkuro merged commit 952c77a into jaegertracing:main Aug 9, 2025
63 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:experimental Change to an experimental part of the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants