Bug 1018540

Summary: Review Request: golang-github-goraft-raft - A Go implementation of the Raft distributed consensus protocol
Product: [Fedora] Fedora Reporter: Lokesh Mandvekar <lsm5>
Component: Package ReviewAssignee: Matthew Miller <mattdm>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jkeck, lacypret, lemenkov, lsm5, mattdm, notting, package-review, tdawson, vbatts
Target Milestone: ---Keywords: Reopened
Target Release: ---Flags: mattdm: fedora-review+
gwync: fedora-cvs+
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: golang-github-goraft-raft-0-0.3.gite138889.fc20 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-10-29 03:48:25 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:
Bug Depends On: 1018057    
Bug Blocks: 1018479    

Description Lokesh Mandvekar 2013-10-13 03:27:19 UTC
Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-github-goraft-raft.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-github-goraft-raft-0-0.1.gite138889.fc21.src.rpm

Description: A Go implementation of the Raft distributed consensus protocol
Fedora Account System Username: lsm5

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6054102

$ rpmlint SPECS/golang-github-goraft-raft.spec SRPMS/golang-github-goraft-raft-0-0.1.gite138889.fc21.src.rpm RPMS/noarch/golang-github-goraft-raft-devel-0-0.1.gite138889.fc21.noarch.rpm
SPECS/golang-github-goraft-raft.spec:35: W: setup-not-quiet
golang-github-goraft-raft.src:35: W: setup-not-quiet
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

Comment 1 Lokesh Mandvekar 2013-10-13 03:28:30 UTC
packaging the upstream instead of the coreos fork

Comment 2 Matthew Miller 2013-10-14 15:17:57 UTC
Should have

BuildRequires: golang-googlecode-goprotobuf-devel

Comment 3 Matthew Miller 2013-10-14 16:41:29 UTC
(Or the equivalent  golang("[import path]") virtual provides.)

Comment 4 Matthew Miller 2013-10-14 17:34:55 UTC
Actually should be Requires, not BuildRequires.

Comment 5 Matthew Miller 2013-10-15 20:26:20 UTC
So, the devel package should have

Requires: golang("code.google.com/p/goprotobuf/proto")



Eventually, we'll make RPM generate these automatically.

Comment 6 Lokesh Mandvekar 2013-10-16 01:29:48 UTC
%changelog
* Tue Oct 15 2013 Lokesh Mandvekar <lsm5>  0-0.2.gite138889
- requires goprotobuf

Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-github-goraft-raft.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-github-goraft-raft-0-0.2.gite138889.fc21.src.rpm

Comment 7 Matthew Miller 2013-10-16 01:51:02 UTC
Review passed, with the caveat that it may need to be updated as the Go packaging guidelines progress beyond draft form.

Also, it wouldn't hurt to have a more descriptive description. But not a big deal for the review.

%description
This is a Go implementation of the Raft distributed consensus protocol.
Raft is a protocol by which a cluster of nodes can maintain a replicated state machine. The state machine is kept in sync through the use of a replicated log.

Comment 8 Lokesh Mandvekar 2013-10-18 19:47:00 UTC
New Package SCM Request
=======================
Package Name: golang-github-goraft-raft
Short Description: A Go implementation of the Raft distributed consensus protocol
Owners: lsm5
Branches: f19 f20 el6
InitialCC:

Comment 9 Gwyn Ciesla 2013-10-19 21:15:00 UTC
Git done (by process-git-requests).

Comment 10 Fedora Update System 2013-10-20 05:20:55 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/golang-github-goraft-raft-0-0.3.gite138889.fc20

Comment 11 Fedora Update System 2013-10-20 05:26:03 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/golang-github-goraft-raft-0-0.3.gite138889.fc19

Comment 12 Fedora Update System 2013-10-20 05:31:10 UTC
golang-github-goraft-raft-0-0.3.gite138889.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/golang-github-goraft-raft-0-0.3.gite138889.el6

Comment 13 Fedora Update System 2013-10-20 16:19:37 UTC
golang-github-goraft-raft-0-0.3.gite138889.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 14 Peter Lemenkov 2013-10-23 11:07:23 UTC
Package Change Request
======================
Package Name: golang-github-goraft-raft
InitialCC: golang-sig

Comment 15 Gwyn Ciesla 2013-10-23 14:12:47 UTC
Done.

Comment 16 Fedora Update System 2013-10-29 03:48:25 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc19 has been pushed to the Fedora 19 stable repository.

Comment 17 Fedora Update System 2013-11-04 17:35:31 UTC
golang-github-goraft-raft-0-0.3.gite138889.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 18 Fedora Update System 2013-11-10 07:37:49 UTC
golang-github-goraft-raft-0-0.3.gite138889.fc20 has been pushed to the Fedora 20 stable repository.

Comment 19 Troy Dawson 2014-07-22 20:17:31 UTC
Package Change Request
======================
Package Name: golang-github-goraft-raft
New Branches: epel7
Owners: tdawson maxamillion

Comment 20 Gwyn Ciesla 2014-07-23 10:05:30 UTC
Git done (by process-git-requests).