Re: [make-initrd] [for-master] Fixes

[Leonid Krivoshein <klark.devel@gmail.com>]

On 2/8/24 19:27, Leonid Krivoshein wrote:
>
> On 2/8/24 18:38, Alexey Gladkov wrote:
>> On Thu, Feb 08, 2024 at 05:16:16PM +0300, Leonid Krivoshein wrote:
>>> Алексей, привет!
>>>
>>>
>>> On 1/31/24 21:18, Alexey Gladkov wrote:
>>>> Привет!
>>>>
>>>> Я вроде поправил все баги, которые нашли в последнем релизе. Я буду
>>>> признателен, если кто-нибудь посмотрит на них. Я не могу 
>>>> протестировать
>>>> multipath и plymouth, кроме как проверить, что в образ попадают файлы.
>>> Multipath остался нерабочий. И файлы в образ не попадают. Разбираемся
>>> уже неделю с этим...
>> Было бы неплохо, если бы вы хоть что-то про это сказали. Я сделал релиз,
>> потому что посчитал, что все проигнорировали моё письмо.
>
> Я взялся этим заниматься, но приболел, и сходу не получилось. Да и 
> чего писать, пока не было результатов. Всё затянулось, только сегодня 
> сам на стенд смог попасть.
>
>
>> Теперь уже проверьте релиз. Скорее всего вы не бэкпортировали все
>> необходимые коммиты.
>>
>>>> https://github.com/osboot/make-initrd/tree/for-master
>>>>
>>>> Alexey Gladkov (18):
>>>>         feature/overlayfs: New feature to take lowerdirs into account
>>>>         Add loop kernel module if needed
>>>>         Add SPDX-License-Identifier
>>> Сначала у меня не накладывалось остальное без этого коммита, я проверял
>>> только multipath, и только на p10, другой возможности не было. Пришлось
>>> этот коммит немного урезать.
>>>
>>>
>>>>         github: Update dependencies
>>>>         feature/pipeline: Add dependency on additional utilities
>>>>         feature/pipeline: Register ping= cmdline parameter
>>>>         sort-services: Ignore services without LSB header without 
>>>> error
>>>>         feature/lkrg: Fix LSM header for service
>>>>         Add service files validation
>>>>         Make System.map optional
>>>>         Remove unused helper udev-version
>>>>         Provide to features a list of library directories
>>>>         find-udev-rule-externals: handle the situation when there 
>>>> is a variable at the beginning of the path
>>>>         find-udev-rule-externals: If the argument begins with - or 
>>>> %, then most likely we could not find the command
>>>>         find-udev-rule-externals: Add more verbosity and do not hit 
>>>> the error
>>>>         feature/multipath: Add plugins directory
>>>>         feature/multipath: Logically separate the multipath and 
>>>> sg3_utils files
>>> И ещё эти два патча использовал. Но там в наборе говорится о трёх 
>>> патчах.
>>   a830801d ("Provide to features a list of library directories")
>>
>> Вы не забыли вот этот патч ? Без него не будет ничего работать.
>
> Да, его я точно упустил. Т.к. проверял точечно только коммиты 
> multipath, причём на p10.
> И даже `git log --oneline 2.42.0.. -- make-initrd/feature/multipath` 
> не помог его обнаружить.
>

Да, оказалось, что именно его не хватало: проверил на стенде с task 
#340125, теперь всё ОК.


>
>>> Пробовали и с последней на тот момент версией из Сизифа на p10 (Task
>>> #339757), и с последней версией в бранче p10 (Task #339804). 
>>> Диагностику
>>> собрал, думаю лучше с ней уйти в личную переписку. Может, что-то не так
>>> делаем?
>> Скорее всего вы взяли не все коммиты. Попробуйте 2.43.0 из сизифа.
>>
>
> Спасибо! Опробую, отпишусь.
> С предыдущим вариантом (#339757) версия 2.42.0 на p10 даже не смогла 
> создать initrd.
>

А с "полным букетом" изменений shaba@ что-то пытался, но там вроде как 
ломается VROC на p10.



-- 
WBR, Leonid Krivoshein.