Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-06
21:20:25 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
21:20:35 mriedem plus, if mnaser's patch creates a instance -> cell mapping dict to keep track, the 2nd loop could use that too
21:20:43 dansmith yeah
21:20:52 openstackgerrit Dan Smith proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452
21:21:02 dansmith gonna work on tests, but pushed this up in case I have to run ^
21:25:16 dansmith hmm, we kinda have a test for this, I'm not sure why it's not failing
21:29:32 mnaser dansmith is it okay that when i try to do git review with your patch below mine, it mentions that it will submit two commits?
21:29:44 dansmith mnaser: yep
21:29:49 mnaser okay cool
21:29:53 openstackgerrit Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408
21:30:05 mnaser oh it didnt send both, nice. that's leveraging your cell_instance_cache
21:30:15 mnaser oh i should update the commit msg
21:30:22 melwitt mnaser: I usually double check to make sure the commit hash of the dependent change is the same as what shows on the review being rebased upon
21:30:41 melwitt that's how you can tell whether it will push more than just your change
21:30:51 mnaser melwitt oh, if it matches it wont submit it? i'm just terrified of the embarassement that happens sometimes when an irc channel gets spammed :p
21:31:10 mriedem mnaser: git review -R also avoids the rebase of the base changes
21:31:19 melwitt mnaser: yeah. if the hash hasn't changed it won't submit it. so you can know before you do it
21:31:28 mriedem melwitt: btw i think i figured out the git review / rebase author change thing,
21:31:36 mriedem it happens when rebasing on a series that involves a merge conflict,
21:31:46 mriedem so you rebase, merge conflict, fix, git add / commit, git rebase continue
21:31:50 mriedem and that has changed the author
21:32:05 openstackgerrit Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408
21:32:19 melwitt weird. I feel like I've rebased other ppl's changes with merge conflict and it didn't change the author before
21:32:32 mnaser there, that should have most of it covered
21:32:34 melwitt but, that's great data to go on. I'll pay attention next time
21:33:31 dansmith okay, the test I thought we had was build not schedule_and_build
21:33:42 dansmith I have to board soon, but I can work on tests for this on the plane unless someone else wants to?
21:34:28 mnaser i'd help but it sits outside my relm of knowledge :) have a safe flight though dansmith
21:35:14 dansmith mnaser: np, thanks
21:35:38 mnaser time to go figure out why horizon pike is logging users out now :> thanks for the help everyone, i'll keep an eye out on reviews for the patch
21:36:01 mriedem mnaser: one issue in your patch
21:36:09 mnaser sure whats up
21:36:15 mnaser oh /me looks
21:36:16 mriedem comment inline
21:36:57 melwitt dansmith: I can write the tests for your change if you want to not do it on the plane
21:38:55 mnaser mriedem took me a while to figure out what you meant till i realized that super hidden for loop in the top lol
21:39:31 mriedem :) everything else looks good
21:42:07 mriedem sdague: low hanging fruit review https://review.openstack.org/#/c/499001/
21:42:16 mriedem skip more cells v1 random failure tests
21:42:27 openstackgerrit Mohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408
21:44:07 openstackgerrit Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137
21:44:08 openstackgerrit Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057
21:45:32 openstackgerrit Merged openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
21:48:42 openstackgerrit Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137
21:52:39 openstackgerrit Chris Dent proposed openstack/nova-specs master: Add a spec for POST /allocations in placement https://review.openstack.org/499259
21:55:52 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Add release note for force live migration allocations https://review.openstack.org/501477
21:58:13 openstackgerrit Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057
22:05:07 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247
22:15:09 mriedem melwitt: i'm going to step away to make dinner but if you get that test written ping me and i'll take a look tonight
22:16:27 melwitt mriedem: okay. I wasn't sure if dansmith got my message before boarding. so I wasn't sure if he's already working on it on the plane
22:46:53 mnaser do live migration jobs often fail
22:46:55 mnaser or did i break something :>
22:47:06 mnaser Details: (LiveAutoBlockMigrationV225Test:test_live_block_migration) Server 44884988-d3f2-4e32-a367-4962acda6346 failed to reach ACTIVE status and task state "None" within the required time (196 s). Current status: BUILD. Current task state: scheduling.
23:21:20 openstackgerrit Merged openstack/nova master: Amend uuid4 hacking rule https://review.openstack.org/488315
23:21:54 openstackgerrit Merged openstack/nova stable/pike: Fix broken link https://review.openstack.org/501403
23:40:07 mriedem mnaser: ah ./nova/conductor/manager.py:1017:20: F821 undefined name 'instance_mapping_cache'
23:40:19 mnaser oopsie
23:40:50 mriedem that's a bug in dan's change
23:40:52 mnaser oh thats not my path
23:40:53 mnaser yeah
23:41:02 mriedem i'll fix quick
23:44:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408
23:44:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452
23:58:42 openstackgerrit Merged openstack/nova master: Add release note for force live migration allocations https://review.openstack.org/501314
#openstack-nova - 2017-09-07
00:00:40 openstackgerrit Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351
00:00:41 openstackgerrit Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516
00:00:42 openstackgerrit Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537
00:00:42 openstackgerrit Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538
00:00:44 openstackgerrit Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423
00:00:45 openstackgerrit Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542
00:00:46 openstackgerrit Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664
00:00:51 openstackgerrit Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
00:00:53 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
00:00:56 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
00:00:59 openstackgerrit Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486
00:01:00 openstackgerrit Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541
00:01:01 openstackgerrit Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354
00:01:07 openstackgerrit Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
00:01:09 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
00:01:10 openstackgerrit Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398
00:01:16 openstackgerrit Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438
00:12:30 mriedem melwitt: i'm pretty close to having a test now
00:18:15 mriedem ok got it
00:20:58 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_certificates to REST API https://review.openstack.org/486204
00:22:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently https://review.openstack.org/501452
00:22:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501408
00:22:28 mriedem melwitt: here we go, ready for review ^
00:22:49 mriedem alex_xu: ^ these fix a couple of serious bugs for cells v2 in pike that we'll want to get backported soon
00:26:12 mikal sdague: that upload should address your concerns with the next patch in the series of doom
00:37:21 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations https://review.openstack.org/501051
00:37:21 openstackgerrit Chris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
01:19:35 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove vnedordata_driver https://review.openstack.org/501507
01:19:35 openstackgerrit jichenjc proposed openstack/nova master: WIP:Remove quota_networks and enable_network_quota https://review.openstack.org/501068
01:24:56 dansmith melwitt: I didn't see it before I left, did you do it?
01:25:43 dansmith mriedem: heh, that is almost exactly what I did on the plane
01:25:53 dansmith but didn't get a full test run to see the failures before I landed
01:26:19 mriedem dansmith: i've got ConductorTaskTestCase working locally with multiple cells, but that made some other tests fail, so i'm splitting that out in the series
01:26:24 mriedem to ungorpify it
01:26:30 dansmith cool
01:28:46 melwitt dansmith: no, I assumed you were doing it bc I assumed you didn't see my message

Earlier   Later