Merge pull request #510 from greendeath/f/docker_absent

F/docker absent

cdist/conf/type/__docker/man.rst

@@ -26,17 +26,24 @@ BOOLEAN PARAMETERS
 experimental
    Install the experimental docker-engine package instead of the latest stable release.
 
+state
+   'present' or 'absent', defaults to 'present'
+
 
 EXAMPLES
 --------
 
 .. code-block:: sh
 
+    # Install docker
     __docker
 
-    # experimental
+    # Install experimental
     __docker --experimental
 
+    # Remove docker
+    __docker --state absent
+
 
 AUTHORS
 -------

cdist/conf/type/__docker/manifest

@@ -20,6 +20,7 @@
 
 
 os=$(cat "$__global/explorer/os")
+state=$(cat "$__object/parameter/state")
 
 case "$os" in
     centos)
@@ -27,31 +28,31 @@ case "$os" in
         if [ -f "$__object/parameter/experimental" ]; then
           component="experimental"
         fi
-      export CDIST_ORDER_DEPENDENCY=on
         __yum_repo docker \
           --name 'Docker Repository' \
           --baseurl "https://yum.dockerproject.org/repo/$component/centos/\$releasever/" \
           --enabled \
-         --gpgcheck \
-         --gpgkey 'https://yum.dockerproject.org/gpg'
-      __package docker-engine
-      unset CDIST_ORDER_DEPENDENCY
+          --gpgcheck 1 \
+          --gpgkey 'https://yum.dockerproject.org/gpg' \
+          --state ${state} 
+        require="__yum_repo/docker" __package docker-engine --state ${state}
     ;;
     ubuntu)
         component="main"
       if [ -f "$__object/parameter/experimental" ]; then
          component="experimental"
       fi
-      __package apparmor
-      __package ca-certificates
-      __package apt-transport-https
-      __apt_key docker --keyid 58118E89F3A912897C070ADBF76221572C52609D
+      __package apparmor --state ${state}
+      __package ca-certificates --state ${state}
+      __package apt-transport-https --state ${state}
+      __apt_key docker --keyid 58118E89F3A912897C070ADBF76221572C52609D --state ${state}
       export CDIST_ORDER_DEPENDENCY=on
       __apt_source docker \
          --uri https://apt.dockerproject.org/repo \
          --distribution "ubuntu-$(cat "$__global/explorer/lsb_codename")" \
+         --state ${state} \
          --component "$component"
-      __package docker-engine
+      __package docker-engine --state ${state}
       unset CDIST_ORDER_DEPENDENCY
    ;;
    debian)
@@ -60,16 +61,17 @@ case "$os" in
          component="experimental"
       fi
 
-      __package apt-transport-https
-      __package ca-certificates
-      __package gnupg2
-      __apt_key docker --keyid 58118E89F3A912897C070ADBF76221572C52609D
+      __package apt-transport-https --state ${state}
+      __package ca-certificates --state ${state}
+      __package gnupg2 --state ${state}
+      __apt_key docker --keyid 58118E89F3A912897C070ADBF76221572C52609D --state ${state}
       export CDIST_ORDER_DEPENDENCY=on
       __apt_source docker \
           --uri https://apt.dockerproject.org/repo \
           --distribution "debian-$(cat "$__global/explorer/lsb_codename")" \
+          --state ${state} \
           --component "$component"
-      __package docker-engine
+      __package docker-engine --state ${state}
       unset CDIST_ORDER_DEPENDENCY
 
    ;;

cdist/conf/type/__docker/parameter/default/state

@@ -0,0 +1 @@
+present

cdist/conf/type/__docker/parameter/optional

@@ -0,0 +1 @@
+state

cdist/conf/type/__docker_compose/gencode-remote

@@ -20,9 +20,12 @@
 
 # Variables
 version="$(cat "$__object/parameter/version")"
+state="$(cat "$__object/parameter/state")"
 
+if [ ${state} = "present" ]; then
     # Download docker-compose file
         echo 'curl -L "https://github.com/docker/compose/releases/download/'${version}'/docker-compose-$(uname -s)-$(uname -m)" -o /tmp/docker-compose'
         echo 'mv /tmp/docker-compose /usr/local/bin/docker-compose'
     # Change permissions
     echo 'chmod +x /usr/local/bin/docker-compose'
+fi

cdist/conf/type/__docker_compose/man.rst

@@ -9,6 +9,8 @@ cdist-type__docker_compose - install docker-compose
 DESCRIPTION
 -----------
 Installs docker-compose package.
+State 'absent' will not remove docker binary itself,
+only docker-compose binary will be removed
 
 
 REQUIRED PARAMETERS
@@ -21,6 +23,9 @@ OPTIONAL PARAMETERS
 version
    Define docker_compose version, defaults to "1.9.0" 
 
+state
+   'present' or 'absent', defaults to 'present'
+
 
 BOOLEAN PARAMETERS
 ------------------
@@ -32,11 +37,15 @@ EXAMPLES
 
 .. code-block:: sh
 
+    # Install docker-compose
     __docker_compose
 
     # Install version 1.9.0-rc4
     __docker_compose --version 1.9.0-rc4
 
+    # Remove docker-compose 
+    __docker_compose --state absent
+
 
 AUTHORS
 -------

cdist/conf/type/__docker_compose/manifest

@@ -19,6 +19,15 @@
 #
 #
 
+state="$(cat "$__object/parameter/state")"
+
 # Needed packages
+if [ ${state} = "present" ]; then
     __docker
     __package curl
+elif [ ${state} = "absent" ]; then
+    __file /usr/local/bin/docker-compose --state absent
+else
+    echo "Unknown state: ${state}" >&2
+    exit 1
+fi

cdist/conf/type/__docker_compose/parameter/default/state

@@ -0,0 +1 @@
+present

cdist/conf/type/__docker_compose/parameter/optional

@@ -1 +1,2 @@
+state
 version