| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-22 | |||
| 22:04:09 | sean-k-mooney | we remove a layer of indirection in train | |
| 22:04:26 | sean-k-mooney | so ther is 1 addtion plays we need to add the parmater in stien | |
| 22:04:26 | melwitt | you mean it does not accept a ageing kwarg? | |
| 22:04:36 | melwitt | oh | |
| 22:05:40 | sean-k-mooney | correct it does not currently the patch that was backported did not have to update that file because we removed it in master | |
| 22:19:31 | sean-k-mooney | melwitt: mnaser in attion to adding the aging parmater to 1 more location i will need to update the functional test or backport a second patch | |
| 22:19:45 | sean-k-mooney | but yes we shoudl be able to make this work | |
| 22:20:26 | dansmith | mriedem_away: I assume by "outside" you mean either (a) out in super mario world or (b) three feet from your door to plant something | |
| 22:21:11 | melwitt | super mario world? | |
| 22:23:36 | dansmith | melwitt: i.e. some video game as opposed to actually going outside | |
| 22:24:24 | melwitt | haha, ok. that's an old super nintendo game | |
| 22:24:40 | sean-k-mooney | it is a good game. unfortunetly i dont have my snes anymore. i do have a saga meadrive with sonic plug into by tv | |
| 22:25:00 | melwitt | I didn't know mriedem_away is a SNES person | |
| 22:25:51 | melwitt | I liked it ok but not a huge fan of super mario. I prefer rpgs | |
| 22:25:54 | sean-k-mooney | ... tries not to feel old for having fond memories of snes and sage games | |
| 22:27:24 | melwitt | I <3 snes | |
| 22:28:24 | openstackgerrit | sean mooney proposed openstack/os-vif stable/stein: only disable mac ageing for ovs hybrid plug https://review.opendev.org/678098 | |
| 22:28:46 | sean-k-mooney | the againg parmater was jsut missing in that one spot after all | |
| 22:29:02 | sean-k-mooney | but now i need to fiture out why i was getting test failures on rhel8 | |
| 22:29:37 | dansmith | melwitt: yeah, I know.. mriedem_away likes snes, which is why I chose it | |
| 22:45:05 | sean-k-mooney | efried: if you have the link to the cyborg privsep patch that would be useful i have several cyborg patches added to my review queue and i did comment on there use of privsep i think in the past bug i cant recall which one it was | |
| 22:47:35 | sean-k-mooney | efried: ah its https://review.opendev.org/#/c/673957/ | |
| 22:48:15 | sean-k-mooney | i was added to it but i have not reviewd it yet ill take a look tomorow. | |
| 22:48:53 | sean-k-mooney | "This code is modelled strongly on how nova and cinder[0] has chosen | |
| 22:48:55 | sean-k-mooney | to use privsep. | |
| 22:48:57 | sean-k-mooney | " | |
| 22:49:04 | sean-k-mooney | ^ that is not a good start. | |
| 23:49:27 | openstackgerrit | Merged openstack/nova master: Handle websockify v0.9.0 in console proxy https://review.opendev.org/677856 | |
| #openstack-nova - 2019-08-23 | |||
| 00:19:12 | alex_xu | mriedem_away: thanks :) | |
| 00:31:57 | mriedem_away | i love SNES | |
| 00:32:12 | mriedem_away | and am playing FF1 on PS1 lately because | |
| 00:32:19 | mriedem_away | but i was actually physically at a park with my kid | |
| 00:32:24 | mriedem_away | with air and sun | |
| 00:32:25 | mriedem_away | wild | |
| 00:38:59 | openstackgerrit | Brin Zhang proposed openstack/nova master: Remove the invalid AZ server create test https://review.opendev.org/678116 | |
| 01:17:24 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 01:58:40 | openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626 | |
| 02:11:58 | openstackgerrit | Merged openstack/nova master: Add docstring to check_availability_zone function https://review.opendev.org/677858 | |
| 03:30:31 | openstackgerrit | Guo Jingyu proposed openstack/nova master: Define and catch InstanceAgentNotRunning for set-password failure https://review.opendev.org/661466 | |
| 03:38:47 | openstackgerrit | zhufl proposed openstack/nova master: [Trivial]Remove used helper get_vif_devname_with_prefix https://review.opendev.org/678136 | |
| 03:43:25 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 03:54:56 | openstackgerrit | Merged openstack/nova master: objects: Rename 'nova.objects.instance_numa_topology' https://review.opendev.org/671789 | |
| 03:55:04 | openstackgerrit | Merged openstack/nova master: libvirt: Remove unnecessary try-catch around 'getCPUMap' https://review.opendev.org/671790 | |
| 03:55:13 | openstackgerrit | Merged openstack/nova master: libvirt: Remove unnecessary argument https://review.opendev.org/674396 | |
| 03:55:23 | openstackgerrit | Merged openstack/nova master: claims: Remove useless caching https://review.opendev.org/671791 | |
| 05:40:17 | openstackgerrit | zhufl proposed openstack/nova master: [Trivial]Remove unused helper get_vif_devname_with_prefix https://review.opendev.org/678136 | |
| 06:16:03 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 06:19:51 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 07:41:13 | AJaeger | nova stable cores, could you review https://review.opendev.org/677178, https://review.opendev.org/677176 https://review.opendev.org/677177 ? Those remove experimental openSUSE 42.3 jobs since the distro is end-of-life now. | |
| 07:43:55 | lyarwood | AJaeger: ack, I'll take a look now | |
| 07:45:08 | lyarwood | AJaeger: nothing on stable/stein? | |
| 07:45:36 | AJaeger | lyarwood: no, that one uses the newer opensuse-150 job already | |
| 07:45:45 | AJaeger | thanks, lyarwood | |
| 07:46:27 | lyarwood | Ib0b49fead613d06001df348831c022edfe8d0fd1 - got it, thanks | |
| 08:47:28 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 08:51:44 | cervigni | hello | |
| 08:51:52 | cervigni | I am playing around with vgpu | |
| 08:52:11 | cervigni | It seems that the nova scheduler does not assign properly the cpu-> gpu | |
| 08:52:33 | cervigni | essentially the first vgpu that is always created by the scheduler, goes to my 2nd gpu | |
| 08:52:46 | cervigni | while the cores are assigned to first cpu (first numa node) | |
| 08:53:22 | cervigni | is there a way to tweak the scheduling algorithm or to specify somehow that there is an affinity between CPU and VGPUs ? | |
| 08:53:52 | cervigni | as it stands, instances always get first cpus / second gpu | |
| 08:54:17 | cervigni | therefore there is a big numa issue and performance penality where instances are communicating with the "opposite" numa node in the other cpu socket | |
| 08:56:15 | cervigni | it seems to me a real nova scheduling issue, but I would like someone to confirm that is the case before opening a bug report | |
| 08:56:24 | openstackgerrit | Mitsuhiro Tanino proposed openstack/nova master: Smaller project quota value than user quota value can't be set https://review.opendev.org/678183 | |
| 08:57:35 | cervigni | GPU 0 has a pci id lower than GPU 1, but for reasons I don't understand, the VGPUs are always first assigned to GPU1 | |
| 08:57:39 | cervigni | |===============================+======================+======================| | |
| 08:57:39 | cervigni | | 0 Tesla V100-PCIE... On | 00000000:37:00.0 Off | Off | | |
| 08:57:39 | cervigni | | N/A 38C P0 27W / 250W | 39MiB / 16383MiB | 0% Default | | |
| 08:57:39 | cervigni | +-------------------------------+----------------------+----------------------+ | |
| 08:57:39 | cervigni | | 1 Tesla V100-PCIE... On | 00000000:86:00.0 Off | Off | | |
| 08:57:39 | cervigni | | N/A 35C P0 29W / 250W | 51MiB / 16383MiB | 0% Default | | |
| 08:57:42 | cervigni | +-------------------------------+----------------------+----------------------+ | |
| 08:59:02 | openstackgerrit | Mitsuhiro Tanino proposed openstack/nova master: Smaller project quota value than user quota value can't be set https://review.opendev.org/678183 | |
| 09:05:25 | openstackgerrit | Mitsuhiro Tanino proposed openstack/nova master: Smaller project quota than user quota shouldn't be set https://review.opendev.org/678183 | |
| 09:53:31 | openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626 | |
| 10:48:52 | stephenfin | alex_xu: You should probably look at https://review.opendev.org/#/c/674894/ given your comments at https://review.opendev.org/#/c/664420/42/nova/scheduler/request_filter.py@206 | |
| 11:18:52 | efried | sean-k-mooney: I agree. I skimmed over your review last night, sounds like what I was expecting. | |
| 11:19:28 | sean-k-mooney | efried: i might try and submit a basic patch to show how to do it as i suggested instead | |
| 11:19:41 | sean-k-mooney | efried: i assume we are talking about cyborg privsep | |
| 11:19:48 | efried | yes | |
| 11:20:46 | sean-k-mooney | i think i was a little harsh in the review but i also have strong feeling about this topic. | |
| 11:26:28 | efried | Swhy I was hoping you would look at it. | |
| 11:47:05 | sean-k-mooney | efried: well i review in cyborg octionally and is mostly negitve when i do. not alway but my +/- raition there is proably not that positive. so i do try to make sure my review that i do there are at least constructive rather then saying "no this is all terible" and suggest how it can be improved. but if i do my usuall 50 billion comments on each indiviaual issue that can come across badly if in a | |
| 11:47:07 | sean-k-mooney | "drive by" review. | |
| 11:53:30 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for setting instance id https://review.opendev.org/659690 | |
| 11:53:31 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/659691 | |
| 11:53:31 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/670213 | |
| 11:54:53 | efried | sean-k-mooney: Any update on that libvirt attribute ordering bug? Not sure if I missed something going past... | |
| 11:58:12 | sean-k-mooney | efried: its still on my todo list. i was dealing with a downstream backport the last day or two but i should get to it today or monday. | |
| 11:58:50 | efried | cool, thanks. It's a "requirements freeze" thing fwiw. | |
| 11:59:12 | sean-k-mooney | oh ya when is that next week? | |
| 11:59:31 | sean-k-mooney | i was aware of that but i proably should expidite looking at it | |
| 11:59:56 | sean-k-mooney | non-client lib frezee shoudl m3 -2 weeks | |
| 11:59:57 | efried | oh, I thought it was, but no, it's the week of Sept 9 | |
| 12:00:05 | sean-k-mooney | and oslo shoudl be m3 -3 | |
| 12:00:16 | sean-k-mooney | i guess requirement freeze is m3-1 | |
| 12:01:12 | sean-k-mooney | actully it looks like its m3 | |
| 12:01:35 | sean-k-mooney | so september 12th | |
| 12:01:58 | sean-k-mooney | but ill try and repoduce anyway this/next week and get a patch up so we dont run up against it | |