Skip to content

Fix failed download and copy of artifact rpms - #5108

Open
bajertom wants to merge 2 commits into
mainfrom
tbajer-failed-download-and-copy-of-rpms-artifact
Open

Fix failed download and copy of artifact rpms#5108
bajertom wants to merge 2 commits into
mainfrom
tbajer-failed-download-and-copy-of-rpms-artifact

Conversation

@bajertom

@bajertom bajertom commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Fixes #4886 and #5018.

Second commit might be irrelevant since the first one raises on the no-download problem. Not sure if a scenario where the try/except block would actually go with the except clause could happen now, but #5020 should introduce an option to allow for this behavior, so I kept the checks.

  • implement the feature
  • extend the test coverage?

@bajertom bajertom changed the title Tbajer failed download and copy of rpms artifact Fix failed download and copy of artifact rpms Aug 6, 2026
@bajertom bajertom added this to planning Aug 6, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Aug 6, 2026
@bajertom bajertom moved this from backlog to review in planning Aug 6, 2026
Comment thread tmt/steps/prepare/artifact/providers/copr_build.py
Comment thread tmt/steps/prepare/artifact/providers/file.py
@bajertom
bajertom force-pushed the tbajer-failed-download-and-copy-of-rpms-artifact branch from 5eeff46 to f19f6a6 Compare August 6, 2026 20:22
@bajertom
bajertom force-pushed the tbajer-failed-download-and-copy-of-rpms-artifact branch from f19f6a6 to c5ac41c Compare August 6, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: review

Development

Successfully merging this pull request may close these issues.

prepare/artifact: fail with clear error when no RPMs are downloaded

1 participant