Skip to content

Move to download images using requests library.#698

Open
mykaul wants to merge 1 commit into
lago-project:masterfrom
mykaul:download_using_requests
Open

Move to download images using requests library.#698
mykaul wants to merge 1 commit into
lago-project:masterfrom
mykaul:download_using_requests

Conversation

@mykaul

@mykaul mykaul commented Mar 15, 2018

Copy link
Copy Markdown

More modern, hopefully somewhat faster.

@mykaul mykaul force-pushed the download_using_requests branch from 2fc5a9f to b831a10 Compare March 15, 2018 11:05
@eedri

eedri commented Mar 15, 2018

Copy link
Copy Markdown
Contributor

Is python-requests available on RHEL and Centos? this will be needed for lago demo tool and running lago on RHEL.

@mykaul

mykaul commented Mar 15, 2018

Copy link
Copy Markdown
Author

@eedri I'd be very surprised if it's not, but apparently things are failing :(

Comment thread requirements.txt
Jinja2
xmltodict
netaddr
requests

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be added to the spec file as well.

Comment thread lago/templates.py
import shutil
import urllib
import sys
import urllib

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we still need urllib?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is still used in the code.

More modern, hopefully somewhat faster.
@mykaul mykaul force-pushed the download_using_requests branch from a0a5b2b to c4e1107 Compare April 11, 2018 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants