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

[extension/zpages] zpagesextension using confighttp.ServerConfig #10276

Merged
merged 3 commits into from
Jun 12, 2024

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented May 31, 2024

Description

use confighttp.ServerConfig as part of the configuration of the zpagesextension

Link to tracking issue

Fixes #9368

@atoulme atoulme requested review from a team and djaglowski May 31, 2024 05:22
@atoulme atoulme force-pushed the zpagesextension_confighttp branch 2 times, most recently from e08ee4e to 2d8b5d9 Compare May 31, 2024 05:55
@atoulme atoulme force-pushed the zpagesextension_confighttp branch 4 times, most recently from 5c3608b to cb7c310 Compare May 31, 2024 06:56
Copy link

codecov bot commented May 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.39%. Comparing base (422eaf2) to head (6ba4ca3).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10276      +/-   ##
==========================================
+ Coverage   92.38%   92.39%   +0.01%     
==========================================
  Files         387      387              
  Lines       18308    18312       +4     
==========================================
+ Hits        16913    16920       +7     
+ Misses       1049     1046       -3     
  Partials      346      346              

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

@atoulme atoulme force-pushed the zpagesextension_confighttp branch 5 times, most recently from 19bf9d8 to 2f1e830 Compare May 31, 2024 20:28
@atoulme atoulme force-pushed the zpagesextension_confighttp branch from 2f1e830 to 2293c1a Compare May 31, 2024 20:55
@atoulme atoulme force-pushed the zpagesextension_confighttp branch from 2293c1a to 9a168a6 Compare June 5, 2024 15:55
@atoulme atoulme requested a review from codeboten June 5, 2024 23:04
@atoulme
Copy link
Contributor Author

atoulme commented Jun 12, 2024

Ready to merge?

@codeboten codeboten merged commit 1889d58 into open-telemetry:main Jun 12, 2024
49 checks passed
@github-actions github-actions bot added this to the next release milestone Jun 12, 2024
@atoulme atoulme deleted the zpagesextension_confighttp branch July 19, 2024 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[extension/zpages] zpages extension should use confighttp.HTTPServerSettings
2 participants