Post backup scripts in v1.0 about mysql-backup HOT 3 CLOSED

I appreciate the civility of, "I'm not sure if it's a typo". 😁 It most certainly is a typo, and thank you for catching it.

Since you are comfortable with go, did you try running it with that changed and see if it resolves your issue? Obviously will open a PR to fix that, but it would be good to understand if that alone is your issue, or something else.

from mysql-backup.

I wondered why the integration tests didn't catch it. I just realized why. The integration tests do not run the actual command-line executable. Instead they call the library that is called from the CLI, setting the parameters explicitly. It is much more difficult to write and manage tests without that.

There is a separate set (within the unit tests) that check if parameters get passed properly to those library calls, but that specifically parameter is not exercised. We shall fix that.

from mysql-backup.

Hi, I've been running the docker images so haven't actually needed the go build chain just yet. I'm happy to make a PR with that change.

There is another issue where AWS_ENDPOINT_URL doesn't seem to be respected if in an ENV var vs passed as --aws-endpoint-url, but I didn't want to muddy the issue. Could that be a similar omission in config handling?

Edit: and thank you for the super quick reply!

from mysql-backup.