Conversation
| # | ||
| # @todo Perform a real D-Bus call. | ||
| def AvailableAll(_package_names) | ||
| true |
There was a problem hiding this comment.
Can we always assume true?
There was a problem hiding this comment.
No. We can't. It's the same TODO we have for Available in the line above.
There was a problem hiding this comment.
But not that you mention it, I will search for the corresponding Trello card/Github issue/Whatever and will reference it here.
There was a problem hiding this comment.
Everything originated here (it says "We need to add support for Package#Available and Package#Installed to D-Bus):
But I don't see any follow-up card or issue to track it. So I will create one.
There was a problem hiding this comment.
Done. Now we have https://trello.com/c/oRcZ5P5k/251-implement-missing-software-methods
joseivanlopez
left a comment
joseivanlopez
left a comment
There was a problem hiding this comment.
Please, add a changelog entry. The rest LGTM.
3 participants
Problem
Encryption with TPM-unlocking was implemented (in a pretty hacky way) directly at Agama.
Solution
This PR goes together with yast/yast-storage-ng#1363
Now
Y2Storagetakes care of most things.As a bonus, the storage finisher is now called. We overlooked that in the past and it takes care of some relevant things like copying the multipath configuration to the target system.