Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

queryfrontend: grant jaeger-query access to pki certs #1051

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

frzifus
Copy link
Collaborator

@frzifus frzifus commented Oct 8, 2024

Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>
@frzifus frzifus force-pushed the fix/jaeger-query-certs branch from 2cea897 to cf8288a Compare October 8, 2024 00:35
@codecov-commenter
Copy link

codecov-commenter commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.20%. Comparing base (c7eb01b) to head (d06ac27).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1051   +/-   ##
=======================================
  Coverage   69.20%   69.20%           
=======================================
  Files         110      110           
  Lines        7049     7049           
=======================================
  Hits         4878     4878           
  Misses       1881     1881           
  Partials      290      290           
Flag Coverage Δ
unittests 69.20% <100.00%> (ø)

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.

@andreasgerstmayr andreasgerstmayr changed the title queryfrontend: grant jaeer-query access to pki certs queryfrontend: grant jaeger-query access to pki certs Oct 8, 2024
change_type: bug_fix

# The name of the component, or a single word describing the area of concern, (e.g. tempostack, tempomonolithic, github action)
component: tempostack, tempomonolithic
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
component: tempostack, tempomonolithic
component: tempostack

only affects tempostack, in tempomonolithic all runs in the same pod and therefore doesn't require mTLS

@andreasgerstmayr andreasgerstmayr enabled auto-merge (squash) October 8, 2024 11:15
@andreasgerstmayr andreasgerstmayr merged commit 642fc96 into grafana:main Oct 8, 2024
11 checks passed
@frzifus frzifus deleted the fix/jaeger-query-certs branch October 8, 2024 11:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants