-
Notifications
You must be signed in to change notification settings - Fork 4
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
support structure.sql #17
Comments
For: #17 This checks common CA paths and adds it to the connection string.
I’ve been running into this. It’s great to see you're already on it! 🙏 |
Oh great! We've been struggling to recreate it consistently. We tried this branch on one persons machine but it did not resolve it. Could you share your mysqldump version? |
It’s 100% consistent on my machine. Happy to help out or even jump on a call if it would help. This is my version:
|
Thank you! I now have it reproducing on my own laptop. A solution feels close. 😄 |
@mscoutermarsh it seems to mostly work, but there’s an incompatibility with the This is the error I’m getting
My workaround for now is to:
|
Which Rails version? Latest? It should be skipping the dump step. |
Ah, I've found the problem. Will try to get a fix done soon. |
A fix for the schema dump has been released in In if ENV["DISABLE_SCHEMA_DUMP"]
config.active_record.dump_schema_after_migration = false
end |
@mscoutermarsh this seems to work perfectly now. It does leave me wondering how anyone would notice drift between the local |
@joeldrapper great! We've found schema drift to be pretty unlikely when using this flow. The top cause is people modifying production directly. The other common cause of schema drift in most Rails apps is failed production migrations (running halfway and failing). This isn't an issue either when using the branching flow since we run every schema change through vitess online DDL. It must be successful before we write to the So while it is unlikely because of the above. I think the best way to check would be to do the following:
|
Currently,
psdb:migrate
errors for apps that usestructure.sql
for their schema.Due to an SSL error with
mysqldump
. We likely need to modify some setting so that mysqldump passes the correct certificate when connecting.The text was updated successfully, but these errors were encountered: