Skip to content

[APMSP-1976] Add check of parsed config and return action needed for tracer flare #1085

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

anais-raison
Copy link
Contributor

@anais-raison anais-raison commented Jun 3, 2025

What does this PR do?

This PR add a function that check the config received by the listener of RemoteConfig. This way it returns, through an enum, which action have to be done by the tracer flare depending of the config.

Motivation

What inspired you to submit this pull request?

Additional Notes

Also I updated the CODEOWNER file because of the crate datadog-dynamic-configuration was remove in my previous PR.

How to test the change?

Describe here in detail how the change can be validated.

@pr-commenter
Copy link

pr-commenter bot commented Jun 3, 2025

Benchmarks

Comparison

Benchmark execution time: 2025-06-06 10:09:13

Comparing candidate commit 26e5ac3 in PR branch anais/check-config-tracer-flare with baseline commit 6b73967 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 52 metrics, 2 unstable metrics.

Candidate

Candidate benchmark details

Group 1

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
benching string interning on wordpress profile execution_time 149.541µs 150.422µs ± 0.375µs 150.383µs ± 0.184µs 150.564µs 150.866µs 151.420µs 153.239µs 1.90% 3.209 20.052 0.25% 0.027µs 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
benching string interning on wordpress profile execution_time [150.370µs; 150.474µs] or [-0.035%; +0.035%] None None None

Group 2

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
concentrator/add_spans_to_concentrator execution_time 8.150ms 8.166ms ± 0.012ms 8.164ms ± 0.006ms 8.172ms 8.182ms 8.196ms 8.272ms 1.32% 3.951 30.283 0.15% 0.001ms 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
concentrator/add_spans_to_concentrator execution_time [8.165ms; 8.168ms] or [-0.020%; +0.020%] None None None

Group 3

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
ip_address/quantize_peer_ip_address_benchmark execution_time 5.031µs 5.090µs ± 0.040µs 5.073µs ± 0.027µs 5.132µs 5.147µs 5.150µs 5.155µs 1.61% 0.257 -1.648 0.78% 0.003µs 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
ip_address/quantize_peer_ip_address_benchmark execution_time [5.085µs; 5.096µs] or [-0.109%; +0.109%] None None None

Group 4

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
normalization/normalize_name/normalize_name/Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Lo... execution_time 204.338µs 204.779µs ± 0.307µs 204.712µs ± 0.195µs 204.930µs 205.420µs 205.729µs 205.763µs 0.51% 1.151 1.060 0.15% 0.022µs 1 200
normalization/normalize_name/normalize_name/Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Lo... throughput 4859957.692op/s 4883321.521op/s ± 7313.556op/s 4884914.388op/s ± 4660.027op/s 4889034.121op/s 4891581.533op/s 4892957.012op/s 4893846.517op/s 0.18% -1.143 1.037 0.15% 517.147op/s 1 200
normalization/normalize_name/normalize_name/bad-name execution_time 18.583µs 18.713µs ± 0.113µs 18.701µs ± 0.042µs 18.743µs 18.810µs 18.932µs 20.020µs 7.05% 7.862 86.925 0.60% 0.008µs 1 200
normalization/normalize_name/normalize_name/bad-name throughput 49949901.942op/s 53441901.247op/s ± 309466.477op/s 53473754.865op/s ± 121174.840op/s 53596659.620op/s 53719622.977op/s 53763597.900op/s 53812599.901op/s 0.63% -7.382 79.464 0.58% 21882.584op/s 1 200
normalization/normalize_name/normalize_name/good execution_time 10.639µs 10.725µs ± 0.036µs 10.722µs ± 0.022µs 10.747µs 10.782µs 10.821µs 10.834µs 1.04% 0.256 0.239 0.33% 0.003µs 1 200
normalization/normalize_name/normalize_name/good throughput 92302467.435op/s 93243999.757op/s ± 309888.034op/s 93263724.931op/s ± 188537.687op/s 93435646.556op/s 93754554.674op/s 93888650.440op/s 93994201.060op/s 0.78% -0.234 0.219 0.33% 21912.393op/s 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
normalization/normalize_name/normalize_name/Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Lo... execution_time [204.737µs; 204.822µs] or [-0.021%; +0.021%] None None None
normalization/normalize_name/normalize_name/Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Long-.Too-Lo... throughput [4882307.933op/s; 4884335.110op/s] or [-0.021%; +0.021%] None None None
normalization/normalize_name/normalize_name/bad-name execution_time [18.697µs; 18.728µs] or [-0.084%; +0.084%] None None None
normalization/normalize_name/normalize_name/bad-name throughput [53399012.169op/s; 53484790.324op/s] or [-0.080%; +0.080%] None None None
normalization/normalize_name/normalize_name/good execution_time [10.720µs; 10.730µs] or [-0.046%; +0.046%] None None None
normalization/normalize_name/normalize_name/good throughput [93201052.256op/s; 93286947.258op/s] or [-0.046%; +0.046%] None None None

