[openstack-dev] [heat][telemetry] gate-ceilometer-dsvm-integration broken
shardy at redhat.com
Mon Jan 4 10:11:48 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
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
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.
More information about the OpenStack-dev