Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-16
13:53:30 efried dansmith: Doc-only fixes are kosher :) ---^
13:53:46 dansmith ...yeah, and/
13:54:39 openstack Launchpad bug 1763966 in devstack "python-psutil dependency error on ubuntu 16.04.4" [Critical,Fix released] - Assigned to hongbin (hongbin034)
13:54:39 efried dansmith: cdent was telling me earlier we think this has something to do with neutron and requirements, with this being one of the culprits: https://bugs.launchpad.net/devstack/+bug/1763966
13:54:56 dansmith efried: right I know,
13:54:57 efried not nova-specific, beind discussed in #openstack-dev
13:55:00 dansmith I know
13:55:01 efried okay.
13:55:07 dansmith I'm asking if it's 100%, because I assume it is,
13:55:14 dansmith thus no reason to send anything into recheck or the gate
13:55:16 efried It seems to be. Except for doc fixes.
13:55:17 efried Right.
13:56:04 mriedem master is fine
13:56:05 efried dansmith: So yeah, you should hold off +A on the zvm series.
13:56:17 efried but only til the gate is fixed
13:56:29 mriedem https://review.openstack.org/#/q/I8bf80efc04883cd754c19bea0303064080112c6e
13:56:56 dansmith oh as of an hour ago, okay
13:57:00 efried oh, okay, cool.
13:57:06 dansmith all the ones I'm looking at were before that
13:59:22 edleafe Scheduler subteam meeting in 1 minute in #openstack-meeting-alt
14:12:03 M0z122 mriedem: thanks, will ask to cfriesen
14:26:02 openstackgerrit Merged openstack/nova-specs master: Spec for isolating configuration of placement database https://review.openstack.org/552927
14:33:02 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Stop printing flavor details on successful flavor-delete https://review.openstack.org/561634
14:51:30 kosamara Hi mriedem, I've updated https://review.openstack.org/#/c/555861/ Can you take a look if you have the time?
14:52:28 mriedem i'll put it in the browser tab queue
15:13:41 kashyap Hi folks, wonder if anyone has time to look at why is 'mox' raising UnexpectedMethodCallError with this unit test: http://paste.openstack.org/show/719304/
15:14:20 kashyap I posted two code fragments the test points to (from guest.py & driver.py in nova/libvirt)
15:18:17 kashyap mdbooth: Wonder if you have a minute to look at the above (read prior 3 lines of scroll for context)? Last I recall, you made the change to that surrounding code. :-)
15:19:33 kashyap mdbooth: So, that UnexpectedMethodCallError is raised when "if a method is called with incorrect parameters, or out of the specified order".
15:19:46 kashyap Neither is the case, from my reading.
15:20:07 mdbooth kashyap: What have you changed?
15:20:37 mdbooth kashyap: pastebin your local diff
15:20:48 kashyap Fixing the fall out of bumping versions: https://review.openstack.org/#/c/558783/
15:21:18 mdbooth kashyap: I'll fire it up locally
15:21:31 kashyap mdbooth: Then, wait -- let me post the merge-conflict fixed series
15:22:40 mdbooth I can't pull it right now
15:23:36 kashyap Yeah, it's merge conflict; I fixed it this morning locally. Pushing the new rev in a couple. I'll notify
15:23:51 efried kashyap: Note that takashin is doing a whole slew of patches to remove mox.
15:24:13 kashyap efried: So, he'll be replacing it w/ Mock, is it?
15:24:36 efried kashyap: Yes. There's a bp for that, so I'm sure he would welcome the help if you'd like to propose a "fix" for this problem that happens to swap out mox for mock.
15:25:09 efried kashyap: https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/mox-removal
15:25:29 kashyap efried: I'll see about it. Python unit tests is one my Python weakness area
15:25:47 kashyap First trying to remove all the now-needless compat code due to the version bumps
15:26:10 efried kashyap: No pressure :)
15:27:39 kashyap Thanks for the (non-null) pointer
15:28:28 mdbooth efried kashyap: I'd recommend against mixing those 2 changes, though.
15:28:41 kashyap mdbooth: Yeah, I'm strictly following the "One logical change per patch"
15:28:43 kashyap No madness
15:28:58 kashyap But appreciate the reminder :-)
15:29:28 mdbooth_ kashyap: Don't know if ^^^ sent, but I recommend against mixing those 2 changes.
15:29:53 kashyap mdbooth_: Yep, got sent. And then I wrote: < kashyap> mdbooth: Yeah, I'm strictly following the "One logical change per patch" rule
15:30:06 cfriesen mriedem: M0z122: I think this was the last time I tried to push LVM resize/migration upstream: https://review.openstack.org/#/c/337334/
15:31:53 mdbooth_ cfriesen: Aside: do you get a performance benefit from lvm?
15:32:05 openstackgerrit Merged openstack/nova master: Add config drive link to api-guide https://review.openstack.org/561113
15:32:54 cfriesen mdbooth_: last time we checked, I think there was a difference, but I don't know that we've checked in a while. Have you seen more recent benchmarks?
15:33:23 mdbooth_ cfriesen: No, but I vaguely recall that it was marginal at best.
15:33:32 mdbooth_ cfriesen: Over raw, that is.
15:33:56 mdbooth_ Just wondering if you had data which showed it's worth the bother.
15:34:00 M0z122 cfriesen: many thks will have a look on it
15:35:03 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add DifferentHostFilter to enabled_filters default https://review.openstack.org/561651
15:37:12 mdbooth_ cfriesen: Intuitively, I'd expect that if you fallocate a raw file which isn't fragmented and open it O_DIRECT, all of which I think we do, the performance should be the same as LVM, as the kernel is basically doing the same in both cases.
15:38:41 mdbooth_ cfriesen: Numbers beat intuition of course, though :)
15:52:57 Nisha_Agarwal None at some point. We commented out this line https://github.com/openstack/nova/blob/stable/pike/nova/objects/instance.py#L824 and the deploy goes thru.
15:52:57 Nisha_Agarwal Hi team, I have 2 stable pike devstack setups- one stacked last week on a freshly imaged system and another stacked around a month ago. When i was trying "nova boot" for ironic nodes on both the setups i land up in this error http://paste.openstack.org/show/719143/ due to which it send the "delete" event to ironic node as soon as it goes to "wait-call-back" state. On debugging we found that it happens because instance's context is set to
15:55:13 dansmith melwitt: did you see my comment on that bottom certs patch? I think she reverted the changes she made for you
15:57:07 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add (Same|Different)HostFilter to defaults https://review.openstack.org/561651
15:57:12 Nisha_Agarwal dansmith, could you help me with above? Should i go ahead and open the defect against Nova for the OrphanedObject error reported on stable/pike?
15:57:59 Nisha_Agarwal or there is some configuration in the config file which i should be setting so that the issue is not hit?
16:01:57 dansmith Nisha_Agarwal: orphaned objects are those without a context set, which prevents them from doing things like loading or saving from the database
16:02:04 dansmith Nisha_Agarwal: obviously no config option should cause/prevent that
16:03:46 cfriesen mriedem: we're running into problems with the code that adds a uuid to services on the fly. it seems that calling service.save() on a deleted service doesn't work. it chokes down in service_get()
16:04:04 mriedem cfriesen: where do we save() a deleted service?
16:04:20 cfriesen Service._from_db_object(), if there's no uuid
16:04:35 mriedem might be a bug that melwitt has recently fixed
16:05:32 mriedem cfriesen: https://review.openstack.org/#/c/555923/
16:05:39 mriedem is that it?
16:06:04 dansmith yeah
16:06:06 dansmith was going to say
16:06:53 mriedem if it's not that, then i will need to see a stacktrace rather than a dead cat
16:07:35 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
16:07:36 openstackgerrit Jackie Truong proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
16:07:37 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
16:07:39 openstackgerrit Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158
16:07:49 cfriesen mriedem: so previously we were hitting the bug that commit was fixing...so we applied that fix and hit this other problem. Stacktrace at http://paste.openstack.org/show/719308/
16:08:04 Nisha_Agarwal dansmith, the issue is seen on both the stable pike setups as soon as nova calls this https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1102 which in turn hits https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L452
16:08:16 mriedem gibi: welcome back - you might have an opinion on the last comment i left here about notifications https://review.openstack.org/#/c/486204/
16:08:24 dansmith cfriesen: is that service deleted?
16:08:27 mriedem i'll throw it in the notification meeting
16:08:34 cfriesen dansmith: yes
16:08:51 dansmith cfriesen: that's why, but I thought we determined it shouldn't even be processing deleted services.. did you tweak something in there?
16:09:35 cfriesen dansmith: the sequence of events was: boot instance on compute node, migrate instance, delete compute node and service, upgrade to pike, create new compute node with same name, migrate instance back to that compute node.
16:09:56 cfriesen dansmith: so now when we look up the service by name, there are two matching entries, one of which is deleted
16:10:24 dansmith cfriesen: but we're using model_query which should filter out the deleted one, IIRC
16:10:28 openstackgerrit Jay Pipes proposed openstack/nova-specs master: Numbered request groups use different providers https://review.openstack.org/560974
16:10:32 mriedem cfriesen: i need more of the stacktrace
16:10:37 mriedem not the middle of it
16:11:10 stephenfin bauzas: Want to weigh in on this comment? https://review.openstack.org/#/c/379525/9/nova/scheduler/weights/cpu.py
16:11:17 cfriesen mriedem: will try to get it.
16:11:24 stephenfin bauzas: We don't do this for RAM or disk. Should we?
16:12:36 mriedem stephenfin: the core filter does

Earlier   Later