*** tetsuro has joined #openstack-placement | 00:40 | |
*** efried has joined #openstack-placement | 00:43 | |
*** tetsuro has quit IRC | 01:25 | |
*** tetsuro has joined #openstack-placement | 01:27 | |
*** tetsuro has quit IRC | 01:32 | |
*** tetsuro has joined #openstack-placement | 01:33 | |
*** tetsuro has quit IRC | 01:37 | |
*** tetsuro has joined #openstack-placement | 04:31 | |
*** tetsuro has quit IRC | 04:41 | |
*** tetsuro has joined #openstack-placement | 04:52 | |
*** tetsuro has quit IRC | 05:30 | |
*** tetsuro has joined #openstack-placement | 05:34 | |
*** tetsuro has quit IRC | 05:41 | |
*** tetsuro has joined #openstack-placement | 07:02 | |
*** tetsuro has quit IRC | 07:22 | |
*** tetsuro has joined #openstack-placement | 07:23 | |
*** tssurya has joined #openstack-placement | 07:29 | |
*** ttsiouts has joined #openstack-placement | 07:38 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/placement master: WIP: Support resourceless request before same_subtree https://review.opendev.org/663009 | 07:40 |
---|---|---|
openstackgerrit | Tetsuro Nakamura proposed openstack/placement master: WIP: Support `same_subtree` queryparam https://review.opendev.org/668376 | 07:40 |
*** helenafm has joined #openstack-placement | 07:48 | |
*** ttsiouts has quit IRC | 07:49 | |
*** ttsiouts has joined #openstack-placement | 07:49 | |
*** tetsuro has quit IRC | 07:50 | |
*** ttsiouts has quit IRC | 07:54 | |
*** ttsiouts has joined #openstack-placement | 08:16 | |
*** e0ne has joined #openstack-placement | 08:40 | |
*** takashin has quit IRC | 09:18 | |
*** cdent has joined #openstack-placement | 09:55 | |
*** ttsiouts has quit IRC | 10:19 | |
*** ttsiouts has joined #openstack-placement | 10:20 | |
*** ttsiouts has quit IRC | 10:25 | |
tssurya | cdent: welcome back | 10:51 |
cdent | thanks, hello | 10:51 |
tssurya | hope you had great hols | 10:51 |
tssurya | consumer types spec is almost merged | 10:51 |
tssurya | waiting for your final ack | 10:51 |
tssurya | also 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/versions | 10:52 |
tssurya | with the correct revision number ? | 10:52 |
tssurya | I couldn;t find it in the contributor docs | 10:52 |
cdent | great, will be catching up on that stuff today and tomorrow | 10:52 |
tssurya | unless I messed up the reading order | 10:52 |
cdent | there's a section, but probably hard to find, one sec | 10:52 |
cdent | https://docs.openstack.org/placement/latest/contributor/index.html#database-schema-changes The version number and a stub script is generated for you | 10:53 |
tssurya | great thanks cdent, sorry I became blind to that section | 10:54 |
tssurya | although I did go through that particular doc | 10:54 |
cdent | docs never work as well as we would all like | 10:56 |
*** takashin has joined #openstack-placement | 11:06 | |
*** takashin has quit IRC | 11:49 | |
*** ttsiouts has joined #openstack-placement | 11:50 | |
openstackgerrit | Chris Dent proposed openstack/placement master: Fix up some inaccuracies in perfload comments and logs https://review.opendev.org/668449 | 11:55 |
*** takashin has joined #openstack-placement | 11:58 | |
*** cdent has quit IRC | 12:55 | |
*** ttsiouts has quit IRC | 13:00 | |
*** takashin has left #openstack-placement | 13:00 | |
*** ttsiouts has joined #openstack-placement | 13:01 | |
*** cdent has joined #openstack-placement | 13:01 | |
*** ttsiouts has quit IRC | 13:05 | |
openstackgerrit | Merged openstack/placement master: Bump os-traits minimum to 0.15.0 https://review.opendev.org/668278 | 13:10 |
openstackgerrit | Merged openstack/placement master: Remove gate/post_test_hook.sh https://review.opendev.org/667996 | 13:10 |
*** ttsiouts has joined #openstack-placement | 13:14 | |
efried | o/ cdent, how was your break? | 13:17 |
cdent | bit exhausting...much driving | 13:18 |
cdent | it wasn't really a break. it was more "go do the job of driving family around" | 13:18 |
efried | one of those holidays you come back from feeling like you need a vacation. | 13:18 |
efried | well, hopefully you were able to give your OpenStack muscles a needed rest. | 13:19 |
cdent | yeah, i'm out of practice | 13:20 |
gibi | cdent: welcome back! | 13:21 |
cdent | thanks | 13:21 |
efried | cdent: did we decide resourceless could only be suffixed? I don't see that in the spec. | 13:24 |
cdent | that's my recollection, but only based on remembering you saying so, as a result of root_required coming into existence | 13:24 |
* cdent is not yet up to date on the-recently-merged | 13:25 | |
efried | ah, 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 |
efried | cdent: Nothing is recently merged. Everything needs your +2W :) | 13:25 |
*** mriedem has joined #openstack-placement | 13:27 | |
cdent | i was thinking that nested magic 1 spec merged after I faded out | 13:27 |
cdent | (but I had already +2d it anyway, so...) | 13:27 |
*** amodi has joined #openstack-placement | 13:30 | |
openstackgerrit | Merged openstack/placement master: Centralize and clarify pip in the docs https://review.opendev.org/667383 | 13:48 |
cdent | efried, mriedem : you both willing to be release liaison types for placement? | 13:53 |
efried | sure | 13:54 |
*** belmoreira has joined #openstack-placement | 14:07 | |
mriedem | sure | 14:21 |
cdent | tssurya: i had a couple of questions on the consumer type spec, clarifications mostly | 14:32 |
*** dklyle has joined #openstack-placement | 14:37 | |
efried | cdent: Nice work by tetsuro an same_subtree and resourceless. | 14:50 |
cdent | yeah, reading through that now | 14:51 |
*** belmoreira has quit IRC | 14:53 | |
*** belmoreira has joined #openstack-placement | 15:00 | |
efried | cdent: 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 |
efried | I was actually going to approach it by porting (pieces of) ProviderTree from nova. | 15:02 |
tssurya | cdent: ack looking at them now | 15:04 |
cdent | efried: well thank goodness you got distracted then ;) | 15:04 |
tssurya | cdent: would you like me to revise the current patch and you FA or do you prefer a FUP ? | 15:06 |
cdent | tssurya: 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 placde | 15:07 |
tssurya | cdent: yea I 'll update the current spec | 15:07 |
cdent | would be useful, in fact, to get Eric or Matt's input on the migration query | 15:07 |
tssurya | you mean completely alembic versus separate online migration ? | 15:08 |
tssurya | former 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 sure | 15:09 |
cdent | I think we should at least mention that some investigation will be done to choose from the options | 15:12 |
efried | I can't spell alembic | 15:15 |
efried | cdent: 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 |
cdent | efried: 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 add | 15:16 |
cdent | I feel bad about people needing to be told about pip and pip3... | 15:17 |
efried | kind of thing where addressing it is more to encourage people to keep opening bugs and stuff | 15:17 |
efried | and not get discouraged because their bug gets ignored or closed as don't-care. | 15:17 |
* cdent nods | 15:17 | |
cdent | the rest of the content (the not pip3 part) is very useful, so as a bit of inspriration it was very useful | 15:18 |
efried | cdent, tssurya: I looked at the consumer types spec again and can confirm: I can't weigh in on the data migration issue. | 15:21 |
efried | and | 15:21 |
efried | I'm around to re+2, or +2 a fup, either way is fine with me. | 15:21 |
efried | alex_xu's +2 carries with these minor adjustments IMO. | 15:22 |
openstackgerrit | Surya Seetharaman proposed openstack/placement master: Spec: Support Consumer Types https://review.opendev.org/654799 | 15:45 |
efried | tssurya: tweak that regex real quick? or I can. | 15:48 |
*** ttsiouts has quit IRC | 15:49 | |
*** ttsiouts has joined #openstack-placement | 15:49 | |
openstackgerrit | Eric Fried proposed openstack/placement master: Spec: Support Consumer Types https://review.opendev.org/654799 | 15:50 |
efried | done | 15:50 |
efried | and +2 | 15:50 |
cdent | +W | 15:52 |
*** tssurya has quit IRC | 15:52 | |
*** helenafm has quit IRC | 15:53 | |
*** ttsiouts has quit IRC | 15:54 | |
efried | cdent: https://review.opendev.org/657510 dead? | 15:56 |
cdent | yeah, abandoned | 15:56 |
efried | thx | 15:57 |
openstackgerrit | Eric Fried proposed openstack/placement master: Fix up some inaccuracies in perfload comments and logs https://review.opendev.org/668449 | 15:58 |
efried | rebase to pick up os-traits bump | 15:59 |
*** e0ne has quit IRC | 16:00 | |
*** belmoreira has quit IRC | 16:08 | |
*** tssurya has joined #openstack-placement | 16:36 | |
*** cdent has quit IRC | 16:36 | |
*** cdent has joined #openstack-placement | 16:42 | |
cdent | efried: 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...meh | 16:54 |
cdent | it's looking like having it on AR might not be too bad | 16:56 |
cdent | but we'll see | 16:56 |
* cdent waves goodnight | 16:56 | |
*** cdent has quit IRC | 16:56 | |
*** tssurya has quit IRC | 16:57 | |
*** efried has quit IRC | 17:30 | |
*** efried has joined #openstack-placement | 17:38 | |
mriedem | speaking of releases, we should consider doing an osc-placement release | 18:53 |
*** jaypipes has joined #openstack-placement | 21:15 | |
openstackgerrit | Merged openstack/placement master: Spec: Support Consumer Types https://review.opendev.org/654799 | 21:19 |
*** mriedem has quit IRC | 22:13 | |
*** jaypipes has quit IRC | 22:35 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!