Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-29
22:19:25 mriedem so if the key was in specs but not the pci whitelist, i think we'd reject it because
22:19:25 mriedem pci_dev_v != v
22:20:08 mriedem also
22:20:08 mriedem pci_dev_v != v
22:20:10 mriedem oops
22:20:17 mriedem 'spec': fields.ListOfDictOfNullableStringsField(),
22:20:20 mriedem NICE!
22:20:33 mriedem ListOfSchmorgazboard
22:27:02 efried mriedem: Okay, so the stuff on L53-6 does what I said, I think. That is, if you ask for k=v in the spec, the dev in the (intersected whitelist + virt-reported) dev list must have that k=v. But any k in the dev list that's not present in the spec is ignored.
22:28:02 efried ...and I believe pci_dev is the (intersected) thingy.
22:33:38 mriedem oh yeah, extra garbage in the pci passthrough_whitelist is ignored,
22:33:44 mriedem but stuff that's in the spec must be in the whitelist
22:33:48 mriedem which is what i want
22:36:07 openstackgerrit Matt Riedemann proposed openstack/nova master: pci: don't consider case when match tags specs https://review.openstack.org/565808
22:36:08 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514
22:36:09 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: add vf_trusted field for network metadata https://review.openstack.org/566343
22:36:10 openstackgerrit Matt Riedemann proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344
22:36:11 openstackgerrit Matt Riedemann proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820
22:53:26 mriedem so, are the pci passthrough_whitelist entries supposed to be lists, or dicts, or either as long as they are serialized json?
22:54:49 mriedem i guess each entry is a serialized dict and we convert it into a list
22:55:29 mriedem er, list or dict, and can be multiples of them
23:03:56 efried mriedem: Yeah, it can be any of those things. It's super confusing.
#openstack-nova - 2018-05-30
01:14:20 openstackgerrit Merged openstack/nova master: Fix doc mistakes https://review.openstack.org/570894
02:28:17 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832
02:28:18 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
02:28:19 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401
03:42:11 openstackgerrit tianhui proposed openstack/nova master: Fix doc mistakes https://review.openstack.org/571082
05:54:42 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
06:55:30 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401
07:11:59 openstackgerrit Chris Friesen proposed openstack/nova-specs master: Add support for emulated virtual TPM https://review.openstack.org/571111
07:18:13 openstackgerrit zhaodan7597 proposed openstack/nova master: fix a bug, when creating a vmware instance from a volume,and it goes to error state, the volume still in "in use" state. https://review.openstack.org/571112
08:56:29 openstackgerrit zhaodan7597 proposed openstack/nova master: fix a bug, when creating a vmware instance from a volume,and it goes to error state, the volume still in "in use" state. Closes-Bug:#1774137 Change-Id: I987d9b46c82b66ecc0ef2453bdd7fce6d3a7810d https://review.openstack.org/571112
09:23:52 openstackgerrit tianhui proposed openstack/nova master: Fix doc mistakes https://review.openstack.org/571147
09:26:48 openstackgerrit zhaodan7597 proposed openstack/nova master: when creating a vmware instance from volume failed ,detach the volume Closes-Bug:#1774137 Change-Id: I987d9b46c82b66ecc0ef2453bdd7fce6d3a7810d https://review.openstack.org/571112
09:32:53 openstackgerrit hyunsik Yang proposed openstack/nova master: support pxe boot between the VM https://review.openstack.org/570685
10:00:44 openstackgerrit Merged openstack/nova stable/queens: Avoid unnecessary joins in HostManager._get_instances_by_host https://review.openstack.org/570083
10:01:52 openstackgerrit zhaodan7597 proposed openstack/nova master: Unable to delete volume when a vmware instance bfv is failed. https://review.openstack.org/571112
10:34:02 openstackgerrit sahid proposed openstack/nova master: pci: don't consider case when match tags specs https://review.openstack.org/565808
10:34:03 openstackgerrit sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514
10:34:04 openstackgerrit sahid proposed openstack/nova master: libvirt: add vf_trusted field for network metadata https://review.openstack.org/566343
10:34:05 openstackgerrit sahid proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344
10:34:06 openstackgerrit sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820
10:39:44 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166
10:51:32 openstackgerrit Merged openstack/nova stable/pike: Document how to disable notifications https://review.openstack.org/563254
11:03:43 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
11:03:44 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401
11:03:45 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166
11:47:30 nguyenhai_ hi guys, how can I enable again nova-compute after it is disable by "Auto-disabled due to 10 build failures"
12:04:30 Tahvok Hey guys! On Ocata here. I've enable auto-convergence on all my nova computes, and since then, doing a live migration will shutdown the instance and start it on the destination node
12:04:57 Tahvok I'm pretty sure that's not what should be happening. How should I investigate such issue?
12:22:19 openstackgerrit Merged openstack/nova master: Remove mox in tests/unit/api/openstack/compute https://review.openstack.org/566528
13:22:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Set scope for remaining placement policy rules https://review.openstack.org/571201
13:25:37 openstackgerrit hyunsik Yang proposed openstack/nova master: support pxe boot between the VM https://review.openstack.org/570685
13:25:47 stephenfin bauzas: Friendly reminder to take a look at https://review.openstack.org/#/c/540887/, if you can :)
13:25:55 stephenfin Also, no API meeting today?
13:26:24 bauzas stephenfin: LOL
13:26:44 bauzas stephenfin: I should give you a t-shirt saying 'I love RST'
13:27:00 stephenfin bauzas: And I would wear that t-shirt :)
13:31:20 efried stephenfin: Does :program: render the same as ``double backticks`` ? (It appears to under sphinx build, anyway.)
13:31:43 efried Just wondering why that was used vs using double backticks.
13:31:43 stephenfin efried: Sure does, but it also has the benefits of rendering in the glossary
13:31:56 stephenfin Which we currently ignore :(
13:31:59 efried oh, a glossary...
13:32:01 stephenfin But I'm planning to fix that
13:32:10 efried as in we don't build the glossary?
13:33:06 stephenfin We do but we don't include it anywhere, to the best of my knowledge
13:33:10 efried Looking forward to seeing what that looks like. And wondering how e.g. https://review.openstack.org/#/c/540887/3/doc/source/cli/nova-rootwrap.rst@42 will fold into it.
13:41:51 openstackgerrit Merged openstack/nova master: doc: Don't use single backticks in man pages https://review.openstack.org/540887
13:43:56 stephenfin bauzas: Ta!
13:44:12 stephenfin That was a fast merge. Guess they fixed that ignore-files rule thing for zuul.conf
13:45:05 bauzas stephenfin: you know my opinion on docs reviews
13:45:37 bauzas efried: have you seen my thread on upgrade concerns for n-rp ?
13:46:23 bauzas probably only VGPU resource class is impacted, but I guess the upgrade concern can be there for any RC once we work on NUMA
13:46:37 bauzas at least VCPU and MEMORY_MB
13:49:32 bhagyashris efried: Hi,
13:51:50 bhagyashris efried: I am still struggling to write the that functional test case as per our discussion I have made the FakeLibvirtDriver class as *subclass* of LibvirtDriver but still i am gertting the issue as File "nova/virt/fake.py", line 754, in __init__
13:51:51 bhagyashris super(FakeLibvirtDriver, self).__init__(virtapi, read_only)
13:51:51 bhagyashris File "nova/virt/libvirt/driver.py", line 314, in __init__
13:51:51 bhagyashris libvirt = importutils.import_module('libvirt')
13:51:51 bhagyashris File "/opt/stack/nova/.tox/functional/local/lib/python2.7/site-packages/oslo_utils/importutils.py", line 73, in import_module
13:51:54 bhagyashris __import__(import_str)
13:51:56 bhagyashris ImportError: No module named libvirt
13:55:23 jichen hi dansmith sorry to disturb , could you please help to approve this spec https://review.openstack.org/#/c/562154/ as it's mostly address the questions in the ML?
13:55:45 bhagyashris efried: I have moved the same functional test case under the https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_rt_servers.py#L27 but there at the time of start of compute service it's not creating the compute node resource provider in placement db
13:56:10 dansmith jichen: I will leave that to someone else, but I'm replying to your question on the ML right now
13:57:29 jichen dansmith: ok, then I will ask someone else to review and approve it , as it got some +1 and one +2 already
13:57:37 dansmith ack
13:59:20 jichen dansmith: and could you please check this -2 on https://review.openstack.org/#/c/523387/ ? it's been reviewed by some core and non-cores, will there be some criteria to remove the -2 so I can get more core to help (efried has reviewed the patch and it's dependency patches previously thoroughly)
14:00:09 dansmith jichen: that -2 is just to hold the base patch until that patch and the rest have +2+W on them so we don't land a driver with no function
14:00:25 dansmith jichen: so yes, I'll drop that procedural -2 when the approvals are stacked up behind it
14:00:48 dansmith I said that on the patch, so it shouldn't be preventing anyone from reviewing it
14:00:53 jichen dansmith: ok, I will get more people to review and have +2 +w on them then ping you later, thanks a lot
14:00:57 dansmith yep
14:05:12 mriedem sahid: some comments in your neutron docs patch for trusted vf https://review.openstack.org/#/c/571167/
14:06:38 mriedem dansmith: just noticed this https://review.openstack.org/#/c/570685/ above, but wasn't there something similar that bnemec did for tripleo?
14:07:11 dansmith I think he proposed something
14:08:34 bhagyashris kashyap: any idea about ^
14:09:52 kashyap bhagyashris: Sorry, which one?
14:11:15 openstackgerrit Moshe Levi proposed openstack/nova stable/pike: Don't overwrite binding-profile https://review.openstack.org/563501
14:11:55 mriedem so something that's been bothering me about the FFU sessions which i don't think anyone has brought up,

Earlier   Later