| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-26 | |||
| 13:05:03 | efried | okay | |
| 13:05:15 | efried | I'm sure I'll understand better when I see it | |
| 13:05:30 | mdbooth | Some language/library designers failed to consider their users | |
| 13:05:39 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Block automatic transport_url update for cell0 https://review.openstack.org/605414 | |
| 13:08:26 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Skip more rebuild tests for cells v1 job https://review.openstack.org/605416 | |
| 13:08:38 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Block automatic transport_url update for cell0 https://review.openstack.org/605414 | |
| 13:08:48 | edleafe | mdbooth: more likely is that someone proposed the ExitStack addition after py2 was put into maintenance mode. The mistake was OpenStack starting with py2 just because some vendors didn't distribute py3 yet | |
| 13:08:59 | sean-k-mooney | mdbooth: well to be fair they have said they will continue to support it byond 2020 if enough people have not migrated to py3 earlier this year and python3 is 10 years old in december so we have had time to stop using it | |
| 13:09:36 | mdbooth | edleafe: py3 wasn't a safe choice in 2010 :/ | |
| 13:09:48 | edleafe | Sure it was | |
| 13:10:01 | mdbooth | Ok | |
| 13:10:01 | sean-k-mooney | mdbooth: it was not in 2010 but it was in 2014 when 3.4 shipped | |
| 13:10:10 | edleafe | It's just that Ubuntu didn't include it by default, and Xen required py2.4 | |
| 13:10:26 | mdbooth | edleafe: That's 2 reasons right there | |
| 13:10:40 | mdbooth | There were many, many more | |
| 13:10:52 | edleafe | Right, so it isn't python that's the issue; it's the vendors dragging their butts | |
| 13:11:07 | mdbooth | And libraries | |
| 13:11:18 | sean-k-mooney | mdbooth: yes but at this point its well past the time that libary mantains can claim they did not have time to migrate. | |
| 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 :) | |