Skip to content

Include BPF FS mount status in sysdump output#12191

Merged
joestringer merged 1 commit intocilium:masterfrom
soumynathan:bpf-fs-mount
Jul 9, 2020
Merged

Include BPF FS mount status in sysdump output#12191
joestringer merged 1 commit intocilium:masterfrom
soumynathan:bpf-fs-mount

Conversation

@soumynathan
Copy link
Copy Markdown

This PR adds in a new cilium bpf command to show the status of
the bpffs filesystem mount.

Fixes: #12057
Signed-off-by: Swaminathan Vasudevan svasudevan@suse.com

@soumynathan soumynathan requested a review from a team as a code owner June 18, 2020 22:31
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@coveralls
Copy link
Copy Markdown

coveralls commented Jun 18, 2020

Coverage Status

Coverage decreased (-0.03%) to 37.12% when pulling 268808f621b10c8076b396354643c131201bfe21 on soumynathan:bpf-fs-mount into 93d32dd on cilium:master.

@aanm aanm requested a review from joestringer June 19, 2020 08:22
@aanm aanm added the release-note/minor This PR changes functionality that users may find relevant to operating Cilium. label Jun 19, 2020
Comment thread cilium/cmd/bpf_mountfs_show.go Outdated
Copy link
Copy Markdown
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Looks good overall thanks, some minor suggestions below.

Comment thread cilium/cmd/bpf_mountfs_show.go Outdated
Comment thread cilium/cmd/bpf_mountfs_show.go Outdated
Comment thread cilium/cmd/bpf_mountfs_show.go Outdated
This PR adds in a new cilium bpf command to show the status of
the bpffs filesystem mount.

Fixes: cilium#12057
Signed-off-by: Swaminathan Vasudevan <svasudevan@suse.com>
@soumynathan
Copy link
Copy Markdown
Author

@joestringer Is there anything else that we need to do in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/minor This PR changes functionality that users may find relevant to operating Cilium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Include BPF FS mount status in sysdump output

7 participants