Fix retry for Pyxis request timeout

In 31cc4ca, we added retry mechanism when Pyxis request timeout, however
we detect the Pyxis request timeout by:

    if "Pyxis API was unable to fetch data from MongoDB in time" in error

`error` is actually a dict, not a string, so this doesn't work.

JIRA: CWFHEALTH-2315

freshmaker/pyxis_gql.py

@@ -98,7 +98,10 @@ def query(self, query_dsl):
         if error is not None:
             trace_id = self._client.transport.response_headers.get("trace_id", False)
 
-            if "Pyxis API was unable to fetch data from MongoDB in time" in error:
+            if (
+                error["detail"]
+                and "Pyxis API was unable to fetch data from MongoDB in time" in error["detail"]
+            ):
                 raise PyxisGQLRequestTimeout(error=error, trace_id=trace_id)
             raise PyxisGQLRequestError(error=error, trace_id=trace_id)
 

tests/test_pyxis_gql.py

@@ -552,7 +552,7 @@ def test_log_trace_id(mock_client, mock_transport):
     result = {
         "find_images": {
             "data": [],
-            "error": ["something went wrong"],
+            "error": {"status": 500, "detail": "something went wrong"},
             "page": 0,
             "page_size": 250,
             "total": 2,
@@ -574,6 +574,5 @@ def test_log_trace_id(mock_client, mock_transport):
             content_sets=["rhel-8-for-x86_64-baseos-rpms"],
         )
     except PyxisGQLRequestError as e:
-        print(e)
         assert e.error == str(result["find_images"]["error"])
         assert e.trace_id == mock_transport.return_value.response_headers["trace_id"]