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

Invoke xdpbpfexport.exe from MSI install/uninstall #519

Merged
merged 9 commits into from
Sep 6, 2024

Conversation

mtfriesen
Copy link
Contributor

@mtfriesen mtfriesen commented May 29, 2024

XDP isn't registering its eBPF characteristics within the MSI installer, which causes certain eBPF features to break. Somewhat surprisingly, the libbpf APIs covered by the functional tests work fine.

Add an optional feature to our MSI installer and runtime installer script.

This was missed in #443. Also add an automated test to catch the scenario.

Resolves #567

@mtfriesen mtfriesen added bug Something isn't working ebpf labels May 29, 2024
@mtfriesen mtfriesen added this to the Release 1.1 milestone May 29, 2024
@mtfriesen mtfriesen requested a review from shankarseal May 29, 2024 21:02
@mtfriesen mtfriesen requested a review from a team as a code owner May 29, 2024 21:02
nibanks
nibanks previously approved these changes May 29, 2024
Copy link
Member

@nibanks nibanks left a comment

Choose a reason for hiding this comment

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

LGTM

@nibanks
Copy link
Member

nibanks commented Jul 18, 2024

Looks like this PR has conflicts too.

@mtfriesen
Copy link
Contributor Author

Looks like this PR has conflicts too.

yeah, this PR has been shelved for weeks. converting to draft.

@mtfriesen mtfriesen marked this pull request as draft July 18, 2024 13:19
@mtfriesen mtfriesen changed the title Invoke xdp_bpfexport.exe from MSI install/uninstall Invoke xdpbpfexport.exe from MSI install/uninstall Sep 5, 2024
@mtfriesen mtfriesen marked this pull request as ready for review September 5, 2024 21:31
@mtfriesen mtfriesen merged commit 6242219 into main Sep 6, 2024
31 checks passed
@mtfriesen mtfriesen deleted the mtfriesen/xdp_bpfexport_msi branch September 6, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ebpf
Projects
None yet
Development

Successfully merging this pull request may close these issues.

xdpbpfexport should be invoked from MSI
2 participants