search

projectatomic / commissaire-mvp

A lightweight REST interface for upgrading, restarting, and bootstrapping new hosts into an existing Container Management cluster.
http://commissaire.readthedocs.org/en/latest/
GNU General Public License v3.0
15 stars 9 forks source link

[merged] commissaire.spec corrections #138

Closed mbarnes closed 8 years ago

mbarnes commented 8 years ago

Capturing spec file corrections from the completed Fedora package review.

https://bugzilla.redhat.com/show_bug.cgi?id=1328892

I've also converted everything over to Python 3 for Fedora, but not here. I'm not sure if we want the repo's spec file to represent Fedora or RHEL.

ashcrow commented 8 years ago

IMO once we have packages in Fedora and RHEL I think we drop the contrib version and update the docs to point to where one can grab the SPECs and SRPMs. That way we are not needing to keep 3 spec files updated at all times.

ashcrow commented 8 years ago

LGTM

@rh-atomic-bot try

rh-atomic-bot commented 8 years ago

:hourglass: Trying commit 5b6a50c with merge 342d7c6...

rh-atomic-bot commented 8 years ago

:sunny: Test successful - travis State: approved= try=True

mbarnes commented 8 years ago

Agreed on dropping contrib spec eventually. Feeling a sense of doom about getting this in RHEL, but that's a separate conversation.

ashcrow commented 8 years ago

@rh-atomic-bot r+

rh-atomic-bot commented 8 years ago

:pushpin: Commit 5b6a50c has been approved by ashcrow

rh-atomic-bot commented 8 years ago

:hourglass: Testing commit 5b6a50c with merge e3a445d...

rh-atomic-bot commented 8 years ago

:sunny: Test successful - travis Approved by: ashcrow Pushing e3a445d40d5a81cb955142eea9e3c0d38c8950ae to master...