[openstack-dev] Urgent PBR code reviews
Mark McLoughlin
markmc at redhat.com
Tue Jun 11 09:56:08 UTC 2013
Hi Julien,
On Tue, 2013-06-11 at 10:58 +0200, Julien Danjou wrote:
> Hi there,
>
> I'd like to emphasize that Ceilometer CI is failing for now 5 days, and
> that we can't get anything merged, due to a pbr issue. Monty has fixed
> this a few day ago, but it would be really great if we could speed up
> things by reviewing, approving his work and get a new release:
>
> https://review.openstack.org/#/q/status:open+project:openstack-dev/pbr+branch:master+topic:27519,n,z
I don't think the issue here is "people aren't reviewing fast enough",
frankly. The patches are getting a pretty decent amount of attention
from reviewers.
There are a bunch of issues we need to think about here:
- I don't see a bug report for this:
https://bugs.launchpad.net/pbr
https://bugs.launchpad.net/ceilometer
I've just dug about and found e.g.:
http://logs.openstack.org/32108/1/check/gate-ceilometer-python27/2657/console.html.gz
pkg_resources.VersionConflict: (requests 1.2.3 (/home/jenkins/workspace/gate-ceilometer-python27/.tox/py27/lib/python2.7/site-packages), Requirement.parse('requests>=1.1,<1.2.1'))
Is that the issue?
The details should be in a Critical bug.
- It's not clear to me what change in pbr caused the problem. Was it
the use of easy_install ?
https://review.openstack.org/31778
- Whatever the patch that caused the issue was, why haven't we
reverted it?
- If we can't revert it, why haven't we just removed the broken pbr
version from pypi?
- Which of Monty's 9 patches fixes the issue?
https://review.openstack.org/#/q/project:openstack-dev/pbr+status:open+owner:mordred,n,z
Does it depend on patches which aren't absolutely required to fix
this critical issue?
- We now have this ceilometer issue, an issue with quantumclient:
https://bugs.launchpad.net/pbr/+bug/1187452
and a oslo.config related issue:
https://review.openstack.org/30938
all waiting on a fixed pbr release.
I'm going to review the latest updates to the patches, but at a time
like this we actually need the minimum possible churn and some clear
thinking ... not super fast reviewing of a big stack of changes. What's
the quickest and least risky thing we can do to get Ceilometer
unblocked?
Cheers,
Mark.
More information about the OpenStack-dev
mailing list