teemtee / fmf

Flexible Metadata Format
GNU General Public License v2.0
22 stars 28 forks source link

A bunch of adjustments for the epel-7 build #120

Closed psss closed 3 years ago

psss commented 3 years ago

Enable the epel-7 target in packit. Build only the python2-fmf package for rhel-7. Resolve several Python 2 incompatibilities. Enable unit tests, correctly mark all web tests. Skip test which needs pytest.warns module. Skip selected tests when run as root. Adjust requires accordingly, add missing git-core.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging a486de00d2c4d0b386894e9bbefb31acbdda352d into 0799662013df2907ab0caf516b67022f6b57eb3d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 9af3b180c836f7a1bbafc9155f35fa591599d8b1 into 0799662013df2907ab0caf516b67022f6b57eb3d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 9dc19edd9c791ac3c84ee68b6639891fc040ec3f into 0799662013df2907ab0caf516b67022f6b57eb3d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging a11c9bf59a800dcda70bae3c11b4e846a951d2fa into 0799662013df2907ab0caf516b67022f6b57eb3d - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging d51be0e53b5e33617764d77770b9042a10a7a1d9 into 0799662013df2907ab0caf516b67022f6b57eb3d - view on LGTM.com

new alerts:

lukaszachy commented 3 years ago

/packit test

lukaszachy commented 3 years ago

/packit test

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 819807a0abe1a31c4b6b9516208c35909dbec9a6 into 0799662013df2907ab0caf516b67022f6b57eb3d - view on LGTM.com

new alerts:

lukaszachy commented 3 years ago

@psss I'm not sure how to disable "/plans/integration" for rhel-7 only. The rest passes however I made them to be skipped when they run as root or when pytest is too old respectively. I think we could make permission denied to be mocked, but I'd leave that for next time.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 1b3987fe8d6d7b23f32736d346a53e5caedcc83a into 0799662013df2907ab0caf516b67022f6b57eb3d - view on LGTM.com

new alerts:

psss commented 3 years ago

@lukaszachy, thanks a lot for working on this together!