Skip to content

Conversation

@dangome3
Copy link
Contributor

@dangome3 dangome3 commented Nov 28, 2025

Updates configuration steps with minikube-install-hook.sh script .

Description

Updates the Setup Cluster section in the RunTime Hooks page as the tetragon-oci-hook-setup script does not longer exist. it was replaced by minikube-install-hook. See #2995

See page updated in https://deploy-preview-4386--tetragon.netlify.app/docs/installation/runtime-hooks/

The script has been tested so it is safe to update the documentation.

Simplify the runtime-hooks installation documentation by removing redundant
Helm setup section and replacing manual containerd configuration steps with
the minikube-install-hook.sh script. Also fix shell code block typo.

Signed-off-by: Daniel Gomez <[email protected]>
@dangome3 dangome3 requested review from a team and mtardy as code owners November 28, 2025 00:09
@netlify
Copy link

netlify bot commented Nov 28, 2025

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 5784c8a
🔍 Latest deploy log https://app.netlify.com/projects/tetragon/deploys/6928e8508aa1e3000862b5ee
😎 Deploy Preview https://deploy-preview-4386--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

We use `minikube` as the example platform because it supports both `cri-o` and `containerd`, but the
same steps can be applied to other platforms.

### Setup Helm
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed this section as it is repeated in ### Install Tetragon:

Screenshot 2025-11-27 at 7 11 14 PM

+ plugin_request_timeout = "2s"
+ socket_path = "/var/run/nri/nri.sock"
```
This script updates the configuration and restarts containerd.
Copy link
Contributor Author

@dangome3 dangome3 Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirmed that the script handles the configuration update and restarts containerd by itself. No more steps are needed to follow next steps in the guide.

{"time":"2025-11-27T23:14:09.292112562Z","level":"INFO","msg":"hook request to agent succeeded","hook":"createRuntime","start-time":"2025-11-27T23:14:09.278533805Z","req-cgroups":"/kubepods.slice/kubepods-besteffort.slice/kubepods-besteffort-podab5f6a59_2fc0_44a3_8650_7d8c70149c40.slice/cri-containerd-fbfc2551cde334ed1a101b47073be93aba5db7bf08aee4f1ca83cc6b4c4da141.scope","req-rootdir":"/run/containerd/io.containerd.runtime.v2.task/k8s.io/fbfc2551cde334ed1a101b47073be93aba5db7bf08aee4f1ca83cc6b4c4da141/rootfs","req-containerName":"test-ubuntu","req-containerID":"","req-podName":"test-ubuntu","req-podUID":"ab5f6a59-2fc0-44a3-8650-7d8c70149c40","req-podNamespace":"default"}

JIC in my local setup I didn't have NRI enabled by default

@FedeDP
Copy link
Contributor

FedeDP commented Nov 28, 2025

Should we add Fixes https://github.com/cilium/tetragon/issues/2995 to the PR description?

@FedeDP FedeDP added area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation. and removed area/documentation Improvements or additions to documentation labels Nov 28, 2025
@mtardy mtardy linked an issue Dec 1, 2025 that may be closed by this pull request
Copy link
Member

@mtardy mtardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!!

@mtardy mtardy merged commit 93c89d5 into cilium:main Dec 1, 2025
18 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/docs This PR updates the documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs(tetragon-rthooks): outdated command for containerd setup with NRI

4 participants