Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-03-11
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
21:26:18 efried aka "mosquito repellant"
21:26:22 dansmith maybe they should sell aerosol bbq sauce next to the aerosol bear spray?
21:26:39 dansmith "choose your strategy"
21:26:42 dansmith "offense or defense"
21:29:20 dansmith https://www.youtube.com/watch?v=RZiYpDw_UM8
21:29:24 dansmith found it ^
21:31:36 cburgess mikal: Yes I still lurk here most days. You all are the only friends I have.
21:31:48 cburgess dansmith: Tax them? I mean argue with them I get, but tax them?
21:32:12 dansmith cburgess: yeah, I don't understand the logic either, but.. alas :D
21:32:19 cburgess LOL
21:32:49 dansmith but seriously, I'm thinking if you're the slow one, a long-range BBQ sauce spray bottle could be real handy
21:33:16 dansmith and, you didn't even technically perform an aggressive act, you just sauced up a buddy
21:33:27 dansmith so you get to keep your membership card and everything
21:37:40 mikal So that was the time dansmith defined covering a person with bbq sauce while being attacked by bears as an act of friendship.
21:37:44 efried You just know the early iterations of that spray were flammable
21:38:20 dansmith mikal: can you imagine a more loving way to feed someone to a bear than cover them in delicious BBQ sauce?
21:47:01 openstackgerrit Jeremy Stanley proposed openstack/nova master: [DNM] Exercising base-test for a master change https://review.openstack.org/642580
21:48:28 openstackgerrit Jeremy Stanley proposed openstack/nova master: [DNM] Exercising base-test for a master change https://review.openstack.org/642580
21:50:20 openstackgerrit Jeremy Stanley proposed openstack/nova stable/rocky: [DNM] Exercising base-test for a rocky change https://review.openstack.org/642582
22:43:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Add finish_snapshot_based_resize_at_dest compute method https://review.openstack.org/635080
22:43:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add FinishResizeAtDestTask https://review.openstack.org/635646
22:43:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Destination.allow_cross_cell_move field https://review.openstack.org/614035
22:43:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute CrossCellMigrationTask from MigrationTask https://review.openstack.org/635668
22:43:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Plumb allow_cross_cell_resize into compute API resize() https://review.openstack.org/635684
22:43:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Filter duplicates from compute API get_migrations_sorted() https://review.openstack.org/636224
22:43:22 openstackgerrit Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037
22:43:22 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.openstack.org/614353
22:43:22 openstackgerrit Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.openstack.org/636253
22:43:23 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add confirm_snapshot_based_resize_at_source https://review.openstack.org/637058
22:43:24 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add ConfirmResizeTask https://review.openstack.org/637070
22:43:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.openstack.org/637075
22:43:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.openstack.org/637316
22:43:25 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add revert_snapshot_based_resize_at_dest compute method https://review.openstack.org/637630
22:43:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.openstack.org/639453
22:43:26 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add finish_revert_snapshot_based_resize_at_source compute method https://review.openstack.org/637647
22:43:27 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add RevertResizeTask https://review.openstack.org/638046
22:43:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.openstack.org/638047

Earlier   Later