-
Notifications
You must be signed in to change notification settings - Fork 76
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
feat(agent): agent development #3081
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* feat: add structure for agent * add short and long description for agent command * fix build * add go.work
…2970) * wip: connect to grpc server * feat: add connection workflow * add protobuf for trigger request * add flow to receive trigger requests * add comments and rename files * cleanup makefile * configure CI to run agent tests * fix makefile * add flow for sending trigger result back to the server
* add flow for listening for polling requests * feat: send spans to server
* feat: add empty trigger worker * move executor/trigger package to agent and adapt it * adapt test * implement trigger * test trigger flow * send trigger response to server * add TODO
* feat: implement trace polling on agent * initialize polling worker on agent start * pass worker functions directly to client hooks
* feat: add otlp and http servers with no operation yet * feat: implement a very basic collector in the agent * remove tracer from otlp_server structure * Update agent/collector/http_server.go Co-authored-by: Daniel Baptista Dias <[email protected]> * Update agent/collector/http_server.go Co-authored-by: Daniel Baptista Dias <[email protected]> * reuse otlp servers * apply daniel's comments to the otlp server code * remove dependency on ioutil from http otlp server * delete http server from agent collector * make collector ingester use otlp ingester interface --------- Co-authored-by: Daniel Baptista Dias <[email protected]>
* shutdown agent when receive shutdown request * remove unused function * fix brittle test * centrilize stop function
feat: add agent identification to agent client proto
fix(agent): rename traceid response property to traceid
add kafka response
* feat(agent): inject env variables in agent configuration * add tests to cover all config values are configurable via env vars * add log at startup and set hostname as default agent name * remove dev mode config * fix config defaults test
feat(agent): make agent wait until connection end This prevents the agent from exiting right after connecting to the server
mathnogueira
changed the title
feat(agent): add initial agent development
feat(agent): agent development
Aug 30, 2023
xoscar
approved these changes
Aug 31, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds the initial agent structure with most of the flows implemented. Still some things to be done after we have a working control plane
Checklist