Closed harshad16 closed 1 year ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mayaCostantini
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@harshad16: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
aicoe-ci/prow/pre-commit | e2fb2cab51e9c1fc6610109b7ca9942ae7a7c6e5 | link | true | /test pre-commit |
Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.
Merging it for release.
Iterate over each artifacts instead of capturing in a list Signed-off-by: Harshad Reddy Nalla hnalla@redhat.com
Related Issues and Dependencies
Related-to: https://github.com/thoth-station/thoth-application/issues/2690#issuecomment-1331752960
Description
get_package_artifacts
directly intoget_package_hashes
As each artifact object, is linked to the downloaded artifacts, they get deleted on obj deletion. With this , each artifact would be live for short period till the hash is gathered.Each artifact would be iterated one after another, instead of gathering them all in one list.
get_package_data
didn't seem to be used anywhere so it is now merged inget_package_hashes