Gracefully handle df errors when Dataflow not enabled on CDP tenant

cloudera-labs / cdpy

A Simple Pythonic Client wrapper for Cloudera CDP CLI, designed for use with the Ansible framework

Apache License 2.0

10 stars 21 forks source link

Gracefully handle df errors when Dataflow not enabled on CDP tenant #45

Closed jimright closed 2 years ago

jimright commented 2 years ago

Added Squelch to catch PATH_DISABLED error code that is returned when Dataflow entitlement is not enabled.

For completeness added to the following methods:

list_services
describe_service
list_deployments
describe_deployment
list_readyflows
list_flow_definitions
describe_readyflow
import_readyflow
delete_added_readyflow
describe_added_readyflow
describe_customflow

Signed-off-by: Jim Enright jenright@cloudera.com

jimright commented 2 years ago

Approved, but you can drop the field parameter in the Squelch objects because the parameter defaults to error_code. See https://github.com/cloudera-labs/cdpy/blob/devel/src/cdpy/common.py#L121

If you want to make these changes before I commit, go ahead. Otherwise, we can clean up in a subsequent PR.

Thanks @wmudge - I've made that suggested change.