-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Remote debug heartbeat #34224
Remote debug heartbeat #34224
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
/test |
The failure in CI is:
Never seen this, but Stackoverflow suggests the ownership of the new directory might be wrong: https://stackoverflow.com/questions/72978485/git-submodule-update-failed-with-fatal-detected-dubious-ownership-in-repositor |
@cmacknz I am not sure how to fix this though. I have checked the ownership of the new folder on my machine but I can't see anything wrong with it. Any other suggestions? |
@v1v it looks like elastic/golang-crossbuild#232 is hitting us now |
3b683de
to
ccb0ccd
Compare
The build error should be fixed now if you merge/rebase with main. |
@cmacknz Thanks, I saw the commit to main by chance and this PR seems to be building correctly now. Thanks |
What does this PR do?
Changes to Tiltfile + dockerfile + manifests to remote debug Heartbeat similarly to how it works already for Metricbeat and Filebeat.
Why is it important?
I found absolutely necessity to be able to remote debug heartbeat and similar beats to investigate bugs
Checklist
None of the below applies since it is a dev tool
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.How to test this PR locally
Instructions at #32326
Related issues