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
netdata: 1.18.1 -> 1.19.0 #75173
netdata: 1.18.1 -> 1.19.0 #75173
Conversation
@GrahamcOfBorg test netdata |
@GrahamcOfBorg test netdata |
@GrahamcOfBorg build netdata |
Builds fine. Needs a few folders created for the default log,cache,lib to work. Even then there is an access permission problem when accessing the API/Web port. [NixOS] |
@tomberek, if build package with the $ out / var folder, the service does not work:
|
562c5b1
to
980b53f
Compare
@GrahamcOfBorg test netdata |
Commit
|
cc @Mic92 |
Thanks! |
@@ -25,6 +25,7 @@ import ./make-test-python.nix ({ pkgs, ...} : { | |||
|
|||
# check if the netdata main page loads. | |||
netdata.succeed("curl --fail http://localhost:19999/") | |||
netdata.succeed("sleep 4") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was this needed? Should this not be covered by wait_until_succeeds
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
without sleep 4
- error:
netdata: waiting for success: curl -s http://localhost:19999/api/v1/data\?chart=users.preads | jq -e '[.data[range(10)][.labels | indices("root")[0]]] | add | . > 0'
netdata # parse error: Invalid numeric literal at line 1, column 6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The netdata process cannot collect statistics users.preads
Motivation for this change
Update netdata to version 1.19.0.
Changelog - https://docs.netdata.cloud/changelog/#v1190-2019-11-27
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @