Red Hat Bugzilla – Bug 842891
Review Request: stickshift-broker - StickShift broker components
Last modified: 2012-07-27 18:49:41 EDT
Spec URL: http://www.yortnoswad.org/packages/review/stickshift-broker.spec
SRPM URL: http://www.yortnoswad.org/packages/review/stickshift-broker-0.6.7-2.fc18.src.rpm
Description: This contains the broker 'controlling' components of StickShift.
This includes the public APIs for the client tools.
Fedora Account System Username: tdawson
[me@rawhide]$ rpmlint stickshift-broker.spec stickshift-broker-0.6.7-2.fc18.src.rpm
1 packages and 1 specfiles checked; 0 errors, 0 warnings.
mod_ridealong is not packaged ( in f17, didn't check in rawhide ), and neither is the ruby module ( I assume that's a fork of passenger ? )
And I think that it work with a remote mongodb server or bind, so i am not sure this is good to pull mongodb-server, bind ( especially if it use a plugin to modify dns, like uplift-bind ).
For the selinux part, wouldn't it be easier to push them directly in selinux-policy ( I am not sure that's good idea to modify boolean from %post without warning )
Among the various deps, there is rcov, but this is not listed as useful for runtime, just for testing ( like cucumber, who is not in Requires ). Shouldn't it be left out ?
There is also no mention of multimap in code, and the following gem are not listed in the spec while they are in the Gemfile :
Could you check with upstream the requirement ?
Also, can the unit test be run in %check ( and so that would require rcov at built time) ?
And it need to have a systemd file, I could write one but I am not sure.
Stickshift-broker should be started separately, with a separate apache + passenger ( + mod_ssl ) instance running on port 8080 ?
( also, why is there a requirement on oddjob since oddjob is deprecated upstream, replaced by mcollective, and oddjob do not seems to be used in the code ).
Thank you for being so thorough. It helps when I have so many of these that I am working on.
ridealong - That should be having a review request in by the end of the week.
Ridealong also has it it's own selinux policy and selinux setup. So you are correct, that part should not be in this rpm.
I see what you are saying about mongodb-server and bind. I will talk with the developer. Especially with bind, since that is a dependancy of uplift.
I didn't even look in the gemfile, since this package isn't technically a gem. Didn't see those dependancies.
oddjob - yep, need to fix that, should be doing mcollective.
systemd and startup scripts. I'll talk to the developer about that.
This package is being renamed
I have also talked to the developers and will be removing several of the dependancies that you talked about. One exception is mongodb-server, which they say currently has to stay, but hopefully the next release can be taken out as a dependancy.
When I have a review-request up for openshift-origin-broker I will convert this over to that.
This package has been renamed and a new review request has been created.
*** This bug has been marked as a duplicate of bug 844013 ***