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 Driver not available, cannot use postgres sink. #7043

Closed
brennhill opened this issue Oct 3, 2021 · 3 comments
Closed

Postgres Driver not available, cannot use postgres sink. #7043

brennhill opened this issue Oct 3, 2021 · 3 comments
Assignees
Labels
T:bug Type Bug (Confirmed)

Comments

@brennhill
Copy link

brennhill commented Oct 3, 2021

Tendermint version (use tendermint version or git rev-parse --verify HEAD if installed from source):
0.34.13

ABCI app (name for built-in, URL for self-written if it's publicly available):
Any

Environment:

  • OS (e.g. from /etc/os-release): OSX
  • Install tools:
  • Others:

What happened:
When configuring config.toml to use postgres driver, this will fail to function properly, even if postgres is available and the connection string is valid. This is because there is no postgres driver available.

What you expected to happen:
I would expect tendermint to connect to postgres without further configuration OR very clear instructions that we need to import a postgres driver in our app, e.g.
import (_ "github.com/lib/pq")

Have you tried the latest version: yes

How to reproduce it (as minimally and precisely as possible):
Try using the postgres sink without importing a postgres driver in your app.

Logs (paste a small part showing an error (< 10 lines) or link a pastebin, gist, etc. containing more of the log file):

Config (you can paste only the changes you've made):

node command runtime flags:

Please provide the output from the http://<ip>:<port>/dump_consensus_state RPC endpoint for consensus bugs

Anything else we need to know:

@brennhill brennhill changed the title Postgres Driver Postgres Driver not available, cannot use postgres sink. Oct 3, 2021
@creachadair
Copy link
Contributor

Sorry for the confusion. I agree that whatever else we do, we should definitely document this requirement more clearly. And, while it is an "optional" dependency, it probably makes sense for the node to include it, since it's the node rather than the application that depends on it.

@creachadair creachadair self-assigned this Oct 4, 2021
creachadair pushed a commit that referenced this issue Oct 4, 2021
I accidentally omitted this from the backport in #6906.
Fixes #7043.
@creachadair creachadair added the T:bug Type Bug (Confirmed) label Oct 4, 2021
creachadair pushed a commit that referenced this issue Oct 4, 2021
I accidentally omitted this from the backport in #6906.
Fixes #7043.
@creachadair
Copy link
Contributor

The fix is merged and will go out with v0.34.14. It is also already addressed in the forthcoming v0.35 release.

@brennhill
Copy link
Author

Great,thank you so much @creachadair

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T:bug Type Bug (Confirmed)
Projects
None yet
Development

No branches or pull requests

2 participants