Skip to content

[Cherry-pick][3.4ea1] fix: replace broken S3 cleanup with LlamaStack files API#1223

Merged
jgarciao merged 1 commit into3.4ea1from
CherryPick-3.4ea1-c3f4
Mar 16, 2026
Merged

[Cherry-pick][3.4ea1] fix: replace broken S3 cleanup with LlamaStack files API#1223
jgarciao merged 1 commit into3.4ea1from
CherryPick-3.4ea1-c3f4

Conversation

@rhods-ci-bot
Copy link
Copy Markdown
Contributor

Cherry-picked #1210 into 3.4ea1. Requested by: jgarciao

The S3 file cleanup was failing because boto3 is not a project
dependency. Replace the direct S3 access with the LlamaStack
client's files API (client.files.list/delete), which requires
no additional dependencies.

- Remove _cleanup_s3_files helper and its boto3 dependency
- Remove S3 credential parameters from distribution fixtures
- Add _cleanup_files using the LlamaStack files API
- Snapshot existing file IDs before tests to only delete files
  created during the current test execution

Signed-off-by: Ignas Baranauskas <ibaranau@redhat.com>
Co-authored-by: Jorge <jgarciao@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

The following are automatically added/executed:

  • PR size label.
  • Run pre-commit
  • Run tox
  • Add PR author as the PR assignee
  • Build image based on the PR

Available user actions:

  • To mark a PR as WIP, add /wip in a comment. To remove it from the PR comment /wip cancel to the PR.
  • To block merging of a PR, add /hold in a comment. To un-block merging of PR comment /hold cancel.
  • To mark a PR as approved, add /lgtm in a comment. To remove, add /lgtm cancel.
    lgtm label removed on each new commit push.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
  • To Cherry-pick a merged PR /cherry-pick <target_branch_name> to the PR. If <target_branch_name> is valid,
    and the current PR is merged, a cherry-picked PR would be created and linked to the current PR.
  • To build and push image to quay, add /build-push-pr-image in a comment. This would create an image with tag
    pr-<pr_number> to quay repository. This image tag, however would be deleted on PR merge or close action.
Supported labels

{'/lgtm', '/hold', '/cherry-pick', '/verified', '/wip', '/build-push-pr-image'}

@jgarciao jgarciao merged commit 653d356 into 3.4ea1 Mar 16, 2026
8 checks passed
@jgarciao jgarciao deleted the CherryPick-3.4ea1-c3f4 branch March 16, 2026 12:23
@github-actions
Copy link
Copy Markdown

Status of building tag 3.4ea1: success.
Status of pushing tag 3.4ea1 to image registry: success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants