| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-11 | |||
| 13:46:48 | zhubx | Can anyone else here take some time to help to review https://review.openstack.org/#/c/640271/ Thanks. | |
| 13:47:23 | mriedem | gibi: a few small things in https://review.openstack.org/#/c/642387/ | |
| 13:48:48 | gibi | mriedem: looking | |
| 13:54:48 | gibi | mriedem: gerrit died on me but as far as I see you are correct. I will respin the patch soon | |
| 13:55:23 | mriedem | ack | |
| 13:55:26 | mriedem | yeah dead for me too | |
| 13:55:43 | edleafe | I had just tried to post a long comment when it died on me. <sigh> | |
| 14:19:52 | openstackgerrit | sahid proposed openstack/os-vif master: ci: re-enable openstack-python35-jobs https://review.openstack.org/642465 | |
| 14:26:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug #1819460 in functional test https://review.openstack.org/642444 | |
| 14:26:41 | openstack | bug 1819460 in OpenStack Compute (nova) "instance stuck in BUILD state due to unhandled exceptions in conductor" [Low,New] https://launchpad.net/bugs/1819460 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 14:26:41 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Handle placement error during re-schedule https://review.openstack.org/642470 | |
| 14:30:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Handle placement error during re-schedule https://review.openstack.org/639608 | |
| 14:36:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use Selection object to fill request group mapping https://review.openstack.org/638711 | |
| 14:36:28 | mriedem | easy optimization for the bw series ^ | |
| 14:38:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix race in test_interface_detach_with_port_with_bandwidth_request https://review.openstack.org/642387 | |
| 14:39:05 | gibi | mriedem, efried: fixed the nits in ^^ | |
| 14:39:42 | efried | gibi: +2 thanks | |
| 14:39:45 | gibi | efried: thanks | |
| 14:40:08 | gibi | efried: there is a functinal test coverage only patch as well https://review.openstack.org/#/c/641309/ that needs a second core | |
| 14:40:37 | efried | gibi: Yeah, I saw that and tried to grok it last week but gave up :) | |
| 14:40:57 | gibi | efried: no worries I will try to hook bauzas into that | |
| 14:41:11 | efried | or somebody who knows wtf a macvtap is | |
| 14:41:36 | gibi | yeah :) | |
| 14:41:39 | efried | one of these days I'm going to have to learn *anything* about networking. | |
| 14:42:34 | gibi | efried: sean-k-mooney | |
| 14:42:37 | gibi | efried: sean-k-mooney can help | |
| 15:01:06 | mriedem | efried: i can't say i fully grok this ironicclient interface change https://review.openstack.org/#/c/640879/ - questions inline | |
| 15:05:35 | mriedem | is the issue that on stable (queens or pike), valid_interfaces as an option didn't exist, only interface, and they have interface=internal but we don't pass that along or something, so when we get the ksa adapter it returns the public endpoint which we use to construct the ironicclient? | |
| 15:07:46 | efried | mriedem: ack, looking. | |
| 15:34:58 | mriedem | efried: also comments on your 'stop using PlacementDirect' change https://review.openstack.org/#/c/640887/ | |
| 15:40:04 | efried | ack | |
| 15:53:52 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Documentation for bandwidth support https://review.openstack.org/642064 | |
| 16:20:26 | zbr | we need some input regarding changing default NovaHWMachineType value to make it cross distro compatible: https://review.openstack.org/#/c/642443/ | |
| 16:20:56 | weshay | dansmith bauzas can you folks recommend a libvirt vm machine types? https://review.openstack.org/#/c/642443/1/deployment/nova/nova-compute-container-puppet.yaml is pc > pc-i440fx-rhel7.6.0 | |
| 16:21:36 | dansmith | weshay: kashyap is probably your best bet for that question, but he and bauzas are at a church retreat this week | |
| 16:21:51 | weshay | ah k | |
| 16:22:06 | weshay | thanks dansmith | |
| 16:22:13 | dansmith | yup | |
| 16:23:18 | mriedem | zbr: i'd ask kashyap but he's not in channel right now | |
| 16:23:27 | mriedem | oh heh | |
| 16:23:29 | mriedem | jinx | |
| 16:23:49 | zbr | mriedem: please add those that can help to this review. | |
| 16:24:04 | mriedem | done | |
| 16:24:21 | zbr | i am curious why we used a pinned version instead of the "pc" one. | |
| 16:58:36 | openstackgerrit | Merged openstack/nova master: fix race in test_interface_detach_with_port_with_bandwidth_request https://review.openstack.org/642387 | |
| 17:39:01 | openstackgerrit | guang-yee proposed openstack/nova master: pass endpoint interface to Ironic client https://review.openstack.org/640879 | |
| 17:44:39 | openstackgerrit | Chris Dent proposed openstack/os-traits master: Update bug links for storyboard https://review.openstack.org/642529 | |
| 17:51:42 | openstackgerrit | Chris Dent proposed openstack/os-resource-classes master: Update bug links for storyboard https://review.openstack.org/642530 | |
| 19:16:33 | melwitt | o/ | |
| 19:34:13 | openstackgerrit | Adrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115 | |
| 19:34:28 | openstackgerrit | Merged openstack/os-traits master: Update bug links for storyboard https://review.openstack.org/642529 | |
| 19:36:47 | openstackgerrit | Eric Fried proposed openstack/nova master: Stop using PlacementDirect https://review.openstack.org/640887 | |
| 19:37:00 | efried | mriedem, cdent: ^ | |
| 19:38:10 | efried | Hi melwitt. In case you haven't seen this: https://review.openstack.org/#/c/641391/ | |
| 19:38:23 | efried | I don't think there's really contention anymore, but in case you want to weigh in. | |
| 19:39:20 | openstackgerrit | Adrian Chiris proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | |
| 19:39:30 | melwitt | efried: I saw and had thought it looked like folks had agreed to hold until train. thanks for confirming | |
| 19:39:39 | efried | rgr | |
| 19:42:56 | openstackgerrit | Merged openstack/os-resource-classes master: Update bug links for storyboard https://review.openstack.org/642530 | |
| 19:43:44 | mriedem | melwitt: https://blueprints.launchpad.net/nova/stein should be fully up to date now as well | |
| 19:44:32 | melwitt | mriedem: fantastic, thank you | |
| 20:23:36 | openstackgerrit | Eric Fried proposed openstack/nova master: DNM: inline fake_libvirt_utils.create_image mocks https://review.openstack.org/642445 | |
| 20:23:54 | openstackgerrit | Michael Still proposed openstack/nova master: Make privsep.chown mocking for libvirt snapshot tests less magic. https://review.openstack.org/642134 | |
| 20:23:55 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from the cache concurrency tests. https://review.openstack.org/642556 | |
| 20:23:55 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from connection tests. https://review.openstack.org/642557 | |
| 20:23:56 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from snapshot tests. https://review.openstack.org/642558 | |
| 20:25:07 | efried | mikal: o/ | |
| 20:27:45 | mikal | efried: Hi | |
| 20:28:27 | efried | mikal: In case you wanted to chat realtime about https://review.openstack.org/#/c/641489/ (fake_libvirt_utils) | |
| 20:29:18 | mikal | I'm happy to chat, but wont really be online for a couple of hours. I need to get the kids off to school first. I would be interested in what you think of the alternative approach used above though. | |
| 20:30:10 | efried | ah, looking. | |
| 20:32:08 | efried | mikal: oic, this way you gotta do it (at least) one whole module at a time. I was going to do it one mocked method at a time. I think the former makes for fewer, bigger patches. But I like it. | |
| 20:32:58 | efried | I thought there was a base class somewhere that was inherited by multiple test modules though. If that's the case, you'll either have to do all of those in one big swatch; or move the monkeypatch from the base into the sub-modules so you can piecemeal em. | |
| 20:43:11 | mikal | efried: there isn't much left after those four patches -- test_virt_drivers (small); and test_imagebackend (bigger, but not huge). There is some cleanup afterwards as the module is removed, which I was going to do method by method, but it keeps it reviewable, incremental, and games the statistics to make us all look good. | |
| 20:43:45 | mikal | efried: Oh, and don't discount the added benefit of long patch series using a lot more gate resources than a single big patch. We wouldn't want the gate to get bored. | |
| 20:51:14 | efried | mikal: my series was probably going to be longer. | |
| 20:51:23 | efried | mikal: I'm on board with your approach | |
| 20:51:30 | efried | especially since you seem to be signed up to do the work :P | |
| 20:51:46 | efried | (unless you consider that reviewing these is hard. I don't. zuul does most of the hard stuff there) | |
| 20:52:09 | efried | mikal: I've abandoned my efforts. There's no going back now. | |
| 20:52:19 | efried | And have I mentioned how trim your waistline is looking these days? | |
| 20:55:29 | mikal | LOL | |
| 20:55:45 | mikal | Its because of my regular predator escape simulations. | |
| 20:57:25 | efried | Simulations? You lived in the land of the box jellyfish and the funnel web spider. | |
| 20:57:34 | efried | One might say mocks are unnecessary | |
| 20:59:07 | mikal | Oh I dunno. When Chet explained to me the correct procedure for fending off a bear in the Rockies it seemed a bit like you people downplay exactly how doomed you are. | |
| 20:59:23 | cburgess | Wait I did what? | |
| 20:59:26 | cburgess | Oh.... | |
| 21:02:56 | mikal | cburgess: oh, I didn't realise you still lurk here. | |
| 21:05:08 | openstackgerrit | Eric Fried proposed openstack/nova master: Stop using PlacementDirect https://review.openstack.org/640887 | |
| 21:05:56 | mikal | So we're tools down on privsep now and will finish in the next release right? If so I'll go back to just rebasing them on Rebase Wednesdays. | |
| 21:11:30 | melwitt | mikal: correct. thanks :) | |
| 21:15:25 | mikal | I look forward to smashing the gate over and over. | |
| 21:15:30 | mikal | We all have our hobbies. | |
| 21:24:21 | dansmith | mikal: I can only imagine that cburgess' strategy was to argue with and tax the bear until he gives up and agrees to be as passive as the squrrels, in which case I would agree with you about our doomed-ness | |
| 21:24:58 | mikal | dansmith: well, there was bear repellant at REI, and I nearly bought some until I realised I only had to run faster than cburgess to survive. | |
| 21:25:32 | dansmith | yep, always adventure with a slower person | |
| 21:25:33 | mikal | Anyway, the scariest things in the US are the humans as best as I can tell. | |
| 21:25:59 | mikal | And if you can convince your walking partner to regularly rub themselves down with bbq sauce all the better. | |
| 21:26:05 | dansmith | lol | |