Bug 2266310 - Review Request: crun-vm - An OCI Runtime that runs VM images
Summary: Review Request: crun-vm - An OCI Runtime that runs VM images
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lokesh Mandvekar
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/containers/crun-vm
Whiteboard:
Depends On: 2266303
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-02-27 14:47 UTC by Alberto Faria
Modified: 2024-03-19 13:26 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2024-03-19 13:26:18 UTC
Type: ---
Embargoed:
lsm5: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 7179821 to 7182693 (1.40 KB, patch)
2024-03-19 12:09 UTC, Fedora Review Service
no flags Details | Diff

Description Alberto Faria 2024-02-27 14:47:23 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/afaria/crun-vm/srpm-builds/07064857/crun-vm.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/afaria/crun-vm/srpm-builds/07064857/crun-vm-0.1.2-1.src.rpm
Description: An OCI Runtime that runs VM images
Fedora Account System Username: afaria

Comment 1 Fedora Review Service 2024-02-27 14:51:19 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7065051
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2266310-crun-vm/fedora-rawhide-x86_64/07065051-crun-vm/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 3 Alberto Faria 2024-03-18 21:05:08 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/afaria/crun-vm/srpm-builds/07179185/crun-vm.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/afaria/crun-vm/srpm-builds/07179185/crun-vm-0.1.3-1.src.rpm
Description: An OCI Runtime that runs VM images
Fedora Account System Username: afaria

Comment 4 Fedora Review Service 2024-03-18 23:55:50 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7179821
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2266310-crun-vm/fedora-rawhide-x86_64/07179821-crun-vm/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 5 Fedora Review Service 2024-03-18 23:55:57 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7179820
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2266310-crun-vm/fedora-rawhide-x86_64/07179820-crun-vm/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 6 Fabio Valentini 2024-03-19 11:19:44 UTC
Quick comments from a Rust packaging perspective:

1. Please replace "__cargo_is_lib() 0" (which is a private macro and was previously needed as a workaround) with "cargo_install_lib 0" (which is now the official, supported way to override this behaviour as of rust-packaging v26).

2. Bump cargo-rpm-macros dependency to >= 26 (and / or regenerate the spec with rust2rpm v26).

3. Please include the full output of "%cargo_license_summary" from the build.log as a spec comment above the "License" tag.

4. The license tag contains duplicates and can be simplified.

> (MIT OR Apache-2.0) AND Unicode-DFS-2016 AND Apache-2.0 AND (Apache-2.0 OR BSL-1.0) AND (Apache-2.0 OR MIT) AND GPL-2.0-or-later AND MIT AND (MIT OR Apache-2.0) AND MPL-2.0 AND MPL-2.0+ AND (Unlicense OR MIT)

Eliminating duplicates and sorting results in:

Apache-2.0 AND GPL-2.0-or-later AND MIT AND MPL-2.0 AND MPL.2.0+ AND Unicode-DFS-2016 AND (Apache-2.0 OR BSL-1.0) AND (Apache-2.0 OR MIT) AND (Unlicense OR MIT)

Looks good from Rust perspective otherwise.

Comment 7 Alberto Faria 2024-03-19 12:01:41 UTC
Thank you Fabio, I addressed your comments.

Spec URL: https://download.copr.fedorainfracloud.org/results/afaria/crun-vm/srpm-builds/07182639/crun-vm.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/afaria/crun-vm/srpm-builds/07182639/crun-vm-0.1.3-1.src.rpm
Description: An OCI Runtime that runs VM images
Fedora Account System Username: afaria

Comment 8 Fedora Review Service 2024-03-19 12:09:30 UTC
Created attachment 2022548 [details]
The .spec file difference from Copr build 7179821 to 7182693

Comment 9 Fedora Review Service 2024-03-19 12:09:33 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7182693
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2266310-crun-vm/fedora-rawhide-x86_64/07182693-crun-vm/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 10 Lokesh Mandvekar 2024-03-19 12:13:06 UTC
LGTM given Fabio's comments are addressed and we have a successful copr build.

Comment 11 Fedora Admin user for bugzilla script actions 2024-03-19 13:06:14 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/crun-vm

Comment 12 Fedora Update System 2024-03-19 13:23:49 UTC
FEDORA-2024-be71bfe8e2 (crun-vm-0.1.3-1.fc41) has been submitted as an update to Fedora 41.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-be71bfe8e2

Comment 13 Fedora Update System 2024-03-19 13:26:18 UTC
FEDORA-2024-be71bfe8e2 (crun-vm-0.1.3-1.fc41) has been pushed to the Fedora 41 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.