verified, this works for me

Il giorno gio 28 ott 2021 alle ore 13:41 Harel Braha <hbr...@redhat.com> ha
scritto:

> Hi,
> I have posted a patch with the suggested workaround [1].
> If need anything else,  please let me know.
>
> [1] https://gerrit.ovirt.org/c/vdsm/+/117372
>
> Harel
>
>
> On Thu, Oct 28, 2021 at 2:08 PM Marcin Sobczyk <msobc...@redhat.com>
> wrote:
>
>> Hi,
>>
>> On 10/28/21 11:59, Sandro Bonazzola wrote:
>> > hi,
>> > I'm trying to enable copr builds for vdsm (
>> > https://gerrit.ovirt.org/c/vdsm/+/117368
>> > <https://gerrit.ovirt.org/c/vdsm/+/117368> )
>> >
>> > And it's currently failing to rebuild src.rpm (generated on Fedora 34)
>> > for el8 with the following error:
>> > (
>> https://download.copr.fedorainfracloud.org/results/ovirt/ovirt-master-snapshot/centos-stream-8-x86_64/02912480-vdsm/build.log.gz
>> > <
>> https://download.copr.fedorainfracloud.org/results/ovirt/ovirt-master-snapshot/centos-stream-8-x86_64/02912480-vdsm/build.log.gz>
>>
>> > )
>> >
>> > make[2]: Entering directory
>> '/builddir/build/BUILD/vdsm-4.50.0.1/lib/vdsm'
>> > Making all in api
>> > make[3]: Entering directory
>> '/builddir/build/BUILD/vdsm-4.50.0.1/lib/vdsm/api'
>> >    Generate vdsm-api.html
>> > chmod u+w .
>> > PYTHONPATH=./../../:./../../vdsm \
>> >       ./schema_to_html.py vdsm-api ./vdsm-api.html
>> > Traceback (most recent call last):
>> >    File "./schema_to_html.py", line 250, in <module>
>> >      main()
>> >    File "./schema_to_html.py", line 245, in main
>> >      api_schema = vdsmapi.Schema((schema_type,), strict_mode=False)
>> >    File "/builddir/build/BUILD/vdsm-4.50.0.1/lib/vdsm/api/vdsmapi.py",
>> line 145, in __init__
>> >      loaded_schema = pickle.loads(f.read())
>> > ValueError: unsupported pickle protocol: 5
>> > make[3]: *** [Makefile:697: vdsm-api.html] Error 1
>> > make[3]: Leaving directory
>> '/builddir/build/BUILD/vdsm-4.50.0.1/lib/vdsm/api'
>> > make[2]: *** [Makefile:644: all-recursive] Error 1
>> > make[2]: Leaving directory
>> '/builddir/build/BUILD/vdsm-4.50.0.1/lib/vdsm'
>> > make[1]: *** [Makefile:466: all-recursive] Error 1
>> > make[1]: Leaving directory '/builddir/build/BUILD/vdsm-4.50.0.1/lib'
>> > make: *** [Makefile:539: all-recursive] Error 1
>> > error: Bad exit status from /var/tmp/rpm-tmp.nDfLzv (%build)
>> >
>> > Sounds like the make dist process ran on Fedora 34 brings a source file
>> used at build time on el8 in a cpickle format which is not backward
>> compatible.
>> >
>> > Seems to be a bug, the cpickle shouldn't be included in the tar.gz, it
>> should be generated at build time.
>> >
>> > Comments?
>>
>> The pickles were introduced here:
>>
>> https://gerrit.ovirt.org/c/vdsm/+/94196
>>
>> AFAIR they were added to the vdsm-api package because previously we were
>> generating them in during rpm installation in %post section which caused
>> issues with oVirt Node.
>>
>> I'm not sure how easy will it be to not have them in the srpm/tar.gz but
>> have them in the rpm. As a quick workaround I can change the pickle
>> protocol version that we use [1] to '4', which should work for both
>> Fedora 34 and el8.
>>
>> Regards, Marcin
>>
>> [1]
>>
>> https://github.com/oVirt/vdsm/blob/1969ab99c371ad498ea8693671cec60e2d0d49c2/lib/vdsm/api/schema_to_pickle.py#L46
>>
>>
>> >
>> > --
>> >
>> > Sandro Bonazzola
>> >
>> > MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV
>> >
>> > Red Hat EMEA <https://www.redhat.com/>
>> >
>> > sbona...@redhat.com <mailto:sbona...@redhat.com>
>> >
>> > <https://www.redhat.com/>
>> >
>> > **
>> > *Red Hat respects your work life balance. Therefore there is no need to
>> > answer this email out of your office hours.
>> > *
>> > *
>> >
>> > *
>>
>>

-- 

Sandro Bonazzola

MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV

Red Hat EMEA <https://www.redhat.com/>

sbona...@redhat.com
<https://www.redhat.com/>

*Red Hat respects your work life balance. Therefore there is no need to
answer this email out of your office hours.*
_______________________________________________
Devel mailing list -- devel@ovirt.org
To unsubscribe send an email to devel-le...@ovirt.org
Privacy Statement: https://www.ovirt.org/privacy-policy.html
oVirt Code of Conduct: 
https://www.ovirt.org/community/about/community-guidelines/
List Archives: 
https://lists.ovirt.org/archives/list/devel@ovirt.org/message/QFSUBH62ZB4ROYP26XU2Q2RARHOO7DDA/

Reply via email to