Earlier  
Posted Nick Remark
#openstack-nova - 2017-07-07
01:27:35 openstackgerrit yuanyue proposed openstack/nova master: Add log info in scheduler to mark start of scheduling https://review.openstack.org/481340
01:53:50 Kevin_Zheng Hi, when we boot instance, in cells v2, we call schedule_and_build_instances, but if we have to reschedule, we called build_instances, why is that?
01:54:38 Kevin_Zheng shouldn't we call schedule_and_build_instances again?
01:54:39 mriedem because nova-compute has always done a reschedule by doing an rpc cast to build_instances
01:54:51 mriedem Kevin_Zheng: no, build_instances is in the cell-level conductor,
01:54:54 mriedem so the compute can reach it
01:55:04 Kevin_Zheng AH, ok
01:55:04 mriedem but schedule_and_build_instances talks to the scheduler, which is all at the top
01:55:09 mriedem and we don't want the computes to be able to reach the top
01:55:16 mriedem "super" conductor
01:55:21 Kevin_Zheng I understood now
01:55:28 mriedem although, build_instances calls the scheduler too...
01:55:49 mriedem build_instances isn't cell aware either,
01:56:06 mriedem schedule_and_build_instances is working under the assumption that the instance does not exist in a cell yet
01:56:11 mriedem that's the better reason why the compute doesn't call it
01:57:44 Kevin_Zheng error isntances should stay in cell0 or cellN?
01:57:54 mriedem cell0
01:58:04 mriedem if you're talking about an instance that fails to build at the top and doesn't reach a cell
01:58:15 mriedem if it reaches a cell and eventually goes to error, then it's in the cell it was scheduled in
01:58:51 Kevin_Zheng OK
01:59:03 Kevin_Zheng got it
01:59:34 mriedem we should probably have something in https://docs.openstack.org/nova/latest/cells.html#faqs about cells0
01:59:40 mriedem since that's a common question
02:00:15 mriedem although higher up it describes cell0,
02:00:16 mriedem "Instances that are never scheduled are relegated to the cell0 database, which is effectively a graveyard of instances that failed to start. All successful/running instances are stored in “cell1”."
02:00:48 mriedem an instance can reach a cell and still fail to start,
02:00:53 mriedem so that sentence is not entirely clear
02:03:44 Kevin_Zheng yeah, not only successful
02:53:42 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
02:57:04 openstackgerrit Merged openstack/nova master: placement: filter usage records by resource provider id https://review.openstack.org/480130
03:37:30 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
04:19:25 openstackgerrit Merged openstack/nova master: Fix comment for API binary name in WSGIService https://review.openstack.org/480648
04:28:11 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach https://review.openstack.org/480392
04:28:42 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - Enable cold migration with target host https://review.openstack.org/406707
04:28:47 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - List/Show all server migration types https://review.openstack.org/430839
04:41:06 openstackgerrit sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
05:25:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671
05:25:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle Cinder 3.27 style attachments in swap_volume https://review.openstack.org/456971
05:25:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
05:25:08 mriedem ildikov: tests and stuff are done in the swap volume patch now ^
05:25:38 mriedem i think i'll +1 since i'm good with it now
05:29:19 openstackgerrit Arundhati Surpur proposed openstack/python-novaclient master: Removed extra 'method' from the NOTE https://review.openstack.org/480988
05:30:42 openstackgerrit Arundhati Surpur proposed openstack/python-novaclient master: Removed extra word 'method' from the NOTE https://review.openstack.org/480988
05:43:26 openstackgerrit joohyun proposed openstack/nova master: Remove TODO message https://review.openstack.org/481436
05:44:30 openstackgerrit joohyun proposed openstack/nova master: Remove TODO message https://review.openstack.org/481436
05:50:34 Abhishek Hi..My nova-compute is keep on getting inactive after few seconds.Can anyone suggest what could be the issue
05:51:18 Abhishek http://www.heypasteit.com/clip/0IIO6P ->here is the log file
05:55:14 mriedem 2017-07-07 11:10:51.754 23129 ERROR oslo_service.service AttributeError: 'module' object has no attribute 'VIR_MIGRATE_POSTCOPY'
05:55:28 mriedem your libvirt-python package likely has a problem
05:55:41 mriedem and needs to be rebuilt
05:57:14 Abhishek mriedem:Should i remove the package and re-install it
05:57:29 mriedem that probably won't fix it
05:57:46 mriedem i can't remember what the exact nature of this usually is, but it's something to do with the packaging,
05:58:02 mriedem it's saying it's a newer libvirt version than the libvirt-python was built against
05:58:16 mriedem but the libvirt-python code was built against an older version
06:03:00 Abhishek mriedem:Basically,is it a version mismatch between the version installed in Ubuntu16.04 and the nova-compute conf file
06:03:10 mriedem has nothing to do with nova
06:04:05 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add ability to signal and perform online volume size change https://review.openstack.org/454322
06:04:18 Abhishek mriedem:Any way to fix this
06:05:05 mriedem clarkb has tricks, but i don't remember them. i think it ultimately means the libvirt-python package wasn't built against the same version of libvirt that you have on the system, so you need to make sure those are the same
06:05:15 mriedem ?
06:05:15 mriedem rpm -qa | grep libvirt
06:05:22 mriedem make sure those are the same
06:05:35 Abhishek root@01hw467434:/var/lib/dpkg# rpm -qa | grep libvirt root@01hw467434:/var/lib/dpkg#
06:05:42 Abhishek i do not see any
06:07:09 mriedem ?
06:07:09 mriedem apt list | grep libvirt
06:08:08 openstackgerrit KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443
06:08:42 Abhishek mriedem:http://www.heypasteit.com/clip/0IIO6S
06:11:37 Abhishek mriedem:apt list | grep libvirt output is in the location ->http://www.heypasteit.com/clip/0IIO6S
06:12:42 mriedem sorry but i can't help you much further here
06:13:21 Abhishek mriedem:No problem,Thank you for your time
06:15:23 openstackgerrit KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443
06:16:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove reverts_task_state decorator from swap_volume https://review.openstack.org/481447
06:18:20 openstackgerrit Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
06:25:05 mriedem Kevin_Zheng: want to fix the little issue in here? https://review.openstack.org/#/c/459593/
06:25:06 mriedem then i'm +2
06:25:58 Kevin_Zheng sure thanks, wow, you still up yet?
06:26:09 mriedem yeah, decided it was going to be a long night
06:28:14 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
06:28:31 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
06:29:12 Kevin_Zheng 4/4 also rebased to the latest version
06:29:47 mriedem cool, now just need to get alex_xu to look at https://review.openstack.org/#/c/459593/ again, i remember he was +2 on it before
06:31:36 Kevin_Zheng and the notification patch is also on the way, I messed up with a bad rebase last time ... will fix latter today
06:37:53 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP https://review.openstack.org/480379
06:47:28 openstackgerrit sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
06:54:06 mriedem alex_xu: i've replied to your comments in https://review.openstack.org/#/c/471899/
06:54:14 mriedem thanks for going through that, it's a massive change
06:54:24 alex_xu mriedem: np, will check soon
06:54:50 ildikov mriedem: thank you!
07:01:58 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle Cinder 3.27 style attachments in swap_volume https://review.openstack.org/456971
07:01:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671
07:02:00 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
07:02:05 mriedem Kevin_Zheng: it would be good if you could review that bottom swap volume change as well ^
07:02:19 mriedem Kevin_Zheng: the new cinder v3 attach flow needs some more people involved on the nova side,
07:02:26 mriedem for reviews and understanding
07:02:43 Kevin_Zheng sure, will do
07:04:56 alex_xu mriedem: replied
07:06:08 openstackgerrit Alex Szarka proposed openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582
07:06:22 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660

Earlier   Later