Skip to content

Conversation

@paha77
Copy link
Contributor

@paha77 paha77 commented May 21, 2025

@deitch
Copy link
Collaborator

deitch commented May 21, 2025

Great, thanks. Please sign your commits with git commit -s

Signed-off-by: Péter Bakonyi-Kárpáti <pbk@vivid-planet.com>
@paha77 paha77 force-pushed the addCronTestCaseForSunday branch from 84c8588 to f8c5ca0 Compare May 22, 2025 04:01
@paha77
Copy link
Contributor Author

paha77 commented May 22, 2025

Great, thanks. Please sign your commits with git commit -s

@deitch
Copy link
Collaborator

deitch commented May 23, 2025

Kicked off CI, hopefully it goes red, and then we can try to solve.

@deitch
Copy link
Collaborator

deitch commented May 23, 2025

And it does pass. It should have turned red. Which means, at least , that the lower-level processing is working correctly.

How can we recreate it?

@paha77
Copy link
Contributor Author

paha77 commented May 25, 2025

Is it possible to turn on debugging / logging?

I have

DB_DUMP_DEBUG: true

in my docker-compose.yml but there's nothing in the logs.

@deitch
Copy link
Collaborator

deitch commented May 25, 2025

Try DB_DEBUG=true. See the var in configuration doc.

Also needs rebase, but shouldn't affect anything.

@deitch
Copy link
Collaborator

deitch commented May 28, 2025

Anything interesting out of debugging? I really would like to see this replicated and fixed.

@paha77
Copy link
Contributor Author

paha77 commented May 29, 2025

DB_DEBUG=true

With this option I get this message after starting the container:

backup-mysql-1  | time="2025-05-29T04:15:22Z" level=debug msg="starting dump"

Of course I have to wait until Sunday to see if anything happens and is written in the log.

@deitch
Copy link
Collaborator

deitch commented May 29, 2025

Here is my trick for things like this. I spin up a VM, change the clock, and run the program there. Make it Sunday at 0150 inside the VM. Just be sure to disable ntpd so it doesn't change it back on you.

@paha77
Copy link
Contributor Author

paha77 commented Jun 2, 2025

Quite strange, it seems to be working now, the backup was made at 2:00 on Sunday, sorry. You can close the PR or if you'd like to habe a further test case, you can merge it. Thank you again for your support.

@deitch
Copy link
Collaborator

deitch commented Jun 3, 2025

It isn't the first strange and unexplained thing I have seen in software, I am sure it won't be the last. 😁

I will merge it in. The extra test always is good.

@deitch deitch merged commit 461c3d7 into databacker:master Jun 3, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants