| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-05 | |||
| 10:00:36 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911 | |
| 10:17:55 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | |
| 10:27:44 | openstackgerrit | Matthew Booth proposed openstack/nova: Port libvirt.test_firewall to Python 3 https://review.openstack.org/403423 | |
| 10:27:45 | openstackgerrit | Matthew Booth proposed openstack/nova: Port libvirt.test_vif to Python 3 https://review.openstack.org/403419 | |
| 10:28:59 | mdbooth | sfinucan: Thanks for the +2s yesterday. Just rebased and fixed test-py3 merge conflicts in ^^^. I've also made one depend on the other so it doesn't happen again. | |
| 10:30:07 | sfinucan | mdbooth: Sweet. I'll take a look again shortly | |
| 10:30:12 | mdbooth | sfinucan: Thanks | |
| 10:41:36 | openstackgerrit | Jiri Suchomel proposed openstack/nova: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | |
| 11:44:26 | openstackgerrit | Rafael Folco proposed openstack/nova: DELETE all inventory for a resource provider https://review.openstack.org/416669 | |
| 11:49:24 | cdent | rfolco: things going okay with ^ ? | |
| 11:49:46 | sfinucan | claudiub: I have that vNUMA patch reviewed again. If you get a chance, could you take a shot at updating this page [1] in openstack-manuals to reflect the new functionality? [1] http://docs.openstack.org/admin-guide/compute-cpu-topologies.html | |
| 11:50:38 | sfinucan | ...assuming any of the HyperV stuff in documented in that guide | |
| 11:51:08 | rfolco | cdent, minor issue with reno, another one with latest microversion test, fixed. waiting test results to see if all good now | |
| 11:51:20 | sfinucan | claudiub: I'd guess you'd want to update the feature matrix too. johnthetubaguy might have some input on what you should do there | |
| 11:51:22 | rfolco | cdent, thanks for your great assistance | |
| 11:51:49 | cdent | you're very welcome, thanks for picking up the work. the more people involved the better. | |
| 11:52:41 | claudiub | sfinucan: hi. thanks for reviewing. :) there isn't any hyper-v documentation in the docs about vnuma, since it wasn't merged yet. will start working on it as soon as i can. | |
| 11:54:38 | sfinucan | claudiub: Sure thing. Might be worth asking sahid to take a look at that too | |
| 11:57:55 | openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce basic framework for security proxying https://review.openstack.org/345396 | |
| 11:57:56 | openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 11:57:56 | openstackgerrit | Stephen Finucane proposed openstack/nova: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 11:57:57 | openstackgerrit | Stephen Finucane proposed openstack/nova: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 12:02:04 | frickler | wznoinsk: Intel NFV CI looks unstable, see e.g. https://review.openstack.org/416822 | |
| 12:11:55 | gibi | diga: I'm still here | |
| 12:17:25 | openstackgerrit | Balazs Gibizer proposed openstack/nova: add availability_zone field in service.update https://review.openstack.org/415857 | |
| 12:18:12 | sfinucan | johnthetubaguy: Floating IP quotas only apply when using nova-network, right? | |
| 12:19:15 | johnthetubaguy | sfinucan: I believe so | |
| 12:19:35 | johnthetubaguy | sfinucan: there might be a config for that... | |
| 12:19:59 | sfinucan | johnthetubaguy: There is, as part of https://review.openstack.org/#/c/346869/10 | |
| 12:20:16 | sfinucan | which unfortunately for poor, long-suffering macsz got in a merge conflict | |
| 12:21:03 | johnthetubaguy | sfinucan: honestly, I recon we could move that one, its can't get removed till will kill cells v1 | |
| 12:21:24 | diga_ | gibi: Hi | |
| 12:21:31 | diga_ | sorry got disconnected | |
| 12:21:46 | diga_ | gibi: you still there ? | |
| 12:21:57 | sfinucan | True. What about the 'driver' one? | |
| 12:22:47 | johnthetubaguy | sfinucan: well I don't quite get why we deprecated that I suppose, I guess we will kill the no op driver, but I suspect folks use that | |
| 12:22:53 | gibi | diga_: yes. how can I help you? | |
| 12:23:18 | diga_ | gibi: jaypipes told me to work with you on nova-placement-notifications | |
| 12:23:35 | johnthetubaguy | sfinucan: I thought it was just going to move to stevedore, in which case, the setting remains, its just we do a fallback on the values used | |
| 12:24:06 | diga_ | gibi: just want to know, what is remaining, what should take up on first priority | |
| 12:24:13 | sfinucan | johnthetubaguy: Looks like the work of dansmith (https://github.com/openstack/nova/commit/43063888) | |
| 12:24:33 | gibi | diga_: I think nothing was done on the notification side in the placement work | |
| 12:24:36 | johnthetubaguy | yeah, was just digging into that | |
| 12:24:58 | gibi | diga_: one I saw a WIP patch from cdent | |
| 12:25:05 | gibi | diga_: let me dig up a link | |
| 12:25:09 | diga_ | gibi: ok | |
| 12:25:12 | johnthetubaguy | sfinucan: I mean if you set everything to -1, you get the same behaviour as no-op I think | |
| 12:25:16 | openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Deprecate yet another nova-net option https://review.openstack.org/416959 | |
| 12:25:20 | cdent | diga_, gibi : https://review.openstack.org/#/c/308503/ | |
| 12:25:24 | cdent | there's very little there | |
| 12:25:30 | johnthetubaguy | sfinucan: I totally approved the change, just looked odd at a second glance | |
| 12:26:04 | diga_ | cdent: yes, this is one you posted in the meeting | |
| 12:26:05 | cdent | diga_: feel free to take that over, or ignore it, whichever seems best | |
| 12:26:07 | gibi | cdent: thanks, that one :) | |
| 12:26:11 | sfinucan | johnthetubaguy: Ah, so we only have two "plugins" and one of them can do the work of the other? Definitely worth killing then | |
| 12:26:18 | johnthetubaguy | sfinucan: tempting to not move the driver one, given its needed rework | |
| 12:26:22 | diga_ | cdent: sure | |
| 12:26:38 | johnthetubaguy | sfinucan: well, one is much slower at doing nothing, I presume, but I don't know so | |
| 12:26:46 | diga_ | cdent: just want to know how to run placement-api seperately | |
| 12:26:48 | gibi | diga_: also jaypipes drafted the blueprint with some TODO list https://blueprints.launchpad.net/nova/+spec/placement-notifications | |
| 12:26:58 | johnthetubaguy | sfinucan: I suspect its only a test hang over thing | |
| 12:27:01 | diga_ | gibi: yes, he assigned to me | |
| 12:27:04 | sfinucan | johnthetubaguy: I can't see why someone would want to disable quotas entirely, tbh | |
| 12:27:06 | sfinucan | Yes, exactly | |
| 12:27:34 | gibi | diga_: based on that bp, the next thing is a small spec describing what notifications the placement service wants to emit | |
| 12:28:00 | diga_ | gibi: okay | |
| 12:28:13 | gibi | diga_: do you need a link for the spec template? | |
| 12:28:20 | cdent | diga_: What do you mean by "separately"? It is automatically enabled in devstack if nova is enabled and runs under mod_wsgi. Do you want to run it without devstack? | |
| 12:28:27 | diga_ | gibi: yeah, will be helpful | |
| 12:28:37 | sfinucan | Hmm, OK I hold off on a reworked version that doesn't move driver | |
| 12:28:48 | diga_ | cdent: I just see n-api in devstack | |
| 12:28:59 | diga_ | cdent: is that all ? | |
| 12:29:06 | sfinucan | I also have this to deprecate that floating IPs opt so perhaps we shouldn't move that too (but I don't care so much about this one) https://review.openstack.org/#/c/416959/ | |
| 12:29:12 | gibi | diga_: here is the template you need to fill https://github.com/openstack/nova-specs/blob/master/specs/pike-template.rst | |
| 12:29:23 | diga_ | gibi: okay | |
| 12:29:58 | cdent | diga: the service is called placement-api and it will be enabled unless it is explicitly turned off | |
| 12:29:59 | gibi | diga_: then push the spec up for review in the nova-specs repo | |
| 12:30:01 | diga_ | gibi: will go through it first | |
| 12:30:28 | diga_ | okay | |
| 12:30:31 | diga_ | cdent: okay | |
| 12:31:43 | diga_ | gibi: will submit the spec soon | |
| 12:32:08 | diga_ | gibi: I think it is necessary to submit the spec spec & then start the implementation | |
| 12:32:11 | diga_ | ? | |
| 12:32:26 | sfinucan | mdbooth: +2'd | |
| 12:33:16 | gibi | diga_: you can start the implementation as early as you would like but you can only expect code review on your implementation after the related spec is approved | |
| 12:33:40 | diga_ | gibi: ohh..kk | |
| 12:33:59 | diga_ | gibi: got it, will start on spec first then | |
| 12:33:59 | gibi | diga_: you don't have to rush as nova folks are pretty busy finishing Ocata these days | |
| 12:34:28 | gibi | diga_: cool | |
| 12:34:33 | diga_ | gibi: yep | |
| 12:37:19 | openstackgerrit | Stephen Finucane proposed openstack/os-vif: vif_plug_ovs: Always set MTU when plugging devices https://review.openstack.org/408779 | |
| 12:38:56 | sfinucan | johnthetubaguy: Ah, mriedem already pipped us to the mark https://review.openstack.org/#/c/410996/ | |
| 12:39:35 | gibi | diga_: here are two example spec about adding new notifications https://review.openstack.org/#/c/321336/ and https://review.openstack.org/#/c/340715/ | |
| 12:40:34 | diga_ | gibi: Thanks for sharing this, this is very helpful :) | |
| 12:41:43 | diga_ | gibi: will go through it, add it accordingly | |
| 12:41:47 | diga_ | gibi: brb | |
| 12:46:44 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571 | |
| 13:07:07 | openstackgerrit | Chris Dent proposed openstack/nova: [WIP] Structure for simply managing placement-api-ref https://review.openstack.org/409340 | |
| 13:08:00 | cdent | sdague: if you get a chance to look at that ^ to sanity check that would be awesome. It's not done, but I want to make sure I'm not heading down a bad path. You'll also likely have some insight on how to make the testing it tries to do happen in the right place. | |
| 13:12:57 | sdague | cdent: looking | |