Monday, 2019-07-01

*** tetsuro has joined #openstack-placement00:40
*** efried has joined #openstack-placement00:43
*** tetsuro has quit IRC01:25
*** tetsuro has joined #openstack-placement01:27
*** tetsuro has quit IRC01:32
*** tetsuro has joined #openstack-placement01:33
*** tetsuro has quit IRC01:37
*** tetsuro has joined #openstack-placement04:31
*** tetsuro has quit IRC04:41
*** tetsuro has joined #openstack-placement04:52
*** tetsuro has quit IRC05:30
*** tetsuro has joined #openstack-placement05:34
*** tetsuro has quit IRC05:41
*** tetsuro has joined #openstack-placement07:02
*** tetsuro has quit IRC07:22
*** tetsuro has joined #openstack-placement07:23
*** tssurya has joined #openstack-placement07:29
*** ttsiouts has joined #openstack-placement07:38
openstackgerritTetsuro Nakamura proposed openstack/placement master: WIP: Support resourceless request before same_subtree  https://review.opendev.org/66300907:40
openstackgerritTetsuro Nakamura proposed openstack/placement master: WIP: Support `same_subtree` queryparam  https://review.opendev.org/66837607:40
*** helenafm has joined #openstack-placement07:48
*** ttsiouts has quit IRC07:49
*** ttsiouts has joined #openstack-placement07:49
*** tetsuro has quit IRC07:50
*** ttsiouts has quit IRC07:54
*** ttsiouts has joined #openstack-placement08:16
*** e0ne has joined #openstack-placement08:40
*** takashin has quit IRC09:18
*** cdent has joined #openstack-placement09:55
*** ttsiouts has quit IRC10:19
*** ttsiouts has joined #openstack-placement10:20
*** ttsiouts has quit IRC10:25
tssuryacdent: welcome back10:51
cdentthanks, hello10:51
tssuryahope you had great hols10:51
tssuryaconsumer types spec is almost merged10:51
tssuryawaiting for your final ack10:51
tssuryaalso I had some doubts about the implementation.. how do I geenrate the version script https://github.com/openstack/placement/tree/master/placement/db/sqlalchemy/alembic/versions10:52
tssuryawith the correct revision number ?10:52
tssuryaI couldn;t find it in the contributor docs10:52
cdentgreat, will be catching up on that stuff today and tomorrow10:52
tssuryaunless I messed up the reading order10:52
cdentthere's a section, but probably hard to find, one sec10:52
cdenthttps://docs.openstack.org/placement/latest/contributor/index.html#database-schema-changes The version number and a stub script is generated for you10:53
tssuryagreat thanks cdent, sorry I became blind to that section10:54
tssuryaalthough I did go through that particular doc10:54
cdentdocs never work as well as we would all like10:56
*** takashin has joined #openstack-placement11:06
*** takashin has quit IRC11:49
*** ttsiouts has joined #openstack-placement11:50
openstackgerritChris Dent proposed openstack/placement master: Fix up some inaccuracies in perfload comments and logs  https://review.opendev.org/66844911:55
*** takashin has joined #openstack-placement11:58
*** cdent has quit IRC12:55
*** ttsiouts has quit IRC13:00
*** takashin has left #openstack-placement13:00
*** ttsiouts has joined #openstack-placement13:01
*** cdent has joined #openstack-placement13:01
*** ttsiouts has quit IRC13:05
openstackgerritMerged openstack/placement master: Bump os-traits minimum to 0.15.0  https://review.opendev.org/66827813:10
openstackgerritMerged openstack/placement master: Remove gate/post_test_hook.sh  https://review.opendev.org/66799613:10
*** ttsiouts has joined #openstack-placement13:14
efriedo/ cdent, how was your break?13:17
cdentbit exhausting...much driving13:18
cdentit wasn't really a break. it was more "go do the job of driving family around"13:18
efriedone of those holidays you come back from feeling like you need a vacation.13:18
efriedwell, hopefully you were able to give your OpenStack muscles a needed rest.13:19
cdentyeah, i'm out of practice13:20
gibicdent: welcome back!13:21
cdentthanks13:21
efriedcdent: did we decide resourceless could only be suffixed? I don't see that in the spec.13:24
cdentthat's my recollection, but only based on remembering you saying so, as a result of root_required coming into existence13:24
* cdent is not yet up to date on the-recently-merged13:25
efriedah, by extension, since same_subtree doesn't apply to unsuffixed (which *is* in the spec), and we're requiring resourceless suffixes to exist in same_subtree... yes, resourceless can only apply to suffixed.13:25
efriedcdent: Nothing is recently merged. Everything needs your +2W :)13:25
*** mriedem has joined #openstack-placement13:27
cdenti was thinking that nested magic 1 spec merged after I faded out13:27
cdent(but I had already +2d it anyway, so...)13:27
*** amodi has joined #openstack-placement13:30
openstackgerritMerged openstack/placement master: Centralize and clarify pip in the docs  https://review.opendev.org/66738313:48
cdentefried, mriedem : you both willing to be release liaison types for placement?13:53
efriedsure13:54
*** belmoreira has joined #openstack-placement14:07
mriedemsure14:21
cdenttssurya: i had a couple of questions on the consumer type spec, clarifications mostly14:32
*** dklyle has joined #openstack-placement14:37
efriedcdent: Nice work by tetsuro an same_subtree and resourceless.14:50
cdentyeah, reading through that now14:51
*** belmoreira has quit IRC14:53
*** belmoreira has joined #openstack-placement15:00
efriedcdent: I was trying to get a start on that over the weekend when I ran across that bug, which took me the rest of the day to sort out.15:01
efriedI was actually going to approach it by porting (pieces of) ProviderTree from nova.15:02
tssuryacdent: ack looking at them now15:04
cdentefried: well thank goodness you got distracted then ;)15:04
tssuryacdent: would you like me to revise the current patch and you FA or do you prefer a FUP ?15:06
cdenttssurya: whichever is easier for you. I think eric is around enough that destroying his +2 will not set us back timewise, if you prefer to fix in placde15:07
tssuryacdent: yea I 'll update the current spec15:07
cdentwould be useful, in fact, to get Eric or Matt's input on the migration query15:07
tssuryayou mean completely alembic versus separate online migration ?15:08
tssuryaformer has the advantage of not having to wait around for people to run the migration and deal with NULL stuff. but yea from perf perspective I am not sure15:09
cdentI think we should at least mention that some investigation will be done to choose from the options15:12
efriedI can't spell alembic15:15
efriedcdent: in case it wasn't clear, that pip-in-docs patch was a response to a "bug" opened by someone. Not something I would have thought to do on my own.15:15
cdentefried: yeah, I saw that bug when it was created and I was very ambivalent about doing anything about it, but since you did, that's an good add15:16
cdentI feel bad about people needing to be told about pip and pip3...15:17
efriedkind of thing where addressing it is more to encourage people to keep opening bugs and stuff15:17
efriedand not get discouraged because their bug gets ignored or closed as don't-care.15:17
* cdent nods15:17
cdentthe rest of the content (the not pip3 part) is very useful, so as a bit of inspriration it was very useful15:18
efriedcdent, tssurya: I looked at the consumer types spec again and can confirm: I can't weigh in on the data migration issue.15:21
efriedand15:21
efriedI'm around to re+2, or +2 a fup, either way is fine with me.15:21
efriedalex_xu's +2 carries with these minor adjustments IMO.15:22
openstackgerritSurya Seetharaman proposed openstack/placement master: Spec: Support Consumer Types  https://review.opendev.org/65479915:45
efriedtssurya: tweak that regex real quick? or I can.15:48
*** ttsiouts has quit IRC15:49
*** ttsiouts has joined #openstack-placement15:49
openstackgerritEric Fried proposed openstack/placement master: Spec: Support Consumer Types  https://review.opendev.org/65479915:50
efrieddone15:50
efriedand +215:50
cdent+W15:52
*** tssurya has quit IRC15:52
*** helenafm has quit IRC15:53
*** ttsiouts has quit IRC15:54
efriedcdent: https://review.opendev.org/657510 dead?15:56
cdentyeah, abandoned15:56
efriedthx15:57
openstackgerritEric Fried proposed openstack/placement master: Fix up some inaccuracies in perfload comments and logs  https://review.opendev.org/66844915:58
efriedrebase to pick up os-traits bump15:59
*** e0ne has quit IRC16:00
*** belmoreira has quit IRC16:08
*** tssurya has joined #openstack-placement16:36
*** cdent has quit IRC16:36
*** cdent has joined #openstack-placement16:42
cdentefried: i'm outtie shortly. left some comments on your mappings fixes. gonna think on the "put the suffix somewhere else" thing but unless I have a revelation...meh16:54
cdentit's looking like having it on AR might not be too bad16:56
cdentbut we'll see16:56
* cdent waves goodnight16:56
*** cdent has quit IRC16:56
*** tssurya has quit IRC16:57
*** efried has quit IRC17:30
*** efried has joined #openstack-placement17:38
mriedemspeaking of releases, we should consider doing an osc-placement release18:53
*** jaypipes has joined #openstack-placement21:15
openstackgerritMerged openstack/placement master: Spec: Support Consumer Types  https://review.opendev.org/65479921:19
*** mriedem has quit IRC22:13
*** jaypipes has quit IRC22:35

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