| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-16 | |||
| 16:16:45 | cdent | superdan: I was trying to (quickly) come up with a regexp on your statement but I got lazy and instead printed the output. (meaning, yes, I agree) | |
| 16:17:44 | superdan | heh | |
| 16:18:37 | figleaf | superdan: it was losing the microversion race that made the member_of patch fail. For once, can't blame fried_rice | |
| 16:18:48 | superdan | figleaf: ah | |
| 16:19:41 | superdan | 1.21 is a better microversion anyway | |
| 16:19:49 | fried_rice | It can drink | |
| 16:20:03 | fried_rice | (wheat, unfiltered IIRC) | |
| 16:32:28 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates https://review.openstack.org/552098 | |
| 16:32:36 | figleaf | leakypipes: superdan: fried_rice: ^^ func tests fixed | |
| 16:33:04 | figleaf | Had to resort to the gabbi HISTORY stuff, since I couldn't assume generation=0 | |
| 16:33:22 | figleaf | Besides, we're not supposed to assume that, right? :) | |
| 16:48:33 | fried_rice | figleaf: Ideally you should get it from the RP creation. I did that in a couple of places in my other patch. It's a PITA in gabbi. (Not such a problem in real code.) | |
| 16:48:51 | fried_rice | FWIW, I wouldn't -1 hardcoded generation numbers in a gabbi test. | |
| 16:49:03 | fried_rice | and would even write them myself :) | |
| 16:51:11 | cdent | fried_rice: it's not a pita in gabbi, just do what figleaf has done | |
| 16:52:12 | superdan | lyaaaaaaaaaaaaar: jgwentworth: looks like those reverts are failing some tests.. on two of the three patches at least | |
| 16:52:13 | fried_rice | cdent: Sorry, I just meant you have to use a pretty long string to get there. | |
| 16:52:21 | cdent | ow, my fingers | |
| 16:52:46 | cdent | is more explicit than e.g.. 'upt' | |
| 16:53:18 | superdan | cdent: was that an idiocracy reference? | |
| 16:53:23 | superdan | if so, kudos | |
| 16:53:24 | lyaaaaaaaaaaaaar | superdan: yup looking | |
| 16:53:44 | cdent | superdan: sadly no, entirely unintentional | |
| 16:54:23 | superdan | cdent: ah then no kudos for you | |
| 16:54:32 | cdent | /o\ | |
| 16:54:39 | superdan | cdent: https://www.youtube.com/watch?v=r_4jrMwvZ2A | |
| 16:56:25 | superdan | kashyap: speaking of all that, any ETA on finishing that cpu features patch? | |
| 16:57:04 | superdan | oh I guess you just said you'd work on it next week in -infra | |
| 16:57:13 | kashyap | superdan: I have been slow; you put me on the hook sometime mid next week. I am trying to figure out the unit test breakages :-( | |
| 16:57:27 | superdan | I put you on the hook? | |
| 16:57:28 | kashyap | superdan: You can keep me on the hook for next week, and shame me here if I don't finish it | |
| 16:57:31 | kashyap | Haha | |
| 16:57:36 | kashyap | We were writing the same thing :P | |
| 16:57:45 | superdan | okay | |
| 17:08:27 | openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | |
| 17:08:27 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 17:08:28 | openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | |
| 17:08:28 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | |
| 17:09:12 | mnaser | kashyap: yeah so just to follow up, it looks like older libvirt versions have to do a whole lot of stuff on start up to get cpu capabilities | |
| 17:11:26 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: ironic: stop lying to the RT when ironic is down https://review.openstack.org/545479 | |
| 17:11:43 | jroll | this is what we were thinking, right? ^ | |
| 17:19:18 | superdan | fried_rolls: can you ack this again before I send it? https://review.openstack.org/#/c/552098/9 | |
| 17:22:00 | kashyap | mnaser: Yeah; noted. | |
| 17:25:10 | cdent | finucannot: do you mind if I unparent the db_driver conf removal change https://review.openstack.org/#/c/508487/5 it's parent looks stuck, but itself is indepdent | |
| 17:25:21 | finucannot | cdent: go for it | |
| 17:25:23 | openstackgerrit | melanie witt proposed openstack/nova master: only increment disk address unit for scsi devices https://review.openstack.org/538310 | |
| 17:25:40 | openstackgerrit | Chris Dent proposed openstack/nova master: conf: Remove 'db_driver' config opt https://review.openstack.org/508487 | |
| 17:25:45 | cdent | finucannot: thanks | |
| 17:26:24 | cdent | johnthetubaguy: you happy to put a +w on https://review.openstack.org/#/c/508487/ now that it has no parent? you +2'd it a while back. | |
| 17:33:36 | cdent | superdan: I'm not fried_rolls but looking through his comments, and checking the changes since last I looked, assuming we are good with the "followon" nits that eric mentioned on ps8 being followons, looks good to me | |
| 17:37:16 | superdan | cdent: yeah I just figured I'd give him another go at it, but if he doesn't show up soonish I can just send it | |
| 17:41:48 | superdan | mriedem: looks like it failed on glance again | |
| 17:41:51 | mnaser | is legacy-tempest-dsvm-cells failing more than usual? | |
| 17:42:02 | mriedem | mnaser: everything is failng more than usual | |
| 17:42:07 | mriedem | until the cinder revert merges | |
| 17:42:07 | mnaser | oh nice | |
| 17:42:09 | mnaser | its friday | |
| 17:42:17 | mnaser | okay, so ill hold rechecking stuff endlessly then | |
| 17:42:21 | mriedem | mnaser: https://review.openstack.org/#/c/553794/ | |
| 17:42:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/553067 | |
| 17:42:59 | mriedem | superdan: well that's just not cool | |
| 17:43:18 | mriedem | superdan: oh that was the one i saw fail, pre-lunch | |
| 17:43:20 | mriedem | need to recheck | |
| 17:44:12 | superdan | oh I thought you already did | |
| 17:44:22 | superdan | heh I guess not, okay | |
| 17:44:37 | superdan | I was comparing your log link to the latest and like "yep, looks exactly the same" | |
| 17:44:42 | mriedem | the ovs multinode job, g-api started up right away on the subnode http://logs.openstack.org/08/553608/3/check/neutron-tempest-multinode-full/94a71ec/logs/subnode-2/devstacklog.txt.gz#_2018-03-16_14_40_43_011 | |
| 17:44:45 | mriedem | so not sure what's the difference | |
| 17:45:01 | superdan | unless LB networking means the nodes can't talk to each other | |
| 17:45:09 | mriedem | it might | |
| 17:45:27 | mriedem | https://review.openstack.org/#/c/553608/3/playbooks/legacy/neutron-tempest-linuxbridge/run.yaml@a33 | |
| 17:45:30 | mriedem | i can throw that back in there | |
| 17:45:35 | mriedem | but not sure if it will help | |
| 17:45:42 | superdan | I really thought we had ruled this out as an easy thing to do | |
| 17:45:53 | mriedem | well, the patch itself is pretty easy :) | |
| 17:45:54 | superdan | maybe I was just psychic | |
| 17:46:22 | mriedem | i.e. it's easy to copy the ovs multinode job, tweak for lb, and run it through ci until it works | |
| 17:46:27 | mriedem | not like i'm spending a lot of time on it | |
| 17:47:03 | mriedem | add PHYSICAL_NETWORK=default back in | |
| 17:47:05 | mriedem | *adding | |
| 17:55:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add a note in DELETE /os-services about deleting computes https://review.openstack.org/553598 | |
| 18:00:05 | jgwentworth | superdan, mriedem, leakypipes, fried_rolls, anyone else interested: here's a really rough draft on runways, please add comments and questions and if we can polish it a bit, I can send it out to the dev ML to start using the process https://etherpad.openstack.org/p/nova-runways-rocky | |
| 18:00:39 | jgwentworth | noting that this is going to be really not perfect and we just want to start with something. I should probably write that on the etherpad too | |
| 18:58:00 | fried_rolls | superdan: Gah, I thought I did that before I went to roll, sorry about that. Done. | |
| 18:58:21 | superdan | fried_rolls: thanks | |
| 19:02:18 | fried_rice | superdan, cdent: and by the way, here's my contribution: https://youtu.be/RYwgtJ7kJTI?t=130 | |
| 19:03:22 | superdan | fried_rice: now that's just silly | |
| 19:03:55 | fried_rice | Well, compared to the high cinema you presented, of course. | |
| 19:05:11 | fried_rice | superdan: I think the volume fails will continue to fail until https://review.openstack.org/#/c/553794/ merges? | |
| 19:05:43 | superdan | fried_rice: intermittently I think | |
| 19:06:03 | fried_rice | mm. Well, I was gonna wait to recheck mine, which is underneath his, so nyah. | |
| 19:06:25 | superdan | well, whatever, I was trying to help | |
| 19:06:32 | fried_rice | :) | |
| 19:06:39 | superdan | gonna go cry in my corner now | |
| 19:07:21 | fried_rice | What does that make the score? efried 1, dansmith 10000? | |
| 19:07:34 | fried_rice | I'm making my comeback | |
| 19:07:40 | mriedem | gah we don't have a Dict<?> field? | |
| 19:07:59 | superdan | mriedem: DictOfStrings | |
| 19:08:10 | mriedem | yeah but i've got a big ol bag of goodies | |
| 19:08:16 | mriedem | i know what to do, | |