Skip to content

Conversation

timvaillancourt
Copy link
Contributor

To make the Dockerfile small when we started I made it simply curl/wget a built-binary from our Github releases page. This will fix the Travis CI failure for PR #200 too.

Now that it will be used in travis-ci tests we need to build the actual code we changed into a Docker container. Secondly, I added debian 8 and 9 builds and made all builds happen in Docker containers of the os they build for.

Lastly, I made the docker container run MCB as non-root by installing the RPM and using the user the RPM creates to run MCB for the container.

'make release' will build docker, centos7 (rpm+bin) and debian 8 and 9 bins:

[tim@centos7 mongodb_consistent_backup]$ ls -alh ./bin/
total 17M
drwxr-xr-x.  2 root root 4.0K Sep  1 01:28 .
drwxrwxr-x. 11 tim  tim  4.0K Sep  1 01:03 ..
-rwxr-xr-x.  1 root root 5.6M Sep  1 01:19 mongodb-consistent-backup.debian8.x86_64
-rwxr-xr-x.  1 root root 5.6M Sep  1 01:28 mongodb-consistent-backup.debian9.x86_64
-rwxr-xr-x.  1 root root 5.6M Sep  1 01:10 mongodb-consistent-backup.el7.centos.x86_64
[tim@centos7 mongodb_consistent_backup]$ ls build/rpm/RPMS/x86_64/mongodb_consistent_backup-1.1.0-1.el7.centos.x86_64.rpm 
build/rpm/RPMS/x86_64/mongodb_consistent_backup-1.1.0-1.el7.centos.x86_64.rpm

@timvaillancourt timvaillancourt added this to the 1.2.0 milestone Sep 1, 2017
@dbmurphy dbmurphy merged commit 4a0752f into Percona-Lab:master Sep 7, 2017
@timvaillancourt timvaillancourt deleted the build_bin_docker branch September 7, 2017 15:31
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