Re: [openstack-dev] [nova][oslo] pep8 gating fails due to tools/config/check_uptodate.sh

2014-02-04 Thread Daniel P. Berrange
On Mon, Feb 03, 2014 at 09:49:15PM -0600, Matt Riedemann wrote: This broke us (nova) again today after python-keystoneclient-0.5.0 was released with a new config option. Joe Gordon pushed the patch to fix nova [1] so everyone will need to recheck their patches again once that merges. This

Re: [openstack-dev] [nova][oslo] pep8 gating fails due to tools/config/check_uptodate.sh

2014-02-04 Thread Julien Danjou
On Tue, Feb 04 2014, Daniel P. Berrange wrote: I think the pretty obvious solution here is to *stop* storing automatically generated files in GIT. This kind of problem crops up time again for any project I've seen which stores auto-generated files in GIT. The sample config file should only

Re: [openstack-dev] [nova][oslo] pep8 gating fails due to tools/config/check_uptodate.sh

2014-02-04 Thread Julien Danjou
On Tue, Feb 04 2014, Doug Hellmann wrote: This does seem like the correct solution, except that it would require having oslo.config and related tools installed just to build the sdist for nova, and I imagine that will break distro packaging. Distro have build dependencies, so I don't think

Re: [openstack-dev] [nova][oslo] pep8 gating fails due to tools/config/check_uptodate.sh

2014-02-04 Thread Daniel P. Berrange
On Tue, Feb 04, 2014 at 09:19:08AM -0500, Doug Hellmann wrote: On Tue, Feb 4, 2014 at 5:09 AM, Julien Danjou jul...@danjou.info wrote: On Tue, Feb 04 2014, Daniel P. Berrange wrote: I think the pretty obvious solution here is to *stop* storing automatically generated files in GIT.

Re: [openstack-dev] [nova][oslo] pep8 gating fails due to tools/config/check_uptodate.sh

2014-02-03 Thread Matt Riedemann
On 1/13/2014 10:49 AM, Sahid Ferdjaoui wrote: Hello all, It looks 100% of the pep8 gate for nova is failing because of a bug reported, we probably need to mark this as Critical. https://bugs.launchpad.net/nova/+bug/1268614 Ivan Melnikov has pushed a patchset waiting for review: