Skip to content

fix: Force remove tetragon-clang container for the tetragon-bpf target in case it's still running. #3935

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 28, 2025
Merged

fix: Force remove tetragon-clang container for the tetragon-bpf target in case it's still running. #3935

merged 1 commit into from
Jul 28, 2025

Conversation

acamatcisco
Copy link
Contributor

Description

After make clean, I got an error when trying to rebuild the tetragon-bpf target:

acam@lima-default:~/src/hubble-fgs/modules/tetragon-oss$ make -j3 tetragon-bpf
docker rm tetragon-clang || true
Error response from daemon: cannot remove container "tetragon-clang": container is running: stop the container before removing or force remove

This PR contains a simple change to force-remove the tetragon-clang container if it's already running.

@acamatcisco acamatcisco requested a review from a team as a code owner July 24, 2025 15:21
@acamatcisco acamatcisco requested a review from mtardy July 24, 2025 15:21
will-isovalent
will-isovalent approved these changes Jul 24, 2025
View reviewed changes
Copy link
Contributor

@will-isovalent will-isovalent left a comment

Choose a reason for hiding this comment

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

Let's do it

mtardy
mtardy approved these changes Jul 24, 2025
View reviewed changes
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.

sure

@mtardy mtardy added the release-note/misc This PR makes changes that have no direct user impact. label Jul 24, 2025
@acamatcisco acamatcisco force-pushed the pr/ac/force-remove-tetragon-clang-container branch from 26da95e to 0f69a17 Compare July 24, 2025 15:50
@acamatcisco acamatcisco force-pushed the pr/ac/force-remove-tetragon-clang-container branch from 0f69a17 to 8a4384c Compare July 24, 2025 17:05
@mtardy mtardy merged commit 067a0e8 into cilium:main Jul 28, 2025
60 of 69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mtardy mtardy mtardy approved these changes

@will-isovalent will-isovalent will-isovalent approved these changes

Assignees
No one assigned
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@acamatcisco @mtardy @will-isovalent