| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-27 | |||
| 09:28:22 | opendevreview | Bence Romsics proposed openstack/nova stable/2023.1: Reproduce bug #2025480 in a functional test https://review.opendev.org/c/openstack/nova/+/904372 | |
| 09:28:23 | opendevreview | Bence Romsics proposed openstack/nova stable/2023.1: Do not untrack resources of a server being unshelved https://review.opendev.org/c/openstack/nova/+/904373 | |
| 09:28:43 | opendevreview | Bence Romsics proposed openstack/nova stable/zed: Reproduce bug #2025480 in a functional test https://review.opendev.org/c/openstack/nova/+/904374 | |
| 09:28:44 | opendevreview | Bence Romsics proposed openstack/nova stable/zed: Do not untrack resources of a server being unshelved https://review.opendev.org/c/openstack/nova/+/904375 | |
| 13:14:33 | opendevreview | Rajesh Tailor proposed openstack/nova-specs master: List requested availability zones https://review.opendev.org/c/openstack/nova-specs/+/904368 | |
| 14:46:40 | opendevreview | Tobias Urdin proposed openstack/nova master: Fail nova-compute startup when RP cannot be created https://review.opendev.org/c/openstack/nova/+/904381 | |
| 14:49:11 | opendevreview | Tobias Urdin proposed openstack/nova master: DNM: Alternative to parent patch https://review.opendev.org/c/openstack/nova/+/904382 | |
| 16:52:36 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/yoga: Fix the PCI device capability dict creation https://review.opendev.org/c/openstack/nova/+/899252 | |
| 16:53:46 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Store pf_mac_address and vf_num in extra_info https://review.opendev.org/c/openstack/nova/+/899253 | |
| 16:53:54 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Translate VF network capabilities to port binding https://review.opendev.org/c/openstack/nova/+/899254 | |
| 17:47:20 | tj90241 | Hello folks - happy holidays. Noticed today that: https://review.opendev.org/c/openstack/nova/+/700062 may have a regression and it does not seem fixed in at least a fairly-recent Yoga. | |
| 17:48:04 | tj90241 | Namely, after this change it is not possible to (cold) migrate a VM off a compute node that is strictly backed by Ceph, whereas it was previously. Does this ring a bell/stand out to anyone else? | |
| 17:48:20 | tj90241 | off a compute node, which is down/off/disalbed*** | |
| 17:49:53 | tj90241 | I understand the RPC call will hang forever and it's non-ideal for local storage, just wondering if anyone else thinks this may need to be gated by a flag like --on-shared-storage in older Novas or something somehow (or if this is already fixed and I haven't seen it). | |
| 17:54:01 | tj90241 | Totally unrelated -- would love another +1 or feedback on: https://review.opendev.org/c/openstack/nova/+/892614 as well. Been sitting for a couple months with the last feedback being lets not merge before 2023.2. | |
| #openstack-nova - 2023-12-28 | |||
| 18:44:19 | JayF | sean-k-mooney[m]: https://review.opendev.org/q/topic:%22codespell%22+repo:openstack/ironic-python-agent talent borrows; genius steals :P (seriously; thanks for doing that in nova; hadn't seen the tool before and it looks awesome, going to see if Ironic community agrees) | |
| 18:52:53 | sean-k-mooney[m] | givven how much time i spend respining my code for typos it felt worth my time to try out a few of the options and codespell seams to be one the nicer solutions. i wanted something that was pure python, i.e. had not deps on programs on your system, and that was designed for code i.e. was designed to reduce false positives | |
| 18:53:34 | sean-k-mooney[m] | JayF: i was kind of surprised at the number of issues found in nova | |
| 18:54:45 | sean-k-mooney[m] | the fact it automatically has suggestions for converting uk to us english is also really useful | |
| 18:55:40 | JayF | Honestly I'd consider that a downside, not an upside TBH. | |
| 18:55:55 | JayF | Any correct spelling being accepted would be better than it enforcing a specific regional standard | |
| 18:56:10 | sean-k-mooney[m] | you can disable that but our offcial docs policy for openstack say we use us english | |
| 18:56:16 | JayF | but I personally don't care *which* standard | |
| 18:56:34 | sean-k-mooney[m] | its just an additoanl correction dictionary | |
| 18:57:05 | JayF | that's fair, but I care more about not running someone away with CI churn than I do if a pydoc has an extra u | |
| 18:57:35 | sean-k-mooney[m] | https://github.com/codespell-project/codespell/blob/master/codespell_lib/data/dictionary_en-GB_to_en-US.txt | |
| 18:59:06 | JayF | yeah, my MVP in IPA is super simple, just invoking with the smallest correction list possible | |
| 18:59:24 | JayF | not sure if that's the right path or not, but bluntly, don't wanna invest that much time in a PoC that I made without talking to anyone else ironic-y | |
| 18:59:46 | JayF | I'm mildly tempted to do a similar POC with black/ruff/autopep8 or a similar tool | |
| 19:01:31 | sean-k-mooney[m] | we use autopep8 | |
| 19:01:55 | sean-k-mooney[m] | we talked about black but so far its a hard no form some so not pushing that at the moment | |
| 19:02:26 | sean-k-mooney[m] | we are currently trying to move as much of this to precommit wih optional tox targets for those that dont wnt to use precommit | |
| 19:02:54 | sean-k-mooney[m] | ruff is very interesting although i dont know if its packaged widely | |
| 19:03:35 | sean-k-mooney[m] | usign pre-commit to run flake8 has similar effect in that it only checks the files you changed | |
| 19:03:56 | sean-k-mooney[m] | so it gives a nice speedup althogh im sure its still slower the ruff | |
| 19:04:26 | sean-k-mooney[m] | the main problem with ruff would be hacking i.e. hacking checks would still need to be run elsewhere or ported | |
| 19:06:02 | JayF | the only reason I'm considering autopep8 at all is Nova's use of it | |
| 19:06:20 | JayF | yep, that's basically the #1 primary reason why ruff makes me :( | |
| 19:07:33 | JayF | ruff has some neat features around autoupgrading and dropping old python compat code | |
| 19:08:06 | JayF | I think it could have some real utility in OpenStack if we could bridge that gap, but I do think it's a bit early in the lifecycle compared to most tools we'd pick up | |
| 19:10:37 | sean-k-mooney[m] | JayF: i proposed using pyupgrade for that for nova | |
| 19:10:55 | sean-k-mooney[m] | but people were kind of uncomfortable with that being used automtically | |
| 19:11:00 | sean-k-mooney[m] | so i abandoned the serise | |
| 19:11:38 | JayF | pyupgrade is nasty for a handful of reasons | |
| 19:11:43 | JayF | ruff is better designed but just younger | |
| 19:11:52 | JayF | pyupgrade only running on a per-file basis is really strange | |
| 19:12:32 | JayF | let me put it this way: pyupgrade is written in an extremely opinionated way, and CI-style tooling like that with strong opinions I find offputting, as opposed to things with knobs you can turn to make them work in different ways/environments/runners/etc | |
| 19:12:38 | sean-k-mooney[m] | pyupgarde is using the python ast and validating that the upgraded line has the same behavior | |
| 19:12:42 | JayF | you're already on pre-commit, so that probably was easily glossed over :) | |
| 19:13:27 | sean-k-mooney[m] | welll the way we have been trating this is if the linters would change the code then you reject the patch in ci | |
| 19:13:41 | sean-k-mooney[m] | and provide a way to run that same check locally before its submitted | |
| 19:13:46 | JayF | sean-k-mooney[m]: yeah, my complaints are mostly about the interface/style than they are about the technical aspects of it. It's a useful tool; but I'd never trust it in a CI environment or anywhere that a human would not review it afterwards | |
| 19:13:48 | sean-k-mooney[m] | with pre-commit or tox or both | |
| 19:14:22 | sean-k-mooney[m] | ya same we generally have a policy of nothing other than a huma should modify the code | |
| 19:14:51 | sean-k-mooney[m] | i.e. it not modifed after its reviewed before its submited | |
| 19:16:06 | sean-k-mooney[m] | anyway hopefully the ironic comunitiy will find codespell useful too | |
| 19:18:04 | sean-k-mooney[m] | JayF: on a different note, while im off im catching up on some python confernce videos ectra. i have been thinking about replaceing eventlet and lookign at https://anyio.readthedocs.io/en/stable/ as a possible lib to use if we are to prot off of eventlet | |
| 19:18:42 | JayF | I'm going to be completely honest and transparent with you: I'm maybe one of the worst folks to have this conversation with. | |
| 19:19:12 | JayF | I don't have a good mind for this style of programming or a lot of background on it, it's part of why I pulled in the big brains with that knowledge from GR-OSS to help with that stuff | |
| 19:19:52 | JayF | but I will say please lets pick the most vanilla basic implementation that 90% of everyone else is using, asyncio, trio, something like that, not another weirdo thing that someone will have to rescue next decade LOL | |
| 19:19:55 | sean-k-mooney[m] | well the thing is i dont really want most people to start writing low level asyncio or threaded code | |
| 19:20:31 | sean-k-mooney[m] | right so anyio provides trios api over trio or vanila ayncio | |
| 19:21:38 | sean-k-mooney[m] | trio and asyncio are not directly comtible. anyio give you taskgroups and cancelation_scopes(timeouts) form trio but on plain asyncio | |
| 19:21:55 | sean-k-mooney[m] | its also supports py38+ | |
| 19:22:14 | sean-k-mooney[m] | so while 3.11 has taskgroups anyio gives them to you on 3.8 | |
| 19:24:23 | sean-k-mooney[m] | anyway im going to go cooks something and chill for the evening o/ | |
| 19:25:31 | JayF | o/ have a good one | |
| #openstack-nova - 2023-12-29 | |||
| 01:40:47 | opendevreview | songjie proposed openstack/nova master: libvirt: stop enabling hyperv feature reenlightenment https://review.opendev.org/c/openstack/nova/+/904183 | |
| 02:19:18 | opendevreview | sean mooney proposed openstack/nova master: add bin dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 02:21:53 | opendevreview | sean mooney proposed openstack/nova master: removew use of eventlet.queue.LightQueue https://review.opendev.org/c/openstack/nova/+/900681 | |
| 03:19:08 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 03:22:10 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 03:26:11 | opendevreview | sean mooney proposed openstack/nova master: removew use of eventlet.queue.LightQueue https://review.opendev.org/c/openstack/nova/+/900681 | |
| 03:26:11 | opendevreview | sean mooney proposed openstack/nova master: add bin dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 03:26:12 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 03:29:17 | opendevreview | sean mooney proposed openstack/nova master: add bin dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 03:29:18 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 03:29:18 | opendevreview | sean mooney proposed openstack/nova master: removew use of eventlet.queue.LightQueue https://review.opendev.org/c/openstack/nova/+/900681 | |
| 03:30:55 | opendevreview | sean mooney proposed openstack/nova master: removew use of eventlet.queue.LightQueue https://review.opendev.org/c/openstack/nova/+/900681 | |
| 03:30:55 | opendevreview | sean mooney proposed openstack/nova master: add bin dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 03:30:56 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 03:51:05 | opendevreview | sean mooney proposed openstack/nova master: removew use of eventlet.queue.LightQueue https://review.opendev.org/c/openstack/nova/+/900681 | |
| 03:51:05 | opendevreview | sean mooney proposed openstack/nova master: add bin dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 03:51:06 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 10:07:45 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 15:25:58 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 16:09:21 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| #openstack-nova - 2023-12-30 | |||
| 02:39:22 | opendevreview | sean mooney proposed openstack/nova master: add bin dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 02:39:23 | opendevreview | sean mooney proposed openstack/nova master: [WIP] remmove eventlet form nova api wsgi https://review.opendev.org/c/openstack/nova/+/904425 | |
| 02:39:23 | opendevreview | sean mooney proposed openstack/nova master: removew use of eventlet.queue.LightQueue https://review.opendev.org/c/openstack/nova/+/900681 | |
| #openstack-nova - 2024-01-02 | |||
| 07:08:03 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| #openstack-nova - 2024-01-03 | |||
| 13:26:12 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing requested az in output https://review.opendev.org/c/openstack/nova/+/904568 | |
| 23:13:19 | opendevreview | melanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells https://review.opendev.org/c/openstack/nova/+/650172 | |
| #openstack-nova - 2024-01-04 | |||
| 01:45:19 | opendevreview | melanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells https://review.opendev.org/c/openstack/nova/+/650172 | |
| 03:15:02 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/osc-placement/+/904612 | |
| 03:15:49 | opendevreview | Ghanshyam proposed openstack/os-vif master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/os-vif/+/904613 | |
| 03:27:06 | opendevreview | Ghanshyam proposed openstack/os-resource-classes master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/os-resource-classes/+/904614 | |
| 03:27:12 | opendevreview | Ghanshyam proposed openstack/os-traits master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/os-traits/+/904615 | |
| 03:27:29 | opendevreview | Ghanshyam proposed openstack/nova master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/nova/+/904616 | |
| 03:27:34 | opendevreview | Ghanshyam proposed openstack/python-novaclient master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/python-novaclient/+/904617 | |