Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add ability to register ADP as a remote agent to the Datadog Agent #377
Add ability to register ADP as a remote agent to the Datadog Agent #377
Changes from 13 commits
4afb63b
28ae9b4
400cdc5
f10568a
66029c8
437deec
3b605cc
60d3dad
d21a5fd
8d85d46
dd3e41c
a652ec1
6729691
69673c2
2ca83f8
9816494
6c6a8b1
ecbe7d2
e41f955
758a2bd
63c9300
46b8ff4
b00e0e5
ae38c04
efd436b
ac5f5c6
24cbdab
59a124e
3f21ab4
a25cb80
84a6058
fe6aa0d
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would say this isn't missing anymore. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This component is only built if
adp_standalone_mode == false
, but this call will keep failing ifis not set on the
datadog-agent
side.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤷🏻
I think that's just something we'll have to deal with until we make it the default to enable the remote agent registry. We don't get any visible errors, right, since it's just debug logging if the call fails? Or does it have any other user-visible impact in ADP when the remote agent registry is disabled?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct, it will just spam the debug logs if it fails.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Purely as a sidenote that we should add a TODO for, but isn't blocking for this PR: this call might take a while if the Agent is temporarily down or slow to respond... which would in turn slow down the accepting of events/responding to health check requests.
I don't know the exact code I would want to use to do it, but we would likely want to consider figuring out how we could spawn this call as a background task so that the component can keep polling, but limit ourselves to one in-flight request at a time.
Typing this all up also reminds me that we don't have any request timeout configuration in
RemoteAgentClient
. 🤔