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

Attribution check, do not merge #1

Draft
wants to merge 8 commits into
base: gabor/otel-review-2-check
Choose a base branch
from

Conversation

javorszky
Copy link
Owner

Tag check

avahahn and others added 8 commits November 7, 2024 18:06
This is purely the source code of the rust end of opentelemetry. It does
not have build tooling wired up yet, nor is this used from the C code.

Co-authored-by: Gabor Javorszky <[email protected]>
Signed-off-by: Gabor Javorszky <[email protected]>
Signed-off-by: Ava Hahn <[email protected]>
Adds the --otel flag to the configure command and the various build time
variables and checks that are needed in this flow.

It also includes the nxt_otel.c and nxt_otel.h files that are needed for
the rest of Unit to talk to the compiled static library that's generated
from the rust crate.

Co-authored-by: Gabor Javorszky <[email protected]>
Signed-off-by: Gabor Javorszky <[email protected]>
Signed-off-by: Ava Hahn <[email protected]>
Enables Unit to parse the tracestate and traceparent headers and add it
to the list, as well as calls to nxt_otel_test_and_call_state.

Signed-off-by: Ava Hahn <[email protected]>
Adds code responsible for users to apply the `telemetry` configuration
options.

configuration snippet as follows:
{
    "settings": {
        "telemetry": {
            "batch_size": 20,
            "endpoint": "http://lgtm:4318/v1/traces",
            "protocol": "http",
            "sampling_ratio": 1
        }
    },
    "listeners": {
        "*:80": {
            "pass": "routes"
        }
    },
    "routes": [
        {
            "match": {
                "headers": {
                    "accept": "*text/html*"
                }
            },
            "action": {
                "share": "/usr/share/unit/welcome/welcome.html"
            }
        },
        {
            "action": {
                "share": "/usr/share/unit/welcome/welcome.md"
            }
        }
    ]
}

Signed-off-by: Gabor Javorszky <[email protected]>
Signed-off-by: Ava Hahn <[email protected]>
Tiny bracket balance fix.

Signed-off-by: Gabor Javorszky <[email protected]>
Signed-off-by: Ava Hahn <[email protected]>
These changes are generated by the openapi generator through a make
command.

Signed-off-by: Gabor Javorszky <[email protected]>
Signed-off-by: Ava Hahn <[email protected]>
This is a body for testing purposes

Co-developed-by: Ava Hahn <[email protected]>
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.

2 participants