fedora-infra / fedimg

Service to automatically upload built Fedora images to internal and external cloud providers.
GNU Affero General Public License v3.0
22 stars 24 forks source link

Fix invalid parameter error when calling euca-import-volume command #162

Open phsmoura opened 2 years ago

phsmoura commented 2 years ago

Changes description

trigger_upload.py wasn't working because when it calls euca-import-volume, that command was throwing an error about an invalid format of the manifest url when concatenating it with the path of the file. This PR works around that problem because euca-import-volume works just fine when executed directly from /tmp directory.

Here is last lines of the log with those changes:

[2022-02-17 01:11:40][fedimg.services.ec2.ec2imgpublisher    INFO] Publish image (ami-0499d828467cb4cf1) in eu-west-2 started
[2022-02-17 01:11:40][fedimg.services.ec2.ec2imgpublisher    INFO] Publish image (ami-0499d828467cb4cf1) in eu-west-2 completed
[2022-02-17 01:11:40][fedimg.services.ec2.ec2imgpublisher    INFO] Publish snaphsot for image (ami-0499d828467cb4cf1) in eu-west-2 started
[2022-02-17 01:11:41][fedimg.services.ec2.ec2imgpublisher    INFO] Fetched snapshot for image (ami-0499d828467cb4cf1): snap-0bfc5543eb0cd474e
[2022-02-17 01:11:41][fedimg.services.ec2.ec2imgpublisher    INFO] Publish snaphsot for image (ami-0499d828467cb4cf1) in eu-west-2 completed
[2022-02-17 01:11:41][fedimg.uploader    INFO] AWS EC2Service process is completed.

Related Issue

Signed-off-by: Pedro Moura pmoura@redhat.com