| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-23 | |||
| 18:09:57 | sean-k-mooney | does not return or use the same name as pip | |
| 18:10:02 | sean-k-mooney | specificly pip freeze | |
| 18:10:17 | sean-k-mooney | we can make this work by doign pkg_resources.require(name.replace('.','_')) | |
| 18:10:23 | sean-k-mooney | instead of pkg_resources.require(name) | |
| 18:10:27 | sean-k-mooney | on python 3.8 | |
| 18:10:39 | sean-k-mooney | however that is not really teh correct fix | |
| 18:10:52 | sean-k-mooney | we could do that here https://github.com/openstack/requirements/blob/master/openstack_requirements/cmds/check_conflicts.py#L51C25-L51C52 | |
| 18:11:04 | sean-k-mooney | and gard i by the python version. | |
| 18:11:32 | Uggla | yep I agree this is another possibility. | |
| 18:11:45 | sean-k-mooney | but it would be beter to undersand why it is behviing diffently | |
| 18:13:06 | Uggla | yes that was the reason why I asked you. If you have an idea about the reason. | |
| 18:13:48 | sean-k-mooney | my guess is somethign in pkg_resouces has changed | |
| 18:14:00 | sean-k-mooney | i woudl have to see if there are any recent releases | |
| 18:15:09 | sean-k-mooney | https://setuptools.pypa.io/en/latest/pkg_resources.html | |
| 18:15:17 | sean-k-mooney | so pkg_resouces is deprecated for removal | |
| 18:15:39 | sean-k-mooney | we may want to swap to import lib instead | |
| 18:17:48 | sean-k-mooney | pkg_resocue i think is gon in python 3.13 | |
| 18:18:07 | sean-k-mooney | i think we are ment to use https://docs.python.org/3/library/importlib.resources.html instead | |
| 18:18:27 | sean-k-mooney | or well import lib in general | |
| 20:26:42 | masahito | sean-k-mooney: hi, please review the API bug fix, again. gmaan_pto +2ed so all comments are solved. https://review.opendev.org/c/openstack/nova/+/939658 | |
| 20:27:15 | sean-k-mooney | ah sure. im just about to finsih for todday but i can take a look quickly | |
| 20:28:50 | sean-k-mooney | masahito: i agree it looks like gmanns comment have been addressed and i was happy with it before so +2w | |
| 20:30:51 | masahito | sean-k-mooney: thanks! I appreciate you check it even though you're going to finish today. | |
| 23:16:48 | opendevreview | Merged openstack/nova master: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/939658 | |
| #openstack-nova - 2025-04-24 | |||
| 10:08:10 | Uggla | sean-k-mooney[m], I proposed this as a fix: https://review.opendev.org/c/openstack/requirements/+/947558/1..2 please let me know what you think about it. | |
| 10:08:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: debug gate https://review.opendev.org/c/openstack/nova/+/948057 | |
| 10:36:50 | opendevreview | sean mooney proposed openstack/nova stable/2025.1: Use dict object for request_specs_dict in the _list_view https://review.opendev.org/c/openstack/nova/+/948061 | |
| 11:02:11 | opendevreview | Balazs Gibizer proposed openstack/nova master: [quota]Refactor group counting to scatter-gather https://review.opendev.org/c/openstack/nova/+/948064 | |
| 11:03:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: [quota]Refactor group counting to scatter-gather https://review.opendev.org/c/openstack/nova/+/948064 | |
| 11:14:07 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet sleep with time.sleep https://review.opendev.org/c/openstack/nova/+/947212 | |
| 12:09:31 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet sleep with time.sleep https://review.opendev.org/c/openstack/nova/+/947212 | |
| 12:58:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP:Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 12:58:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 13:29:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 13:40:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: [hacking] N373 do not use eventlet primitives https://review.opendev.org/c/openstack/nova/+/947265 | |
| 13:58:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 14:08:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 14:17:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_to https://review.opendev.org/c/openstack/nova/+/948079 | |
| 14:21:46 | JayF | Does a doc like https://docs.openstack.org/nova/latest/reference/vm-states.html exist except for instance.status? | |
| 14:52:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 14:52:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 15:29:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 15:46:20 | dansmith | bauzas: https://review.opendev.org/c/openstack/nova/+/891036 | |
| 15:47:52 | gibi | sean-k-mooney[m]: this is ready to land now I think https://review.opendev.org/c/openstack/nova/+/947212 | |
| 15:48:17 | sean-k-mooney | gibi: sure. about the eventlet removal | |
| 15:48:40 | sean-k-mooney | im a littel confused wny alot of the work i had doen in my orginal patches for that | |
| 15:48:45 | sean-k-mooney | is not been redone | |
| 15:49:01 | sean-k-mooney | i tought you and kamil were going to take over my orginal patches | |
| 15:50:11 | sean-k-mooney | like i had already added exectuor and the cooperative yeild https://review.opendev.org/c/openstack/nova/+/922497/5/nova/utils.py | |
| 15:51:16 | bauzas | dansmith: +2d, I'm cool with the testing pattern | |
| 15:51:24 | sean-k-mooney | we now have 3 version fo the scather gater rewtire | |
| 15:51:50 | dansmith | bauzas: thanks | |
| 15:54:05 | gibi | sean-k-mooney: in some cases I did it | |
| 15:54:20 | gibi | sean-k-mooney: in other cases it was hard to take over due to changes in direction | |
| 15:54:34 | gibi | but if you want I can re-use change IDs | |
| 15:54:57 | sean-k-mooney | well i think we just need to abandon any that your are not going to use/update | |
| 15:55:10 | sean-k-mooney | its confugin to kwno which oens are gogn to proceed | |
| 15:55:11 | gibi | sean-k-mooney: I can do that, and make some linking | |
| 15:55:21 | gibi | made a note, I will do that tomorrow | |
| 15:55:51 | sean-k-mooney | its more about know which ones to review. | |
| 16:00:41 | sean-k-mooney | gibi: the one that propmeted thsi was https://review.opendev.org/c/openstack/nova/+/948076 i wrote a version of that years ago https://review.opendev.org/c/openstack/nova/+/892722 | |
| 16:02:39 | sean-k-mooney | so i jsut didnt want to repeat work for no reasons. | |
| 16:04:25 | sean-k-mooney | looks like yoru version is passing ci so ill abandon min in favor fo it | |
| 19:10:24 | opendevreview | Dan Smith proposed openstack/placement master: Reproduce bug 2104040: allocate in over-capacity https://review.opendev.org/c/openstack/placement/+/945464 | |
| 19:10:25 | opendevreview | Dan Smith proposed openstack/placement master: Fix placement allocate while over-capacity https://review.opendev.org/c/openstack/placement/+/945465 | |
| 19:56:36 | opendevreview | Merged openstack/nova master: Support glance's new location API https://review.opendev.org/c/openstack/nova/+/891036 | |
| #openstack-nova - 2025-04-25 | |||
| 06:56:03 | gibi | bauzas: replied in https://review.opendev.org/c/openstack/nova/+/947507 | |
| 06:56:08 | gibi | bauzas: we merged a relnote | |
| 06:58:27 | gibi | dansmith, sean-k-mooney[m]: this is ready to land and need a second core https://review.opendev.org/c/openstack/nova/+/922496 (rip out the remote debugger) | |
| 10:35:06 | sean-k-mooney | ... i have been using kde for the last week to test somethign and i keep forgetting that its configure to sleep so my irc client keeps disconnecting in the evening | |
| 10:57:42 | opendevreview | Merged openstack/nova master: [doc]Remove eventlet based API endpoints https://review.opendev.org/c/openstack/nova/+/947507 | |
| 10:57:54 | opendevreview | Merged openstack/nova master: [doc]Describe file based GMR triggering https://review.opendev.org/c/openstack/nova/+/947508 | |
| 11:14:21 | opendevreview | Merged openstack/nova master: Remove WSGIServer related config options https://review.opendev.org/c/openstack/nova/+/947595 | |
| 11:18:11 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP:Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 11:18:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 11:18:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 11:18:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 11:18:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 11:18:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 11:18:14 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 11:18:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 11:18:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 11:23:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 11:23:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 11:23:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 11:24:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 11:24:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 11:58:14 | sean-k-mooney | melwitt: gibi: before i forget https://review.opendev.org/c/openstack/nova/+/946950 is my change to get rid of the ovn live migration hack can we try and merge that sooner rather then later. it seams to work but i want to get as much soak time in the ci this cycle as we can | |
| 11:58:48 | sean-k-mooney | we cant backport this due to the ovn version in ubuntu22.04 so this is a purly master only fix | |
| 12:26:19 | gibi | sean-k-mooney[m]: +2 | |
| 12:26:32 | sean-k-mooney | thanks | |
| 12:27:03 | sean-k-mooney | i need ot go find and approve that debuger remvoeal pacth actully | |
| 12:28:31 | sean-k-mooney | my irc client disconened last night but i got the ping on slack do you happen to have the link if not ill find it its definetly in my review dashbaord | |
| 12:29:03 | frickler | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/922496 | |
| 12:29:53 | frickler | looks like it is in merge conflict now, though | |
| 12:30:11 | sean-k-mooney | damit i shoudl have merged it this morning | |
| 12:30:30 | sean-k-mooney | but thanks | |
| 12:30:50 | sean-k-mooney | this isnt urgent | |
| 12:33:04 | sean-k-mooney | on the plus sid ethe reason its in merge conflict is becasue we merged more imporant eventlet removal patches so its nice that we are finally making progress on this | |
| 12:36:03 | gibi | () | |