fix: prevent race condition with docker/moby install #447
Conversation
@@ -15,7 +15,7 @@ removeEtcd() { | |||
} | |||
|
|||
removeMoby() { | |||
sudo apt-get purge moby-engine moby-cli | |||
sudo apt-get purge -y moby-engine moby-cli |
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.
forgot to commit this in #407, needed to avoid force purge without prompt
@@ -165,6 +164,10 @@ echo `date`,`hostname`, endcustomscript>>/opt/m | |||
mkdir -p /opt/azure/containers && touch /opt/azure/containers/provision.complete | |||
ps auxfww > /opt/azure/provision-ps.log & | |||
|
|||
if ! $FULL_INSTALL_REQUIRED; then | |||
cleanUpContainerImages |
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.
moved this down to after container runtime is installed to avoid cleanup colliding with docker uninstall (if new version is needed)
lgtm |
Codecov Report
@@ Coverage Diff @@
## master #447 +/- ##
=======================================
Coverage 53.42% 53.42%
=======================================
Files 95 95
Lines 14385 14385
=======================================
Hits 7685 7685
Misses 6037 6037
Partials 663 663 |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CecileRobertMichon, mboersma The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
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.
lgtm pending e2e
Reason for Change:
Cleaning up docker images in the background before installing docker can cause a race condition with docker install if we need to change the version of moby installed.
Issue Fixed:
Requirements:
Notes: