Restarting the docker service causes kubelet to restart

[Bart Wensley]

Brief Description
-----------------
Originally, docker was used as the container runtime. The kubelet service and puppet dependencies were set up so kubelet ran after docker and was restarted when docker was restarted. However, we have now changed the container runtime to containerd, so kubelet should no longer be dependant on docker for initialization or restarts.

Severity
--------
Major: Restarts of the docker service cause kubelet restarts and unnecessarily impact kubernetes availability.

Steps to Reproduce
------------------
Restart the docker service - this can happen if the service fails or if configuration changes (e.g. docker proxy configuration change).

Expected Behavior
------------------
Restarts of the docker service should not impact kubelet.

Actual Behavior
----------------
Restarts of the docker service cause kubelet to be restarted.

Reproducibility
---------------
Reproducible

System Configuration
--------------------
All

Branch/Pull Time/Commit
-----------------------
starlingx master - since the switch to containerd

Last Pass
---------
prior to the introduction of containerd

Timestamp/Logs
--------------
None

Test Activity
-------------
Evaluation

Workaround
----------
None

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/736841

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/736841
Committed: https://git.openstack.org/cgit/starlingx/integ/commit/?id=88f29d0b98c943b13bf52e6c47b2025b49a8300a
Submitter: Zuul
Branch: master

commit 88f29d0b98c943b13bf52e6c47b2025b49a8300a
Author: Chris Friesen <email address hidden>
Date: Sun Jun 14 19:16:40 2020 -0400

    remove docker dependency from kubelet.service

    It's incorrect for kubelet.service to depend on docker as
    we no longer use docker as our container runtime. Instead,
    just wait for networking to be up.

    The %patch directive is a little odd, as we're patching
    the "contrib" directory which is actually unpacked from
    a separate tarball.

    Change-Id: I16a1aed98b15ac3d1fe3870ce01cc9237bdb3426
    Closes-Bug: 1882604
    Signed-off-by: Chris Friesen <email address hidden>