Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-04
22:48:29 artom o/
23:27:06 openstackgerrit Merged openstack/nova master: doc: link Kashyap's cpu model talk to the libvirt driver config docs https://review.openstack.org/634731
23:33:10 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891
23:33:11 openstackgerrit Matt Riedemann proposed openstack/nova master: Move resize.(start|end) notification sending to helper method https://review.openstack.org/634831
23:33:11 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.openstack.org/634832
#openstack-nova - 2019-02-05
00:35:16 jackding cfriesen: need your opinion on https://review.openstack.org/#/c/603844/
01:37:54 openstackgerrit Merged openstack/nova master: Reject evacuate with port having resource request https://review.openstack.org/630724
02:05:23 openstackgerrit Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for cyborg access https://review.openstack.org/631242
02:05:23 openstackgerrit Sundar Nadathur proposed openstack/nova master: WIP: Add Cyborg device profile groups to spec obj. https://review.openstack.org/631243
02:05:24 openstackgerrit Sundar Nadathur proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.openstack.org/631244
02:05:24 openstackgerrit Sundar Nadathur proposed openstack/nova master: WIP: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.openstack.org/631245
02:17:19 openstackgerrit Merged openstack/nova master: Add support for vrouter HW datapath offloads https://review.openstack.org/572082
04:16:16 openstackgerrit Michael Still proposed openstack/nova master: Move setting mac addresses for network devices to privsep. https://review.openstack.org/621529
04:16:16 openstackgerrit Michael Still proposed openstack/nova master: Move interface disabling to privsep. https://review.openstack.org/622150
04:16:17 openstackgerrit Michael Still proposed openstack/nova master: Move binding ips to privsep. https://review.openstack.org/622151
04:16:17 openstackgerrit Michael Still proposed openstack/nova master: create_veth_pair is unused, remove it. https://review.openstack.org/624226
04:16:18 openstackgerrit Michael Still proposed openstack/nova master: Create specialist set_macaddr_and_vlan helper. https://review.openstack.org/624227
04:16:18 openstackgerrit Michael Still proposed openstack/nova master: Move create_tap_dev into privsep. https://review.openstack.org/624228
04:16:19 openstackgerrit Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to be with its only caller. https://review.openstack.org/624229
04:16:19 openstackgerrit Michael Still proposed openstack/nova master: Move DHCP releasing to privsep. https://review.openstack.org/624230
04:16:20 openstackgerrit Michael Still proposed openstack/nova master: Convert additional IP management calls to privsep. https://review.openstack.org/624231
04:16:20 openstackgerrit Michael Still proposed openstack/nova master: Move route management to privsep. https://review.openstack.org/624232
04:16:21 openstackgerrit Michael Still proposed openstack/nova master: Move additional IP address management to privsep. https://review.openstack.org/624591
04:16:21 openstackgerrit Michael Still proposed openstack/nova master: Move slight bonkers IP management to privsep. https://review.openstack.org/624592
04:16:22 openstackgerrit Michael Still proposed openstack/nova master: Move arping to privsep. https://review.openstack.org/624593
04:16:22 openstackgerrit Michael Still proposed openstack/nova master: Move cleaning conntrack to privsep. https://review.openstack.org/624594
07:20:24 openstackgerrit Yury Kulazhenkov proposed openstack/nova master: rename ScaleIO driver to VxFlex Os https://review.openstack.org/634866
09:59:39 tinwood Morning. I'm still waiting, please, for somebody to nay/yay on my very small nova review: https://review.openstack.org/#/c/633279/ Many thanks.
10:54:52 openstackgerrit Balazs Gibizer proposed openstack/nova master: Calculate RequestGroup resource provider mapping https://review.openstack.org/616239
10:54:52 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fill the RequestGroup mapping during schedule https://review.openstack.org/619528
10:54:53 openstackgerrit Balazs Gibizer proposed openstack/nova master: Pass resource provider mapping to neutronv2 api https://review.openstack.org/616240
10:54:53 openstackgerrit Balazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule https://review.openstack.org/619529
10:54:54 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459
10:54:54 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
10:54:55 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
10:54:55 openstackgerrit Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310
10:54:56 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311
10:54:56 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
11:51:47 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657
11:59:11 openstackgerrit Stephen Finucane proposed openstack/nova master: API: Remove evacuate/live-migrate 'force' parameter https://review.openstack.org/634600
11:59:12 openstackgerrit Stephen Finucane proposed openstack/nova master: Don't force evacuate/live migrate in notification sample tests https://review.openstack.org/634920
13:01:41 openstackgerrit Merged openstack/nova master: Follow up for per-instance serial number change https://review.openstack.org/634743
13:01:57 openstackgerrit Jan Gutter proposed openstack/nova master: Ignore some PendingDeprecationWarnings for os-vif https://review.openstack.org/634928
13:10:00 jangutter mriedem: ^ apologies for contributing to g8h8 2019
13:18:21 mriedem https://blueprints.launchpad.net/nova/+spec/libvirt-neutron-sriov-livemigration and https://blueprints.launchpad.net/nova/+spec/handling-down-cell are now in runway slots
13:23:09 openstackgerrit Merged openstack/nova master: Move setting mac addresses for network devices to privsep. https://review.openstack.org/621529
13:23:16 openstackgerrit Merged openstack/nova master: Move interface disabling to privsep. https://review.openstack.org/622150
13:45:52 openstackgerrit Sundar Nadathur proposed openstack/nova master: WIP: Add Cyborg device profile groups to spec obj. https://review.openstack.org/631243
13:45:53 openstackgerrit Sundar Nadathur proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.openstack.org/631244
13:45:53 openstackgerrit Sundar Nadathur proposed openstack/nova master: WIP: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.openstack.org/631245
13:48:42 openstackgerrit Merged openstack/nova master: Reject unshelve with port having resource request https://review.openstack.org/630725
14:14:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop nova-multiattach job https://review.openstack.org/606981
14:15:08 mriedem jaypipes: next 2 in the bw provider series are awaiting your loving embrace https://review.openstack.org/#/c/567268/
14:16:34 gibi_off mriedem, jaypipes: thank your for the reviews!
14:17:10 cdent gibi_off: I hope you're feeling well, or at least headed that way
14:19:57 gibi_off cdent: thanks, I'm pretty OK
14:21:19 gibi_off cdent: I still have some extra medical steps ahead but so far so good
14:21:32 cdent glad to hear it, good luck with it all
14:21:36 gibi_off cdent: thanks
14:22:01 gibi_off I think I will remove the _off postfix from next week
14:22:40 cdent whatever you do, don't let openstack rush things
14:23:31 gibi_off I use openstack (and software dev in general) as something agains boredom
14:32:37 efried tinwood: I left a +2. I'm hoping the second comes from someone like dansmith who knows how to spell OVO
14:33:05 tinwood efried, thanks very much.
14:34:55 efried You're welcome, fwiw :)
14:38:20 dansmith I commented
14:38:47 dansmith I'm not really in favor of tracking every possible out of tree driver in our objects
14:38:59 dansmith so I'll let someone else +2 that
14:41:29 efried dansmith: An alternative would be to make that a string field so drivers can put their name in there without having to exist in an enum. Is that enum used for anything else anywhere ever?
14:41:59 dansmith yeah I dunno why it's an enum, and don't know where it's used
14:42:14 efried I looked yesterday and couldn't find anywhere other than in the diagnostics object.
14:42:20 efried I think
14:42:38 efried yeah
14:47:02 bauzas afternoon, folks
14:47:23 bauzas here and now, I begin my upstream day
14:47:59 efried \o/
14:48:04 bauzas any fancy reviews to make ?
14:49:41 efried dansmith, tinwood: I looked at the in-tree usages of get_instance_diagnostics, and they're all using a string to set that field; none of them are using the actual enum class. So changing that to a string field would require no changes *in tree*. Out of tree, I suppose it would be harmless if we kept the enum around (deprecated) and let folks know to transition to using a plain string.
14:50:13 tinwood efried, I tried that, but it errored out as the field checks the enum for the string.
14:50:30 tinwood That's how I found I needed to update the enum.
14:50:53 efried tinwood: You tried just using a string in your oot get_instance_diagnostics impl?
14:52:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop the integrated-gate (py27) template https://review.openstack.org/634949
14:53:07 tinwood efried, yes, I wouldn't have changed nova except it wouldn't accept 'lxd'. The equivalent change in nova-lxd needing it is a string: https://review.openstack.org/#/c/633270/3/nova/virt/lxd/driver.py
14:53:38 tinwood efried, line 1175
14:53:44 efried Yeah, so that's what I'm saying. Instead of augmenting the enum every time some OOT (or in-tree for that matter) driver needs to implement get_instance_diagnostics, let's just change the field in the Diagnostics object to a StringField. We can deprecate the HypervisorDriver enum (or not, I guess it doesn't really matter if drivers want to use it to get their string). That way, any impl - in tree or out - can use any string they l
14:53:44 efried without having to dork with this enum.
14:54:33 tinwood efried, ah, I see - well, it would help me ... :)
14:54:44 efried I mean, I get the value in having a discrete set of values allowed in that field, but I don't think it's probably worth the hassle in this case.
14:55:05 efried dansmith: would you be supportive of such a change?
14:55:47 dansmith efried: we should ask whoever added that
14:55:56 dansmith but this falls into the realm of my previous meh on the topic
14:56:09 efried sergei nitikin
14:56:18 efried nikitin
14:56:25 efried blueprint restore-vm-diagnostics
14:57:48 cdent mriedem: to avoid a pointless "woot, yeah" response to your recent email, I say "woot, yeah" here.
14:57:50 cdent woot, yeah
14:58:49 efried tinwood: ^
14:59:06 tinwood efried, taking a look
14:59:14 mriedem cdent: woot acknowledged
14:59:46 efried The spec doesn't say it's an enum. It says it's a string. The enum-ness must have been decided during impl time.

Earlier   Later