-
Notifications
You must be signed in to change notification settings - Fork 11
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
Upgrade ansible-docker, Docker, and Docker Compose #800
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine, however I was able to provision fine without the Docker Engine/Compose version bump. I would consider leaving that out of this PR. For example, we're using an older Compose file format (2.0
) and I'm not sure what the implications of using that on a newer Docker Engine would be.
Also, it's worth surfacing that I had issues provisioning because I do not have NFS configured on my workstation. I had to make the following changes:
diff --git a/Vagrantfile b/Vagrantfile
index 8ebb89c..6af7326 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -39,7 +39,7 @@ Vagrant.configure("2") do |config|
config.vm.network :forwarded_port, guest: 9401, host: ENV.fetch("PFB_TILEGARDEN_DEBUG_PORT", 9401)
config.vm.synced_folder "~/.aws", "/home/vagrant/.aws"
- config.vm.synced_folder '.', ROOT_VM_DIR, type: PFB_SHARED_FOLDER_TYPE, mount_options: PFB_MOUNT_OPTIONS
+ config.vm.synced_folder '.', ROOT_VM_DIR
config.vm.provision "shell" do |s|
s.path = 'deployment/vagrant/cd_shared_folder.sh'
This isn't something to address in this PR, just something to be aware of.
You were able to provision without the Docker Engine version bump? That failed for me after a
I agree that the Compose version bump isn't necessary. I'm marginally more concerned about the implications of running a significantly older Compose version with a new Engine version than the compatibility between a new Compose version and an older Compose file format, since Docker Compose hasn't formally deprecated or dropped support for any Compose file versions thus far (would be declared here). Development only and not something I feel strongly about though, your call! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I created azavea/ansible-docker#17 to track upgrading the default Docker version to |
Overview
Upgrades the Ansible Docker module to
5.0.0
, which uses the non-deprecateddownload.docker.com
repository URL. Also upgrades Docker to18.*
, the latest version available in the main repo and the version currently running on ECS; and Docker Compose to1.23.*
, the latest version that will work with the current VM configuration.Notes
I upgraded Docker Compose to
1.23.*
, which is not the most recent version. Upgrading to1.24.*
or1.25.*
resulted in a segmentation fault for any subsequent step. I didn't dig very far into the root cause of this problem and instead created #799 to track it.Testing Instructions
scripts/setup
and verify that VM is created successfully.vagrant ssh
to SSH into VM.scripts/update
to build Docker images.scripts/test
and verify that all tests pass.Checklist
Connects #798