Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-03-14
16:33:22 cfriesen okay, works for me.
16:33:44 mdbooth But they all definitely have the same theme.
16:34:27 mdbooth cfriesen: And we can't test virtuozzo
16:34:38 kashyap Exactly, the "potential for reverts" is the thing
16:35:25 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: qemu: Make disk image conversion dramatically faster https://review.openstack.org/640781
16:35:25 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Use 'writeback' QEMU cache mode for Nova instances https://review.openstack.org/641981
16:35:26 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: vzstorage: Use 'writeback' QEMU cache mode https://review.openstack.org/643376
16:35:26 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: smbfs: Use 'writeback' QEMU cache mode https://review.openstack.org/643377
16:35:34 kashyap cfriesen: ^ Mind re-acking?
16:36:16 openstackgerrit Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899
16:37:42 cfriesen I suspect the biggest impact will be I/O usage patterns on the host, since now we'll be queueing up a bunch of writes and flushing them all at once.
16:37:48 cfriesen kashyap: done
16:38:57 kashyap cfriesen: Thank you
16:45:34 openstackgerrit Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899
16:50:00 openstackgerrit Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899
17:03:57 jackding Could cores look at https://review.openstack.org/#/c/621646/? It's been too long.
17:18:15 kashyap cfriesen: Fancy having a gander at: https://review.openstack.org/#/c/631154/ (WIP: Gracefully handle QEMU machine types for guests)
17:27:53 dansmith mriedem: who was asking about the infinite recursion ssl thing yesterday?
17:28:04 dansmith oh EmilienM
17:28:05 dansmith https://review.openstack.org/#/c/626952/
17:29:21 mdbooth dansmith: Yeah, so I was previously going on faith, but it didn't appear to fix the test.
17:29:35 mdbooth But it *did* just fix the reproducer system I had direct access to.
17:30:01 dansmith hrm, this looks weird, I'll have to do some poking
17:30:23 mdbooth It's very weird. If you look in the BZ I have a minimal non-nova python reproducer.
17:31:13 mdbooth Based on my understanding of the eventlet bug, the issue is that we need to do monkey patching really early before importing most things.
17:31:29 mdbooth ...as some code runs at import time.
17:31:32 dansmith yeah
17:31:40 mdbooth So that was the goal of my patch.
17:31:49 cfriesen kashyap: I think the blueprint name is supposed to match the spec filename
17:31:59 mdbooth Have one thing that you import first that does the monkey patching without importing anything else.
17:32:45 mdbooth OSP15 bz is here: https://bugzilla.redhat.com/show_bug.cgi?id=1686817
17:32:45 openstack bugzilla.redhat.com bug 1686817 in openstack-nova "[osp15] nova is broken on the undercloud with ssl" [Urgent,Assigned] - Assigned to mbooth
17:33:17 dansmith mdbooth: if you're patching now before importing oslo.context, I imagine that reload is no longer necessary
17:33:24 kashyap cfriesen: Yeah, noticed it. Can I still update the BP URL?
17:33:39 kashyap cfriesen: I wrote that BP while still the discussions were still hot on QEMU and libvirt lists on the default
17:34:02 cfriesen kashyap: no idea
17:34:08 kashyap cfriesen: Where they were on the verge to change the default to 'q35', but backed out due to "maybe we're breaking backwards compat"
17:34:34 mdbooth dansmith: Quite possibly, but I just moved it.
17:34:39 dansmith yeah I know
17:34:47 dansmith just answering your TODO
17:35:18 dansmith er, note
17:35:35 kashyap cfriesen: Yep, I could change it.
17:37:30 dansmith yikun: FYI https://review.openstack.org/#/c/592285/
17:37:39 openstackgerrit Kashyap Chamarthy proposed openstack/nova-specs master: WIP: Gracefully handle QEMU machine types for guests https://review.openstack.org/631154
17:39:53 dansmith mdbooth: why are there two identical LP bugs?
17:39:58 dansmith or am I confusing something?
17:40:18 kashyap cfriesen: Fixed.
17:40:36 mdbooth dansmith: Unclear.
17:41:21 mdbooth dansmith: Is the other one a tripleo tracker?
17:41:33 dansmith shouldn't be necessary with LP
17:42:16 dansmith mdbooth: so, the cmd/__init__.py one.. does that get run by the entrypoint before anything in something like cmd/compute ? because at first glance, it seems to not move it for those commands, if it comes in the first time they import cmd.common
17:42:27 dansmith I guess it must..
17:42:54 mdbooth dansmith: Yes, it does. The BZ is against nova compute in the undercloud, which isn't wsgi.
17:43:49 dansmith right, okay so this ends up actually running sooner in the compute case because we've done the patching before we import utils, which imported oslo stuff?
17:44:15 mdbooth Yes.
17:44:39 dansmith because you changed things to run at import time, it's maybe a little more obscure, but really the fix here is detaching the monkeypatch from the utils module (or anything else that has a bunch of stuff in it)
17:45:40 mdbooth Yeah. I don't think there's any way to avoid running anything at import time, tbh, because that's the fundamental underlying issue
17:45:50 mdbooth you need to run monkey_patch() before importing stuff
17:46:29 dansmith no, I know.. I get that part
17:47:18 mdbooth Yes. The purpose of the monkey_patch module is to do monkey patching with minimal dependencies.
17:55:33 mdbooth dansmith: Thanks
17:55:41 dansmith aye
17:56:49 kashyap dansmith: If you want some fresh change of scenery, a simple catch: https://review.openstack.org/#/q/topic:bug/1818847+(status:open+OR+status:merged)
17:57:08 dansmith I really don't
17:57:11 kashyap Hehe
17:58:18 kashyap dansmith: It brings 15 times faster performance for image conversion :D -- https://review.openstack.org/#/c/640781/
17:58:51 kashyap I ran it through the QEMU Block Layer maintainer, who also reviewed the patches over e-mail. And gave their: "Looks-good-to-me'd-by", FWIW
18:46:05 mriedem someone want to approve these for gate goodness? https://review.openstack.org/#/q/topic:bug/1819944+(status:open+OR+status:merged)+project:openstack/nova
18:46:17 mriedem gets the ceph portion of our live migration (+grenade) jobs running again
18:46:55 mriedem efried: maybe you want to hit https://review.openstack.org/#/c/643150/ and melwitt can get the one below
18:47:47 efried mriedem: is that reverting something?
18:47:56 mriedem yeah in part
18:47:58 mriedem revert + fix
18:48:09 mriedem it's reverting this https://review.openstack.org/#/c/643122/
18:48:29 efried k, Ima update commit message accordingly
18:48:58 openstackgerrit Eric Fried proposed openstack/nova master: Re-enable Ceph in live migration testing https://review.openstack.org/643150
18:50:07 efried mriedem: done
18:52:34 mriedem thanks
19:01:47 openstackgerrit Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899
19:36:58 edleafe python
19:37:03 edleafe doh!
19:59:18 mriedem your secret is out
20:00:11 martinkennelly Hey all, I just inherited an openstack deployment that I do not think is consuming the whole node. It is a single node deployment of OS. How can I help it consume the whole node?
20:01:36 mriedem drop reserved host config values for cpu/ram/disk, create finer grained flavors
20:01:42 mriedem really pack it tight
20:02:09 mriedem and then not be able to resize anything....
20:23:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.openstack.org/636253
20:23:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional test for cross-cell migrate with target host https://review.openstack.org/642591
20:23:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.openstack.org/642592
20:23:40 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add confirm_snapshot_based_resize_at_source https://review.openstack.org/637058
20:23:41 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add ConfirmResizeTask https://review.openstack.org/637070
20:23:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.openstack.org/637075
20:23:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.openstack.org/637316
20:23:42 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add revert_snapshot_based_resize_at_dest compute method https://review.openstack.org/637630
20:23:43 openstackgerrit Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.openstack.org/639453
20:23:43 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add finish_revert_snapshot_based_resize_at_source compute method https://review.openstack.org/637647
20:23:44 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add RevertResizeTask https://review.openstack.org/638046
20:23:44 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.openstack.org/638047
20:23:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.openstack.org/638048
20:23:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server https://review.openstack.org/638268

Earlier   Later