Re: [openstack-dev] [stable/liberty] [requirements]/[oslo] ceilometer jobs failing.

2015-10-17 Thread Tony Breeds
On Sat, Oct 17, 2015 at 08:56:01AM +1100, Tony Breeds wrote:
 
> We actually need both https://review.openstack.org/233854 
> and https://review.openstack.org/#/c/235536  to land at this point.

Okay https://review.openstack.org/233854 has merged
https://review.openstack.org/#/c/235536 is in check now.
 
> as 2.6.0 and 2.6.1 are problematic.
>  
> > Then we have https://review.openstack.org/235815 to strip the
> > pre-versioning setting and fold in the requirements updates for
> > ceilometer.
> 
> I'll address the review feedback  and republish.

Once 235536 merges we need either of the reviews at:
https://review.openstack.org/#/q/topic:next-liberty-stable+project:openstack/ceilometer,n,z

Yours Tony.


pgp4kmwaIJBOK.pgp
Description: PGP signature
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [stable/liberty] [requirements]/[oslo] ceilometer jobs failing.

2015-10-16 Thread Davanum Srinivas
Tony,

yes, planning to do a oslo.messaing 2.6.2 on monday.

-- dims

On Fri, Oct 16, 2015 at 7:21 AM, Tony Breeds 
wrote:

> Hi All,
> We have an issue for any review that causes a new commit to
> ceilometer, it
> will fail with[1]:
>
> ValueError: git history requires a target version of
> pbr.version.SemanticVersion(5.0.1), but target version is
> pbr.version.SemanticVersion(5.0.0)
>
> Switching to post-versioning[2] can't land because of [3]  The fix for
> that is
> to ban 2.6.1 (and 2.6.0) from requirements.  That can't land because of[1],
> and we've come full circle.
>
> I can't use Depends-On as the patches in each repo need to land at the same
> time.  So I tried the "obvious squash" patch at [4], but that isn't going
> to
> work because of the sensible requirements checking.
>
> The only way it fix this I can think of is to release 2.6.2 of
> oslo.messaging
> with the fixes needed by ceilometer (which have merged).
>
> I can think of a couple of rather terrible ways to hack around this but
> lets
> not go there unless there is a good reason to not release 2.6.2
>
> Yours Tony.
>
> [1] https://bugs.launchpad.net/ceilometer/+bug/1506805
> [2] https://review.openstack.org/#/c/235207/
> [3] https://bugs.launchpad.net/oslo.messaging/+bug/1505730
> [4] https://review.openstack.org/#/c/235815/
> [5] https://review.openstack.org/#/c/235536/
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>


-- 
Davanum Srinivas :: https://twitter.com/dims
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [stable/liberty] [requirements]/[oslo] ceilometer jobs failing.

2015-10-16 Thread Tony Breeds
On Fri, Oct 16, 2015 at 03:32:21PM -0400, Doug Hellmann wrote:
> Excerpts from Tony Breeds's message of 2015-10-16 22:21:27 +1100:
> > Hi All,
> > We have an issue for any review that causes a new commit to ceilometer, 
> > it
> > will fail with[1]:
> > 
> > ValueError: git history requires a target version of
> > pbr.version.SemanticVersion(5.0.1), but target version is
> > pbr.version.SemanticVersion(5.0.0)
> > 
> > Switching to post-versioning[2] can't land because of [3]  The fix for that 
> > is
> > to ban 2.6.1 (and 2.6.0) from requirements.  That can't land because of[1],
> > and we've come full circle.
> 
> The fix to allow global requirements changes to land in stable/liberty
> is in https://review.openstack.org/235924

Ahh thanks Doug.  That's a neat fix.  I wasn't sure why integration.sh
and the clone of a clone.  Now I understand, just as it's gone ;P
 
> After that has merged, https://review.openstack.org/233854 will block
> oslo.messaging 2.6 from stable/liberty.

We actually need both https://review.openstack.org/233854 
and https://review.openstack.org/#/c/235536  to land at this point.

as 2.6.0 and 2.6.1 are problematic.
 
> Then we have https://review.openstack.org/235815 to strip the
> pre-versioning setting and fold in the requirements updates for
> ceilometer.

I'll address the review feedback  and republish.
 
> We need to land the patches in that order.
> 
> > I can't use Depends-On as the patches in each repo need to land at the same
> > time.  So I tried the "obvious squash" patch at [4], but that isn't going to
> > work because of the sensible requirements checking.
> > 
> > The only way it fix this I can think of is to release 2.6.2 of 
> > oslo.messaging
> > with the fixes needed by ceilometer (which have merged).
> > 
> > I can think of a couple of rather terrible ways to hack around this but lets
> > not go there unless there is a good reason to not release 2.6.2
> 
> Releasing 2.6.2 would also be OK, if that happens first.

Cool.  I'd hope we can land the 4 patches required before Monday.

Yours Tony.


pgpTD5C7Zscsn.pgp
Description: PGP signature
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [stable/liberty] [requirements]/[oslo] ceilometer jobs failing.

2015-10-16 Thread Doug Hellmann
Excerpts from Tony Breeds's message of 2015-10-16 22:21:27 +1100:
> Hi All,
> We have an issue for any review that causes a new commit to ceilometer, it
> will fail with[1]:
> 
> ValueError: git history requires a target version of
> pbr.version.SemanticVersion(5.0.1), but target version is
> pbr.version.SemanticVersion(5.0.0)
> 
> Switching to post-versioning[2] can't land because of [3]  The fix for that is
> to ban 2.6.1 (and 2.6.0) from requirements.  That can't land because of[1],
> and we've come full circle.

The fix to allow global requirements changes to land in stable/liberty
is in https://review.openstack.org/235924

After that has merged, https://review.openstack.org/233854 will block
oslo.messaging 2.6 from stable/liberty.

Then we have https://review.openstack.org/235815 to strip the
pre-versioning setting and fold in the requirements updates for
ceilometer.

We need to land the patches in that order.

> I can't use Depends-On as the patches in each repo need to land at the same
> time.  So I tried the "obvious squash" patch at [4], but that isn't going to
> work because of the sensible requirements checking.
> 
> The only way it fix this I can think of is to release 2.6.2 of oslo.messaging
> with the fixes needed by ceilometer (which have merged).
> 
> I can think of a couple of rather terrible ways to hack around this but lets
> not go there unless there is a good reason to not release 2.6.2

Releasing 2.6.2 would also be OK, if that happens first.

Doug

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [stable/liberty] [requirements]/[oslo] ceilometer jobs failing.

2015-10-16 Thread Tony Breeds
Hi All,
We have an issue for any review that causes a new commit to ceilometer, it
will fail with[1]:

ValueError: git history requires a target version of
pbr.version.SemanticVersion(5.0.1), but target version is
pbr.version.SemanticVersion(5.0.0)

Switching to post-versioning[2] can't land because of [3]  The fix for that is
to ban 2.6.1 (and 2.6.0) from requirements.  That can't land because of[1],
and we've come full circle.

I can't use Depends-On as the patches in each repo need to land at the same
time.  So I tried the "obvious squash" patch at [4], but that isn't going to
work because of the sensible requirements checking.

The only way it fix this I can think of is to release 2.6.2 of oslo.messaging
with the fixes needed by ceilometer (which have merged).

I can think of a couple of rather terrible ways to hack around this but lets
not go there unless there is a good reason to not release 2.6.2

Yours Tony.

[1] https://bugs.launchpad.net/ceilometer/+bug/1506805
[2] https://review.openstack.org/#/c/235207/
[3] https://bugs.launchpad.net/oslo.messaging/+bug/1505730
[4] https://review.openstack.org/#/c/235815/
[5] https://review.openstack.org/#/c/235536/


pgpK5e0UUHwYT.pgp
Description: PGP signature
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev