| gmann | mriedem, good morning. can you check this spec, i updated the quota class APi fix spec for network quota filter out - https://review.openstack.org/#/c/480815/ | 00:06 |
|---|---|---|
| *** sree_ has joined #openstack-nova | 00:06 | |
| *** sree_ is now known as Guest89443 | 00:06 | |
| *** gmann has quit IRC | 00:10 | |
| *** gmann has joined #openstack-nova | 00:10 | |
| *** yangyapeng has quit IRC | 00:14 | |
| *** slaweq has joined #openstack-nova | 00:14 | |
| *** yangyapeng has joined #openstack-nova | 00:14 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631 | 00:16 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632 | 00:16 |
| jaypipes | mriedem: ^ k, fixed up. | 00:16 |
| *** abalutoiu has quit IRC | 00:17 | |
| *** yangyapeng has quit IRC | 00:18 | |
| *** slaweq has quit IRC | 00:18 | |
| *** Sukhdev has quit IRC | 00:21 | |
| *** thorst has joined #openstack-nova | 00:23 | |
| *** thorst has quit IRC | 00:26 | |
| *** armax has joined #openstack-nova | 00:26 | |
| *** felipemonteiro has quit IRC | 00:28 | |
| mriedem | cool | 00:30 |
| *** Guest89443 has quit IRC | 00:30 | |
| *** moshele has quit IRC | 00:34 | |
| jaypipes | mriedem: sorry for the delay. was out this afternoon. | 00:35 |
| mriedem | np, one teeny tiny thing https://review.openstack.org/#/c/476631/17 | 00:35 |
| mriedem | and then +2 to the extreme | 00:35 |
| *** rajathagasthya has quit IRC | 00:39 | |
| jaypipes | mriedem: done. | 00:41 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631 | 00:41 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632 | 00:41 |
| alex_xu | jaypipes: easy fix for AllocationCandidates https://review.openstack.org/480379 :) | 00:42 |
| alex_xu | jaypipes: sorry, that is the hard one, need your feedback, the easy one is https://review.openstack.org/480130 | 00:42 |
| *** claudiub has quit IRC | 00:42 | |
| jaypipes | alex_xu: cool, will review both later tonight. | 00:42 |
| alex_xu | jaypipes: thanks! | 00:42 |
| *** zhurong has joined #openstack-nova | 00:44 | |
| *** ijw has joined #openstack-nova | 00:46 | |
| openstackgerrit | yuanyue proposed openstack/nova master: Add log info in scheduler to mark start of scheduling https://review.openstack.org/481340 | 00:46 |
| jaypipes | alex_xu: approved the one. nice catch! :) | 00:47 |
| alex_xu | jaypipes: thanks \o/ | 00:47 |
| 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:48 |
| alex_xu | jaypipes: you mean a functional test for usage filtered by resource_class_id? | 00:49 |
| jaypipes | alex_xu: no, I mean add a functional test that showed the missing WHERE condition that was fixed in your patch. | 00:51 |
| 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 |
| *** ijw has quit IRC | 00:52 | |
| *** Shunli has joined #openstack-nova | 00:52 | |
| mriedem | jaypipes: +2 | 00:52 |
| mriedem | dansmith: for your morning, +2 on the allocation candidates in scheduler change https://review.openstack.org/#/c/476631/ | 00:53 |
| *** dixiaoli has joined #openstack-nova | 00:53 | |
| *** iceyao has joined #openstack-nova | 00:53 | |
| 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:55 |
| *** dixiaoli has quit IRC | 00:55 | |
| alex_xu | jaypipes: ah, I see now | 00:56 |
| *** lbragstad has quit IRC | 00:56 | |
| *** trungnv has quit IRC | 00:56 | |
| alex_xu | I will check that | 00:56 |
| *** iceyao has quit IRC | 00:58 | |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/481189 | 01:03 |
| *** fragatin_ has joined #openstack-nova | 01:05 | |
| *** sree_ has joined #openstack-nova | 01:05 | |
| *** sree_ is now known as Guest12148 | 01:06 | |
| *** thorst has joined #openstack-nova | 01:06 | |
| *** thorst has quit IRC | 01:07 | |
| *** fragati__ has joined #openstack-nova | 01:07 | |
| *** ijw has joined #openstack-nova | 01:08 | |
| *** fragatina has quit IRC | 01:08 | |
| *** lbragstad has joined #openstack-nova | 01:10 | |
| *** fragatin_ has quit IRC | 01:10 | |
| *** kaisers_ has joined #openstack-nova | 01:11 | |
| *** fragati__ has quit IRC | 01:12 | |
| *** ijw has quit IRC | 01:13 | |
| *** slaweq has joined #openstack-nova | 01:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test new style volume attach with live migration https://review.openstack.org/481290 | 01:15 |
| openstackgerrit | yuanyue proposed openstack/nova master: Add log info in scheduler to mark start of scheduling https://review.openstack.org/481340 | 01:15 |
| *** kaisers_ has quit IRC | 01:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test new style volume attach with live migration https://review.openstack.org/481290 | 01:16 |
| *** jwcroppe_ has joined #openstack-nova | 01:18 | |
| *** trungnv has joined #openstack-nova | 01:19 | |
| *** yamahata has quit IRC | 01:19 | |
| *** slaweq has quit IRC | 01:20 | |
| *** yangyapeng has joined #openstack-nova | 01:20 | |
| *** iceyao has joined #openstack-nova | 01:21 | |
| 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 |
| jaypipes | alex_xu: :) done that many times myself. | 01:21 |
| *** jwcroppe has quit IRC | 01:21 | |
| *** yangyapeng has quit IRC | 01:21 | |
| *** yangyapeng has joined #openstack-nova | 01:21 | |
| alex_xu | jaypipes: for 2637, it should be English problem, I shouldn't use 'filter out', I will reparse | 01:22 |
| jaypipes | kk | 01:22 |
| alex_xu | jaypipes: bauzas expect one SQL to done all the things, do you expect that, or you are happy the current way. | 01:22 |
| jaypipes | alex_xu: I don't mind if there's >1 SQL statement, no. | 01:23 |
| mriedem | are you asking jaypipes if he thinks sql can do everything? | 01:23 |
| jaypipes | heh | 01:23 |
| alex_xu | mriedem: :) | 01:23 |
| *** itlinux has joined #openstack-nova | 01:23 | |
| *** itlinux has quit IRC | 01:23 | |
| mriedem | jay's quest in life is to find the one single sql statement that can rule them all | 01:24 |
| jaypipes | SELECT 42 FROM DUAL. | 01:24 |
| alex_xu | jaypipes: can you rewrite the whole nova api with one sql? please | 01:24 |
| jaypipes | lol. dansmith would kill me. | 01:24 |
| *** itlinux has joined #openstack-nova | 01:25 | |
| openstackgerrit | yuanyue proposed openstack/nova master: Add log info in scheduler to mark start of scheduling https://review.openstack.org/481340 | 01:27 |
| *** acormier has joined #openstack-nova | 01:30 | |
| *** Apoorva has quit IRC | 01:30 | |
| *** acormier has quit IRC | 01:31 | |
| *** acormier has joined #openstack-nova | 01:32 | |
| *** liujiong has joined #openstack-nova | 01:34 | |
| *** ijw has joined #openstack-nova | 01:36 | |
| *** wong_ has joined #openstack-nova | 01:37 | |
| *** smatzek has joined #openstack-nova | 01:38 | |
| *** ijw has quit IRC | 01:47 | |
| *** sapcc-bot has quit IRC | 01:47 | |
| *** sapcc-bot has joined #openstack-nova | 01:47 | |
| 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:53 |
| Kevin_Zheng | shouldn't we call schedule_and_build_instances again? | 01:54 |
| mriedem | because nova-compute has always done a reschedule by doing an rpc cast to build_instances | 01:54 |
| mriedem | Kevin_Zheng: no, build_instances is in the cell-level conductor, | 01:54 |
| mriedem | so the compute can reach it | 01:54 |
| mriedem | but schedule_and_build_instances talks to the scheduler, which is all at the top | 01:55 |
| Kevin_Zheng | AH, ok | 01:55 |
| mriedem | and we don't want the computes to be able to reach the top | 01:55 |
| mriedem | "super" conductor | 01:55 |
| Kevin_Zheng | I understood now | 01:55 |
| mriedem | although, build_instances calls the scheduler too... | 01:55 |
| mriedem | build_instances isn't cell aware either, | 01:55 |
| *** hongbin has joined #openstack-nova | 01:55 | |
| mriedem | schedule_and_build_instances is working under the assumption that the instance does not exist in a cell yet | 01:56 |
| mriedem | that's the better reason why the compute doesn't call it | 01:56 |
| Kevin_Zheng | error isntances should stay in cell0 or cellN？ | 01:57 |
| mriedem | cell0 | 01:57 |
| mriedem | if you're talking about an instance that fails to build at the top and doesn't reach a cell | 01:58 |
| mriedem | if it reaches a cell and eventually goes to error, then it's in the cell it was scheduled in | 01:58 |
| Kevin_Zheng | OK | 01:58 |
| Kevin_Zheng | got it | 01:59 |
| mriedem | we should probably have something in https://docs.openstack.org/nova/latest/cells.html#faqs about cells0 | 01:59 |
| mriedem | since that's a common question | 01:59 |
| mriedem | although higher up it describes cell0, | 02:00 |
| 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 |
| mriedem | an instance can reach a cell and still fail to start, | 02:00 |
| mriedem | so that sentence is not entirely clear | 02:00 |
| Kevin_Zheng | yeah, not only successful | 02:03 |
| *** litao__ has joined #openstack-nova | 02:05 | |
| *** gjayavelu has quit IRC | 02:14 | |
| *** acormier_ has joined #openstack-nova | 02:19 | |
| *** acormier has quit IRC | 02:22 | |
| *** gyee has quit IRC | 02:23 | |
| *** smatzek has quit IRC | 02:39 | |
| *** Guest12148 has quit IRC | 02:51 | |
| *** hieulq_ has joined #openstack-nova | 02:52 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 02:53 |
| *** lucasxu has joined #openstack-nova | 02:53 | |
| openstackgerrit | Merged openstack/nova master: placement: filter usage records by resource provider id https://review.openstack.org/480130 | 02:57 |
| *** hieulq_ has quit IRC | 02:58 | |
| *** hieulq_ has joined #openstack-nova | 02:58 | |
| *** hieulq_ has quit IRC | 02:59 | |
| *** lucasxu has quit IRC | 02:59 | |
| *** acormier_ has quit IRC | 03:03 | |
| *** hieulq_ has joined #openstack-nova | 03:07 | |
| *** karthiks has joined #openstack-nova | 03:07 | |
| *** thorst has joined #openstack-nova | 03:07 | |
| *** thorst has quit IRC | 03:13 | |
| *** felipemonteiro has joined #openstack-nova | 03:23 | |
| *** Shunli has quit IRC | 03:25 | |
| *** Shunli has joined #openstack-nova | 03:26 | |
| *** psachin has joined #openstack-nova | 03:27 | |
| *** Nel1x has quit IRC | 03:29 | |
| *** hieulq_ has quit IRC | 03:32 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995 | 03:37 |
| *** felipemonteiro has quit IRC | 03:43 | |
| *** cNilesh has joined #openstack-nova | 03:51 | |
| *** Shunli has quit IRC | 03:54 | |
| *** Shunli has joined #openstack-nova | 03:55 | |
| *** yamamoto has joined #openstack-nova | 03:56 | |
| *** kaisers_ has joined #openstack-nova | 03:57 | |
| *** kaisers_ has quit IRC | 04:02 | |
| *** kaisers_ has joined #openstack-nova | 04:05 | |
| *** thorst has joined #openstack-nova | 04:09 | |
| *** karthiks has quit IRC | 04:10 | |
| *** iceyao has quit IRC | 04:10 | |
| *** cNilesh is now known as cNilesh|bf | 04:13 | |
| *** thorst has quit IRC | 04:14 | |
| *** lbragstad has quit IRC | 04:14 | |
| *** yamahata has joined #openstack-nova | 04:16 | |
| *** iceyao has joined #openstack-nova | 04:18 | |
| *** slaweq has joined #openstack-nova | 04:19 | |
| openstackgerrit | Merged openstack/nova master: Fix comment for API binary name in WSGIService https://review.openstack.org/480648 | 04:19 |
| *** iceyao has quit IRC | 04:22 | |
| *** slaweq has quit IRC | 04:24 | |
| *** vladikr_ has quit IRC | 04:24 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach https://review.openstack.org/480392 | 04:28 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - Enable cold migration with target host https://review.openstack.org/406707 | 04:28 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - List/Show all server migration types https://review.openstack.org/430839 | 04:28 |
| *** links has joined #openstack-nova | 04:29 | |
| *** iceyao has joined #openstack-nova | 04:32 | |
| *** claudiub has joined #openstack-nova | 04:35 | |
| *** iceyao has quit IRC | 04:36 | |
| *** adisky__ has joined #openstack-nova | 04:41 | |
| openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 04:41 |
| *** faizy has joined #openstack-nova | 04:42 | |
| *** trinaths has joined #openstack-nova | 04:43 | |
| *** trinaths1 has joined #openstack-nova | 04:44 | |
| *** armax has quit IRC | 04:45 | |
| *** armax has joined #openstack-nova | 04:46 | |
| *** trinaths has quit IRC | 04:47 | |
| *** liujiong is now known as liujiong|away | 04:52 | |
| *** hongbin has quit IRC | 04:55 | |
| *** zzzeek has joined #openstack-nova | 04:55 | |
| *** dixiaoli has joined #openstack-nova | 04:56 | |
| *** zzzeek has quit IRC | 04:56 | |
| *** zzzeek has joined #openstack-nova | 05:02 | |
| *** zzzeek has quit IRC | 05:02 | |
| *** zzzeek has joined #openstack-nova | 05:04 | |
| *** zzzeek has quit IRC | 05:04 | |
| *** cNilesh has joined #openstack-nova | 05:07 | |
| *** karthiks has joined #openstack-nova | 05:07 | |
| *** zzzeek has joined #openstack-nova | 05:08 | |
| *** zzzeek has quit IRC | 05:09 | |
| *** cNilesh|bf has quit IRC | 05:09 | |
| *** zzzeek has joined #openstack-nova | 05:10 | |
| *** iceyao has joined #openstack-nova | 05:11 | |
| *** cNilesh has quit IRC | 05:15 | |
| *** cNilesh has joined #openstack-nova | 05:16 | |
| *** zzzeek has quit IRC | 05:16 | |
| *** slaweq has joined #openstack-nova | 05:19 | |
| *** thorst has joined #openstack-nova | 05:21 | |
| *** zzzeek has joined #openstack-nova | 05:22 | |
| *** zzzeek has quit IRC | 05:23 | |
| *** slaweq has quit IRC | 05:25 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle Cinder 3.27 style attachments in swap_volume https://review.openstack.org/456971 | 05:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671 | 05:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 05:25 |
| mriedem | ildikov: tests and stuff are done in the swap volume patch now ^ | 05:25 |
| *** prateek has joined #openstack-nova | 05:25 | |
| mriedem | i think i'll +1 since i'm good with it now | 05:25 |
| *** thorst has quit IRC | 05:25 | |
| *** ratailor has joined #openstack-nova | 05:28 | |
| openstackgerrit | Arundhati Surpur proposed openstack/python-novaclient master: Removed extra 'method' from the NOTE https://review.openstack.org/480988 | 05:29 |
| openstackgerrit | Arundhati Surpur proposed openstack/python-novaclient master: Removed extra word 'method' from the NOTE https://review.openstack.org/480988 | 05:30 |
| *** sapcc-bot2 has joined #openstack-nova | 05:32 | |
| *** sapcc-bot has quit IRC | 05:32 | |
| *** yamahata has quit IRC | 05:32 | |
| *** yangyape_ has joined #openstack-nova | 05:33 | |
| *** yangyapeng has quit IRC | 05:35 | |
| *** mdnadeem has joined #openstack-nova | 05:37 | |
| *** zzzeek has joined #openstack-nova | 05:40 | |
| *** zzzeek has quit IRC | 05:40 | |
| *** sridharg has joined #openstack-nova | 05:41 | |
| openstackgerrit | joohyun proposed openstack/nova master: Remove TODO message https://review.openstack.org/481436 | 05:43 |
| openstackgerrit | joohyun proposed openstack/nova master: Remove TODO message https://review.openstack.org/481436 | 05:44 |
| *** rcernin has joined #openstack-nova | 05:46 | |
| *** Abhishek has joined #openstack-nova | 05:47 | |
| *** shaohe_feng has quit IRC | 05:50 | |
| Abhishek | Hi..My nova-compute is keep on getting inactive after few seconds.Can anyone suggest what could be the issue | 05:50 |
| *** markvoelker has quit IRC | 05:50 | |
| *** markvoelker has joined #openstack-nova | 05:51 | |
| *** shaohe_feng has joined #openstack-nova | 05:51 | |
| Abhishek | http://www.heypasteit.com/clip/0IIO6P ->here is the log file | 05:51 |
| mriedem | 2017-07-07 11:10:51.754 23129 ERROR oslo_service.service AttributeError: 'module' object has no attribute 'VIR_MIGRATE_POSTCOPY' | 05:55 |
| *** ianw is now known as ianw_pto | 05:55 | |
| mriedem | your libvirt-python package likely has a problem | 05:55 |
| *** markvoelker has quit IRC | 05:55 | |
| mriedem | and needs to be rebuilt | 05:55 |
| Abhishek | mriedem:Should i remove the package and re-install it | 05:57 |
| mriedem | that probably won't fix it | 05:57 |
| mriedem | i can't remember what the exact nature of this usually is, but it's something to do with the packaging, | 05:57 |
| mriedem | it's saying it's a newer libvirt version than the libvirt-python was built against | 05:58 |
| mriedem | but the libvirt-python code was built against an older version | 05:58 |
| *** junbo has quit IRC | 06:01 | |
| *** zzzeek has joined #openstack-nova | 06:02 | |
| Abhishek | mriedem:Basically,is it a version mismatch between the version installed in Ubuntu16.04 and the nova-compute conf file | 06:03 |
| mriedem | has nothing to do with nova | 06:03 |
| 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 |
| Abhishek | mriedem:Any way to fix this | 06:04 |
| *** andreas_s has joined #openstack-nova | 06:04 | |
| 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 |
| mriedem | rpm -qa | grep libvirt | 06:05 |
| mriedem | ? | 06:05 |
| mriedem | make sure those are the same | 06:05 |
| Abhishek | root@01hw467434:/var/lib/dpkg# rpm -qa | grep libvirt root@01hw467434:/var/lib/dpkg# | 06:05 |
| Abhishek | i do not see any | 06:05 |
| mriedem | apt list | grep libvirt | 06:07 |
| mriedem | ? | 06:07 |
| *** armax has quit IRC | 06:07 | |
| *** slaweq has joined #openstack-nova | 06:07 | |
| *** armax has joined #openstack-nova | 06:07 | |
| openstackgerrit | KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443 | 06:08 |
| *** armax has quit IRC | 06:08 | |
| *** armax has joined #openstack-nova | 06:08 | |
| Abhishek | mriedem:http://www.heypasteit.com/clip/0IIO6S | 06:08 |
| *** armax has quit IRC | 06:08 | |
| *** armax has joined #openstack-nova | 06:09 | |
| *** Oku_OS-away is now known as Oku_OS | 06:09 | |
| *** armax has quit IRC | 06:09 | |
| *** armax has joined #openstack-nova | 06:10 | |
| *** armax has quit IRC | 06:10 | |
| *** yangyape_ has quit IRC | 06:10 | |
| *** vvargaszte has joined #openstack-nova | 06:10 | |
| *** armax has joined #openstack-nova | 06:10 | |
| *** yangyapeng has joined #openstack-nova | 06:11 | |
| *** armax has quit IRC | 06:11 | |
| Abhishek | mriedem:apt list | grep libvirt output is in the location ->http://www.heypasteit.com/clip/0IIO6S | 06:11 |
| *** armax has joined #openstack-nova | 06:11 | |
| *** armax has quit IRC | 06:12 | |
| mriedem | sorry but i can't help you much further here | 06:12 |
| *** yangyape_ has joined #openstack-nova | 06:13 | |
| Abhishek | mriedem:No problem,Thank you for your time | 06:13 |
| *** shaohe_feng has quit IRC | 06:14 | |
| openstackgerrit | KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443 | 06:15 |
| *** yangyapeng has quit IRC | 06:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove reverts_task_state decorator from swap_volume https://review.openstack.org/481447 | 06:16 |
| *** zzzeek has quit IRC | 06:17 | |
| *** Abhishek has quit IRC | 06:18 | |
| openstackgerrit | Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023 | 06:18 |
| *** markus_z has joined #openstack-nova | 06:24 | |
| mriedem | Kevin_Zheng: want to fix the little issue in here? https://review.openstack.org/#/c/459593/ | 06:25 |
| mriedem | then i'm +2 | 06:25 |
| *** shaohe_feng has joined #openstack-nova | 06:25 | |
| Kevin_Zheng | sure thanks, wow, you still up yet? | 06:25 |
| *** lpetrut has joined #openstack-nova | 06:25 | |
| mriedem | yeah, decided it was going to be a long night | 06:26 |
| *** liujiong|away is now known as liujiong | 06:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 06:28 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 06:28 |
| Kevin_Zheng | 4/4 also rebased to the latest version | 06:29 |
| 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:29 |
| *** yangyape_ has quit IRC | 06:30 | |
| *** yangyapeng has joined #openstack-nova | 06:30 | |
| 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:31 |
| *** yangyapeng has quit IRC | 06:32 | |
| *** yangyapeng has joined #openstack-nova | 06:33 | |
| *** yangyapeng has quit IRC | 06:33 | |
| *** yangyapeng has joined #openstack-nova | 06:34 | |
| *** gabor_antal_ has joined #openstack-nova | 06:35 | |
| *** sridharg has quit IRC | 06:35 | |
| *** thomasem_ has quit IRC | 06:36 | |
| *** sneti_ has quit IRC | 06:36 | |
| *** mrodden has quit IRC | 06:37 | |
| *** beagles has quit IRC | 06:37 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP https://review.openstack.org/480379 | 06:37 |
| *** htruta has quit IRC | 06:38 | |
| *** NostawRm has quit IRC | 06:38 | |
| *** gabor_antal has quit IRC | 06:38 | |
| *** thomasem_ has joined #openstack-nova | 06:38 | |
| *** sneti has joined #openstack-nova | 06:38 | |
| *** mrodden has joined #openstack-nova | 06:39 | |
| *** diga has joined #openstack-nova | 06:39 | |
| *** sridharg has joined #openstack-nova | 06:39 | |
| *** rha has quit IRC | 06:41 | |
| *** htruta has joined #openstack-nova | 06:42 | |
| *** NostawRm has joined #openstack-nova | 06:43 | |
| *** b3nt_pin has joined #openstack-nova | 06:43 | |
| *** b3nt_pin is now known as Guest9640 | 06:43 | |
| *** rha has joined #openstack-nova | 06:46 | |
| *** gouthamr has quit IRC | 06:47 | |
| openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 06:47 |
| *** ijw has joined #openstack-nova | 06:48 | |
| *** ijw has quit IRC | 06:52 | |
| mriedem | alex_xu: i've replied to your comments in https://review.openstack.org/#/c/471899/ | 06:54 |
| mriedem | thanks for going through that, it's a massive change | 06:54 |
| alex_xu | mriedem: np, will check soon | 06:54 |
| ildikov | mriedem: thank you! | 06:54 |
| *** liujiong has quit IRC | 07:01 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle Cinder 3.27 style attachments in swap_volume https://review.openstack.org/456971 | 07:01 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671 | 07:01 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 07:02 |
| mriedem | Kevin_Zheng: it would be good if you could review that bottom swap volume change as well ^ | 07:02 |
| mriedem | Kevin_Zheng: the new cinder v3 attach flow needs some more people involved on the nova side, | 07:02 |
| mriedem | for reviews and understanding | 07:02 |
| Kevin_Zheng | sure, will do | 07:02 |
| alex_xu | mriedem: replied | 07:04 |
| *** slaweq has quit IRC | 07:05 | |
| *** mkucia has quit IRC | 07:05 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582 | 07:06 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 07:06 |
| mriedem | alex_xu: got it, thanks again. i would say i will deal with that tomorrow, but it's already tomorrow :) | 07:07 |
| mriedem | so i'll deal with it in a few hours | 07:07 |
| alex_xu | mriedem: :) yea, really shock you still awake up now | 07:09 |
| openstackgerrit | iswarya vakati proposed openstack/nova master: Fixed wrap from taking negative values https://review.openstack.org/481465 | 07:10 |
| mriedem | not for long, i'm signing off - see you in awhile | 07:10 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | 07:10 |
| *** mriedem has quit IRC | 07:10 | |
| *** lpetrut has quit IRC | 07:13 | |
| *** alexchadin has joined #openstack-nova | 07:16 | |
| *** itlinux has quit IRC | 07:16 | |
| *** tesseract has joined #openstack-nova | 07:17 | |
| *** jpena|off is now known as jpena | 07:18 | |
| *** thorst has joined #openstack-nova | 07:21 | |
| *** damien_r has joined #openstack-nova | 07:24 | |
| *** thorst has quit IRC | 07:27 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add device tag support info in support matrix https://review.openstack.org/481478 | 07:44 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add description for tag support in attach https://review.openstack.org/481479 | 07:44 |
| *** ralonsoh has joined #openstack-nova | 07:48 | |
| *** rajathagasthya has joined #openstack-nova | 07:51 | |
| openstackgerrit | jichenjc proposed openstack/nova master: add description for supports_migrate_to_same_host https://review.openstack.org/481483 | 07:51 |
| *** markvoelker has joined #openstack-nova | 07:51 | |
| openstackgerrit | KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443 | 07:52 |
| *** iceyao has quit IRC | 07:54 | |
| *** efoley has joined #openstack-nova | 07:59 | |
| *** zzzeek_ has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** abalutoiu has joined #openstack-nova | 08:09 | |
| *** diga has quit IRC | 08:10 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: remove extension pop https://review.openstack.org/481491 | 08:11 |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove nova.api.extensions.server.extensions usage https://review.openstack.org/481494 | 08:15 |
| *** wong_ has quit IRC | 08:16 | |
| *** david-lyle has quit IRC | 08:19 | |
| *** david-lyle has joined #openstack-nova | 08:19 | |
| *** iceyao has joined #openstack-nova | 08:19 | |
| *** iceyao has quit IRC | 08:24 | |
| *** yangyap__ has joined #openstack-nova | 08:25 | |
| *** yangyapeng has quit IRC | 08:26 | |
| *** markvoelker has quit IRC | 08:26 | |
| openstackgerrit | jichenjc proposed openstack/nova master: add cli to support matrix https://review.openstack.org/481498 | 08:26 |
| *** pabardina has joined #openstack-nova | 08:28 | |
| pabardina | hello, Is it normal I don't see "locked_by" attribute when I'm using "nova show XX" on mitaka? I have "Locked: True" information, no more... But in the code there is the "locked_by" attr... any ideas ? | 08:31 |
| *** lucas-afk is now known as lucasagomes | 08:33 | |
| *** iceyao has joined #openstack-nova | 08:37 | |
| *** iceyao has quit IRC | 08:42 | |
| *** purplerbot has quit IRC | 08:44 | |
| *** purplerbot has joined #openstack-nova | 08:44 | |
| *** dixiaoli has quit IRC | 08:45 | |
| *** dixiaoli has joined #openstack-nova | 08:45 | |
| *** cdent has joined #openstack-nova | 08:49 | |
| *** Oku_OS has quit IRC | 08:49 | |
| *** priteau has joined #openstack-nova | 08:51 | |
| *** Oku_OS has joined #openstack-nova | 08:51 | |
| *** tojuvone has quit IRC | 08:53 | |
| *** tojuvone has joined #openstack-nova | 08:54 | |
| stephenfin | johnthetubaguy: Next time you're about, would appreciate it if you could remove the -2 on this change as it no longer applies https://review.openstack.org/#/c/449584 | 08:54 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - manager.py https://review.openstack.org/359868 | 08:58 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871 | 09:04 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - libvirt/config.py https://review.openstack.org/359879 | 09:04 |
| *** faizy has quit IRC | 09:04 | |
| *** psachin_ has joined #openstack-nova | 09:04 | |
| *** psachin_ has quit IRC | 09:05 | |
| *** faizy has joined #openstack-nova | 09:05 | |
| openstackgerrit | Merged openstack/nova master: Add 'networks' quota in quota sample files https://review.openstack.org/479227 | 09:06 |
| *** lpetrut has joined #openstack-nova | 09:14 | |
| *** yangyapeng has joined #openstack-nova | 09:19 | |
| *** yangyap__ has quit IRC | 09:22 | |
| *** markvoelker has joined #openstack-nova | 09:23 | |
| *** thorst has joined #openstack-nova | 09:23 | |
| *** reedip has quit IRC | 09:25 | |
| *** thorst has quit IRC | 09:28 | |
| *** Shunli has quit IRC | 09:29 | |
| gmann | pabardina, that will be present if you request API with microversion >2.9 | 09:29 |
| gmann | pabardina, and its there since liberty so mitaka is all fine to get that | 09:30 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 09:32 |
| pabardina | gmann, yes with microversion > 2.9, "locked" is present, but no "locked_by" | 09:32 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use URIOpt https://review.openstack.org/479215 | 09:33 |
| gmann | pabardina, ahh, yes its boolean 'locked' | 09:34 |
| *** karthiks has quit IRC | 09:34 | |
| pabardina | gmann, ok, it's fine then. thanks :) | 09:35 |
| gmann | pabardina, API never publish 'locked_by' i think it was only internally used | 09:35 |
| pabardina | pabardina, alright | 09:35 |
| pabardina | gmann, alright | 09:36 |
| *** takashin has left #openstack-nova | 09:36 | |
| *** aloga has quit IRC | 09:38 | |
| *** aloga has joined #openstack-nova | 09:38 | |
| *** reedip has joined #openstack-nova | 09:39 | |
| *** rajathagasthya has quit IRC | 09:39 | |
| *** yamahata has joined #openstack-nova | 09:41 | |
| *** alexchadin has quit IRC | 09:42 | |
| *** dixiaoli_ has joined #openstack-nova | 09:42 | |
| *** dixiaoli_ has quit IRC | 09:42 | |
| *** dixiaoli has quit IRC | 09:46 | |
| *** sulo has quit IRC | 09:47 | |
| *** sulo has joined #openstack-nova | 09:47 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: remove extension pop https://review.openstack.org/481491 | 09:48 |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** openstackgerrit has joined #openstack-nova | 09:48 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 09:48 |
| *** ZZelle has joined #openstack-nova | 09:51 | |
| *** sulo has quit IRC | 09:52 | |
| *** slaweq has joined #openstack-nova | 09:53 | |
| *** sulo has joined #openstack-nova | 09:55 | |
| *** markvoelker has quit IRC | 09:56 | |
| *** pabardina has quit IRC | 09:56 | |
| *** avolkov has quit IRC | 10:00 | |
| *** sambetts|afk is now known as sambetts | 10:06 | |
| cdent | jaypipes: another gabbi change you might be interested in: https://github.com/cdent/gabbi/pull/216 | 10:08 |
| *** jmlowe_ has joined #openstack-nova | 10:10 | |
| *** jmlowe has quit IRC | 10:11 | |
| bauzas | cdent: jaypipes: you knew that we had a problem with traits not synced in gabbi testing ? | 10:16 |
| bauzas | that's fixed by Monday but just to make sure you knew it | 10:16 |
| *** slaweq has quit IRC | 10:20 | |
| *** slaweq has joined #openstack-nova | 10:21 | |
| cdent | bauzas: this last monday? yeah | 10:25 |
| *** slaweq has quit IRC | 10:26 | |
| *** yamahata has quit IRC | 10:26 | |
| bauzas | cdent: kewl | 10:31 |
| bauzas | was just a FYI | 10:32 |
| cdent | yeah, thanks | 10:32 |
| *** alexchadin has joined #openstack-nova | 10:35 | |
| *** cNilesh has quit IRC | 10:39 | |
| *** cNilesh has joined #openstack-nova | 10:40 | |
| *** alexchadin has quit IRC | 10:42 | |
| *** yamamoto has quit IRC | 10:46 | |
| *** zhurong has quit IRC | 10:46 | |
| openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 10:46 |
| *** slaweq has joined #openstack-nova | 10:49 | |
| *** slaweq has quit IRC | 10:50 | |
| *** slaweq has joined #openstack-nova | 10:50 | |
| *** markvoelker has joined #openstack-nova | 10:53 | |
| *** ratailor has quit IRC | 10:58 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova master: Trivial: Remove unnecessary format specifier https://review.openstack.org/481574 | 10:59 |
| *** thorst has joined #openstack-nova | 11:00 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 11:01 |
| jaypipes | bauzas: yeah, that was the thing edleafe fixed, yeah? | 11:02 |
| bauzas | jaypipes: nope, he missed something | 11:05 |
| bauzas | jaypipes: https://review.openstack.org/#/c/480198/ | 11:05 |
| *** thorst has quit IRC | 11:05 | |
| *** cNilesh has quit IRC | 11:06 | |
| *** cNilesh has joined #openstack-nova | 11:06 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112 | 11:17 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits https://review.openstack.org/474550 | 11:17 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for traits https://review.openstack.org/474186 | 11:17 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations https://review.openstack.org/470933 | 11:17 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563 | 11:17 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924 | 11:17 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105 | 11:17 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550 | 11:17 |
| jaypipes | cdent: nice on that full JSON gabbi enhancement. :) | 11:21 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112 | 11:22 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits https://review.openstack.org/474550 | 11:22 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations https://review.openstack.org/470933 | 11:22 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563 | 11:22 |
| *** trinaths1 has left #openstack-nova | 11:22 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924 | 11:22 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105 | 11:22 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550 | 11:22 |
| cdent | jaypipes: if you want to put a thumbs up of some kind on either or both of #215 and #216 that will help convince some of the other people. | 11:23 |
| jaypipes | cdent: sure thing. | 11:23 |
| cdent | thanks | 11:23 |
| jaypipes | cdent: and... done. :) | 11:24 |
| cdent | word | 11:24 |
| *** markvoelker has quit IRC | 11:26 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112 | 11:26 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924 | 11:26 |
| *** lucasagomes is now known as lucas-hungry | 11:27 | |
| jaypipes | cdent: the bird. | 11:27 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112 | 11:27 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924 | 11:27 |
| jaypipes | I wonder what avolkov` is working on ... :) | 11:28 |
| avolkov` | jaypipes: sorry ) | 11:28 |
| jaypipes | avolkov`: why? it's a much-needed patch series! ++ | 11:28 |
| jaypipes | avolkov`: thank you very much for leading these doc efforts! | 11:29 |
| avolkov` | jaypipes: I'm about all these rebases those spam chat ), always miss something | 11:30 |
| *** yamamoto has joined #openstack-nova | 11:30 | |
| jaypipes | avolkov`: heh, no worries :) | 11:30 |
| cdent | avolkov`++ | 11:30 |
| *** gbarros has quit IRC | 11:34 | |
| *** iceyao has joined #openstack-nova | 11:34 | |
| *** psachin has quit IRC | 11:36 | |
| avolkov` | cdent: jaypipes: btw, there is some initiative to migrate docs, probably placement api ref would be the first one, https://review.openstack.org/#/c/480991/ | 11:36 |
| *** sdague has joined #openstack-nova | 11:36 | |
| cdent | avolkov`: yeah, saw that. I guess it makes sense given the docs migration spec (which feels a bit complicated but necessary) | 11:37 |
| cdent | I reckon we should merge all the pending placement-api-ref before moving stuff? | 11:38 |
| jaypipes | cdent: ++ | 11:39 |
| *** zhurong has joined #openstack-nova | 11:41 | |
| avolkov` | cdent: I think yes, we should have some accomplished thing before migration | 11:42 |
| *** yangyapeng has quit IRC | 11:44 | |
| *** markvoelker has joined #openstack-nova | 11:44 | |
| *** zhurong has quit IRC | 11:46 | |
| *** slaweq has quit IRC | 11:48 | |
| *** pcaruana has joined #openstack-nova | 11:49 | |
| bhagyashris | cdent: Hi, Thanks for your responses and suggestions. | 11:52 |
| cdent | bhagyashris: thank you for bringing it up | 11:52 |
| cdent | I'll be making a new release of gabbi probalby today | 11:53 |
| cdent | and then doing the global requirements changes | 11:53 |
| bhagyashris | cdent: Actually I am new to gabbi trying to understand it. | 11:53 |
| bhagyashris | cdent: Ohh, that will be great and thanks | 11:53 |
| bhagyashris | cdent: Just for my understanding i am going to changes #215 and #216. and trying take it as assignment so I have few difficulties. | 11:56 |
| *** thorst has joined #openstack-nova | 11:56 | |
| cdent | bhagyashris: what do you mean by "take it as assigment"? | 11:56 |
| *** vladikr_ has joined #openstack-nova | 11:58 | |
| *** cNilesh has quit IRC | 11:59 | |
| *** cNilesh has joined #openstack-nova | 12:00 | |
| *** jpena is now known as jpena|lunch | 12:01 | |
| bhagyashris | cdent: I have separately clone gabbi project and install in my environment. And applied the changes as in #216. But i am unaware about how to test the sample test. | 12:02 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | 12:03 |
| cdent | bhagyashris: the new changes can be tested with: tox -epy27 -- test_intercept.data | 12:03 |
| cdent | that will test just the tests in the data.yaml | 12:04 |
| bhagyashris | cdent: ok | 12:04 |
| *** litao__ has quit IRC | 12:04 | |
| bhagyashris | cdent: thanks | 12:05 |
| *** iceyao has quit IRC | 12:09 | |
| *** tbachman has quit IRC | 12:10 | |
| *** hongda has joined #openstack-nova | 12:10 | |
| *** iceyao has joined #openstack-nova | 12:11 | |
| *** jamesdenton has joined #openstack-nova | 12:14 | |
| *** edmondsw has joined #openstack-nova | 12:21 | |
| *** sapcc-bot2 has quit IRC | 12:23 | |
| *** carthaca_ has quit IRC | 12:23 | |
| *** carthaca_ has joined #openstack-nova | 12:24 | |
| *** mkoderer_ has joined #openstack-nova | 12:24 | |
| *** sapcc-bot has joined #openstack-nova | 12:24 | |
| *** tpatzig_ has joined #openstack-nova | 12:24 | |
| *** david_1 has joined #openstack-nova | 12:24 | |
| *** dgonzalez_ has joined #openstack-nova | 12:24 | |
| *** lucas-hungry is now known as lucasagomes | 12:25 | |
| *** mkoderer_ has quit IRC | 12:26 | |
| *** tpatzig_ has quit IRC | 12:26 | |
| *** david_1 has quit IRC | 12:26 | |
| *** dgonzalez_ has quit IRC | 12:26 | |
| *** hongda has quit IRC | 12:26 | |
| *** smatzek has joined #openstack-nova | 12:27 | |
| *** faizy has quit IRC | 12:28 | |
| *** smatzek has quit IRC | 12:31 | |
| *** ijw has joined #openstack-nova | 12:36 | |
| *** smatzek has joined #openstack-nova | 12:37 | |
| *** _pewp_ has quit IRC | 12:37 | |
| masber | good morning, may I ask a question? | 12:38 |
| kashyap | masber: Just ask :-) | 12:39 |
| *** catintheroof has joined #openstack-nova | 12:40 | |
| masber | I am trying to deploy a baremetal using ironic, my issue is that nova is telling me that I am exceeding my quota, however that is not true, I can see that I have enough resources | 12:40 |
| *** _pewp_ has joined #openstack-nova | 12:40 | |
| kashyap | Hmm, afraid not sure about it myself, as I don't know much about Ironic. | 12:41 |
| *** ijw has quit IRC | 12:41 | |
| masber | is there a command to show the resources used? | 12:41 |
| kashyap | A gentle note: Operator questions are best asked on #openstack-operators (or is it 'users'?) | 12:41 |
| kashyap | masber: `nova quota-show`? | 12:42 |
| kashyap | Depeding on what you're looking for, you may have to specify tenant | 12:43 |
| masber | that will show me the whole quota, is there a way to see the free resources but from a quota point of view? | 12:44 |
| *** acormier has joined #openstack-nova | 12:44 | |
| masber | like an overview to see how many cpus, ram, disks I am currently using? | 12:44 |
| masber | something like openstack hypervisor stats show but with quotas | 12:45 |
| *** lyan has joined #openstack-nova | 12:46 | |
| *** slaweq has joined #openstack-nova | 12:48 | |
| *** jaypipes is now known as leakypipes | 12:50 | |
| *** liverpooler has joined #openstack-nova | 12:50 | |
| *** fragatina has joined #openstack-nova | 12:52 | |
| *** acormier has quit IRC | 12:53 | |
| *** acormier has joined #openstack-nova | 12:53 | |
| *** slaweq has quit IRC | 12:54 | |
| *** cNilesh has quit IRC | 12:55 | |
| *** fragatina has quit IRC | 12:57 | |
| *** acormier has quit IRC | 12:59 | |
| *** acormier has joined #openstack-nova | 12:59 | |
| *** lucasxu has joined #openstack-nova | 12:59 | |
| *** edleafe is now known as figleaf | 12:59 | |
| *** pchavva has joined #openstack-nova | 13:03 | |
| *** gcb has quit IRC | 13:03 | |
| *** acormier has quit IRC | 13:04 | |
| *** links has quit IRC | 13:06 | |
| *** tbachman has joined #openstack-nova | 13:06 | |
| *** slaweq has joined #openstack-nova | 13:07 | |
| *** acormier has joined #openstack-nova | 13:07 | |
| *** takedakn has joined #openstack-nova | 13:08 | |
| *** jpena|lunch is now known as jpena | 13:08 | |
| *** slaweq has quit IRC | 13:11 | |
| *** acormier has quit IRC | 13:11 | |
| *** acormier has joined #openstack-nova | 13:12 | |
| *** mdnadeem has quit IRC | 13:14 | |
| *** mdnadeem has joined #openstack-nova | 13:14 | |
| openstackgerrit | Anthony PERARD proposed openstack/nova master: Skip boot from encrypted volume on Xen+libvirt https://review.openstack.org/481627 | 13:16 |
| *** acormier has quit IRC | 13:16 | |
| *** slaweq has joined #openstack-nova | 13:16 | |
| cdent | figleaf: yay! we don't agree on something! | 13:19 |
| *** zhurong has joined #openstack-nova | 13:23 | |
| *** alexchadin has joined #openstack-nova | 13:24 | |
| *** eharney has joined #openstack-nova | 13:26 | |
| *** esberglu has joined #openstack-nova | 13:28 | |
| *** Dinesh_Bhor has quit IRC | 13:32 | |
| *** slaweq_ has joined #openstack-nova | 13:32 | |
| *** jwcroppe_ has quit IRC | 13:37 | |
| *** slaweq_ has quit IRC | 13:37 | |
| *** jwcroppe has joined #openstack-nova | 13:37 | |
| *** sdague has quit IRC | 13:39 | |
| *** jwcroppe has quit IRC | 13:42 | |
| *** takedakn has quit IRC | 13:43 | |
| *** david_1 has joined #openstack-nova | 13:44 | |
| *** dgonzalez_ has joined #openstack-nova | 13:44 | |
| *** seife_ has joined #openstack-nova | 13:44 | |
| *** mkoderer_ has joined #openstack-nova | 13:44 | |
| *** dgonzalez_ has quit IRC | 13:46 | |
| *** seife_ has quit IRC | 13:46 | |
| *** david_1 has quit IRC | 13:46 | |
| *** mkoderer_ has quit IRC | 13:46 | |
| *** slaweq_ has joined #openstack-nova | 13:48 | |
| figleaf | cdent: oh, there's a lot. I'm just too polite to express myself :) | 13:48 |
| cdent | figleaf: polite is never a word I heard near you | 13:48 |
| *** jwcroppe has joined #openstack-nova | 13:53 | |
| *** burt has joined #openstack-nova | 13:53 | |
| *** baoli has joined #openstack-nova | 13:54 | |
| *** baoli has quit IRC | 13:54 | |
| *** baoli has joined #openstack-nova | 13:55 | |
| leakypipes | dansmith: morning superdan. | 13:55 |
| *** dansmith is now known as superdan | 13:55 | |
| superdan | shame is so effective | 13:55 |
| leakypipes | lol | 13:56 |
| *** efried is now known as fried_rice | 13:56 | |
| leakypipes | ah, knock-on effects. :) | 13:56 |
| fried_rice | Happy Friday! | 13:56 |
| fried_rice | Frieday | 13:56 |
| superdan | we should do more of these three-day workweek deals | 13:56 |
| leakypipes | heh | 13:56 |
| fried_rice | +1 | 13:56 |
| *** slaweq_ has quit IRC | 13:58 | |
| *** mriedem has joined #openstack-nova | 13:58 | |
| mriedem | o/ | 13:58 |
| leakypipes | mriedem: mornin. | 13:59 |
| *** alexchadin has quit IRC | 14:03 | |
| *** adisky__ has quit IRC | 14:08 | |
| *** hongbin has joined #openstack-nova | 14:08 | |
| *** amotoki is now known as amotoki_away | 14:09 | |
| *** slaweq has quit IRC | 14:09 | |
| *** slaweq has joined #openstack-nova | 14:10 | |
| mriedem | gmann: looking at https://review.openstack.org/#/c/480815/1/specs/pike/approved/fix-to-return-server-groups-quota-in-quota-class.rst@51 | 14:10 |
| mriedem | did we also miss those other resources in the quota class api, like fixed ips, floating ips, etc? | 14:11 |
| mriedem | or was it just 'networks'? | 14:11 |
| *** iceyao has quit IRC | 14:12 | |
| *** slaweq has quit IRC | 14:14 | |
| *** prateek has quit IRC | 14:14 | |
| *** slaweq has joined #openstack-nova | 14:14 | |
| *** tbachman has quit IRC | 14:17 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Update quota class spec to filter out networks quota https://review.openstack.org/480815 | 14:18 |
| *** gouthamr has joined #openstack-nova | 14:19 | |
| *** itlinux has joined #openstack-nova | 14:19 | |
| *** slaweq has quit IRC | 14:19 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:19 | |
| *** felipemonteiro__ has joined #openstack-nova | 14:21 | |
| *** felipemonteiro_ has quit IRC | 14:24 | |
| openstackgerrit | Merged openstack/nova-specs master: Update quota class spec to filter out networks quota https://review.openstack.org/480815 | 14:26 |
| *** zhurong has quit IRC | 14:26 | |
| *** slaweq has joined #openstack-nova | 14:26 | |
| *** awaugama has joined #openstack-nova | 14:27 | |
| *** armax has joined #openstack-nova | 14:27 | |
| *** mkucia has joined #openstack-nova | 14:27 | |
| mriedem | so who wants to review this change to use the cinder 3.27 attachments API flow in swap volume? https://review.openstack.org/#/c/456971/ | 14:31 |
| *** mdnadeem has quit IRC | 14:31 | |
| *** slaweq has quit IRC | 14:31 | |
| mriedem | tempest has already passed on it from higher in the stack which triggers the new behavior | 14:31 |
| *** mdnadeem has joined #openstack-nova | 14:32 | |
| *** mdnadeem has quit IRC | 14:32 | |
| leakypipes | mriedem: do you want me to move those two lines in the allocation candidates patch or do it in a followup patch? | 14:37 |
| mriedem | leakypipes: follow up is fine | 14:37 |
| leakypipes | k | 14:37 |
| *** andreas_s has quit IRC | 14:41 | |
| *** sdague has joined #openstack-nova | 14:44 | |
| *** lbragstad has joined #openstack-nova | 14:45 | |
| *** stvnoyes1 has quit IRC | 14:49 | |
| *** catintheroof has quit IRC | 14:52 | |
| mriedem | ruh roh | 14:53 |
| *** catintheroof has joined #openstack-nova | 14:53 | |
| *** itlinux has quit IRC | 14:53 | |
| mriedem | superdan: melwitt: i know you're on a call, but after that, let me ruin your day https://github.com/openstack/nova/blob/56cd608d3a199dcb02ac2ae071ff3057241259da/nova/compute/api.py#L3723 | 14:53 |
| mriedem | when attaching a volume to a shelved offloaded instance, we create the bdm in the api - so cell0 | 14:54 |
| mriedem | we must not have a tempest test for this | 14:54 |
| melwitt | rats | 14:54 |
| mriedem | could probably recreate with a functional test, but i'll start by reporting abug | 14:55 |
| *** catinthe_ has joined #openstack-nova | 14:56 | |
| mriedem | hmm, AttachVolumeShelveTestJSON in tempest should test this | 14:56 |
| superdan | guh | 14:57 |
| mriedem | so either the tempest test is busted, or i'm missing something that makes this work | 14:59 |
| *** catintheroof has quit IRC | 14:59 | |
| *** lucasxu has quit IRC | 15:04 | |
| *** Oku_OS is now known as Oku_OS-away | 15:04 | |
| *** rcernin has quit IRC | 15:06 | |
| *** catintheroof has joined #openstack-nova | 15:06 | |
| *** ociuhandu has joined #openstack-nova | 15:09 | |
| *** catinthe_ has quit IRC | 15:09 | |
| superdan | mriedem: well, that clearly needs to be targeted | 15:10 |
| *** sudipto has joined #openstack-nova | 15:10 | |
| *** sudipto_ has joined #openstack-nova | 15:10 | |
| leakypipes | cdent: thx for continuing your resource providers weekly summaries. good job catching up on all the change while you were on PTO. look to have gotten everything correct, AFAICT. | 15:11 |
| cdent | leakypipes: it's my UVP | 15:11 |
| *** markus_z has quit IRC | 15:11 | |
| mriedem | superdan: yeah, just really odd that the tempest test wouldn't fail, unless the test is broken | 15:11 |
| cdent | thanks for the confirmation that I didn't miss anything obvious | 15:12 |
| superdan | mriedem: yeah | 15:12 |
| superdan | mriedem: does that fail with a constraint error, or do we just create it and then it never attaches on unshelve? | 15:12 |
| superdan | because maybe the tempest test isn't validating via ssh | 15:12 |
| mriedem | the tempest test does do some ssh counting of volumes on the guest | 15:13 |
| superdan | mriedem: actually, | 15:13 |
| superdan | mriedem: until my devstack patch, the control services are not pointing at cell0 for their default | 15:14 |
| superdan | so that's why | 15:14 |
| mriedem | hmm, so why doesn't your devstack patch fail on this though? | 15:14 |
| superdan | mriedem: it'd be the canary against nova that d-o it that would fail | 15:14 |
| superdan | this guy: https://review.openstack.org/#/c/473565/ | 15:15 |
| superdan | is that test new or something? | 15:15 |
| superdan | hmm, nope passing in the canary | 15:15 |
| *** acormier has joined #openstack-nova | 15:16 | |
| cdent | leakypipes: requirements change for new gabbi if you're inclined to move that along: https://review.openstack.org/481665 | 15:16 |
| *** zzzeek_ has joined #openstack-nova | 15:17 | |
| *** zzzeek_ has quit IRC | 15:17 | |
| mriedem | tempest is configured to run ssh validatoin, so it should do this: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/volumes/test_attach_volume.py#n181 | 15:19 |
| mriedem | maybe the counter is broken so it always reports 0 and it's comparing 0 to 0 and passes? | 15:19 |
| leakypipes | cdent: done | 15:20 |
| superdan | mriedem: heh | 15:20 |
| mriedem | although this would fail http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/volumes/test_attach_volume.py#n219 | 15:20 |
| mriedem | num_vol + 1 | 15:20 |
| *** burgerk has joined #openstack-nova | 15:22 | |
| mriedem | superdan: we need to update your canary patch to depend on I08d7da843d18b426dda8a8a231039d950a4c0ce5 now | 15:25 |
| mriedem | shall i push the button? | 15:25 |
| superdan | mriedem: I'm working on a target change, so I will do that when I'm done here | 15:26 |
| cdent | thanks leakypipes | 15:26 |
| *** ckopper has quit IRC | 15:26 | |
| openstackgerrit | Merged openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 15:27 |
| * cdent dines | 15:27 | |
| *** cdent has quit IRC | 15:27 | |
| *** ckopper has joined #openstack-nova | 15:28 | |
| superdan | mriedem: was there a bug for this shelved issue? | 15:31 |
| mriedem | no, i can create one | 15:31 |
| superdan | thanks | 15:31 |
| mriedem | you know, | 15:33 |
| mriedem | another thing, | 15:33 |
| mriedem | when we shelve offload a server, we never clean up it's InstanceMapping, | 15:34 |
| mriedem | and when we unshelve it, conductor asks the scheduler for a new host, which could be in a different cell, | 15:34 |
| mriedem | but we never create a new instance mapping, or update the existing instance mapping, for that new cell | 15:34 |
| *** catinthe_ has joined #openstack-nova | 15:35 | |
| superdan | mriedem: we can't move it between cells, | 15:35 |
| superdan | so if that's true, we need to be asking the scheduler to limit to one cell, | 15:35 |
| superdan | like we do for resize | 15:35 |
| mriedem | i don't see anything in conductor that's doing that | 15:35 |
| mriedem | is it in the req spec? | 15:35 |
| superdan | yep | 15:35 |
| superdan | reqspec.destination.cell | 15:35 |
| *** ckopper has quit IRC | 15:36 | |
| *** catintheroof has quit IRC | 15:36 | |
| mriedem | ok this https://github.com/openstack/nova/blob/56cd608d3a199dcb02ac2ae071ff3057241259da/nova/conductor/tasks/migrate.py#L63-L66 | 15:38 |
| mriedem | that's the only place we set destination.cell from what i can tell | 15:39 |
| *** amotoki_away is now known as amotoki | 15:39 | |
| superdan | yep | 15:40 |
| mriedem | ok i'll open a separate bug for that | 15:41 |
| *** abalutoiu has quit IRC | 15:42 | |
| superdan | cool | 15:42 |
| *** chyka has joined #openstack-nova | 15:42 | |
| superdan | well, not cool, but.. | 15:42 |
| mriedem | about as cool as a kick in the pants | 15:42 |
| mriedem | maybe another nail in the shelve coffin | 15:42 |
| superdan | meh, anyone that uses shelve is probably used to shelve being broken | 15:43 |
| mriedem | "Days since shelve failed: 0" | 15:43 |
| mriedem | ^ in their office | 15:43 |
| superdan | hah yeah | 15:44 |
| *** tbachman has joined #openstack-nova | 15:47 | |
| mriedem | hmm, in the multi-cell job, the cpu config database is also pointing at cell0 | 15:48 |
| mriedem | http://logs.openstack.org/65/473565/4/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fcfedba/logs/etc/nova/nova-cpu.conf.txt.gz | 15:48 |
| mriedem | but it should go through conductor when the compute manager queries for the list of bdms right? | 15:48 |
| mriedem | and cell conductor should be using http://logs.openstack.org/65/473565/4/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fcfedba/logs/etc/nova/nova_cell1.conf.txt.gz ? | 15:48 |
| superdan | yeah | 15:48 |
| *** amotoki is now known as amotoki_away | 15:49 | |
| *** itlinux has joined #openstack-nova | 15:49 | |
| superdan | really we should unset the db connection for both main and api in that cpu config, | 15:49 |
| superdan | but the way I generate it is kinda weird | 15:49 |
| *** tbachman_ has joined #openstack-nova | 15:49 | |
| mriedem | the [database] one for sure, | 15:49 |
| mriedem | but the api_database one is needed for the retry and affinity crap | 15:50 |
| superdan | not in compute | 15:50 |
| mriedem | well, at least affinity | 15:50 |
| mriedem | sure it is, the compute manager pulls server groups from the api db right? | 15:50 |
| *** damien_r has quit IRC | 15:50 | |
| superdan | no, it would ask conductor like anything else | 15:51 |
| superdan | and conductor in cell1 is configured to not be able to find the api db: http://logs.openstack.org/65/473565/4/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fcfedba/logs/etc/nova/nova_cell1.conf.txt.gz | 15:51 |
| superdan | which means it can't retry in that configuration nor do the affinity check, | 15:51 |
| mriedem | so how does the affinity stuff not all blow up? | 15:51 |
| superdan | which is why that workaround config was added and the devstack patch sets it | 15:51 |
| mriedem | oh | 15:51 |
| *** tbachman has quit IRC | 15:52 | |
| *** tbachman_ is now known as tbachman | 15:52 | |
| superdan | mriedem: got a bug for the offload target thing yet? | 15:52 |
| mriedem | http://logs.openstack.org/65/473565/4/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/fcfedba/logs/etc/nova/nova_cell1.conf.txt.gz | 15:52 |
| mriedem | oops | 15:52 |
| mriedem | i meant, if not CONF.workarounds.disable_group_policy_check_upcall: | 15:52 |
| *** sridharg has quit IRC | 15:52 | |
| mriedem | ok | 15:52 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1702932 | 15:53 |
| openstack | Launchpad bug 1702932 in OpenStack Compute (nova) "Unshelving an offloaded server with volume attachments may not attach to the guest in multi-cell env" [Undecided,New] | 15:53 |
| mriedem | i'll crank out another one for the unshelve limit to same cell thing | 15:53 |
| superdan | thanks | 15:53 |
| *** yangyapeng has joined #openstack-nova | 15:54 | |
| *** klipka has quit IRC | 15:54 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary https://review.openstack.org/473565 | 15:54 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix targeting cell when attaching a volume to a shelved instance https://review.openstack.org/481683 | 15:54 |
| superdan | here's that first fix and updated canary ^ | 15:54 |
| *** Apoorva has joined #openstack-nova | 15:55 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1702935 | 15:56 |
| openstack | Launchpad bug 1702935 in OpenStack Compute (nova) "Unshelving an offloaded server does not restrict to the original cell" [Undecided,New] | 15:56 |
| *** lucasagomes is now known as lucas-afk | 15:58 | |
| *** gyee has joined #openstack-nova | 15:58 | |
| mriedem | cool https://review.openstack.org/#/c/481683/ is easier than i thought, but makes sense. still really wondering how this wasn't caught in tempest | 16:01 |
| *** ckopper has joined #openstack-nova | 16:04 | |
| *** jmlowe_ has quit IRC | 16:05 | |
| *** yamahata has joined #openstack-nova | 16:07 | |
| *** acormier has quit IRC | 16:08 | |
| mriedem | so normal attach volume works to create the volume bdm in the same cell as the instance because when we lookup the instance, set target the context | 16:12 |
| mriedem | and we create that bdm in the cell compute service | 16:13 |
| *** rajathagasthya has joined #openstack-nova | 16:14 | |
| *** thorst has quit IRC | 16:15 | |
| *** catinthe_ has quit IRC | 16:19 | |
| *** catintheroof has joined #openstack-nova | 16:19 | |
| *** mkucia has quit IRC | 16:21 | |
| *** READ10 has joined #openstack-nova | 16:21 | |
| *** sapcc-bot has quit IRC | 16:24 | |
| *** dfisher has joined #openstack-nova | 16:24 | |
| *** carthaca_ has quit IRC | 16:24 | |
| *** sapcc-bot has joined #openstack-nova | 16:24 | |
| *** carthaca_ has joined #openstack-nova | 16:24 | |
| *** iceyao has joined #openstack-nova | 16:29 | |
| mriedem | another one https://bugs.launchpad.net/nova/+bug/1702959 | 16:34 |
| openstack | Launchpad bug 1702959 in OpenStack Compute (nova) "MigrationNotFound in multi-cell setup doing server external events processing" [High,Triaged] | 16:34 |
| *** ociuhandu has quit IRC | 16:36 | |
| *** iceyao has quit IRC | 16:36 | |
| *** dtp has joined #openstack-nova | 16:36 | |
| mriedem | i'll take a crack at that one | 16:38 |
| *** jpena is now known as jpena|off | 16:38 | |
| dfisher | anybody know of attach/detach issues with current nova/cinder (via devstack) bits? I can't seem to get a simple volume to attach to a simple Ubuntu or Cirros VM | 16:38 |
| mriedem | no known issues | 16:39 |
| dfisher | hmm, ok. | 16:39 |
| dfisher | thanks | 16:39 |
| *** amotoki_away is now known as amotoki | 16:40 | |
| superdan | mriedem: is that one resolved by the patch that is up already? | 16:43 |
| superdan | mriedem: https://review.openstack.org/#/c/445142/ | 16:43 |
| clarkb | mriedem: http://logs.openstack.org/31/476631/18/gate/gate-nova-tox-functional-ubuntu-xenial/2cafb17/console.html#_2017-07-07_15_29_18_416631 known issue? that reset the gate not too long ago. Looks like test is attempting admin request and failing without admin rights | 16:46 |
| *** psachin has joined #openstack-nova | 16:47 | |
| *** amotoki is now known as amotoki_away | 16:49 | |
| mriedem | clarkb: i'm not aware of that one | 16:50 |
| mriedem | superdan: maybe given https://review.openstack.org/#/c/445142/10/nova/compute/api.py@4263 | 16:50 |
| mriedem | superdan: are you ok with me making the fleetify change depend on that patch as well? | 16:51 |
| superdan | mriedem: sure | 16:51 |
| *** jmlowe has joined #openstack-nova | 16:52 | |
| *** tbachman has quit IRC | 16:53 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381 | 16:56 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations https://review.openstack.org/325414 | 16:56 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add console connection object https://review.openstack.org/320063 | 16:56 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add access_url_base to console_auth_tokens table https://review.openstack.org/334614 | 16:56 |
| openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 16:56 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add console_auth_token_get() method to DB API https://review.openstack.org/481700 | 16:56 |
| mriedem | ok, fleeitify updated, will check out the api logs when that's done to see if the migration not found is gone | 16:56 |
| *** psachin has quit IRC | 16:57 | |
| *** efoley has quit IRC | 16:58 | |
| mriedem | clarkb: have you checked logstash? | 16:58 |
| *** READ10 has quit IRC | 16:59 | |
| mriedem | probably can't really | 17:00 |
| clarkb | mriedem: not yet trying to diagnose problems with my desktop :/ | 17:00 |
| mriedem | in general we've had very weird failures like that with the nova functional tests, | 17:00 |
| mriedem | something is making a request to the placement service and another is making a request to the compute api service, and the response goes the wrong way, | 17:01 |
| mriedem | assuming eventlet weirdness | 17:01 |
| *** ralonsoh has quit IRC | 17:03 | |
| mriedem | aha | 17:05 |
| mriedem | vdrok: found that weird sql connection thing in the fleetify patch | 17:05 |
| openstackgerrit | John Griffith proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 17:06 |
| mriedem | http://logs.openstack.org/56/477556/3/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/ee3e9b6/logs/subnode-2/screen-n-cpu.txt.gz#_Jul_05_20_44_19_705646 | 17:06 |
| mriedem | the subnode n-cpu is doing the server group affinity check, which blows up b/c the cell conductor doesn't have an api db connection, | 17:06 |
| mriedem | the nova-cpu.conf on the subnode is missing the config option to disable that upcall | 17:06 |
| *** sdague has quit IRC | 17:09 | |
| *** jmlowe has quit IRC | 17:10 | |
| vdrok | mriedem: cool, thanks! Do we need to set it in ironic plugin or it's going to be handled on Nova side? | 17:10 |
| mriedem | not sure, it needs to be handled in https://review.openstack.org/#/c/477556/4/lib/nova though | 17:10 |
| *** trungnv has quit IRC | 17:11 | |
| mriedem | i think this is where the nova-cpu.conf is used https://review.openstack.org/#/c/477556/4/lib/nova@818 | 17:11 |
| *** iceyao has joined #openstack-nova | 17:11 | |
| *** hieulq has quit IRC | 17:12 | |
| *** fragatina has joined #openstack-nova | 17:13 | |
| vdrok | mriedem: aha, ok. | 17:13 |
| mriedem | i don't know what nova.conf it's copying though, or where that is configured | 17:15 |
| *** Swami has joined #openstack-nova | 17:15 | |
| *** iceyao has quit IRC | 17:16 | |
| mriedem | ah i think i know where to set it | 17:17 |
| mriedem | if dan is ok with me touching his change | 17:17 |
| mriedem | superdan: i think i know how to fix the subnode nova.conf thing for the upcall in https://review.openstack.org/#/c/477556/ - ok with me doing that? | 17:19 |
| *** tbachman has joined #openstack-nova | 17:20 | |
| superdan | mriedem: not sure what you mean "fix the upcall" | 17:20 |
| * superdan just got off the phone | 17:21 | |
| mriedem | the subnode n-cpu conf isn't configured to disable the server group upcall | 17:21 |
| mriedem | i know where to fix it in the fleetify change | 17:21 |
| mriedem | asking permission | 17:21 |
| superdan | it's not? | 17:21 |
| mriedem | f no | 17:21 |
| superdan | ohh, subnode | 17:21 |
| superdan | gotcha | 17:21 |
| superdan | yes please | 17:21 |
| mriedem | aye aye | 17:21 |
| mriedem | is it weird when i can see the mail man out of the corner of my eye and look at him and he nods at me cross my yard to drop something off? | 17:22 |
| mriedem | should we have such a familiar relationship at this point? | 17:22 |
| mriedem | ok it's up https://review.openstack.org/#/c/477556/4..5/lib/nova | 17:23 |
| mriedem | and now it's taco time | 17:23 |
| *** dave-mccowan has joined #openstack-nova | 17:24 | |
| smcginnis | mriedem: But it's not Tuesday. | 17:24 |
| *** links has joined #openstack-nova | 17:25 | |
| *** jmlowe has joined #openstack-nova | 17:25 | |
| *** trungnv has joined #openstack-nova | 17:26 | |
| *** hieulq has joined #openstack-nova | 17:26 | |
| *** baoli has quit IRC | 17:27 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | 17:28 |
| *** baoli has joined #openstack-nova | 17:28 | |
| *** links has quit IRC | 17:29 | |
| *** links has joined #openstack-nova | 17:30 | |
| *** sambetts is now known as sambetts|afk | 17:34 | |
| melwitt | unless I'm missing something, we have some bugs in nova/conductor/manager.py where RequestSpec objects are being passed to _set_vm_state_and_notify() that doesn't handle them | 17:34 |
| *** sudipto has quit IRC | 17:35 | |
| *** sudipto_ has quit IRC | 17:35 | |
| *** ociuhandu has joined #openstack-nova | 17:36 | |
| *** links has quit IRC | 17:36 | |
| *** slaweq has joined #openstack-nova | 17:37 | |
| *** gjayavelu has joined #openstack-nova | 17:39 | |
| melwitt | nevermind, I think I see where it's being converted to a dict before calling the notify | 17:39 |
| *** baoli has quit IRC | 17:40 | |
| *** amotoki_away is now known as amotoki | 17:40 | |
| *** baoli has joined #openstack-nova | 17:41 | |
| *** fragatina has quit IRC | 17:42 | |
| *** fragatina has joined #openstack-nova | 17:43 | |
| mriedem | melwitt: i need you to solve the riddle of why shelved offloaded volume attach wasn't failing in the multi-cell setup | 17:43 |
| mriedem | only you are capable of cracking that nut | 17:43 |
| melwitt | now that you've said that, you'll probably figure it out within a few minutes :) | 17:44 |
| *** bnemec is now known as beekneemech | 17:45 | |
| mriedem | no i gave up | 17:45 |
| melwitt | dangit | 17:46 |
| *** fragatina has quit IRC | 17:47 | |
| *** slaweq has quit IRC | 17:49 | |
| *** yamahata has quit IRC | 17:49 | |
| *** thorst has joined #openstack-nova | 17:49 | |
| *** slaweq has joined #openstack-nova | 17:50 | |
| *** amotoki is now known as amotoki_away | 17:50 | |
| *** kaisers_ has quit IRC | 17:50 | |
| *** kaisers_ has joined #openstack-nova | 17:51 | |
| *** fragatina has joined #openstack-nova | 17:51 | |
| *** fragatina has quit IRC | 17:51 | |
| melwitt | I keep reading _BaseTaskTestCase as BasketCase | 17:52 |
| *** iceyao has joined #openstack-nova | 17:52 | |
| *** fragatina has joined #openstack-nova | 17:52 | |
| *** thorst has quit IRC | 17:52 | |
| *** thorst has joined #openstack-nova | 17:52 | |
| figleaf | do you have the time to listen to me whine? | 17:52 |
| *** iceyao has quit IRC | 17:57 | |
| * superdan wonders who that was intended to be a /msg for | 17:59 | |
| *** gjayavelu has quit IRC | 17:59 | |
| melwitt | it was for the channel. important stuff | 18:01 |
| figleaf | for all the basket cases in the channel | 18:01 |
| *** kaisers_ has quit IRC | 18:01 | |
| figleaf | superdan: https://www.youtube.com/watch?v=NUTGr5t3MoY | 18:02 |
| *** gjayavelu has joined #openstack-nova | 18:02 | |
| superdan | ah, dang, I didn't make the connection | 18:02 |
| *** stvnoyes has joined #openstack-nova | 18:06 | |
| *** lpetrut has quit IRC | 18:08 | |
| *** dtantsur|afk has quit IRC | 18:10 | |
| *** dtantsur has joined #openstack-nova | 18:13 | |
| *** tesseract has quit IRC | 18:21 | |
| mriedem | leakypipes: is this complete now? https://blueprints.launchpad.net/nova/+spec/shared-resources-pike | 18:23 |
| mriedem | there are no open changes, and the allocation request candidate stuff handles RPs associated via aggregate | 18:24 |
| *** yamahata has joined #openstack-nova | 18:24 | |
| leakypipes | mriedem: yeah. #4 on that list is the only one not done, and the allocation candidates will handle that (not in the scheduler reporting client, but rather int he scheduler during claims) | 18:24 |
| mriedem | leakypipes: ok cool i'm going to close it out then | 18:25 |
| mriedem | thanks | 18:25 |
| leakypipes | kk | 18:25 |
| mriedem | fried_rice: same with https://blueprints.launchpad.net/nova/+spec/powervm-nova-compute-driver - i think i'm going to close that out | 18:26 |
| mriedem | i don't see anything critical in there that we need to get in for pike | 18:26 |
| fried_rice | mriedem Critical, no. Some nice-to-haves. But closing the bp shouldn't preclude further changes in pike, right? | 18:27 |
| *** itlinux has quit IRC | 18:27 | |
| mriedem | closing it means i'm not going to look at anything proposed for it | 18:29 |
| mriedem | in pike | 18:29 |
| mriedem | because i need to narrow my focus on what still needs to get done | 18:29 |
| *** pchavva has quit IRC | 18:31 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631 | 18:31 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: claim resources in scheduler https://review.openstack.org/476632 | 18:31 |
| leakypipes | mriedem, superdan: ^^ | 18:32 |
| leakypipes | definitely should add some functional testing for that. | 18:32 |
| leakypipes | but it's good enough to get reviews on IMO. | 18:32 |
| superdan | did you just kick that out of the gate with that push? | 18:34 |
| superdan | yup | 18:34 |
| mriedem | git review -R | 18:35 |
| mriedem | ftw | 18:35 |
| superdan | yeah | 18:35 |
| *** bit_lySLH2uSZHed has joined #openstack-nova | 18:36 | |
| leakypipes | mriedem, superdan: yeah, sorry about that :( | 18:36 |
| leakypipes | figleaf had a worry about the previous patch, so perhaps you could see his review on the previous revision of https://review.openstack.org/476632 | 18:37 |
| *** bit_lySLH2uSZHed has quit IRC | 18:37 | |
| clarkb | fwiw git review -R shouldn't really be necessary anymore. You should only get a rebase from git review if it hits a merge conflict | 18:39 |
| clarkb | in which case you hvae to push a rebase at some point | 18:39 |
| *** amotoki_away is now known as amotoki | 18:41 | |
| superdan | clarkb: gerrit identified the bottom as a rebase only. | 18:42 |
| superdan | so I don't think that is accurate for this case | 18:42 |
| leakypipes | superdan: I rebased to master explicitly (by mistake) | 18:43 |
| figleaf | leakypipes: did that comment make any sense? | 18:43 |
| superdan | leakypipes: right I know | 18:43 |
| leakypipes | figleaf: well, I tried to answer as best I could. did you see my response? | 18:43 |
| figleaf | just looking... | 18:43 |
| leakypipes | guys, I need to run out for a couple hours... I will be back later on. sorry! | 18:44 |
| figleaf | leakypipes: np - I have to run out now too | 18:44 |
| figleaf | leakypipes: but I'm still confused as to how this is going to work w/o returning alloc candidates | 18:44 |
| *** kaisers_ has joined #openstack-nova | 18:47 | |
| *** itlinux has joined #openstack-nova | 18:47 | |
| *** slaweq has quit IRC | 18:49 | |
| *** slaweq has joined #openstack-nova | 18:49 | |
| *** amotoki is now known as amotoki_away | 18:51 | |
| *** slaweq has quit IRC | 18:54 | |
| *** lpetrut has joined #openstack-nova | 18:54 | |
| *** Sukhdev has joined #openstack-nova | 18:58 | |
| mriedem | superdan: booya fleetify is passing the multinode job http://logs.openstack.org/56/477556/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/aed9486/ | 19:01 |
| superdan | schwing | 19:01 |
| mriedem | and the migration not found errors are gone | 19:02 |
| mriedem | subnode nova-cpu.conf also has the disable flag http://logs.openstack.org/56/477556/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/aed9486/logs/subnode-2/etc/nova/nova-cpu.conf.txt.gz | 19:02 |
| *** bit_lySLH2uSZHed has joined #openstack-nova | 19:04 | |
| *** bit_lySLH2uSZHed has left #openstack-nova | 19:05 | |
| *** jamesden_ has joined #openstack-nova | 19:06 | |
| *** jamesdenton has quit IRC | 19:07 | |
| mriedem | jgriffith: clarified what i meant in https://review.openstack.org/#/c/330285/88/nova/compute/api.py@3901 | 19:11 |
| openstackgerrit | Sumedh Degaonkar proposed openstack/nova master: Adds 'sata' as a valid disk bus for qemu and kvm hypervisors https://review.openstack.org/459794 | 19:12 |
| jgriffith | mriedem thanks, once they get network back up in the lab I'll go back through and fix the unit tests and pep8 I appear to have broken | 19:12 |
| jgriffith | also will see if I can figure out what broke after the rebase | 19:12 |
| jgriffith | mriedem oh... looks like you didn't see the update | 19:13 |
| jgriffith | mriedem https://review.openstack.org/#/c/330285/89/nova/compute/api.py@3901 | 19:13 |
| mriedem | ha | 19:14 |
| jgriffith | :) | 19:14 |
| mriedem | volume_id doesn't exist | 19:14 |
| mriedem | volume['id'] is what you need | 19:14 |
| jgriffith | awww crap!!! | 19:14 |
| mriedem | otherwise looks good | 19:14 |
| openstackgerrit | Sumedh Degaonkar proposed openstack/nova master: Adds 'sata' as a valid disk bus for qemu and kvm hypervisors https://review.openstack.org/459794 | 19:14 |
| mriedem | let me pad some stats here | 19:14 |
| jgriffith | mriedem thanks, knew I was screwed when I couldn't test | 19:15 |
| mriedem | figleaf: we have some other changes to do in https://blueprints.launchpad.net/nova/+spec/custom-resource-classes-in-flavors right? there is some data migration thing that is supposed to happen on startup of the nova-compute service with ironic, so it can retrofit older flavors | 19:16 |
| mriedem | ah yeah this https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/custom-resource-classes-in-flavors.html#data-model-impact | 19:18 |
| mriedem | superdan: in ^ should that say "will not be schedulable" ? | 19:19 |
| superdan | mriedem: I think it's saying that until you account for the new resource types, | 19:20 |
| superdan | nova will think it could schedule instances to nodes that are already used, because they haven't yet reported that the instance is consuming one of the new resources | 19:21 |
| mriedem | ah | 19:21 |
| superdan | you buy that? | 19:21 |
| *** kaisers_ has left #openstack-nova | 19:21 | |
| mriedem | so CUSTOM_BAREMETAL_INFINIBAND_GOLD is used but not reported as used | 19:21 |
| superdan | yar | 19:22 |
| mriedem | ok, | 19:22 |
| mriedem | so, | 19:22 |
| mriedem | if we move the claim/allocation stuff to the scheduler, can we still do this data migration sync up on nova-compute start? | 19:22 |
| mriedem | i mean, we can, we have the scheduler report client, | 19:22 |
| mriedem | and the update_available_resource periodic, | 19:23 |
| mriedem | just wondering if that will handle it | 19:23 |
| superdan | mriedem: the compute node also isn't going to report any GOLD resources available until it starts up on new code either | 19:23 |
| superdan | so I think if you restart onto the new scheduler, | 19:23 |
| *** ckopper has quit IRC | 19:24 | |
| superdan | we'll start asking for resources that expose GOLD once the flavors are updated, but no computes will have any GOLDs to give until they are restarted and start exposing that | 19:24 |
| superdan | wait | 19:24 |
| superdan | did we already do that part in ocata? | 19:24 |
| superdan | yeah, okay yeah | 19:24 |
| superdan | I think I've got this now | 19:24 |
| superdan | lemme try again: | 19:24 |
| superdan | In ocata, we got the ironic driver to expose 1 $ironic_node_resource_class for each node to placement | 19:24 |
| superdan | if we update the real flavors with the new resource type, | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700 | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699 | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 19:25 |
| superdan | scheduler would send instances to busy computes thinking there was room | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 19:25 |
| superdan | what we need, | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 19:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: Limit maximum pending resizes for preventing over quota https://review.openstack.org/481739 | 19:25 |
| superdan | is to data migrate the _instance_ flavors to also report that they use one of the new resources, | 19:25 |
| superdan | so that the computes start reporting 1 total, 1 used, 0 available | 19:25 |
| superdan | so really, | 19:26 |
| superdan | ops just need to run data migrations before they update the main flavors to use the override I think | 19:26 |
| superdan | make sense/ | 19:26 |
| mriedem | yeah, that's where i was going in my head, | 19:27 |
| mriedem | that the compute updates the existing instance embedded flavor on startup so we can fix the allocations reported in, | 19:27 |
| superdan | right | 19:28 |
| mriedem | and it's good that none of it turns on until you fiddle the flavors | 19:28 |
| superdan | yeah | 19:28 |
| mriedem | although, we don't actually pull the custom resource off the instance.flavor in the compute, we pull it off the node | 19:28 |
| mriedem | so i'm a bit fuzzy on where the existing instance.flavor comes into play | 19:29 |
| mriedem | this https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L658 | 19:29 |
| superdan | we report inventory based on the node | 19:29 |
| superdan | we report allocations based on the instance/flavor | 19:29 |
| superdan | and we request from placement based on the real flavor | 19:30 |
| mriedem | ok so RT updates allocations here https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L1012 | 19:30 |
| *** dfisher has left #openstack-nova | 19:31 | |
| mriedem | and this is where we get the allocations off the instance right? https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L141 | 19:31 |
| mriedem | which is only ram, cpu and disk | 19:32 |
| mriedem | not fancy pants ironic stuff | 19:32 |
| *** smatzek has quit IRC | 19:32 | |
| superdan | yeah | 19:32 |
| superdan | we kinda need the ironic driver to do the instance flavor migration though I think | 19:32 |
| mriedem | sure, but there are 2 parts to this if i'm understanding, | 19:32 |
| superdan | because it'd be hard to do that from nova-manage, not being 100% sure the hypervisor type of an instance I think | 19:33 |
| mriedem | 1. ironic driver updates the instance.flavor if it doesn't have the custome rresource class but the node.resource_class is set | 19:33 |
| mriedem | 2. scheduler report client needs to account for custom resource class allocations off the instance flavor | 19:33 |
| mriedem | then 3. deployer fiddles the bits | 19:33 |
| mriedem | 4. cleanup | 19:33 |
| *** Apoorva has quit IRC | 19:34 | |
| mriedem | i think #2 is missing from the spec, unless i'm missing something | 19:34 |
| superdan | yeah I suppose you're right | 19:34 |
| *** yamamoto has quit IRC | 19:34 | |
| superdan | it's implied | 19:34 |
| mriedem | heh | 19:35 |
| *** jjung has joined #openstack-nova | 19:35 | |
| mriedem | mayhap i'll update the spec quick | 19:35 |
| melwitt | superdan: can you tell where I would need to put an "objects.register_all()" to call object classmethods in the websocketproxy? would it have to be here or is there a way it could only be done once? https://github.com/openstack/nova/blob/master/nova/console/websocketproxy.py#L71 | 19:38 |
| *** jjung has quit IRC | 19:39 | |
| superdan | melwitt: you could do it at module-level.. it is only needed once per process | 19:39 |
| melwitt | superdan: okay, cool. thanks | 19:40 |
| superdan | you wouldn't want to do it per-request like that | 19:40 |
| melwitt | yeah, that's what I was thinking | 19:41 |
| *** amotoki_away is now known as amotoki | 19:42 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990 | 19:42 |
| *** awaugama has quit IRC | 19:44 | |
| *** dtantsur has quit IRC | 19:45 | |
| *** sdague has joined #openstack-nova | 19:46 | |
| *** slaweq has joined #openstack-nova | 19:49 | |
| *** slaweq_ has joined #openstack-nova | 19:50 | |
| *** jamesden_ has quit IRC | 19:50 | |
| *** dtantsur has joined #openstack-nova | 19:50 | |
| *** amotoki is now known as amotoki_away | 19:52 | |
| *** slaweq has quit IRC | 19:53 | |
| *** esberglu has quit IRC | 19:53 | |
| *** esberglu has joined #openstack-nova | 19:54 | |
| mriedem | superdan: oh i guess that allocations thing was mentioned in the deps section https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/custom-resource-classes-in-flavors.html#dependencies | 19:55 |
| mriedem | maybe that was supposed to or expected to happen in http://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/custom-resource-classes.html | 19:55 |
| superdan | mriedem: that part is done already | 19:55 |
| mriedem | "This depends on work to change the resource tracker to allow drivers to expose allocations of custom resource classes" | 19:55 |
| superdan | the migration of existing instances is different from what is described there, and is specific to ironic | 19:55 |
| superdan | right that depends-on is done | 19:55 |
| mriedem | we don't expose allocations for custom resource classes | 19:55 |
| mriedem | we expose inventory for custom resource classess | 19:56 |
| superdan | we can from the driver, hang on | 19:56 |
| *** iceyao has joined #openstack-nova | 19:56 | |
| superdan | oh | 19:56 |
| mriedem | that's what i was talking about in https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L141 | 19:56 |
| *** catinthe_ has joined #openstack-nova | 19:56 | |
| superdan | yeah okay | 19:56 |
| superdan | I was glossing over the allocation word there | 19:56 |
| *** dtantsur has quit IRC | 19:57 | |
| superdan | there's still the migration work that needs doing which is neither of these | 19:57 |
| *** tjones has joined #openstack-nova | 19:57 | |
| mriedem | yeah i think i've covered that here in this spec update, hold on | 19:57 |
| *** tjones has quit IRC | 19:58 | |
| superdan | cool, I gotsta jump on a call | 19:58 |
| *** tjones has joined #openstack-nova | 19:58 | |
| *** esberglu has quit IRC | 19:58 | |
| *** catintheroof has quit IRC | 19:59 | |
| mriedem | geez another one | 19:59 |
| mriedem | these aren't 900 numbers are they? | 19:59 |
| superdan | I have a problem | 19:59 |
| *** jmlowe has quit IRC | 20:00 | |
| *** iceyao has quit IRC | 20:00 | |
| *** dtantsur has joined #openstack-nova | 20:02 | |
| *** jamesdenton has joined #openstack-nova | 20:05 | |
| *** liverpooler has quit IRC | 20:09 | |
| melwitt | grr, internet is on the fritz again | 20:11 |
| *** jamesdenton has quit IRC | 20:11 | |
| *** jamesdenton has joined #openstack-nova | 20:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Amend spec for "Allow custom resource classes in flavor extra specs" https://review.openstack.org/481748 | 20:15 |
| mriedem | superdan: leakypipes: figleaf: ^ | 20:15 |
| melwitt | mriedem: I think that tempest test works because the common.get_instance() method gets the instance via compute/api and that does a set_target_cell() that targets the context to the cell the instance is in | 20:15 |
| mriedem | melwitt: so the same reason that normal attach works | 20:15 |
| mriedem | so even though the instance isn't on a host, | 20:15 |
| mriedem | it's still mapped to the original cell | 20:15 |
| mriedem | tricksy baggins | 20:16 |
| mriedem | melwitt: i told you you were the only one that could figure it out | 20:16 |
| * mriedem goes to +2 dan's fix then | 20:16 | |
| * melwitt wipes nails on lapel | 20:16 | |
| mriedem | ha | 20:16 |
| mriedem | mayhap you'd like to +W https://review.openstack.org/#/c/481683/ | 20:17 |
| *** esberglu has joined #openstack-nova | 20:18 | |
| mriedem | actually, | 20:18 |
| mriedem | so maybe this means we don't need that fix | 20:18 |
| mriedem | yeah, it was'nt failing b/c the context is already targeted | 20:19 |
| melwitt | mriedem: yeah I was wondering, what broke that brought up the bug? | 20:19 |
| mriedem | nothing :( | 20:20 |
| mriedem | i noticed the local creation part during a review of something unrelated, | 20:20 |
| mriedem | and because of how local delete has bit us with cells, | 20:20 |
| melwitt | ah, okay | 20:20 |
| mriedem | i immediately thought this was also busted in a similar way | 20:20 |
| melwitt | yeah, the set_target_cell is so hidden that it's hard to tell when we need an extra target cell or not | 20:21 |
| *** burgerk_ has joined #openstack-nova | 20:22 | |
| *** burgerk has quit IRC | 20:25 | |
| *** markvoelker has quit IRC | 20:27 | |
| *** markvoelker has joined #openstack-nova | 20:27 | |
| *** burgerk_ has quit IRC | 20:28 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Fix cropping the endpoint url https://review.openstack.org/479818 | 20:33 |
| *** Apoorva has joined #openstack-nova | 20:35 | |
| *** yamamoto has joined #openstack-nova | 20:35 | |
| *** iceyao has joined #openstack-nova | 20:37 | |
| *** ijw has joined #openstack-nova | 20:38 | |
| *** yamamoto has quit IRC | 20:39 | |
| *** catinthe_ has quit IRC | 20:39 | |
| *** Apoorva has quit IRC | 20:39 | |
| *** mriedem1 has joined #openstack-nova | 20:40 | |
| *** iceyao has quit IRC | 20:41 | |
| *** abalutoiu has joined #openstack-nova | 20:41 | |
| *** sambetts|afk has quit IRC | 20:42 | |
| *** priteau has quit IRC | 20:42 | |
| *** mriedem has quit IRC | 20:42 | |
| *** abalutoiu_ has joined #openstack-nova | 20:43 | |
| *** amotoki_away is now known as amotoki | 20:43 | |
| *** sambetts has joined #openstack-nova | 20:43 | |
| *** Apoorva has joined #openstack-nova | 20:45 | |
| *** ijw has quit IRC | 20:45 | |
| *** abalutoiu has quit IRC | 20:46 | |
| *** ijw has joined #openstack-nova | 20:46 | |
| *** owalsh has quit IRC | 20:47 | |
| *** abalutoiu has joined #openstack-nova | 20:48 | |
| *** abalutoiu_ has quit IRC | 20:49 | |
| *** ckopper has joined #openstack-nova | 20:50 | |
| *** amotoki is now known as amotoki_away | 20:52 | |
| superdan | mriedem1: ah yeah, so we probably don't need that | 20:53 |
| superdan | mriedem1: hence my asking where/how this came hup | 20:53 |
| superdan | up even | 20:53 |
| *** abalutoiu has quit IRC | 20:54 | |
| mriedem1 | sorry | 20:55 |
| *** abalutoiu has joined #openstack-nova | 20:55 | |
| *** mriedem1 is now known as mriedem | 20:55 | |
| mriedem | the local create thing scared me because of local delete issues | 20:55 |
| superdan | hah | 20:55 |
| mriedem | leakypipes: i went through https://review.openstack.org/#/c/476632/ - we can't actually allocate in the scheduler until we've done the changes in the compute to not claim if the allocations are already done, right? | 20:55 |
| mriedem | otheriwse we allocate in scheduler and still claim in compute | 20:56 |
| superdan | mriedem: we already have that right? | 20:56 |
| mriedem | do we? | 20:56 |
| superdan | mriedem: compute just checks the allocations in placement, changing them only if need be | 20:56 |
| mriedem | but i thought the big win was not doing the claim at all | 20:56 |
| mriedem | in the resourc etracker | 20:56 |
| superdan | we can't drop the RT until this is in place | 20:56 |
| mriedem | plus, | 20:56 |
| mriedem | there should be a minimum compute version check somewhere i thought, | 20:57 |
| mriedem | we had the whole back and forth about sending the actual allocations down to the compute over rpc | 20:57 |
| *** abalutoiu_ has joined #openstack-nova | 20:57 | |
| superdan | mriedem: I dunno, ocata computes did the allocation stuff right? | 20:57 |
| mriedem | to tell the compute if it should claim | 20:57 |
| *** ijw has quit IRC | 20:57 | |
| superdan | that's to tell it if it should forego the normal RT claiming right? | 20:57 |
| mriedem | and i thought we settled on not sending hte allocations claimed over rpc, but instead just have the compute check to see if the allocations were already done for it | 20:57 |
| mriedem | and by pass the claim (noop) | 20:57 |
| superdan | yeah | 20:58 |
| superdan | here's the short and curly of it: | 20:59 |
| superdan | there is BBQ hurling its way towards me at about 60MPH, so I stop caring about this when it gets here | 20:59 |
| *** ijw has joined #openstack-nova | 20:59 | |
| *** abalutoiu has quit IRC | 21:00 | |
| mriedem | at 2pm? | 21:00 |
| superdan | hell yes.. taylor gets half day fridays in the summer | 21:00 |
| *** owalsh has joined #openstack-nova | 21:01 | |
| mriedem | so https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/placement-claims.html#other-deployer-impact | 21:01 |
| mriedem | and https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/placement-claims.html#work-items | 21:01 |
| mriedem | "Make compute nodes GET /allocations/<instance_id> for verifying if already created, and if so, don’t POST allocations to Placement service." | 21:01 |
| mriedem | that's what i'm thinking of | 21:01 |
| mriedem | we can sort it out on monday when everyone is actually around | 21:02 |
| mriedem | including his bauzasness | 21:02 |
| superdan | https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L859-L877 | 21:02 |
| superdan | L868 stops if they're equal | 21:02 |
| mriedem | yeah i know about that, | 21:03 |
| *** ckopper has quit IRC | 21:03 | |
| superdan | bypassing the existing RT still needs to happen | 21:03 |
| mriedem | but we're still doing all of this other claim checking stuff in the compute that seems unnecessary - although there are going to be things the compute side claim will pick up that we don't in the scheduler, like pci/numa and overhead | 21:03 |
| superdan | yep | 21:03 |
| mriedem | i wasn't sure what we were going to do about the nfv resources | 21:04 |
| mriedem | and i thought in boston for overhead we said, well, you adjust the reserved amount configs to buffer your inventory | 21:04 |
| superdan | we said we'd do the simple thing first, expecting the op to reserve enough, | 21:04 |
| *** ijw has quit IRC | 21:04 | |
| superdan | potentially updating reserved automatically after that yeah | 21:05 |
| mriedem | i'll just go back to my corner with https://review.openstack.org/#/c/471899/ where things make sense | 21:06 |
| *** abalutoiu has joined #openstack-nova | 21:07 | |
| *** yamamoto has joined #openstack-nova | 21:08 | |
| *** smatzek has joined #openstack-nova | 21:08 | |
| *** abalutoiu_ has quit IRC | 21:09 | |
| *** thorst has quit IRC | 21:16 | |
| *** yamamoto has quit IRC | 21:17 | |
| *** iceyao has joined #openstack-nova | 21:18 | |
| *** owalsh has quit IRC | 21:19 | |
| *** abalutoiu has quit IRC | 21:21 | |
| *** owalsh has joined #openstack-nova | 21:21 | |
| *** acormier has joined #openstack-nova | 21:22 | |
| *** iceyao has quit IRC | 21:22 | |
| *** acormier has quit IRC | 21:24 | |
| *** itlinux has quit IRC | 21:26 | |
| Apoorva | Our Cinder CI is failing with following error http://paste.openstack.org/show/614811/. Is this a known issue? | 21:30 |
| *** catintheroof has joined #openstack-nova | 21:33 | |
| *** edmondsw has quit IRC | 21:34 | |
| *** lpetrut has quit IRC | 21:34 | |
| *** thorst has joined #openstack-nova | 21:38 | |
| *** thorst has quit IRC | 21:38 | |
| *** dave-mccowan has quit IRC | 21:41 | |
| *** lyan has quit IRC | 21:42 | |
| *** amotoki_away is now known as amotoki | 21:43 | |
| sgordon | so i am looking at this thread http://lists.openstack.org/pipermail/openstack/2017-July/thread.html#45139 | 21:44 |
| sgordon | "[Openstack] Live Migration and LibVirt CPU Mode" | 21:44 |
| sgordon | and i could swear there was a legit bug around this at some point but am struggling to pin down where it was discussed/fixed | 21:44 |
| *** yamamoto has joined #openstack-nova | 21:45 | |
| sgordon | specifically "Therefore it appears if a VM XML CPU definition contains "cpu mode='host-model'" the cpu model is ignored and the 2 host CPU models are compared." | 21:45 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | 21:45 |
| *** openstackgerrit has quit IRC | 21:47 | |
| *** smatzek has quit IRC | 21:49 | |
| sgordon | ring any bells for anyone? | 21:50 |
| *** thorst has joined #openstack-nova | 21:50 | |
| *** yamamoto has quit IRC | 21:51 | |
| *** ijw has joined #openstack-nova | 21:51 | |
| superdan | sgordon: I think maybe kashyap was discussing that with someone at some point? | 21:51 |
| *** priteau has joined #openstack-nova | 21:52 | |
| *** esberglu has quit IRC | 21:52 | |
| *** esberglu has joined #openstack-nova | 21:52 | |
| sgordon | superdan, yeah - it definitely sounds familiar i just cant remember the outcome | 21:53 |
| superdan | yeah that's about all I know | 21:53 |
| *** amotoki is now known as amotoki_away | 21:53 | |
| *** gjayavelu has quit IRC | 21:56 | |
| cburgess | superdan sgordon That sounds about right. What is the specific question? | 21:56 |
| *** priteau has quit IRC | 21:56 | |
| *** openstackstatus has quit IRC | 21:56 | |
| *** openstack has joined #openstack-nova | 21:58 | |
| cburgess | if so then... working as intended. | 21:58 |
| sgordon | yeah but host model is not really host model | 21:58 |
| sgordon | it's an approximation of the host model | 21:59 |
| cburgess | Right | 21:59 |
| *** iceyao has joined #openstack-nova | 21:59 | |
| sgordon | if you go downthread i think they are saying they get the same when using custom and specifying westmere | 21:59 |
| cburgess | Let me keep reading... | 22:00 |
| cburgess | I wonder if he has different versions of libvirt/kvm between the hosts. | 22:02 |
| sgordon | possibly | 22:02 |
| cburgess | "I'm beginning to suspect that because the VM has "cpu mode='host-model'" in its xml CPU definition," | 22:02 |
| sgordon | there is not much/any info about what versions of anything are in use | 22:02 |
| cburgess | As I said.. working as intended. | 22:02 |
| sgordon | yeah - in that email they contradict themselves | 22:03 |
| cburgess | If its using host-model.. you are screwed. | 22:03 |
| cburgess | Period.. | 22:03 |
| sgordon | say it's not working with custom, then say it is | 22:03 |
| sgordon | right | 22:03 |
| cburgess | Right.. | 22:03 |
| cburgess | This message... | 22:03 |
| cburgess | http://lists.openstack.org/pipermail/openstack/2017-July/045145.html | 22:03 |
| sgordon | and there is no way to get out of that without rebooting instances | 22:03 |
| cburgess | basically describes thats in the XML | 22:03 |
| cburgess | and based upon that XML.. what he is seeing is whats expected. | 22:03 |
| cburgess | Correct. | 22:04 |
| *** iceyao has quit IRC | 22:04 | |
| cburgess | He could disable the CPU checks but... he runs the risk of the VM crashing on live migration because it sees the flags change. | 22:04 |
| cburgess | I know he doesn't want to reboot but once you are using host-model there is absolutely no safe/guaranteed way to got from new to old. | 22:05 |
| sgordon | right | 22:05 |
| cburgess | So I feel for them.. but there is no solution. They could comment the checks out of nova if they wanted to but even that is a just rolling the device on the VMs crashing on migration. | 22:06 |
| cburgess | Or libvirt just refusing to do it. | 22:06 |
| *** felipemonteiro__ has quit IRC | 22:07 | |
| sgordon | i dont think that helps them anyway | 22:07 |
| sgordon | because it's passed the nova checks | 22:07 |
| sgordon | it's the libvirt checks kicking it out | 22:07 |
| cburgess | Oh yeah.. well.. like I said... | 22:07 |
| sgordon | thanks for being a sounding board, clearly i have friday afternoon brain | 22:08 |
| sgordon | i will try and frame this in an email | 22:08 |
| *** claudiub has quit IRC | 22:08 | |
| *** zzzeek_ has joined #openstack-nova | 22:08 | |
| *** zzzeek_ has quit IRC | 22:09 | |
| cburgess | sgordon No worries, we dealt with this very issue a lot in the essex/folsom days. Trying to figure out exacly what CPU type and model info to use to guarnatee the best mix of performance and portability. | 22:10 |
| *** figleaf is now known as edleafe | 22:10 | |
| *** jwcroppe has quit IRC | 22:11 | |
| *** burt has quit IRC | 22:11 | |
| *** jwcroppe has joined #openstack-nova | 22:11 | |
| sgordon | cburgess, yeah - it's basically impossible to set defaults this because any given selection is a tradeoff | 22:12 |
| sgordon | cburgess, depends on the characteristics and requirements of each environment | 22:12 |
| cburgess | sgordon I think the key bit to explains is that all the config values are basically useless once you get down to the libvirt internal code. It will *always* use whats defined in the running VMs XML. | 22:12 |
| sgordon | (i mean obviously we *do* set defaults for this, it's just that they arent going to fit for everyone) | 22:12 |
| cburgess | sgordon Exactly. So we have to play around with a sane "default" for our product and then enable customers to change it for their deployment based upon their needs. | 22:13 |
| cburgess | Though to be fair most use our default since its designed around maximum portability and most of customers can't have downtime for their pets. :( | 22:13 |
| *** zzzeek_ has joined #openstack-nova | 22:14 | |
| *** zzzeek_ has quit IRC | 22:14 | |
| *** edleafe- has joined #openstack-nova | 22:14 | |
| *** zzzeek_ has joined #openstack-nova | 22:14 | |
| *** zzzeek_ has quit IRC | 22:15 | |
| *** jwcroppe has quit IRC | 22:15 | |
| *** edleafe has quit IRC | 22:16 | |
| sgordon | yeah the other aspect of course is someone may not notice when doing a greenfield deployment | 22:16 |
| sgordon | the problem comes when they expand later and add a new generation of hosts | 22:16 |
| cburgess | Yup. we didn't notice it until our first client did a server refresh after about a year. | 22:18 |
| *** zzzeek_ has joined #openstack-nova | 22:19 | |
| *** zzzeek_ has quit IRC | 22:24 | |
| *** jamesdenton has quit IRC | 22:24 | |
| *** jamesdenton has joined #openstack-nova | 22:25 | |
| *** edleafe- is now known as edleafe | 22:25 | |
| *** zzzeek_ has joined #openstack-nova | 22:29 | |
| *** jamesdenton has quit IRC | 22:29 | |
| *** zzzeek_ has quit IRC | 22:30 | |
| *** zzzeek_ has joined #openstack-nova | 22:31 | |
| *** jamesdenton has joined #openstack-nova | 22:31 | |
| *** zzzeek- has joined #openstack-nova | 22:33 | |
| *** zzzeek_ has quit IRC | 22:35 | |
| *** iceyao has joined #openstack-nova | 22:40 | |
| *** zzzeek- has quit IRC | 22:41 | |
| *** catintheroof has quit IRC | 22:42 | |
| *** leakypipes has quit IRC | 22:43 | |
| *** amotoki_away is now known as amotoki | 22:44 | |
| *** iceyao has quit IRC | 22:45 | |
| *** sdague has quit IRC | 22:47 | |
| *** slaweq_ has quit IRC | 22:47 | |
| *** slaweq has joined #openstack-nova | 22:48 | |
| *** acormier has joined #openstack-nova | 22:51 | |
| *** slaweq has quit IRC | 22:53 | |
| *** amotoki is now known as amotoki_away | 22:54 | |
| *** acormier has quit IRC | 22:54 | |
| *** acormier has joined #openstack-nova | 22:54 | |
| *** gbarros has joined #openstack-nova | 22:55 | |
| *** tjones has left #openstack-nova | 22:56 | |
| *** jmlowe has joined #openstack-nova | 23:07 | |
| *** openstackgerrit has joined #openstack-nova | 23:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuids in os-hypervisors API https://review.openstack.org/471899 | 23:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use uuid for id in os-services API https://review.openstack.org/464280 | 23:12 |
| *** zzzeek_ has joined #openstack-nova | 23:17 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuids in os-hypervisors API https://review.openstack.org/471899 | 23:19 |
| *** zzzeek_ has quit IRC | 23:21 | |
| *** zzzeek_ has joined #openstack-nova | 23:21 | |
| *** acormier has quit IRC | 23:22 | |
| *** abalutoiu has joined #openstack-nova | 23:23 | |
| mriedem | mgagne: you overwrote my changes and re-introduced a bug https://review.openstack.org/#/c/454322/29..31 :) | 23:24 |
| mriedem | i'll just push my version again | 23:24 |
| *** iceyao has joined #openstack-nova | 23:25 | |
| *** Apoorva has quit IRC | 23:26 | |
| *** acormier has joined #openstack-nova | 23:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | 23:26 |
| *** rajathagasthya has quit IRC | 23:26 | |
| *** iceyao has quit IRC | 23:29 | |
| *** gbarros has quit IRC | 23:34 | |
| *** chyka has quit IRC | 23:34 | |
| *** chyka has joined #openstack-nova | 23:35 | |
| mriedem | fixed, now i just need to flesh out the tests and docs for the os-instance-actions details changes in there, probably over the weekend | 23:35 |
| mriedem | the tempest/devstack test stack had a clean run last night though, which is awesome | 23:35 |
| *** gbarros has joined #openstack-nova | 23:37 | |
| *** chyka has quit IRC | 23:39 | |
| *** dtp has quit IRC | 23:40 | |
| *** iceyao has joined #openstack-nova | 23:44 | |
| *** abalutoiu has quit IRC | 23:45 | |
| *** amotoki_away is now known as amotoki | 23:45 | |
| *** abalutoiu has joined #openstack-nova | 23:47 | |
| *** Apoorva has joined #openstack-nova | 23:47 | |
| *** jmlowe has quit IRC | 23:48 | |
| *** abalutoiu has quit IRC | 23:48 | |
| *** iceyao has quit IRC | 23:49 | |
| *** abalutoiu has joined #openstack-nova | 23:49 | |
| *** acormier has quit IRC | 23:50 | |
| *** jmlowe has joined #openstack-nova | 23:50 | |
| *** ijw has quit IRC | 23:52 | |
| *** acormier has joined #openstack-nova | 23:52 | |
| *** abalutoiu_ has joined #openstack-nova | 23:52 | |
| *** ijw has joined #openstack-nova | 23:52 | |
| *** abalutoiu has quit IRC | 23:54 | |
| *** amotoki is now known as amotoki_away | 23:55 | |
| *** abalutoiu__ has joined #openstack-nova | 23:56 | |
| *** ijw has quit IRC | 23:57 | |
| *** abalutoiu has joined #openstack-nova | 23:57 | |
| *** thorst has joined #openstack-nova | 23:57 | |
| *** ijw has joined #openstack-nova | 23:58 | |
| *** Apoorva has quit IRC | 23:58 | |
| *** abalutoiu_ has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!