[make-initrd] [for-master] Fixes

[make-initrd] [for-master] Fixes

[Alexey Gladkov]

Привет!

Я вроде поправил все баги, которые нашли в последнем релизе. Я буду
признателен, если кто-нибудь посмотрит на них. Я не могу протестировать
multipath и plymouth, кроме как проверить, что в образ попадают файлы.

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
      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
      feature/plymouth: Use the library list to find the plymouth directory

Alexey Shabalin (1):
      Feature guestfs: Add gost checksum utilities

-- 
Rgrds, legion

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

[Leonid Krivoshein]

Алексей, привет!


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

И ещё эти два патча использовал. Но там в наборе говорится о трёх патчах.

Пробовали и с последней на тот момент версией из Сизифа на p10 (Task 
#339757), и с последней версией в бранче p10 (Task #339804). Диагностику 
собрал, думаю лучше с ней уйти в личную переписку. Может, что-то не так 
делаем?


>        feature/plymouth: Use the library list to find the plymouth directory
>
> Alexey Shabalin (1):
>        Feature guestfs: Add gost checksum utilities
>


-- 
WBR, Leonid Krivoshein.

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

[Alexey Gladkov]

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")

Вы не забыли вот этот патч ? Без него не будет ничего работать.

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

Скорее всего вы взяли не все коммиты. Попробуйте 2.43.0 из сизифа.

-- 
Rgrds, legion

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

[Leonid Krivoshein]

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` не 
помог его обнаружить.


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

Спасибо! Опробую, отпишусь.
С предыдущим вариантом (#339757) версия 2.42.0 на p10 даже не смогла 
создать initrd.


-- 
WBR, Leonid Krivoshein.

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

[Leonid Krivoshein]

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.

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

[Alexey Gladkov]

On Thu, Feb 08, 2024 at 08:40:07PM +0300, Leonid Krivoshein wrote:
> >>> Пробовали и с последней на тот момент версией из Сизифа на p10 (Task
> >>> #339757), и с последней версией в бранче p10 (Task #339804). 
> >>> Диагностику
> >>> собрал, думаю лучше с ней уйти в личную переписку. Может, что-то не так
> >>> делаем?
> >> Скорее всего вы взяли не все коммиты. Попробуйте 2.43.0 из сизифа.
> >>
> >
> > Спасибо! Опробую, отпишусь.
> > С предыдущим вариантом (#339757) версия 2.42.0 на p10 даже не смогла 
> > создать initrd.
> >
> 
> А с "полным букетом" изменений shaba@ что-то пытался, но там вроде как 
> ломается VROC на p10.

Я не очень понял. Что-то не работает ?

-- 
Rgrds, legion

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

[Leonid Krivoshein]

Привет!


On 2/9/24 13:37, Alexey Gladkov wrote:
> On Thu, Feb 08, 2024 at 08:40:07PM +0300, Leonid Krivoshein wrote:
>>>>> Пробовали и с последней на тот момент версией из Сизифа на p10 (Task
>>>>> #339757), и с последней версией в бранче p10 (Task #339804).
>>>>> Диагностику
>>>>> собрал, думаю лучше с ней уйти в личную переписку. Может, что-то не так
>>>>> делаем?
>>>> Скорее всего вы взяли не все коммиты. Попробуйте 2.43.0 из сизифа.
>>>>
>>> Спасибо! Опробую, отпишусь.
>>> С предыдущим вариантом (#339757) версия 2.42.0 на p10 даже не смогла
>>> создать initrd.
>>>
>> А с "полным букетом" изменений shaba@ что-то пытался, но там вроде как
>> ломается VROC на p10.
> Я не очень понял. Что-то не работает ?

Деталей не знаю, только то, что выше со слов shaba@ написал. Могу только 
сказать, что #340125 чинит multipath в p10 и c10, я проверял только 
multipath, т.к. стенд с ним ещё добыть надо.


-- 
WBR, Leonid Krivoshein.

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

[Alexey Gladkov]

On Fri, Feb 09, 2024 at 04:38:33PM +0300, Leonid Krivoshein wrote:
> Привет!
> 
> 
> On 2/9/24 13:37, Alexey Gladkov wrote:
> > On Thu, Feb 08, 2024 at 08:40:07PM +0300, Leonid Krivoshein wrote:
> >>>>> Пробовали и с последней на тот момент версией из Сизифа на p10 (Task
> >>>>> #339757), и с последней версией в бранче p10 (Task #339804).
> >>>>> Диагностику
> >>>>> собрал, думаю лучше с ней уйти в личную переписку. Может, что-то не так
> >>>>> делаем?
> >>>> Скорее всего вы взяли не все коммиты. Попробуйте 2.43.0 из сизифа.
> >>>>
> >>> Спасибо! Опробую, отпишусь.
> >>> С предыдущим вариантом (#339757) версия 2.42.0 на p10 даже не смогла
> >>> создать initrd.
> >>>
> >> А с "полным букетом" изменений shaba@ что-то пытался, но там вроде как
> >> ломается VROC на p10.
> > Я не очень понял. Что-то не работает ?
> 
> Деталей не знаю, только то, что выше со слов shaba@ написал. Могу только 
> сказать, что #340125 чинит multipath в p10 и c10, я проверял только 
> multipath, т.к. стенд с ним ещё добыть надо.

Я уже общаюсь с ним и делаем фикс.

-- 
Rgrds, legion

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

[Alexey Gladkov]

On Fri, Feb 09, 2024 at 01:57:19PM +0000, Alexey Gladkov wrote:
> On Fri, Feb 09, 2024 at 04:38:33PM +0300, Leonid Krivoshein wrote:
> > Привет!
> > 
> > 
> > On 2/9/24 13:37, Alexey Gladkov wrote:
> > > On Thu, Feb 08, 2024 at 08:40:07PM +0300, Leonid Krivoshein wrote:
> > >>>>> Пробовали и с последней на тот момент версией из Сизифа на p10 (Task
> > >>>>> #339757), и с последней версией в бранче p10 (Task #339804).
> > >>>>> Диагностику
> > >>>>> собрал, думаю лучше с ней уйти в личную переписку. Может, что-то не так
> > >>>>> делаем?
> > >>>> Скорее всего вы взяли не все коммиты. Попробуйте 2.43.0 из сизифа.
> > >>>>
> > >>> Спасибо! Опробую, отпишусь.
> > >>> С предыдущим вариантом (#339757) версия 2.42.0 на p10 даже не смогла
> > >>> создать initrd.
> > >>>
> > >> А с "полным букетом" изменений shaba@ что-то пытался, но там вроде как
> > >> ломается VROC на p10.
> > > Я не очень понял. Что-то не работает ?
> > 
> > Деталей не знаю, только то, что выше со слов shaba@ написал. Могу только 
> > сказать, что #340125 чинит multipath в p10 и c10, я проверял только 
> > multipath, т.к. стенд с ним ещё добыть надо.
> 
> Я уже общаюсь с ним и делаем фикс.

Поправочка. И человек тот же, и про make-initrd, но shaba@ про другое
сообщил.

-- 
Rgrds, legion