https://prefect.io logo
m

Marwan Sarieddine

10/21/2020, 1:58 AM
Hi folks, I am getting an INTERVAL_SERVER_ERROR trying to enable heartbeat using a graphql mutation query to prefect cloud
c

Chris White

10/21/2020, 1:59 AM
Hi @Marwan Sarieddine - to be precise, what graphql call are you making?
m

Marwan Sarieddine

10/21/2020, 1:59 AM
query:
Copy code
mutation {
            enable_flow_heartbeat(input: { flow_id: "$flow_id" }) {
                success
            }
        }
traceback
Copy code
File "/Users/marwansarieddine/.pyenv/versions/etl-embs/lib/python3.8/site-packages/infima/toolkit/prefect/runner.py", line 174, in _toggle_heartbeat
    return _toggle_flow_option(client=client, flow_id=flow_id, action=action)
  File "/Users/marwansarieddine/.pyenv/versions/etl-embs/lib/python3.8/site-packages/infima/toolkit/prefect/runner.py", line 169, in _toggle_flow_option
    client.graphql(query=query)
  File "/Users/marwansarieddine/.pyenv/versions/etl-embs/lib/python3.8/site-packages/prefect/client/client.py", line 294, in graphql
    raise ClientError(result["errors"])
prefect.utilities.exceptions.ClientError: [{'path': ['enable_flow_heartbeat'], 'message': '"\'Box\' object has no attribute \'enable_heartbeat_for_flow\'"', 'extensions': {'code': 'INTERNAL_SERVER_ERROR'}}]
hi @Chris White - please see the query and traceback above (thank you)
c

Chris White

10/21/2020, 2:00 AM
ok interesting, this definitely looks like a bug on our side
m

Marwan Sarieddine

10/21/2020, 2:01 AM
disable_flow_heartbeat
seems to be working fine on the other hand ...
Copy code
mutation {
            disable_flow_heartbeat(input: { flow_id: "$flow_id" }) {
                success
            }
        }
c

Chris White

10/21/2020, 2:07 AM
AH! We recently did an internal refactor of the API for flow settings and it appears you found a route that wasn’t correctly updated. I can prioritize this and we should be able to release it fairly quickly (on the order of 1-2 days)
m

Marwan Sarieddine

10/21/2020, 2:08 AM
I see - thanks for looking into this, it is not mission critical from our end given there are other ways to enable heartbeat ... but it would be nice to have it fixed 🙂
c

Chris White

10/21/2020, 2:08 AM
yup we’ll definitely get it fixed 👍 I’ll report back when it’s queued up for release
m

Marwan Sarieddine

10/21/2020, 2:09 AM
sounds good - thank you!
c

Chris White

10/22/2020, 9:57 PM
Hi @Marwan Sarieddine - the fix has been released, you should be able to re-enable heartbeats for your flows now! Thanks for bringing this to our attention
m

Marwan Sarieddine

10/22/2020, 10:07 PM
Hi @Chris White - thank you for the update 🙂
👍 1
3 Views