*** tetsuro has joined #openstack-placement | 01:18 | |
*** lei-zh has joined #openstack-placement | 01:58 | |
*** takashin has quit IRC | 03:33 | |
*** takashin has joined #openstack-placement | 03:51 | |
*** tetsuro has quit IRC | 03:57 | |
*** lei-zh has quit IRC | 04:38 | |
*** lei-zh has joined #openstack-placement | 05:03 | |
*** lei-zh has quit IRC | 05:07 | |
*** lei-zh has joined #openstack-placement | 05:07 | |
openstackgerrit | Yongli He proposed openstack/nova-specs master: add 'show-server-group' spec https://review.openstack.org/612255 | 06:37 |
---|---|---|
openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256 | 06:37 |
*** e0ne has joined #openstack-placement | 06:45 | |
*** lei-zh has quit IRC | 06:49 | |
*** lei-zh has joined #openstack-placement | 06:49 | |
*** lei-zh has quit IRC | 07:06 | |
*** lei-zh has joined #openstack-placement | 07:06 | |
*** helenafm has joined #openstack-placement | 07:22 | |
*** lei-zh has quit IRC | 07:28 | |
*** lei-zh has joined #openstack-placement | 07:42 | |
*** tetsuro has joined #openstack-placement | 08:11 | |
*** tetsuro has quit IRC | 08:12 | |
*** tetsuro_ has joined #openstack-placement | 08:12 | |
*** finucannot is now known as stephenfin | 08:20 | |
*** ttsiouts has joined #openstack-placement | 08:25 | |
*** ttsiouts has quit IRC | 08:29 | |
*** ttsiouts has joined #openstack-placement | 09:17 | |
*** ttsiouts has quit IRC | 09:23 | |
*** lei-zh has quit IRC | 09:27 | |
*** ttsiouts has joined #openstack-placement | 09:43 | |
*** sean-k-mooney has joined #openstack-placement | 10:03 | |
*** s10 has joined #openstack-placement | 10:25 | |
*** cdent has joined #openstack-placement | 10:30 | |
cdent | tetsuro_: do you know if this bug is still valid: https://bugs.launchpad.net/nova/+bug/1792503 ? | 10:36 |
openstack | Launchpad bug 1792503 in OpenStack Compute (nova) "allocation candidates "?member_of=" doesn't work with nested providers" [Undecided,New] | 10:36 |
openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779 | 10:41 |
*** cdent has quit IRC | 10:50 | |
openstackgerrit | Jie Li proposed openstack/nova-specs master: Support volume-backed server rebuild https://review.openstack.org/532407 | 11:46 |
*** cdent has joined #openstack-placement | 11:52 | |
tetsuro_ | cdent : it's fixed in the extracted placement repository. | 12:06 |
cdent | tetsuro_: that's what I thought. Could you indicate that on the bug please? | 12:07 |
tetsuro_ | cdent: I commented on https://bugs.launchpad.net/nova/+bug/1792503/comments/4. I didn't touch the status since on nova/master the bug still exists. | 12:11 |
openstack | Launchpad bug 1792503 in OpenStack Compute (nova) "allocation candidates "?member_of=" doesn't work with nested providers" [Undecided,New] | 12:12 |
cdent | thanks | 12:13 |
*** efried_pto is now known as efried | 12:14 | |
*** smcginnis_vaca is now known as smcginnis | 12:26 | |
*** jaypipes has joined #openstack-placement | 12:27 | |
*** tetsuro_ has quit IRC | 12:33 | |
*** jroll has quit IRC | 12:53 | |
*** jroll has joined #openstack-placement | 12:54 | |
*** edleafe has joined #openstack-placement | 12:55 | |
*** mriedem has joined #openstack-placement | 13:11 | |
*** tetsuro has joined #openstack-placement | 13:12 | |
sean-k-mooney | cdent: o/ | 13:19 |
sean-k-mooney | cdent: thanks for reviewing my wsgi patch. it turns out we use the context manager staitcly via decorators so we cannot safly recreate the context manager unless we remove the decorators | 13:20 |
sean-k-mooney | i just left a comment say the same https://review.openstack.org/#/c/610034/1/nova/api/openstack/placement/db_api.py@20 | 13:21 |
sean-k-mooney | i could refactor the code to remove the decorators that create teh read and write contexts but im not sure i should. | 13:22 |
sean-k-mooney | from a pure api perspective i think the decorator patteren we are using is cleaner but it does man the module must have static lifetime. | 13:24 |
*** rubasov has joined #openstack-placement | 13:26 | |
openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779 | 13:30 |
*** takashin has quit IRC | 13:37 | |
cdent | sean-k-mooney: ah, yeah, I should have remembered that: it used to be that all the decorated methods were in classes, but that changed along the way | 13:38 |
cdent | so yeah, I guess type error catching is the way to go | 13:39 |
sean-k-mooney | cdent: ok in that case is there anything else i should do such as store the conf and check if it changed? or are you happy with it as is | 13:41 |
*** takashin has joined #openstack-placement | 13:43 | |
openstackgerrit | Markus Hentsch proposed openstack/nova-specs master: Spec for the Nova part of Image Encryption https://review.openstack.org/608696 | 13:43 |
cdent | sean-k-mooney: I'd say one issue at a time | 13:47 |
cdent | rather than trying to guess the next thing that might come up | 13:47 |
sean-k-mooney | cdent: ok so if i just file the bug and then propose the same fix to placement are you happy with it as is ill double check the comment but if im not refactoring i dont think there was any matial change requested | 13:50 |
cdent | sean-k-mooney: That sounds right, but who knows, we'll probably find something else along the way :) | 13:53 |
efried | n-sch meeting in 5 minutes in #openstack-meeting-alt | 13:55 |
openstackgerrit | Chris Dent proposed openstack/placement master: Clean up and clarify tox.ini https://review.openstack.org/611719 | 14:04 |
openstackgerrit | Chris Dent proposed openstack/placement master: Fix genpolicy tox job https://review.openstack.org/611927 | 14:10 |
*** SteelyDan is now known as dansmith | 14:11 | |
openstackgerrit | Chris Dent proposed openstack/placement master: Fix genpolicy tox job https://review.openstack.org/611927 | 14:24 |
*** tetsuro has quit IRC | 14:44 | |
*** takashin has left #openstack-placement | 15:03 | |
*** s10 has quit IRC | 15:07 | |
*** e0ne has quit IRC | 15:15 | |
*** e0ne has joined #openstack-placement | 15:28 | |
*** e0ne has quit IRC | 15:32 | |
*** helenafm has quit IRC | 15:56 | |
cdent | edleafe, efried, mriedem: https://etherpad.openstack.org/p/placement-extract-stein-4 has some top-level items, now going back through and adding links, details, etc, if I have them | 16:03 |
efried | cdent: ack, updated https://wiki.openstack.org/wiki/Meetings/NovaScheduler#Agenda_for_next_meeting | 16:05 |
edleafe | Thanks cdent | 16:07 |
*** ttsiouts has quit IRC | 16:30 | |
openstackgerrit | Ed Leafe proposed openstack/placement master: Added alembic environment https://review.openstack.org/611441 | 16:32 |
openstackgerrit | Ed Leafe proposed openstack/placement master: Clean up the old migrations and models https://review.openstack.org/611440 | 16:32 |
edleafe | cdent: if you're so inclilned ^^ | 16:32 |
edleafe | functional tests still failing for not being able to see the tables | 16:33 |
openstackgerrit | sean mooney proposed openstack/placement master: harden placement init under wsgi https://review.openstack.org/612444 | 16:34 |
cdent | edleafe: aye aye | 16:34 |
*** ttsiouts has joined #openstack-placement | 16:36 | |
cdent | edleafe: so the expectation at this point is it seems like the functional tests ought to work, but don't or is it is that it's not wired up yet? | 16:39 |
*** ttsiouts has quit IRC | 16:39 | |
edleafe | It's not wired up | 16:39 |
edleafe | The call to alembic is creating the tables and such, but I can't figure out how to link the sqla engine to that db | 16:40 |
cdent | edleafe: okay, thanks, will see if I can figure anything out, as I'm curious etc | 16:43 |
*** efried has quit IRC | 16:49 | |
*** efried has joined #openstack-placement | 16:49 | |
*** e0ne has joined #openstack-placement | 16:52 | |
*** e0ne has quit IRC | 17:06 | |
*** rubasov has quit IRC | 17:18 | |
*** rubasov has joined #openstack-placement | 17:21 | |
*** rubasov has quit IRC | 17:27 | |
*** e0ne has joined #openstack-placement | 17:30 | |
*** irclogbot_4 has joined #openstack-placement | 17:34 | |
sean-k-mooney | o/ | 17:51 |
sean-k-mooney | anyone have time to look at https://review.openstack.org/#/q/I704196711d30c1124e713ac31111a8ea6fa2f1ba | 17:52 |
sean-k-mooney | they should be as close to identical as i can make them realistically | 17:52 |
*** irclogbot_4 has quit IRC | 17:56 | |
*** ttsiouts has joined #openstack-placement | 17:58 | |
*** irclogbot_4 has joined #openstack-placement | 18:06 | |
*** e0ne has quit IRC | 18:24 | |
*** rubasov has joined #openstack-placement | 18:38 | |
*** rubasov has quit IRC | 18:47 | |
cdent | edleafe: you still around. I've made some headway | 18:56 |
edleafe | yeah, but I have a meeting in 3 minutes | 18:57 |
cdent | just wanted to know what you wanted me to do with the stuff | 18:57 |
cdent | should I push up a new patchset with some notes? | 18:58 |
edleafe | If you have improvements, push a new revision | 18:58 |
cdent | roger that | 18:58 |
edleafe | Great. Maybe I'll learn something about sqla :) | 18:58 |
cdent | after I do that I pass the ball back to you. It's not perfect yet, but closer | 18:58 |
cdent | one thing not yet fixed: the allocations table isn't included in the initial migration | 18:58 |
edleafe | Was it something simple I was missing? Just curious | 18:59 |
cdent | the env.py needs to be customized for "our" environment | 18:59 |
cdent | it's used whenever an alembic command is called | 18:59 |
cdent | so that needs to know about placement.conf, essentially | 19:00 |
*** ttsiouts has quit IRC | 19:01 | |
openstackgerrit | Chris Dent proposed openstack/placement master: Added alembic environment https://review.openstack.org/611441 | 19:06 |
openstackgerrit | Chris Dent proposed openstack/placement master: Clean up the old migrations and models https://review.openstack.org/611440 | 19:07 |
*** irclogbot_4 has quit IRC | 19:14 | |
edleafe | cdent: thanks - will look at those soon | 19:18 |
*** ttsiouts has joined #openstack-placement | 19:18 | |
*** ttsiouts has quit IRC | 20:04 | |
*** ttsiouts has joined #openstack-placement | 20:05 | |
*** ttsiouts has quit IRC | 20:09 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.openstack.org/612497 | 20:16 |
efried | cdent, jaypipes: ^ | 20:16 |
cdent | enqueued | 20:17 |
jaypipes | also enqueued | 20:18 |
cdent | i r dun | 20:22 |
cdent | night night | 20:22 |
*** cdent has quit IRC | 20:22 | |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/612500 | 20:27 |
*** ttsiouts has joined #openstack-placement | 20:39 | |
*** ttsiouts has quit IRC | 20:43 | |
*** ttsiouts has joined #openstack-placement | 21:01 | |
openstackgerrit | Eric Fried proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/612500 | 21:30 |
*** mriedem is now known as mriedem_away | 22:07 | |
openstackgerrit | Merged openstack/placement master: Add recreate test for bug 1798163 https://review.openstack.org/611164 | 22:12 |
openstack | bug 1798163 in OpenStack Compute (nova) rocky "Placement incomplete consumers online migration fails" [Critical,Fix committed] https://launchpad.net/bugs/1798163 - Assigned to Matt Riedemann (mriedem) | 22:12 |
openstackgerrit | Merged openstack/placement master: Use unique consumer_id when doing online data migration https://review.openstack.org/611165 | 22:13 |
openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779 | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Per-instance serial number (spec) https://review.openstack.org/612531 | 22:56 |
*** mriedem_away has quit IRC | 23:27 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!