Skip to content

Document all CLI options#606

Merged
giuseppe merged 1 commit intocontainers:mainfrom
jnovy:missing-docs
Sep 24, 2025
Merged

Document all CLI options#606
giuseppe merged 1 commit intocontainers:mainfrom
jnovy:missing-docs

Conversation

@jnovy
Copy link
Collaborator

@jnovy jnovy commented Sep 24, 2025

Some of CLI options were missing from conmon.8.md. This commit adds all available options there.

@jnovy jnovy requested a review from giuseppe September 24, 2025 11:29
docs/conmon.8.md Outdated
Comment on lines +139 to +144
**--seccomp-notify-plugins**
Plugins to use for managing the seccomp notifications.

**--seccomp-notify-socket**
Path to the socket where the seccomp notification fd is received.

Copy link
Member

Choose a reason for hiding this comment

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

do we want to document these or just drop the functionality?

@jnovy
Copy link
Collaborator Author

jnovy commented Sep 24, 2025

Seems the removal is the best if it never worked.

Some of CLI options were missing from conmon.8.md. This commit
adds all available options there.

Signed-off-by: Jindrich Novy <jnovy@redhat.com>
@jnovy
Copy link
Collaborator Author

jnovy commented Sep 24, 2025

I just updated the man page not mentioning these options @giuseppe

Copy link
Member

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

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

LGTM

@giuseppe giuseppe merged commit 4d68d7a into containers:main Sep 24, 2025
31 of 34 checks passed
@jnovy jnovy mentioned this pull request Sep 24, 2025
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.

2 participants