Skip to content
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 support for executing agent-local workloads #305

Closed
wants to merge 5 commits into from

Conversation

kthomas
Copy link
Contributor

@kthomas kthomas commented Jul 1, 2024

This PR allows the workload deploy location to be specified using a file:// or nats:// URL scheme.

File-based deploy locations are treated as "agent-local" workload deploy requests, meaning the agent will attempt to resolve the named file as an executable installed on its rootfs. File-based workload deploy requests can only be executed in sandbox mode.

@kthomas kthomas requested a review from a team as a code owner July 1, 2024 20:30
@autodidaddict
Copy link
Contributor

File-based workload deploy requests can only be executed in sandbox mode.
So there's no way for someone to specify a path to a file that didn't ship with the rootfs or when agents are running without a rootfs?

I can't find the error message in the PR that rejects this.. maybe git's hiding the diff.

@autodidaddict
Copy link
Contributor

As discussed I think if we put this behind a feature flag like agent_local_workloads and default that to false and it can only be enabled via node configuration, then this should be good.

Short version is nodes should default to not allowing this behavior and it should be explicitly opt-in to enable

Workload deploy location can be specified with file:// or nats://
schemes. File-based deploy locations are treated as "agent-local"
workload deploy requests, meaning the agent will attempt to resolve the
file as an executable installed on its rootfs. File-based workload
deploy requests can only be executed in sandbox mode.
@kthomas kthomas force-pushed the feat/agent-local-workloads branch from 665c64e to 2d4014a Compare July 12, 2024 19:41
@kthomas
Copy link
Contributor Author

kthomas commented Jul 12, 2024

Closing in favor of #325

@kthomas kthomas closed this Jul 12, 2024
@kthomas kthomas deleted the feat/agent-local-workloads branch August 5, 2024 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants