Monday, 2018-10-22

*** tetsuro has joined #openstack-placement01:18
*** lei-zh has joined #openstack-placement01:58
*** takashin has quit IRC03:33
*** takashin has joined #openstack-placement03:51
*** tetsuro has quit IRC03:57
*** lei-zh has quit IRC04:38
*** lei-zh has joined #openstack-placement05:03
*** lei-zh has quit IRC05:07
*** lei-zh has joined #openstack-placement05:07
openstackgerritYongli He proposed openstack/nova-specs master: add 'show-server-group' spec  https://review.openstack.org/61225506:37
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225606:37
*** e0ne has joined #openstack-placement06:45
*** lei-zh has quit IRC06:49
*** lei-zh has joined #openstack-placement06:49
*** lei-zh has quit IRC07:06
*** lei-zh has joined #openstack-placement07:06
*** helenafm has joined #openstack-placement07:22
*** lei-zh has quit IRC07:28
*** lei-zh has joined #openstack-placement07:42
*** tetsuro has joined #openstack-placement08:11
*** tetsuro has quit IRC08:12
*** tetsuro_ has joined #openstack-placement08:12
*** finucannot is now known as stephenfin08:20
*** ttsiouts has joined #openstack-placement08:25
*** ttsiouts has quit IRC08:29
*** ttsiouts has joined #openstack-placement09:17
*** ttsiouts has quit IRC09:23
*** lei-zh has quit IRC09:27
*** ttsiouts has joined #openstack-placement09:43
*** sean-k-mooney has joined #openstack-placement10:03
*** s10 has joined #openstack-placement10:25
*** cdent has joined #openstack-placement10:30
cdenttetsuro_: do you know if this bug is still valid: https://bugs.launchpad.net/nova/+bug/1792503 ?10:36
openstackLaunchpad bug 1792503 in OpenStack Compute (nova) "allocation candidates "?member_of=" doesn't work with nested providers" [Undecided,New]10:36
openstackgerritAdam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances  https://review.openstack.org/60977910:41
*** cdent has quit IRC10:50
openstackgerritJie Li proposed openstack/nova-specs master: Support volume-backed server rebuild  https://review.openstack.org/53240711:46
*** cdent has joined #openstack-placement11:52
tetsuro_cdent : it's fixed in the extracted placement repository.12:06
cdenttetsuro_: 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
openstackLaunchpad bug 1792503 in OpenStack Compute (nova) "allocation candidates "?member_of=" doesn't work with nested providers" [Undecided,New]12:12
cdentthanks12:13
*** efried_pto is now known as efried12:14
*** smcginnis_vaca is now known as smcginnis12:26
*** jaypipes has joined #openstack-placement12:27
*** tetsuro_ has quit IRC12:33
*** jroll has quit IRC12:53
*** jroll has joined #openstack-placement12:54
*** edleafe has joined #openstack-placement12:55
*** mriedem has joined #openstack-placement13:11
*** tetsuro has joined #openstack-placement13:12
sean-k-mooneycdent: o/13:19
sean-k-mooneycdent: 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 decorators13:20
sean-k-mooneyi just left a comment say the same https://review.openstack.org/#/c/610034/1/nova/api/openstack/placement/db_api.py@2013:21
sean-k-mooneyi 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-mooneyfrom 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-placement13:26
openstackgerritAdam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances  https://review.openstack.org/60977913:30
*** takashin has quit IRC13:37
cdentsean-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 way13:38
cdentso yeah, I guess type error catching is the way to go13:39
sean-k-mooneycdent: 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 is13:41
*** takashin has joined #openstack-placement13:43
openstackgerritMarkus Hentsch proposed openstack/nova-specs master: Spec for the Nova part of Image Encryption  https://review.openstack.org/60869613:43
cdentsean-k-mooney: I'd say one issue at a time13:47
cdentrather than trying to guess the next thing that might come up13:47
sean-k-mooneycdent: 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 requested13:50
cdentsean-k-mooney: That sounds right, but who knows, we'll probably find something else along the way :)13:53
efriedn-sch meeting in 5 minutes in #openstack-meeting-alt13:55
openstackgerritChris Dent proposed openstack/placement master: Clean up and clarify tox.ini  https://review.openstack.org/61171914:04
openstackgerritChris Dent proposed openstack/placement master: Fix genpolicy tox job  https://review.openstack.org/61192714:10
*** SteelyDan is now known as dansmith14:11
openstackgerritChris Dent proposed openstack/placement master: Fix genpolicy tox job  https://review.openstack.org/61192714:24
*** tetsuro has quit IRC14:44
*** takashin has left #openstack-placement15:03
*** s10 has quit IRC15:07
*** e0ne has quit IRC15:15
*** e0ne has joined #openstack-placement15:28
*** e0ne has quit IRC15:32
*** helenafm has quit IRC15:56
cdentedleafe, 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 them16:03
efriedcdent: ack, updated https://wiki.openstack.org/wiki/Meetings/NovaScheduler#Agenda_for_next_meeting16:05
edleafeThanks cdent16:07
*** ttsiouts has quit IRC16:30
openstackgerritEd Leafe proposed openstack/placement master: Added alembic environment  https://review.openstack.org/61144116:32
openstackgerritEd Leafe proposed openstack/placement master: Clean up the old migrations and models  https://review.openstack.org/61144016:32
edleafecdent: if you're so inclilned ^^16:32
edleafefunctional tests still failing for not being able to see the tables16:33
openstackgerritsean mooney proposed openstack/placement master: harden placement init under wsgi  https://review.openstack.org/61244416:34
cdentedleafe: aye aye16:34
*** ttsiouts has joined #openstack-placement16:36
cdentedleafe: 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 IRC16:39
edleafeIt's not wired up16:39
edleafeThe call to alembic is creating the tables and such, but I can't figure out how to link the sqla engine to that db16:40
cdentedleafe: okay, thanks, will see if I can figure anything out, as I'm curious etc16:43
*** efried has quit IRC16:49
*** efried has joined #openstack-placement16:49
*** e0ne has joined #openstack-placement16:52
*** e0ne has quit IRC17:06
*** rubasov has quit IRC17:18
*** rubasov has joined #openstack-placement17:21
*** rubasov has quit IRC17:27
*** e0ne has joined #openstack-placement17:30
*** irclogbot_4 has joined #openstack-placement17:34
sean-k-mooneyo/17:51
sean-k-mooneyanyone have time to look at https://review.openstack.org/#/q/I704196711d30c1124e713ac31111a8ea6fa2f1ba17:52
sean-k-mooneythey should be as close to identical as i can make them realistically17:52
*** irclogbot_4 has quit IRC17:56
*** ttsiouts has joined #openstack-placement17:58
*** irclogbot_4 has joined #openstack-placement18:06
*** e0ne has quit IRC18:24
*** rubasov has joined #openstack-placement18:38
*** rubasov has quit IRC18:47
cdentedleafe: you still around. I've made some headway18:56
edleafeyeah, but I have a meeting in 3 minutes18:57
cdentjust wanted to know what you wanted me to do with the stuff18:57
cdentshould I push up a new patchset with some notes?18:58
edleafeIf you have improvements, push a new revision18:58
cdentroger that18:58
edleafeGreat. Maybe I'll learn something about sqla :)18:58
cdentafter I do that I pass the ball back to you. It's not perfect yet, but closer18:58
cdentone thing not yet fixed: the allocations table isn't included in the initial migration18:58
edleafeWas it something simple I was missing? Just curious18:59
cdentthe env.py needs to be customized for "our" environment18:59
cdentit's used whenever an alembic command is called18:59
cdentso that needs to know about placement.conf, essentially19:00
*** ttsiouts has quit IRC19:01
openstackgerritChris Dent proposed openstack/placement master: Added alembic environment  https://review.openstack.org/61144119:06
openstackgerritChris Dent proposed openstack/placement master: Clean up the old migrations and models  https://review.openstack.org/61144019:07
*** irclogbot_4 has quit IRC19:14
edleafecdent: thanks - will look at those soon19:18
*** ttsiouts has joined #openstack-placement19:18
*** ttsiouts has quit IRC20:04
*** ttsiouts has joined #openstack-placement20:05
*** ttsiouts has quit IRC20:09
openstackgerritEric Fried proposed openstack/nova-specs master: Spec: Provider config YAML file  https://review.openstack.org/61249720:16
efriedcdent, jaypipes: ^20:16
cdentenqueued20:17
jaypipesalso enqueued20:18
cdenti r dun20:22
cdentnight night20:22
*** cdent has quit IRC20:22
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/61250020:27
*** ttsiouts has joined #openstack-placement20:39
*** ttsiouts has quit IRC20:43
*** ttsiouts has joined #openstack-placement21:01
openstackgerritEric Fried proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/61250021:30
*** mriedem is now known as mriedem_away22:07
openstackgerritMerged openstack/placement master: Add recreate test for bug 1798163  https://review.openstack.org/61116422:12
openstackbug 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
openstackgerritMerged openstack/placement master: Use unique consumer_id when doing online data migration  https://review.openstack.org/61116522:13
openstackgerritAdam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances  https://review.openstack.org/60977922:23
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Per-instance serial number (spec)  https://review.openstack.org/61253122:56
*** mriedem_away has quit IRC23:27

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!