-
Notifications
You must be signed in to change notification settings - Fork 1.4k
POC for Agent Lite: make Python and integrations optional #43629
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
base: main
Are you sure you want to change the base?
Conversation
| # Used for memory profiling with the `status py` agent subcommand | ||
| dependency 'pympler' | ||
| # For lite flavor, pympler goes with Python runtime artifacts | ||
| unless ENV['AGENT_FLAVOR'] == 'lite' |
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.
nit: I'd add an accessor for this (see ostools.rb / project_helpers.rb)
|
What would be the approach for the final artifact? Would this be a separated deb/rpm packages, and some additional layers for the containers? |
We'd want to package them for the Agent Installer. I'm investigating how to do that. |
So the lite agent would only be available through the installer and the deb/rpm & usual packages would be the full agent? In any case, for the installer you'll want to package as OCI |
The lite agent itself should be available as a deb pkg! the python stuff as OCI then |
Static quality checks✅ Please find below the results from static quality gates Successful checksInfo
|
What does this PR do?
Here I made a quick and dirty poc for a new flavor of the agent. Calling it “lite” for now, but not married to the name.
The point is that we split the base agent into 2 parts that can be assembled later:
I'm aware that this will become obsolete with the bzl migration, I'm just trying out the concept.
Motivation
Want to see how easy it is to split the Python and integrations artifacts.
I'd also like to get some concrete feedback from the build folks.