| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-08 | |||
| 10:02:14 | kashyap | Oh right! | |
| 10:02:19 | kashyap | stephenfin: I recall the 'none' thingie. Good idea. | |
| 10:02:28 | kashyap | Thanks :-) | |
| 10:03:16 | stephenfin | kashyap: Actually, looks like Sphinx can render complex tables like those. Left comment inline | |
| 10:04:04 | kashyap | Oh, yeah. I recalled the "+" syntax. Wonderful rST + Sphinx. Endless joy :-) | |
| 10:12:00 | kashyap | stephenfin: Okay, even with '+', the words "or" and "type" are highlighted in the rendering. | |
| 10:12:31 | kashyap | (NB: Rendering was not the problem; with +, it rendering was equivalent to the one before.) | |
| 10:13:08 | stephenfin | kashyap: If you switch that to '+', you can dedent it and remove the '::' or '.. code-block' | |
| 10:13:19 | stephenfin | so it'll render as a table instead of a code block | |
| 10:14:37 | kashyap | Ah-ha, so after switching it required 'dedenting'... | |
| 10:14:50 | stephenfin | Yeah, sorry, I wasn't clear | |
| 10:15:23 | stephenfin | The reason it's indented is so it'll render as code because I didn't think docutils would handle complex tables like that, but it turns out it can | |
| 10:15:27 | stephenfin | and with gusto, no less | |
| 10:15:35 | kashyap | (No problem.) | |
| 10:15:37 | kashyap | Hehe, noted. | |
| 10:17:38 | openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.openstack.org/612626 | |
| 10:22:16 | cdent | "with gusto" | |
| 10:24:38 | kashyap | stephenfin: Hmm, so with "+" (de-dendeted, and no '::' or '.. code-block'), the table rendering is "naked": https://kashyapc.fedorapeople.org/Plain-rendering-of-table-Sphinx.png | |
| 10:24:49 | kashyap | (Assuming if you can even spot the table :D) | |
| 10:26:55 | sean-k-mooney | o/ | |
| 10:31:44 | sean-k-mooney | stephenfin: efried: gibi: jaypipes: ill send a email about this later but https://review.openstack.org/#/c/650573/ as an fyi i finished setting up my thirdparty ci over the weekend so we have limited capastity to trigger some dpdk and pinning testing manually now. | |
| 10:33:17 | sean-k-mooney | if people comment "seans-nfv-ci: recheck" or "check experimental" on any patch to nova,neutron,os-vif,networking-ovs-dpdk or devstack then it will run | |
| 10:34:15 | stephenfin | sounds good to me | |
| 10:34:33 | sean-k-mooney | i currently have two singel node jobs running for nova one that tests ovs-dpdk(with hugepages) and another that tests numa+cpu pinning+realtime | |
| 10:36:19 | sean-k-mooney | im going to convert them to multinode jobs so that migration/resize tests run and add a whitbox plugin job too i think. | |
| 10:38:17 | sean-k-mooney | i also need to enable more tests. ill proably change the trigger to run atomatically once i come up with a sutible file ignore list but i am going to see if i can run some of these test upstream too in the future. | |
| 10:55:42 | efried | sean-k-mooney: nice! | |
| 11:03:52 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Re-propose detach root volume spec https://review.openstack.org/650580 | |
| 11:08:24 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: Re-propose volume backed server rebuild https://review.openstack.org/650582 | |
| 11:19:27 | openstackgerrit | Eric Fried proposed openstack/nova master: Hacking N363: Don't use spec[_set]='string' https://review.openstack.org/650370 | |
| 11:20:49 | openstackgerrit | Merged openstack/nova-specs master: Re-propose detach root volume spec https://review.openstack.org/650580 | |
| 11:24:58 | openstackgerrit | Merged openstack/nova-specs master: Re-propose volume backed server rebuild https://review.openstack.org/650582 | |
| 11:38:53 | kashyap | sean-k-mooney: sfinucan: Remind me again: why are we messing with VNC whe configuring live migration here: http://git.openstack.org/cgit/openstack/nova/tree/doc/source/admin/configuring-migrations.rst | |
| 11:40:25 | sean-k-mooney | kashyap by default our install guides did not used to tell people to make vnc listen to 0.0.0.0 i think | |
| 11:40:45 | sean-k-mooney | as a result if you live migrated without haveing mand that change the console would not work on the new host | |
| 11:41:12 | sean-k-mooney | as it defaulted to listening to on the ip of the source host and we dont update that in the libvirt xml on migration | |
| 11:41:42 | sean-k-mooney | so the vnc consle woudl try to bind to a tcp port on a ip that is not on the dstionation and fail | |
| 11:43:15 | kashyap | Ah, I see. I think we should spell that reasoning out in the doc. (I'll send a patch with "Thanks: Sean Mooney") | |
| 11:43:54 | sean-k-mooney | well it says "You must not make the VNC server listen to the IP address of its compute | |
| 11:43:56 | sean-k-mooney | host, since that addresses changes when the instance is migrated. | |
| 11:43:58 | sean-k-mooney | " | |
| 11:44:19 | sean-k-mooney | but ya it could be clearer | |
| 11:45:24 | lyarwood | sean-k-mooney / kashyap ; isn't that out of date now given https://github.com/openstack/nova/blob/65874326404717b58ce4927c21d3502d7e53022b/nova/virt/libvirt/migration.py#L84 | |
| 11:45:41 | kashyap | sean-k-mooney: Yeah, the meaning is implicit. But it can definitely be clearer | |
| 11:45:59 | lyarwood | I thought we did update the listen address now | |
| 11:46:21 | sean-k-mooney | we do according to that code | |
| 11:46:35 | sean-k-mooney | so its nolonger requried but it might be desirable anyway | |
| 11:46:44 | sean-k-mooney | it may not be | |
| 11:46:52 | sean-k-mooney | when was that added | |
| 11:47:15 | sean-k-mooney | 3 years ago | |
| 11:47:36 | sean-k-mooney | there is a slight change that the dock is out dated | |
| 11:48:07 | kashyap | Yeah, definitely making sure that is set does not hurt. | |
| 11:48:18 | kashyap | lyarwood: Thanks for the pointer | |
| 11:56:50 | kashyap | sean-k-mooney: Do we always go through VNC proxy? | |
| 11:57:34 | sean-k-mooney | technically you can connect directly but typically an enduser will always connect via the vnc proxy | |
| 11:57:52 | sean-k-mooney | we do not want endusers connecting directly to the hyperviors | |
| 11:57:55 | sean-k-mooney | host | |
| 12:02:34 | kashyap | Nod; also how about security? Isn't _always_ listening on 0.0.0.0 insecure? | |
| 12:03:04 | sean-k-mooney | yes which is why the note is present in the doc. | |
| 12:03:18 | kashyap | Which note? /me reads | |
| 12:03:29 | kashyap | Ah, this one: | |
| 12:03:30 | kashyap | Since this setting allows VNC clients from any IP address to connect to instance consoles, you must take additional measures like secure networks or firewalls to prevent potential attackers from gaining access to instances. | |
| 12:03:46 | sean-k-mooney | yes | |
| 12:04:02 | kashyap | Okay, thanks again :-) | |
| 12:21:24 | jaypipes | sean-k-mooney: cool, thanks :) | |
| 12:48:32 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add in_tree field to RequestGroup object https://review.openstack.org/649534 | |
| 12:48:33 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Query `in_tree` to placement https://review.openstack.org/649535 | |
| 12:48:33 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: WIP: Add get_compute_nodes_by_host_or_node() https://review.openstack.org/650877 | |
| 12:48:34 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Pass target host to RequestGroup.in_tree https://review.openstack.org/650878 | |
| 12:49:08 | openstackgerrit | François Palin proposed openstack/nova master: WIP - nova diagnostics command is not working with all interfaces https://review.openstack.org/648123 | |
| 13:10:22 | mnaser | https://github.com/openstack/nova/blob/73edcfae7d5ea498fd17fa8e548ae1bd690f408a/nova/db/sqlalchemy/api.py#L5508-L5537 | |
| 13:10:39 | mnaser | so max rows isn't really max rows.. its scanning a full table | |
| 13:29:27 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient stable/stein: Revert "Fix crashing console-log" https://review.openstack.org/650363 | |
| 13:29:51 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient stable/rocky: Revert "Fix crashing console-log" https://review.openstack.org/650364 | |
| 13:41:59 | sean-k-mooney | cdent: jaypipes dansmith thanks for your feedback on https://review.openstack.org/#/c/647578/2/specs/train/approved/image-metadata-prefiltering.rst by the way. im gong to rework the spec with your changes today | |
| 13:46:39 | openstackgerrit | Boris Bobrov proposed openstack/nova-specs master: Re-approve AMD SEV support for Train https://review.openstack.org/641994 | |
| 13:50:34 | openstackgerrit | Boris Bobrov proposed openstack/nova-specs master: Re-approve AMD SEV support for Train https://review.openstack.org/641994 | |
| 15:35:06 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Do not log a warning about not using compute monitors https://review.openstack.org/650929 | |
| 15:36:04 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: libvirt: disconnect volume when encryption fails https://review.openstack.org/650931 | |
| 15:58:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1823370 https://review.openstack.org/650424 | |
| 15:58:44 | openstack | bug 1823370 in OpenStack Compute (nova) "Evacuations are not restricted to the source cell during scheduling" [Medium,In progress] https://launchpad.net/bugs/1823370 - Assigned to Matt Riedemann (mriedem) | |
| 15:58:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Restrict RequestSpec to cell when evacuating https://review.openstack.org/650429 | |
| 15:58:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Use RequestSpec.requested_destination when rebuilding with new image https://review.openstack.org/650376 | |
| 16:05:07 | openstackgerrit | sean mooney proposed openstack/nova-specs master: add spec for image metadata prefiltering https://review.openstack.org/647578 | |
| 16:09:02 | efried | gibi: Who creates the bw RPs? Nova or neutron? | |
| 16:09:02 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Re-propose the spec to allow specifying a list of CPU models https://review.openstack.org/642030 | |
| 16:10:10 | mriedem | efried: neutron | |
| 16:10:16 | efried | thanks | |
| 16:10:26 | mriedem | https://docs.openstack.org/neutron/latest/admin/config-qos-min-bw.html | |
| 16:10:44 | kashyap | sean-k-mooney: When you get a moment this week, can you please give the latest iteration a "once-over" -- https://review.openstack.org/#/c/642030/5/specs/train/approved/cpu-model-selection.rst | |
| 16:11:34 | kashyap | jackding: ^ You originally proposed it; I made some adjustments. Might want to take a look at it. | |
| 16:13:36 | kashyap | I noticed a few mistakes in the code, which led me to look at the spec, one thing leads to another ... and suddenly I "inherited" the above spec :D | |
| 16:29:40 | stephenfin | lyarwood: Mind taking a look at these if you're still about? https://review.openstack.org/650363 https://review.openstack.org/650364 | |
| 16:31:24 | lyarwood | stephenfin: yarp, I'll take a look before I leave | |
| 16:33:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Prepare _heal_allocations_for_instance for nested allocations https://review.openstack.org/637954 | |
| 16:33:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.openstack.org/637955 | |
| 16:33:21 | stephenfin | dansmith, jaypipes: Briefly, do you recall any previous examples of us removing a model/table from 'nova/db/sqlalchemy/models.py'. I assume I need a migration? | |
| 16:33:37 | stephenfin | only example I can see 0951cf3d194edbee213673a525c1291245958264. I assume that's mostly all there is to it? | |
| 16:34:32 | stephenfin | (context is that I want to remove 'nova.db.sqlalchemy.models.Cell' without having the tests b**** at me) | |
| 16:35:24 | sean-k-mooney | stephenfin: quick question | |