Group 5

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
credit_card/is_card_number/ execution_time 3.891µs 3.912µs ± 0.005µs 3.912µs ± 0.002µs 3.914µs 3.918µs 3.922µs 3.952µs 1.02% 4.470 35.986 0.13% 0.000µs 1 200
credit_card/is_card_number/ throughput 253047179.890op/s 255594556.014op/s ± 326255.688op/s 255637036.598op/s ± 111146.361op/s 255738610.137op/s 255852364.058op/s 255925287.207op/s 256973457.667op/s 0.52% -4.401 35.452 0.13% 23069.761op/s 1 200
credit_card/is_card_number/ 3782-8224-6310-005 execution_time 76.543µs 78.615µs ± 0.832µs 78.596µs ± 0.597µs 79.194µs 80.000µs 80.587µs 80.890µs 2.92% 0.174 -0.253 1.06% 0.059µs 1 200
credit_card/is_card_number/ 3782-8224-6310-005 throughput 12362413.819op/s 12721605.254op/s ± 134372.328op/s 12723349.509op/s ± 96201.278op/s 12816068.354op/s 12930061.324op/s 13006849.519op/s 13064502.613op/s 2.68% -0.120 -0.272 1.05% 9501.558op/s 1 200
credit_card/is_card_number/ 378282246310005 execution_time 70.014µs 71.237µs ± 0.658µs 71.174µs ± 0.455µs 71.656µs 72.368µs 72.899µs 73.761µs 3.64% 0.682 0.365 0.92% 0.047µs 1 200
credit_card/is_card_number/ 378282246310005 throughput 13557241.269op/s 14038760.905op/s ± 128934.815op/s 14050173.205op/s ± 90041.572op/s 14139527.406op/s 14217633.434op/s 14256912.446op/s 14282898.225op/s 1.66% -0.631 0.230 0.92% 9117.068op/s 1 200
credit_card/is_card_number/37828224631 execution_time 3.892µs 3.912µs ± 0.003µs 3.912µs ± 0.002µs 3.914µs 3.917µs 3.919µs 3.921µs 0.21% -0.937 7.849 0.08% 0.000µs 1 200
credit_card/is_card_number/37828224631 throughput 255067393.775op/s 255610015.727op/s ± 200319.732op/s 255613240.766op/s ± 130064.998op/s 255739793.767op/s 255861731.199op/s 255887257.269op/s 256919332.154op/s 0.51% 0.958 7.993 0.08% 14164.744op/s 1 200
credit_card/is_card_number/378282246310005 execution_time 66.782µs 68.247µs ± 0.677µs 68.213µs ± 0.465µs 68.696µs 69.347µs 70.008µs 70.071µs 2.72% 0.218 -0.251 0.99% 0.048µs 1 200
credit_card/is_card_number/378282246310005 throughput 14271308.447op/s 14654046.134op/s ± 145191.722op/s 14659946.419op/s ± 100409.415op/s 14757170.591op/s 14901495.242op/s 14939891.194op/s 14974003.297op/s 2.14% -0.168 -0.292 0.99% 10266.605op/s 1 200
credit_card/is_card_number/37828224631000521389798 execution_time 52.148µs 52.208µs ± 0.032µs 52.208µs ± 0.022µs 52.229µs 52.266µs 52.292µs 52.313µs 0.20% 0.474 0.049 0.06% 0.002µs 1 200
credit_card/is_card_number/37828224631000521389798 throughput 19115787.311op/s 19154159.071op/s ± 11861.658op/s 19154173.640op/s ± 8202.950op/s 19162899.190op/s 19171544.349op/s 19174043.007op/s 19176227.231op/s 0.12% -0.470 0.043 0.06% 838.746op/s 1 200
credit_card/is_card_number/x371413321323331 execution_time 6.027µs 6.034µs ± 0.004µs 6.033µs ± 0.002µs 6.036µs 6.040µs 6.043µs 6.073µs 0.65% 3.796 29.783 0.07% 0.000µs 1 200
credit_card/is_card_number/x371413321323331 throughput 164673769.536op/s 165731785.642op/s ± 119274.589op/s 165749749.803op/s ± 61086.451op/s 165808588.045op/s 165863230.982op/s 165885265.524op/s 165916903.168op/s 0.10% -3.759 29.337 0.07% 8433.987op/s 1 200
credit_card/is_card_number_no_luhn/ execution_time 3.890µs 3.911µs ± 0.003µs 3.911µs ± 0.002µs 3.913µs 3.916µs 3.918µs 3.920µs 0.23% -1.588 14.705 0.07% 0.000µs 1 200
credit_card/is_card_number_no_luhn/ throughput 255073310.384op/s 255656714.778op/s ± 188184.232op/s 255665216.748op/s ± 109981.160op/s 255771014.840op/s 255864504.251op/s 255899162.253op/s 257071088.651op/s 0.55% 1.619 14.959 0.07% 13306.635op/s 1 200
credit_card/is_card_number_no_luhn/ 3782-8224-6310-005 execution_time 64.226µs 64.452µs ± 0.147µs 64.422µs ± 0.089µs 64.530µs 64.722µs 64.859µs 65.258µs 1.30% 1.359 3.721 0.23% 0.010µs 1 200
credit_card/is_card_number_no_luhn/ 3782-8224-6310-005 throughput 15323851.929op/s 15515573.146op/s ± 35232.339op/s 15522597.345op/s ± 21311.073op/s 15538456.060op/s 15560183.358op/s 15565053.240op/s 15569948.443op/s 0.31% -1.333 3.560 0.23% 2491.303op/s 1 200
credit_card/is_card_number_no_luhn/ 378282246310005 execution_time 58.144µs 58.383µs ± 0.145µs 58.348µs ± 0.064µs 58.426µs 58.686µs 58.860µs 58.977µs 1.08% 1.549 2.796 0.25% 0.010µs 1 200
credit_card/is_card_number_no_luhn/ 378282246310005 throughput 16955792.503op/s 17128370.371op/s ± 42250.074op/s 17138523.261op/s ± 18701.377op/s 17156686.230op/s 17175280.135op/s 17185457.581op/s 17198772.008op/s 0.35% -1.531 2.726 0.25% 2987.531op/s 1 200
credit_card/is_card_number_no_luhn/37828224631 execution_time 3.896µs 3.911µs ± 0.002µs 3.911µs ± 0.001µs 3.913µs 3.916µs 3.918µs 3.918µs 0.19% -0.659 6.871 0.06% 0.000µs 1 200
credit_card/is_card_number_no_luhn/37828224631 throughput 255202406.348op/s 255660010.175op/s ± 162122.915op/s 255683150.481op/s ± 97001.330op/s 255761082.152op/s 255846382.509op/s 255935700.477op/s 256675392.158op/s 0.39% 0.675 6.966 0.06% 11463.821op/s 1 200
credit_card/is_card_number_no_luhn/378282246310005 execution_time 54.502µs 54.874µs ± 0.250µs 54.766µs ± 0.101µs 54.996µs 55.388µs 55.655µs 55.834µs 1.95% 1.439 1.757 0.45% 0.018µs 1 200
credit_card/is_card_number_no_luhn/378282246310005 throughput 17910305.776op/s 18223871.395op/s ± 82570.019op/s 18259477.654op/s ± 33840.712op/s 18283884.586op/s 18298854.039op/s 18315112.753op/s 18347853.036op/s 0.48% -1.417 1.665 0.45% 5838.582op/s 1 200
credit_card/is_card_number_no_luhn/37828224631000521389798 execution_time 52.147µs 52.233µs ± 0.048µs 52.224µs ± 0.029µs 52.265µs 52.314µs 52.362µs 52.400µs 0.34% 0.684 0.132 0.09% 0.003µs 1 200
credit_card/is_card_number_no_luhn/37828224631000521389798 throughput 19083848.067op/s 19144929.008op/s ± 17459.872op/s 19148259.488op/s ± 10813.548op/s 19157555.003op/s 19169137.497op/s 19173550.394op/s 19176451.318op/s 0.15% -0.680 0.122 0.09% 1234.599op/s 1 200
credit_card/is_card_number_no_luhn/x371413321323331 execution_time 6.027µs 6.035µs ± 0.005µs 6.035µs ± 0.003µs 6.038µs 6.043µs 6.049µs 6.067µs 0.54% 1.747 7.558 0.08% 0.000µs 1 200
credit_card/is_card_number_no_luhn/x371413321323331 throughput 164826010.082op/s 165691650.740op/s ± 135880.328op/s 165708575.429op/s ± 77401.179op/s 165785288.242op/s 165873134.271op/s 165899627.189op/s 165916753.647op/s 0.13% -1.731 7.438 0.08% 9608.190op/s 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
credit_card/is_card_number/ execution_time [3.912µs; 3.913µs] or [-0.018%; +0.018%] None None None
credit_card/is_card_number/ throughput [255549340.114op/s; 255639771.915op/s] or [-0.018%; +0.018%] None None None
credit_card/is_card_number/ 3782-8224-6310-005 execution_time [78.500µs; 78.730µs] or [-0.147%; +0.147%] None None None
credit_card/is_card_number/ 3782-8224-6310-005 throughput [12702982.542op/s; 12740227.966op/s] or [-0.146%; +0.146%] None None None
credit_card/is_card_number/ 378282246310005 execution_time [71.146µs; 71.329µs] or [-0.128%; +0.128%] None None None
credit_card/is_card_number/ 378282246310005 throughput [14020891.779op/s; 14056630.030op/s] or [-0.127%; +0.127%] None None None
credit_card/is_card_number/37828224631 execution_time [3.912µs; 3.913µs] or [-0.011%; +0.011%] None None None
credit_card/is_card_number/37828224631 throughput [255582253.339op/s; 255637778.115op/s] or [-0.011%; +0.011%] None None None
credit_card/is_card_number/378282246310005 execution_time [68.153µs; 68.341µs] or [-0.138%; +0.138%] None None None
credit_card/is_card_number/378282246310005 throughput [14633923.958op/s; 14674168.310op/s] or [-0.137%; +0.137%] None None None
credit_card/is_card_number/37828224631000521389798 execution_time [52.204µs; 52.212µs] or [-0.009%; +0.009%] None None None
credit_card/is_card_number/37828224631000521389798 throughput [19152515.160op/s; 19155802.983op/s] or [-0.009%; +0.009%] None None None
credit_card/is_card_number/x371413321323331 execution_time [6.033µs; 6.034µs] or [-0.010%; +0.010%] None None None
credit_card/is_card_number/x371413321323331 throughput [165715255.331op/s; 165748315.953op/s] or [-0.010%; +0.010%] None None None
credit_card/is_card_number_no_luhn/ execution_time [3.911µs; 3.912µs] or [-0.010%; +0.010%] None None None
credit_card/is_card_number_no_luhn/ throughput [255630634.253op/s; 255682795.302op/s] or [-0.010%; +0.010%] None None None
credit_card/is_card_number_no_luhn/ 3782-8224-6310-005 execution_time [64.431µs; 64.472µs] or [-0.032%; +0.032%] None None None
credit_card/is_card_number_no_luhn/ 3782-8224-6310-005 throughput [15510690.283op/s; 15520456.009op/s] or [-0.031%; +0.031%] None None None
credit_card/is_card_number_no_luhn/ 378282246310005 execution_time [58.363µs; 58.403µs] or [-0.034%; +0.034%] None None None
credit_card/is_card_number_no_luhn/ 378282246310005 throughput [17122514.917op/s; 17134225.825op/s] or [-0.034%; +0.034%] None None None
credit_card/is_card_number_no_luhn/37828224631 execution_time [3.911µs; 3.912µs] or [-0.009%; +0.009%] None None None
credit_card/is_card_number_no_luhn/37828224631 throughput [255637541.498op/s; 255682478.852op/s] or [-0.009%; +0.009%] None None None
credit_card/is_card_number_no_luhn/378282246310005 execution_time [54.840µs; 54.909µs] or [-0.063%; +0.063%] None None None
credit_card/is_card_number_no_luhn/378282246310005 throughput [18212427.985op/s; 18235314.806op/s] or [-0.063%; +0.063%] None None None
credit_card/is_card_number_no_luhn/37828224631000521389798 execution_time [52.227µs; 52.240µs] or [-0.013%; +0.013%] None None None
credit_card/is_card_number_no_luhn/37828224631000521389798 throughput [19142509.237op/s; 19147348.778op/s] or [-0.013%; +0.013%] None None None
credit_card/is_card_number_no_luhn/x371413321323331 execution_time [6.035µs; 6.036µs] or [-0.011%; +0.011%] None None None
credit_card/is_card_number_no_luhn/x371413321323331 throughput [165672819.034op/s; 165710482.447op/s] or [-0.011%; +0.011%] None None None

Group 6

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
benching deserializing traces from msgpack to their internal representation execution_time 72.251ms 72.448ms ± 0.388ms 72.391ms ± 0.045ms 72.440ms 72.629ms 73.016ms 77.517ms 7.08% 11.448 144.947 0.53% 0.027ms 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
benching deserializing traces from msgpack to their internal representation execution_time [72.394ms; 72.502ms] or [-0.074%; +0.074%] None None None

Group 7

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
normalization/normalize_service/normalize_service/A0000000000000000000000000000000000000000000000000... execution_time 532.238µs 533.099µs ± 0.791µs 533.008µs ± 0.280µs 533.313µs 533.808µs 534.383µs 542.556µs 1.79% 8.653 100.597 0.15% 0.056µs 1 200
normalization/normalize_service/normalize_service/A0000000000000000000000000000000000000000000000000... throughput 1843127.314op/s 1875828.934op/s ± 2747.299op/s 1876145.690op/s ± 985.889op/s 1877037.586op/s 1878203.539op/s 1878689.419op/s 1878857.460op/s 0.14% -8.530 98.629 0.15% 194.263op/s 1 200
normalization/normalize_service/normalize_service/Data🐨dog🐶 繋がっ⛰てて execution_time 381.353µs 381.699µs ± 0.192µs 381.674µs ± 0.112µs 381.792µs 382.072µs 382.308µs 382.466µs 0.21% 1.034 1.613 0.05% 0.014µs 1 200
normalization/normalize_service/normalize_service/Data🐨dog🐶 繋がっ⛰てて throughput 2614614.764op/s 2619864.944op/s ± 1315.958op/s 2620036.697op/s ± 766.303op/s 2620751.630op/s 2621674.044op/s 2622121.050op/s 2622243.580op/s 0.08% -1.030 1.601 0.05% 93.052op/s 1 200
normalization/normalize_service/normalize_service/Test Conversion 0f Weird !@#$%^&**() Characters execution_time 189.131µs 189.892µs ± 0.249µs 189.859µs ± 0.153µs 190.046µs 190.318µs 190.447µs 190.625µs 0.40% 0.046 0.417 0.13% 0.018µs 1 200
normalization/normalize_service/normalize_service/Test Conversion 0f Weird !@#$%^&**() Characters throughput 5245893.172op/s 5266167.195op/s ± 6908.687op/s 5267078.350op/s ± 4238.141op/s 5270398.291op/s 5276753.404op/s 5283918.880op/s 5287329.768op/s 0.38% -0.037 0.423 0.13% 488.518op/s 1 200
normalization/normalize_service/normalize_service/[empty string] execution_time 37.664µs 37.764µs ± 0.047µs 37.758µs ± 0.032µs 37.796µs 37.839µs 37.883µs 37.946µs 0.50% 0.260 0.309 0.12% 0.003µs 1 200
normalization/normalize_service/normalize_service/[empty string] throughput 26353345.659op/s 26480485.381op/s ± 33061.917op/s 26484468.154op/s ± 22173.909op/s 26501099.096op/s 26535031.974op/s 26548141.187op/s 26550581.450op/s 0.25% -0.252 0.292 0.12% 2337.831op/s 1 200
normalization/normalize_service/normalize_service/test_ASCII execution_time 44.929µs 45.094µs ± 0.157µs 45.074µs ± 0.138µs 45.252µs 45.335µs 45.402µs 45.437µs 0.80% 0.373 -1.409 0.35% 0.011µs 1 200
normalization/normalize_service/normalize_service/test_ASCII throughput 22008715.941op/s 22176090.260op/s ± 77238.287op/s 22185670.646op/s ± 67956.664op/s 22252441.587op/s 22255345.870op/s 22256430.246op/s 22257438.436op/s 0.32% -0.368 -1.416 0.35% 5461.572op/s 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
normalization/normalize_service/normalize_service/A0000000000000000000000000000000000000000000000000... execution_time [532.989µs; 533.208µs] or [-0.021%; +0.021%] None None None
normalization/normalize_service/normalize_service/A0000000000000000000000000000000000000000000000000... throughput [1875448.185op/s; 1876209.683op/s] or [-0.020%; +0.020%] None None None
normalization/normalize_service/normalize_service/Data🐨dog🐶 繋がっ⛰てて execution_time [381.673µs; 381.726µs] or [-0.007%; +0.007%] None None None
normalization/normalize_service/normalize_service/Data🐨dog🐶 繋がっ⛰てて throughput [2619682.565op/s; 2620047.323op/s] or [-0.007%; +0.007%] None None None
normalization/normalize_service/normalize_service/Test Conversion 0f Weird !@#$%^&**() Characters execution_time [189.857µs; 189.926µs] or [-0.018%; +0.018%] None None None
normalization/normalize_service/normalize_service/Test Conversion 0f Weird !@#$%^&**() Characters throughput [5265209.718op/s; 5267124.673op/s] or [-0.018%; +0.018%] None None None
normalization/normalize_service/normalize_service/[empty string] execution_time [37.757µs; 37.770µs] or [-0.017%; +0.017%] None None None
normalization/normalize_service/normalize_service/[empty string] throughput [26475903.317op/s; 26485067.444op/s] or [-0.017%; +0.017%] None None None
normalization/normalize_service/normalize_service/test_ASCII execution_time [45.072µs; 45.116µs] or [-0.048%; +0.048%] None None None
normalization/normalize_service/normalize_service/test_ASCII throughput [22165385.776op/s; 22186794.744op/s] or [-0.048%; +0.048%] None None None

