Skip to content

Address incosinstency#423

Merged
thiagoftsm merged 1 commit intonetdata:masterfrom
thiagoftsm:pidgid
Mar 20, 2026
Merged

Address incosinstency#423
thiagoftsm merged 1 commit intonetdata:masterfrom
thiagoftsm:pidgid

Conversation

@thiagoftsm
Copy link
Contributor

Summary

This PR fixes inconsistency between filesystem variable value and name.

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 *.xz`; 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 ./kernel/legacy_test --netdata-path ../artifacts --content --iteration 1 --pid $i --log-path file_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 requested a review from Ferroin as a code owner March 20, 2026 01:18
@thiagoftsm thiagoftsm merged commit e226ed0 into netdata:master Mar 20, 2026
40 checks passed
@thiagoftsm thiagoftsm deleted the pidgid branch March 20, 2026 01:25
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