sparkfabrik / sparkdock

Automatic MacOS system provisioning
20 stars 8 forks source link

Stable support for Ubuntu from 17.04 to 18.04 LTS + #20

Closed stickgrinder closed 6 years ago

stickgrinder commented 6 years ago

Changes in this MR:

NOTE: This MR can be merged but requires real-world testing. I planned to test it in virtualbox but still didn't do this. We can ask someone in Spark to test it against 18.04 maybe?

grota commented 6 years ago

LGTM

Pinolo commented 6 years ago

Something was wrong trying to use the new installer on an upgraded 18.04:

Here's a diff I used to get through (linux-image-extra was already installed in my system, but does not match my kernel version)

diff --git a/ansible/ubuntu/ubuntu18.yml b/ansible/ubuntu/ubuntu18.yml
index 25f80e1..288d1f2 100644
--- a/ansible/ubuntu/ubuntu18.yml
+++ b/ansible/ubuntu/ubuntu18.yml
@@ -5,7 +5,7 @@
   become_user: root

   vars:
-    apt_repository: deb [arch=amd64] https://download.docker.com/linux/ubuntu {{ ansible_lsb.id|lower }}-{{ ansible_lsb.codename|lower }} stable
+    apt_repository: deb [arch=amd64] https://download.docker.com/linux/ubuntu {{ ansible_lsb.codename|lower }} stable
     docker_compose_version: 1.21.0
     dnsdock_image: aacebedo/dnsdock:v1.16.4-amd64

@@ -30,11 +30,10 @@
     - name: Install the base required dependencies and suggested packages
       apt:
         pkg:
-          - "linux-image-extra-{{ ansible_kernel }}"
           - linux-image-extra-virtual
           - apt-transport-https
           - ca-certificates
-          - docker-ce
+          - docker-engine
           - git
         state: latest
         update_cache: yes
@@ -70,4 +69,4 @@
       become: no

     - name: Restart systemd-resolved
-      service: name=systemd-resolved state=restarted
\ No newline at end of file
+      service: name=systemd-resolved state=restarted
stickgrinder commented 6 years ago

You're right for the package, it has been removed by 18.04. It used to work during beta thou... (at least, in my memory :D)

For the other changes:

Let me check it twice and I'll add the right reference to the playbook. Thanks for the test!

stickgrinder commented 6 years ago

@Pinolo @MarianoFranzese here is the problem: https://github.com/docker/for-linux/issues/290

Now the repository is there but the package is not. I'm changing the repo to test channel so we can learn if RC1 is working. We can still use artful repo instead if you experience any issue. June will see stable 18.05 release and we'll switch the repo back to stable.

I'm also updating the docs to reflect this.

stickgrinder commented 6 years ago

OK, this PR has been tested on two PCs now and it seems to work (or at least it doesn't harm).

I'm going to merge it to develop and if it needs refinement, we'll add them with other PRs.