[GitHub] [trafficcontrol] ocket8888 opened a new issue #4954: 'admin' tool should use pq

[GitBox <gi...@apache.org>]

ocket8888 opened a new issue #4954:
URL: https://github.com/apache/trafficcontrol/issues/4954


   ## I'm submitting a ...
   -  improvement request (usability, performance, tech debt, etc.)
   
   ## Traffic Control components affected ...
   - other
   
   ## Current behavior:
   `admin` shells out to subprocess using a `psql` binary` for database operations.
   
   ## Expected / new behavior:
   The `admin` tool should use a postgresql adapter to make database connections instead of relying on shelling out to a `psql` binary somewhere in the system's path. That'd make database errors more transparent (I'm currently getting the extremely helpful error message "unable to determine if user already exists: exit status 2"), and would eliminate the dependency on the postgresql client.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] MylesBock commented on issue #4954: 'admin' tool should use pq

[GitBox <gi...@apache.org>]

MylesBock commented on issue #4954:
URL: https://github.com/apache/trafficcontrol/issues/4954#issuecomment-732473716


   I think I have something that could end up solving this, may I be assigned this issue?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org