[Debian-astro-maintainers] Bug#1119842: astroquery: autopkgtest failure in testing: Forbidden for url: https://vizier.cds.unistra.fr/viz-bin/votable

[Paul Gevers]

Source: astroquery
Version: 0.4.11+dfsg-3
Severity: serious
User: debian-ci at lists.debian.org
Usertags: flaky

Dear maintainer(s),

Your package has an autopkgtest, great. However, the last run(s) in 
testing on amd64 failed [1]. Can you please investigate the situation 
and fix it? Your test seems to depend on external resources that 
occasionally block our hosts. Your test also fails to declare it needs 
external resources via the needs-internet restriction (although that 
wouldn't help for this case). Does it make sense to skip tests that need 
external resources (the name of the package suggest querying external 
sources might be the purpose of the package)? If not, maybe handle 
external resource availability more gracefully.

The release team has announced [2] that failing autopkgtest on amd64 and 
arm64 are considered RC in testing.

If you believe the failure is related to the infrastructure rather than 
the test of the package, don't hesitate to reach out to the ci-team on 
IRC (#debci on oftc) or via debian-ci at lists.debian.org.

Paul

PS: this bug submission was largely automated, please contact me if you 
think I should improve my workflow.

[1] https://ci.debian.net/packages/a/astroquery/testing/amd64
[2] https://lists.debian.org/debian-devel-announce/2019/07/msg00002.html

https://ci.debian.net/packages/a/astroquery/testing/amd64/65607864/

  96s =================================== FAILURES 
===================================
  96s _______________________ TestTap.test_query_target_error 
________________________
  96s
  96s self = <astroquery.esa.jwst.tests.test_jwsttap.TestTap object at 
0x7f8b729c4310>
  96s
  96s     def test_query_target_error(self):
  96s         # need to patch simbad query object here
  96s         with patch("astroquery.simbad.SimbadClass.query_object",
  96s                    side_effect=lambda object_name: parse_single_table(
  96s                        Path(__file__).parent / "data" / 
f"simbad_{object_name}.vot"
  96s                    ).to_table()):
  96s             jwst = JwstClass(show_messages=False)
  96s             simbad = SimbadClass()
  96s             ned = Ned()
  96s             vizier = Vizier()
  96s             # Testing default parameters
  96s             with pytest.raises((ValueError, TableParseError)) as err:
  96s                 jwst.query_target(target_name="M1", 
target_resolver="")
  96s                 assert "This target resolver is not allowed" in 
err.value.args[0]
  96s             with pytest.raises((ValueError, TableParseError)) as err:
  96s >               jwst.query_target("TEST")
  96s
  96s 
/usr/lib/python3/dist-packages/astroquery/esa/jwst/tests/test_jwsttap.py:970: 

  96s _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
_ _ _ _ _ _ _
  96s /usr/lib/python3/dist-packages/astroquery/esa/jwst/core.py:549: in 
query_target
  96s     coordinates = 
self.resolve_target_coordinates(target_name=target_name,
  96s /usr/lib/python3/dist-packages/astroquery/esa/jwst/core.py:593: in 
resolve_target_coordinates
  96s     result_table = Vizier.query_object(target_name,
  96s 
/usr/lib/python3/dist-packages/astroquery/utils/class_or_instance.py:25: 
in f
  96s     return self.fn(obj, *args, **kwds)
  96s            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  96s 
/usr/lib/python3/dist-packages/astroquery/utils/process_asyncs.py:34: in 
newmethod
  96s     response.raise_for_status()
  96s _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
_ _ _ _ _ _ _
  96s
  96s self = <Response [403]>
  96s
  96s     def raise_for_status(self):
  96s         """Raises :class:`HTTPError`, if one occurred."""
  96s
  96s         http_error_msg = ""
  96s         if isinstance(self.reason, bytes):
  96s             # We attempt to decode utf-8 first because some servers
  96s             # choose to localize their reason strings. If the string
  96s             # isn't utf-8, we fall back to iso-8859-1 for all other
  96s             # encodings. (See PR #3538)
  96s             try:
  96s                 reason = self.reason.decode("utf-8")
  96s             except UnicodeDecodeError:
  96s                 reason = self.reason.decode("iso-8859-1")
  96s         else:
  96s             reason = self.reason
  96s
  96s         if 400 <= self.status_code < 500:
  96s             http_error_msg = (
  96s                 f"{self.status_code} Client Error: {reason} for 
url: {self.url}"
  96s             )
  96s
  96s         elif 500 <= self.status_code < 600:
  96s             http_error_msg = (
  96s                 f"{self.status_code} Server Error: {reason} for 
url: {self.url}"
  96s             )
  96s
  96s         if http_error_msg:
  96s >           raise HTTPError(http_error_msg, response=self)
  96s E           requests.exceptions.HTTPError: 403 Client Error: 
Forbidden for url: https://vizier.cds.unistra.fr/viz-bin/votable
  96s
  96s /usr/lib/python3/dist-packages/requests/models.py:1026: HTTPError
  96s ----------------------------- Captured stdout call 
-----------------------------
  96s INFO: SIMBAD could not resolve this target [astroquery.esa.jwst.core]
  96s INFO: NED could not resolve this target [astroquery.esa.jwst.core]
  96s ------------------------------ Captured log call 
-------------------------------
  96s INFO     astropy:core.py:582 SIMBAD could not resolve this target
  96s INFO     astropy:core.py:590 NED could not resolve this target

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 585 bytes
Desc: OpenPGP digital signature
URL: <http://alioth-lists.debian.net/pipermail/debian-astro-maintainers/attachments/20251101/56225603/attachment.sig>