Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-03-14
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
20:23:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353
20:23:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Add cross-cell resize policy rule and enable in API https://review.openstack.org/638269
20:23:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.openstack.org/643450
20:23:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.openstack.org/643451
20:30:34 mriedem sob, time to rebase
20:30:37 mriedem 44 changes
20:34:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1818914 https://review.openstack.org/641521
20:34:33 openstack bug 1818914 in OpenStack Compute (nova) "Hypervisor resource usage on source still shows old flavor usage after resize confirm until update_available_resource periodic runs" [Low,In progress] https://launchpad.net/bugs/1818914 - Assigned to Matt Riedemann (mriedem)
20:34:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove unused context parameter from RT._get_instance_type https://review.openstack.org/641792
20:34:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Update usage in RT.drop_move_claim during confirm resize https://review.openstack.org/641806
20:34:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix ProviderUsageBaseTestCase._run_periodics for multi-cell https://review.openstack.org/641179
20:34:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Improve CinderFixtureNewAttachFlow https://review.openstack.org/639382
20:34:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
20:34:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
20:34:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.openstack.org/631123
20:34:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Add TargetDBSetupTask https://review.openstack.org/627892
20:34:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.openstack.org/631581
20:34:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask https://review.openstack.org/633853
20:34:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method https://review.openstack.org/621313
20:34:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_dest compute method https://review.openstack.org/633293
20:34:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add PrepResizeAtDestTask https://review.openstack.org/627890
20:34:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.openstack.org/634832
20:34:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova.compute.utils.delete_image https://review.openstack.org/637605
20:34:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Add PrepResizeAtSourceTask https://review.openstack.org/627891
20:34:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor ComputeManager.remove_volume_connection https://review.openstack.org/642183
20:34:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.openstack.org/639453
20:34:53 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add finish_revert_snapshot_based_resize_at_source compute method https://review.openstack.org/637647
20:34:53 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add RevertResizeTask https://review.openstack.org/638046
20:34:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.openstack.org/638047
20:34:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.openstack.org/638048
20:34:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server https://review.openstack.org/638268
20:34:56 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353

Earlier   Later