Skip to content
This repository was archived by the owner on Oct 8, 2025. It is now read-only.

Conversation

@ac000
Copy link
Member

@ac000 ac000 commented Nov 12, 2024

    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
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 <a.clayton@nginx.com>
@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
@ac000 ac000 deleted the ci-otel branch February 4, 2025 14:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants