Bug 1086378 - Review Request: geard - CLI client for installing Docker images as containers
Summary: Review Request: geard - CLI client for installing Docker images as containers
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Vincent Batts
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1023769 1085019 1085030 1085840 1085873 1085881 1085890
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-04-10 17:50 UTC by Lokesh Mandvekar
Modified: 2014-07-01 23:00 UTC (History)
4 users (show)

Fixed In Version: geard-0-0.3.gitb31df16.fc20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-04-25 04:25:45 UTC
Type: ---
Embargoed:
walters: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Lokesh Mandvekar 2014-04-10 17:50:10 UTC
Spec URL: http://lsm5.fedorapeople.org/geard/geard.spec
SRPM URL: http://lsm5.fedorapeople.org/geard/SRPMS/geard-0-0.1.fc21.src.rpm

Description: 
geard is an opinionated tool for installing Docker images as containers onto a systemd-enabled Linux operating system (systemd 207 or newer).

Fedora Account System Username: lsm5

Comment 1 Lokesh Mandvekar 2014-04-11 21:15:34 UTC
just fyi, all deps have been submitted as buildroot overrides

Comment 2 Colin Walters 2014-04-11 22:22:44 UTC
There's some bits in this spec file that I'm not sure are necessary, e.g.:

%global __os_install_post %{_rpmconfigdir}/brp-compress

# btrfs not available for rhel yet


As for:

%description
%{summary}

Now for a lot of shared libraries I think this is fine, but this package probably merits a bit of an attempt at a description =)

Just take the upstream "A client and daemon for installing and linking Docker containers into systemd across hosts" ?

Can also just drop:

%if 0%{?rhel}
#%patch0 -p1 -b ignore-btrfs-for-rhel
#%patch90 -p1 -b docker-0.9-el6-lxc
%endif
#%patch1 -p1 -b upstream-patched-archive-tar


And drop:

%pre

%post

%preun

%postun

Theoretically we should use %systemd_postun and such right?  


<lsm5

Missing a trailing >

Comment 3 Lokesh Mandvekar 2014-04-12 20:52:15 UTC
aah yup, i copied over the docker-io spec file so that's why most of the mess, cleaned up now:

Spec URL: http://lsm5.fedorapeople.org/geard/geard.spec
SRPM URL: http://lsm5.fedorapeople.org/geard/SRPMS/geard-0-0.2.gitb31df16.fc21.src.rpm

Comment 4 Colin Walters 2014-04-12 21:03:06 UTC
* This bit looks unused:

%if 0%{?fedora} >= 18 || 0%{?rhel} >= 7
%bcond_without  systemd
%endif

* And this comment doesn't apply:

# btrfs not available for rhel yet

* As you mentioned on IRC the %description should be filled in.

Other than that, looks like it's following the guidelines to me.  Marking as fedora-review+ under the assumption these minor bits can be fixed.

Comment 6 Lokesh Mandvekar 2014-04-12 21:12:28 UTC
New Package SCM Request
=======================
Package Name: geard
Short Description: CLI client for installing Docker images as containers
Owners: lsm5
Branches: f19 f20 el6 epel7
InitialCC: golang-sig

Comment 7 Kevin Fenzi 2014-04-13 20:39:24 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2014-04-13 22:52:46 UTC
geard-0-0.3.gitb31df16.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/geard-0-0.3.gitb31df16.fc20

Comment 9 Fedora Update System 2014-04-13 23:00:16 UTC
geard-0-0.3.gitb31df16.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/geard-0-0.3.gitb31df16.fc19

Comment 10 Fedora Update System 2014-04-15 15:33:50 UTC
geard-0-0.3.gitb31df16.fc20 has been pushed to the Fedora 20 testing repository.

Comment 11 Lokesh Mandvekar 2014-04-23 23:41:06 UTC
Jessica, copying you to this bug. It'll send out an email notification when geard gets pushed to stable, so you'll know when to update the geard install instructions

Comment 12 Fedora Update System 2014-04-25 04:25:45 UTC
geard-0-0.3.gitb31df16.fc19 has been pushed to the Fedora 19 stable repository.

Comment 13 Fedora Update System 2014-04-25 04:27:21 UTC
geard-0-0.3.gitb31df16.fc20 has been pushed to the Fedora 20 stable repository.


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