Skip to content

cli/sysdump: capture more operator and clustermesh-apiserver troubleshooting information#46503

Merged
giorio94 merged 3 commits into
cilium:mainfrom
giorio94:mio/sysdump-operator-dbg
Jun 22, 2026
Merged

cli/sysdump: capture more operator and clustermesh-apiserver troubleshooting information#46503
giorio94 merged 3 commits into
cilium:mainfrom
giorio94:mio/sysdump-operator-dbg

Conversation

@giorio94

@giorio94 giorio94 commented Jun 11, 2026

Copy link
Copy Markdown
Member

Please review commit by commit, and refer to the individual commit messages for additional details.

The db/dump command is introduced in cilium/statedb#164.

giorio94 added 3 commits June 11, 2026 15:17
Extend the clustermesh-related sysdump tasks to additionally capture
the outputs of the `db/dump` and `kvstore/list` statedb commands on
the clustermesh-apiserver replicas (both apiserver and kvstoremesh
containers), to aid during troubleshooting. The `kvstore/list` output
is expected to match in both cases, but it is retrieved twice anyways
for resiliency in case only one of the two containers is enabled in
the target environment.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
Extend the sysdump collection logic to additionally capture Cilium
operator debug information, including the dump of the statedb tables
and kvstore/clustermesh related information. The collection logic
mimics the one already used for the clustermesh-apiserver replicas.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
[1] moved the `gops` binary from `/bin/gops` to `/usr/bin/gops`. The
commit message claims that the change is fully backwards compatible
since `/bin` symlinks to `/usr/bin`. However, that does not apply in
case of the Cilium operator, as based on a scratch image that contains
no such symlink. In turn, this broke the collection of gops data for
the Cilium operator in sysdumps, as it still invokes the binary at
the old location. Let's get this fixed by trying both paths, and
picking the first command which succeeds.

[1]: a927457 ("images: Only build `gops` for the relevant platform")

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 added release-note/misc This PR makes changes that have no direct user impact. cilium-cli This PR contains changes related with cilium-cli labels Jun 11, 2026
@github-actions github-actions Bot added the cilium-cli-exclusive This PR only impacts cilium-cli binary label Jun 11, 2026
@giorio94 giorio94 marked this pull request as ready for review June 11, 2026 13:56
@giorio94 giorio94 requested a review from a team as a code owner June 11, 2026 13:56
@giorio94 giorio94 requested a review from asauber June 11, 2026 13:56
@giorio94

Copy link
Copy Markdown
Member Author

/test

@giorio94 giorio94 enabled auto-merge June 12, 2026 06:52
@giorio94 giorio94 added this pull request to the merge queue Jun 22, 2026
@maintainer-s-little-helper maintainer-s-little-helper Bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 22, 2026
Merged via the queue into cilium:main with commit 166fd9b Jun 22, 2026
83 of 86 checks passed
@giorio94 giorio94 deleted the mio/sysdump-operator-dbg branch June 22, 2026 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cilium-cli This PR contains changes related with cilium-cli cilium-cli-exclusive This PR only impacts cilium-cli binary ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants