| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-07 | |||
| 00:47:15 | alex_xu | jaypipes: thanks \o/ | |
| 00:48:07 | jaypipes | alex_xu: would be good to add a functional test that verified that (though, I understand that all of our fixtures and the way we currently do testing would *not* have shown that issue since the usages table was appropriately filtered by resource_class_id already. | |
| 00:49:57 | alex_xu | jaypipes: you mean a functional test for usage filtered by resource_class_id? | |
| 00:51:33 | jaypipes | alex_xu: no, I mean add a functional test that showed the missing WHERE condition that was fixed in your patch. | |
| 00:52:30 | alex_xu | jaypipes: emm..I changed the function tests to expose that bug https://review.openstack.org/#/c/480130/3/nova/tests/functional/db/test_resource_provider.py | |
| 00:52:54 | mriedem | jaypipes: +2 | |
| 00:53:05 | mriedem | dansmith: for your morning, +2 on the allocation candidates in scheduler change https://review.openstack.org/#/c/476631/ | |
| 00:55:06 | jaypipes | alex_xu: understood. but those tests, while exposing the bug, didn't directly call _get_usages_by_provider_and_rc(). that's all I was saying, sorry, I wasn't being clear! | |
| 00:56:05 | alex_xu | jaypipes: ah, I see now | |
| 00:56:55 | alex_xu | I will check that | |
| 01:03:31 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/481189 | |
| 01:15:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test new style volume attach with live migration https://review.openstack.org/481290 | |
| 01:15:10 | openstackgerrit | yuanyue proposed openstack/nova master: Add log info in scheduler to mark start of scheduling https://review.openstack.org/481340 | |
| 01:16:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test new style volume attach with live migration https://review.openstack.org/481290 | |
| 01:21:17 | alex_xu | jaypipes: https://review.openstack.org/#/c/480379/2/nova/objects/resource_provider.py@2422...i'm doing stupid thing...why I join the table! | |
| 01:21:30 | jaypipes | alex_xu: :) done that many times myself. | |
| 01:22:15 | alex_xu | jaypipes: for 2637, it should be English problem, I shouldn't use 'filter out', I will reparse | |
| 01:22:26 | jaypipes | kk | |
| 01:22:51 | alex_xu | jaypipes: bauzas expect one SQL to done all the things, do you expect that, or you are happy the current way. | |
| 01:23:16 | jaypipes | alex_xu: I don't mind if there's >1 SQL statement, no. | |
| 01:23:20 | mriedem | are you asking jaypipes if he thinks sql can do everything? | |
| 01:23:25 | jaypipes | heh | |
| 01:23:27 | alex_xu | mriedem: :) | |
| 01:24:08 | mriedem | jay's quest in life is to find the one single sql statement that can rule them all | |
| 01:24:23 | jaypipes | SELECT 42 FROM DUAL. | |
| 01:24:40 | alex_xu | jaypipes: can you rewrite the whole nova api with one sql? please | |
| 01:24:51 | jaypipes | lol. dansmith would kill me. | |
| 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 | mriedem | but schedule_and_build_instances talks to the scheduler, which is all at the top | |
| 01:55:04 | Kevin_Zheng | AH, ok | |
| 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: Handle Cinder 3.27 style attachments in swap_volume https://review.openstack.org/456971 | |
| 05:25:04 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671 | |
| 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 | rpm -qa | grep libvirt | |
| 06:05:15 | mriedem | ? | |
| 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 | apt list | grep libvirt | |
| 06:07:09 | mriedem | ? | |
| 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 | |