Monday, 2017-10-30

*** yangyapeng has quit IRC00:02
*** markvoelker has quit IRC00:08
*** thorst has joined #openstack-nova00:18
*** thorst has quit IRC00:20
*** tbachman_ has joined #openstack-nova00:22
*** tbachman has quit IRC00:22
*** tbachman_ is now known as tbachman00:22
*** rmcallis has quit IRC00:35
*** Jeffrey4l has quit IRC00:42
*** Jeffrey4l has joined #openstack-nova00:42
*** salv-orlando has joined #openstack-nova00:46
*** salv-orlando has quit IRC00:51
*** huanxie has joined #openstack-nova00:51
*** TuanLA has joined #openstack-nova00:54
*** hieulq has joined #openstack-nova01:02
*** artom has quit IRC01:04
*** markvoelker has joined #openstack-nova01:05
*** jichen has joined #openstack-nova01:07
*** catintheroof has joined #openstack-nova01:08
*** yangyapeng has joined #openstack-nova01:11
openstackgerritjichenjc proposed openstack/nova master: check query param for server groups function  https://review.openstack.org/50034701:12
*** phuongnh has joined #openstack-nova01:15
*** Shunli has joined #openstack-nova01:15
*** liusheng has quit IRC01:20
*** takashin has joined #openstack-nova01:20
*** trungnv has joined #openstack-nova01:20
*** catintheroof has quit IRC01:22
openstackgerritYikun Jiang proposed openstack/nova master: Add migration db and object pagination support.  https://review.openstack.org/51490401:26
*** TuanLA has quit IRC01:28
*** yangyapeng has quit IRC01:29
*** TuanLA has joined #openstack-nova01:29
*** yangyapeng has joined #openstack-nova01:29
*** markvoelker has quit IRC01:38
*** namnh has joined #openstack-nova01:38
*** zzzeek has quit IRC01:40
*** yangyape_ has joined #openstack-nova01:40
*** yangyapeng has quit IRC01:41
*** hieulq has quit IRC01:50
*** TuanLA has quit IRC01:50
*** phuongnh has quit IRC01:50
*** phuongnh has joined #openstack-nova01:51
*** TuanLA has joined #openstack-nova01:51
*** hieulq has joined #openstack-nova01:51
*** cheneydc has joined #openstack-nova01:53
*** itlinux has quit IRC02:10
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895502:10
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896402:11
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix an example in "Delete Assisted Volume Snapshot"  https://review.openstack.org/51412802:12
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing marker functions  https://review.openstack.org/51457902:12
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - Enable cold migration with target host  https://review.openstack.org/40670702:13
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751402:13
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix test_get_volume_config method  https://review.openstack.org/48946702:13
*** tetsuro has joined #openstack-nova02:14
openstackgerritTakashi NATSUME proposed openstack/nova master: Update document related to host aggregate  https://review.openstack.org/51449902:14
*** thorst has joined #openstack-nova02:21
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103502:22
*** phuongnh has quit IRC02:23
*** hieulq has quit IRC02:23
*** TuanLA has quit IRC02:23
*** namnh has quit IRC02:23
*** namnh has joined #openstack-nova02:24
*** TuanLA has joined #openstack-nova02:24
*** phuongnh has joined #openstack-nova02:24
*** hieulq has joined #openstack-nova02:24
*** thorst has quit IRC02:25
efriedalex_xu How about now?02:31
*** liusheng has joined #openstack-nova02:31
efriedI was thinking it might work if we did something more like:  SELECT ... FROM rp INNER JOIN (SELECT inventory & allocation stuff 1 LEFT JOIN (SELECT inventory & allocation stuff 2 LEFT JOIN ...))02:32
efriedThat first INNER JOIN would mean we only get a row if *something* on the right-hand side isn't NULL.02:33
*** markvoelker has joined #openstack-nova02:35
*** mriedem has quit IRC02:38
*** felipemonteiro_ has joined #openstack-nova02:44
*** salv-orlando has joined #openstack-nova02:47
*** litao__ has joined #openstack-nova02:49
*** Laurene has joined #openstack-nova02:49
openstackgerritYikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination  https://review.openstack.org/51616802:51
*** salv-orlando has quit IRC02:51
*** annp has joined #openstack-nova02:53
*** zsli_ has joined #openstack-nova02:57
*** Tom_ has joined #openstack-nova03:00
*** Shunli has quit IRC03:00
openstackgerritYikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination  https://review.openstack.org/51616803:02
*** gongysh has joined #openstack-nova03:06
*** Tom_ has quit IRC03:08
*** markvoelker has quit IRC03:09
*** tetsuro has quit IRC03:16
*** namnh has quit IRC03:18
*** phuongnh has quit IRC03:18
*** TuanLA has quit IRC03:18
openstackgerritYikun Jiang proposed openstack/nova master: Add migration db and object pagination support.  https://review.openstack.org/51490403:18
*** phuongnh has joined #openstack-nova03:19
*** TuanLA has joined #openstack-nova03:19
*** namnh has joined #openstack-nova03:19
*** felipemonteiro_ has quit IRC03:21
*** cheneydc_ has joined #openstack-nova03:25
*** cheneydc has quit IRC03:29
*** jichen has quit IRC03:38
*** tetsuro has joined #openstack-nova03:38
*** hieulq has quit IRC03:40
*** TuanLA has quit IRC03:40
*** TuanLA has joined #openstack-nova03:41
*** hieulq has joined #openstack-nova03:41
*** udesale has joined #openstack-nova03:45
*** rcernin has quit IRC03:47
*** rcernin_ has joined #openstack-nova03:47
*** dave-mccowan has quit IRC03:48
*** mdnadeem has joined #openstack-nova03:50
*** stvnoyes has quit IRC03:51
*** takashin has left #openstack-nova03:54
*** vks1 has joined #openstack-nova04:03
*** markvoelker has joined #openstack-nova04:06
*** phuongnh has quit IRC04:13
*** TuanLA has quit IRC04:13
*** namnh has quit IRC04:13
*** namnh has joined #openstack-nova04:14
*** phuongnh has joined #openstack-nova04:14
*** TuanLA has joined #openstack-nova04:14
*** Dinesh_Bhor has joined #openstack-nova04:14
*** thorst has joined #openstack-nova04:22
*** thorst has quit IRC04:27
*** huanxie has quit IRC04:27
*** janki has joined #openstack-nova04:35
*** cheneydc_ has quit IRC04:37
*** markvoelker has quit IRC04:39
alex_xuefried: sorry, I'm on PTO last friday04:41
*** gongysh has quit IRC04:43
*** cheneydc_ has joined #openstack-nova04:44
*** salv-orlando has joined #openstack-nova04:48
*** salv-orlando has quit IRC04:52
*** jaosorior has joined #openstack-nova04:54
*** huanxie has joined #openstack-nova04:58
*** udesale__ has joined #openstack-nova04:59
*** udesale has quit IRC04:59
*** brault has joined #openstack-nova05:11
*** bkopilov has joined #openstack-nova05:16
*** brault has quit IRC05:16
*** yangyape_ has quit IRC05:24
*** yangyapeng has joined #openstack-nova05:25
*** sridharg has joined #openstack-nova05:28
*** markvoelker has joined #openstack-nova05:37
*** gongysh has joined #openstack-nova05:41
*** ratailor has joined #openstack-nova05:45
*** trinaths has joined #openstack-nova05:46
*** zhurong has joined #openstack-nova05:48
*** rnoriega has quit IRC05:52
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896405:57
*** huanxie has quit IRC06:01
*** Kevin_Zheng has quit IRC06:01
*** lpetrut has joined #openstack-nova06:04
*** avolkov has joined #openstack-nova06:04
*** slaweq has quit IRC06:05
*** hoonetorg has quit IRC06:05
*** slaweq has joined #openstack-nova06:06
*** salv-orlando has joined #openstack-nova06:08
*** markvoelker has quit IRC06:10
*** slaweq has quit IRC06:10
*** hshiina has joined #openstack-nova06:11
*** takashin has joined #openstack-nova06:13
*** udesale__ has quit IRC06:14
openstackgerritYikun Jiang proposed openstack/nova master: Add migration db and object pagination support.  https://review.openstack.org/51490406:20
*** thorst has joined #openstack-nova06:23
*** hoonetorg has joined #openstack-nova06:23
*** krtaylor has quit IRC06:27
*** thorst has quit IRC06:27
*** hamzy has quit IRC06:28
*** brault has joined #openstack-nova06:32
*** cheneydc_ has quit IRC06:35
*** slaweq has joined #openstack-nova06:37
*** cheneydc_ has joined #openstack-nova06:38
*** hamzy has joined #openstack-nova06:42
*** krtaylor has joined #openstack-nova06:43
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - testing check-requirements  https://review.openstack.org/51619506:45
*** salv-orlando has quit IRC06:46
openstackgerritYikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination  https://review.openstack.org/51616806:46
*** salv-orlando has joined #openstack-nova06:48
*** lpetrut has quit IRC06:58
*** spectr has joined #openstack-nova07:02
openstackgerritJianghua Wang proposed openstack/nova master: vgpu: add enabled white list  https://review.openstack.org/51258007:04
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor  https://review.openstack.org/51296507:04
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - testing check-requirements  https://review.openstack.org/51619507:06
*** markvoelker has joined #openstack-nova07:07
*** Oku_OS-away is now known as Oku_OS07:08
*** brault has quit IRC07:08
*** lpetrut has joined #openstack-nova07:09
*** slaweq has quit IRC07:10
openstackgerritYikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination  https://review.openstack.org/51616807:14
*** slaweq has joined #openstack-nova07:16
*** rcernin_ has quit IRC07:17
*** Tom_ has joined #openstack-nova07:19
*** AlexeyAbashkin has joined #openstack-nova07:23
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - testing check-requirements  https://review.openstack.org/51619507:25
*** avolkov has quit IRC07:25
*** avolkov has joined #openstack-nova07:26
*** lajoskatona has joined #openstack-nova07:26
*** brault has joined #openstack-nova07:32
*** tesseract has joined #openstack-nova07:33
*** salv-orlando has quit IRC07:35
*** brault has quit IRC07:35
*** salv-orlando has joined #openstack-nova07:36
*** markvoelker has quit IRC07:40
*** cheneydc_ has quit IRC07:40
*** cheneydc_ has joined #openstack-nova07:40
*** aarefiev_afk is now known as aarefiev07:41
*** lpetrut has quit IRC07:45
openstackgerritYikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination  https://review.openstack.org/51616807:45
*** sahid has joined #openstack-nova07:48
*** rgerganov has joined #openstack-nova07:49
*** yamamoto has quit IRC07:55
*** pow_2_30 is now known as gibi07:57
*** salv-orlando has quit IRC07:59
*** salv-orlando has joined #openstack-nova08:00
gmannalex_xu: can you check this - https://review.openstack.org/#/c/516104/08:00
gmannalex_xu: matt BP about adding PUT description in flavor will be much easy after this08:01
*** Tom_ has quit IRC08:01
alex_xugmann: got it, I will check it before EOD08:01
*** Tom_ has joined #openstack-nova08:02
*** salv-orl_ has joined #openstack-nova08:02
gmannalex_xu: thanks08:02
alex_xugmann: np08:02
openstackgerritAlex Xu proposed openstack/nova master: WIP: Add _get_providers_with_any_resource method  https://review.openstack.org/51419708:03
openstackgerritAlex Xu proposed openstack/nova master: Enable _get_usages_by_provider_and_rc not filter on the resource class id  https://review.openstack.org/51464908:03
openstackgerritAlex Xu proposed openstack/nova master: Add _get_traits_by_rp_ids method  https://review.openstack.org/51118408:03
openstackgerritAlex Xu proposed openstack/nova master: Add ProviderSummaryList object  https://review.openstack.org/51419808:03
openstackgerritAlex Xu proposed openstack/nova master: Add more caches for the ProviderSummary and ProviderSummaryList  https://review.openstack.org/51419908:03
openstackgerritAlex Xu proposed openstack/nova master: Add AggregatedProviderSummary and AggregatedProviderSummaryList objects  https://review.openstack.org/51420008:03
openstackgerritAlex Xu proposed openstack/nova master: Add has_resources method to the AggregatedProviderSummary object  https://review.openstack.org/51465008:04
openstackgerritAlex Xu proposed openstack/nova master: Add as_allocation_request_obj method to AggregatedProvierSummary obj  https://review.openstack.org/51465108:04
openstackgerritAlex Xu proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters  https://review.openstack.org/47976608:04
*** salv-orlando has quit IRC08:04
*** gongysh has quit IRC08:05
*** Tom_ has quit IRC08:06
*** Tom_ has joined #openstack-nova08:11
*** vivsoni__ has quit IRC08:14
*** alexchadin has joined #openstack-nova08:15
*** kuzko has quit IRC08:17
*** kuzko has joined #openstack-nova08:20
*** slaweq has quit IRC08:22
*** rcernin has joined #openstack-nova08:22
*** thorst has joined #openstack-nova08:24
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - testing check-requirements  https://review.openstack.org/51619508:25
*** huanxie has joined #openstack-nova08:27
*** jpena|off is now known as jpena08:27
*** udesale has joined #openstack-nova08:27
*** thorst has quit IRC08:28
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for aggregates (v1.1)  https://review.openstack.org/50564308:28
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for resource classes (v1.2)  https://review.openstack.org/51118208:28
openstackgerritAndrey Volkov proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4)  https://review.openstack.org/51118308:29
openstackgerritAndrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5)  https://review.openstack.org/51464208:29
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6)  https://review.openstack.org/51464308:29
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7)  https://review.openstack.org/51464408:29
openstackgerritAndrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9)  https://review.openstack.org/51464608:29
openstackgerritAndrey Volkov proposed openstack/osc-placement master: [WIP] CLI allocation candidates (v1.10)  https://review.openstack.org/51464708:29
*** arxcruz|pto is now known as arxcruz08:30
*** brault has joined #openstack-nova08:31
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor  https://review.openstack.org/51296508:31
*** brault has quit IRC08:31
*** salv-orl_ has quit IRC08:32
*** _pewp_ has quit IRC08:32
*** salv-orlando has joined #openstack-nova08:33
*** salv-orlando has quit IRC08:33
*** salv-orlando has joined #openstack-nova08:33
*** _pewp_ has joined #openstack-nova08:34
*** slaweq has joined #openstack-nova08:36
*** markvoelker has joined #openstack-nova08:37
*** evrardjp has quit IRC08:40
*** evrardjp has joined #openstack-nova08:41
*** gongysh has joined #openstack-nova08:46
*** ragiman has joined #openstack-nova08:47
*** moshele has joined #openstack-nova08:48
*** alexchadin has quit IRC08:49
*** vivsoni has joined #openstack-nova08:52
*** Tom_ has quit IRC08:54
*** Tom_ has joined #openstack-nova08:54
openstackgerritDavid Rabel proposed openstack/nova master: VMware: add support for graceful shutdown of instances  https://review.openstack.org/49416908:54
*** ociuhandu has joined #openstack-nova08:55
*** yamamoto has joined #openstack-nova08:55
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node  https://review.openstack.org/51621708:56
*** Tom_ has quit IRC08:58
*** alexchadin has joined #openstack-nova09:04
*** yamamoto has quit IRC09:04
*** rabel has joined #openstack-nova09:04
*** giuseppe1992 has joined #openstack-nova09:06
*** slaweq has quit IRC09:07
*** Tom_ has joined #openstack-nova09:08
*** markvoelker has quit IRC09:10
rabelheyho!09:13
*** yamamoto has joined #openstack-nova09:13
rabelhttps://review.openstack.org/#/c/494169/ is ready for review (again) :)09:13
*** slaweq has joined #openstack-nova09:15
*** evrardjp has quit IRC09:22
*** evrardjp has joined #openstack-nova09:23
*** alexchadin has quit IRC09:25
*** alexchadin has joined #openstack-nova09:25
openstackgerritjichenjc proposed openstack/nova master: Add more functional test for placement.usage  https://review.openstack.org/51326409:27
openstackgerritjichenjc proposed openstack/nova master: fix race condition of instance host  https://review.openstack.org/49445809:28
openstackgerritjichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix  https://review.openstack.org/48239009:29
*** zsli_ has quit IRC09:29
*** gmann is now known as gmann_afk09:29
*** vks1 has quit IRC09:29
*** vks1 has joined #openstack-nova09:31
openstackgerritjichenjc proposed openstack/nova master: check query param for used_limits function  https://review.openstack.org/49909109:32
openstackgerritjichenjc proposed openstack/nova master: emove glance usage inside compute  https://review.openstack.org/51139209:36
openstackgerritjichenjc proposed openstack/nova master: Remove usage of glance.generate_image_url  https://review.openstack.org/51139709:36
openstackgerritjichenjc proposed openstack/nova master: Remove usage of glance.generate_image_url  https://review.openstack.org/51139709:38
*** lpetrut has joined #openstack-nova09:40
*** edmondsw has joined #openstack-nova09:47
*** slaweq has quit IRC09:48
*** sdague has joined #openstack-nova09:50
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Fix missing metavar in aggregate-update  https://review.openstack.org/51248509:50
*** edmondsw has quit IRC09:52
openstackgerritjichenjc proposed openstack/nova master: WIP: conf: remove deprecated 'project_id_regex' opt  https://review.openstack.org/50105709:53
*** slaweq has joined #openstack-nova09:53
*** yamamoto has quit IRC09:54
*** slaweq_ has joined #openstack-nova09:59
*** gongysh has quit IRC10:00
*** cheneydc__ has joined #openstack-nova10:02
*** cheneydc__ has quit IRC10:02
*** giuseppe1992 has quit IRC10:04
*** cheneydc_ has quit IRC10:04
*** zhurong has quit IRC10:05
openstackgerritTakashi NATSUME proposed openstack/nova master: [Placement] Fix parameter order in placement API ref  https://review.openstack.org/51623310:07
*** markvoelker has joined #openstack-nova10:07
*** yangyapeng has quit IRC10:08
*** yangyapeng has joined #openstack-nova10:09
*** Lirt has joined #openstack-nova10:11
*** vivsoni has quit IRC10:11
*** yangyapeng has quit IRC10:13
*** yamamoto has joined #openstack-nova10:16
*** annp has quit IRC10:17
*** zhurong has joined #openstack-nova10:20
*** udesale has quit IRC10:20
openstackgerritTuan Luong-Anh proposed openstack/nova master: Using --option ARGUMENT  https://review.openstack.org/51623810:22
*** moshele has quit IRC10:22
*** vivsoni has joined #openstack-nova10:22
*** yamamoto has quit IRC10:24
*** thorst has joined #openstack-nova10:25
*** jsheeren has joined #openstack-nova10:26
*** slaweq has quit IRC10:27
jsheerenhi, i'm having an issue with nova resize actions.  the instance dir /var/lib/nova/instances/instance_uuid gets deleted; on suspend and resume this causes the error that libvirt cannot find the libvirt xml file10:27
*** moshele has joined #openstack-nova10:28
jsheerenif i watch the folder, an instance_uuid_resize folder get's created.  after the resize, this folder gets deleted.   as well as the /var/lib/nova/instances/instance_uuid folder10:28
jsheerenhas anyone seen this before?10:28
*** slaweq has joined #openstack-nova10:29
*** thorst has quit IRC10:30
*** xinliang has quit IRC10:30
*** sshwarts has joined #openstack-nova10:31
*** brault has joined #openstack-nova10:32
*** brault has quit IRC10:37
*** gszasz has joined #openstack-nova10:37
*** sapcc-bot has quit IRC10:39
*** mkoderer_ has joined #openstack-nova10:39
*** sapcc-bot has joined #openstack-nova10:39
*** dgonzalez_ has joined #openstack-nova10:39
*** tpatzig_ has joined #openstack-nova10:39
*** david_1 has joined #openstack-nova10:39
*** TuanLA has quit IRC10:40
*** markvoelker has quit IRC10:41
*** mkoderer_ has quit IRC10:41
*** dgonzalez_ has quit IRC10:41
*** tpatzig_ has quit IRC10:41
*** david_1 has quit IRC10:41
*** xinliang has joined #openstack-nova10:42
*** jsheeren has quit IRC10:43
-openstackstatus- NOTICE: Zuul has been restarted due to an unexpected issue. Please recheck any jobs that were in progress10:46
*** Tom_ has quit IRC10:48
*** Tom_ has joined #openstack-nova10:51
*** tbachman has quit IRC10:52
*** aloga has quit IRC10:55
*** mvk has quit IRC10:55
*** Tom_ has quit IRC10:55
*** aloga has joined #openstack-nova10:55
*** pcaruana has joined #openstack-nova10:56
openstackgerritTakashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell'  https://review.openstack.org/51377110:58
*** erlon has joined #openstack-nova10:58
*** slaweq has quit IRC11:01
*** Laurene has quit IRC11:01
*** slaweq has joined #openstack-nova11:02
*** smatzek has joined #openstack-nova11:07
*** nicolasbock has joined #openstack-nova11:07
*** cdent has joined #openstack-nova11:11
openstackgerritYikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040611:16
*** namnh has quit IRC11:17
*** moshele has quit IRC11:19
*** sgordon has quit IRC11:21
*** sgordon has joined #openstack-nova11:22
*** yamamoto has joined #openstack-nova11:25
*** salv-orlando has quit IRC11:28
*** mvk has joined #openstack-nova11:29
*** salv-orlando has joined #openstack-nova11:29
*** takashin has left #openstack-nova11:29
*** pchavva has joined #openstack-nova11:29
*** alexchadin has quit IRC11:31
*** alexchadin has joined #openstack-nova11:32
*** yamamoto has quit IRC11:32
*** dave-mccowan has joined #openstack-nova11:35
*** slaweq has quit IRC11:36
*** gszasz has quit IRC11:37
*** markvoelker has joined #openstack-nova11:38
*** yamamoto has joined #openstack-nova11:39
openstackgerritChris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007311:41
*** vks1 has quit IRC11:41
*** tetsuro has quit IRC11:44
*** slaweq has joined #openstack-nova11:45
*** dillaman has joined #openstack-nova11:45
*** hemna_ has joined #openstack-nova11:48
*** vivsoni has quit IRC11:52
efriedalex_xu So... my test case was buggy.  That's a relief.11:53
*** gszasz has joined #openstack-nova11:57
*** spectr has quit IRC11:59
*** salv-orlando has quit IRC11:59
*** spectr has joined #openstack-nova12:00
openstackgerritBalazs Gibizer proposed openstack/nova master: Document the real behavior of notify_on_state_change  https://review.openstack.org/51626412:03
*** udesale has joined #openstack-nova12:03
efriedalex_xu If you're around, let's talk about how you'd like to split up those test cases and I can work on that today.12:04
*** litao__ has quit IRC12:05
*** tbachman has joined #openstack-nova12:08
openstackgerritAndrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10)  https://review.openstack.org/51464712:09
*** thorst has joined #openstack-nova12:09
*** giuseppe1992 has joined #openstack-nova12:09
*** edmondsw has joined #openstack-nova12:09
*** moshele has joined #openstack-nova12:10
alex_xuefried: hey, i'm here12:10
efriedalex_xu Hi.  I'm glad you figured out that bug in my test.12:11
efriedAt some point I'd like to get some tips on how you debug this stuff.12:11
alex_xuefried: I drop in the same trap before, that is really hard thing to remember12:11
efriedAh, okay.12:11
*** markvoelker has quit IRC12:11
efriedWell, I'll know for the future.12:11
efriedI was of course thinking my "add allocation" was actually going to *add* an allocation.12:12
efriedLike, why couldn't you have two separate allocations rows for the same RP/RC12:12
efriedBut I guess if you're e.g. resizing, you're supposed to do the math on the client side and replace the whole record.12:13
efriedAnyway - how would you like to split up that one test case?  Where the three inventories all have something that excludes them from the result.12:13
alex_xuefried: yea, something like that as i remember12:13
alex_xuefried: I like your current way12:14
efriedTalking about this: https://review.openstack.org/#/c/514197/4/nova/tests/functional/db/test_resource_provider.py@238712:14
alex_xuah12:15
efriedYou want to create separate RPs for each different way an inventory can be exclude-worthy?12:15
*** jaypipes has joined #openstack-nova12:15
efriedMorning jaypipes !12:16
efried(sorry, too cheerful?)12:16
alex_xuefried: we can create three RPs , one hasn;t enough total vcpu, one hasn't current max_unit, the last one reserved a lot12:17
efriedalex_xu And is the idea for those RPs to be included or excluded?12:17
*** kylek3h_ has quit IRC12:17
*** moshele has quit IRC12:17
alex_xuefried: all of us are cheerful when seeing jaypipes online :)12:17
efriedalex_xu i.e. do we want to create any other inventory in those RPs that makes them show up in the resulting candidate list?12:17
alex_xuefried: exclude12:17
efriedOkay.12:17
alex_xuefried: we should create inventory which match the request for other resource12:18
*** slaweq has quit IRC12:18
efriedSorry, I didn't follow that one.12:18
alex_xuefried: the reason is due to this https://review.openstack.org/#/c/514197/4/nova/objects/resource_provider.py@84612:18
alex_xuefried: your first VCPU inventory https://review.openstack.org/#/c/514197/4/nova/tests/functional/db/test_resource_provider.py@2381 doesn't have enough vcpus12:19
*** phuongnh has quit IRC12:19
*** thorst_ has joined #openstack-nova12:19
alex_xuthen the where conds will return False directly after https://review.openstack.org/#/c/514197/4/nova/objects/resource_provider.py@85412:20
*** yangyapeng has joined #openstack-nova12:20
efriedalex_xu But just for that one resource.12:20
alex_xuthat means the mem and disk invetories acutally weren't tested12:20
*** thorst__ has joined #openstack-nova12:20
efriedOh, I don't think that's true.  I hope not, anyway.12:20
efriedI have a different RP later on that tests that case...12:21
efriedor I thought I did...12:21
*** jamesden_ has joined #openstack-nova12:21
alex_xuBut the comment message said three inventory for three different failure case12:21
efriedalex_xu The one at L238912:22
efriedThat one has one inventory that's good, one that's bad.12:22
efriedIn that case, the 'bad' one is because the inventory is exhausted by allocations.12:22
efriedBut that shouldn't matter, should it?12:22
jaypipesefried, alex_xu: mornin, fellas!12:22
*** ociuhandu has quit IRC12:23
*** thorst has quit IRC12:23
alex_xuefried: that shounds good12:23
*** jamesdenton has quit IRC12:24
efriedalex_xu In any case, I can expand out all the conditions we're checking for in the SQL.12:24
efriedalex_xu But I still don't quite understand which way you're wanting to test.12:24
*** thorst_ has quit IRC12:24
efriedalex_xu For each case, do you want (one inventory bad, one inventory good <= this RP should be *included*) or do you want (one inventory bad, no other inventory <= this RP should be *excluded*)12:25
alex_xuefried: based on what you want to test12:26
efriedalex_xu Well, I'm not sure what your concern is.12:26
alex_xuefried: the comment https://review.openstack.org/#/c/514197/4/nova/tests/functional/db/test_resource_provider.py@2387 is only about line 2383 and 2386 aren't tested actually12:27
efriedalex_xu Are you concerned that one bad inventory will cause the whole thing to be excluded when it really shouldn't be?12:27
*** slaweq has joined #openstack-nova12:27
*** ociuhandu has joined #openstack-nova12:28
efriedalex_xu Oh, I think they must be.12:28
efriedalex_xu If you were to "fix" either one of those so they're viable, that RP would show up in the result.12:28
efriedThey have to *all* fail in order to exclude this RP.12:28
efriedDon't they?12:28
jaypipesefried, alex_xu: did you see my suggestion on renaming that "root providers" to "non-sharing providers"?12:29
*** markvoelker has joined #openstack-nova12:29
*** jsheeren has joined #openstack-nova12:29
alex_xuefried: no, they are exclue, one RP hasn't enough VCPU, but has enough disk and mem, will be exclude12:29
efriedjaypipes I saw you "abstained" from my leetle survey, but I hadn't yet seen that, no.12:29
jaypipesefried: I didn't abstain. I voted no on all options :)12:30
efriedjaypipes Cool.  So you're in favor of "non-sharing"?12:30
jaypipesefried: you fancy taking over the resource providers summary email for cdent the next 3 weeks?12:30
alex_xujaypipes: sorry, I didn't see that, where is it?12:31
jaypipesefried: I am in favor of "non-sharing providers", yes12:31
jaypipesalex_xu: in the commit message comment..12:31
* jaypipes checks that he actually saved the darn gerrit comment....12:31
alex_xuwhich patch....?12:31
jaypipesalex_xu: https://review.openstack.org/#/c/480379/12:32
efriedjaypipes Oh, uh, I could try to do that.  I don't really have a handle on some of the sub-sub-pieces.12:32
*** Eran_Kuris has quit IRC12:32
jaypipesefried: well, doing the email would give you that handle, ya? :)12:32
efriedThat's one way to look at it.12:32
jaypipeshehe12:32
jaypipesefried: we can split the work between the two of us if you'd prefer.12:33
jaypipesefried: are you going to Sydney?12:33
alex_xujaypipes: yea, non-shared provider sounds the right way12:33
efriedjaypipes Fraid not.12:33
*** catintheroof has joined #openstack-nova12:33
jaypipesefried: I think you meant "Fried not".12:33
*** brault has joined #openstack-nova12:34
jaypipesgeez, I am turning into my dad.12:34
efriedThat's way better than I got in grade school.12:34
alex_xujaypipes: initial I thought that should be RP for compute node, the compute node always the root of nested resource provider, but yes, the child resource provider also can share something with others, so non-shared resource provider sounds right12:34
jaypipesefried: I'm not going to Sydney either, so we can tackle the summary email between the two of us.12:34
efriedjaypipes Okay, sounds good to me.12:34
jaypipesalex_xu: makes sense for compute nodes, yes, but eventually we'd like to detach the placement service from being compute-specific.12:35
alex_xujaypipes: yea12:35
efried++12:35
efriedjaypipes I added 'non-sharing' as an option to the poll.12:36
*** hemna_ has quit IRC12:36
alex_xujaypipes: still looking for your feedback on https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-trait-support-in-allocation-candidates12:36
efriedjaypipes Not that it's a democracy or anything.12:36
*** hemna_ has joined #openstack-nova12:36
jsheerenhi all, i already asked in #openstack but i think it's actually a dev issue (but don't shoot me if i'm wrong ..:)12:37
* efried prepares his M&P 912:37
jaypipeslol12:37
jaypipesjsheeren: what's up?12:37
jsheereni'm running into an issue with instance resize of instances with a cinder backed root disk12:37
jsheerenthe resize does a resize_cleanup action, which checks if a root disk exists12:37
jsheereninstances with a cinder root disk, do not have a nova root disk12:38
*** brault has quit IRC12:38
jsheerenso this code gets executed: https://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L114912:38
*** trinaths1 has joined #openstack-nova12:38
jsheerenthe instance base path gets deleted12:38
*** trinaths has quit IRC12:38
jsheerenthis is a problem on nfs shared storage.  when we suspend the instance after a resize, and then resume it.  nova cannot find the libvirt xml definition12:38
jsheereni think the issue is introduced after https://bugs.launchpad.net/nova/+bug/166683112:38
openstackLaunchpad bug 1666831 in OpenStack Compute (nova) ocata "Nova recreates instance directory after migration/resize" [Low,Fix committed] - Assigned to Lee Yarwood (lyarwood)12:38
efriedIt's the 666 in that bug number.12:39
jaypipesheh12:39
jsheerenlol12:39
jaypipesjsheeren: are you trying to resize the root disk along with the flavor?12:39
jsheerenjaypipes: the resize is triggered through horizon, going from flavor a to flavor b. not sure about the resize of the root disk along with the flavor, let me check real quick12:41
jaypipesjsheeren: just check if the root_gb of the flavor is different from a to b.12:41
jsheerenyes, that is the case12:41
*** dtantsur|afk is now known as dtantsur12:42
jaypipesjsheeren: k. I don't *think* that is the cause of this, but one sec, I'm going to give you some code to patch that one line and retry the operation.... one sec.12:42
*** gszasz has quit IRC12:42
jsheerenjaypipes: thanks12:42
*** yamamoto has quit IRC12:43
jaypipesjsheeren: change the line so that it looks like this:12:45
jaypipesif os.path.exists(inst_base) and not root_disk.exists() and not compute_utils.is_volume_backed_instance(instance._context, instance):12:45
jaypipesjsheeren: restart the nova-compute service, retry your resize operation and lemme know if that fixes things.12:46
*** giuseppe1992 has quit IRC12:46
jsheerenjaypipes: ok, i will, thanks!12:47
*** trinaths1 has quit IRC12:47
jaypipesjsheeren: np. lemme know if that works and I'll add a note to the bug and submit a new bug for it.12:47
*** Eran_Kuris has joined #openstack-nova12:47
*** catintheroof has quit IRC12:47
*** catintheroof has joined #openstack-nova12:49
*** hemna_ has quit IRC12:49
*** zhurong has quit IRC12:50
*** vladikr has joined #openstack-nova12:54
jsheerenjaypipes: your patch works for us12:55
*** jpena is now known as jpena|lunch12:55
*** kylek3h has joined #openstack-nova12:55
jsheerenwe're going to use this for now, as it is nfs shared storage.  i assume people with no nfs shared storage will not run into this issue12:56
jsheerenjaypipes: thanks for the fast response AND fix!12:57
*** Tom_ has joined #openstack-nova12:57
*** mriedem has joined #openstack-nova12:58
*** Tom_ has quit IRC12:58
*** rcernin has quit IRC12:58
*** slaweq has quit IRC13:00
*** salv-orlando has joined #openstack-nova13:00
*** mvk has quit IRC13:01
*** rnoriega has joined #openstack-nova13:02
*** slaweq has joined #openstack-nova13:03
*** rnoriega has quit IRC13:05
*** Eran_Kuris has quit IRC13:05
*** baoli has joined #openstack-nova13:05
*** salv-orlando has quit IRC13:06
*** janki has quit IRC13:06
*** rnoriega has joined #openstack-nova13:07
efriedjaypipes Point of design for GET /allocation_candidates.  Let's say I request inventory in three RCs.  There's a RP with inventory in two of those RCs.  But one of them is exhausted or otherwise unsuitable (wrong step_size, whatever).  Should I *still* get a candidate with that RP in it (assuming another RP in the tree/aggregate can satisfy the other two pieces of the resource request)?13:08
*** felipemonteiro_ has joined #openstack-nova13:09
bhagyashrisHi all, Is nova-cpu.conf is used by only n-cpumte service or this is used by other service also like n-sch, n-cond.. ?13:10
*** lyan has joined #openstack-nova13:11
efriedbhagyashris You talking devstack?13:11
*** felipemonteiro has joined #openstack-nova13:11
*** hemna_ has joined #openstack-nova13:11
bhagyashrisefried: yes13:12
*** gongysh has joined #openstack-nova13:12
*** gongysh has quit IRC13:12
efriedbhagyashris Easiest thing is to take a look at the service commands.  The conf file is sent to the executables as a parameter.13:13
*** felipemonteiro_ has quit IRC13:14
*** mvk has joined #openstack-nova13:16
efriedbhagyashris sudo systemctl cat devstack@* | grep nova-cpu.conf   <== For my devstack, this shows me only the nova-compute service.13:16
bhagyashrisefried: yes13:16
efriedbhagyashris Does that answer your question?13:17
bhagyashrisefried: yes13:17
bhagyashrisefried: thank you :)13:17
efriedbhagyashris You're welcome!13:17
*** gszasz has joined #openstack-nova13:17
*** liverpooler has joined #openstack-nova13:21
*** namnh has joined #openstack-nova13:21
cdentefried: on your inventory question above, if I’m understanding the question correctly (I’m not sure I am) the answer is “yes”. Why, in the dances of your mind, would it be "no"?13:24
efriedcdent I agree.  alex_xu and I have been debating it.13:25
*** vladikr has quit IRC13:25
efriedcdent He raised a valid point, which was, "Give me a real world example where this could happen."13:25
*** lbragstad has quit IRC13:25
cdentalways a useful point13:25
*** hshiina has quit IRC13:26
efriedcdent It took a while, and eventually I had to resort back to the "DISK_GB provided by both local storage and a shared RP" thing.13:26
cdent“real world example” depends on the answer to “can a compute node have local and shared disk” if the answer is yes, we get all sorts of examples, yeah?13:26
cdentjinx!13:26
efriedhah!13:26
*** takashin has joined #openstack-nova13:26
openstackgerritMatthew Booth proposed openstack/nova master: Use Migration object in ComputeManagerMigrationTestCase  https://review.openstack.org/50212613:26
openstackgerritMatthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination  https://review.openstack.org/46252113:26
alex_xuyea, finally, efried beats me13:27
cdentefried: I think the api should allow for that possibility, even if compute nodes don’t or won't13:28
cdentas we can’t see the future of placement13:28
*** superdan is now known as dansmith13:28
cdentand limiting placement for the models presented by today’s compute nodes is limiting13:28
efriedcdent The first example I came up with was: You're on a compute that's NUMA-capable and modeled with its CPU and memory resources in NUMA nodes, but you're making a request that doesn't care about NUMA affinity, so you should be able to get your VCPU from one NUMA node and your MEMORY_MB from a different one.13:28
cdentI still find that modeling hard to stomach, but yeah, if you’re doing it that way, I agree13:29
*** salv-orlando has joined #openstack-nova13:30
efriedAnyway, I agree we don't want to paint ourselves into a corner.  Which means we still have some SQL work to do on https://review.openstack.org/#/c/514197/13:30
*** mdnadeem has quit IRC13:30
*** smatzek has quit IRC13:30
*** amodi has joined #openstack-nova13:30
*** smatzek has joined #openstack-nova13:30
*** smatzek has quit IRC13:31
efriedI had one test case bug that "fixed" one aspect of the test and revealed this design hole at the same time.  Just glad alex_xu caught it (in both places).13:31
efriedalex_xu So what's the path forward from here?  Looks like we need to rework the logic some more.13:32
jaypipesjsheeren: awesomesauce. I'll file a bug :)13:32
alex_xuefried: emm...feel like, https://review.openstack.org/#/c/514197 should be revert to the patchset2?13:33
*** lbragstad has joined #openstack-nova13:33
efriedalex_xu But with the new test case?13:33
jaypipesefried: if there's no constraints saying that all three RCs need to be provided by the same RP, then sure.13:34
efriedjaypipes ++13:34
alex_xuefried: yea13:34
efriedalex_xu Okay.  I'll bring back PS2, expand the test case to include some more permutations of RPs, and we'll see if it passes.13:35
efriedalex_xu If it doesn't, you get to fix the SQL :)13:35
alex_xuefried: cool, thanks13:35
alex_xuok :)13:35
efriedcause I'm still too rusty13:35
efriedDid I mention the last time I did any remotely serious SQL was circa 2003?13:35
efriedEven then I didn't do anything this complicated.13:36
*** efried is now known as efried_brb13:36
*** slaweq has quit IRC13:36
*** vks1 has joined #openstack-nova13:37
*** mlavalle has joined #openstack-nova13:37
alex_xuefried_brb: yea, without jaypipes, I don't know the SQL can't such complicated :)13:37
*** vladikr has joined #openstack-nova13:38
* alex_xu is exhausted, and want to escape the scheduler meeting13:38
*** yamamoto has joined #openstack-nova13:39
*** thorst__ has quit IRC13:40
edleafealex_xu: if you skip the meeting, we'll assign all the tasks to you! :)13:41
*** gcb has joined #openstack-nova13:41
jaypipesalex_xu: no worries, we can do the conversation on your patches.13:41
*** slaweq has joined #openstack-nova13:42
Dinesh_Bhorcdent: Hi, do you have 5 min? I have a question: http://paste.openstack.org/show/624973/13:43
*** brault has joined #openstack-nova13:44
cdenthi Dinesh_Bhor, reading that13:44
edleafeScheduler subteam meeting in 15 minutes in #openstack-meeting-alt13:45
cdentDinesh_Bhor: let’s take it to #openstack-sdks13:45
*** efried_brb is now known as efried13:46
*** awaugama has joined #openstack-nova13:48
*** brault has quit IRC13:48
*** amodi has quit IRC13:49
*** tbachman has quit IRC13:51
*** ratailor has quit IRC13:51
jaypipesefried: luckily, the SQL language has not changed too significantly since 2003. :)13:56
efriedjaypipes And we seem to be using a subset of it anyway, in case different backends.13:57
*** jpena|lunch is now known as jpena13:57
jaypipesefried: ack13:57
*** armax has joined #openstack-nova13:58
*** jmlowe has joined #openstack-nova13:59
edleafeefried: and sqla makes it *so* easy!!13:59
efriededleafe I had a mind-blown moment this weekend when I realized I could actually just pass textual SQL down.  I was like, "uhh, why aren't we just doing that?"13:59
*** eharney has joined #openstack-nova14:00
*** alexchadin has quit IRC14:00
*** hongbin has joined #openstack-nova14:00
edleafeBecause ORMs insulate us from the rough edges of SQL, and make life easy!14:00
*** amodi has joined #openstack-nova14:01
edleafeScheduler subteam meeting running now in #openstack-meeting-alt14:01
*** salv-orlando has quit IRC14:02
*** Eran_Kuris has joined #openstack-nova14:02
*** smatzek has joined #openstack-nova14:03
*** smatzek has quit IRC14:03
*** smatzek has joined #openstack-nova14:04
*** smatzek has quit IRC14:04
*** brault has joined #openstack-nova14:05
*** smatzek has joined #openstack-nova14:05
jaypipesjsheeren: feel free to provide color commentary on https://bugs.launchpad.net/nova/+bug/1728603 :)14:06
openstackLaunchpad bug 1728603 in OpenStack Compute (nova) "Resize a boot-from-volume instance with NFS destroys instance" [High,Confirmed]14:06
*** coreywright has quit IRC14:08
mriedemjaypipes: guh, and that change was backported to ocata https://review.openstack.org/#/c/441037/14:09
*** brault has quit IRC14:09
mriedemwe do have an NFS job in the nova experimental queue, not sure if it tests resize though - but probably should14:10
*** marst has joined #openstack-nova14:11
*** alexchadin has joined #openstack-nova14:13
*** lbragstad has quit IRC14:13
*** lbragstad has joined #openstack-nova14:13
*** tbachman has joined #openstack-nova14:17
*** lbragstad has quit IRC14:18
*** coreywright has joined #openstack-nova14:21
*** lbragstad has joined #openstack-nova14:22
*** salv-orlando has joined #openstack-nova14:24
*** felipemonteiro has quit IRC14:30
*** thorst has joined #openstack-nova14:30
*** thorst has quit IRC14:31
*** thorst has joined #openstack-nova14:31
*** amodi has quit IRC14:33
*** Apoorva has joined #openstack-nova14:33
cdentdtantsur: tomorrow’s fine for traits for me too14:34
dtantsurnice! sorry all14:34
dtantsurmoving to a new country and new flat means sudden appointments here and there..14:34
*** vishwanathj has joined #openstack-nova14:34
*** udesale has quit IRC14:37
*** READ10 has joined #openstack-nova14:37
*** artom has joined #openstack-nova14:42
jaypipesdtantsur: not a problem, man. :)14:42
edleafedtantsur: I can make it tomorrow, too14:43
jaypipesdtantsur: it's 3pm UTC yeah?14:44
bhagyashrisHi All,14:45
dtantsurjaypipes: I think so, yeah14:45
bhagyashrisI am checking the below scenario by enabling the AggregateImagePropertiesIsolation filter14:45
bhagyashrisCreated two host aggregate groups and add one compute node to each14:45
bhagyashrisHost aggregate group A ->add 'os_type=license_ubuntu' host aggregate metadata14:45
bhagyashrisHost aggregate group B-> add 'k1=v1' host aggregate metadata14:45
*** amodi has joined #openstack-nova14:45
bhagyashriscreated the two images by setting the properties14:45
jaypipesmriedem: lol. forever in bluejeans.14:45
bhagyashrisImage1 -> add 'os_type=license_ubuntu'14:46
bhagyashris Image2 -> add 'k1=v1'14:46
bhagyashriscreated the inatence using the Image2 so it should boot on Host aggregate group B but it's booting on Host aggregate group A14:46
*** slaweq has quit IRC14:46
bhagyashrisI have also set the aggregate_image_properties_isolation_namespace = os_type_114:47
bhagyashris aggregate_image_properties_isolation_separator = = in /etc/nova/nova.conf on both the Host14:47
*** zzzeek has joined #openstack-nova14:47
bhagyashriss/inatence/instance14:48
jsheerenjaypipes: :)14:49
mriedemjaypipes: i assume dtantsur will get any and all neil diamond references14:49
jaypipesheh14:49
*** slaweq has joined #openstack-nova14:49
dtantsur:)14:50
*** dikonoor has joined #openstack-nova14:50
jsheerenjaypipes: fixed my name, rest looks good and as we experienced it14:50
*** mvk has quit IRC14:51
jaypipesjsheeren: oops, sorry about that!14:51
bauzaswow, just found http://45.55.105.55:3000/dashboard/db/openstack-bugs14:56
* bauzas loves markus_z14:56
jsheerenno problem! i'm used to fixing my name, it's that ' that messes with most people though :)14:59
*** lyan has quit IRC14:59
*** gcb has quit IRC15:00
*** yamahata has joined #openstack-nova15:01
*** lyan has joined #openstack-nova15:03
*** salv-orlando has quit IRC15:04
*** alexchadin has quit IRC15:04
efriedjsheeren What's the provenance of that name?  (I'm a collector.)15:04
efriedFlemish?15:05
*** salv-orlando has joined #openstack-nova15:06
*** esberglu has joined #openstack-nova15:07
*** esberglu has quit IRC15:07
*** esberglu has joined #openstack-nova15:07
*** takashin has left #openstack-nova15:12
*** mvk has joined #openstack-nova15:13
jsheerenFlemish indeed15:14
jsheerenwe're a rare breed: http://forebears.io/surnames/s-heeren15:15
jsheerenhah apparently i have family in Canada.. hmm15:16
bauzasmriedem: any patch up for https://specs.openstack.org/openstack/nova-specs/priorities/queens-priorities.html ?15:16
mriedembauzas: not yet but i will15:16
bauzasmriedem: I'm just writing a presentation and I'd love to speak about priorities15:16
mriedemfor sydney?15:16
bauzasyup15:16
mriedemwhich presentation is that?15:16
bauzasbug triage15:17
mriedemwhat does this have to do with that?15:17
bauzasnothing really related15:17
bauzasbut just explaining about our tags15:17
mriedemumm, ok15:17
bauzasbut meh15:17
jaypipesefried: what happened to the patch about parsing numbered request groups that I commented on?15:17
bauzasmriedem: no worries, I'll just pass a list of tags15:17
*** itlinux has joined #openstack-nova15:18
efriedjaypipes If you're talking about https://review.openstack.org/#/c/515151/ -- I reworked it for you & gibi.  gibi left a couple little tweaks that I'll get to later today, so if you want to hit it again, I can also take care of anything else you find.15:19
mriedembauzas: were you going to drop your -1 on this? https://review.openstack.org/#/c/51553015:19
jaypipesefried: k, thanks, yeah, that's the one. I was looking at https://review.openstack.org/#/c/514091/ and wondering where it was...15:20
efriedjaypipes Yeah, that one's coming from the placement side.15:20
mriedembauzas: also fyi in case you didn't notice this from the weekend https://review.openstack.org/#/c/515530/15:20
jaypipesefried: k15:20
efriedjaypipes Also needs a review, if you please :)15:20
bauzasmriedem: yup, just forgot to remove it15:20
jaypipesefried: yup, I'm on it. just finished reviewing cdent's series on POST /allocations. Moving on to yours.15:20
efriedjaypipes Thanks.15:21
jaypipesedleafe is still reworking the Selection object series, right?15:21
mriedembauzas: i think we should try and get https://review.openstack.org/#/c/515530/ into newton before eol15:21
mriedemsince we shipped a broken feature basically15:21
edleafejaypipes: the object is fine. It's the RPC change that's being re-worked15:22
*** slaweq has quit IRC15:22
*** salv-orl_ has joined #openstack-nova15:23
jaypipesedleafe: gotcha.15:24
openstackgerritJay Pipes proposed openstack/nova master: placement: add nested resource providers  https://review.openstack.org/37713815:25
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721515:25
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480715:25
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569315:25
jaypipesdansmith, gibi, edleafe, cdent, bauzas, mriedem: ok ^ should be good to review. First one to microversion 1.12 wins some Halloween candy.15:26
*** salv-orlando has quit IRC15:27
mriedemyou can't tempt me with halloween candy15:29
*** AlexeyAbashkin has quit IRC15:30
*** AlexeyAbashkin has joined #openstack-nova15:31
*** fragatina has joined #openstack-nova15:31
*** lajoskatona has left #openstack-nova15:33
*** stvnoyes has joined #openstack-nova15:33
*** jsheeren has quit IRC15:34
*** jsheeren has joined #openstack-nova15:34
efriedalex_xu Still awake?15:34
*** AlexeyAbashkin has quit IRC15:35
*** jsheeren has left #openstack-nova15:35
cdentjaypipes: I don’t think we want zero value resources? See my response on https://review.openstack.org/#/c/500073/2715:35
*** felipemonteiro has joined #openstack-nova15:36
*** catintheroof has quit IRC15:37
*** felipemonteiro_ has joined #openstack-nova15:38
*** gyee has joined #openstack-nova15:38
*** yamamoto has quit IRC15:40
jaypipescdent: yeah, guess I was confused.15:40
namnhjaypipes: hi Jay, could I ask you a question? have you created a etherpad or something to note a list of members who will join fast forward upgrade session at the sydney summit?15:41
openstackgerritMatt Riedemann proposed openstack/nova master: Add microversion to allow setting flavor description  https://review.openstack.org/51521315:41
jaypipescdent: the magic zeroed allocations are only internal to the AllocationList.create_all() method then...15:41
mriedemnamnh: that's a question for lyarwood probably15:41
*** yamamoto has joined #openstack-nova15:41
*** felipemonteiro has quit IRC15:42
jaypipesnamnh: yeah, especially since I won't be in Sydney :)15:42
cdentyes, I think I may have said that was the intent on a comment on the change that adjusted create_all()15:42
cdent(that it was an internal thing to be jiggered at the api controller level for special powers)15:42
*** penick has joined #openstack-nova15:42
jaypipescdent: k, cool with me then.15:43
cdent✔\15:43
mriedemnamnh: should eventually show up here https://wiki.openstack.org/wiki/Forum/Sydney201715:43
namnhmriedem: I asked him, but he does not know :) I've just asked Arkady, he suggest me to ask Jay :(15:44
mriedemwho is the actual session leader?15:44
mriedemhttps://www.openstack.org/summit/sydney-2017/summit-schedule/events/20451/fast-forward-upgrades15:45
mriedemeric mccormick15:45
mriedemfind him15:45
*** yamamoto has quit IRC15:45
*** yamamoto has joined #openstack-nova15:45
* mriedem asks in -operators15:45
namnhmriedem: according to this mailling-list http://lists.openstack.org/pipermail/openstack-dev/2017-September/122361.html . that is Arkady15:46
mriedemgreat...15:47
mriedemdansmith: is lyarwood not going to be at the summit?15:47
dansmithI don't think so15:47
mdboothmelwitt: mriedem Any chance we can merge this one: https://review.openstack.org/#/c/507202/15:47
mriedemok and the person moderating the session is sending people to jay...who won't be at the summit, nor is involved in fast forward upgrades15:48
namnhmriedem: i asked both of them, it seems there is no a list of members for now15:48
mdboothNot sure what the current hold up is, but it's still a data corruptor15:48
mriedemmdbooth: i believe the hold up is getting a run with https://review.openstack.org/#/c/508163/215:49
mriedemin the live migration job to make sure nothing is regressed15:49
mriedemnamnh: ok i'll ask in the ML then15:49
namnhmriedem: thanks in advance, because I'd like to know a list of members to prepare something for discussion :)15:50
mdboothmriedem: Thanks. I see melwitt spammed it a few times already. I've spammed it again.15:51
namnhmriedem: are you joining FFU-session?15:51
mdboothEurgh, gonna have to rebase it15:52
mriedemnamnh: idk15:52
mriedemFFU isn't really a focus area of mine15:52
*** slaweq has joined #openstack-nova15:53
openstackgerritChris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007315:53
namnhmriedem: i see, thanks for your time15:53
openstackgerritMatthew Booth proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks  https://review.openstack.org/50720215:55
openstackgerritMatthew Booth proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test  https://review.openstack.org/50816315:55
namnhjaypipes: :)) i see15:55
*** slaweq has quit IRC15:57
*** spectr has quit IRC16:02
*** ijw has joined #openstack-nova16:03
*** catintheroof has joined #openstack-nova16:07
*** fragatina has quit IRC16:07
*** fragatina has joined #openstack-nova16:08
*** Oku_OS is now known as Oku_OS-away16:09
mriedemdansmith: fyi added starter etherpads for the nova sessions i'm aware of https://wiki.openstack.org/wiki/Forum/Sydney2017#List_of_Sessions_.26_Etherpads16:10
dansmithmriedem: cool16:10
*** itlinux has quit IRC16:12
*** catintheroof has quit IRC16:12
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480716:14
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569316:14
*** ircuser-1 has quit IRC16:15
*** fragatina has quit IRC16:17
*** jamesden_ has quit IRC16:22
*** jamesdenton has joined #openstack-nova16:23
*** sridharg has quit IRC16:26
*** slaweq has joined #openstack-nova16:27
*** rmcallis has joined #openstack-nova16:28
*** fragatina has joined #openstack-nova16:30
openstackgerritMatt Riedemann proposed openstack/nova master: rp: break functions out of _set_traits()  https://review.openstack.org/50990816:32
*** fragatina has quit IRC16:33
*** fragatina has joined #openstack-nova16:33
*** yamahata has quit IRC16:34
openstackgerritDoug Hellmann proposed openstack/nova master: add whereto for testing redirect rules  https://review.openstack.org/51638516:36
openstackgerritDoug Hellmann proposed openstack/nova master: add whereto for testing redirect rules  https://review.openstack.org/51638516:37
*** ragiman has quit IRC16:38
efriedjaypipes I'm about to be afk for a couple of hours, but: We do need to distinguish between unnumbered and numbered, remember, because the unnumbered can come from any RP in the tree or aggregate; whereas numbered has to come from exactly the same RP.16:40
efriedI'm not a fan of naming them 'numbered' and 'unnumbered', but thus far haven't been able to come up with better nomenclature.16:40
*** gouthamr has joined #openstack-nova16:41
jaypipesefried: no, the request group just means "in the same provider tree".16:41
efriedjaypipes This was the whole thing the spec was about.16:41
efriedjaypipes Same provider tree doesn't work for e.g. a VF and its bandwidth.16:41
*** Swami has joined #openstack-nova16:41
efriedjaypipes See use cases in the spec.16:42
jaypipesefried: Sorry, by "same provider tree" I mean "same provider or its descendants"16:42
jaypipesefried: and each request group means exactly the same thing.16:42
efriedjaypipes Whoah.  That's totally not what happened in the spec.  To be continued...16:43
*** efried is now known as efried_rollin16:43
*** jpena is now known as jpena|brb16:44
*** markvoelker_ has joined #openstack-nova16:47
*** markvoelker has quit IRC16:49
*** fragatina has quit IRC16:49
*** rmcallis has quit IRC16:49
*** namnh has quit IRC16:49
*** slaweq has quit IRC16:50
*** namnh has joined #openstack-nova16:50
*** slaweq has joined #openstack-nova16:50
*** markvoelker_ has quit IRC16:51
*** catintheroof has joined #openstack-nova16:52
*** rmcallis has joined #openstack-nova16:53
*** tesseract has quit IRC16:55
*** slaweq has quit IRC16:55
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: libvirt: do not remove inst_base when volume-backed during resize  https://review.openstack.org/51639516:55
mriedemjaypipes: this should allow reproducing that bug in the NFS job ^16:55
*** gszasz has quit IRC16:56
*** catintheroof has quit IRC16:56
*** markvoelker has joined #openstack-nova17:00
*** markvoelker has quit IRC17:05
*** yamahata has joined #openstack-nova17:07
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Pass the correct image to build_request_spec in conductor.rebuild_instance  https://review.openstack.org/51640017:07
*** Apoorva has quit IRC17:10
*** stvnoyes has quit IRC17:11
*** namnh has quit IRC17:11
*** harlowja has joined #openstack-nova17:19
*** mvk has quit IRC17:20
*** kfarr has joined #openstack-nova17:21
*** jpena|brb is now known as jpena17:21
*** lpetrut has quit IRC17:22
*** fragatina has joined #openstack-nova17:22
*** markvoelker has joined #openstack-nova17:23
*** catintheroof has joined #openstack-nova17:27
*** markvoelker has quit IRC17:28
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Pass the correct image to build_request_spec in conductor.rebuild_instance  https://review.openstack.org/51640417:29
*** markvoelker has joined #openstack-nova17:30
*** AlexeyAbashkin has joined #openstack-nova17:30
*** nikhil has joined #openstack-nova17:31
*** itlinux has joined #openstack-nova17:31
*** Apoorva has joined #openstack-nova17:32
*** Apoorva has quit IRC17:32
*** catintheroof has quit IRC17:32
*** Apoorva has joined #openstack-nova17:32
*** vks1 has quit IRC17:34
*** markvoelker has quit IRC17:34
*** slaweq has joined #openstack-nova17:34
*** AlexeyAbashkin has quit IRC17:35
*** catintheroof has joined #openstack-nova17:37
*** slaweq has quit IRC17:39
*** catintheroof has quit IRC17:42
*** slaweq has joined #openstack-nova17:45
*** sahid has quit IRC17:45
*** jmlowe has quit IRC17:47
*** sshwarts has quit IRC17:49
*** purplerbot has quit IRC17:50
*** purplerbot has joined #openstack-nova17:51
*** lpetrut has joined #openstack-nova17:56
openstackgerritMerged openstack/nova master: rp: remove _HasAResourceProvider mixin  https://review.openstack.org/50903617:59
*** ijw has quit IRC18:00
*** avolkov` has joined #openstack-nova18:01
*** ijw has joined #openstack-nova18:01
*** dikonoor has quit IRC18:02
*** avolkov has quit IRC18:02
*** jmlowe has joined #openstack-nova18:04
*** markvoelker has joined #openstack-nova18:05
*** nicolasbock has quit IRC18:05
*** slaweq has quit IRC18:08
*** markvoelker has quit IRC18:09
*** markvoelker has joined #openstack-nova18:10
*** namnh has joined #openstack-nova18:12
*** slaweq has joined #openstack-nova18:13
*** namnh has quit IRC18:17
*** slaweq has quit IRC18:18
*** jpena is now known as jpena|off18:18
*** markvoelker has quit IRC18:19
*** markvoelker has joined #openstack-nova18:19
openstackgerritDavid Rabel proposed openstack/nova master: Fix format in live-migration-usage.rst  https://review.openstack.org/51641618:19
*** markvoelker_ has joined #openstack-nova18:20
*** markvoelker has quit IRC18:23
*** markvoelker has joined #openstack-nova18:32
*** markvoelker_ has quit IRC18:33
*** markvoelker has quit IRC18:37
*** efried_rollin is now known as efried18:37
*** dtantsur is now known as dtantsur|afk18:40
*** felipemonteiro_ has quit IRC18:41
*** markvoelker has joined #openstack-nova18:41
*** catintheroof has joined #openstack-nova18:43
*** mikal has quit IRC18:43
*** felipemonteiro has joined #openstack-nova18:44
*** felipemonteiro_ has joined #openstack-nova18:45
*** catintheroof has quit IRC18:48
*** felipemonteiro has quit IRC18:49
*** catintheroof has joined #openstack-nova18:49
*** salv-orlando has joined #openstack-nova18:52
*** mvk has joined #openstack-nova18:53
*** pchavva has quit IRC18:53
*** ijw has quit IRC18:54
*** mikal has joined #openstack-nova18:55
*** salv-orl_ has quit IRC18:55
*** baoli has quit IRC18:59
efriedjaypipes Now a good time?18:59
*** jdurgin has joined #openstack-nova19:00
*** ijw has joined #openstack-nova19:06
*** baoli has joined #openstack-nova19:09
*** AlexeyAbashkin has joined #openstack-nova19:10
jaypipesefried: sure19:13
efriedjaypipes So first of all I'd like to set the baseline that what I've got proposed thus far is what's described in the spec.19:14
efriedjaypipes Then we can go into whether the spec is wrong, and what to do about that.19:14
efriedjaypipes I was just going over the comments on the spec myself...19:14
efriedhttps://review.openstack.org/#/c/510244/19:14
efriedjaypipes Unfortunately I realize that this spec was basically written, reviewed, and approved while you were out.  (I do remember you acking it when you got back, but I recognize that's not the same thing as being involved in the review process.)19:16
*** yamamoto has quit IRC19:16
*** READ10 has quit IRC19:17
jaypipesefried: the only difference I see between the resources= key that exists now and the resourcesN= keys that are in that spec is that the resources= key can have part of its request satisfied by a sharing provider.19:19
efriedjaypipes In which case we nevertheless need to keep them semantically separate when processing them, nah?19:21
efriedjaypipes I mean, I think we still wanted to get narrower than that, but that at the very least should be enough motivation to have a distinction.19:21
jaypipesefried: sure, but I'd prefer we just annotate the RequestGroup with a "can_use_shared_resources" attribute or similar19:22
efriedjaypipes Oh, rather than calling it 'numbered' vs 'unnumbered', sure I'm open to better ways of describing.19:22
jaypipesright19:22
efriedjaypipes But IIUC, we can't share an object between the scheduler side and the placement side, can we?19:22
jaypipescuz numbered and unnumbered doesn't indicate what the diff is.19:23
jaypipesthat was my complaint.19:23
jaypipesefried: no, we can't share an object, but that's not the point. The point is to have an attribute that indicates what the difference between the things is. "numbered" and "unnumbered" doesn't do that.19:23
jaypipesthat's what I was trying to say19:24
efriedjaypipes Cool, I'm good with that for sure.19:24
efriedjaypipes Back to the other issue...19:24
jaypipesefried: the over-the-HTTP-wire syntax of using numbers is fine. I'm just saying when creating objects that represent these things, let's not call them "unnumbered" or "numbered"19:24
efriedjaypipes Dig19:25
jaypipesk, cool.19:25
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - testing check-requirements  https://review.openstack.org/51619519:25
efriedjaypipes So the "non-sharing" group can get resources from any RP in the tree or any RP in the same aggregate.  We're agreed on that.19:25
efriedjaypipes Bah, I meant the "sharing" group.19:25
efriedCause this wasn't confusing enough.19:25
efriedjaypipes Per the spec, the "non-sharing" group is limited to getting its resources from exactly one RP.  No trees, sub-branches, etc.19:26
jaypipesefried: ok, if that's what was agreed to in the spec, cool.19:27
efriedjaypipes Okay, good deal.  I think the thing about using branches *might* work, but only because we can't think of a case right now where a parent and child would both provide inventory in the same RC.19:27
efriedCleaner to keep it same-RP.19:27
jaypipesright19:28
jaypipesefried: how about RequestGroup.use_same_provider as the attribute name and make the parser set that to True for any request group with a number in the query param.19:29
efriedjaypipes Dig.19:29
jaypipescoolio.19:29
efriedjaypipes So do you want me to have a (separately-defined) RequestGroup class on the placement side of the world as well?19:29
efriedNot a DB object, obviously; just a util/helper.19:30
mriedemsdague: want to send this through? https://review.openstack.org/#/c/508271/19:33
mriedemotherwise i'm just going to self approve19:33
*** markvoelker has quit IRC19:34
*** markvoelker has joined #openstack-nova19:34
dansmithmriedem: I got it19:35
mriedemthanks19:35
dansmithsorry I wasn't paying attention to that19:35
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test  https://review.openstack.org/50816319:36
jaypipesefried: yeah, prolly.19:38
efriedjaypipes ight.19:39
jaypipesefried: you can put it in nova/api/openstack/placement/19:39
efriedack19:39
jaypipesefried: and the other in /nova/scheduler/ somewhere19:39
efriedjaypipes It's in utils at the moment, will leave it there.19:39
jaypipesefried: should just be a POD object (plain ol' data)19:39
jaypipesefried: sure, that's cool19:39
efriedyuh19:39
*** ijw has quit IRC19:40
sdaguemriedem: is ceph going to eventually work on the super conductor stuff?19:40
mriedemsdague: the problem isn't superconductor + ceph,19:40
sdagueoh, I guess the issue is grenade doesn't do that yet19:40
mriedemthe problem there is grenade19:40
*** ociuhandu has quit IRC19:40
mriedemyup19:40
mriedemsdague: which i half heartedly posted https://review.openstack.org/#/c/509803/ just to see how bad things blew up19:41
mriedembut didn't really dig into it19:41
mriedemappears to be a discover_hosts issue19:41
mriedemhttp://logs.openstack.org/03/509803/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/d8414a5/logs/grenade.sh.txt.gz19:41
*** rmcallis has quit IRC19:42
*** awaugama has quit IRC19:42
mriedemyeah the configs don't get set properly, because e.g. nova-cpu.conf has api_database stuff in it http://logs.openstack.org/03/509803/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/d8414a5/logs/etc/nova/nova-cpu.conf.txt.gz19:44
openstackgerritMerged openstack/nova master: Pass the correct image to build_request_spec in conductor.rebuild_instance  https://review.openstack.org/51553019:46
openstackgerritJay Pipes proposed openstack/nova master: placement: add nested resource providers  https://review.openstack.org/37713819:52
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721519:52
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480719:52
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569319:52
jaypipescdent: turned out that you identified some useless code that should be removed. so I did. ^19:52
*** jmlowe has quit IRC19:57
*** kylek3h has quit IRC20:00
*** penick has quit IRC20:00
sdaguemriedem: yeh, let me try a thing with that20:02
cdentjaypipes: yay less code!20:02
*** baoli has quit IRC20:04
*** liverpooler has quit IRC20:06
*** oanson has quit IRC20:08
cfriesenwhen live-migrating with block-migration = "auto", does anyone know offhand where we do the check to see if block migration is needed or not?20:08
openstackgerritEric Fried proposed openstack/nova master: WIP: Add _get_providers_with_any_resource method  https://review.openstack.org/51419720:10
openstackgerritEric Fried proposed openstack/nova master: _get_usages_by_provider_and_rc not filter on RC ID  https://review.openstack.org/51464920:10
openstackgerritEric Fried proposed openstack/nova master: Add _get_traits_by_rp_ids method  https://review.openstack.org/51118420:10
openstackgerritEric Fried proposed openstack/nova master: Add ProviderSummaryList object  https://review.openstack.org/51419820:10
openstackgerritEric Fried proposed openstack/nova master: More caches: ProviderSummary, ProviderSummaryList  https://review.openstack.org/51419920:10
openstackgerritEric Fried proposed openstack/nova master: Add AggregatedProviderSummary[List] objects  https://review.openstack.org/51420020:10
openstackgerritEric Fried proposed openstack/nova master: Add has_resources to AggregatedProviderSummary  https://review.openstack.org/51465020:10
openstackgerritEric Fried proposed openstack/nova master: AggregatedProvierSummary.as_allocation_request_obj  https://review.openstack.org/51465120:10
openstackgerritEric Fried proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters  https://review.openstack.org/47976620:10
efriedjaypipes In other news, if you have some time to look at this WIP patch ^ there's either a SQL bug or a test case bug.20:10
efriedjaypipes This one: https://review.openstack.org/51419720:11
jaypipesefried: wasn't that alex_xu's?20:11
efriedjaypipes alex_xu and I have been pair programming it.20:12
jaypipescfriesen: mriedem would know best.20:12
jaypipesefried: k.20:12
mriedemummm20:12
efriedjaypipes Gah, pair "developing", per edleafe20:12
*** rabel has quit IRC20:12
efriedwhich is different.20:12
mriedemcfriesen: you're going to have to check the code20:13
jaypipesmriedem: you're all about that flag.20:13
* jaypipes runs for the hills20:14
*** kylek3h has joined #openstack-nova20:14
*** slaweq has joined #openstack-nova20:14
cfriesenjaypipes: mriedem: heh...I think I found it.   git history is my friend20:14
*** penick has joined #openstack-nova20:14
*** kylek3h_ has joined #openstack-nova20:14
cfriesenefried: someone's developing a pair?20:14
*** baoli has joined #openstack-nova20:15
efriedcfriesen It's a slow process.  But actually more efficient than one at a time.20:16
*** salv-orlando has quit IRC20:17
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: document caveats with scheduler hints  https://review.openstack.org/51644220:17
mriedemsdague: how is this for documented ambiguity? :) ^20:17
*** yamamoto has joined #openstack-nova20:17
*** salv-orlando has joined #openstack-nova20:17
sdaguemriedem: yeh, pretty much20:18
*** eharney has quit IRC20:18
mriedemmordred: do you ever have to rely on scheduler hints?20:18
*** kylek3h has quit IRC20:18
clarkbmriedem: I think any of that would be largely transparent to infra at least since we are just "public" cloud ocnsumers for the most part. and since infracloud is single tenant we don't need any special scheduling we just schedule to any of our nodes20:19
mriedemclarkb: ok. i just figure anyone that is a user of openstack and has to rely on scheduler hints for their app to work, probably has a hell of a time making that app work across different clouds - but i have no idea if something like that exists in the wild20:20
clarkbI know ovh schedules us to dedicate resources so we don't noisy neighbor their customers20:21
clarkbthey do this with a flavor just for us not sure if that involves scheduler hints20:21
clarkbbut ya we don't directly interact with scheduler hints at least20:21
*** salv-orlando has quit IRC20:21
mriedemclarkb: yeah that wouldn't be scheduler hints, just extra specs and host aggregates20:21
openstackgerritEric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs  https://review.openstack.org/51515120:22
openstackgerritEric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec}  https://review.openstack.org/51522320:22
openstackgerritEric Fried proposed openstack/nova master: ResourceRequest.to_querystring()  https://review.openstack.org/51581120:22
mriedemi need to know if cloud x has a scheduler hint so i can tell it which lpar i want my powervm instance on20:23
mriedembecause lpar 44 is the one i want20:23
*** larainema has quit IRC20:26
jaypipesmriedem: and how would a cloud user know that lpar 44 is the one they want? :)20:26
*** yamamoto has quit IRC20:26
sdaguejaypipes: probably the same way they'd know which az they wanted, it was documented somewhere.20:27
jaypipessdague: heh20:28
jaypipessdague: my point is that the AZ is a publicly-visible grouping construct that OpenStack exposes. LPARs are not publicly-visible constructs...20:29
jaypipessdague: in the same sense that on AWS I cannot request a particular compute host (because, well, as a cloud user, I don't *know* what the compute hosts are...)20:30
*** stvnoyes has joined #openstack-nova20:30
jaypipessdague: but maybe you're just pulling my leg... I can't tell :)20:30
mriedemi'm just using a ridiculous example20:31
mriedembut yeah, just saying, one cloud/product docs could have any number of weird things tailored for their users20:32
mriedemif we actually cared, we'd have some kind of scheduler_hints API to return what *this* cloud supports for scheduler hints, so it would at least be discoverable20:34
*** markvoelker has quit IRC20:35
jaypipesmriedem: sure, agreed. discoverable, not interoperable...20:35
mriedemright, neither are flavors20:35
jaypipesmriedem: ack20:35
jaypipesmriedem: but standard resource classes and os-traits are getting us much closer in that respect.20:35
*** stvnoyes has quit IRC20:35
*** penick has quit IRC20:36
*** AlexeyAbashkin has quit IRC20:37
*** penick has joined #openstack-nova20:38
*** penick has quit IRC20:39
*** ijw has joined #openstack-nova20:40
*** cdent has quit IRC20:42
*** penick has joined #openstack-nova20:44
openstackgerritMatt Riedemann proposed openstack/nova master: Transform keypair.import notification  https://review.openstack.org/46751420:44
*** penick has quit IRC20:45
*** markvoelker has joined #openstack-nova20:45
*** ijw has quit IRC20:46
*** ijw has joined #openstack-nova20:46
*** markvoelker has quit IRC20:49
*** markvoelker has joined #openstack-nova20:49
*** jmlowe has joined #openstack-nova20:55
*** rmcallis has joined #openstack-nova20:58
*** penick has joined #openstack-nova20:59
mriedemthis is fun, http://logs.openstack.org/96/516396/1/check/legacy-tempest-dsvm-py35/ecb9db4/logs/screen-n-api.txt.gz#_Oct_30_18_01_18_12238021:00
mriedemcan't find that request in the scheduler logs to see why NoValidHost21:00
*** markvoelker_ has joined #openstack-nova21:02
*** markvoelker has quit IRC21:04
mriedemah because we overwrite the NoValidHost message before returning to the api21:04
mriedemdansmith: http://logs.openstack.org/96/516396/1/check/legacy-tempest-dsvm-py35/ecb9db4/logs/screen-n-super-cond.txt.gz?level=TRACE#_Oct_30_18_01_18_00314821:04
*** stvnoyes has joined #openstack-nova21:04
mriedem(409 {"errors": [{"status": 409, "request_id": "req-7eec8dd2-f65c-43fa-b3df-cdf7a236aa03", "title": "Conflict", "detail": "There was a conflict when trying to complete your request.\n\n Inventory changed while attempting to allocate: Another thread concurrently updated the data. Please retry your update  "}]})21:05
*** stvnoyes has quit IRC21:07
*** smatzek has quit IRC21:09
*** smatzek has joined #openstack-nova21:10
*** slaweq has quit IRC21:11
*** esberglu has quit IRC21:11
*** slaweq has joined #openstack-nova21:12
*** esberglu has joined #openstack-nova21:12
*** smatzek has quit IRC21:14
*** pcaruana has quit IRC21:15
*** slaweq has quit IRC21:16
*** esberglu has quit IRC21:16
*** slaweq has joined #openstack-nova21:16
*** slaweq has quit IRC21:16
mriedeme-r seems to be dead so can't tell how bad this is http://status.openstack.org/elastic-recheck/21:17
*** salv-orlando has joined #openstack-nova21:18
*** kylek3h_ has quit IRC21:20
*** dave-mccowan has quit IRC21:20
*** kylek3h has joined #openstack-nova21:21
mriedemhttps://bugs.launchpad.net/nova/+bug/172872221:21
openstackLaunchpad bug 1728722 in OpenStack Compute (nova) "Resize test fails in conductor during migration/instance allocation swap: "Unable to replace resource claim on source host"" [High,New]21:22
mriedemok, so what's probably happening is conductor is trying to 'claim' allocations in placement at the same time as the scheduler for a concurrent test, and capacity changed so we have to retry client-side, just like in the scheduler21:22
*** salv-orlando has quit IRC21:22
*** esberglu has joined #openstack-nova21:23
openstackgerritMerged openstack/nova master: rp: break functions out of _set_traits()  https://review.openstack.org/50990821:25
*** kylek3h has quit IRC21:25
openstackgerritMerged openstack/nova master: Fix live migration grenade ceph setup  https://review.openstack.org/50827121:25
openstackgerritMerged openstack/nova master: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622521:25
*** avolkov` has quit IRC21:27
cfriesenmriedem: is there a bug for the fact that rebuilding a boot-from-volume instance changes the reported image without changing the volume contents?  Or are we just tracking that under https://bugs.launchpad.net/nova/+bug/1482040 ?21:30
openstackLaunchpad bug 1482040 in OpenStack Compute (nova) "rebuild volume-backed instance with image does not replace image in root disk" [Undecided,Confirmed]21:31
mriedemjust tracking that under that bug21:31
*** esberglu has quit IRC21:31
*** esberglu has joined #openstack-nova21:32
cfriesenis there any concensus on how to handle that?  if not, should we bring it up at the Forum?21:32
*** esberglu has quit IRC21:32
*** esberglu has joined #openstack-nova21:32
mriedemwe should fail fast in the api if it's a volume-backed instance21:32
cfriesensounds reasonable21:33
*** jmlowe has quit IRC21:35
*** smatzek has joined #openstack-nova21:36
*** thorst has quit IRC21:38
kukaczhi, is there a method how to ensure user data script execution on instance creation without making the user to use eg. the --user-data CLI argument?21:38
*** kfarr has quit IRC21:39
kukaczI'm thinking of doing it based eg. on some pre-defined metadata value presence21:39
*** smatzek has quit IRC21:40
*** jmlowe has joined #openstack-nova21:42
*** baoli has quit IRC21:42
mriedemkukacz: check this out https://docs.openstack.org/nova/latest/user/vendordata.html21:43
mriedemconfigure nova for force_config_drive=True so every instance gets a config drive, and then you can setup a vendordata v2 service for injecting whatever you need into the guest21:43
mriedemjaypipes: for this todo, we'd want a microversion so the client can detect if it can rely on the server doing the retries or if the client has to do them https://github.com/openstack/nova/blob/965f56d7d2ca1f668f70d24d4dcc20e418bb5b9c/nova/objects/resource_provider.py#L188721:46
mriedemjaypipes: but i'm wondering if we want to make that a hard-coded retry value in placement (or configurable), or allow the user to pass in the number of retries (probably not)21:46
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103521:47
mriedemor allow the user to request a certain number of retries but have a configurable max in placement, defaulted to 3?21:47
kukaczmriedem: thanks! looking into that21:47
*** Sukhdev has joined #openstack-nova21:48
dansmithmriedem: is that a generation mismatch thing?21:48
mriedemdansmith: yup21:48
cfriesenmriedem: does a rebuild in the boot-from-vol case rebuild any of the volume connection stuff in a way that wouldn't happen on a stop/start?  if so then maybe we'd only want to fail if we try to rebuild to a different image21:48
dansmithmriedem: so we're not retrying those in the reportclient I guess?21:48
mriedemdansmith: not in put_allocations21:48
mriedemwhich is what the migrate task uses21:48
dansmithack okay21:49
mriedemthe claim_resources method in the reportclient is the thing the scheduler calls and that does the double up and retry logic21:49
dansmithokay21:49
dansmithmriedem: you gots a bug yet?21:49
mriedemso we could hacky sack a bit and have migrate call claim_resources but pass a "don't double things"21:49
mriedemdansmith: yeah https://bugs.launchpad.net/nova/+bug/172872221:49
openstackLaunchpad bug 1728722 in OpenStack Compute (nova) "Resize test fails in conductor during migration/instance allocation swap: "Unable to replace resource claim on source host"" [High,Triaged]21:49
dansmithmriedem: nah, we can just make a wrapper for put that does it I think and refactor a bit21:50
mriedemthat's what i started doing,21:50
dansmithmriedem: I'll work on that tomorrow21:50
mriedembut then you end up copying most of the put_allocations method from placement, and have to get the response back for the status_code21:50
kukaczmriedem: which release was this vendordata feature introduced in? Pike?21:50
mriedemand reportclient.put doesn't have the safe_connect on it21:50
mriedembut i think we said we could remove that...21:50
dansmithmriedem: I think we want the wrapper in reportclient though21:51
mriedemkukacz: newton i think, and enhanced in ocata21:51
mriedemdansmith: ah yeah that would work21:51
dansmithmriedem: I would think we could make claim_resources use the wrapped version too21:51
dansmithto avoid that duplication21:51
dansmithanything calling put_allocations shouldn't have to worry about generation retries21:51
*** takashin has joined #openstack-nova21:51
*** rcernin has joined #openstack-nova21:51
openstackgerritMerged openstack/nova master: use context mgr in instance.delete  https://review.openstack.org/44376421:51
kukaczmriedem: great, seems this could help our usecase. thanks a lot!21:52
mriedemkukacz: yw21:52
melwittmriedem: I'm not seeing where https://review.openstack.org/#/c/508163 ran the test_volume_backed_live_migration test in the results21:54
mriedemmelwitt: http://logs.openstack.org/63/508163/4/check/legacy-tempest-dsvm-multinode-live-migration/633c914/job-output.txt.gz#_2017-10-30_20_50_05_84252021:55
melwittmriedem: oh, thanks. I was looking at the http://logs.openstack.org/63/508163/4/check/legacy-tempest-dsvm-multinode-live-migration/633c914/logs/testr_results.html.gz and I guess it doesn't drill down that much21:55
mriedemnope21:55
jaypipesmriedem: I think a hard-coded server-side retry is what I would go with.21:55
mriedemthese are post test hook driven21:55
melwittoh :\21:56
mriedemmelwitt: looks like i didn't uncomment the part to run it with ceph as the ephemeral backend, but not sure that would make any difference21:56
jaypipesmriedem: but right now, dinner...21:56
*** jmlowe has quit IRC21:56
mriedemhttps://review.openstack.org/#/c/508163/4/nova/tests/live_migration/hooks/run_tests.sh@6721:56
melwittoh, I see21:56
melwittI *think* that doesn't make a difference but we could do it just to be sure21:57
*** thorst has joined #openstack-nova21:58
mriedemmelwitt: yeah it might given the comment for VIR_MIGRATE_NON_SHARED_INC says "Migration with non-shared storage with incremental disk copy"22:00
mriedemso i guess i'll rev that test patch to see what happens22:00
melwittyeah, I think that'd be best. I'll keep an eye out for the result22:00
*** penick has quit IRC22:01
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test  https://review.openstack.org/50816322:01
*** penick has joined #openstack-nova22:01
*** thorst has quit IRC22:02
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks  https://review.openstack.org/50720222:02
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test  https://review.openstack.org/50816322:02
mriedemi don't know why it says it can't merge22:03
melwittI never figured out why it says that22:05
*** jmlowe has joined #openstack-nova22:06
*** penick has quit IRC22:06
*** oanson has joined #openstack-nova22:09
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103522:11
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103522:15
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103522:15
*** marst has quit IRC22:15
*** salv-orlando has joined #openstack-nova22:19
*** salv-orlando has quit IRC22:23
*** felipemonteiro_ has quit IRC22:23
*** rmcallis has quit IRC22:25
*** mlavalle has quit IRC22:25
*** slaweq has joined #openstack-nova22:25
*** salv-orlando has joined #openstack-nova22:26
*** AlexeyAbashkin has joined #openstack-nova22:29
*** AlexeyAbashkin has quit IRC22:33
*** lpetrut has quit IRC22:33
*** cdent has joined #openstack-nova22:43
cdentefried, jaypipes you see https://twitter.com/amitry/status/925112698102255616 ?22:44
efriedcdent I don't twit.22:44
efriedcdent How do I "ping" this guy?22:44
cdentone moment please22:45
efried(cdent I mean, without joining twitter)22:45
cdentyou can use me as a sock puppet if you like22:45
cdentI can give him your email address22:46
cdentif that works for yhou22:46
efriedcdent For sure.22:46
*** catintheroof has quit IRC22:48
*** lyan has quit IRC22:49
mikalmriedem: you still around?22:52
*** slaweq has quit IRC22:52
cdentefried: he’s already responded, says he’ll start an email thread22:54
efriednoyce, thanks.22:55
efriedThat twitter thing sure is responsive22:55
cdentdo not be tempted22:56
*** cdent has quit IRC22:59
*** jaypipes has quit IRC23:06
*** liverpooler has joined #openstack-nova23:09
melwittmriedem: I tried rebasing your tempest change and rechecking and it didn't work. sorry about that :(23:15
melwittit needs the -W reapplied https://review.openstack.org/#/c/481346/23:15
*** sdague has quit IRC23:16
*** yamamoto has joined #openstack-nova23:17
*** hongbin has quit IRC23:23
*** baoli has joined #openstack-nova23:25
*** dave-mccowan has joined #openstack-nova23:28
*** AlexeyAbashkin has joined #openstack-nova23:29
*** Swami has quit IRC23:30
*** lbragstad has quit IRC23:32
*** AlexeyAbashkin has quit IRC23:33
*** gyee has quit IRC23:33
*** takashin has quit IRC23:37
*** baoli has quit IRC23:38
*** thorst has joined #openstack-nova23:39
*** jmlowe has quit IRC23:45
*** vladikr has quit IRC23:45
*** vladikr has joined #openstack-nova23:45
*** esberglu_ has joined #openstack-nova23:51
openstackgerritmelanie witt proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks  https://review.openstack.org/50720223:51
openstackgerritmelanie witt proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test  https://review.openstack.org/50816323:51
melwittmriedem: there was a real merge conflict from the other ceph test improvement that got merged earlier today. oops23:52
*** ijw has quit IRC23:53
*** ijw has joined #openstack-nova23:54
*** esberglu has quit IRC23:54
openstackgerritEric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs  https://review.openstack.org/51515123:57
openstackgerritEric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec}  https://review.openstack.org/51522323:57
openstackgerritEric Fried proposed openstack/nova master: ResourceRequest.to_querystring()  https://review.openstack.org/51581123:57
efriedgibi ^23:57
*** aloga has quit IRC23:59
*** aloga has joined #openstack-nova23:59
*** jmlowe has joined #openstack-nova23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!