Add Ubuntu kernel for aws instances. #1552
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Michael Ducy <[email protected]>
leodido
approved these changes
Dec 4, 2019
fntlnz
approved these changes
Dec 5, 2019
…ewly built probe (#1558) * install script now loads latest built probe at tend of script * fixed modprobe command to remove -r flag * reverting -r flag in modprobe
…PF probe. (#1556) sysdig-CLA-1.0-signed-off-by: James Alseth <[email protected]>
* SMAGENT-2165: Disable pre-built probes for 5.4.0 The probe builder needs some work to build modules for Linux 5.4.0. This change temporarily disables builds for that version.
After 7209e28, we end up with a repo in `build/googletest-src` that `git clean -d -f -x` does not remove automatically, causing errors in `mkdir build`. Since we don't use the test framework to build the driver, disable it.
Create an interface in the container manager to re-request docker container info that includes the size of the container layer. This is relevant when running overlay2. The size request isn't done during the initial request because determining the size can easily take seconds.
* cmake: Look for gtest headers in 'gtest' path suffix The gtest headers are installed into /usr/include/gtest on Gentoo. Account for that in the default search path. sysdig-CLA-1.0-signed-off-by: Michał Górny <[email protected]> * cmake: Fail if system gtest is not found sysdig-CLA-1.0-signed-off-by: Michał Górny <[email protected]> * cmake: Link -lgtest for tests as well Find and link -lgtest when using system gtest. It is insufficient to link -lgtest_main when the tests are directly referencing symbols from -lgtest as well. sysdig-CLA-1.0-signed-off-by: Michał Górny <[email protected]>
* add the container cache as a member in container_engine_base * CRIO support for rw.bytes.used
|
Hi Folks, any ETA on getting this PR merged? |
Signed-off-by: Lorenzo Fontana <[email protected]>
The old probe builder used whatever gcc happened to be installed on the build host, which led to incompatibilities with very old or very new kernels. The new builder moves the actual build process to a Docker container that is chosen based on the actual kernel to build. It also features a more flexible command line interface (run it without parameters to see a help message). A compatibility wrapper for tools using the old script is provided.
Otherwise when building everything, all the kernels get built with the first builder distro that runs (CentOS if Artifactory creds for RHEL are present, Ubuntu otherwise)
|
Bump - any update here? |
Signed-off-by: Michael Ducy <[email protected]>
…ios/sysdig into mfdii/add-ubuntu-aws-kernels
|
Any updates on this fix? |
|
Any updates? |
|
Hi all, sorry for the long delay. We've recently checked in a series of improvements to our probe build process. A question to everyone asking for updates: has your particular problem gotten any better? |
|
We now build Ubuntu AWS probes (since #1576 got merged, with changes greatly inspired by this PR, i.e. taken wholesale), closing this one. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change is needed to compile kernel modules for the AWS specific kernel Ubuntu provides. Tested the script and URLs return as expected. Below is just a snippet.