Can you update the usage text to include the new parameter? -re On Tue, Mar 13, 2018 at 7:54 AM, Sayan Chowdhury <sayan.chowdhury2012@xxxxxxxxx> wrote: > Hi, > > fedimg contains a trigger upload script that helps in uploading the > raw images in case the upload fails. This patch fixes the script to > use the third parameter in the function that would push the compose > metadata. > > +1s? > > From: Sayan Chowdhury <sayan.chowdhury2012@xxxxxxxxx> > Date: Tue, 13 Mar 2018 17:07:53 +0530 > Subject: [PATCH 1/1] fedimg: Fix the trigger_upload script to create AMIs > manually > > --- > roles/fedimg/files/trigger_upload.py | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/roles/fedimg/files/trigger_upload.py > b/roles/fedimg/files/trigger_upload.py > index 5a5208d..29907f1 100644 > --- a/roles/fedimg/files/trigger_upload.py > +++ b/roles/fedimg/files/trigger_upload.py > @@ -16,7 +16,7 @@ from fedimg.services.ec2 import EC2Service, > EC2ServiceException > import fedimg.uploader > from fedimg.util import virt_types_from_url > > -if len(sys.argv) != 2: > +if len(sys.argv) != 3: > print 'Usage: trigger_upload.py <rawxz_image_url>' > sys.exit(1) > > @@ -26,5 +26,10 @@ log = logging.getLogger('fedmsg') > upload_pool = multiprocessing.pool.ThreadPool(processes=4) > > url = sys.argv[1] > +compose_id = sys.argv[2] > > -fedimg.uploader.upload(upload_pool, [url]) > +compose_meta = { > + 'compose_id': compose_id > +} > + > +fedimg.uploader.upload(upload_pool, [url], compose_meta=compose_meta) > -- > 2.9.4 > > > -- > Sayan Chowdhury <https://sayanchowdhury.dgplug.org/> > Senior Software Engineer, Fedora Engineering - Emerging Platform > GPG Fingerprint : 0F16 E841 E517 225C 7D13 AB3C B023 9931 9CD0 5C8B > _______________________________________________ > infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx > To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx _______________________________________________ infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx