| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-02-12 | |||
| 17:47:27 | sean-k-mooney | medtadef time | |
| 17:47:55 | sean-k-mooney | actully im not sure this is even a metadef it migh be directly on the image | |
| 17:48:30 | stephenfin | Hmm, we do also have 'hw_numa_cpus' there | |
| 17:48:41 | stephenfin | That's wrong too, IMO :( | |
| 17:49:00 | stephenfin | I wonder if we should do this just to be compatible even if it's not a great design though | |
| 17:49:54 | stephenfin | We have a 'img_bittorrent' field on the 'ImageMetaProps' object. Good Lord | |
| 17:50:43 | sean-k-mooney | oh looks like it only has min disk and min ram | |
| 17:50:45 | sean-k-mooney | https://docs.openstack.org/python-openstackclient/pike/cli/command-objects/image.html#image-create | |
| 17:50:59 | sean-k-mooney | stephenfin: its by design | |
| 17:51:12 | sean-k-mooney | again the intent was to allow the image to descibr the toplogy | |
| 17:51:20 | sean-k-mooney | and the flavor to describe the quantity | |
| 17:51:45 | stephenfin | sean-k-mooney: But remember, that's a not a simple list | |
| 17:51:58 | stephenfin | or actually, maybe it is for the image metadata | |
| 17:52:45 | stephenfin | what happens if you create an N node instance that lists M CPUs in the 'hw_numa_cpus' image metadata option? | |
| 17:52:50 | sean-k-mooney | stephenfin: i agree that we should not mix it with the resouces:* syntax | |
| 17:53:28 | stephenfin | yeah, I really don't think we need the image metadata option. If anything, I'd be looking to deprecate the likes of 'hw_numa_cpus' | |
| 17:53:52 | sean-k-mooney | but if we are seting hw_cpu_policy=mixed in the image seting hw_pinned_cpus=1-5 i think is ok | |
| 17:54:07 | sean-k-mooney | stephenfin: i dont think that we shoudl do that | |
| 17:54:22 | sean-k-mooney | we could but its there for a reason | |
| 17:54:27 | stephenfin | so if we boot using a flavor with 4 cores, what happens? | |
| 17:54:30 | sean-k-mooney | partly to prevent falvor explosion | |
| 17:54:30 | stephenfin | or with 8 cores? | |
| 17:55:06 | sean-k-mooney | with 4 course it woudl be rejected by the numa constraint in the api | |
| 17:55:07 | stephenfin | by having image explosion instead? | |
| 17:55:24 | sean-k-mooney | before we even create the instance record | |
| 17:55:55 | sean-k-mooney | stephenfin: for 8 cors 0,6,7 would be un pinned | |
| 17:56:25 | stephenfin | Yeah, that seems wrong to me | |
| 17:56:27 | stephenfin | I'd be okay with a ratio | |
| 17:56:36 | stephenfin | i.e. 3:1 pinned:unpinned | |
| 17:56:44 | stephenfin | because it's a hint | |
| 17:56:45 | sean-k-mooney | right but we said its a list | |
| 17:56:52 | stephenfin | but I don't think anyone's asked for that | |
| 17:57:12 | sean-k-mooney | a ratio is not ok in general | |
| 17:57:12 | stephenfin | has anyone actually said they need the image metadata option? | |
| 17:57:30 | sean-k-mooney | thats a question for huaqiang and alex_xu | |
| 17:57:33 | stephenfin | well we have that in a manner already | |
| 17:57:54 | stephenfin | that's what e.g. the 'hw_cpu_cores' thing is doing | |
| 17:58:02 | stephenfin | the ratio of cores to threads and sockets | |
| 17:58:18 | sean-k-mooney | yes although its not really a raitio | |
| 17:58:24 | sean-k-mooney | it indireclty is | |
| 17:58:27 | sean-k-mooney | but ya | |
| 17:58:53 | stephenfin | a flat number of unpinned cores would also be okay for me | |
| 17:59:06 | redrobot | Hi friends! I have a question about your pep8 gate configuration. | |
| 17:59:08 | sean-k-mooney | its really socket per vm, cores per socket and then threads per core | |
| 17:59:08 | stephenfin | but again, I don't know how much that achieves | |
| 17:59:12 | stephenfin | redrobot: shoot | |
| 17:59:15 | redrobot | Seeing this error right now: | |
| 17:59:17 | redrobot | > F841 local variable '_' is assigned to but never used | |
| 17:59:31 | redrobot | I'm used to _ being used for throwaways in tests | |
| 17:59:47 | redrobot | but I was wondering if y'all handled it differently? | |
| 17:59:52 | stephenfin | Yeah, I thought that was usually ignored | |
| 18:00:04 | sean-k-mooney | it depens on the context i think | |
| 18:00:09 | stephenfin | Got a link to a build? We're just using flake8 + hacking | |
| 18:00:13 | sean-k-mooney | if you do _ = 4 | |
| 18:00:21 | sean-k-mooney | i think we will raise an error | |
| 18:00:27 | stephenfin | yeah, maybe | |
| 18:00:33 | sean-k-mooney | if you for _ , i in x: | |
| 18:00:38 | sean-k-mooney | i think it will be fine | |
| 18:00:42 | stephenfin | yup | |
| 18:00:43 | sean-k-mooney | or if you do multiple assingment | |
| 18:00:48 | stephenfin | in my experience, anyway | |
| 18:00:58 | redrobot | stephenfin, https://zuul.opendev.org/t/openstack/build/342ef93cbf464e01b72564695b2d610d | |
| 18:01:04 | sean-k-mooney | so _, x = sum_fun() | |
| 18:01:39 | stephenfin | redrobot: I'm not seeing that issue | |
| 18:01:45 | stephenfin | I'm seeing this | |
| 18:01:46 | stephenfin | ./nova/tests/unit/cmd/test_baseproxy.py:93:50: E241 multiple spaces after ',' | |
| 18:01:46 | stephenfin | ./nova/console/websocketproxy.py:326:32: F632 use ==/!= to compare str, bytes, and int literals | |
| 18:02:02 | sean-k-mooney | ya same i dont see it compain about _ | |
| 18:02:38 | redrobot | oh d'oh. It's complaining about _ when I run tox -e pep8 locally | |
| 18:02:44 | redrobot | but also the extra space | |
| 18:02:56 | redrobot | I'll follow up the patch with the fix for the space and hopefully that'll work. | |
| 18:03:09 | openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Proposes NUMA topology with RPs https://review.opendev.org/552924 | |
| 18:03:17 | stephenfin | redrobot: Ah, yes, this https://review.opendev.org/#/c/679502/10/nova/tests/unit/console/test_websocketproxy.py@642 | |
| 18:03:21 | stephenfin | You don't need that assignment | |
| 18:03:29 | stephenfin | If you don't want the value, don't assign to anything | |
| 18:03:36 | stephenfin | so just | |
| 18:03:37 | stephenfin | websocketproxy.NovaWebSocketProxy(ssl_minimum_version=minver) | |
| 18:03:41 | stephenfin | instead of | |
| 18:03:42 | stephenfin | _ = websocketproxy.NovaWebSocketProxy(ssl_minimum_version=minver) | |
| 18:03:43 | sean-k-mooney | so yes dont do https://review.opendev.org/#/c/679502/10/nova/tests/unit/console/test_websocketproxy.py@631 | |
| 18:03:55 | stephenfin | 'zactly ^ | |
| 18:03:56 | sean-k-mooney | that should raise a style error | |
| 18:04:33 | redrobot | stephenfin, ack, I'll change that | |
| 18:04:39 | redrobot | thanks for the help, y'all! | |
| 18:05:11 | sean-k-mooney | no worries | |
| 18:05:13 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Correctly resize encrypted LUKSv1 volumes https://review.opendev.org/706900 | |
| 18:05:52 | bauzas | efried: sean-k-mooney: gibi: dansmith: stephenfin: just wrote the last round of the NUMA/placement spec | |
| 18:06:06 | bauzas | it's fuckingly 7pm and I'm deprived of meat | |
| 18:06:22 | openstackgerrit | Douglas Mendizábal proposed openstack/nova master: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/679502 | |
| 18:06:42 | sean-k-mooney | bauzas: go get foods hehe | |
| 18:06:56 | bauzas | to make it clear, I turn-voted into *not* reshaping by default in Ussuri because i think it's important for us to be very conservative and super cautious | |
| 18:07:28 | bauzas | we will have time to make it mandatory for Victoria and write some nice nova-status command later in Ussuri | |
| 18:08:27 | bauzas | sean-k-mooney: well, I also have to join a city council meeting since I run for a mayor's list :) | |
| 18:09:16 | sean-k-mooney | ireland had some interesting political change over the weekend too. | |
| 18:09:58 | sean-k-mooney | im going to similarly go to the store and get stuff for dinner but ill review it in the next hour or so | |
| 18:20:03 | gmann | melwitt: stephenfin you can see the jobs running for policy-only changes with proposed change- https://review.opendev.org/#/c/707041/ | |
| 18:20:48 | melwitt | gmann: cool thanks | |
| 18:22:32 | lyarwood | sean-k-mooney: sorry missed your pings earlier, yeah can you take a look at elod's comments in the first change and address them if you have time this week? | |
| 18:22:49 | lyarwood | sean-k-mooney: I was just rebasing to get them passing in CI again now that gmann++ fixed up stable/rocky | |
| 19:15:52 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing deferred_delete policies https://review.opendev.org/707455 | |