Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-05
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
08:43:54 openstackgerrit zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
08:49:12 openstackgerrit zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
08:51:30 openstackgerrit Zhenyu Zheng proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip https://review.openstack.org/525505
08:55:42 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
09:00:47 openstackgerrit Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395
09:05:12 openstackgerrit Zhenyu Zheng proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip https://review.openstack.org/525505
09:05:44 openstackgerrit Gyorgy Szombathelyi proposed openstack/nova stable/ocata: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/525997
09:17:53 openstackgerrit zhurong proposed openstack/nova master: Remove the unused request_id filter from api-paste.ini https://review.openstack.org/526001
09:30:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242
09:32:59 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242
09:51:34 openstackgerrit Stephen Finucane proposed openstack/nova master: test: Store the OutputStreamCapture fixture https://review.openstack.org/515146
09:51:35 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-status: Migrate to cliff https://review.openstack.org/515147
09:51:35 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Rename 'policy_check' -> 'policy' https://review.openstack.org/515148
09:51:36 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff https://review.openstack.org/515149
10:12:41 openstackgerrit Merged openstack/python-novaclient master: Avoid tox_install.sh for constraints support https://review.openstack.org/524837
10:30:27 openstackgerrit Merged openstack/python-novaclient master: Microversion 2.56 - Enable cold migration with target host https://review.openstack.org/406707
10:43:15 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386
10:49:18 openstackgerrit Merged openstack/nova master: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660
10:49:24 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/525386
12:31:28 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386
12:42:40 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.57 - List/Show all server migration types https://review.openstack.org/430839
12:43:27 openstackgerrit Merged openstack/nova master: Genericify the instance_list stuff https://review.openstack.org/524253
12:43:34 openstackgerrit Merged openstack/nova master: Split instance_list into instance and multi_cell https://review.openstack.org/524302

Earlier   Later