Skip to content

Implement agent injection routine. #621

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.

Sign up for GitHub

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

Jump to bottom
Open
DanielleMaywood opened this issue May 7, 2025 · 0 comments
Open

Implement agent injection routine. #621

DanielleMaywood opened this issue May 7, 2025 · 0 comments
Assignees
@mafredri
Labels
m estimated at less than 1 week of work

Comments

@DanielleMaywood
Copy link

The agent should have a routine listening for newly created dev containers.

Once a new dev container is detected, it should:

  1. Communicate with coderd to create a workspace_agents record and receive an auth_token.
  2. Detect the architecture of the dev container
  3. Check if a copy of an agent with that architecture is stored, otherwise download 4 new one.
  4. Copy that binary onto the agent.
  5. Attempt to add CAP_NET_ADMIN to the injected dev-container binary.
  6. Run the coder agent command in the dev container using the auth_token obtained earlier.
@DanielleMaywood DanielleMaywood added the m estimated at less than 1 week of work label May 7, 2025
mafredri added a commit to coder/coder that referenced this issue May 21, 2025
@mafredri
feat(agent/agentcontainers): update containers periodically
5ecf9b1 
This change introduces a significant refactor to the agentcontainers API
and enabled periodic updates of Docker containers rather than on-demand.
Consequently this change also allows us to move away from using a
locking channel and replace it with a mutex, which simplifies usage.

Additionally a previous oversight was fixed, and testing added, to clear
devcontainer running/dirty status when the container has been removed.

Updates #16424
Updates coder/internal#621
@mafredri mafredri mentioned this issue May 21, 2025
Merged
mafredri added a commit to coder/coder that referenced this issue May 21, 2025
@mafredri
feat(agent/agentcontainers): update containers periodically
7fc062b 
This change introduces a significant refactor to the agentcontainers API
and enabled periodic updates of Docker containers rather than on-demand.
Consequently this change also allows us to move away from using a
locking channel and replace it with a mutex, which simplifies usage.

Additionally a previous oversight was fixed, and testing added, to clear
devcontainer running/dirty status when the container has been removed.

Updates #16424
Updates coder/internal#621
mafredri added a commit to coder/coder that referenced this issue May 21, 2025
@mafredri
feat(agent/agentcontainers): update containers periodically
c381c45 
This change introduces a significant refactor to the agentcontainers API
and enabled periodic updates of Docker containers rather than on-demand.
Consequently this change also allows us to move away from using a
locking channel and replace it with a mutex, which simplifies usage.

Additionally a previous oversight was fixed, and testing added, to clear
devcontainer running/dirty status when the container has been removed.

Updates #16424
Updates coder/internal#621
mafredri added a commit to coder/coder that referenced this issue May 21, 2025
@mafredri
feat(agent/agentcontainers): update containers periodically
84e7fb1 
This change introduces a significant refactor to the agentcontainers API
and enabled periodic updates of Docker containers rather than on-demand.
Consequently this change also allows us to move away from using a
locking channel and replace it with a mutex, which simplifies usage.

Additionally a previous oversight was fixed, and testing added, to clear
devcontainer running/dirty status when the container has been removed.

Updates #16424
Updates coder/internal#621
mafredri added a commit to coder/coder that referenced this issue May 21, 2025
@mafredri
feat(agent/agentcontainers): update containers periodically
25ead7b 
This change introduces a significant refactor to the agentcontainers API
and enabled periodic updates of Docker containers rather than on-demand.
Consequently this change also allows us to move away from using a
locking channel and replace it with a mutex, which simplifies usage.

Additionally a previous oversight was fixed, and testing added, to clear
devcontainer running/dirty status when the container has been removed.

Updates #16424
Updates coder/internal#621
mafredri added a commit to coder/coder that referenced this issue May 21, 2025
@mafredri
feat(agent/agentcontainers): update containers periodically
0b002ec 
This change introduces a significant refactor to the agentcontainers API
and enabled periodic updates of Docker containers rather than on-demand.
Consequently this change also allows us to move away from using a
locking channel and replace it with a mutex, which simplifies usage.

Additionally a previous oversight was fixed, and testing added, to clear
devcontainer running/dirty status when the container has been removed.

Updates #16424
Updates coder/internal#621
mafredri added a commit to coder/coder that referenced this issue May 22, 2025
@mafredri
feat(agent/agentcontainers): update containers periodically (#17972)
d6c14f3 
This change introduces a significant refactor to the agentcontainers API
and enables periodic updates of Docker containers rather than on-demand.
Consequently this change also allows us to move away from using a
locking channel and replace it with a mutex, which simplifies usage.

Additionally a previous oversight was fixed, and testing added, to clear
devcontainer running/dirty status when the container has been removed.

Updates #16424
Updates coder/internal#621
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mafredri mafredri

Labels
m estimated at less than 1 week of work
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mafredri @DanielleMaywood