Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fs/fsx:Fixing URLError for ltp link #2794

Merged
merged 1 commit into from
Apr 6, 2024

Conversation

AYUSHJAIN951
Copy link
Member

Fix issue caused by last commit of extra spaces in link

Before:

L0141 DEBUG| PARAMS (key=url, path=*, default=https: // github.com/linux-test-project/ltp/archive/master.zip) => 'https: // github.com/linux-test-project/ltp/archive/master.zip'
[stdlog] 2024-04-05 12:33:22,940 avocado.utils.asset asset            L0398 INFO | Fetching asset ltp-master.zip
[stdlog] 2024-04-05 12:33:22,940 avocado.utils.asset asset            L0402 INFO | Asset not in cache, fetching it.
[stdlog] 2024-04-05 12:33:22,941 avocado.utils.asset asset            L0152 DEBUG| Asset not in cache after lock, fetching it.
[stdlog] 2024-04-05 12:33:22,941 avocado.utils.download download         L0096 INFO | Fetching https:/// // github.com/linux-test-project/ltp/archive/master.zip -> /root/avocado/data/cache/by_location/6ec3298404f928ec28638d7c0e5bfa2ba2aa7771/ltp-master.zip.76ec2324-b94f-48b3-a163-82cec77ec954
[stderr] Process Process-1:1:
[stderr] Traceback (most recent call last):
[stderr]   File "/usr/lib/python3.10/multiprocessing/process.py", line 314, in _bootstrap
[stderr]     self.run()
[stderr]   File "/usr/lib/python3.10/multiprocessing/process.py", line 108, in run
[stderr]     self._target(*self._args, **self._kwargs)
[stderr]   File "/usr/local/lib/python3.10/dist-packages/avocado/utils/download.py", line 69, in _url_download
[stderr]     src_file = url_open(url, data=data)
[stderr]   File "/usr/local/lib/python3.10/dist-packages/avocado/utils/download.py", line 48, in url_open
[stderr]     result = urlopen(url, data=data, timeout=timeout)
[stderr]   File "/usr/lib/python3.10/urllib/request.py", line 216, in urlopen
[stderr]     return opener.open(url, data, timeout)
[stderr]   File "/usr/lib/python3.10/urllib/request.py", line 516, in open
[stderr]     req = meth(req)
[stderr]   File "/usr/lib/python3.10/urllib/request.py", line 1272, in do_request_
[stderr]     raise URLError('no host given')
[stderr] urllib.error.URLError: <urlopen error no host given>
[stdlog] 2024-04-05 12:33:22,948 avocado.utils.asset asset            L0165 INFO | Temporary asset file unavailable due to failed download attempt.
[stdlog] 2024-04-05 12:33:22,948 avocado.utils.asset asset            L0437 ERROR| FileNotFoundError: [Errno 2] No such file or directory: '/root/avocado/data/cache/by_location/6ec3298404f928ec28638d7c0e5bfa2ba2aa7771/ltp-master.zip.76ec2324-b94f-48b3-a163-82cec77ec954'

After:

(key=url, path=*, default=https://github.com/linux-test-project/ltp/archive/master.zip) => 'https://github.com/linux-test-project/ltp/archive/master.zip'
[stdlog] 2024-04-05 12:33:49,037 avocado.utils.asset asset            L0398 INFO | Fetching asset ltp-master.zip

Fix issue caused by last commit of extra spaces in link

Signed-off-by: Ayush Jain <[email protected]>
@AYUSHJAIN951 AYUSHJAIN951 changed the title Fixing URLError for ltp link fs/fsx:Fixing URLError for ltp link Apr 5, 2024
@narasimhan-v
Copy link
Member

@PraveenPenguin thanks for the review, merging.

@narasimhan-v narasimhan-v merged commit e3043c2 into avocado-framework-tests:master Apr 6, 2024
3 checks passed
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