| *** mriedem_lawnboy is now known as mriedem | 01:04 | |
| *** mriedem has quit IRC | 01:10 | |
| *** lei-zh has joined #openstack-placement | 01:52 | |
| *** lei-zh has quit IRC | 02:31 | |
| *** lei-zh1 has joined #openstack-placement | 02:31 | |
| openstackgerrit | Leopard Ma proposed openstack/nova master: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/598415 | 03:15 |
|---|---|---|
| openstackgerrit | huanhongda proposed openstack/nova master: Fix instance delete stuck in deleting task_state https://review.openstack.org/598084 | 03:21 |
| *** lei-zh1 has quit IRC | 03:32 | |
| *** nicolasbock has quit IRC | 03:33 | |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976 | 03:44 |
| *** jaypipes has quit IRC | 04:04 | |
| *** jaypipes has joined #openstack-placement | 04:04 | |
| *** lei-zh1 has joined #openstack-placement | 04:06 | |
| *** lei-zh1 has quit IRC | 04:11 | |
| *** lei-zh1 has joined #openstack-placement | 05:46 | |
| openstackgerrit | Leopard Ma proposed openstack/nova master: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/598415 | 06:25 |
| *** e0ne has joined #openstack-placement | 06:50 | |
| *** e0ne has quit IRC | 06:52 | |
| *** lei-zh1 has quit IRC | 06:58 | |
| *** lei-zh1 has joined #openstack-placement | 06:59 | |
| *** tssurya has joined #openstack-placement | 07:12 | |
| openstackgerrit | Chen proposed openstack/nova master: Fix SOFT_DELETE vm_state string https://review.openstack.org/598482 | 07:56 |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976 | 08:45 |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976 | 08:48 |
| *** ttsiouts has joined #openstack-placement | 08:49 | |
| *** e0ne has joined #openstack-placement | 09:01 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consumer gen support for delete instance allocations https://review.openstack.org/591597 | 09:06 |
| *** lei-zh1 has quit IRC | 09:21 | |
| openstackgerrit | Chen proposed openstack/nova master: Fix filter server list with SOFT_DELETED status https://review.openstack.org/597443 | 09:45 |
| -openstackstatus- NOTICE: Jobs using devstack-gate (legacy devstack jobs) have been failing due to an ara update. We use now a newer ansible version, it's safe to recheck if you see "ImportError: No module named manager" in the logs. | 09:57 | |
| *** ttsiouts has quit IRC | 10:09 | |
| openstackgerrit | Merged openstack/nova master: Delete instance_group_member records from API DB during archive https://review.openstack.org/588943 | 10:13 |
| *** cdent has joined #openstack-placement | 10:21 | |
| *** nicolasbock has joined #openstack-placement | 10:33 | |
| *** stephenfin is now known as finucannot | 10:46 | |
| *** gibi is now known as giblet | 10:53 | |
| *** ttsiouts has joined #openstack-placement | 11:12 | |
| cdent | giblet, efried, edleafe, jaypipes, melwitt: it lives! https://git.openstack.org/cgit/openstack/placement | 11:22 |
| giblet | woot | 11:22 |
| cdent | input is desired on https://review.openstack.org/#/c/598380/ if you haven't already provided it | 11:22 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler https://review.openstack.org/598611 | 11:38 |
| *** e0ne has quit IRC | 11:44 | |
| cdent | edleafe: I've gone ahead and done https://review.openstack.org/#/c/598602/ as my curiosity didn't want to wait. Any automation you do should probably base off that. | 11:52 |
| cdent | I've also made this change https://review.openstack.org/#/c/598603/ so that the new repo will report here | 11:52 |
| giblet | cdent: left feedback on https://review.openstack.org/#/c/598602 | 11:58 |
| giblet | soo good to see that repo separation is moving forward | 11:59 |
| cdent | thanks | 12:04 |
| cdent | yes | 12:04 |
| cdent | giblet: responded, I hope it's clear why I've chosen the way I've chosen | 12:11 |
| openstackgerrit | sahid proposed openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168 | 12:13 |
| giblet | cdent: I don't use gitreview so I basically don't know what we loose if the .gitreview file is not correct. | 12:15 |
| cdent | old school! | 12:15 |
| giblet | cdent: but if this change is needed for the gitreview users I'm fine with it | 12:15 |
| cdent | do you use some other automation, or just your own calls to git? | 12:15 |
| giblet | cdent: I just use git push review HEAD:refs/for/master/<topic> | 12:17 |
| cdent | i've become so habituated to git review that I'm pretty sure I've forgotten the underlying steps | 12:18 |
| giblet | cdent: it seems that placement-core is empty https://review.openstack.org/#/admin/groups/1936,members so nobody can approve code at the moment | 12:18 |
| cdent | cd nova; git review -v -d <id> ; do some stuff ; git review is may daily mantra | 12:18 |
| cdent | giblet: yeah, I've made a request to infra to get nova-core in there | 12:19 |
| giblet | cdent: that git review is like black magic to me :) I like to know what happen in git | 12:19 |
| cdent | I assume that's the right starting point (as in, I shouldn't have myself added and then add nova-core) | 12:19 |
| giblet | I think that was the agreed initial setup | 12:20 |
| cdent | giblet: i'm going to be away for some of the afternoon, so if there's no action on that soonish, if you have the time to poke infra about it, that would be helpful | 12:21 |
| giblet | cdent: I still have about 2 hours in the office before the the weekend so I can check back to infra | 12:22 |
| cdent | thanks | 12:22 |
| giblet | cdent: thank you for your effort on the repo separation | 12:23 |
| cdent | you're welcome, it's nice to see it finally taking shape | 12:23 |
| *** jaypipes is now known as leakypipes | 12:44 | |
| *** cdent has quit IRC | 12:55 | |
| *** e0ne has joined #openstack-placement | 12:55 | |
| *** tssurya has quit IRC | 12:59 | |
| * edleafe gets to the keyboard with a nice cup of coffee | 13:01 | |
| edleafe | Guess we need to change the gerrit bot to look at openstack/placement for this channel now | 13:17 |
| edleafe | giblet: efried: cdent: I've pushed my stack of changes on top of cdent's zuul job change, starting with https://review.openstack.org/#/c/598941/ | 13:18 |
| edleafe | ^^ These are all the mechanical movements and replacements that are needed to make placement sort of close to working. I broke them down into 10 logical steps so that they would be easy to review | 13:20 |
| *** ttsiouts has quit IRC | 13:24 | |
| *** ttsiouts has joined #openstack-placement | 13:30 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Delete instance_group_member records from API DB during archive https://review.openstack.org/598953 | 13:36 |
| giblet | edleafe: ack | 13:37 |
| giblet | cdent, edleafe, efried: infra fixed the placement-core group to contain nova-core so now we have approve rights as well | 13:37 |
| efried | woot. I'm still working through the gov patches | 13:38 |
| *** efried is now known as fried_rice | 13:38 | |
| giblet | edleafe: https://review.openstack.org/#/c/598940 will not be mergabel without a noop-jobs template | 13:39 |
| edleafe | giblet: Heh, it wouldn't import it with a .zuul.yaml file, which is why cdent added it back in the first patch | 13:40 |
| edleafe | Should I squash his patch into this? | 13:40 |
| giblet | edleafe: squash seems appropriate in this case | 13:41 |
| edleafe | giblet: do you know why cdent's patch says "Cannot Merge", while the base one doesn't? | 13:42 |
| giblet | edleafe: I have no idea | 13:42 |
| * edleafe is learning more infra than he cares to | 13:42 | |
| giblet | I'm guessing that it is the missing jobs | 13:42 |
| edleafe | So... add at least one gate job? | 13:43 |
| giblet | noop-jobs template will be enough based on the docs | 13:43 |
| edleafe | ok, off to squash! | 13:43 |
| giblet | edleafe: https://docs.openstack.org/infra/manual/creators.html#add-jobs-for-your-project | 13:44 |
| edleafe | thanks | 13:44 |
| * giblet have to leave soon but will wait for the squash to leave +2 on it :) | 13:44 | |
| *** mriedem has joined #openstack-placement | 13:45 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consumer gen support for put allocations https://review.openstack.org/591647 | 13:45 |
| *** ttsiouts has quit IRC | 13:47 | |
| *** alex_xu has quit IRC | 13:47 | |
| *** alex_xu has joined #openstack-placement | 13:53 | |
| edleafe | giblet: got a merge conflict on gerrit. Might take a little while to straighten out | 13:56 |
| giblet | edleafe: thanks for the notice. The I will leave now and try to check back later | 13:57 |
| edleafe | Thanks giblet | 13:57 |
| *** giblet is now known as giblet_off | 13:57 | |
| fried_rice | edleafe: Anyone propose the gerritbot patches to make placement repo changes show up in this channel? | 13:57 |
| edleafe | fried_rice: not that I know of | 13:58 |
| fried_rice | k, I can look into that, if I can remember how. | 13:58 |
| fried_rice | Cause that's, like, important. | 13:58 |
| giblet_off | fried_rice: cden did that | 13:58 |
| fried_rice | okay. | 13:58 |
| edleafe | giblet_off: fried_rice: cdent: Pushed the stack with cdent's patch squashed | 13:58 |
| giblet_off | fried_rice: https://review.openstack.org/#/c/598603/ | 13:59 |
| edleafe | oops - hold off | 13:59 |
| edleafe | the merge didn't do what I needed | 13:59 |
| edleafe | Thought that was too easy :) | 13:59 |
| *** ttsiouts has joined #openstack-placement | 14:08 | |
| fried_rice | edleafe: Let me know if I did something wrong here, but... | 14:12 |
| fried_rice | I cloned the placement repo from the link that shows up in gerrit, then diffed it against nova | 14:12 |
| fried_rice | and it looks like we're missing some of the more recent commits. | 14:12 |
| fried_rice | like from the reshaper series. | 14:12 |
| fried_rice | e.g. https://review.openstack.org/#/c/596497/ | 14:13 |
| * fried_rice scrambles to see whether that merged before or after we said we'd pull the trigger... | 14:14 | |
| *** mriedem has quit IRC | 14:14 | |
| fried_rice | damn. | 14:14 |
| *** mriedem has joined #openstack-placement | 14:15 | |
| fried_rice | by ten minutes. | 14:15 |
| edleafe | fried_rice: I pulled a fresh master for nova before running the extraction | 14:15 |
| fried_rice | yeah, that one merged ten minutes later. | 14:15 |
| edleafe | It shouldn't be too hard to apply the diff to the extracted code, no? | 14:15 |
| fried_rice | edleafe: We missed it when looking for patches that needed to be included. | 14:15 |
| fried_rice | No, it should be easy enough, but then it needs to merge before all the pathing changes. | 14:16 |
| fried_rice | Also | 14:16 |
| fried_rice | there were patches further up in the reshaper series that were "all nova side" but which touched e.g. placement.rst | 14:16 |
| edleafe | No, the pathing changes are trivial to handle | 14:16 |
| fried_rice | edleafe: But procedurally, I want the two patches to look identical. | 14:17 |
| edleafe | fried_rice: I don't see how that's possible. And I don't see why the pathing in some import statements is critical to be identical | 14:18 |
| fried_rice | Well, the diff itself would be identical, since the import lines are far enough away from the actual changes. But the deltas would be to different files depending whether it landed before or after the repathing. | 14:23 |
| fried_rice | whoops, I guess there's some git config patches I need to be on top of first, so my review winds up being proposed to the correct repo? | 14:25 |
| *** cdent has joined #openstack-placement | 14:26 | |
| cdent | edleafe: you've done something weird with https://review.openstack.org/#/c/598940/ | 14:26 |
| cdent | it doesn't not need to exist and is the thing blocking the children | 14:26 |
| cdent | sigh: s/not// | 14:26 |
| edleafe | cdent: | 14:27 |
| edleafe | cdent: yeah, I know | 14:27 |
| cdent | k | 14:27 |
| edleafe | I think it was an artifact of the juggling done in my GitHub repo | 14:27 |
| cdent | i left a comment anyway | 14:27 |
| edleafe | sure | 14:27 |
| fried_rice | cdent: In other news, we missed https://review.openstack.org/#/c/596497/ | 14:27 |
| fried_rice | I'm trying to propose it to the placement repo, but I'm probably missing some kind of git-review bootstrapping... | 14:27 |
| edleafe | I'm working in your patch into my series | 14:27 |
| fried_rice | edleafe: was this ^ to me? | 14:28 |
| edleafe | nope, for cdent | 14:28 |
| cdent | fried_rice: you can't propose until after https://review.openstack.org/#/c/598602/ merges | 14:28 |
| edleafe | shit | 14:28 |
| edleafe | pushed a new series based on cdent's stuff, and https://review.openstack.org/#/c/598940/ still is in there | 14:29 |
| cdent | and there's no point proposing anything until after we get the tests working, so best that we just remember that that missed thing needs to be cherry picked later | 14:29 |
| cdent | edleafe: you want I to try to fix it? | 14:30 |
| edleafe | cdent: what hash do you have for master? | 14:30 |
| fried_rice | I can't even git-review -d anything. | 14:30 |
| cdent | actually edleafe you can just abandon your change https://review.openstack.org/#/c/598602/ is parented to master | 14:30 |
| cdent | fried_rice: change .gitreview to have the right repo value | 14:30 |
| fried_rice | yeah, still freezing. | 14:30 |
| * cdent tries | 14:31 | |
| edleafe | cdent: on my machine that patch doesn't exist. I have your patch rebased onto 1a45f11, which shows up as master for me | 14:31 |
| fried_rice | I think I effed something up by getting partway without having done that. I recloned and am making progress now. | 14:31 |
| fried_rice | cdent: How come it's necessary to wait until things get working? I wanted to start off by having a clean diff between nova/ and placement/. And I need this commit (and part of one other) to make that happen. | 14:32 |
| edleafe | fried_rice: for one thing, it effs up the process we worked out | 14:32 |
| cdent | fried_rice: it's not _necessary_ to wait, I just figured we would want to wait until the tests were working | 14:32 |
| edleafe | fried_rice: the initial commit was going to be one thing, but at your and others request, it was broken into steps | 14:33 |
| cdent | edleafe: my master is 1a45f11a060d6 | 14:33 |
| cdent | edleafe: are you working in a repo that has remotes of both openstack/placement and edleafe/placement? | 14:33 |
| edleafe | cdent: so I abandoned the bad patch, but it still shows your change based on it | 14:34 |
| fried_rice | edleafe: Yeah, I would have had a much harder time finding this snafu if we had done the monolithic commit. | 14:34 |
| edleafe | cdent: no, remotes are updated. | 14:34 |
| edleafe | fried_rice: agreed. So we know the problem, and we'll correct it as soon as we can | 14:34 |
| cdent | the abandoned patch will continue to show my patch based on it forever. that doesn't matter | 14:34 |
| cdent | what matters is that https://review.openstack.org/#/c/598602/ is based on master | 14:34 |
| cdent | and it is | 14:35 |
| edleafe | cdent: yeah. Hey, the noop jobs passed for it!!! | 14:36 |
| cdent | edleafe, fried_rice, giblet_off, leakypipes : basically if want to continue working (at least using git-review) without too much faffing he base of https://review.openstack.org/#/c/598602/ needs to merge | 14:37 |
| fried_rice | Yeah, I can see that being the case. Why is edleafe -1 on it? | 14:37 |
| cdent | nova-core is now placement-core so they should have the power | 14:37 |
| cdent | because he hasn't caught up with himself yet | 14:37 |
| edleafe | fried_rice: and because I had a question about gate jobs. I was +1 before | 14:38 |
| edleafe | It's a very soft -1 | 14:38 |
| openstackgerrit | Merged openstack/nova master: api-ref: add a warning about calling swap volume directly https://review.openstack.org/596492 | 14:38 |
| fried_rice | cdent, edleafe: +2 | 14:38 |
| cdent | thanks fried_rice | 14:38 |
| edleafe | Now that the Cannot Merge is gone, I'm back to +1 | 14:39 |
| fried_rice | "8:44:52 AM * giblet have to leave soon but will wait for the squash to leave +2 on it :)" | 14:41 |
| fried_rice | What squash? Did that happen? | 14:41 |
| cdent | I have no idea what that's about | 14:42 |
| fried_rice | (trying to suss whether I can legitimately fast-approve that one) | 14:42 |
| cdent | thanks leakypipes | 14:42 |
| leakypipes | yep | 14:43 |
| edleafe | fried_rice: go back to 8:39 to the start of that conversation | 14:44 |
| edleafe | It was essentially a way of getting rid of the artifact patch that I just abandoned | 14:44 |
| edleafe | That was before we understood where that patch came from | 14:45 |
| fried_rice | cdent, edleafe: I proposed https://review.openstack.org/598988 | 14:46 |
| cdent | fried_rice: I'm not sure how we're going to get that to work with https://review.openstack.org/#/c/598941/ and the automated stack above it | 14:47 |
| cdent | are you saying you want your change in before all that stuff? | 14:47 |
| fried_rice | yes, let me restate the reasoning, might have been before you entered the room. | 14:47 |
| edleafe | It's gonna f up the rest of the extraction series | 14:47 |
| fried_rice | cdent: It's because I want to be able to `diff placement/ nova/` cleanly as a baseline to start reviewing gerrit patches. | 14:48 |
| * edleafe goes to make more coffee | 14:49 | |
| fried_rice | edleafe: Well, fwiw, I just rebased that whole series on top of my patch and it went fine. | 14:49 |
| cdent | yeah, I heard that fried_rice, but we messed up and missed that change. If we special case that one in, it messed up all the already existing patches we have for fixing stuff. Do we want to redo all those patches because of this one missed thing? Or can we just accept that we made a mistaked and deal? | 14:49 |
| cdent | if the rebase is okay, then maybe it is fine | 14:49 |
| fried_rice | That patch had no renames and no deltas near the imports. | 14:49 |
| cdent | I don't really mind as long as other cores are not going to be surprised and freak out | 14:50 |
| fried_rice | that's the thing. Cores who are using the same strategy (diff initial placement/ and nova/) are going to be freaked out by the difference, be like where tf did this come from | 14:51 |
| cdent | I have no insight to what strategies people are using, so I don't have an opinion | 14:51 |
| fried_rice | without proposing the rebased series, is there some way to figure out if it's okay? | 14:52 |
| cdent | what do you mean by okay? | 14:53 |
| cdent | the top of that stack is still not a working system, so there's no formal definition of okay, yet | 14:54 |
| *** rubasov has quit IRC | 14:55 | |
| fried_rice | right, swhy I'm asking. Is the fact that the rebase went through enough? | 14:56 |
| cdent | I guess you just want to make sure the changes in your patch are present in the file you expect them to be in in the top of the stack | 14:58 |
| cdent | if so, then disco | 14:58 |
| fried_rice | that's correct. | 14:59 |
| fried_rice | so | 14:59 |
| fried_rice | should I push the rebased series? | 14:59 |
| cdent | huzzah: https://review.openstack.org/#/c/598602/ merged! first commit. woot. | 15:00 |
| cdent | yes | 15:00 |
| cdent | but rebase again | 15:00 |
| fried_rice | why? | 15:00 |
| fried_rice | my patch was already on top of 602 | 15:00 |
| cdent | well, if you use git-review it will take care of it | 15:00 |
| fried_rice | I mean, I don't mind doing it again | 15:00 |
| cdent | no, is fine | 15:00 |
| fried_rice | I think I have auto-rebase switched off. | 15:00 |
| fried_rice | cdent: Done. | 15:01 |
| cdent | we need another infra for https://review.openstack.org/#/c/598603/ | 15:01 |
| fried_rice | btw, gibi said you proposed the patches to make placement repo changes show up in this channel. Wherezat? | 15:01 |
| cdent | jinx | 15:01 |
| fried_rice | thx, +1. You bugging infra about that one? | 15:02 |
| openstackgerrit | Merged openstack/nova master: Remove old check_attach version check in API https://review.openstack.org/588348 | 15:02 |
| edleafe | cdent: Do we still want nova and nova-specs showing up here? They already show up in -nova | 15:03 |
| fried_rice | I agree they could/should be removed later. | 15:03 |
| fried_rice | We definitely still want nova-specs, since that's where placement specs still live until the divorce is final. | 15:04 |
| cdent | edleafe: for now, yeah, I figured it was temporarily the right thing, as specs are still in nova-specs and to assuage some of the concerns about nova-side stuff being missed out | 15:04 |
| fried_rice | cdent, edleafe: There's one more change needed to make the repositories match. | 15:09 |
| edleafe | "...there's one more thing" | 15:10 |
| fried_rice | literally | 15:10 |
| fried_rice | https://review.openstack.org/#/c/585034/20/doc/source/user/placement.rst | 15:10 |
| fried_rice | efried@efried-ThinkPad-W520:~/Neo$ diff --exclude=.git --exclude=.zuul.yaml --exclude=.gitreview -r nova placement/ 2>&1 | grep -v '^Only in nova' | 15:11 |
| fried_rice | diff '--exclude=.git' '--exclude=.zuul.yaml' '--exclude=.gitreview' -r nova/doc/source/user/placement.rst placement/doc/source/user/placement.rst | 15:11 |
| fried_rice | 60d59 | 15:11 |
| fried_rice | < * `inventory and allocation data migration`_ (reshaping provider trees) | 15:11 |
| fried_rice | 66d64 | 15:11 |
| fried_rice | < .. _inventory and allocation data migration: http://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/reshape-provider-tree.html | 15:11 |
| cdent | fried_rice: that's going to be a bit more challenging | 15:12 |
| fried_rice | Because other stuff is changing nearby? | 15:12 |
| fried_rice | I'll propose it on top. This one should be relatively easy to explain away. | 15:12 |
| cdent | given that it is docs, I think you're better off fixing that by hand and not worrying about history | 15:12 |
| fried_rice | ack | 15:12 |
| cdent | if you're going to do it on top, don't do it yet | 15:13 |
| cdent | if you're going to put it near the bottom, then go ahead | 15:13 |
| cdent | but if you're able to cope with the diff, I'd say wait until we have running docs jobs. If you want a clean diff, put it in the stack you just made | 15:13 |
| fried_rice | I'll see if it rebases cleanly again. | 15:14 |
| *** e0ne has quit IRC | 15:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't persist zero allocation ratios in ResourceTracker https://review.openstack.org/598365 | 15:14 |
| fried_rice | cdent, edleafe: That rebase went clean as well. Okay to push? | 15:17 |
| edleafe | fried_rice: on top of your other patch? | 15:18 |
| cdent | okay with me, if it makes your life easier. | 15:18 |
| fried_rice | Yes, on top of the other one, and underneath the rest of the series. | 15:18 |
| edleafe | I only want to have to retool my patch series once | 15:18 |
| edleafe | fried_rice: ok, then cool | 15:18 |
| fried_rice | Do you have to retool anything if the rebase was clean? | 15:18 |
| fried_rice | done | 15:19 |
| edleafe | yes - I have to change what the patch series is based on | 15:19 |
| edleafe | You think I rebase 10 patches by hand? | 15:19 |
| fried_rice | what do you mean? | 15:19 |
| fried_rice | I already rebased the rest of the series. | 15:19 |
| edleafe | scripts, baby! | 15:19 |
| fried_rice | sorry, I don't follow. | 15:20 |
| fried_rice | anyway, I'm happy with the "baseline" now. | 15:20 |
| edleafe | I automate rebasing locally. | 15:20 |
| cdent | yeah, I'm not sure what edleafe is talking about | 15:20 |
| cdent | no new patches need to be pushed, fried_rice's stuff is applying cleaning in the middle | 15:20 |
| cdent | edleafe: is all your automation already pushed, or is there more? | 15:20 |
| cdent | we don't need to do it again | 15:21 |
| edleafe | cdent: it's pushed. If a change is needed, though, then I have to update the series | 15:21 |
| edleafe | that's all | 15:21 |
| cdent | if a change is needed wouldn't we pull the series down and update it by hand at this stage? | 15:22 |
| * cdent thinks he's missing some detail | 15:22 | |
| edleafe | I think you don't manage git branches the same way I do, so you don't understand my approach. Suffice it to say that it isn't a major hassle, just a minor one. | 15:24 |
| fried_rice | cdent, edleafe: I left an explanatory note on https://review.openstack.org/#/c/599001/ | 15:25 |
| cdent | i gotta go pick up my failed car, biab | 15:30 |
| *** ttsiouts has quit IRC | 15:38 | |
| *** ttsiouts has joined #openstack-placement | 15:39 | |
| *** ttsiouts has quit IRC | 15:43 | |
| *** e0ne has joined #openstack-placement | 15:45 | |
| *** e0ne has quit IRC | 15:56 | |
| *** openstackgerrit has quit IRC | 16:06 | |
| *** mriedem is now known as mriedem_away | 16:21 | |
| *** fried_rice is now known as fried_rolls | 16:27 | |
| *** mriedem_away is now known as mriedem | 17:50 | |
| *** openstackgerrit has joined #openstack-placement | 18:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675 | 18:15 |
| openstackgerrit | Adam Harwell proposed openstack/nova stable/pike: Add apply_cells to nova-manage https://review.openstack.org/599050 | 18:48 |
| openstackgerrit | Merged openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049 | 18:51 |
| openstackgerrit | Merged openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | 18:51 |
| openstackgerrit | Merged openstack/nova master: Do test_reshape with an actual startup https://review.openstack.org/597218 | 18:52 |
| openstackgerrit | Merged openstack/nova master: Fix reshaper report client functonal test nits https://review.openstack.org/598330 | 18:52 |
| *** openstackstatus has quit IRC | 18:58 | |
| *** fried_rolls is now known as fried_rice | 19:23 | |
| *** e0ne has joined #openstack-placement | 19:36 | |
| *** openstackstatus has joined #openstack-placement | 19:38 | |
| *** ChanServ sets mode: +v openstackstatus | 19:38 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Move conductor wait_until_ready() delay before manager init https://review.openstack.org/598353 | 19:50 |
| *** openstackstatus has quit IRC | 19:55 | |
| *** openstackstatus has joined #openstack-placement | 19:58 | |
| *** ChanServ sets mode: +v openstackstatus | 19:58 | |
| cdent | goodnight all | 20:01 |
| *** cdent has quit IRC | 20:02 | |
| *** e0ne has quit IRC | 20:06 | |
| *** openstackstatus has quit IRC | 20:11 | |
| *** openstackstatus has joined #openstack-placement | 20:12 | |
| *** ChanServ sets mode: +v openstackstatus | 20:12 | |
| *** nicolasbock has quit IRC | 20:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Fix image-defined numa claims during evacuate https://review.openstack.org/599062 | 20:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Move conductor wait_until_ready() delay before manager init https://review.openstack.org/599064 | 20:22 |
| *** openstackstatus has quit IRC | 20:36 | |
| *** openstackstatus has joined #openstack-placement | 20:38 | |
| *** ChanServ sets mode: +v openstackstatus | 20:38 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Other host allocs may appear in gafpt during evac https://review.openstack.org/597301 | 20:51 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Other host allocs may appear in gafpt during evac https://review.openstack.org/597301 | 20:55 |
| openstackgerrit | sean mooney proposed openstack/nova master: add caching to _build_regex_range https://review.openstack.org/599071 | 20:55 |
| *** e0ne has joined #openstack-placement | 20:56 | |
| *** e0ne has quit IRC | 21:00 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Null out instance.availability_zone on shelve offload https://review.openstack.org/599087 | 22:20 |
| *** mriedem has quit IRC | 22:32 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!