Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-26
08:40:46 kashyap I know Matt is right "strictly speaking", this isn't a "bug": https://bugs.launchpad.net/nova/+bug/1766208
08:40:46 openstack Launchpad bug 1766208 in OpenStack Compute (nova) "Lift the restriction on choices for `cpu_model_extra_flags` config attribute" [Undecided,Invalid] - Assigned to Kashyap Chamarthy (kashyapc)
08:40:57 kashyap But I filed it as a mostly "item tracker"
08:41:19 kashyap Maybe it's a habit out of dealing with Bugzilla
08:42:03 openstackgerrit Stephen Finucane proposed openstack/nova master: Handle deprecation of inspect.getargspec https://review.openstack.org/530777
09:02:59 bauzas cores, easy +W for a High priority backport potential bugfix https://review.openstack.org/#/c/564257/
09:03:08 bauzas gibi, stephenfin: ^
09:03:30 stephenfin bauzas: Um, I already did that?
09:03:44 bauzas oops my bad
09:03:57 stephenfin Some people are so ungrateful
09:04:20 bauzas stephenfin: <3
09:04:40 bauzas stephenfin: I can spoil you InfinityWars if you want
09:04:53 stephenfin bauzas: I'm pretty sure I know where you live
09:09:46 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:09:47 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:14:52 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:14:53 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:15:57 bauzas stephenfin: /me is afraid
09:17:23 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:17:24 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:27:51 stephenfin How to confuse oneself: fire up 'tox -e py35' that was recreated a while back and is missing dependencies. Try to figure out what's going on from the wall of text
09:27:58 stephenfin That's got to be a bug with stestr
09:40:40 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
09:40:41 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:40:42 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:49:10 lyarwood stephenfin: thanks again
10:08:30 openstackgerrit Merged openstack/nova master: libvirt: fix hard reboot issue with mdevs https://review.openstack.org/564257
10:23:28 stephenfin patchbomb incoming
10:23:51 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add NUMANetworkTopology object https://review.openstack.org/564439
10:23:52 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
10:23:53 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.networks field https://review.openstack.org/564441
10:23:55 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add NetworkRequest.physnet, tunnelled https://review.openstack.org/564442
10:23:56 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Unchain _get_phynet_info from _get_port_vnic_info https://review.openstack.org/564443
10:23:57 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Add 'populate_networks' to network API https://review.openstack.org/564444
10:23:58 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Add '_get_network_tunnel_status' https://review.openstack.org/564445
10:23:59 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Add network_requests to instance_extra https://review.openstack.org/564446
10:24:00 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add Instance.network_requests https://review.openstack.org/564447
10:24:01 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Start accounting for network requests https://review.openstack.org/564448
10:24:02 openstackgerrit Stephen Finucane proposed openstack/nova master: rt: Start utilizing Instance.network_requests https://review.openstack.org/564449
10:24:03 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add RequestSpec.requested_networks https://review.openstack.org/564450
10:24:04 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Remove unnecessary default parameter https://review.openstack.org/564451
10:24:05 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing network_requests https://review.openstack.org/564452
10:24:47 stephenfin gibi, bauzas, jaypipes, sean-k-mooney[m]: voilà ^
10:30:39 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: fix hard reboot issue with mdevs https://review.openstack.org/564454
11:02:07 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION https://review.openstack.org/563984
11:02:07 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC https://review.openstack.org/564010
11:02:09 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: Drop BAD_LIBVIRT_NUMA_VERSIONS https://review.openstack.org/564011
11:02:09 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012
11:02:10 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD https://review.openstack.org/564013
11:04:08 kashyap Darn, one more time
11:04:15 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION https://review.openstack.org/563984
11:04:16 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC https://review.openstack.org/564010
11:04:17 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_NUMA_VERSIONS https://review.openstack.org/564011
11:04:18 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012
11:04:19 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD https://review.openstack.org/564013
11:11:02 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Fix dropped check for boot_index 0 in _validate_bdm https://review.openstack.org/564461
11:18:21 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: xenapi: handle InstanceNotFound in detach_interface() https://review.openstack.org/564468
11:36:11 efried ō/
11:36:31 mriedem efried: ha, that inspect.getargspec warnings filter explodes on anything using mox
11:36:59 efried mriedem: Niiice. I didn't look too closely at the failures.
11:37:15 efried mriedem: We should... get rid of mox.
11:38:18 mriedem - Failed: 718
11:39:29 mriedem https://github.com/openstack/mox3/blob/285bc2a1760d460080df79f1ed3b0988cab44f7b/mox3/mox.py#L909
11:39:37 mriedem could patch mox3
11:40:42 kashyap efried: Hehe, unrelated: I saw you comment about the typo: "Teset" -- https://review.openstack.org/#/c/554438/3/nova/tests/unit/test_utils.py
11:40:50 kashyap efried: That is ... much better than the typo mriedem makes :P
11:40:56 kashyap (You know what it is.)
11:41:02 mriedem testes
11:41:11 kashyap Indeed, :D
11:41:54 efried We *could* patch mox3 -- to use mock.
11:42:11 efried surely somebody's done that somewhere.
11:42:13 mriedem it looks like it's usage is best effort and optional
11:42:25 mriedem for inspect.getargspec
11:42:41 mriedem so easy thing would just be adding an "if six.PY3: raise ValueError()"
11:42:57 mriedem anywho, the warnings filter is low priority now
11:43:20 efried Yeah, totally. But leaving the patch open is goodness. And we can revisit it when mox is all gone.
11:43:38 efried takashin has been doing a good job there.
11:44:32 kashyap mriedem: Morning, meta comment: on your remark here, if you say: "I insist", I'll close it as you prefer. I'm normally mindful of it, but in this instance I'd prefer to track it via LP bug too: https://bugs.launchpad.net/nova/+bug/1766208
11:44:32 openstack Launchpad bug 1766208 in OpenStack Compute (nova) "Lift the restriction on choices for `cpu_model_extra_flags` config attribute" [Undecided,In progress] - Assigned to Kashyap Chamarthy (kashyapc)
11:44:50 kashyap See my last comment, before you say the phrase :-)
11:45:17 kashyap And thanks for the actual substantial review, though! _That_ appreciate the most
11:45:34 mriedem kashyap: i insist
11:46:13 kashyap Heh, okay; moooooooving on.
11:51:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle deprecation of inspect.getargspec https://review.openstack.org/530777
11:54:18 mriedem kashyap: i think you can throw https://review.openstack.org/#/c/563926/ into runways
11:54:25 mriedem pickings in the queue are starting to get slim
11:54:30 mriedem for non-blocked things
11:55:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Add a warnings filter for inspect.getargspec usage https://review.openstack.org/564343
11:59:27 kashyap mriedem: Hi, will click in a min; making some quick late lunch
12:00:07 kashyap mriedem: Do you have a (non-null) pointer as to how I can "throw it into a runway"? If you have to look up too, disregard; I'll get to it
12:00:34 efried kashyap: https://etherpad.openstack.org/p/nova-runways-rocky
12:00:50 efried kashyap: Add it at the bottom of the queue (L89)
12:01:56 kashyap Gracias
12:02:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove explicit instance.info_cache.delete() https://review.openstack.org/563689
12:03:32 kashyap Done
12:03:35 kashyap Thanks, Eric!
12:05:01 efried yahyoubetcha
12:15:22 kashyap Has anyone here tinkered with modifying the 'post-config' for Nova here: http://git.openstack.org/cgit/openstack-dev/devstack/tree/.zuul.yaml#n281
12:17:15 mriedem sean-k-mooney[m]: i think https://review.openstack.org/#/c/551370/ is ready for your review. i've confirmed via the live migration CI run in the patch at the top of the series that enables this code that it's hitting _update_vif_xml correctly and linked the logs inline
12:17:32 mriedem sean-k-mooney[m]: however, note the TODO i have in here https://review.openstack.org/#/c/551370/19/nova/virt/libvirt/migration.py@252

Earlier   Later