Earlier  
Posted Nick Remark
#openstack-nova - 2017-09-27
20:02:00 sdague as policy.json is blank
20:02:00 dansmith oh, right,
20:02:29 dansmith was thinking I had to return the actual thing
20:05:01 sdague though, I think the impact is going to be minimal, given that the issue was 1000 of these calls, which added 3s in your env, so were looking at 3ms per policy check
20:05:24 dansmith yeah
20:05:25 sdague so, unless we have another nested thing, it's going to be hard to see the impact
20:05:36 dansmith I see no impact as the first numbers are coming out
20:09:01 mriedem melwitt: couple comments on the test, but looks great otherwise
20:13:02 melwitt thanks, looking
20:13:48 dansmith mriedem: sdague: not likely any difference: https://imgur.com/a/gSIlq
20:18:47 mriedem sdague: i just git log -i --grep'ed for the first time i think
20:18:55 mriedem after you've said it in channel once per week i think
20:19:14 mriedem it might be the first thing you say when you wake up
20:19:15 mriedem not sure
20:22:22 efried oo. TIL.
20:26:05 sdague dansmith: yeh, it's clear the current call is down to 2 policy checks, which is good
20:27:51 openstackgerrit Merged openstack/nova master: _rollback_live_migration in live-migration seqdiag https://review.openstack.org/507871
20:30:32 openstackgerrit melanie witt proposed openstack/nova master: Make setenv consistent for functional and api_sample_tests https://review.openstack.org/507976
20:32:05 mriedem cdent: i know what's going on in my devstack patch
20:32:09 mriedem oh do i ever
20:32:18 cdent oh?
20:32:18 mriedem i've been mtreinish'd i think
20:32:58 dansmith mriedem: unfortunately a unit test failure snuck past me in that fix patch
20:33:14 mriedem where is my gd trombone
20:33:25 openstackgerrit Dan Smith proposed openstack/nova master: Fix policy check performance in 2.47+ https://review.openstack.org/507948
20:34:41 mriedem cdent: this case statement used to have a wildcard https://github.com/openstack-dev/devstack/blob/master/stackrc#L682
20:34:42 openstackgerrit melanie witt proposed openstack/nova master: Set group_members when converting to legacy request spec https://review.openstack.org/507938
20:36:46 cdent mriedem: whyreaka
20:36:49 mriedem GAH
20:36:50 mriedem f119121d21fa0446197b26378091677daac1606a
20:36:53 mriedem sdague'ed
20:37:27 sdague mriedem: ok, so what's the issue?
20:37:51 mriedem VIRT_DRIVER=fake means you don't get any default image downloaded
20:38:02 mriedem so i assume you don't want the wildcard back
20:38:07 johnsom FYI, after collecting a mountain of logs on this missing network interface issue I decided to force a PCI bus rescan inside the instance, boom, the interface appears as it should have. So, ubuntu/kernel/something issue and not nova/neutron
20:38:09 mriedem so i'll just add a case for fake and make it the same as libvirt?
20:39:09 johnsom stock, current, ubuntu 16.04 cloud image
20:39:09 cdent sounds right to me
20:39:21 cdent I’m dead, will check back in the morn
20:39:23 sdague mriedem: ok, because later everything assumes a real image?
20:40:07 mriedem sdague: well, tempest tries to get the image from glance to put in tempest.conf
20:40:11 mriedem the image id i mean
20:41:54 mriedem working a patch
20:43:34 sdague yeh, so, honestly we should probably figure out a better setup strategy for "throw away this image"
20:44:13 sdague mriedem: also, the qemu 2.10 thing wasn't a race, it's actually python2.7 and 3.5 evaluating the mock sentinel differently in the comparison
20:44:30 sdague for one of them it passes a >= check and the other it does not
20:45:53 openstackgerrit Sean Dague proposed openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505673
20:47:44 mriedem oh
20:49:43 mriedem melwitt: pep8
20:50:18 mriedem 507938
20:50:20 mriedem oops
20:50:30 mriedem ./nova/tests/functional/regressions/test_bug_1719730.py:17:1: F401 'cast_as_call' imported but unused
20:50:37 melwitt sigh, of course
20:50:48 melwitt thanks
20:50:50 mriedem i still have the trombone out if you want to borrow it
20:50:54 melwitt haha yeah
20:51:12 melwitt I thought, pep8 can't complain if I just deleted a line. yeah it could
20:52:27 openstackgerrit melanie witt proposed openstack/nova master: Set group_members when converting to legacy request spec https://review.openstack.org/507938
20:57:23 melwitt gdi locally I'm failing a notifications func test because I'm missing a notification from conductor. but the code that's supposed to send that notification is running, so I don't know how it could be missing
21:38:08 openstackgerrit Merged openstack/nova master: Have one list of reboot task_states https://review.openstack.org/219981
21:38:45 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
21:38:46 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
21:38:46 openstackgerrit Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486
21:38:47 openstackgerrit Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516
21:38:47 openstackgerrit Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
21:38:48 openstackgerrit Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538
21:38:48 openstackgerrit Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537
21:38:49 openstackgerrit Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
21:38:49 openstackgerrit Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
21:38:50 openstackgerrit Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354
21:38:50 openstackgerrit Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351
21:38:51 openstackgerrit Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398
21:42:24 mriedem sydney travel approved, hotel booked!
21:44:16 melwitt yay
21:44:41 mriedem time to go through the blistering visa process
21:45:33 melwitt it's easy to do on their site. there's no free option for US ppl, have to get the ETA visa and it costs $20 AUD
21:46:15 mriedem yeah, i was joking
21:46:22 melwitt oh
21:47:08 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix policy check performance in 2.47+ https://review.openstack.org/507965
21:48:48 mriedem ooo the hotel has an infinity pool
21:48:54 mriedem can we do the sessions from poolside?!
21:49:08 mriedem with kangerbangers
21:49:11 melwitt what? what kinda fancy hotel are you staying at
21:49:33 mriedem the sofitel next to the conference center, for the non-refundable pay immediately and never get your money back rate
21:49:51 melwitt nice
21:51:45 mtreinish mriedem: the sofitel has an inifinity pool? I must have missed that on the website
21:51:53 mtreinish not that I looked closely
21:52:51 openstackgerrit Merged openstack/nova master: Implement query param schema for agent index https://review.openstack.org/506950
21:53:31 mriedem well, it will, it's not open yet
21:54:46 melwitt hm, it looks like the prices are lower than when I first looked several weeks ago
21:54:50 cfriesen_ melwitt: yeah, I was surprised that Canadians don't get the free option. I figured what with commonwealth and all...
21:55:14 mriedem the only thing you share,
21:55:22 mriedem is your love of groveling for the royal family
21:55:29 mriedem there are no other benefits
21:55:36 cfriesen_ our love of beer, more like.
21:55:44 mriedem germans don't love beer?
21:55:51 cfriesen_ I didn't say it was exclusive
21:56:02 mriedem but germans can't love the royalty the same way
21:56:20 cfriesen_ half the british royalty was german anyway
21:56:46 mriedem oh breeding

Earlier   Later