Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-08
18:25:28 melwitt I know, I'm saying I wrote a func test back then that does all of that
18:25:59 melwitt it was this one https://review.openstack.org/#/c/428505/19/nova/tests/functional/test_boot_from_volume.py
18:27:37 melwitt it does resize, shelve, unshelve, rebuild
18:30:59 mriedem does it julienne cut?
18:32:54 melwitt well, I thought it might help if you're finding gaps with local_gb not being handled correctly after server actions. those were the other ones I tested
18:33:37 mdbooth_bus The shelve-o-matic
18:33:43 mriedem yeah, i know, thanks. :) i'm just joking about julienne cuts.
18:33:51 melwitt at the time that I was trying to fix the bug with the patch that's now part of starlingX
18:37:34 mriedem yeah i can restore that test locally and see what fails still with context on this resize_claim bug
19:01:48 mdbooth_bus b
19:02:15 mdbooth_bus mriedem: Do you happen to remember where the local ephemeral root bdm is defined?
19:02:42 mdbooth_bus I'm trying to re-grok the bdm code in compute api, and I'm running out of mental cache
19:02:58 mdbooth_bus Also, it's really late and I'm on a bus
19:03:49 mdbooth_bus There's a comment which suggests that the client does it, but I couldn't find it there, either
19:04:45 mriedem yeah the api creates a thing, sec
19:09:31 mriedem nova.block_device.create_image_bdm creates it,
19:09:40 mriedem called from nova.block_device.from_legacy_mapping,
19:09:44 mdbooth_bus mriedem: Thanks
19:09:58 mriedem called from the api
19:09:59 mriedem at some point
19:10:06 mriedem totally not obvious
19:10:51 mdbooth_bus Wow, I'd already found and dismissed the from_legacy_mapping call
19:11:00 mdbooth_bus Yeah, that's not obvious, thanks
19:11:31 mriedem right b/c even if you use bdm_v2 you end up down that legacy path which is confusin
19:11:39 mriedem since legacy should mean bdm v1
19:12:21 mriedem legacy_image_defined = not image_properties.get('bdm_v2', False)
19:12:33 mriedem ^ is the source of the confusion
19:12:39 mriedem in _get_image_defined_bdms
19:12:44 openstackgerrit Merged openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116
19:13:05 mdbooth_bus 'legacy' there presumably meant you specified --image <imageid> rather than --blockdevice ...image-fu...
19:13:26 mriedem maybe
19:13:54 mdbooth_bus Anyway, that's saved me a bunch of bleary-eyed staring at code, thanks
19:14:21 mriedem mayhap i should throw some comments in that code
19:16:35 mdbooth_bus Oh, I think I'm about to arrive after that. Night...
19:23:09 melwitt I marked that bp as complete ^ after removing it from the runway
19:37:01 mriedem we can remove nova-cyborg-interaction from the runways queue right?
19:37:44 melwitt yeah, I wasn't sure if we should leave the note on it for whoever put it there, so they know why it's not going to be added to a runway
19:37:57 mriedem i'm assuming sundar added it
19:38:14 mriedem i would probably just remove it
19:39:08 melwitt ok
19:40:24 efried I suspect Sundar wanted us to review the spec.
19:40:45 efried we should have separate "spec runways".
19:40:52 efried for specs that would otherwise get ignored
20:09:39 sean-k-mooney efried: should or could. they could just go into the stuck review or open discustion section
20:09:44 sean-k-mooney of the nova meeting
20:09:53 efried I was being (mostly) facetious
20:09:55 sean-k-mooney that said i do need to go review it again
20:36:43 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
21:05:59 mriedem melwitt: ^
21:05:59 openstack bug 1796737 in OpenStack Compute (nova) rocky "resize: hypervisor local_gb_used still reports usage even with volume-backed instances after fix for bug 1469179" [Medium,Triaged] https://launchpad.net/bugs/1796737
21:05:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Add regression test for bug 1796737 https://review.openstack.org/608771
21:17:10 melwitt coolness. will take a look
21:37:57 openstackgerrit Matt Riedemann proposed openstack/nova master: Properly track local root disk usage during moves https://review.openstack.org/608777
21:47:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove ServersTestBase.setUp() https://review.openstack.org/608780
22:08:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Make ResourceTracker.tracked_instances a set https://review.openstack.org/608781
22:21:56 openstackgerrit Matt Riedemann proposed openstack/nova master: Make ResourceTracker.tracked_instances a set https://review.openstack.org/608781
23:01:08 alex_xu jaypipes: thanks, no futher answer, that is my expect answer, will recheck the spec again today.
23:01:20 alex_xu s/futher answer/futher question/
23:12:53 openstackgerrit sean mooney proposed openstack/nova master: [DNM] stub out sriov live migration https://review.openstack.org/608788
23:42:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
23:42:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106
23:42:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110
23:43:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
23:43:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
#openstack-nova - 2018-10-09
00:07:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
00:07:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581
00:07:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.openstack.org/576017
00:18:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.openstack.org/576018
00:18:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.openstack.org/576019
00:32:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.openstack.org/576020
00:38:36 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.openstack.org/576027
00:38:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031
00:41:35 openstackgerrit liuming proposed openstack/nova master: Deletes evacuated instance files when source host is ok https://review.openstack.org/605987
00:44:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Remove a description in servers-actions.inc https://review.openstack.org/608796
00:49:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299
00:49:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344
00:57:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673
01:12:55 openstackgerrit Merged openstack/nova master: Fix nits in choices documentation https://review.openstack.org/608310
01:14:44 naichuans bauzas: Thanks.
01:44:15 openstackgerrit lei zhang proposed openstack/nova master: Remove useless TODO section https://review.openstack.org/608802
02:05:05 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676
02:05:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689
02:21:32 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Replace MB with MiB https://review.openstack.org/608807
02:28:52 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Remove unnecessary redirect https://review.openstack.org/607400
02:34:53 openstackgerrit Brin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type`` https://review.openstack.org/579360
02:51:03 openstackgerrit Merged openstack/nova master: Remove redundant irrelevant-files from neutron-tempest-linuxbridge https://review.openstack.org/606989
03:23:55 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: doc: Start using openstackdoctheme's extlink extension https://review.openstack.org/608829
04:48:32 openstackgerrit Brin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type`` https://review.openstack.org/579360
04:51:38 alex_xu jaypipes: sorry, I get more question on it https://review.openstack.org/#/c/555081/20, it's my last question I think, hope won't make you annoying on no ending spec review
05:26:25 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
05:28:00 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
05:28:14 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041
05:28:27 openstackgerrit Naichuan Sun proposed openstack/nova master: os-xenapi(n-rp): add traits for vgpu n-rp https://review.openstack.org/604269
06:02:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Replace non UUID string with UUID https://review.openstack.org/608854
06:28:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345
06:29:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
06:30:56 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
07:29:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709
07:29:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712
07:36:52 bauzas good morning nova

Earlier   Later