| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-30 | |||
| 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 | stephenfin | efried: Sure does, but it also has the benefits of rendering in the glossary | |
| 13:31:43 | efried | Just wondering why that was used vs using double backticks. | |
| 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 | File "/opt/stack/nova/.tox/functional/local/lib/python2.7/site-packages/oslo_utils/importutils.py", line 73, in import_module | |
| 13:51:51 | bhagyashris | libvirt = importutils.import_module('libvirt') | |
| 13:51:51 | bhagyashris | File "nova/virt/libvirt/driver.py", line 314, in __init__ | |
| 13:51:51 | bhagyashris | super(FakeLibvirtDriver, self).__init__(virtapi, read_only) | |
| 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, | |
| 14:12:07 | mriedem | besides just saying, the config options churn a lot, | |
| 14:12:07 | bhagyashris | kashyap: actually i have written functional test here: https://review.openstack.org/#/c/560459/11/nova/tests/functional/test_servers.py the patch is having libvirt related changes but in the functional test case i have not initialized the actual libvirt driver | |
| 14:12:34 | mriedem | is that for the upgrade support tag, we can't drop config options within a release, but we can across boundaries, | |
| 14:12:45 | mriedem | which includes dropping support for deprecated config option names, | |
| 14:13:09 | mriedem | sometimes we leave those deprecated config option aliases around for much more than 1 release, like pci_passthrough_whitelist and the scheduler_default_filters | |
| 14:13:17 | bhagyashris | kashyap: so as per the review comments I am trying to initialized the actual libvirt driver so that the change in code will be tested but i am facing some issues | |
| 14:13:24 | mriedem | i'm kind of surprised the FFU folk haven't pushed for config option drop/renames to last more than 1 release | |
| 14:13:58 | mriedem | lyarwood: have you heard anything like ^ in your FFU travels? | |
| 14:14:09 | kashyap | bhagyashris: If you don't mind, I'm in the middle of something. This requires me to see all the context, will get to it tonight. | |
| 14:15:11 | bhagyashris | kashyap: ok np but i am not sure will be present because i am working in IST time zone so here it's almost EOD | |
| 14:15:27 | mriedem | e.g. pci_passthrough_whitelist was deprecated in ocata https://review.openstack.org/#/c/356604/ and renamed to [pci]/passthrough_whitelist, certainly that's a fair enough amount of time to drop the old deprecated name | |
| 14:15:43 | kashyap | bhagyashris: I'll comment on the review tomm. I need to be AFK soon | |
| 14:16:16 | bhagyashris | kashyap: ok thank you :) | |
| 14:18:40 | lyarwood | mriedem: yeah some ops/diy folks have asked for that but we (TripleO) don't need it, heat/puppet takes care of everything for us under the covers. | |
| 14:20:02 | mriedem | i imagine the pain is really just needing to read through the release notes to see which config options are deprecated/dropped each release | |
| 14:20:10 | mriedem | and digesting that across all projects per release you're FFing through | |
| 14:23:18 | lyarwood | mriedem: yeah, something this group is trying to address for ops/diy folks AFAIK - https://www.openstack.org/videos/sydney-2017/making-fast-forward-upgrade-openstack-possible-with-configuration-mapping-tool | |
| 14:25:18 | lyarwood | https://review.openstack.org/#/c/526314/ maybe? | |
| 14:31:38 | mriedem | yup i heard about that for the first time at the summit | |
| 14:38:45 | efried | bhagyashris: Looks like the libvirt library isn't a [test-]requirement. If you install it into your functional venv, you should be able to get around that import error. | |
| 14:39:08 | openstackgerrit | Jay Pipes proposed openstack/nova master: mirror nova host aggregate members to placement https://review.openstack.org/553597 | |
| 14:39:25 | mriedem | jaypipes: figured out massimo's cold migrate issue in the ML | |
| 14:39:34 | efried | bhagyashris: As for the second thing, not sure I can help without seeing the code. Can you upload your patch set? | |