[openstack-dev] [heat][telemetry] gate-ceilometer-dsvm-integration broken
ramishra at redhat.com
Mon Jan 4 10:40:06 UTC 2016
> On Mon, Dec 28, 2015 at 01:52:45PM +0100, Julien Danjou wrote:
> > On Mon, Dec 28 2015, Rabi Mishra wrote:
> > > Yes, this has started happening after keystone/trusts config changes by
> > > the
> > > devstack patch you mentioned. I've no idea how this can be fixed. As
> > > Steve
> > > Hardy is away, either someone with keystone knowledge should fix this or
> > > we
> > > merge the devstack patch revert that I tested few days ago.
> > Why don't you just revert the devstack change?
> > This is way saner than disabling the test! Steve will be able to rework
> > his initial change when he come back.
> Firstly, I'm very sorry for the breakage here, and I agree that in general
> a quick-revert is the best policy when something like this happens.
> I'm a little unclear how this occurred tho, since I had a clear CI run on
> this patch:
> Which had a Depends-On to the devstack change, anyone know why that didn't
> fail with the CeilometerAlarmTest.test_alarm before the devstack change
It seems the test was skipped, as it was disabled for another bug.
> Regardless, we've got several fixes now which can be considered:
> 1. Rabi's devstack revert:
> 2. Fix the actual issue in heat:
> Given that the review latency on Devstack is quite high, it seems possible
> we'll land (2) before (1) lands, but if not then I'll re-propose it and
> hopefully figure out where I went wrong with Depends-On to confirm all is
> fixed before it lands.
> Also, there's this fix:
> I've not yet confirmed if this also fixes the issue referencing the default
> domain which broke the alarm tests.
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
More information about the OpenStack-dev