Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-05
21:13:03 ildikov mriedem: and I owe you a lot, I'm sure we can negotiate, lemme know
21:13:03 mriedem ildikov: i need to fix another functional test, and i'm working on adding the unit test coverage for the new flow changes in test_block_device
21:13:21 mriedem which makes me realize that in the new attach flow in block_device.py, if attachment_complete fails, we don't do any cleanup
21:13:24 mriedem unlike the old flow
21:14:35 ildikov yeah, we moved that around a bit
21:14:43 ildikov so I guess that's why it's uncovered
21:15:32 ildikov the volume is connected though, however the volume state would still be attaching if that call doesn't get to Cinder or fails before the db update
21:20:00 mriedem that's why we handle it in the legacy flow and rollback (terminate the connection and detach the volume so it's "available")
21:22:22 mriedem anyway, i'm just going to leave a note in the code
21:22:37 ildikov I know, was thinking out loud
21:43:13 mriedem we are going to want to break out these block_device.py changes with the unit tests - this gets pretty hairy in the tests
21:43:19 mriedem i'm down to 10 failures
21:45:05 ildikov we should find a superhero name for you
21:45:32 ildikov can do it after my next meeting and breaking out those changes
21:52:28 ildikov and sorry, there were two self.save() calls in the new attach call in block_device.py
21:52:57 ildikov I obviously deleted the wrong duplicate...
21:57:40 openstackgerrit Lance Bragstad proposed openstack/nova master: Add scope_types to server policies https://review.openstack.org/525772
21:59:07 mriedem ildikov: i'm actively working on the attach patch so don't make any changes there
21:59:21 lbragstad cc johnthetubaguy ^
21:59:44 ildikov mriedem: I didn't intend to until you explicitly say so
22:00:18 ildikov mriedem: otherwise things can get messy, practiced that with jgriffith not that long ago
22:29:02 faridda Hi guys, to Configure Hyper-V virtual switching, we have to specify an interface. Which one should be use (internal api, tenant network ...)?
22:55:29 melwitt mriedem: heads up, regression live-migrating with a volume that was backported to ocata. I'm gonna add stable/pike and stable/ocata to the bug https://bugs.launchpad.net/nova/+bug/1715569
22:55:30 openstack Launchpad bug 1715569 in OpenStack Compute (nova) "Live migration fails with an attached non-bootable Cinder volume (Pike)" [Low,In progress] - Assigned to Mike Lowe (jomlowe)
22:55:59 melwitt "nominate to series pike and ocata
22:56:44 mriedem faridda: ask claudiub or lpetrut when they are around
22:57:13 mriedem melwitt: you mean something was changed which introduced the regression and that change was backported to stable?
22:57:56 melwitt mriedem: yes, that
22:58:08 mriedem and https://review.openstack.org/#/c/459741 was the chnage
22:58:20 melwitt yes
22:58:28 mriedem super
22:58:33 melwitt yeah :(
22:59:13 melwitt the good news? is the fix looks correct to me and fixes the problem without undoing what the other change [that introduced the regression] fixed
23:02:43 melwitt so I think we can just roll forward and backport the fix that's up for review right now
23:20:42 mriedem gd finally got these unit tests passing
23:21:05 mriedem 2.5 hours
23:22:22 melwitt I know your pain
23:28:33 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
23:28:33 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Implement new attach Cinder flow https://review.openstack.org/330285
23:28:34 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047
23:29:27 ildikov mriedem: that would've taken me waaaaay more than that
23:29:45 ildikov I know it doesn't give you back the wasted time, but still
23:30:32 mriedem i'll see if i can split out block_device.py quick
23:34:16 melwitt mriedem: on this https://review.openstack.org/#/c/525622 how can we know we have cinder microversion 3.44 available? or does the check not hurt anything in the case that it's < 3.44?
23:34:37 ildikov sounds like a famous last sentence, but hearing from you makes it believable :)
23:37:06 openstackgerrit Takashi NATSUME proposed openstack/nova stable/pike: [placement] Fix foreign key constraint error https://review.openstack.org/525620
23:38:57 mriedem melwitt: at this point we won't check because bdm.attachment_id won't be set
23:39:21 mriedem bdm.attachment_id only gets set once (1) computes are all upgraded to support new flow and (2) cinder 3.44 is available
23:39:31 melwitt ah, cool. thanks
23:39:35 mriedem existing bdms won't have that attachment_id field set, so we'll just do the old stuff
23:45:07 mriedem here it comes
23:45:09 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
23:45:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
23:45:10 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add new style volume attachment support to block_device.py https://review.openstack.org/525787
23:45:10 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047
23:48:47 mriedem ildikov: jgriffith: i split this out https://review.openstack.org/#/c/525787/ - and left fixme/todo stuff in there from when i was adding the unit tests
23:48:55 mriedem too tired to really go over any of that now
23:50:44 mriedem that tempest test for attaching a volume to a shelved offloaded test is passing too, even after the revert
23:51:06 mriedem so at this point i think the questions in https://review.openstack.org/#/c/525787/ are going to be the sticking point, and hopefully those aren't too bad
23:55:08 mriedem melwitt: so in https://review.openstack.org/#/c/525622/3/nova/compute/api.py - you want a specific test for the case that we find a bdm and fail?
23:55:18 mriedem via objects.BlockDeviceMapping.get_by_volume_and_instance
23:56:36 melwitt mriedem: no, I'm thinking of just pushing a follow up that changes from stubbing out the entire "check if already attached" to mocking only the BDM get. so that the real code raises InvalidVolume instead of unit test code raising it
23:57:16 mriedem ok
23:57:16 melwitt since we're not unit testing "check if already attached" anywhere. I think it would be two birds with one stone by just making that small change
23:57:20 ildikov mriedem: thank you
23:57:40 ildikov I will go through the changes once I slept a bit
23:59:00 ildikov mriedem: is the fixme waiting for me now or there are things you still plan to change?
23:59:21 mriedem i don't plan on making changes at this point
23:59:32 mriedem i've pointed out the open questoins in the block_device change
23:59:42 mriedem but i've got to run
#openstack-nova - 2017-12-06
00:00:23 ildikov mriedem_away: ok, I'll look into those, thanks!
01:02:20 openstackgerrit Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395
02:10:54 mriedem edleafe: i led you astray on the compute rpcapi conversion stuff from https://review.openstack.org/#/c/516707/
02:11:19 mriedem you need that isinstance check in the rpcapi client to convert the SchedulerLimits object to a dict before sending to compute, that's why the functional tests are failing,
02:11:34 mriedem we can't send the object form until the computes are upgraded to handle that, which would be an rpc api version change
04:11:59 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
05:35:51 openstackgerrit jichenjc proposed openstack/nova master: remove glance usage inside compute https://review.openstack.org/511392
05:35:52 openstackgerrit jichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url https://review.openstack.org/511397
05:41:52 openstackgerrit jichenjc proposed openstack/nova master: Remove 'nova-manage host' and 'nova-manage agent' https://review.openstack.org/521837
05:41:53 openstackgerrit jichenjc proposed openstack/nova master: Remove 'nova-manage account' and 'nova-manage project' https://review.openstack.org/521833
05:41:53 openstackgerrit jichenjc proposed openstack/nova master: Remove 'nova-manage shell' command https://review.openstack.org/521835
05:41:54 openstackgerrit jichenjc proposed openstack/nova master: Remove 'nova-manage logs' command https://review.openstack.org/522133
05:46:50 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641
05:53:23 openstackgerrit wanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached https://review.openstack.org/525540
06:05:19 frickler could someone take a look at https://bugs.launchpad.net/bugs/1733933 and possibly give it some increased importance? seems it is masking any error happening on the compute side towards the user
06:05:21 openstack Launchpad bug 1733933 in OpenStack Compute (nova) "nova-conductor is masking error when rescheduling" [Undecided,Confirmed]
06:12:19 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
06:19:07 openstackgerrit Rajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys https://review.openstack.org/504885
06:21:55 Rambo hello!everyone,I am new to this,please correct me!
06:55:38 openstackgerrit jichenjc proposed openstack/nova master: Refactor placement version check https://review.openstack.org/512497
07:00:04 openstackgerrit Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334
07:28:44 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for snapshot instances https://review.openstack.org/525921
07:31:59 openstackgerrit Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395
07:34:20 openstackgerrit Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395
07:47:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix a description for 'guest_format' https://review.openstack.org/525928
08:20:49 Rambo__ Now the boot from volume 's instance of rebuild operation has a problem.For example,after the rebuild operation,the instance 's root disk is not replace.
08:20:50 Rambo__ To solve it,I plan to use CLI 's "metadata" option,to increase a key name "source_type".The "source_type" includes "snapshot" and "image".We can judge from "source_type".If the "source_type" is "snapshot",we can transform the given snapshot to a volume and attach this volume to instance.If the "source_type" is "image",we don't handle it.
08:20:50 Rambo__ I found the reason is that when we use the "_build_resources" function to prepare source,it obtains the block devices according to the previous instance 's uuid and attaches them to instance.So boot from volume 's instance of rebuild operation doesn't update data.
08:20:53 Rambo__ Can you give me some advice?Help in troubleshooting this issue will be appreciated.
08:25:33 openstackgerrit wanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached https://review.openstack.org/525540

Earlier   Later