Source: astroquery
Version: 0.4.11+dfsg-3
Severity: serious
User: [email protected]
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 [email protected].

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

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature

Reply via email to