Skip to content

Improve Algorithms (testers)#431

Open
thiagoftsm wants to merge 2 commits into
netdata:masterfrom
thiagoftsm:improve_algs
Open

Improve Algorithms (testers)#431
thiagoftsm wants to merge 2 commits into
netdata:masterfrom
thiagoftsm:improve_algs

Conversation

@thiagoftsm

Copy link
Copy Markdown
Contributor
Summary

While during tests with netdata/netdata#22469 some issues were detected. This PR fixes them in this repo.

Test Plan
  1. Get binaries according to your C library from [this](ADD ACTIONS LINK HERE) link and extract them inside a directory, for example: ../artifacts.
    You can also get everything for glibc [here](UPLOAD FILE WITH ALL BINARIES TO SIMPLIFY REVIEWERS).

  2. Extract them running:

    $ for i in `ls *.zip`; do unzip $i; rm .gitkeep ; rm $i; done
    $ for i in `ls *.tar`; do tar -xf $i; rm $i* ; done
  3. Compile branch an run the following tests:

    # make clean; make tester
    # for i in `seq 0 3`; do ./tests/legacy_test --netdata-path ../artifacts --content --iteration 1 --pid $i --log-path file_c_pid$i.txt; done
    # for i in `seq 0 3`; do ./gotests/go_tester --netdata-path ../artifacts --content --iteration 1 --pid $i --log-path file_go_pid$i.txt; done
  4. Every test should ends with Success, unless you do not have a specific target (function) available.

Additional information

This PR was tested on:

Linux Distribution Environment Kernel Version Real Parent Parent All PIDs Without PIDs
LINUX DISTRIBUION Bare metal/VM uname -r

@thiagoftsm thiagoftsm marked this pull request as ready for review May 27, 2026 13:12
@thiagoftsm thiagoftsm requested a review from Ferroin as a code owner May 27, 2026 13:12
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.

1 participant