-
Notifications
You must be signed in to change notification settings - Fork 169
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding retry logic on yum commands to avoid resource locking #3511
Changes from all commits
f2ae7c1
55c8b9d
220a14d
99838ea
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,21 @@ | ||
#Adding retry logic to yum commands in order to avoid stalling out on resource locks | ||
echo "running RHUI fix" | ||
yum update -y --disablerepo='*' --enablerepo='rhui-microsoft-azure*' | ||
|
||
yum -y -x WALinuxAgent -x WALinuxAgent-udev update | ||
yum -y install podman-docker | ||
for attempt in {1..5}; do | ||
yum update -y --disablerepo='*' --enablerepo='rhui-microsoft-azure*' && break | ||
if [[ ${attempt} -lt 5 ]]; then sleep 10; else exit 1; fi | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can we break this up into multiple lines? |
||
done | ||
|
||
echo "running yum update" | ||
for attempt in {1..5}; do | ||
yum -y -x WALinuxAgent -x WALinuxAgent-udev update --allowerasing && break | ||
if [[ ${attempt} -lt 5 ]]; then sleep 10; else exit 1; fi | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same as above for the remainder of if statements added |
||
done | ||
|
||
echo "installing podman-docker" | ||
for attempt in {1..5}; do | ||
yum -y install podman-docker && break | ||
if [[ ${attempt} -lt 5 ]]; then sleep 10; else exit 1; fi | ||
done | ||
|
||
firewall-cmd --add-port=443/tcp --permanent | ||
|
||
|
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.
I'm also a fan of breaking up script arguments into multiple lines when there are multiple.
Example: