Skip to content
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.

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

tetragon: Force libbpf link creation #224

Merged
merged 1 commit into from
Jul 11, 2022
Merged

Conversation

olsajiri
Copy link
Contributor

Commit [1] broke libbpf-install target, which fails
if destination files exist:

$ make libbpf-install
mkdir -p ./lib
docker cp 9fb245cca6f49264e1ce716b5db5d6864e734e07b0d1558110880b37f5d0aa85:/go/src/github.com/covalentio/hubble-fgs/src/libbpf.so.0.2.0 ./lib
ln -s libbpf.so.0.2.0 ./lib/libbpf.so.0
ln: failed to create symbolic link './lib/libbpf.so.0': File exists
make: *** [Makefile:176: libbpf-install] Error 1

I need to fix this to get clean update eventhough I'm removing
the whole target in another libbpf remoal PR ;-)

[1] 7c22ff1 create symlinks for libbpf.so for podman

Signed-off-by: Jiri Olsa [email protected]

Commit [1] broke libbpf-install target, which fails
if destination files exist:

  $ make libbpf-install
  mkdir -p ./lib
  docker cp 9fb245cca6f49264e1ce716b5db5d6864e734e07b0d1558110880b37f5d0aa85:/go/src/github.com/covalentio/hubble-fgs/src/libbpf.so.0.2.0 ./lib
  ln -s libbpf.so.0.2.0 ./lib/libbpf.so.0
  ln: failed to create symbolic link './lib/libbpf.so.0': File exists
  make: *** [Makefile:176: libbpf-install] Error 1

I need to fix this to get clean update eventhough I'm removing
the whole target in another libbpf remoal PR ;-)

[1] 7c22ff1 create symlinks for libbpf.so for podman

Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri self-assigned this Jul 11, 2022
@olsajiri olsajiri marked this pull request as ready for review July 11, 2022 12:04
@olsajiri olsajiri requested a review from a team as a code owner July 11, 2022 12:04
@olsajiri olsajiri requested review from sharlns and kkourt July 11, 2022 12:04
@kkourt kkourt merged commit 9b20800 into cilium:main Jul 11, 2022
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.

3 participants