-
Notifications
You must be signed in to change notification settings - Fork 20
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
Build for profile release-with-debug #113
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #113 +/- ##
==========================================
- Coverage 59.59% 59.59% -0.01%
==========================================
Files 33 33
Lines 16068 16069 +1
==========================================
Hits 9576 9576
- Misses 6492 6493 +1 ☔ View full report in Codecov by Sentry. |
Fix clippy regression
Like the idea. I noticed there are some .devcontainer configs not previously there. If they are not part/required for this feature, maybe this is something for a separate PR? |
Pls also check pre-commit line break issues https://github.com/eclipse-kuksa/kuksa-databroker/blob/main/CONTRIBUTING.md#pre-commit-set-up |
A build workflow for the
release-with-debug
profile, which creates a databroker binary with debug symbols enabled.This is useful for the integration into an eBPF-based performance test environments.
As separate testers would need to rebuild databroker with that profile, which requires them to set up their own databroker development environment, we've decided for convenience to release debug versions of databroker for exactly that use case.
The additional
debugsymbols.txt
file in the archive contains the entry point for eBPF probes to instrument the function within databroker where the VSS Signals are being updated by providers: