Skip to content

feat(provider): create sandbox-scoped providers and delete them after - #20

Closed
guyoron1 wants to merge 2 commits into
mainfrom
agent/2294-sandbox-scoped-providers
Closed

feat(provider): create sandbox-scoped providers and delete them after#20
guyoron1 wants to merge 2 commits into
mainfrom
agent/2294-sandbox-scoped-providers

Conversation

@guyoron1

@guyoron1 guyoron1 commented Jun 16, 2026

Copy link
Copy Markdown
Owner

This PR changes providers to be sandbox scoped and to get deleted after the sandbox is deleted. The providers are kept around with --keep-sandbox. This covers fullsend-ai#2294 in that subsequent runs do not try to create providers with the same name, so it does not conflict.

Closes fullsend-ai#2294 .

Supersedes fullsend-ai#2296

Signed-off-by: Hector Martinez <hemartin@redhat.com>
@guyoron1

Copy link
Copy Markdown
Owner Author

/fs-qf

Go tier 1: 0 files → qf-tests/FS-2294/go/
Python tier 2: 0 files → qf-tests/FS-2294/python/
[skip ci]
@fullsend-ai-coder

fullsend-ai-coder Bot commented Jun 16, 2026

Copy link
Copy Markdown

QualityFlow Pipeline Summary

Stage Agent Status
1 STP Builder
2 STP Reviewer ⏭️
3 STP Refiner ⏭️
4 STD Builder ⏭️
5 STD Reviewer ⏭️
6 STD Refiner ⏭️
7 Test Generator ⏭️

Test Output

Type Count Location
Go/Ginkgo tier 1 0 files qf-tests/FS-20/go/
Python/pytest tier 2 0 files qf-tests/FS-20/python/

Ticket: FS-20


Generated by QualityFlow

@guyoron1

Copy link
Copy Markdown
Owner Author

/fs-qf

5 similar comments
@guyoron1

Copy link
Copy Markdown
Owner Author

/fs-qf

@guyoron1

Copy link
Copy Markdown
Owner Author

/fs-qf

@guyoron1

Copy link
Copy Markdown
Owner Author

/fs-qf

@guyoron1

Copy link
Copy Markdown
Owner Author

/fs-qf

@guyoron1

Copy link
Copy Markdown
Owner Author

/fs-qf

@guyoron1

Copy link
Copy Markdown
Owner Author

Closing — QualityFlow test artifacts were inadvertently pushed to this branch.

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.

EnsureProvider should be idempotent — fails with AlreadyExists on repeated runs

2 participants