Group 8

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
redis/obfuscate_redis_string execution_time 34.168µs 34.852µs ± 1.023µs 34.406µs ± 0.143µs 34.599µs 37.017µs 37.064µs 38.256µs 11.19% 1.682 1.037 2.93% 0.072µs 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
redis/obfuscate_redis_string execution_time [34.710µs; 34.993µs] or [-0.407%; +0.407%] None None None

Group 9

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
two way interface execution_time 17.593µs 25.309µs ± 9.661µs 18.014µs ± 0.204µs 34.031µs 43.504µs 45.424µs 72.514µs 302.54% 1.143 1.467 38.08% 0.683µs 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
two way interface execution_time [23.970µs; 26.648µs] or [-5.290%; +5.290%] None None None

Group 10

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
tags/replace_trace_tags execution_time 2.400µs 2.433µs ± 0.015µs 2.428µs ± 0.006µs 2.438µs 2.470µs 2.478µs 2.484µs 2.32% 1.219 1.426 0.63% 0.001µs 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
tags/replace_trace_tags execution_time [2.431µs; 2.435µs] or [-0.088%; +0.088%] None None None

Group 11

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
write only interface execution_time 1.197µs 3.246µs ± 1.484µs 3.021µs ± 0.027µs 3.051µs 3.697µs 14.079µs 15.665µs 418.63% 7.417 56.089 45.61% 0.105µs 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
write only interface execution_time [3.040µs; 3.452µs] or [-6.337%; +6.337%] None None None

Group 12

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
sql/obfuscate_sql_string execution_time 89.530µs 89.952µs ± 0.156µs 89.952µs ± 0.082µs 90.025µs 90.129µs 90.402µs 91.091µs 1.27% 2.379 14.817 0.17% 0.011µs 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
sql/obfuscate_sql_string execution_time [89.930µs; 89.973µs] or [-0.024%; +0.024%] None None None

Group 13

cpu_model git_commit_sha git_commit_date git_branch
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz 26e5ac3 1749203866 anais/check-config-tracer-flare
scenario metric min mean ± sd median ± mad p75 p95 p99 max peak_to_median_ratio skewness kurtosis cv sem runs sample_size
normalization/normalize_trace/test_trace execution_time 247.303ns 257.293ns ± 14.217ns 249.818ns ± 1.967ns 259.719ns 287.615ns 304.184ns 304.382ns 21.84% 1.808 2.299 5.51% 1.005ns 1 200
scenario metric 95% CI mean Shapiro-Wilk pvalue Ljung-Box pvalue (lag=1) Dip test pvalue
normalization/normalize_trace/test_trace execution_time [255.323ns; 259.264ns] or [-0.766%; +0.766%] None None None

Baseline

Omitted due to size.

@codecov-commenter
Copy link

codecov-commenter commented Jun 3, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 22 lines in your changes missing coverage. Please review.

Project coverage is 70.95%. Comparing base (6b73967) to head (26e5ac3).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1085      +/-   ##
==========================================
+ Coverage   70.94%   70.95%   +0.01%     
==========================================
  Files         334      334              
  Lines       50549    50584      +35     
==========================================
+ Hits        35860    35893      +33     
- Misses      14689    14691       +2     
Components Coverage Δ
datadog-crashtracker 44.41% <ø> (+0.10%) ⬆️
datadog-crashtracker-ffi 6.03% <ø> (ø)
datadog-alloc 98.73% <ø> (ø)
data-pipeline 90.43% <ø> (ø)
data-pipeline-ffi 89.26% <ø> (ø)
ddcommon 79.07% <ø> (ø)
ddcommon-ffi 68.64% <ø> (ø)
ddtelemetry 60.56% <ø> (-0.08%) ⬇️
ddtelemetry-ffi 21.32% <ø> (ø)
dogstatsd-client 83.26% <ø> (ø)
datadog-ipc 82.58% <ø> (ø)
datadog-profiling 77.17% <ø> (ø)
datadog-profiling-ffi 62.12% <ø> (ø)
datadog-sidecar 42.32% <ø> (ø)
datdog-sidecar-ffi 10.36% <ø> (ø)
spawn-worker 55.35% <ø> (ø)
tinybytes 90.96% <ø> (ø)
datadog-trace-normalization 98.24% <ø> (ø)
datadog-trace-obfuscation 94.16% <ø> (ø)
datadog-trace-protobuf 77.10% <ø> (ø)
datadog-trace-utils 89.44% <ø> (ø)
datadog-log 66.15% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@r1viollet
Copy link
Contributor

