Skip to content

Commit

Permalink
tests: increase timeout for when exiting cockpit-storage
Browse files Browse the repository at this point in the history
This is needed because of an LVM2 bug which causes re-scanning to take
forever.

The anaconda fix for rawhide is merged and released:
rhinstaller/anaconda#6140
For fedora-42 we need a workaround in the tests as PR is still not merged:
rhinstaller/anaconda#6234.
  • Loading branch information
KKoukiou committed Mar 7, 2025
1 parent 53f6656 commit cf7d3ba
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions test/helpers/storage.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,8 +84,14 @@ def confirm_entering_cockpit_storage(self):

def return_to_installation(self, error=None):
self.browser.click("#cockpit-storage-integration-return-to-installation-button")
if error:
self.browser.wait_in_text("#cockpit-storage-integration-check-storage-dialog", error)
# FIXME: https://github.com/rhinstaller/anaconda/pull/6234
# On Fedora-42 re-scanning takes long when there are LVM devices
# This extra timeout can be removed once the above PR is merged
with self.browser.wait_timeout(90):
if error:
self.browser.wait_in_text("#cockpit-storage-integration-check-storage-dialog", error)
else:
self.browser.wait_not_present("#cockpit-storage-integration-check-storage-dialog")

def return_to_installation_confirm(self):
# FIXME: testBtrfsTopLevelVolume fails sometimes on CI without this workaround
Expand Down

0 comments on commit cf7d3ba

Please sign in to comment.