Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Postgres unclear error message for test connection #13934

Closed
OnkarVO7 opened this issue Nov 10, 2023 · 1 comment · Fixed by #14897
Closed

Postgres unclear error message for test connection #13934

OnkarVO7 opened this issue Nov 10, 2023 · 1 comment · Fixed by #14897
Assignees
Labels
enhancement New feature or request Ingestion

Comments

@OnkarVO7
Copy link
Contributor

OnkarVO7 commented Nov 10, 2023

Postgres test connection gives below error when postgres is running locally and OM is running on docker
image

Describe the task
Improve the error message to point exactly what the error is and show the networking issue
The issue is resolved by following the steps here and putting the hostport as ip_address:5432

@OnkarVO7 OnkarVO7 self-assigned this Nov 10, 2023
@harshach harshach moved this to Ingestion - Bugs & Minor Features in Release 1.3.0 Nov 30, 2023
@pmbrull pmbrull added the enhancement New feature or request label Jan 16, 2024
@pmbrull pmbrull assigned pmbrull and unassigned OnkarVO7 Jan 25, 2024
@pmbrull
Copy link
Collaborator

pmbrull commented Jan 26, 2024

the error is HTTP/1.1 header parser received no bytes.

We are getting that in a quite specific scenario:

  • we point to a known URL (it does not get raised if we go, for example, to random:9876)
  • the url does not allow the connection of the driver

For example, in a testing setting in OM, we can reproduce the error if we add the Postgres host as ingestion:8080 (where Airflow runs), which clearly won't respond properly to the JDCB connection.

Then, Airflow receives the POST, starts trying to connect and does not send anything back to the OM client, which raises the HTTP/1.1 header parser received no bytes

pmbrull added a commit to pmbrull/OpenMetadata that referenced this issue Jan 26, 2024
pmbrull added a commit to pmbrull/OpenMetadata that referenced this issue Jan 26, 2024
harshach pushed a commit that referenced this issue Jan 28, 2024
* #13934 - Update test connection no response error msg

* #13934 - Update test connection no response error msg
@harshach harshach moved this from Ingestion - Bugs & Minor Features to Done in Release 1.3.0 Jan 28, 2024
pmbrull added a commit that referenced this issue Jan 28, 2024
* #13934 - Update test connection no response error msg

* #13934 - Update test connection no response error msg

* MINOR - Fix python CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Ingestion
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants