| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-02 | |||
| 20:21:18 | mriedem | dansmith: was coding | |
| 20:21:26 | mriedem | i see | |
| 20:21:26 | mriedem | oh top gun | |
| 20:21:28 | mriedem | good one mav | |
| 20:22:56 | cdent | edleafe, sdague: it is likely that before we added the consumer_id to the object, it operated as an unintentional delete | |
| 20:23:08 | dansmith | by the way, I'll be taking up a collection to buy one of these for mriedem: https://chummytees.com/products/i-speak-fluent-movie-quotes-t-shirt-hoodie-tank-top | |
| 20:24:05 | cfriesen | I was waiting for "she's real fine my 409" | |
| 20:24:08 | melwitt | he can wear it when he gives summit presentations | |
| 20:25:30 | mriedem | cfriesen: isn't that beach boys ala 60s? | |
| 20:25:34 | mriedem | too old | |
| 20:25:37 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Require at least one allocation when PUT https://review.openstack.org/490195 | |
| 20:26:09 | mriedem | ha yup, 1962 | |
| 20:26:19 | cdent | what’s your limit? | |
| 20:26:29 | mriedem | well i got the reference | |
| 20:26:33 | mriedem | so limit=01 | |
| 20:26:34 | mriedem | *-1 | |
| 20:26:57 | mriedem | the beach boys: playing at a state fair near you! | |
| 20:28:01 | cdent | is the doubling stack still in flux or is some of it ready for review? | |
| 20:28:13 | mriedem | i'm doing it | |
| 20:28:52 | cdent | is that yes or no? | |
| 20:29:00 | mriedem | no | |
| 20:29:13 | dansmith | not in flux or not reviewable? | |
| 20:29:20 | dansmith | I think it's worth a review in its current state, personally | |
| 20:29:30 | mriedem | i'm about to push, | |
| 20:29:36 | mriedem | but should i rebase https://review.openstack.org/#/c/488510/ on top? i suppose i should | |
| 20:31:16 | mriedem | here it comes | |
| 20:31:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | |
| 20:31:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | |
| 20:31:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | |
| 20:33:47 | sdague | mriedem: has anyone attempted to figure out the toc on the sidebar thing? | |
| 20:34:05 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021 | |
| 20:34:09 | sdague | because otherwise I'll look at that tomorrow, because the current toc is driving me nuts | |
| 20:34:33 | sdague | also, this is our top current 404 - https://review.openstack.org/#/c/490181/ | |
| 20:34:47 | sdague | 56 fails on that in the last 2 days | |
| 20:35:00 | mriedem | sdague: nope | |
| 20:36:28 | mriedem | sdague: you should rebase on top of https://review.openstack.org/#/c/489641/ | |
| 20:36:43 | sdague | mriedem: sure | |
| 20:37:00 | openstackgerrit | Sean Dague proposed openstack/nova master: add top 404 redirect https://review.openstack.org/490181 | |
| 20:39:05 | mriedem | you have to re-sort now too | |
| 20:39:20 | openstackgerrit | Sean Dague proposed openstack/nova master: sort redirectmatch lines https://review.openstack.org/490182 | |
| 20:39:29 | sdague | yep, done | |
| 20:39:42 | mriedem | hot | |
| 20:39:43 | mriedem | dog | |
| 20:40:55 | mriedem | all approved | |
| 20:41:26 | sdague | \o/ | |
| 20:46:10 | openstackgerrit | Sean Dague proposed openstack/nova master: fix rpc broken rst comment https://review.openstack.org/490197 | |
| 20:46:12 | sdague | mriedem: one more for you | |
| 20:46:18 | sdague | smallest patch possible | |
| 20:46:53 | mriedem | show it shows up | |
| 20:46:55 | mriedem | -10 | |
| 20:47:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: fix rpc broken rst comment https://review.openstack.org/490197 | |
| 20:55:58 | mriedem | dansmith: there is a false sense of passing in the resize to same host tests https://review.openstack.org/#/c/490085/5/nova/tests/functional/test_servers.py@1300 | |
| 20:56:51 | dansmith | mriedem: I made that change in response to a failure | |
| 20:58:08 | dansmith | that was actually one of the first changes I made, in response to a fail | |
| 20:58:15 | dansmith | lemme pull it out and see | |
| 20:59:51 | dansmith | but if it's dead code, then it's not affecting the passingness right? | |
| 21:00:22 | dansmith | yeah, removing it locally doesn't affect anything | |
| 21:00:48 | mriedem | i'll fix | |
| 21:00:50 | dansmith | maybe I misread the trace and modified it in the wrong place initially | |
| 21:00:52 | dansmith | sorry | |
| 21:00:54 | mriedem | i need to tweak the bottom test change | |
| 21:01:10 | dansmith | can I blame the break-up of everything into non-unified test cases? | |
| 21:01:18 | mriedem | sure | |
| 21:01:50 | mriedem | blame it on the rain | |
| 21:13:10 | jaypipes | ugh... rebase hell after pulling in mriedem's changes. | |
| 21:13:27 | jaypipes | thought this was code that wasn't overlapping :( | |
| 21:13:34 | mriedem | jaypipes: i'm fixing that stack now too | |
| 21:13:39 | mriedem | should be up in a couple minutes | |
| 21:14:45 | jaypipes | mriedem: I was working on the resize-same-host problem | |
| 21:15:49 | mriedem | in the compute? | |
| 21:15:51 | mriedem | or scheduler? | |
| 21:16:16 | jaypipes | mriedem: in the report client. fixing up and adding tests for the section dansmith had marked "this is cheating" | |
| 21:16:27 | mriedem | ok i haven't looked at that yet | |
| 21:16:40 | jaypipes | mriedem: fixing up the disk_gb calculation stuff. | |
| 21:16:50 | cdent | I gotta crash, need to not repeat last night’s too lateness. If there’s unclear state to pick up in the morning, please send a status mail or something like that. | |
| 21:17:09 | jaypipes | k | |
| 21:17:23 | cdent | Speaking of state: I’m not gonig to be around on friday, so if somebody else wants to do an rp/placement update, have at | |
| 21:17:37 | cdent | good night and good luk | |
| 21:17:41 | cdent | and luck even | |
| 21:17:43 | jaypipes | ciao | |
| 21:19:32 | jaypipes | mriedem: I'll wait on you, then. cuz I pulled the latest update you pushed for the "remove provider allocs in confirm/revert resize" patch above and it's all fooey. | |
| 21:19:50 | mriedem | i had just rebased | |
| 21:19:56 | mriedem | and cherry picked that from gerrit | |
| 21:20:06 | dansmith | maybe we should merge some of the turducken | |
| 21:20:11 | dansmith | fewer plates in the air | |
| 21:20:15 | mriedem | yeah, about there | |
| 21:20:26 | mriedem | chris brought up something in my change i need to address | |
| 21:25:54 | jaypipes | dansmith: if you're looking for something to get your mind off this resize stuff, your opinion on https://review.openstack.org/#/c/474914/7/os_vif/objects/vif.py would be useful. | |
| 21:34:08 | mriedem | ok coming up | |
| 21:35:01 | mriedem | i assume someone is going to fix the pep8 failures in the last change | |
| 21:35:10 | dansmith | jaypipes said he was | |
| 21:35:21 | jaypipes | yes | |
| 21:35:43 | dansmith | did we get jenkins runs? | |
| 21:36:21 | dansmith | btw, | |
| 21:36:49 | dansmith | jaypipes: reading your (now) top patch earlier, I was kinda thinking the ocata compat thing could/should be split from the fixing of resize_confirm | |
| 21:36:56 | dansmith | are those tied together in some obscure way I'm not seeing? | |
| 21:37:29 | jaypipes | dansmith: yeah, agree with you | |
| 21:37:38 | jaypipes | dansmith: this all kinda snowballed on me. :( | |
| 21:38:05 | dansmith | cdent rightly noted that the patch is large and gangly at this point | |
| 21:38:18 | jaypipes | yup | |