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.

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

provide auto-delete artifact directory upon completion #739

Closed
cidrblock opened this issue Jun 24, 2021 · 2 comments · Fixed by #1111
Closed

provide auto-delete artifact directory upon completion #739

cidrblock opened this issue Jun 24, 2021 · 2 comments · Fixed by #1111
Assignees
@Akasurde
Labels
enhancement New feature or request

Comments

@cidrblock
Copy link
Contributor

rather than every consumer of runner having to implement the logic, consider an api parameter that instructs runner to delete it's artifacts when no longer needed by runner
@cidrblock cidrblock mentioned this issue Jun 24, 2021
Open
@cidrblock cidrblock added the enhancement New feature or request label Jun 25, 2021
@matburt
Copy link
Member

matburt commented Jun 25, 2021

This somewhat exists already: https://github.com/ansible/ansible-runner/blob/devel/ansible_runner/interface.py#L166

Akasurde added a commit to Akasurde/ansible-runner that referenced this issue Aug 9, 2022
@Akasurde
Update for artifacts_handler
6d34c50 
* Added a new example for artifacts_handler

Fixes: ansible#739

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@Akasurde Akasurde mentioned this issue Aug 9, 2022
Merged
@Akasurde
Copy link
Member

Akasurde commented Aug 9, 2022

resolved_by_pr #1111

@Akasurde Akasurde self-assigned this Aug 9, 2022
Akasurde added a commit to Akasurde/ansible-runner that referenced this issue Aug 9, 2022
@Akasurde
Update for artifacts_handler
bdd9078 
* Added a new example for artifacts_handler

Fixes: ansible#739

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
Akasurde added a commit to Akasurde/ansible-runner that referenced this issue Aug 22, 2022
@Akasurde
Update for artifacts_handler
863fde6 
* Added a new example for artifacts_handler

Fixes: ansible#739

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@Shrews Shrews closed this as completed in db0f66e Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Akasurde Akasurde

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update for artifacts_handler Akasurde/ansible-runner
3 participants
@matburt @Akasurde @cidrblock