Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-26 Thread Gordon Chung
hi, just so this issue doesn't get lost again. Mehdi's bp seems like a good place to track this issue: https://blueprints.launchpad.net/nova/+spec/usage-data-in-notification. Joe, i agree with you that it's too late for this iteration... maybe it's something we should mark low priority for J

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-26 Thread Sean Dague
On 02/26/2014 03:33 PM, Gordon Chung wrote: hi, just so this issue doesn't get lost again. Mehdi's bp seems like a good place to track this issue: https://blueprints.launchpad.net/nova/+spec/usage-data-in-notification. Joe, i agree with you that it's too late for this iteration... maybe

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-26 Thread Gordon Chung
Ceilometer really needs to stop importing server projects in unit tests. By nature this is just going to break all the time. i believe that was the takeaway from the thread -- it's an old thread and i was just doing some back-reading. Cross project interactions need to be tested by

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-05 Thread Julien Danjou
On Tue, Feb 04 2014, Joe Gordon wrote: Ceilometer running a plugin in nova is bad (for all the reasons previously discussed), Well, I partially disagree. Are you saying that nobody is allowed to run a plugin in Nova? So what are these plugins in the first place? Or if you're saying that

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-05 Thread Dan Smith
We don't have to add a new notification, but we have to add some new datas in the nova notifications. At least for the delete instance notification to remove the ceilometer nova notifier. A while ago, I have registered a blueprint that explains which datas are missing in the current nova

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-05 Thread Joe Gordon
On Wed, Feb 5, 2014 at 6:57 AM, Dan Smith d...@danplanet.com wrote: We don't have to add a new notification, but we have to add some new datas in the nova notifications. At least for the delete instance notification to remove the ceilometer nova notifier. A while ago, I have registered a

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-05 Thread Joe Gordon
On Wed, Feb 5, 2014 at 1:35 AM, Julien Danjou jul...@danjou.info wrote: On Tue, Feb 04 2014, Joe Gordon wrote: Ceilometer running a plugin in nova is bad (for all the reasons previously discussed), Well, I partially disagree. Are you saying that nobody is allowed to run a plugin in Nova? So

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-05 Thread Julien Danjou
On Wed, Feb 05 2014, Joe Gordon wrote: Dan explained it well in another email. Is the plugin run in devstack? I couldn't seem to find the code that enables the ceilometer plugin in nova. It was in devstack but got removed because it was hanging IIRC. There was a bug report about that. It

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-05 Thread Julien Danjou
On Wed, Feb 05 2014, Joe Gordon wrote: […] Is that right? Personally, I think #1 is far superior to #2. ++ to #1. I am concerned about the timing of this and don't think we can do this by icehouse though. #1 has been on the radar for – at least – a year. I don't have the courage to dig

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-04 Thread Julien Danjou
On Mon, Feb 03 2014, Joe Gordon wrote: We know, Ceilometer has been broken several times because of that in the past months. We know we shouldn't do that, but for now we don't have enough work force to work on a batter solution unfortunately. Does this issue mean Ceilometer won't work for

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-04 Thread Dan Smith
Whats the underlying problem here? nova notifications aren't versioned? Nova should try to support ceilometer's use case so it sounds like there is may be a nova issue in here as well. Oh you're far from it. Long story short, the problem is that when an instance is detroyed, we need to

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-04 Thread Joe Gordon
On Tue, Feb 4, 2014 at 2:01 AM, Julien Danjou jul...@danjou.info wrote: On Mon, Feb 03 2014, Joe Gordon wrote: We know, Ceilometer has been broken several times because of that in the past months. We know we shouldn't do that, but for now we don't have enough work force to work on a batter

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-04 Thread Adrian Turjak
On 05/02/14 10:07, Joe Gordon wrote: On Tue, Feb 4, 2014 at 2:01 AM, Julien Danjou jul...@danjou.info wrote: On Mon, Feb 03 2014, Joe Gordon wrote: We know, Ceilometer has been broken several times because of that in the past months. We know we shouldn't do that, but for now we don't have

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-04 Thread Mehdi Abaakouk
On Tue, Feb 04, 2014 at 01:11:10PM -0800, Dan Smith wrote: Whats the underlying problem here? nova notifications aren't versioned? Nova should try to support ceilometer's use case so it sounds like there is may be a nova issue in here as well. Oh you're far from it. Long story

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-02-03 Thread Joe Gordon
On Thu, Jan 30, 2014 at 2:28 AM, Julien Danjou jul...@danjou.info wrote: On Thu, Jan 30 2014, Joe Gordon wrote: Hi Joe, While looking at gate failures trying to improve our classification rate I stumbled across this:

Re: [openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch

2014-01-30 Thread Jeremy Stanley
It's also worth pointing out a related side effect of that choice... URL: http://logs.openstack.org/14/68714/3/gate/gate-ceilometer-python27/dc7e987/console.html#_2014-01-30_15_57_30_413 Uploads of branch tarballs are not stable and they're also not atomic. If your job tries to retrieve that