Tuesday, 2018-12-18

*** takashin has joined #openstack-placement00:04
*** N3l1x_ has quit IRC02:02
*** N3l1x_ has joined #openstack-placement02:04
*** alex_xu has joined #openstack-placement02:18
*** mriedem has joined #openstack-placement03:49
openstackgerritMerged openstack/placement master: Add stamp DB version to the migration script  https://review.openstack.org/62173304:00
*** mriedem has quit IRC04:31
*** mgagne_ has quit IRC06:35
*** mgagne has joined #openstack-placement06:39
*** e0ne has joined #openstack-placement07:11
*** avolkov has joined #openstack-placement07:41
*** takashin has left #openstack-placement08:01
*** tetsuro has joined #openstack-placement08:42
*** tetsuro has quit IRC08:47
*** tetsuro has joined #openstack-placement08:48
*** diga has joined #openstack-placement08:48
*** egonzalez has joined #openstack-placement09:41
*** cdent has joined #openstack-placement09:42
egonzalezhi, working on the placement split in kolla-ansible, i'm having errors while doing nova-status upgrade check09:42
egonzalezThere are no compute resource providers in the Placement service but there are 1 compute nodes in the deployment.09:43
egonzalezdeployment is working, placement is running and compute registered with VMs running on it09:43
cdentegonzalez: yeah, that's an issue that we probably need to be more verbose about: the status check is looking in the database for the resource provider count, but it is the wrong database09:44
cdentessentially: that status check needs to be removed or changed09:45
cdentthere's some discussion about that in the comments on this series: https://review.openstack.org/#/c/617941/09:45
egonzalezcdent aha, so fix for now is not run the upgrade check until it looks in the proper site?09:45
cdentpretty much. if you have the chance could you make a bug (in nova) so that we can track this issue. Or if you haven't got time right now, just say and I'll get it a bit later09:47
egonzalezcdent thanks for quick response, will open the bug09:48
cdentegonzalez: thank you. I'm glad to hear you were able to creat a working environment. Any other intersting discoveries?09:49
egonzalezcdent for now only working deployment and upgrade in same version for centos/oraclelinux09:50
egonzalezhavent seen any issue so far, now will work on the db migration09:50
*** ttsiouts has joined #openstack-placement10:10
*** avolkov has quit IRC10:14
*** diga has quit IRC10:57
stephenfincdent: Two trivial enough osc-placement patches here, if you fancy taking a look https://review.openstack.org/#/c/611883/ https://review.openstack.org/#/c/611884/10:58
cdentcan do10:59
cdentalthough my expertise in osc-placement is limited11:00
stephenfinAck. I will note that they are actually trivial though :)11:00
stephenfinmore argparse/tox related than anything11:01
cdentthe tox one i was happy to +w11:01
cdentmoving on11:01
cdentthe other one I'm happy to +2 (I had forgotten we just moved osc-placement somewhere I can do that) but moar people required11:05
stephenfincdent++11:08
*** ttsiouts has quit IRC11:17
*** ttsiouts has joined #openstack-placement11:18
*** sean-k-mooney has quit IRC11:21
*** ttsiouts has quit IRC11:22
openstackgerritMerged openstack/osc-placement master: tox: Hide deprecation warnings from stdlib  https://review.openstack.org/61188411:42
*** ttsiouts has joined #openstack-placement11:47
openstackgerritMerged openstack/osc-placement master: Enforce key-value'ness for 'allocation candidate list --resource'  https://review.openstack.org/61188311:53
*** e0ne has quit IRC12:53
*** sean-k-mooney has joined #openstack-placement13:31
jaypipesedleafe, cdent: left some thoughts on https://bugs.launchpad.net/nova/+bug/1778591. Curious about your thoughts on it.13:39
openstackLaunchpad bug 1778591 in OpenStack Compute (nova) "GET /allocations/{uuid} on a consumer with no allocations provides no generation" [Medium,In progress]13:39
cdentjaypipes: that helps solidify things a bit, thanks for doing that. I need to digest it a bit more before having any real thoughts.13:43
jaypipescdent: ya, no worries. total stream of thought from my end anyway :)13:50
cdentI hope it is clear by now that I yearn daily for more streams of thought13:50
jaypipes:P13:55
*** e0ne has joined #openstack-placement13:57
*** e0ne has quit IRC14:07
*** mriedem has joined #openstack-placement14:27
*** e0ne has joined #openstack-placement14:43
*** alex_xu has quit IRC14:44
*** ttsiouts has quit IRC14:45
*** ttsiouts has joined #openstack-placement14:46
*** alex_xu has joined #openstack-placement14:46
*** ttsiouts has quit IRC14:50
*** ttsiouts has joined #openstack-placement14:58
openstackgerritJack Ding proposed openstack/nova-specs master: Select cpu model from a list of cpu models  https://review.openstack.org/62095915:52
openstackgerritMerged openstack/placement master: Remove writer context from _ensure_aggregates  https://review.openstack.org/62414316:04
openstackgerritMerged openstack/placement master: Retry _ensure_aggregates a limited number of times  https://review.openstack.org/62414416:04
*** e0ne has quit IRC16:50
*** e0ne has joined #openstack-placement17:13
*** ttsiouts has quit IRC17:14
*** ttsiouts has joined #openstack-placement17:15
*** e0ne has quit IRC17:19
*** ttsiouts has quit IRC17:19
mriedemjaypipes: how are you feeling about tetsuro's forbidden aggregates spec? https://review.openstack.org/#/c/603352/17:44
mriedemwe're in feature freeze for the placement api but we can merge specs at least before the jan 10 spec freeze17:44
mriedemi haven't been back on tushar's spec that depends on ^ yet though https://review.openstack.org/#/c/609960/17:44
mriedemit was super unclear when i first read it17:45
jaypipesmriedem: you'd be ok merging one and not the other?17:48
mriedemi would feel better if we had a solid plan for the issue tushar is trying to solve in nova,17:50
mriedembut forbidden aggregates seems worthwhile on its own merit, and it sounds like tetsuro would know how to use it with blazar17:51
cdentit's a rather nice and useful generic operation and it is pretty clear blazar would like it17:51
mriedemjaypipes: not placement related, but can i also get you to take a gander at this fairly straight-forward nova api spec https://review.openstack.org/#/c/393930/18:29
mriedemjust shows the device tags out of GET requests18:29
mriedemwe've been dry humping that idea for awhile now18:29
jaypipesmriedem: k, will try to get to it.18:31
*** e0ne has joined #openstack-placement18:34
*** e0ne has quit IRC18:40
*** e0ne has joined #openstack-placement19:05
*** cdent has quit IRC19:20
*** mriedem is now known as mriedem_afk19:22
*** mriedem_afk is now known as mriedem20:50
*** mriedem has quit IRC21:02
*** mriedem has joined #openstack-placement21:04
*** e0ne has quit IRC21:51
openstackgerritJack Ding proposed openstack/nova-specs master: Flavor Extra Spec and Image Properties Validation  https://review.openstack.org/61854222:12
*** N3l1x_ has quit IRC22:14
openstackgerritJack Ding proposed openstack/nova-specs master: Select cpu model from a list of cpu models  https://review.openstack.org/62095923:03
openstackgerritJack Ding proposed openstack/nova-specs master: Flavor Extra Spec and Image Properties Validation  https://review.openstack.org/61854223:07
*** efried has joined #openstack-placement23:29
*** efried has quit IRC23:34

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