Spec URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient.spec SRPM URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient-0-23.1.dev10.src.rpm Description: Client for Openstack Watcher service (python-watcherclient). Fedora Account System Username: danpawlik copr build: WIP
Clients should be submitted to Fedora (I fixed it) 1. please package latest stable release and fix according Source0 URL 2. simplify the with_python3 block %if 0%{?fedora} >= 24 %global with_python3 1 %endif
I have found some issues: - Doc generation fails, we need to add "PYTHONPATH=." to the sphinx-build call - Even with that, it looks like some dependencies are missing. mock complained about keystoneclient.v2_0 not being available, so the build requirements need to be checked. - There is no need for the %exclude lines, since you are already packaging tests in their own subpackage.
Spec file was based on example client file: https://github.com/openstack-packages/openstack-example-spec/blob/master/python-exampleclient.spec hguemar: 1. I will change it ;) 2. I don't want to change, because I want to be compatible with example. IMHO one global variable is not the main problem of this spec file... Javier Pena: - I will check
Spec URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient.spec SRPM URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient-0-23.0.src.rpm hguemar,Javier Peña: all bugs/suggestions fixed.
* Binaries are not handled correctly, you install the watcher CLI twice. So in Fedora, since python3 version is installed last, python2-watcherclient pulls python3 * You don't %exclude tests directories from pythonX-packages, so they're also included in the the latter. So installing both packages will lead to a conflict. * Don't require python-wheel * Requires: mock >= 1.2 Won't do what you expect, mock packages is a a chrooted build utility for fedora. What you were looking for was python-mock.
I just realized some of the required packages have an Epoch set, like https://github.com/rdo-packages/keystoneclient-distgit/blob/rpm-master/python-keystoneclient.spec#L18 . You should include the Epoch in the Requires/BuildRequires field.
Spec URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient.spec SRPM URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient-0-23.0.src.rpm I made some changes based on your comments.
Hello reviewers, If something is wrong in code, please write comment on github page. It's easier for me to modify spec file ;) Thx.
What changed: -added epoch for required packages, -removed watcherclient epoch -uncommented license file -new watcherclient release: 0.24.0 -changed requirements: python-oslo-utils >= 3.16.0 # was 3.11.0 Spec URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient.spec SRPM URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient-0-24.0.src.rpm If something is wrong with code, pls comment on github page ;) Regards PS I'm sorry that so long...
Before we move on, I'd like to clarify a point: We maintain all OpenStack clients in Fedora, so I'd rather avoid another review for introducing it in Fedora. Daniel, are you willing to maintain it or should I open another review for that?
Package looks much better, but you need to fix: Version: 0.24.0 Release: 1%{?dist} Version field holds upstream versioning, Release is spec file versioning, and you should not redefine upstream_version, as it is used by DLRN. Requires: sphinx is unneeded. prefer python-XXX over python2-XXX requires for python2 packages.
I can be mainaner from python-watcher and python-watcherclient packages ;) Haïkel Guémar : In new commit on github I have changed spec like you propose. Spec URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient.spec SRPM URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient-0.24.0-1.fc24.src.rpm Regards
Hi Daniel, I've added some comments to github. The most important ones are: - BuildRequires: sphinx does not work, you should change it to "BuildRequires: python-sphinx" or remove it (it will be pulled in by oslo-sphinx) - Both the python2 and python3 subpackages include /usr/bin/watcher. This needs to be fixed, https://github.com/rdo-packages/aodhclient-distgit/blob/mitaka-rdo/python-aodhclient.spec#L115-L126 can be used as an example of how to handle this.
Hi, Thanks Javier Peña for reviewing. I made changes and updated version in on github. Spec URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient.spec SRPM URL: https://raw.githubusercontent.com/dduuch/openstack-watcherclient/master/python-watcherclient-0.24.0-1.fc24.src.rpm Github page: https://github.com/dduuch/openstack-watcherclient If something is wrong, pls leave comment on github page ;) Thanks a lot! Regards
Sphinx package removed from BuildRequires. Updated version is on github ;)
Daniel, as discussed on IRC, should we change this review to make it a Fedora package? For client packages, the RDO policy is to keep them in Fedora.
@javier(In reply to Javier Peña from comment #17) > Daniel, as discussed on IRC, should we change this review to make it a > Fedora package? For client packages, the RDO policy is to keep them in > Fedora. Hello, As we talked on Openstack Summit and IRC, watcherclient should be package available in Fedora.
Changed product to Fedora, so this is now a proper Fedora package review.
Ping, still working on that?
Moved to Queens for now, but it will be revisited later.
No activity for 3 years, please reopen if you want to.