Tuesday, 2019-03-19

« Monday, 2019-03-18 Index Wednesday, 2019-03-20 »
*** tetsuro has joined #openstack-placement00:14
openstackgerritRui Zang proposed openstack/nova-specs master: support virtual persistent memory  https://review.openstack.org/60159601:54
*** zzzeek has quit IRC03:10
*** zzzeek has joined #openstack-placement03:16
openstackgerritTetsuro Nakamura proposed openstack/placement master: Create ProviderTreeDBHelperTestCase  https://review.openstack.org/64447804:26
openstackgerritTetsuro Nakamura proposed openstack/placement master: Refactor tests of _get_trees_matching_all()  https://review.openstack.org/64447904:26
*** tetsuro has quit IRC04:33
*** dklyle has quit IRC04:44
*** dklyle has joined #openstack-placement04:45
openstackgerritRui Zang proposed openstack/nova-specs master: Virtual persistent memory libvirt driver implementation  https://review.openstack.org/62289305:11
openstackgerritTetsuro Nakamura proposed openstack/placement master: Refactor tests of _get_trees_matching_all()  https://review.openstack.org/64447906:47
*** tetsuro has joined #openstack-placement07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-placement08:02
*** ttsiouts has joined #openstack-placement08:08
*** tssurya has joined #openstack-placement08:19
*** alex_xu has quit IRC08:21
*** alex_xu has joined #openstack-placement08:23
*** ttsiouts has quit IRC08:25
*** e0ne has joined #openstack-placement08:25
*** helenafm has joined #openstack-placement08:32
*** ttsiouts has joined #openstack-placement08:49
*** dklyle has quit IRC09:08
*** dklyle has joined #openstack-placement09:09
*** alex_xu has quit IRC09:18
*** alex_xu has joined #openstack-placement09:19
*** ttsiouts has quit IRC09:31
*** ttsiouts has joined #openstack-placement09:32
*** e0ne has quit IRC09:56
*** cdent has joined #openstack-placement10:27
openstackgerritChris Dent proposed openstack/placement master: Slightly improve usage documentation  https://review.openstack.org/64391310:49
*** e0ne has joined #openstack-placement10:50
openstackgerritChris Dent proposed openstack/placement master: Rename and restructure install docs  https://review.openstack.org/64391910:58
openstackgerritChris Dent proposed openstack/placement master: Update the from-pypi install instructions  https://review.openstack.org/64393810:58
openstackgerritChris Dent proposed openstack/placement master: Upgrade from rocky-nova docs  https://review.openstack.org/64439610:58
*** ttsiouts has quit IRC11:09
openstackgerritChris Dent proposed openstack/placement master: Rename and restructure install docs  https://review.openstack.org/64391911:18
openstackgerritChris Dent proposed openstack/placement master: Update the from-pypi install instructions  https://review.openstack.org/64393811:18
openstackgerritChris Dent proposed openstack/placement master: Upgrade from rocky-nova docs  https://review.openstack.org/64439611:18
*** tetsuro has quit IRC11:21
openstackgerritMerged openstack/placement master: Create ProviderTreeDBHelperTestCase  https://review.openstack.org/64447812:04
*** finucannot is now known as stephenfin12:05
*** ttsiouts has joined #openstack-placement12:10
*** mriedem has joined #openstack-placement12:54
cdentstephenfin: what's the right way to link to placement docs from releasenotes? Fully qualified url? Something else?13:06
stephenfincdent: Yeah. Release notes aren't versioned so the 'foo-doc' roles that openstackdoctheme provides aren't useful/necessary13:07
cdentcheers, thanks13:07
*** tssurya has quit IRC13:18
openstackgerritChris Dent proposed openstack/placement master: Upgrade from rocky-nova docs  https://review.openstack.org/64439613:25
openstackgerritChris Dent proposed openstack/placement master: Add prelude to release notes  https://review.openstack.org/64456713:25
openstackgerritChris Dent proposed openstack/placement master: Link to more info on service user and endpoints from deployment  https://review.openstack.org/64456813:25
cdentedleafe: there are probably words wrong in that ^ stack. your assistance requested when you have the time13:26
cdentwe get that stack done we're ready to release13:26
edleafecdent: ack. On my todo list13:26
cdentthanks13:26
efriedcdent: do those need my attention again?13:28
cdentefried: yes, especially on the big question i left on your placement Placement comment, and the lack of closing statement13:28
cdenti didn't mention you above because I didn't know you were here yet13:28
efriedI don't care a lot about the capitalization.13:28
efriedI got all the school runs this morning. A solid hour of driving. Not my favorite.13:29
cdentblargh13:32
openstackgerritChris Dent proposed openstack/placement master: Rename and restructure install docs  https://review.openstack.org/64391914:35
openstackgerritChris Dent proposed openstack/placement master: Update the from-pypi install instructions  https://review.openstack.org/64393814:35
openstackgerritChris Dent proposed openstack/placement master: Upgrade from rocky-nova docs  https://review.openstack.org/64439614:35
openstackgerritChris Dent proposed openstack/placement master: Add prelude to release notes  https://review.openstack.org/64456714:36
openstackgerritChris Dent proposed openstack/placement master: Link to more info on service user and endpoints from deployment  https://review.openstack.org/64456814:36
efriedcdent: I called you out for a once-over of a thing in https://review.openstack.org/#/c/644412/ FYI14:39
* cdent looks14:39
openstackgerritEric Fried proposed openstack/placement master: Add oslo.log to genconfig  https://review.openstack.org/64458914:47
cdentah, that makes sense14:51
openstackgerritMerged openstack/placement master: Refactor tests of _get_trees_matching_all()  https://review.openstack.org/64447914:53
openstackgerritEric Fried proposed openstack/placement master: Centralize registration of logging options  https://review.openstack.org/64459114:53
sean-k-mooneycdent: fyi i left a few nits on PS4 while you were uploading PS5 of https://review.openstack.org/#/c/644396/ just incase you miss them14:56
cdentno worries sean-k-mooney, I'm taking backwards through multiple revisions14:57
cdentneed to switch locations, will catch up on all the comments in a few minutes, thanks to everyone14:58
*** cdent has quit IRC14:59
*** cdent has joined #openstack-placement15:10
openstackgerritsahid proposed openstack/os-traits master: ci: re-enable openstack-python35-jobs  https://review.openstack.org/64459515:11
* cdent pulls his barbie string15:24
cdentdocs is hard, let's go drinking15:24
*** ttsiouts has quit IRC15:38
*** ttsiouts has joined #openstack-placement15:40
mriedemhttps://www.youtube.com/watch?v=gqeTb0j2nr415:45
* mriedem digs into docs15:53
cdentmriedem: hold on two tics and you'll get the latest versions15:53
*** ttsiouts has quit IRC15:54
openstackgerritChris Dent proposed openstack/placement master: Rename and restructure install docs  https://review.openstack.org/64391916:00
openstackgerritChris Dent proposed openstack/placement master: Update the from-pypi install instructions  https://review.openstack.org/64393816:00
openstackgerritChris Dent proposed openstack/placement master: Upgrade from rocky-nova docs  https://review.openstack.org/64439616:00
cdentthere go mriedem16:00
openstackgerritChris Dent proposed openstack/placement master: Add prelude to release notes  https://review.openstack.org/64456716:00
openstackgerritChris Dent proposed openstack/placement master: Link to more info on service user and endpoints from deployment  https://review.openstack.org/64456816:00
mriedemack16:00
cdentI think i'll use this moment to go home, then I can be sat in a comfy chair while the torrent of good advice from mriedem comes rushing in16:02
*** ttsiouts has joined #openstack-placement16:02
*** cdent has quit IRC16:06
mriedemflattery will get you everywhere16:12
*** ttsiouts has quit IRC16:25
*** e0ne has quit IRC16:32
*** ttsiouts has joined #openstack-placement16:36
*** cdent has joined #openstack-placement16:42
mriedemcdent: done https://review.openstack.org/#/c/643938/517:01
mriedemi'm setting up a devstack with just keystone installed to step through this as well17:01
cdentyay, that is a torrent17:01
mriedemsome is follow up stuff17:02
mriedemto split out shared content17:02
cdentyeah, I thought about doing some that, but had already gone down so many different holes I was getting a bit "meh, I don't want to do all the docs, dammit"17:02
* cdent reads17:03
mriedemyeah i totally understand, and don't want to burn you out with too much17:04
mriedemi'm also asking in -keystone about the www_authenticate_uri thing17:04
cdentmriedem: yeah, good, because I've struggled to find a straight answer. both the config docs and the debug logs are really ambiguous about it17:08
cdentsometimes it seems one thing, sometimes another17:08
cdentI'm pretty sure the install docs are out of date17:08
cdentespecially with regard to using or not using a versioned endpoint17:08
cdentunversioned is supposed to be right17:08
mriedemyeah devstack uses unversioned auth_url17:09
mriedemand doesn't set www_authenticate_uri at all17:09
*** helenafm has quit IRC17:13
cdentis devstack right?17:16
*** jaypipes has joined #openstack-placement17:16
mriedemdevstack probably doesn't need it17:26
mriedemi'm hitting errors running this17:26
mriedemuwsgi -M --http :80 --wsgi-file /usr/local/bin/placement-api17:26
mriedembut that's what the uwsgi docs say to do as well :/17:26
cdentmriedem: you've got two uwsgi's in your world17:28
cdentone in /usr/bin and the one you installed with pip in /usr/local/bin17:28
cdentthus the warning I stuck in there17:28
mriedemok i suppose keystone is using the former17:28
cdentthe one in /usr/bin is, in typical uwsgi fashion, a somewhat earlier version that has entirely different arguments17:29
cdentwhich is17:29
cdentsigh inducing17:29
cdentbut again, since that doc is designed to be sort of "modern" seems okay17:30
mriedemyeah weird, which uwsgi shows /usr/local/bin/uwsgi and if i run with that fully qualified it's fine17:30
mriedemok got that running now on 877817:32
mriedemalright i'm done17:36
mriedemchecking your replies17:37
*** ttsiouts has quit IRC17:40
cdentbased on your experience there with 8778 I'll change the localhost thing, despite my protests17:40
mriedemi just replied about that17:46
mriedemwith a compromise17:46
cdentthanks for that irc paste17:47
cdentyup, good comments, thanks17:49
mriedemanother thought on a less shitty verify post-install doc than just placement-status upgrade check, would be (1) list providers - there might be none and (2) $ openstack --os-placement-api-version 1.6 trait list17:49
mriedemof which there should be *something(17:49
cdenttrue17:49
mriedemi can follow up on top of this with the verify doc fleshing outing17:49
cdentare you gonna do the remaining patches in that series too? If so I'll hold off17:49
cdenti'm trying to find a reasonable hotel in london. and that's a mission17:50
mriedemprobably not this second, so just address these and then we can get this approved and i'll move on17:50
cdentaye aye17:53
openstackgerritDan Smith proposed openstack/nova-specs master: Add request-filter-image-types spec  https://review.openstack.org/64462518:12
openstackgerritChris Dent proposed openstack/placement master: Update the from-pypi install instructions  https://review.openstack.org/64393818:54
openstackgerritChris Dent proposed openstack/placement master: Upgrade from rocky-nova docs  https://review.openstack.org/64439618:55
openstackgerritChris Dent proposed openstack/placement master: Add prelude to release notes  https://review.openstack.org/64456718:55
openstackgerritChris Dent proposed openstack/placement master: Link to more info on service user and endpoints from deployment  https://review.openstack.org/64456818:55
cdentmriedem: I think I got your suggestions. Not completely positive as I'm knackered18:56
*** ttsiouts has joined #openstack-placement19:20
*** e0ne has joined #openstack-placement19:28
openstackgerritChris Dent proposed openstack/placement master: Upgrade from rocky-nova docs  https://review.openstack.org/64439619:43
openstackgerritChris Dent proposed openstack/placement master: Add prelude to release notes  https://review.openstack.org/64456719:43
openstackgerritChris Dent proposed openstack/placement master: Link to more info on service user and endpoints from deployment  https://review.openstack.org/64456819:43
mriedemack19:43
mriedemi need to coffee up first19:43
* cdent wishes he could coffee at this time of day19:44
*** e0ne has quit IRC19:51
*** ttsiouts has quit IRC20:01
*** cdent has quit IRC20:01
mriedemone small missed bullet in https://review.openstack.org/#/c/643938/ but you could FUP20:18
mriedemefried: want to hit that one again? ^20:19
*** cdent has joined #openstack-placement20:19
openstackgerritMatt Riedemann proposed openstack/placement master: Fix bullet format from I580fa4394cb93b8e8141ee2d546543c174356a47  https://review.openstack.org/64467420:19
openstackgerritMatt Riedemann proposed openstack/placement master: Fix bullet format from I580fa4394cb93b8e8141ee2d546543c174356a47  https://review.openstack.org/64467420:22
openstackgerritMerged openstack/placement master: Slightly improve usage documentation  https://review.openstack.org/64391320:23
openstackgerritMerged openstack/placement master: Add oslo.log to genconfig  https://review.openstack.org/64458920:23
openstackgerritMerged openstack/placement master: Rename and restructure install docs  https://review.openstack.org/64391920:23
*** cdent has quit IRC20:26
*** e0ne has joined #openstack-placement20:41
*** e0ne has quit IRC20:46
*** ttsiouts has joined #openstack-placement20:54
mriedemalright comments in the upgrade doc, https://review.openstack.org/#/c/644396/21:02
mriedemdansmith: you should also take a gander at ^21:02
mriedem+2 with nits and fup fodder21:05
openstackgerritMerged openstack/placement master: Update the from-pypi install instructions  https://review.openstack.org/64393821:26
*** cdent has joined #openstack-placement21:26
cdentefried, mriedem : just stopping by to confirm that neither of you are doing the upgrade-from-rocky fup. If that's the case I'll get on it21:27
efriednot I.21:28
mriedemnope21:28
mriedemi'm going to do a nova change that links to the placement to-stein doc21:28
cdentrad21:28
openstackgerritMerged openstack/placement master: Fix bullet format from I580fa4394cb93b8e8141ee2d546543c174356a47  https://review.openstack.org/64467421:28
mriedemi'm going to try and also tackle task 2994421:31
cdentmore power to you21:32
cdentI marked that one invalid in storyboard in a fit of pique21:32
openstackgerritMatt Riedemann proposed openstack/placement master: Flesh out the post-install verify doc  https://review.openstack.org/64468721:55
openstackgerritChris Dent proposed openstack/placement master: Address followups in the upgrade from nova doc  https://review.openstack.org/64469022:19
*** ttsiouts has quit IRC22:51
*** mriedem is now known as mriedem_away23:07
*** alex_xu has quit IRC23:14
*** alex_xu has joined #openstack-placement23:15
cdentmriedem_away or alex_xu : if you have a chance to look at https://review.openstack.org/#/c/644591/ that would be useful to go ahead and get in, for the sake of cleanness23:26
« Monday, 2019-03-18 Index Wednesday, 2019-03-20 »

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