r1viollet commented Jun 3, 2025

Artifact Size Benchmark Report

aarch64-alpine-linux-musl
Artifact Baseline Commit Change
/aarch64-alpine-linux-musl/lib/libdatadog_profiling.a 70.23 MB 70.23 MB 0% (0 B) 👌
/aarch64-alpine-linux-musl/lib/libdatadog_profiling.so 9.16 MB 9.16 MB 0% (0 B) 👌
/aarch64-alpine-linux-musl/lib/libdatadog_profiling.so.debug 22.17 MB 22.17 MB 0% (0 B) 👌
aarch64-unknown-linux-gnu
Artifact Baseline Commit Change
/aarch64-unknown-linux-gnu/lib/libdatadog_profiling.so.debug 26.03 MB 26.03 MB 0% (0 B) 👌
/aarch64-unknown-linux-gnu/lib/libdatadog_profiling.so 9.08 MB 9.08 MB 0% (0 B) 👌
/aarch64-unknown-linux-gnu/lib/libdatadog_profiling.a 81.89 MB 81.89 MB 0% (0 B) 👌
libdatadog-x64-windows
Artifact Baseline Commit Change
/libdatadog-x64-windows/debug/dynamic/datadog_profiling_ffi.dll 15.99 MB 15.99 MB 0% (0 B) 👌
/libdatadog-x64-windows/debug/dynamic/datadog_profiling_ffi.lib 62.65 KB 62.65 KB 0% (0 B) 👌
/libdatadog-x64-windows/debug/dynamic/datadog_profiling_ffi.pdb 108.95 MB 108.94 MB -0% (-8.00 KB) 👌
/libdatadog-x64-windows/debug/static/datadog_profiling_ffi.lib 577.71 MB 577.71 MB 0% (0 B) 👌
/libdatadog-x64-windows/release/dynamic/datadog_profiling_ffi.dll 5.00 MB 5.00 MB 0% (0 B) 👌
/libdatadog-x64-windows/release/dynamic/datadog_profiling_ffi.lib 62.65 KB 62.65 KB 0% (0 B) 👌
/libdatadog-x64-windows/release/dynamic/datadog_profiling_ffi.pdb 15.53 MB 15.53 MB 0% (0 B) 👌
/libdatadog-x64-windows/release/static/datadog_profiling_ffi.lib 28.56 MB 28.56 MB 0% (0 B) 👌
libdatadog-x86-windows
Artifact Baseline Commit Change
/libdatadog-x86-windows/debug/dynamic/datadog_profiling_ffi.dll 13.55 MB 13.55 MB 0% (0 B) 👌
/libdatadog-x86-windows/debug/dynamic/datadog_profiling_ffi.lib 63.60 KB 63.60 KB 0% (0 B) 👌
/libdatadog-x86-windows/debug/dynamic/datadog_profiling_ffi.pdb 110.54 MB 110.53 MB -0% (-8.00 KB) 👌
/libdatadog-x86-windows/debug/static/datadog_profiling_ffi.lib 568.91 MB 568.91 MB 0% (0 B) 👌
/libdatadog-x86-windows/release/dynamic/datadog_profiling_ffi.dll 3.78 MB 3.78 MB 0% (0 B) 👌
/libdatadog-x86-windows/release/dynamic/datadog_profiling_ffi.lib 63.60 KB 63.60 KB 0% (0 B) 👌
/libdatadog-x86-windows/release/dynamic/datadog_profiling_ffi.pdb 16.51 MB 16.51 MB 0% (0 B) 👌
/libdatadog-x86-windows/release/static/datadog_profiling_ffi.lib 26.78 MB 26.78 MB 0% (0 B) 👌
x86_64-alpine-linux-musl
Artifact Baseline Commit Change
/x86_64-alpine-linux-musl/lib/libdatadog_profiling.a 62.87 MB 62.87 MB 0% (0 B) 👌
/x86_64-alpine-linux-musl/lib/libdatadog_profiling.so 9.78 MB 9.78 MB 0% (0 B) 👌
/x86_64-alpine-linux-musl/lib/libdatadog_profiling.so.debug 20.98 MB 20.98 MB 0% (0 B) 👌
x86_64-unknown-linux-gnu
Artifact Baseline Commit Change
/x86_64-unknown-linux-gnu/lib/libdatadog_profiling.a 76.88 MB 76.88 MB 0% (0 B) 👌
/x86_64-unknown-linux-gnu/lib/libdatadog_profiling.so 9.68 MB 9.68 MB 0% (0 B) 👌
/x86_64-unknown-linux-gnu/lib/libdatadog_profiling.so.debug 23.97 MB 23.97 MB 0% (0 B) 👌

@anais-raison anais-raison marked this pull request as ready for review June 4, 2025 11:31
@anais-raison anais-raison requested a review from a team as a code owner June 4, 2025 11:31
@@ -186,6 +219,7 @@ pub async fn run_remote_config_listener(
Change::Add(file) => {
println!("Added file: {} (version: {})", file.path(), file.version());
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if I asked this on a previous PR, but why are we printing to stdout here?

/// * `Ok(ReturnAction::Stop)` - If AGENT_TASK with the right properties.
/// * `Ok(ReturnAction::None)` - Else.
/// * `FlareError(msg)` - If something fail.
pub fn check_remote_config_file(file: RemoteConfigFile) -> Result<ReturnAction, FlareError> {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this function covered by tests somewhere else? If not, anything preventing us from adding some?

"error" => ReturnAction::StartError,
"critical" => ReturnAction::StartCritical,
"off" => ReturnAction::StartOff,
_ => ReturnAction::None,
Copy link
Contributor

Choose a reason for hiding this comment

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

If we get an unknown log level is just setting it to None the appropriate action? Or should it be an error?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think an error would be the most appropriate. But it could be nice to launch the flare still with a default level.

Comment on lines 44 to 50
StartTrace = 0,
StartDebug = 1,
StartInfo = 2,
StartWarn = 3,
StartError = 4,
StartCritical = 5,
StartOff = 6,
Copy link
Contributor

Choose a reason for hiding this comment

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

Any reason to use explicit discriminants here ?

Copy link
Contributor

@hoolioh hoolioh left a comment

Choose a reason for hiding this comment

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

I would add the crate to the .codecov file so we can the code coverage report.

@@ -23,19 +23,22 @@ pub enum FlareError {
NoFlare(String),
/// Listening to the RemoteConfig failed.
ListeningError(String),
/// Parsing of config failed.
ParsingError(String),
Copy link
Contributor

Choose a reason for hiding this comment

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

It might be beneficial to implement Error trait for FlareError.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants