udisks icon indicating copy to clipboard operation
udisks copied to clipboard

core: Add Encrypted.Convert method

Open xyakimo1 opened this issue 10 months ago • 11 comments

xyakimo1 avatar Apr 15 '24 16:04 xyakimo1

Can one of the admins verify this patch?

StorageGhoul avatar Apr 15 '24 16:04 StorageGhoul

Cockpit tests failed for commit 433ed08644f82f323816738d6b260114c435340e. @martinpitt, @jelly, @mvollmer please check.

OK, that's not cockpit's bug :sweat_smile:

udiskslinuxencrypted.c:1301:9: error: implicit declaration of function 'bd_crypto_luks_convert'; did you mean 'bd_crypto_luks_close'? [-Werror=implicit-function-declaration]
 1301 |   if (! bd_crypto_luks_convert (udisks_block_get_device (block),
      |         ^~~~~~~~~~~~~~~~~~~~~~
      |         bd_crypto_luks_close
udiskslinuxencrypted.c:1301:9: warning: nested extern declaration of 'bd_crypto_luks_convert' [-Wnested-externs]

martinpitt avatar Apr 16 '24 05:04 martinpitt

OK, that's not cockpit's bug 😅

udiskslinuxencrypted.c:1301:9: error: implicit declaration of function 'bd_crypto_luks_convert'; did you mean 'bd_crypto_luks_close'? [-Werror=implicit-function-declaration]
 1301 |   if (! bd_crypto_luks_convert (udisks_block_get_device (block),
      |         ^~~~~~~~~~~~~~~~~~~~~~
      |         bd_crypto_luks_close
udiskslinuxencrypted.c:1301:9: warning: nested extern declaration of 'bd_crypto_luks_convert' [-Wnested-externs]

We need libblockdev daily builds for this, the version with bd_crypto_luks_convert is not yet released.

vojtechtrefny avatar Apr 16 '24 08:04 vojtechtrefny

/packit build

vojtechtrefny avatar Apr 16 '24 13:04 vojtechtrefny

Jenkins, ok to test.

vojtechtrefny avatar Apr 16 '24 13:04 vojtechtrefny

Jenkins, test this please.

vojtechtrefny avatar Apr 18 '24 10:04 vojtechtrefny

/packit build

vojtechtrefny avatar Apr 18 '24 10:04 vojtechtrefny

We need libblockdev daily builds for this

Aren't these supposed to be in https://copr.fedorainfracloud.org/coprs/g/storage/udisks-daily/packages/ ? The last libblockdev build is from 4 days ago, much newer than this PR. What happened here?

martinpitt avatar Apr 22 '24 11:04 martinpitt

/packit test

vojtechtrefny avatar Apr 22 '24 12:04 vojtechtrefny

We need libblockdev daily builds for this

Aren't these supposed to be in https://copr.fedorainfracloud.org/coprs/g/storage/udisks-daily/packages/ ? The last libblockdev build is from 4 days ago, much newer than this PR. What happened here?

There was an issue with the libblockdev daily builds, the version in the udisks-daily repo was behind the version in Fedora for some time. That should be fixed now. (I originally thought that the revdeps test doesn't use the daily build repo and that was the reason for the failures.)

vojtechtrefny avatar Apr 22 '24 12:04 vojtechtrefny