Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-26
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_NUMA_VERSION_PPC https://review.openstack.org/564010
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:09 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012
11:02:09 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: Drop BAD_LIBVIRT_NUMA_VERSIONS https://review.openstack.org/564011
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 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: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: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
12:17:36 mriedem about vif type changing
12:18:01 mriedem remember we talked about whether we could just ignore the original source_vif and just get a new vif config based on the destination vif to generate the new xml *except* for the pci address?
12:18:22 mriedem the problem with that is, the source_vif contains the 'network' info in the vif, which is what we need for things like source bridge and target dev
12:18:36 mriedem so can we even have vif type changes for a vif on the same network?
12:19:54 mriedem sean-k-mooney[m]: what i really need is a 2-node deployment where i've got ovs on one node and linuxbridge on another, and live migrate between them, but getting that setup would probably take me awhile - i'm not sure how easily devstack can be configured to tell the subnode to use linuxbridge when the controller is configured for ovs (but doesn't neutron server by default support ML2 ovs and LB agents?)
12:28:18 openstackgerrit Matthew Booth proposed openstack/nova master: Simplify BDM boot index checking https://review.openstack.org/564480
12:28:25 kashyap mriedem: Re-thinking on your comment here on the confusion of `diff`s: https://review.openstack.org/#/c/563926/3/nova/tests/unit/virt/libvirt/test_driver.py@6418
12:28:49 kashyap mriedem: I added it in the existing way because it keeps all the *_custom* tests together
12:29:10 mdbooth mriedem: ^^^ it was a simple bug. Refactor remains worth it imho, as the old code is mind-bendingly obtuse.
12:29:51 kashyap So someone reading the test along will feel less "cognitive load" if I don't randomly add the new '_multiple_extra_flags' test _after_ '_host_model_with_extra_flags'
12:31:50 mriedem mdbooth: ack
12:33:06 kashyap mriedem: Disregard me until the new iteration passes tests; I still have rearranging to do.
12:39:28 openstackgerrit Martin Midolesov proposed openstack/nova master: Validating video ram against the max video ram allowed https://review.openstack.org/564193
12:51:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334
12:51:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423
12:51:30 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370
12:51:31 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: compute: use port binding extended API during live migration https://review.openstack.org/551371
12:51:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870
12:51:33 openstackgerrit Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537
13:18:04 Tahvok Hey guys! Is it possible to request a release for ocata branch?
13:18:24 mriedem Tahvok: i think that's already up
13:18:35 mriedem https://review.openstack.org/#/c/564044/
13:19:40 Tahvok Oh, nice! I was waiting for some fix to be realesed for a couple of months now :)
13:19:44 Tahvok Thanks!
13:21:14 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926
13:26:20 mordred mriedem, jaypipes: how hard/expensive would it be to return a total count of objects for paginated collections?
13:27:32 mordred (just got a question about being able to get a count of items without having to loop over the iterator in the sdk - but we can't do it because pagination... thought I'd check to see how insane it would be to get that added
13:29:30 cdent mordred: you're after a 'total' bit of metadata?
13:30:07 mordred cdent: yah. atleast, maybe I'm after a total bit of metadata
13:30:42 mordred cdent: I'm honestly not sure how muchit's a thing I want - but cost of implementation probably impacts how much I want it :)
13:31:15 cdent I know how that can be. I'm afrait that I'm not familiar with the pagination handling in nova's api, so can't provide any quick and dirty guess

Earlier   Later