Ingestion api fail having % in ingestion name - Githubissues

open-metadata / OpenMetadata

OpenMetadata is a unified metadata platform for data discovery, data observability, and data governance powered by a central metadata repository, in-depth column level lineage, and seamless team collaboration.

https://open-metadata.org

Apache License 2.0

5.65k stars 1.06k forks source link

Ingestion api fail having % in ingestion name #13575

Closed Ashish8689 closed 1 year ago

Ashish8689 commented 1 year ago

Affected module Does it impact the UI, backend or Ingestion Framework?

Describe the bug create any metadata, usage, etc having % in its ingestion, usage name throwing us an error.

API Query: http://localhost:8585/api/v1/services/ingestionPipelines/%22redshift%25ashish.12%22.redshift%25ashish_12_lineage_7t0zhJsD/pipelineStatus?startTs=1697350193979&endTs=1697436593983
Error: "Airflow Exception [redshift%ashish_12_lineage_7t0zhJsD] due to [Malformed escape pair at index 64: http://ingestion:8080/api/v1/openmetadata/status?dag_id=redshift%ashish_12_lineage_7t0zhJsD&only_queued=true]."

To Reproduce

Screenshots or steps to reproduce

Expected behavior A clear and concise description of what you expected to happen.

Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: [e.g. 0.8]
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context Add any other context about the problem here.

harshach commented 1 year ago

@Ashish8689 what is the name of the service you created

Ashish8689 commented 1 year ago

I have created it for database and it name was like service%data.12

pmbrull commented 1 year ago

built new, service name is my%sql.12 and it LGTM

I can see the API as 200 and the status in the UI. I'll be closing this.

If the error appears again and we can reproduce it we can try to check further