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.

Sign up for GitHub

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

Jump to bottom

ci: Build with --otel on dev distros #1487

Merged
merged 1 commit into from
Dec 5, 2024
Merged

ci: Build with --otel on dev distros #1487

merged 1 commit into from
Dec 5, 2024
+8 −6

Conversation

ac000
Copy link
Member

@ac000 ac000 commented Nov 12, 2024 •
edited
Loading 

    This just builds the OTEL stuff for the Fedora Rawhide and Alpine Edge
    distributions.
    
    If we ever get test cases covering OTEL we can figure out the best way
    to do it in the ci.yaml, but right now I don't see the point in building
    OTEL in every test configuration there...

@ac000 ac000 force-pushed the ci-otel branch 2 times, most recently from 296d118 to 953ceb8 Compare November 12, 2024 20:49
@ac000 ac000 marked this pull request as ready for review November 12, 2024 20:53
@ac000
ci: Build with --otel on dev distros
ea5a4d6 
This just builds the OTEL stuff for the Fedora Rawhide and Alpine Edge
distributions.

If we ever get test cases covering OTEL we can figure out the best way
to do it in the ci.yaml, but right now I don't see the point in building
OTEL in every test configuration there...

Signed-off-by: Andrew Clayton <[email protected]>
@ac000
Copy link
Member Author

ac000 commented Dec 5, 2024

Rebased with master

$ git range-diff 953ceb8b...ea5a4d6e
 -:  -------- >  1:  2d4624f0 Make nxt_tstr_is_js() macro public in header
 -:  -------- >  2:  a92d8149 http: Refactor format field in nxt_router_access_log_conf_t
 -:  -------- >  3:  a760e24a http: Introduce nxt_router_access_log_format_t structure
 -:  -------- >  4:  bc838c5e http: Support JSON format in access log
 -:  -------- >  5:  a3551790 tests: Add tests for JSON format access log
 -:  -------- >  6:  4f041328 Decast nxt_cpymem()
 -:  -------- >  7:  6cc4d706 wasm: Fix build with wasmtime 27.0.0
 -:  -------- >  8:  1dfe5bb0 tools/unitctl: bump anstream from 0.6.5 to 0.6.15
 1:  953ceb8b =  9:  ea5a4d6e ci: Build with --otel on dev distros

@ac000 ac000 merged commit ea5a4d6 into nginx:master Dec 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@avahahn avahahn avahahn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ac000 @avahahn