From caae760c01eeadd59a7ab51b7c8771a06a0ee8d1 Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Mon, 18 Jul 2022 16:29:25 -0400 Subject: [PATCH] Greatly raise `StartLimitBurst` for `rpm-ostreed.service` This is a hack to work around the lack of https://github.com/coreos/rpm-ostree/pull/3523/commits/0556152adb14a8e1cdf6c5d6f234aacbe8dd4e3f for RHEL 8.6 and below. A recent PR in the MCO https://github.com/openshift/machine-config-operator/pull/3243 tipped things over the edge and we now see failures a lot more often. --- manifest.yaml | 1 + .../lib/systemd/system/rpm-ostreed.service.d/startlimit.conf | 4 ++++ tests/kola/rpm-ostree/startlimit | 5 +++++ 3 files changed, 10 insertions(+) create mode 100644 overlay.d/15rhcos-rhel8-workarounds/usr/lib/systemd/system/rpm-ostreed.service.d/startlimit.conf create mode 100755 tests/kola/rpm-ostree/startlimit diff --git a/manifest.yaml b/manifest.yaml index fa78aa8e..60a5f56e 100644 --- a/manifest.yaml +++ b/manifest.yaml @@ -21,6 +21,7 @@ ostree-layers: - overlay/06gcp-routes - overlay/15rhcos-logrotate - overlay/15rhcos-tuned-bits + - overlay/15rhcos-rhel8-workarounds # TODO conditionalize on rhel8 - overlay/20platform-chrony - overlay/21dhcp-chrony - overlay/25rhcos-azure-udev diff --git a/overlay.d/15rhcos-rhel8-workarounds/usr/lib/systemd/system/rpm-ostreed.service.d/startlimit.conf b/overlay.d/15rhcos-rhel8-workarounds/usr/lib/systemd/system/rpm-ostreed.service.d/startlimit.conf new file mode 100644 index 00000000..843e5c4b --- /dev/null +++ b/overlay.d/15rhcos-rhel8-workarounds/usr/lib/systemd/system/rpm-ostreed.service.d/startlimit.conf @@ -0,0 +1,4 @@ +[Unit] +# Work around for lack of https://github.com/coreos/rpm-ostree/pull/3523/commits/0556152adb14a8e1cdf6c5d6f234aacbe8dd4e3f +# on older RHEL +StartLimitBurst=1000 diff --git a/tests/kola/rpm-ostree/startlimit b/tests/kola/rpm-ostree/startlimit new file mode 100755 index 00000000..2b0707d6 --- /dev/null +++ b/tests/kola/rpm-ostree/startlimit @@ -0,0 +1,5 @@ +#!/bin/bash +set -euo pipefail +# https://github.com/coreos/rpm-ostree/pull/3523/commits/0556152adb14a8e1cdf6c5d6f234aacbe8dd4e3f +for x in $(seq 10); do rpm-ostree status >/dev/null; done +echo ok