| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-26 | |||
| 13:11:47 | edleafe | It's chicken-and-egg: if big projects like OpenStack adopt py3, vendors would have motivation to update. If they stay with py2, well, why bother | |
| 13:11:54 | stephenfin | sean-k-mooney: http://py3readiness.org/ | |
| 13:12:00 | mdbooth | edleafe: It was a brand new language which had been around a little over a year whose future was not assured, not least because it marked an epoch in the language. | |
| 13:12:17 | mdbooth | I'd go so far as to say it would have been an insane choice in 2010 | |
| 13:12:22 | sean-k-mooney | almost there what are teh last 11 | |
| 13:13:09 | sean-k-mooney | stephenfin: nothing too critical bar may uwsgi | |
| 13:13:22 | stephenfin | sean-k-mooney: Aye, and I'd imagine that's wrong | |
| 13:13:30 | stephenfin | assuming uWSGI isn't dead | |
| 13:13:44 | stephenfin | https://uwsgi-docs.readthedocs.io/en/latest/Python.html#python-3 | |
| 13:13:46 | sean-k-mooney | stephenfin: was just going to say i assume its outdated | |
| 13:14:29 | stephenfin | Yup https://github.com/unbit/uwsgi/blob/master/setup.py | |
| 13:14:48 | sean-k-mooney | stephenfin: im surprised entrypoints is not updated. | |
| 13:15:15 | sean-k-mooney | stephenfin: has that been pulled into setuptools | |
| 13:15:28 | stephenfin | I don't know that is. We use pkg_resources (or whatever it's called) from setuptools | |
| 13:15:39 | stephenfin | sean-k-mooney: Yeah, I think so. Not sure what that is, tbh | |
| 13:16:35 | stephenfin | Looks like pykube also supports it so that's also wrong https://github.com/kelproject/pykube/blob/master/setup.py#L16-L19 | |
| 13:16:57 | sean-k-mooney | aprarently 200 ish lines of code that has not been update in a while https://github.com/takluyver/entrypoints/blob/master/entrypoints.py#L1 | |
| 13:18:20 | stephenfin | Which also supports Python 3 https://github.com/takluyver/entrypoints/blob/master/flit.ini | |
| 13:18:59 | stephenfin | (they use https://github.com/takluyver/flit) | |
| 13:21:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run ServerMovingTests with nested resources https://review.openstack.org/604084 | |
| 13:36:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | |
| 13:42:12 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360 | |
| 13:45:46 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | |
| 13:45:47 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IPTools deprecated implementation https://review.openstack.org/605422 | |
| 13:46:00 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Block automatic transport_url update for cell0 https://review.openstack.org/605414 | |
| 13:51:51 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | |
| 14:02:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | |
| 14:10:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | |
| 14:22:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add hide server address tests in test_serversV21.py https://review.openstack.org/605435 | |
| 14:24:17 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix a race evacuating instances in an anti-affinity group https://review.openstack.org/605436 | |
| 14:24:43 | openstackgerrit | Merged openstack/nova master: Consumer gen: remove_provider_from_instance_allocation https://review.openstack.org/591784 | |
| 14:25:05 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IPTools deprecated implementation https://review.openstack.org/605422 | |
| 14:25:09 | mdbooth | efried: Lets see if ^^^ passes tests. | |
| 14:25:32 | stephenfin | bauzas, mriedem, tonyb: Could one of you do me a solid and abandon these unnecessary stable requirements patches? They're around since forever https://review.openstack.org/#/c/493146/ https://review.openstack.org/#/c/490256/ | |
| 14:25:40 | mdbooth | efried: NB: it's not central to what I'm working on, and I don't really care what colour the bike shed is. If it doesn't merge, I'll still sleep like a baby. | |
| 14:25:49 | bauzas | stephenfin: sure, lemme look | |
| 14:26:04 | bauzas | stephenfin: thanks for helping me to do something else but writing UTs :p | |
| 14:26:13 | stephenfin | You're...welcome? :) | |
| 14:26:26 | stephenfin | bauzas: Oh, and this one https://review.openstack.org/#/c/543705/ | |
| 14:26:56 | bauzas | stephenfin: wait | |
| 14:27:17 | bauzas | stephenfin: if I'm abandoning a bot change, will the bot be able to give us another change? | |
| 14:27:33 | stephenfin | bauzas: I think that bot is dead now? | |
| 14:27:52 | bauzas | oh right, now that we have the minimums | |
| 14:28:00 | bauzas | mriedem: this ^ ? | |
| 14:28:06 | stephenfin | If not, I'll personally go harass -infra to resolve the issue | |
| 14:29:28 | mdbooth | mriedem: https://review.openstack.org/#/c/604397/ \o/ | |
| 14:29:51 | mdbooth | mriedem: I'll rebase the other patches on it | |
| 14:30:32 | efried | mdbooth: I'll add it to my watchlist, but not sure I can give it a real review. | |
| 14:32:12 | mdbooth | efried: Yeah, np. Just saying I'm kinda throwing it over the fence. If it inspires somebody else to write a better patch then I'm cool with that, but kinks aside I suspect it'll fix the issue at hand today. | |
| 14:32:51 | mdbooth | If somebody wants a massive rewrite they're welcome to massively rewrite it :) | |
| 14:37:19 | openstackgerrit | Matthew Booth proposed openstack/nova master: Run evacuate tests with local/lvm and shared/rbd storage https://review.openstack.org/604400 | |
| 14:38:24 | efried | bauzas: please have a quick look at new comments on https://review.openstack.org/#/c/599208/ before you put up a new patch set. | |
| 14:39:04 | bauzas | efried: ack, thanks for the notice, I was writing the UTs :) | |
| 14:39:11 | efried | I figgered | |
| 14:44:16 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | |
| 14:46:45 | bauzas | efried: about your two points, fair enough for the type, but maybe it's YAGNI | |
| 14:47:29 | efried | bauzas: surely doesn't do any harm | |
| 14:49:03 | openstackgerrit | Merged openstack/nova stable/queens: Skip ServerActionsTestJSON.test_rebuild_server for cells v1 job https://review.openstack.org/605115 | |
| 14:50:14 | mriedem | stephenfin: done | |
| 14:50:25 | stephenfin | mriedem: Ta | |
| 14:53:12 | sean-k-mooney | mriedem: i tried to review https://review.openstack.org/#/c/605199/ this morning. | |
| 14:53:32 | sean-k-mooney | mriedem: your logic makes sense to me but i dont know why its failing | |
| 14:55:22 | melwitt | naichuans: yes, stein runways are open. feel free to add approved ready-for-review blueprints to the queue | |
| 14:59:46 | mriedem | sean-k-mooney: yeah idk either, it's not very important right now, i have a workaround in nova, and it can wait for dan to come back | |
| 15:00:15 | bauzas | naichuans: keep in mind we need first to have both the consumer-gen series and the scheduler using nested allocations series to be merged before your one | |
| 15:00:31 | bauzas | naichuans: and we also need a reshaping change for the xen driver | |
| 15:06:40 | bauzas | efried: I left a reply on my change | |
| 15:06:58 | efried | ack | |
| 15:07:49 | bauzas | efried: tl;dr if someday nvidia wants to name a type "nvidia-mysuperawesometypeforaimlcoinminingandotherbuzzword', they just need to release a new driver version | |
| 15:08:13 | bauzas | and I just don't want to have a vendors war on the placement API :) | |
| 15:09:07 | sean-k-mooney | bauzas: mdev types? ya they can be anything but i think there is a lenght limit in the kernel | |
| 15:09:26 | bauzas | sean-k-mooney: sure, TEXT ? | |
| 15:09:35 | bauzas | or whatever crazy | |
| 15:09:47 | efried | these type strings correspond to what you need to put in the config, right? | |
| 15:09:58 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | |
| 15:10:53 | bauzas | efried: yup, and I hate that | |
| 15:11:06 | sean-k-mooney | dansmith: looking at the kernel docs its not clare i would have to check the cheder for the mdev device | |
| 15:11:22 | sean-k-mooney | dansmith: sorry that was for bauzas ^ | |
| 15:11:27 | bauzas | efried: but there is a big difference in having a vendor-specific information in a config file, and something set in the DB and shown on the API side :) | |
| 15:11:45 | bauzas | s/, and/vs. | |
| 15:11:59 | bauzas | god dammit, my fucking English | |
| 15:12:35 | sean-k-mooney | bauzas: is that different then your normal english :) | |
| 15:12:59 | bauzas | aber ich lerne Deutch | |
| 15:14:21 | bauzas | sean-k-mooney: https://www.kernel.org/doc/Documentation/vfio-mediated-device.txt | |
| 15:16:12 | sean-k-mooney | bauzas: i read that it does not have the max size of a mdev type specified | |
| 15:16:26 | bauzas | yeah it's a struct | |
| 15:16:47 | bauzas | sean-k-mooney: open bar then, right? | |
| 15:17:24 | sean-k-mooney | bauzas: more or less | |
| 15:17:34 | bauzas | oh and look the header | |
| 15:17:38 | bauzas | who wrote this | |
| 15:17:44 | bauzas | now, you'll understand | |
| 15:17:53 | bauzas | it's totally free | |
| 15:18:07 | bauzas | the less I expose, the safer I am :) | |
| 15:18:29 | mnaser | did nova-scheduler go multiprocess in rocky? | |
| 15:20:17 | bauzas | multi-workers yes | |
| 15:21:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 15:21:39 | bauzas | mnaser: https://review.openstack.org/#/c/159382/ | |
| 15:22:12 | mnaser | proposed in 2015, merged in 2018: nice | |
| 15:22:33 | mriedem | it was abandoned for a few years | |
| 15:22:36 | bauzas | mnaser: we had concerns in the past that were resolved | |