Skip to content
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

Add Support for Ports #2

Closed
TPGeek opened this issue Dec 13, 2017 · 4 comments
Closed

Add Support for Ports #2

TPGeek opened this issue Dec 13, 2017 · 4 comments

Comments

@TPGeek
Copy link

TPGeek commented Dec 13, 2017

Hello,

is it possible for you to add support for custom / bespoke ports please as the SQL service we use, doesn't run on the default port.

Tom

@selim13
Copy link
Owner

selim13 commented Dec 14, 2017

I've added DBPORT option.
New build should already be on the docker hub.
Test it and tell if it works for you.

@TPGeek
Copy link
Author

TPGeek commented Dec 14, 2017

On my docker host, It starts up then just stops right away.

@selim13
Copy link
Owner

selim13 commented Dec 15, 2017

I've just tested it out, and everything worked perfectly. Need more info.

Here is an example docker-compose file I've used for testing.

version: "2"

volumes:
  mysql-data: {}
  mysql-backups-data: {}

services:
  mysql:
    image: mysql:latest
    volumes:
      - "mysql-data:/var/lib/mysql"
    environment:
      MYSQL_ROOT_PASSWORD: "my-secret-pw"
    command:
      [
      --port=6666
      ]

  mysqlbackup:
    image: selim13/automysqlbackup
    volumes:
      - "mysql-backups-data:/var/lib/mysql"
    environment:
      USERNAME: root
      PASSWORD: "my-secret-pw"
      DBHOST: mysql
      DBPORT: 6666
      CRON_SCHEDULE: "* * * * *"
    depends_on:
      - mysql

Logs:

...
Backup Information for /backup/daily/performance_schema/performance_schema_2017-12-15_10h34m.Friday.sql
...

@TPGeek TPGeek closed this as completed Dec 16, 2017
@TPGeek
Copy link
Author

TPGeek commented Dec 16, 2017

Perfect, Thank you!

@TPGeek TPGeek reopened this Dec 16, 2017
@TPGeek TPGeek closed this as completed Dec 16, 2017
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

No branches or pull requests

2 participants