Bug 2084107

Summary:	Review Request: rust-vhost-user-backend - Framework to build vhost-user backend service daemon
Product:	[Fedora] Fedora	Reporter:	Sergio Lopez <slopezpa>
Component:	Package Review	Assignee:	Lokesh Mandvekar <lsm5>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	crobinso, decathorpe, fabiano, lsm5, package-review
Target Milestone:	---	Flags:	lsm5: fedora-review+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2022-05-16 11:37:16 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Sergio Lopez 2022-05-11 12:27:58 UTC

Spec URL: https://download.copr.fedorainfracloud.org/results/slp/rust-vhost-user-backend/fedora-rawhide-x86_64/04396024-rust-vhost-user-backend/rust-vhost-user-backend.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/slp/rust-vhost-user-backend/fedora-rawhide-x86_64/04396024-rust-vhost-user-backend/rust-vhost-user-backend-0.2.0-1.fc37.src.rpm
Description:
Framework to build vhost-user backend service daemon.

Fedora Account System Username: slp

Comment 1 Fabio Valentini 2022-05-11 21:13:14 UTC

Is there a reason why you're not packaging the latest version (0.3.0)?

Comment 2 Sergio Lopez 2022-05-12 06:04:56 UTC

(In reply to Fabio Valentini from comment #1)
> Is there a reason why you're not packaging the latest version (0.3.0)?

We're packaging rust-vhost-user-backend as a dependency for virtiofsd. Version 0.3.0 was released this week just after we cut version 1.2.0 of virtiofsd.

As updating a package in Fedora is way faster than requesting a new package, I thought of getting rust-vhost-user-backend 0.2.0 and virtiofsd 1.2.0 first, then update all depedencies in Fedora, cut version 1.2.1 of virtiofsd with a dependency bump in Cargo.toml, and update its package too. The whole process shouldn't take more than a couple weeks and it minimizes the chances of another mid-air collision (happened to us twice already).

Thanks,
Sergio.

Comment 3 Fabio Valentini 2022-05-12 15:40:23 UTC

Yup, that makes sense.

Comment 4 Lokesh Mandvekar 2022-05-13 12:54:19 UTC

Koji build sent by Sergio: https://koji.fedoraproject.org/koji/taskinfo?taskID=86955974

Comment 5 Lokesh Mandvekar 2022-05-13 13:15:41 UTC

- Spec Conforms to packaging guidelines (rust2rpm generated spec)
- license correct and valid
- only sources installed

Package approved.

Might wanna consider using rpm-autospec as it could make life a lot easier https://docs.pagure.org/Fedora-Infra.rpmautospec/index.html .

Comment 6 Gwyn Ciesla 2022-05-13 14:34:03 UTC

(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-vhost-user-backend

Comment 7 Sergio Lopez 2022-05-16 11:37:16 UTC

The package has hit the Rawhide compose. Thanks Lokesh and Gwyn!