Bug 1326116 - Please add ocitools to ship with runc
Summary: Please add ocitools to ship with runc
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: runc
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jan Chaloupka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1326274 1326277
Blocks: golang-packaging-tracker
TreeView+ depends on / blocked
 
Reported: 2016-04-11 21:19 UTC by Daniel Walsh
Modified: 2016-05-18 08:42 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-05-18 08:42:15 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Daniel Walsh 2016-04-11 21:19:03 UTC
ocitools is a handy tool to allow you to easily create OCI config.json files to be used with runc containers.

Comment 1 Jan Chaloupka 2016-04-12 06:48:00 UTC
Assuming this is the correct repo [1]. It has one dependency [2]. I would create a new package for ocitools. Although it has no exported symbols (no devel), they can be introduced later. And not everyone will want to install ocitools with runc at the same time. 

[1] https://github.com/opencontainers/ocitools
[2] https://github.com/opencontainers/runtime-spec

Comment 2 Daniel Walsh 2016-04-12 16:12:26 UTC
I don't see this as a big deal to install both at the same time.  ocitools is not a huge package (I don't think).  There is no library.  I have recently added man pages and bash completions to it.

Comment 3 Jan Chaloupka 2016-04-12 17:07:57 UTC
Though extending the runc.spec with ocitools is fine, I would rather request for another package as both runc and ocitools can be maintained and updated separately. At least for consistency of packaged golang projects.

Comment 4 Daniel Walsh 2016-04-14 20:16:53 UTC
We will be shipping them together in RHEL at least for now. So I don't have a problem with setting up a different package, other then the pain of getting packages into Fedora.

Comment 5 Jan Chaloupka 2016-05-18 08:42:15 UTC
Built as a separate package [1]

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1326277


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