Add blob store support#3606
Conversation
Greptile SummaryThis PR introduces blob/object store support for the console, wiring up S3 and Azure Blob Storage via a Waffle integration to store agent run session recordings, screen recordings, patch files, and workbench memory tarballs.
Confidence Score: 3/5The file-type validation bug in Two defects affect the main use-cases: the
|
| Filename | Overview |
|---|---|
| lib/console/uploads.ex | New Waffle upload definition; validate/1 only allows image extensions, blocking all session, screen-recording, and patch uploads |
| rel/runtime.exs | Adds S3 and Azure waffle config blocks; CONSOLE_S3_ENDPOINT is never consumed here despite being wired in Helm secrets |
| mix.exs | Adds waffle, waffle_ecto, and ex_aws_s3 deps; waffle points to a personal GitHub fork rather than the official package |
| lib/console/schema/agent_run_upload.ex | New schema for agent run uploads with session, screen_recording, and patch Waffle attachment fields; looks correct |
| lib/console/deployments/agents.ex | Adds agent_run_uploads/3 with object-store guard; refactors repeated auth logic into validate_run/3 helper |
| priv/repo/migrations/20260523191654_add_s3_upload_support.exs | Creates agent_run_uploads table and adds memory column to workbenches; migration looks correct |
| charts/console/templates/secrets.yaml | Adds object store env vars to Helm secrets; CONSOLE_S3_ENDPOINT is present but unused in runtime config |
| lib/console/graphql/deployments/agent.ex | Adds create_agent_run_upload mutation with ClusterAuthenticated middleware and upload_url resolver helper |
Reviews (1): Last reviewed commit: "Add blob store support" | Re-trigger Greptile
460f677 to
bfed81d
Compare
|
plural deploy this |
bfed81d to
d7dc381
Compare
|
plural deploy this |
3 similar comments
|
plural deploy this |
|
plural deploy this |
|
plural deploy this |
0b4f303 to
8ebc981
Compare
|
plural deploy this |
0fcf15f to
7a56ac2
Compare
|
plural deploy this |
b210f04 to
40ad430
Compare
|
plural deploy this |
79a54b0 to
1cce296
Compare
|
plural deploy this |
11c936e to
dbe2f9a
Compare
838f30d to
000c220
Compare
|
plural deploy this |
|
plural deploy this |
|
plural deploy this |
|
plural deploy this |
1 similar comment
|
plural deploy this |
f70ab8f to
189fe28
Compare
|
plural deploy this |
6eba169 to
103ef8d
Compare
|
plural deploy this |
1 similar comment
|
plural deploy this |
b1ef6bd to
d770e47
Compare
d770e47 to
db7d43d
Compare
|
plural deploy this |
There's a few usecases for this:
Also implemented:
Test Plan
Test environment: https://console.your-env.onplural.sh/
Checklist
Plural Flow: console