Wednesday, 2018-11-21

*** tetsuro has quit IRC00:07
*** tetsuro has joined #openstack-placement00:10
*** mriedem has quit IRC00:17
openstackgerritTetsuro Nakamura proposed openstack/placement master: WIP: Set root_provider_id in database  https://review.openstack.org/61912601:17
openstackgerritMerged openstack/placement master: Add recreate test for bug 1799892  https://review.openstack.org/61703104:14
openstackbug 1799892 in OpenStack Compute (nova) rocky "Placement API crashes with 500s in Rocky upgrade with downed compute nodes" [Medium,In progress] https://launchpad.net/bugs/1799892 - Assigned to Matt Riedemann (mriedem)04:14
openstackgerritMerged openstack/placement master: Consider root id is None in the database case  https://review.openstack.org/61904104:17
*** diga has quit IRC06:00
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Amend the detach-boot-volume design  https://review.openstack.org/61916107:37
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Make scheduling weight more granular  https://review.openstack.org/59930807:38
openstackgerritTetsuro Nakamura proposed openstack/placement master: Set root_provider_id in the database  https://review.openstack.org/61912607:57
*** sean-k-mooney has quit IRC08:06
*** helenafm has joined #openstack-placement08:21
*** takashin has left #openstack-placement08:30
*** dtantsur|afk is now known as dtantsur08:42
openstackgerritTetsuro Nakamura proposed openstack/placement master: Set root_provider_id in the database  https://review.openstack.org/61912608:51
*** e0ne has joined #openstack-placement09:03
*** tetsuro has quit IRC09:12
*** tssurya has joined #openstack-placement09:32
*** cdent has joined #openstack-placement09:37
*** gibi has quit IRC10:01
*** gibi has joined #openstack-placement10:08
openstackgerritChris Dent proposed openstack/placement master: Initial integration test using gabbits  https://review.openstack.org/60161410:41
openstackgerritChris Dent proposed openstack/placement master: Use gabbi-tempest job from gabbi-tempest plugin  https://review.openstack.org/60750810:41
openstackgerritChris Dent proposed openstack/placement master: WIP: Add placeload to integration test  https://review.openstack.org/60248410:41
openstackgerritChris Dent proposed openstack/placement master: WIP: Add a perfload job.  https://review.openstack.org/60248411:01
openstackgerritChris Dent proposed openstack/placement master: WIP: Add a perfload job.  https://review.openstack.org/60248411:04
openstackgerritChris Dent proposed openstack/placement master: WIP: Add a perfload job.  https://review.openstack.org/60248411:31
cdentgetting so close to goodness11:32
cdentgibi: yesterday I abandoned some of the other work related using placement from nova and the fixtures. Some of my stuff, some of your stuff. Seemed like noise in the face of the thing that we seem to have settled on.11:44
*** sean-k-mooney has joined #openstack-placement11:46
cdentmornin' sean-k-mooney11:51
sean-k-mooneyo/11:52
sean-k-mooneyfor some reason my router disconnected at 8am so getting my vpn working and reconnecting to irc11:53
sean-k-mooneyhow are things in placement land :)11:54
*** tssurya has quit IRC11:55
cdentwe merge a lot of code, so now there's a placement-manage and placement-status command12:01
cdentwhich means that the devstack change can merge soon12:01
cdentat which point it will be time to write a lot of documentation12:01
sean-k-mooneyah yes i saw there was a ML thread on placement-manage12:01
sean-k-mooneydid not have a chance to read it12:01
sean-k-mooneycdent: and update all the other installers to use split out placement12:02
cdentat least tripleo and osa are making some pretty good headway on that already12:02
sean-k-mooneyya. kolla should proably change its containers to refect that placement is a seperate service form nova12:03
gibicdent: thanks for the cleanup.12:07
openstackgerritJan Gutter proposed openstack/nova-specs master: Spec to implement os-vif generic datapath offloads  https://review.openstack.org/60761012:10
cdentsean-k-mooney: yes, true12:10
cdentI wish they would stop using apache as well :)12:10
sean-k-mooneyya12:11
sean-k-mooneythey are open to it12:11
sean-k-mooneywould the wsgi config need to change or can the binary just be swapped12:11
dtantsurcdent: hey, any plans for microversion negotiation in the placement OSC plugin?12:13
sean-k-mooneydtantsur: microvirsion in osc is a topic that goes beyond placement12:14
cdentdtantsur: I've not had much involvement with the plugin, but yeah, what sean-k-mooney said12:14
dtantsursean-k-mooney: well, ironic does it :)12:14
dtantsuressentially, you need to pick the latest version that is known to both client and server12:14
cdentoh, that kind of negotiation12:14
dtantsurI can show anyone interested the relevant ironicclient bits12:14
cdentyou don't mean choosing12:14
dtantsurwell, it's good enough to not confuse people with Operation or argument is not supported with version 1.012:15
sean-k-mooneydtantsur: plugins can do it but osc does not allow the intree service to do the negociation12:15
dtantsursean-k-mooney: who should I talk to about changing this? :)12:15
dtantsurthis is really, REALLY bad for user experience12:15
sean-k-mooneythat will likely be change after osc moves to openstacksdk12:15
dtantsurvery likely indeed12:16
sean-k-mooneywhat i ment is we discussed it at the PTG and they decided that they would change it after that point if people wanted it still12:16
cdentdtantsur: the "who" is a good question. I don't know. I've made a point of avoiding involvement in client-side stuff because my preference is to just use requests :)12:16
cdentor curl even12:16
dtantsuryou don't even support "latest", cmon folks :)12:16
sean-k-mooneydtantsur: again ironic should not be supporting this technically12:17
cdentdtantsur: who is "you"?12:17
dtantsurwell some abstract "you" that claims that "latest" is not a supported version12:17
dtantsurwhich now I remember may be OSC itself. we had to fight with it in ironicclient12:17
* dtantsur can show ironicclient as a collection of OSC-fighting hacks :D12:18
sean-k-mooneyits not that it cant be supported it was not allowed to be for intree services and was not ment to be in plugins for consitancy12:18
dtantsurwell, practicality beats purity. we did it and we do not regret it at all.12:18
cdentdtantsur: I get the impression you're trying to do things with the plugin in a way that the plugin is not oriented towards12:18
dtantsurif by that you mean "try to provide decent user experience with microversions", then yes :)12:19
sean-k-mooneydtantsur: i am one of the people that wants microversion support in osc by the way12:19
cdentdtantsure: I mean what info are you trying to use/have that you aren't able to do now?12:19
dtantsursean-k-mooney: you have a passionate ally here :)12:19
sean-k-mooneydtantsur: consitent user expericance is required for that. if ironic is the only service that negociates that is not a good user experiance12:20
dtantsurcdent: back to placement: I tried openstack resource provider list --resource CUSTOM_BAREMETAL=112:20
dtantsurI naively expected it to just work12:20
dtantsursean-k-mooney: well, ironic is one of few services that do microversions at all. for other services new features just work.12:20
sean-k-mooneydtantsur: it may but what did you want it to do12:20
sean-k-mooneydid you want it to list all RP with that Resouce class or all with that resouce class and free inventory12:21
dtantsursean-k-mooney: I expected it NOT to return Operation or argument is not supported with version 1.012:21
sean-k-mooneydtantsur: ah ok you just wanted the mircroverios to work12:22
dtantsuroverall I'm just playing with Placement, trying to understand how it works. I don't have any specific goal here.12:22
dtantsuryep12:22
dtantsurnow, with the correct microversions explicitly specified, the result is still kind of surprising to me. but now I think it's because I don't understand Placement :)12:23
sean-k-mooneydtantsur: the current behavior is it will show RP that have a resouce of that type free12:23
cdentdtantsur: presumably you followed that with a '--os-placement-api-version 1.4' and it did work. I agree that not being able to say 'latest' there is  pain. However, if your goal is to "understand how placement works" I strongly suggest you avoid using a client. Use something that lets you do http and see it: curl, httppie, gabbi12:23
sean-k-mooneydtantsur: cdent: has a spec for allowing different query to list all resource providers with a resource  i think12:24
dtantsurI care more about free resources, but thanks12:25
cdentdtantsur: I'm not trying to be (at least not intentionally) difficult here, just trying to understand what you personally are trying to do, and what has helped me in similar cases12:25
dtantsuranyway, I think I got what I wanted, and I complained about microversions :) thanks all!12:26
cdentand for me, at least, clients _always_ get in the way12:26
dtantsurheh12:26
dtantsurI have to deal with field/QE ppl using OSC stuff. they don't want curl, and they don't want to do --os-placement-api-version :)12:26
cdentsure, but that's not the same as "I'm just playing with Placement, trying to understand how it works"12:27
dtantsurright, right. two parallel things: "I'm learning placement" and "Oh, btw, you don't do microversion negotiation, that's bad"12:27
cdentplease raise a bug on osc-placement if you have the energy to do so. the main person who was doing that may have gone on to other pastures, so I'm not sure who/how that is going to go12:28
openstackgerritChris Dent proposed openstack/placement master: WIP: Add a perfload job.  https://review.openstack.org/60248412:28
dtantsurcdent: is it storyboard or lp?12:28
cdentI'm not willing for it to be me, because of the aforementioned meh about clients, but also because if I say yes to everything I'll die and nobody else will do anything12:29
cdentI believe https://launchpad.net/placement-osc-plugin12:29
dtantsurlol, will do12:30
*** e0ne has quit IRC12:45
*** dtantsur is now known as dtantsur|brb12:47
openstackgerritChris Dent proposed openstack/placement master: WIP: Add a perfload job.  https://review.openstack.org/60248412:51
*** tssurya has joined #openstack-placement13:02
*** e0ne has joined #openstack-placement13:02
*** e0ne has quit IRC13:03
*** e0ne has joined #openstack-placement13:03
cdentjaypipes: here's an entirely unrealistic bug you might find amusing: https://bugs.launchpad.net/nova/+bug/180445313:33
openstackLaunchpad bug 1804453 in OpenStack Compute (nova) "maximum recursion possible while setting aggregates in placement" [Undecided,New]13:33
jaypipescdent: hmm.13:34
cdentthe log file helps to make it a big more clear about what's going on13:35
*** mriedem has joined #openstack-placement13:49
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924814:01
*** dtantsur|brb is now known as dtantsur14:07
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924814:24
cdenttry try again14:24
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924814:28
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924815:10
*** lyarwood has joined #openstack-placement15:21
openstackgerritChris Dent proposed openstack/placement master: Documentation cleanup: front page  https://review.openstack.org/61927315:36
*** tssurya has quit IRC15:50
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924816:12
openstackgerritChris Dent proposed openstack/placement master: Remove keystoneauth1 opts from placement config group  https://review.openstack.org/61929916:19
openstackgerritChris Dent proposed openstack/placement master: Documentation cleanup: front page  https://review.openstack.org/61927316:33
dtantsurcdent: https://bugs.launchpad.net/placement-osc-plugin/+bug/180448916:45
openstackLaunchpad bug 1804489 in placement-osc-plugin "Microversion negotiation support" [Undecided,New]16:45
cdentthanks16:46
*** efried_back_mon has quit IRC16:51
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924817:00
*** helenafm has quit IRC17:02
*** e0ne has quit IRC17:02
*** alex_xu has quit IRC17:17
*** alex_xu has joined #openstack-placement17:19
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924817:30
*** dtantsur is now known as dtantsur|afk17:49
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924817:50
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924817:58
*** mriedem has quit IRC18:26
openstackgerritChris Dent proposed openstack/placement master: Add a perfload job.  https://review.openstack.org/60248418:29
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924818:29
*** mriedem has joined #openstack-placement18:37
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924818:52
openstackgerritChris Dent proposed openstack/placement master: WIP: Use a smaller base job for the perfload run  https://review.openstack.org/61924819:08
openstackgerritChris Dent proposed openstack/placement master: Use a smaller base job for the perfload run  https://review.openstack.org/61924819:30
*** e0ne has joined #openstack-placement19:33
openstackgerritChris Dent proposed openstack/placement master: Remove keystoneauth1 opts from placement config group  https://review.openstack.org/61929919:36
openstackgerritChris Dent proposed openstack/placement master: Use a smaller base job for the perfload run  https://review.openstack.org/61924819:58
cdentjaypipes: I updated that bug https://bugs.launchpad.net/nova/+bug/1804453 with a bit more info. Turns out that it had an impact while trying to make a better placeload perf job: https://review.openstack.org/#/c/619248/20:16
openstackLaunchpad bug 1804453 in OpenStack Compute (nova) "maximum recursion possible while setting aggregates in placement" [Undecided,New]20:16
cdenteven when it doesn't reach recursion it causes the server to get very congested and the database to get overloaded20:16
cdentand that's the end of my too long coding day. goodnight all. happy thanksgiving to those of you doing that.20:18
*** cdent has quit IRC20:18
*** e0ne has quit IRC21:32
openstackgerritmelanie witt proposed openstack/nova-specs master: Propose counting quota usage from placement and API database  https://review.openstack.org/50904221:44
*** mriedem has quit IRC21:59

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