Skip to content

Fix for diskless installation (bsc#1260378)#3364

Merged
teclator merged 6 commits intomasterfrom
fix_diskless_inst
Apr 8, 2026
Merged

Fix for diskless installation (bsc#1260378)#3364
teclator merged 6 commits intomasterfrom
fix_diskless_inst

Conversation

@teclator
Copy link
Copy Markdown
Contributor

@teclator teclator commented Apr 7, 2026

Problem

If the system has no devices when it starts (diskless system), then the storage proposal is not automatically calculated after activating new devices (zFCP, DASD, iSCSI).

Solution

  • Fix the condition to make a proposal when at least there is a product configuration as in case of no disk there is no proposal at all.

  • Apart of that we fixed an issue with the UI that caused the progress backdrop ("Refreshing data") to be shown when it actually finished because it also waits for proposal changes (related to web: refactor progress backdrop query tracking  #2958)

Testing

  • Tested manually
  • Adapted service unit tests

@teclator teclator requested a review from joseivanlopez April 7, 2026 10:42
Comment thread service/lib/agama/dbus/storage/manager.rb Outdated
Copy link
Copy Markdown
Contributor

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests failing and changelog entry missing

@teclator teclator force-pushed the fix_diskless_inst branch from 4fe0e5d to 460d306 Compare April 8, 2026 09:59
@teclator teclator requested a review from dgdavid April 8, 2026 09:59
Comment on lines +4 to +6
- Fix for diskless installation making a proposal when
there is at least the product configuration notifying the
system and proposal state (bsc#1260378).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NP: Too coupled to the implementation detail, IMHO.

What about just saying in the line of "Always update system and proposal state to avoid progress stalling, even when status hasn’t changed after an action."?

In any case, feel free to keep it as it is.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about just saying in the line of "Always update system and proposal state to avoid progress stalling, even when status hasn’t changed after an action."?

Reading it twice it is kind of coupled as well, and probably "update" is not the right word, but it was just an example for trying to make the changelog entry a bit more for humans. Details are in the code itself and in the bug report. Better to express what it fix/prevents.

Copy link
Copy Markdown
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@teclator teclator merged commit 3791884 into master Apr 8, 2026
19 checks passed
@teclator teclator deleted the fix_diskless_inst branch April 8, 2026 12:16
@imobachgs imobachgs mentioned this pull request Apr 14, 2026
imobachgs added a commit that referenced this pull request Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants