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
osquery: 3.2.8 -> 3.2.9 #43690
osquery: 3.2.8 -> 3.2.9 #43690
Conversation
Latest bugfix release with the following notable changes: * Memory leak resolve for dispatcher (osquery/osquery@06d4865) * Fix include path on status.h (osquery/osquery@5bd4984) Additionally the patch had to be rebased onto the 3.2.9 branch as it added XCode support including some CLang flags (namely `-fno-limit-debug-info`) which are unsupported on GCC. (see osquery/osquery@bccc28d)
as mentioned in IRC, upstream has added XCode support, but I don't have the hardware to test it which is why I left |
@GrahamcOfBorg eval |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: osquery Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: osquery Partial log (click to expand)
|
Evaluation seems to fail due to a lack of memory:
@grahamc is it sufficientto retrigger ofborg or are there any further things tobe done? |
Give it a try, the failing error message indicates the problem was with master not this branch, so trying again can't hurt in case it has been fixed there. However specific to today: there was a bit of an issue with too little memory allocated for the evaluator, which should be fixed now. |
thanks for the fast reply! @GrahamcOfBorg eval |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: osquery Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: osquery Partial log (click to expand)
|
@GrahamcOfBorg test osquery |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.osquery Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: tests.osquery Partial log (click to expand)
|
I'm slightly surprised about the broken test as |
Nevermind, it's just a build timeout. Good enough if the test works on your machine |
Test works fine on my machine too. |
Motivation for this change
Latest bugfix release with the following notable changes:
Memory leak resolve for dispatcher
(osquery/osquery@06d4865)
Fix include path on status.h
(osquery/osquery@5bd4984)
Additionally the patch had to be rebased onto the 3.2.9 branch as it
added XCode support including some CLang flags (namely
-fno-limit-debug-info
)which are unsupported on GCC.
(see osquery/osquery@bccc28d)
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)