Added host to database connection strings #1212
Conversation
dmytrotsko
force-pushed
the
EPID-66-check-for-other-acquisition-processes-that-might-still-be-running-without-a-db-host-specified
branch
from
9e5ab98
to
cc5e1c6
Compare
|
Kudos, SonarCloud Quality Gate passed!
|
There was a problem hiding this comment.
looks good... but before merging (as in #1211), we need to figure out if there are data migrations that need to happen and plan/schedule those. its also possible that many of these jobs may not even be on the run queue right now, but we will need someone with knowledge of and credentials for cronicle to help us determine that.
| # connect | ||
| u, p = secrets.db.auto | ||
| cnx = mysql.connector.connect(user=u, password=p, database="automation") | ||
| cnx = mysql.connector.connect(user=u, password=p, database="automation", host=secrets.db.host) |
There was a problem hiding this comment.
this uses a different database than the others (they use "epidata"), and on prod, the "automation" database has no tables in it. im guessing this job isnt being run in cronicle currently, but its worth checking up on it.
Co-authored-by: melange396 <[email protected]>
|
Kudos, SonarCloud Quality Gate passed!
|
…-that-might-still-be-running-without-a-db-host-specified
|
Kudos, SonarCloud Quality Gate passed!
|
…-that-might-still-be-running-without-a-db-host-specified
|
Summary:
Added missing host to database connection strings to avoid wrong database connection
Prerequisites:
devbranchdev