You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

92 lines
2.4 KiB

Added file and container image caching (#4828) * File and container image downloads are now cached localy, so that repeated vagrant up/down runs do not trigger downloading of those files. This is especially useful on laptops with kubernetes runnig locally on vm's. The total size of the cache, after an ansible run, is currently around 800MB, so bandwidth (=time) savings can be quite significant. * When download_run_once is false, the default is still not to cache, but setting download_force_cache will still enable caching. * The local cache location can be set with download_cache_dir and defaults to /tmp/kubernetes_cache * A local docker instance is no longer required to cache docker images; Images are cached to file. A local docker instance is still required, though, if you wish to download images on localhost. * Fixed a FIXME, wher the argument was that delegate_to doesn't play nice with omit. That is a correct observation and the fix is to use default(inventory_host) instead of default(omit). See ansible/ansible#26009 * Removed "Register docker images info" task from download_container and set_docker_image_facts because it was faulty and unused. * Removed redundant when:download.{container,enabled,run_once} conditions from {sync,download}_container.yml * All features of commit d6fd0d2acaec9f53e75d82db30411f96a5bf2cc9 by Timoses <timosesu@gmail.com>, merged May 1st 2019, are included in this patch. Not all code was included verbatim, but each feature of that commit was checked to be working in this patch. One notable change: The actual downloading of the kubeadm images was moved to {download,sync)_container, to enable caching. Note 1: I considered splitting this patch, but most changes that are not directly related to caching, are a pleasant by-product of implementing the caching code, so splitting would be impractical. Note 2: I have my doubts about the usefulness of the upload, download and upgrade tags in the download role. Must they remain or can they be removed? If anybody knows, then please speak up.
5 years ago
Added file and container image caching (#4828) * File and container image downloads are now cached localy, so that repeated vagrant up/down runs do not trigger downloading of those files. This is especially useful on laptops with kubernetes runnig locally on vm's. The total size of the cache, after an ansible run, is currently around 800MB, so bandwidth (=time) savings can be quite significant. * When download_run_once is false, the default is still not to cache, but setting download_force_cache will still enable caching. * The local cache location can be set with download_cache_dir and defaults to /tmp/kubernetes_cache * A local docker instance is no longer required to cache docker images; Images are cached to file. A local docker instance is still required, though, if you wish to download images on localhost. * Fixed a FIXME, wher the argument was that delegate_to doesn't play nice with omit. That is a correct observation and the fix is to use default(inventory_host) instead of default(omit). See ansible/ansible#26009 * Removed "Register docker images info" task from download_container and set_docker_image_facts because it was faulty and unused. * Removed redundant when:download.{container,enabled,run_once} conditions from {sync,download}_container.yml * All features of commit d6fd0d2acaec9f53e75d82db30411f96a5bf2cc9 by Timoses <timosesu@gmail.com>, merged May 1st 2019, are included in this patch. Not all code was included verbatim, but each feature of that commit was checked to be working in this patch. One notable change: The actual downloading of the kubeadm images was moved to {download,sync)_container, to enable caching. Note 1: I considered splitting this patch, but most changes that are not directly related to caching, are a pleasant by-product of implementing the caching code, so splitting would be impractical. Note 2: I have my doubts about the usefulness of the upload, download and upgrade tags in the download role. Must they remain or can they be removed? If anybody knows, then please speak up.
5 years ago
  1. ---
  2. - name: prep_download | Set a few facts
  3. set_fact:
  4. download_force_cache: "{{ true if download_run_once else download_force_cache }}"
  5. tags:
  6. - facts
  7. - name: Set image info command for containerd
  8. set_fact:
  9. image_info_command: "{{ bin_dir }}/crictl images --verbose | awk -F ': ' '/RepoTags|RepoDigests/ {print $2}' | tr '\n' ','"
  10. when: container_manager == 'containerd'
  11. - name: Register docker images info
  12. shell: "{{ image_info_command }}"
  13. no_log: true
  14. register: docker_images
  15. failed_when: false
  16. changed_when: false
  17. check_mode: no
  18. when: download_container
  19. - name: prep_download | Create staging directory on remote node
  20. file:
  21. path: "{{ local_release_dir }}/images"
  22. state: directory
  23. recurse: yes
  24. mode: 0755
  25. owner: "{{ ansible_ssh_user | default(ansible_user_id) }}"
  26. when:
  27. - ansible_os_family not in ["CoreOS", "Container Linux by CoreOS"]
  28. - name: prep_download | Create local cache for files and images
  29. file:
  30. path: "{{ download_cache_dir }}/images"
  31. state: directory
  32. recurse: yes
  33. mode: 0755
  34. delegate_to: localhost
  35. delegate_facts: no
  36. run_once: true
  37. become: false
  38. tags:
  39. - localhost
  40. - name: prep_download | On localhost, check if passwordless root is possible
  41. command: "true"
  42. delegate_to: localhost
  43. run_once: true
  44. register: test_become
  45. changed_when: false
  46. ignore_errors: true
  47. become: true
  48. when:
  49. - download_localhost
  50. tags:
  51. - localhost
  52. - asserts
  53. - name: prep_download | On localhost, check if user has access to docker without using sudo
  54. shell: "{{ docker_bin_dir }}/docker images"
  55. delegate_to: localhost
  56. run_once: true
  57. register: test_docker
  58. changed_when: false
  59. ignore_errors: true
  60. become: false
  61. when:
  62. - download_localhost
  63. tags:
  64. - localhost
  65. - asserts
  66. - name: prep_download | Parse the outputs of the previous commands
  67. set_fact:
  68. user_in_docker_group: "{{ not test_docker.failed }}"
  69. user_can_become_root: "{{ not test_become.failed }}"
  70. when:
  71. - download_localhost
  72. tags:
  73. - localhost
  74. - asserts
  75. - name: prep_download | Check that local user is in group or can become root
  76. assert:
  77. that: "user_in_docker_group or user_can_become_root"
  78. msg: >-
  79. Error: User is not in docker group and cannot become root. When download_localhost is true, at least one of these two conditions must be met.
  80. when:
  81. - download_localhost
  82. tags:
  83. - localhost
  84. - asserts