Tempest test 'test_image_web_download_import' failing with 'URI for web-download does not pass filtering'

[Douglas Viroel]

The following error is happening in "test_image_web_download_import" test for "periodic-tripleo-ci-centos-9-standalone-full-tempest-api-tempest-wallaby" on component line, which block us from promoting tempest component:

Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/tempest/api/image/v2/test_images.py", line 133, in test_image_web_download_import
    self.client.image_import(image['id'], method='web-download',
  File "/usr/lib/python3.9/site-packages/tempest/lib/services/image/v2/images_client.py", line 244, in image_import
    resp, _ = self.post(url, data, headers=headers)
  File "/usr/lib/python3.9/site-packages/tempest/lib/common/rest_client.py", line 299, in post
    return self.request('POST', url, extra_headers, headers, body, chunked)
  File "/usr/lib/python3.9/site-packages/tempest/lib/common/rest_client.py", line 703, in request
    self._error_checker(resp, resp_body)
  File "/usr/lib/python3.9/site-packages/tempest/lib/common/rest_client.py", line 814, in _error_checker
    raise exceptions.BadRequest(resp_body, resp=resp)
tempest.lib.exceptions.BadRequest: Bad request
Details: b'400 Bad Request\n\nURI for web-download does not pass filtering: /opt/cache/files/cirros-0.5.2-x86_64-disk.img\n\n '

https://logserver.rdoproject.org/56/36356/40/check/periodic-tripleo-ci-centos-9-standalone-full-tempest-api-tempest-wallaby/6d743a4/logs/undercloud/var/log/tempest/stestr_results.html.gz
https://logserver.rdoproject.org/29/37029/29/check/periodic-tripleo-ci-centos-9-standalone-full-tempest-api-tempest-wallaby/35c35ef/logs/undercloud/var/log/tempest/stestr_results.html.gz
https://logserver.rdoproject.org/76/36976/23/check/periodic-tripleo-ci-centos-8-standalone-full-tempest-api-tempest-wallaby/0d7b11e/logs/undercloud/var/log/tempest/stestr_results.html.gz

Note that the same job is hitting another bug described at [1].
https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-9-standalone-full-tempest-api-tempest-wallaby

[1] https://bugs.launchpad.net/tripleo/+bug/1965525

[Douglas Viroel (dviroel)]

Affects master[1] too

[1] https://logserver.rdoproject.org/29/37029/29/check/periodic-tripleo-ci-centos-9-standalone-full-tempest-api-tempest-master/e493a20/logs/undercloud/var/log/tempest/stestr_results.html.gz

[Arx Cruz (arxcruz)]

added on skiplist https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/834321

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835475

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835475
Committed: https://opendev.org/openstack/tripleo-quickstart-extras/commit/3e80768a743c9a30d722cca1f6b146447363a112
Submitter: "Zuul (22348)"
Branch: master

commit 3e80768a743c9a30d722cca1f6b146447363a112
Author: Arx Cruz <email address hidden>
Date: Mon Mar 28 17:58:14 2022 +0200

    Override image.http_image on tempestconf

    The web-download test expect an url in order to work, otherwise it fails
    with an error saying unable to parse the http_image.
    Tempestconf set the http_image to the same as image_path if http_image
    is not overwriten, so, the http_image was pointing to
    /opt/cache/files/cirros. This patch set the http_image to the proper
    cirros url. I check the tempest code, and only the web_download test
    seems to be using the http image, so it should be safe to override the
    http_image here.

    Change-Id: I085177212a6e9b461009c453f7907867b118f859
    Closes-Bug: #1965540

[Marios Andreou (marios-b)]

adding info for more context

the fix proposed at https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835475

was tested there https://review.rdoproject.org/r/c/testproject/+/37916/10#message-e9879053d474e2ce85b7df8ba93362b8b9907fd2

[Ronelle Landy (rlandy)]

https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/835974 was merged - can we close this out?

[Arx Cruz (arxcruz)]

yes