Skip to content

Commit

Permalink
WIP: RHEL/Fedora now defaults to setting crashkernel
Browse files Browse the repository at this point in the history
  • Loading branch information
jelly committed Oct 12, 2023
1 parent 7f8e92f commit 5ed8755
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions test/verify/check-kdump
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,11 @@ from lib.constants import TEST_OS_DEFAULT
class KdumpHelpers(testlib.MachineCase):
def enableKdump(self):
# all current Fedora/CentOS/RHEL images use BootLoaderSpec
self.machine.execute("grubby --args=crashkernel=256M --update-kernel=ALL")
if self.machine.image == "fedora-38":
self.machine.execute("grubby --args=crashkernel=192M --update-kernel=ALL")
self.machine.reboot()

self.machine.execute("mkdir -p /var/crash")
self.machine.reboot()
self.allow_restart_journal_messages()
self.machine.start_cockpit()
self.browser.switch_to_top()
Expand Down Expand Up @@ -68,7 +70,8 @@ class TestKdump(KdumpHelpers):
m = self.machine

b.wait_timeout(120)
m.execute("systemctl enable kdump; systemctl start kdump || true")
if m.image.startswith("fedora"):
m.execute("systemctl enable kdump; systemctl start kdump || true")

self.login_and_go("/kdump")

Expand Down Expand Up @@ -148,7 +151,7 @@ class TestKdump(KdumpHelpers):
self.crashKernel("copied through SSH to root@localhost:/var/crash as vmcore", cancel=True)

# we should have the amount of memory reserved that we indicated
b.wait_in_text("#app", "256 MiB")
b.wait_in_text("#app", "192 MiB")
# service should start up properly and the button should be on
b.wait_in_text("#app", "Service is running")
assertActive(active=True)
Expand Down

0 comments on commit 5ed8755

Please sign in to comment.