*** yangyapeng has quit IRC | 00:02 | |
*** markvoelker has quit IRC | 00:08 | |
*** thorst has joined #openstack-nova | 00:18 | |
*** thorst has quit IRC | 00:20 | |
*** tbachman_ has joined #openstack-nova | 00:22 | |
*** tbachman has quit IRC | 00:22 | |
*** tbachman_ is now known as tbachman | 00:22 | |
*** rmcallis has quit IRC | 00:35 | |
*** Jeffrey4l has quit IRC | 00:42 | |
*** Jeffrey4l has joined #openstack-nova | 00:42 | |
*** salv-orlando has joined #openstack-nova | 00:46 | |
*** salv-orlando has quit IRC | 00:51 | |
*** huanxie has joined #openstack-nova | 00:51 | |
*** TuanLA has joined #openstack-nova | 00:54 | |
*** hieulq has joined #openstack-nova | 01:02 | |
*** artom has quit IRC | 01:04 | |
*** markvoelker has joined #openstack-nova | 01:05 | |
*** jichen has joined #openstack-nova | 01:07 | |
*** catintheroof has joined #openstack-nova | 01:08 | |
*** yangyapeng has joined #openstack-nova | 01:11 | |
openstackgerrit | jichenjc proposed openstack/nova master: check query param for server groups function https://review.openstack.org/500347 | 01:12 |
---|---|---|
*** phuongnh has joined #openstack-nova | 01:15 | |
*** Shunli has joined #openstack-nova | 01:15 | |
*** liusheng has quit IRC | 01:20 | |
*** takashin has joined #openstack-nova | 01:20 | |
*** trungnv has joined #openstack-nova | 01:20 | |
*** catintheroof has quit IRC | 01:22 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | 01:26 |
*** TuanLA has quit IRC | 01:28 | |
*** yangyapeng has quit IRC | 01:29 | |
*** TuanLA has joined #openstack-nova | 01:29 | |
*** yangyapeng has joined #openstack-nova | 01:29 | |
*** markvoelker has quit IRC | 01:38 | |
*** namnh has joined #openstack-nova | 01:38 | |
*** zzzeek has quit IRC | 01:40 | |
*** yangyape_ has joined #openstack-nova | 01:40 | |
*** yangyapeng has quit IRC | 01:41 | |
*** hieulq has quit IRC | 01:50 | |
*** TuanLA has quit IRC | 01:50 | |
*** phuongnh has quit IRC | 01:50 | |
*** phuongnh has joined #openstack-nova | 01:51 | |
*** TuanLA has joined #openstack-nova | 01:51 | |
*** hieulq has joined #openstack-nova | 01:51 | |
*** cheneydc has joined #openstack-nova | 01:53 | |
*** itlinux has quit IRC | 02:10 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 02:10 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 02:11 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix an example in "Delete Assisted Volume Snapshot" https://review.openstack.org/514128 | 02:12 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579 | 02:12 |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - Enable cold migration with target host https://review.openstack.org/406707 | 02:13 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 02:13 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix test_get_volume_config method https://review.openstack.org/489467 | 02:13 |
*** tetsuro has joined #openstack-nova | 02:14 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Update document related to host aggregate https://review.openstack.org/514499 | 02:14 |
*** thorst has joined #openstack-nova | 02:21 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | 02:22 |
*** phuongnh has quit IRC | 02:23 | |
*** hieulq has quit IRC | 02:23 | |
*** TuanLA has quit IRC | 02:23 | |
*** namnh has quit IRC | 02:23 | |
*** namnh has joined #openstack-nova | 02:24 | |
*** TuanLA has joined #openstack-nova | 02:24 | |
*** phuongnh has joined #openstack-nova | 02:24 | |
*** hieulq has joined #openstack-nova | 02:24 | |
*** thorst has quit IRC | 02:25 | |
efried | alex_xu How about now? | 02:31 |
*** liusheng has joined #openstack-nova | 02:31 | |
efried | I 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 |
efried | That 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-nova | 02:35 | |
*** mriedem has quit IRC | 02:38 | |
*** felipemonteiro_ has joined #openstack-nova | 02:44 | |
*** salv-orlando has joined #openstack-nova | 02:47 | |
*** litao__ has joined #openstack-nova | 02:49 | |
*** Laurene has joined #openstack-nova | 02:49 | |
openstackgerrit | Yikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination https://review.openstack.org/516168 | 02:51 |
*** salv-orlando has quit IRC | 02:51 | |
*** annp has joined #openstack-nova | 02:53 | |
*** zsli_ has joined #openstack-nova | 02:57 | |
*** Tom_ has joined #openstack-nova | 03:00 | |
*** Shunli has quit IRC | 03:00 | |
openstackgerrit | Yikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination https://review.openstack.org/516168 | 03:02 |
*** gongysh has joined #openstack-nova | 03:06 | |
*** Tom_ has quit IRC | 03:08 | |
*** markvoelker has quit IRC | 03:09 | |
*** tetsuro has quit IRC | 03:16 | |
*** namnh has quit IRC | 03:18 | |
*** phuongnh has quit IRC | 03:18 | |
*** TuanLA has quit IRC | 03:18 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | 03:18 |
*** phuongnh has joined #openstack-nova | 03:19 | |
*** TuanLA has joined #openstack-nova | 03:19 | |
*** namnh has joined #openstack-nova | 03:19 | |
*** felipemonteiro_ has quit IRC | 03:21 | |
*** cheneydc_ has joined #openstack-nova | 03:25 | |
*** cheneydc has quit IRC | 03:29 | |
*** jichen has quit IRC | 03:38 | |
*** tetsuro has joined #openstack-nova | 03:38 | |
*** hieulq has quit IRC | 03:40 | |
*** TuanLA has quit IRC | 03:40 | |
*** TuanLA has joined #openstack-nova | 03:41 | |
*** hieulq has joined #openstack-nova | 03:41 | |
*** udesale has joined #openstack-nova | 03:45 | |
*** rcernin has quit IRC | 03:47 | |
*** rcernin_ has joined #openstack-nova | 03:47 | |
*** dave-mccowan has quit IRC | 03:48 | |
*** mdnadeem has joined #openstack-nova | 03:50 | |
*** stvnoyes has quit IRC | 03:51 | |
*** takashin has left #openstack-nova | 03:54 | |
*** vks1 has joined #openstack-nova | 04:03 | |
*** markvoelker has joined #openstack-nova | 04:06 | |
*** phuongnh has quit IRC | 04:13 | |
*** TuanLA has quit IRC | 04:13 | |
*** namnh has quit IRC | 04:13 | |
*** namnh has joined #openstack-nova | 04:14 | |
*** phuongnh has joined #openstack-nova | 04:14 | |
*** TuanLA has joined #openstack-nova | 04:14 | |
*** Dinesh_Bhor has joined #openstack-nova | 04:14 | |
*** thorst has joined #openstack-nova | 04:22 | |
*** thorst has quit IRC | 04:27 | |
*** huanxie has quit IRC | 04:27 | |
*** janki has joined #openstack-nova | 04:35 | |
*** cheneydc_ has quit IRC | 04:37 | |
*** markvoelker has quit IRC | 04:39 | |
alex_xu | efried: sorry, I'm on PTO last friday | 04:41 |
*** gongysh has quit IRC | 04:43 | |
*** cheneydc_ has joined #openstack-nova | 04:44 | |
*** salv-orlando has joined #openstack-nova | 04:48 | |
*** salv-orlando has quit IRC | 04:52 | |
*** jaosorior has joined #openstack-nova | 04:54 | |
*** huanxie has joined #openstack-nova | 04:58 | |
*** udesale__ has joined #openstack-nova | 04:59 | |
*** udesale has quit IRC | 04:59 | |
*** brault has joined #openstack-nova | 05:11 | |
*** bkopilov has joined #openstack-nova | 05:16 | |
*** brault has quit IRC | 05:16 | |
*** yangyape_ has quit IRC | 05:24 | |
*** yangyapeng has joined #openstack-nova | 05:25 | |
*** sridharg has joined #openstack-nova | 05:28 | |
*** markvoelker has joined #openstack-nova | 05:37 | |
*** gongysh has joined #openstack-nova | 05:41 | |
*** ratailor has joined #openstack-nova | 05:45 | |
*** trinaths has joined #openstack-nova | 05:46 | |
*** zhurong has joined #openstack-nova | 05:48 | |
*** rnoriega has quit IRC | 05:52 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 05:57 |
*** huanxie has quit IRC | 06:01 | |
*** Kevin_Zheng has quit IRC | 06:01 | |
*** lpetrut has joined #openstack-nova | 06:04 | |
*** avolkov has joined #openstack-nova | 06:04 | |
*** slaweq has quit IRC | 06:05 | |
*** hoonetorg has quit IRC | 06:05 | |
*** slaweq has joined #openstack-nova | 06:06 | |
*** salv-orlando has joined #openstack-nova | 06:08 | |
*** markvoelker has quit IRC | 06:10 | |
*** slaweq has quit IRC | 06:10 | |
*** hshiina has joined #openstack-nova | 06:11 | |
*** takashin has joined #openstack-nova | 06:13 | |
*** udesale__ has quit IRC | 06:14 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | 06:20 |
*** thorst has joined #openstack-nova | 06:23 | |
*** hoonetorg has joined #openstack-nova | 06:23 | |
*** krtaylor has quit IRC | 06:27 | |
*** thorst has quit IRC | 06:27 | |
*** hamzy has quit IRC | 06:28 | |
*** brault has joined #openstack-nova | 06:32 | |
*** cheneydc_ has quit IRC | 06:35 | |
*** slaweq has joined #openstack-nova | 06:37 | |
*** cheneydc_ has joined #openstack-nova | 06:38 | |
*** hamzy has joined #openstack-nova | 06:42 | |
*** krtaylor has joined #openstack-nova | 06:43 | |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM - testing check-requirements https://review.openstack.org/516195 | 06:45 |
*** salv-orlando has quit IRC | 06:46 | |
openstackgerrit | Yikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination https://review.openstack.org/516168 | 06:46 |
*** salv-orlando has joined #openstack-nova | 06:48 | |
*** lpetrut has quit IRC | 06:58 | |
*** spectr has joined #openstack-nova | 07:02 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580 | 07:04 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965 | 07:04 |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM - testing check-requirements https://review.openstack.org/516195 | 07:06 |
*** markvoelker has joined #openstack-nova | 07:07 | |
*** Oku_OS-away is now known as Oku_OS | 07:08 | |
*** brault has quit IRC | 07:08 | |
*** lpetrut has joined #openstack-nova | 07:09 | |
*** slaweq has quit IRC | 07:10 | |
openstackgerrit | Yikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination https://review.openstack.org/516168 | 07:14 |
*** slaweq has joined #openstack-nova | 07:16 | |
*** rcernin_ has quit IRC | 07:17 | |
*** Tom_ has joined #openstack-nova | 07:19 | |
*** AlexeyAbashkin has joined #openstack-nova | 07:23 | |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM - testing check-requirements https://review.openstack.org/516195 | 07:25 |
*** avolkov has quit IRC | 07:25 | |
*** avolkov has joined #openstack-nova | 07:26 | |
*** lajoskatona has joined #openstack-nova | 07:26 | |
*** brault has joined #openstack-nova | 07:32 | |
*** tesseract has joined #openstack-nova | 07:33 | |
*** salv-orlando has quit IRC | 07:35 | |
*** brault has quit IRC | 07:35 | |
*** salv-orlando has joined #openstack-nova | 07:36 | |
*** markvoelker has quit IRC | 07:40 | |
*** cheneydc_ has quit IRC | 07:40 | |
*** cheneydc_ has joined #openstack-nova | 07:40 | |
*** aarefiev_afk is now known as aarefiev | 07:41 | |
*** lpetrut has quit IRC | 07:45 | |
openstackgerrit | Yikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination https://review.openstack.org/516168 | 07:45 |
*** sahid has joined #openstack-nova | 07:48 | |
*** rgerganov has joined #openstack-nova | 07:49 | |
*** yamamoto has quit IRC | 07:55 | |
*** pow_2_30 is now known as gibi | 07:57 | |
*** salv-orlando has quit IRC | 07:59 | |
*** salv-orlando has joined #openstack-nova | 08:00 | |
gmann | alex_xu: can you check this - https://review.openstack.org/#/c/516104/ | 08:00 |
gmann | alex_xu: matt BP about adding PUT description in flavor will be much easy after this | 08:01 |
*** Tom_ has quit IRC | 08:01 | |
alex_xu | gmann: got it, I will check it before EOD | 08:01 |
*** Tom_ has joined #openstack-nova | 08:02 | |
*** salv-orl_ has joined #openstack-nova | 08:02 | |
gmann | alex_xu: thanks | 08:02 |
alex_xu | gmann: np | 08:02 |
openstackgerrit | Alex Xu proposed openstack/nova master: WIP: Add _get_providers_with_any_resource method https://review.openstack.org/514197 | 08:03 |
openstackgerrit | Alex Xu proposed openstack/nova master: Enable _get_usages_by_provider_and_rc not filter on the resource class id https://review.openstack.org/514649 | 08:03 |
openstackgerrit | Alex Xu proposed openstack/nova master: Add _get_traits_by_rp_ids method https://review.openstack.org/511184 | 08:03 |
openstackgerrit | Alex Xu proposed openstack/nova master: Add ProviderSummaryList object https://review.openstack.org/514198 | 08:03 |
openstackgerrit | Alex Xu proposed openstack/nova master: Add more caches for the ProviderSummary and ProviderSummaryList https://review.openstack.org/514199 | 08:03 |
openstackgerrit | Alex Xu proposed openstack/nova master: Add AggregatedProviderSummary and AggregatedProviderSummaryList objects https://review.openstack.org/514200 | 08:03 |
openstackgerrit | Alex Xu proposed openstack/nova master: Add has_resources method to the AggregatedProviderSummary object https://review.openstack.org/514650 | 08:04 |
openstackgerrit | Alex Xu proposed openstack/nova master: Add as_allocation_request_obj method to AggregatedProvierSummary obj https://review.openstack.org/514651 | 08:04 |
openstackgerrit | Alex Xu proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters https://review.openstack.org/479766 | 08:04 |
*** salv-orlando has quit IRC | 08:04 | |
*** gongysh has quit IRC | 08:05 | |
*** Tom_ has quit IRC | 08:06 | |
*** Tom_ has joined #openstack-nova | 08:11 | |
*** vivsoni__ has quit IRC | 08:14 | |
*** alexchadin has joined #openstack-nova | 08:15 | |
*** kuzko has quit IRC | 08:17 | |
*** kuzko has joined #openstack-nova | 08:20 | |
*** slaweq has quit IRC | 08:22 | |
*** rcernin has joined #openstack-nova | 08:22 | |
*** thorst has joined #openstack-nova | 08:24 | |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM - testing check-requirements https://review.openstack.org/516195 | 08:25 |
*** huanxie has joined #openstack-nova | 08:27 | |
*** jpena|off is now known as jpena | 08:27 | |
*** udesale has joined #openstack-nova | 08:27 | |
*** thorst has quit IRC | 08:28 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for aggregates (v1.1) https://review.openstack.org/505643 | 08:28 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for resource classes (v1.2) https://review.openstack.org/511182 | 08:28 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183 | 08:29 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 08:29 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 08:29 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 08:29 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 08:29 |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: [WIP] CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 08:29 |
*** arxcruz|pto is now known as arxcruz | 08:30 | |
*** brault has joined #openstack-nova | 08:31 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965 | 08:31 |
*** brault has quit IRC | 08:31 | |
*** salv-orl_ has quit IRC | 08:32 | |
*** _pewp_ has quit IRC | 08:32 | |
*** salv-orlando has joined #openstack-nova | 08:33 | |
*** salv-orlando has quit IRC | 08:33 | |
*** salv-orlando has joined #openstack-nova | 08:33 | |
*** _pewp_ has joined #openstack-nova | 08:34 | |
*** slaweq has joined #openstack-nova | 08:36 | |
*** markvoelker has joined #openstack-nova | 08:37 | |
*** evrardjp has quit IRC | 08:40 | |
*** evrardjp has joined #openstack-nova | 08:41 | |
*** gongysh has joined #openstack-nova | 08:46 | |
*** ragiman has joined #openstack-nova | 08:47 | |
*** moshele has joined #openstack-nova | 08:48 | |
*** alexchadin has quit IRC | 08:49 | |
*** vivsoni has joined #openstack-nova | 08:52 | |
*** Tom_ has quit IRC | 08:54 | |
*** Tom_ has joined #openstack-nova | 08:54 | |
openstackgerrit | David Rabel proposed openstack/nova master: VMware: add support for graceful shutdown of instances https://review.openstack.org/494169 | 08:54 |
*** ociuhandu has joined #openstack-nova | 08:55 | |
*** yamamoto has joined #openstack-nova | 08:55 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217 | 08:56 |
*** Tom_ has quit IRC | 08:58 | |
*** alexchadin has joined #openstack-nova | 09:04 | |
*** yamamoto has quit IRC | 09:04 | |
*** rabel has joined #openstack-nova | 09:04 | |
*** giuseppe1992 has joined #openstack-nova | 09:06 | |
*** slaweq has quit IRC | 09:07 | |
*** Tom_ has joined #openstack-nova | 09:08 | |
*** markvoelker has quit IRC | 09:10 | |
rabel | heyho! | 09:13 |
*** yamamoto has joined #openstack-nova | 09:13 | |
rabel | https://review.openstack.org/#/c/494169/ is ready for review (again) :) | 09:13 |
*** slaweq has joined #openstack-nova | 09:15 | |
*** evrardjp has quit IRC | 09:22 | |
*** evrardjp has joined #openstack-nova | 09:23 | |
*** alexchadin has quit IRC | 09:25 | |
*** alexchadin has joined #openstack-nova | 09:25 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add more functional test for placement.usage https://review.openstack.org/513264 | 09:27 |
openstackgerrit | jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458 | 09:28 |
openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 09:29 |
*** zsli_ has quit IRC | 09:29 | |
*** gmann is now known as gmann_afk | 09:29 | |
*** vks1 has quit IRC | 09:29 | |
*** vks1 has joined #openstack-nova | 09:31 | |
openstackgerrit | jichenjc proposed openstack/nova master: check query param for used_limits function https://review.openstack.org/499091 | 09:32 |
openstackgerrit | jichenjc proposed openstack/nova master: emove glance usage inside compute https://review.openstack.org/511392 | 09:36 |
openstackgerrit | jichenjc proposed openstack/nova master: Remove usage of glance.generate_image_url https://review.openstack.org/511397 | 09:36 |
openstackgerrit | jichenjc proposed openstack/nova master: Remove usage of glance.generate_image_url https://review.openstack.org/511397 | 09:38 |
*** lpetrut has joined #openstack-nova | 09:40 | |
*** edmondsw has joined #openstack-nova | 09:47 | |
*** slaweq has quit IRC | 09:48 | |
*** sdague has joined #openstack-nova | 09:50 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix missing metavar in aggregate-update https://review.openstack.org/512485 | 09:50 |
*** edmondsw has quit IRC | 09:52 | |
openstackgerrit | jichenjc proposed openstack/nova master: WIP: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | 09:53 |
*** slaweq has joined #openstack-nova | 09:53 | |
*** yamamoto has quit IRC | 09:54 | |
*** slaweq_ has joined #openstack-nova | 09:59 | |
*** gongysh has quit IRC | 10:00 | |
*** cheneydc__ has joined #openstack-nova | 10:02 | |
*** cheneydc__ has quit IRC | 10:02 | |
*** giuseppe1992 has quit IRC | 10:04 | |
*** cheneydc_ has quit IRC | 10:04 | |
*** zhurong has quit IRC | 10:05 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [Placement] Fix parameter order in placement API ref https://review.openstack.org/516233 | 10:07 |
*** markvoelker has joined #openstack-nova | 10:07 | |
*** yangyapeng has quit IRC | 10:08 | |
*** yangyapeng has joined #openstack-nova | 10:09 | |
*** Lirt has joined #openstack-nova | 10:11 | |
*** vivsoni has quit IRC | 10:11 | |
*** yangyapeng has quit IRC | 10:13 | |
*** yamamoto has joined #openstack-nova | 10:16 | |
*** annp has quit IRC | 10:17 | |
*** zhurong has joined #openstack-nova | 10:20 | |
*** udesale has quit IRC | 10:20 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Using --option ARGUMENT https://review.openstack.org/516238 | 10:22 |
*** moshele has quit IRC | 10:22 | |
*** vivsoni has joined #openstack-nova | 10:22 | |
*** yamamoto has quit IRC | 10:24 | |
*** thorst has joined #openstack-nova | 10:25 | |
*** jsheeren has joined #openstack-nova | 10:26 | |
*** slaweq has quit IRC | 10:27 | |
jsheeren | hi, 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 file | 10:27 |
*** moshele has joined #openstack-nova | 10:28 | |
jsheeren | if 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 folder | 10:28 |
jsheeren | has anyone seen this before? | 10:28 |
*** slaweq has joined #openstack-nova | 10:29 | |
*** thorst has quit IRC | 10:30 | |
*** xinliang has quit IRC | 10:30 | |
*** sshwarts has joined #openstack-nova | 10:31 | |
*** brault has joined #openstack-nova | 10:32 | |
*** brault has quit IRC | 10:37 | |
*** gszasz has joined #openstack-nova | 10:37 | |
*** sapcc-bot has quit IRC | 10:39 | |
*** mkoderer_ has joined #openstack-nova | 10:39 | |
*** sapcc-bot has joined #openstack-nova | 10:39 | |
*** dgonzalez_ has joined #openstack-nova | 10:39 | |
*** tpatzig_ has joined #openstack-nova | 10:39 | |
*** david_1 has joined #openstack-nova | 10:39 | |
*** TuanLA has quit IRC | 10:40 | |
*** markvoelker has quit IRC | 10:41 | |
*** mkoderer_ has quit IRC | 10:41 | |
*** dgonzalez_ has quit IRC | 10:41 | |
*** tpatzig_ has quit IRC | 10:41 | |
*** david_1 has quit IRC | 10:41 | |
*** xinliang has joined #openstack-nova | 10:42 | |
*** jsheeren has quit IRC | 10:43 | |
-openstackstatus- NOTICE: Zuul has been restarted due to an unexpected issue. Please recheck any jobs that were in progress | 10:46 | |
*** Tom_ has quit IRC | 10:48 | |
*** Tom_ has joined #openstack-nova | 10:51 | |
*** tbachman has quit IRC | 10:52 | |
*** aloga has quit IRC | 10:55 | |
*** mvk has quit IRC | 10:55 | |
*** Tom_ has quit IRC | 10:55 | |
*** aloga has joined #openstack-nova | 10:55 | |
*** pcaruana has joined #openstack-nova | 10:56 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771 | 10:58 |
*** erlon has joined #openstack-nova | 10:58 | |
*** slaweq has quit IRC | 11:01 | |
*** Laurene has quit IRC | 11:01 | |
*** slaweq has joined #openstack-nova | 11:02 | |
*** smatzek has joined #openstack-nova | 11:07 | |
*** nicolasbock has joined #openstack-nova | 11:07 | |
*** cdent has joined #openstack-nova | 11:11 | |
openstackgerrit | Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 11:16 |
*** namnh has quit IRC | 11:17 | |
*** moshele has quit IRC | 11:19 | |
*** sgordon has quit IRC | 11:21 | |
*** sgordon has joined #openstack-nova | 11:22 | |
*** yamamoto has joined #openstack-nova | 11:25 | |
*** salv-orlando has quit IRC | 11:28 | |
*** mvk has joined #openstack-nova | 11:29 | |
*** salv-orlando has joined #openstack-nova | 11:29 | |
*** takashin has left #openstack-nova | 11:29 | |
*** pchavva has joined #openstack-nova | 11:29 | |
*** alexchadin has quit IRC | 11:31 | |
*** alexchadin has joined #openstack-nova | 11:32 | |
*** yamamoto has quit IRC | 11:32 | |
*** dave-mccowan has joined #openstack-nova | 11:35 | |
*** slaweq has quit IRC | 11:36 | |
*** gszasz has quit IRC | 11:37 | |
*** markvoelker has joined #openstack-nova | 11:38 | |
*** yamamoto has joined #openstack-nova | 11:39 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 11:41 |
*** vks1 has quit IRC | 11:41 | |
*** tetsuro has quit IRC | 11:44 | |
*** slaweq has joined #openstack-nova | 11:45 | |
*** dillaman has joined #openstack-nova | 11:45 | |
*** hemna_ has joined #openstack-nova | 11:48 | |
*** vivsoni has quit IRC | 11:52 | |
efried | alex_xu So... my test case was buggy. That's a relief. | 11:53 |
*** gszasz has joined #openstack-nova | 11:57 | |
*** spectr has quit IRC | 11:59 | |
*** salv-orlando has quit IRC | 11:59 | |
*** spectr has joined #openstack-nova | 12:00 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Document the real behavior of notify_on_state_change https://review.openstack.org/516264 | 12:03 |
*** udesale has joined #openstack-nova | 12:03 | |
efried | alex_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 IRC | 12:05 | |
*** tbachman has joined #openstack-nova | 12:08 | |
openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 12:09 |
*** thorst has joined #openstack-nova | 12:09 | |
*** giuseppe1992 has joined #openstack-nova | 12:09 | |
*** edmondsw has joined #openstack-nova | 12:09 | |
*** moshele has joined #openstack-nova | 12:10 | |
alex_xu | efried: hey, i'm here | 12:10 |
efried | alex_xu Hi. I'm glad you figured out that bug in my test. | 12:11 |
efried | At some point I'd like to get some tips on how you debug this stuff. | 12:11 |
alex_xu | efried: I drop in the same trap before, that is really hard thing to remember | 12:11 |
efried | Ah, okay. | 12:11 |
*** markvoelker has quit IRC | 12:11 | |
efried | Well, I'll know for the future. | 12:11 |
efried | I was of course thinking my "add allocation" was actually going to *add* an allocation. | 12:12 |
efried | Like, why couldn't you have two separate allocations rows for the same RP/RC | 12:12 |
efried | But 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 |
efried | Anyway - 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_xu | efried: yea, something like that as i remember | 12:13 |
alex_xu | efried: I like your current way | 12:14 |
efried | Talking about this: https://review.openstack.org/#/c/514197/4/nova/tests/functional/db/test_resource_provider.py@2387 | 12:14 |
alex_xu | ah | 12:15 |
efried | You want to create separate RPs for each different way an inventory can be exclude-worthy? | 12:15 |
*** jaypipes has joined #openstack-nova | 12:15 | |
efried | Morning jaypipes ! | 12:16 |
efried | (sorry, too cheerful?) | 12:16 |
alex_xu | efried: we can create three RPs , one hasn;t enough total vcpu, one hasn't current max_unit, the last one reserved a lot | 12:17 |
efried | alex_xu And is the idea for those RPs to be included or excluded? | 12:17 |
*** kylek3h_ has quit IRC | 12:17 | |
*** moshele has quit IRC | 12:17 | |
alex_xu | efried: all of us are cheerful when seeing jaypipes online :) | 12:17 |
efried | alex_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_xu | efried: exclude | 12:17 |
efried | Okay. | 12:17 |
alex_xu | efried: we should create inventory which match the request for other resource | 12:18 |
*** slaweq has quit IRC | 12:18 | |
efried | Sorry, I didn't follow that one. | 12:18 |
alex_xu | efried: the reason is due to this https://review.openstack.org/#/c/514197/4/nova/objects/resource_provider.py@846 | 12:18 |
alex_xu | efried: your first VCPU inventory https://review.openstack.org/#/c/514197/4/nova/tests/functional/db/test_resource_provider.py@2381 doesn't have enough vcpus | 12:19 |
*** phuongnh has quit IRC | 12:19 | |
*** thorst_ has joined #openstack-nova | 12:19 | |
alex_xu | then the where conds will return False directly after https://review.openstack.org/#/c/514197/4/nova/objects/resource_provider.py@854 | 12:20 |
*** yangyapeng has joined #openstack-nova | 12:20 | |
efried | alex_xu But just for that one resource. | 12:20 |
alex_xu | that means the mem and disk invetories acutally weren't tested | 12:20 |
*** thorst__ has joined #openstack-nova | 12:20 | |
efried | Oh, I don't think that's true. I hope not, anyway. | 12:20 |
efried | I have a different RP later on that tests that case... | 12:21 |
efried | or I thought I did... | 12:21 |
*** jamesden_ has joined #openstack-nova | 12:21 | |
alex_xu | But the comment message said three inventory for three different failure case | 12:21 |
efried | alex_xu The one at L2389 | 12:22 |
efried | That one has one inventory that's good, one that's bad. | 12:22 |
efried | In that case, the 'bad' one is because the inventory is exhausted by allocations. | 12:22 |
efried | But that shouldn't matter, should it? | 12:22 |
jaypipes | efried, alex_xu: mornin, fellas! | 12:22 |
*** ociuhandu has quit IRC | 12:23 | |
*** thorst has quit IRC | 12:23 | |
alex_xu | efried: that shounds good | 12:23 |
*** jamesdenton has quit IRC | 12:24 | |
efried | alex_xu In any case, I can expand out all the conditions we're checking for in the SQL. | 12:24 |
efried | alex_xu But I still don't quite understand which way you're wanting to test. | 12:24 |
*** thorst_ has quit IRC | 12:24 | |
efried | alex_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_xu | efried: based on what you want to test | 12:26 |
efried | alex_xu Well, I'm not sure what your concern is. | 12:26 |
alex_xu | efried: 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 actually | 12:27 |
efried | alex_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-nova | 12:27 | |
*** ociuhandu has joined #openstack-nova | 12:28 | |
efried | alex_xu Oh, I think they must be. | 12:28 |
efried | alex_xu If you were to "fix" either one of those so they're viable, that RP would show up in the result. | 12:28 |
efried | They have to *all* fail in order to exclude this RP. | 12:28 |
efried | Don't they? | 12:28 |
jaypipes | efried, alex_xu: did you see my suggestion on renaming that "root providers" to "non-sharing providers"? | 12:29 |
*** markvoelker has joined #openstack-nova | 12:29 | |
*** jsheeren has joined #openstack-nova | 12:29 | |
alex_xu | efried: no, they are exclue, one RP hasn't enough VCPU, but has enough disk and mem, will be exclude | 12:29 |
efried | jaypipes I saw you "abstained" from my leetle survey, but I hadn't yet seen that, no. | 12:29 |
jaypipes | efried: I didn't abstain. I voted no on all options :) | 12:30 |
efried | jaypipes Cool. So you're in favor of "non-sharing"? | 12:30 |
jaypipes | efried: you fancy taking over the resource providers summary email for cdent the next 3 weeks? | 12:30 |
alex_xu | jaypipes: sorry, I didn't see that, where is it? | 12:31 |
jaypipes | efried: I am in favor of "non-sharing providers", yes | 12:31 |
jaypipes | alex_xu: in the commit message comment.. | 12:31 |
* jaypipes checks that he actually saved the darn gerrit comment.... | 12:31 | |
alex_xu | which patch....? | 12:31 |
jaypipes | alex_xu: https://review.openstack.org/#/c/480379/ | 12:32 |
efried | jaypipes 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 IRC | 12:32 | |
jaypipes | efried: well, doing the email would give you that handle, ya? :) | 12:32 |
efried | That's one way to look at it. | 12:32 |
jaypipes | hehe | 12:32 |
jaypipes | efried: we can split the work between the two of us if you'd prefer. | 12:33 |
jaypipes | efried: are you going to Sydney? | 12:33 |
alex_xu | jaypipes: yea, non-shared provider sounds the right way | 12:33 |
efried | jaypipes Fraid not. | 12:33 |
*** catintheroof has joined #openstack-nova | 12:33 | |
jaypipes | efried: I think you meant "Fried not". | 12:33 |
*** brault has joined #openstack-nova | 12:34 | |
jaypipes | geez, I am turning into my dad. | 12:34 |
efried | That's way better than I got in grade school. | 12:34 |
alex_xu | jaypipes: 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 right | 12:34 |
jaypipes | efried: I'm not going to Sydney either, so we can tackle the summary email between the two of us. | 12:34 |
efried | jaypipes Okay, sounds good to me. | 12:34 |
jaypipes | alex_xu: makes sense for compute nodes, yes, but eventually we'd like to detach the placement service from being compute-specific. | 12:35 |
alex_xu | jaypipes: yea | 12:35 |
efried | ++ | 12:35 |
efried | jaypipes I added 'non-sharing' as an option to the poll. | 12:36 |
*** hemna_ has quit IRC | 12:36 | |
alex_xu | jaypipes: 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-candidates | 12:36 |
efried | jaypipes Not that it's a democracy or anything. | 12:36 |
*** hemna_ has joined #openstack-nova | 12:36 | |
jsheeren | hi 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 9 | 12:37 | |
jaypipes | lol | 12:37 |
jaypipes | jsheeren: what's up? | 12:37 |
jsheeren | i'm running into an issue with instance resize of instances with a cinder backed root disk | 12:37 |
jsheeren | the resize does a resize_cleanup action, which checks if a root disk exists | 12:37 |
jsheeren | instances with a cinder root disk, do not have a nova root disk | 12:38 |
*** brault has quit IRC | 12:38 | |
jsheeren | so this code gets executed: https://github.com/openstack/nova/blob/stable/ocata/nova/virt/libvirt/driver.py#L1149 | 12:38 |
*** trinaths1 has joined #openstack-nova | 12:38 | |
jsheeren | the instance base path gets deleted | 12:38 |
*** trinaths has quit IRC | 12:38 | |
jsheeren | this 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 definition | 12:38 |
jsheeren | i think the issue is introduced after https://bugs.launchpad.net/nova/+bug/1666831 | 12:38 |
openstack | Launchpad bug 1666831 in OpenStack Compute (nova) ocata "Nova recreates instance directory after migration/resize" [Low,Fix committed] - Assigned to Lee Yarwood (lyarwood) | 12:38 |
efried | It's the 666 in that bug number. | 12:39 |
jaypipes | heh | 12:39 |
jsheeren | lol | 12:39 |
jaypipes | jsheeren: are you trying to resize the root disk along with the flavor? | 12:39 |
jsheeren | jaypipes: 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 quick | 12:41 |
jaypipes | jsheeren: just check if the root_gb of the flavor is different from a to b. | 12:41 |
jsheeren | yes, that is the case | 12:41 |
*** dtantsur|afk is now known as dtantsur | 12:42 | |
jaypipes | jsheeren: 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 IRC | 12:42 | |
jsheeren | jaypipes: thanks | 12:42 |
*** yamamoto has quit IRC | 12:43 | |
jaypipes | jsheeren: change the line so that it looks like this: | 12:45 |
jaypipes | if os.path.exists(inst_base) and not root_disk.exists() and not compute_utils.is_volume_backed_instance(instance._context, instance): | 12:45 |
jaypipes | jsheeren: restart the nova-compute service, retry your resize operation and lemme know if that fixes things. | 12:46 |
*** giuseppe1992 has quit IRC | 12:46 | |
jsheeren | jaypipes: ok, i will, thanks! | 12:47 |
*** trinaths1 has quit IRC | 12:47 | |
jaypipes | jsheeren: 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-nova | 12:47 | |
*** catintheroof has quit IRC | 12:47 | |
*** catintheroof has joined #openstack-nova | 12:49 | |
*** hemna_ has quit IRC | 12:49 | |
*** zhurong has quit IRC | 12:50 | |
*** vladikr has joined #openstack-nova | 12:54 | |
jsheeren | jaypipes: your patch works for us | 12:55 |
*** jpena is now known as jpena|lunch | 12:55 | |
*** kylek3h has joined #openstack-nova | 12:55 | |
jsheeren | we'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 issue | 12:56 |
jsheeren | jaypipes: thanks for the fast response AND fix! | 12:57 |
*** Tom_ has joined #openstack-nova | 12:57 | |
*** mriedem has joined #openstack-nova | 12:58 | |
*** Tom_ has quit IRC | 12:58 | |
*** rcernin has quit IRC | 12:58 | |
*** slaweq has quit IRC | 13:00 | |
*** salv-orlando has joined #openstack-nova | 13:00 | |
*** mvk has quit IRC | 13:01 | |
*** rnoriega has joined #openstack-nova | 13:02 | |
*** slaweq has joined #openstack-nova | 13:03 | |
*** rnoriega has quit IRC | 13:05 | |
*** Eran_Kuris has quit IRC | 13:05 | |
*** baoli has joined #openstack-nova | 13:05 | |
*** salv-orlando has quit IRC | 13:06 | |
*** janki has quit IRC | 13:06 | |
*** rnoriega has joined #openstack-nova | 13:07 | |
efried | jaypipes 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-nova | 13:09 | |
bhagyashris | Hi 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-nova | 13:11 | |
efried | bhagyashris You talking devstack? | 13:11 |
*** felipemonteiro has joined #openstack-nova | 13:11 | |
*** hemna_ has joined #openstack-nova | 13:11 | |
bhagyashris | efried: yes | 13:12 |
*** gongysh has joined #openstack-nova | 13:12 | |
*** gongysh has quit IRC | 13:12 | |
efried | bhagyashris 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 IRC | 13:14 | |
*** mvk has joined #openstack-nova | 13:16 | |
efried | bhagyashris sudo systemctl cat devstack@* | grep nova-cpu.conf <== For my devstack, this shows me only the nova-compute service. | 13:16 |
bhagyashris | efried: yes | 13:16 |
efried | bhagyashris Does that answer your question? | 13:17 |
bhagyashris | efried: yes | 13:17 |
bhagyashris | efried: thank you :) | 13:17 |
efried | bhagyashris You're welcome! | 13:17 |
*** gszasz has joined #openstack-nova | 13:17 | |
*** liverpooler has joined #openstack-nova | 13:21 | |
*** namnh has joined #openstack-nova | 13:21 | |
cdent | efried: 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 |
efried | cdent I agree. alex_xu and I have been debating it. | 13:25 |
*** vladikr has quit IRC | 13:25 | |
efried | cdent He raised a valid point, which was, "Give me a real world example where this could happen." | 13:25 |
*** lbragstad has quit IRC | 13:25 | |
cdent | always a useful point | 13:25 |
*** hshiina has quit IRC | 13:26 | |
efried | cdent 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 |
cdent | jinx! | 13:26 |
efried | hah! | 13:26 |
*** takashin has joined #openstack-nova | 13:26 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Use Migration object in ComputeManagerMigrationTestCase https://review.openstack.org/502126 | 13:26 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521 | 13:26 |
alex_xu | yea, finally, efried beats me | 13:27 |
cdent | efried: I think the api should allow for that possibility, even if compute nodes don’t or won't | 13:28 |
cdent | as we can’t see the future of placement | 13:28 |
*** superdan is now known as dansmith | 13:28 | |
cdent | and limiting placement for the models presented by today’s compute nodes is limiting | 13:28 |
efried | cdent 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 |
cdent | I still find that modeling hard to stomach, but yeah, if you’re doing it that way, I agree | 13:29 |
*** salv-orlando has joined #openstack-nova | 13:30 | |
efried | Anyway, 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 IRC | 13:30 | |
*** smatzek has quit IRC | 13:30 | |
*** amodi has joined #openstack-nova | 13:30 | |
*** smatzek has joined #openstack-nova | 13:30 | |
*** smatzek has quit IRC | 13:31 | |
efried | I 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 |
efried | alex_xu So what's the path forward from here? Looks like we need to rework the logic some more. | 13:32 |
jaypipes | jsheeren: awesomesauce. I'll file a bug :) | 13:32 |
alex_xu | efried: emm...feel like, https://review.openstack.org/#/c/514197 should be revert to the patchset2? | 13:33 |
*** lbragstad has joined #openstack-nova | 13:33 | |
efried | alex_xu But with the new test case? | 13:33 |
jaypipes | efried: if there's no constraints saying that all three RCs need to be provided by the same RP, then sure. | 13:34 |
efried | jaypipes ++ | 13:34 |
alex_xu | efried: yea | 13:34 |
efried | alex_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 |
efried | alex_xu If it doesn't, you get to fix the SQL :) | 13:35 |
alex_xu | efried: cool, thanks | 13:35 |
alex_xu | ok :) | 13:35 |
efried | cause I'm still too rusty | 13:35 |
efried | Did I mention the last time I did any remotely serious SQL was circa 2003? | 13:35 |
efried | Even then I didn't do anything this complicated. | 13:36 |
*** efried is now known as efried_brb | 13:36 | |
*** slaweq has quit IRC | 13:36 | |
*** vks1 has joined #openstack-nova | 13:37 | |
*** mlavalle has joined #openstack-nova | 13:37 | |
alex_xu | efried_brb: yea, without jaypipes, I don't know the SQL can't such complicated :) | 13:37 |
*** vladikr has joined #openstack-nova | 13:38 | |
* alex_xu is exhausted, and want to escape the scheduler meeting | 13:38 | |
*** yamamoto has joined #openstack-nova | 13:39 | |
*** thorst__ has quit IRC | 13:40 | |
edleafe | alex_xu: if you skip the meeting, we'll assign all the tasks to you! :) | 13:41 |
*** gcb has joined #openstack-nova | 13:41 | |
jaypipes | alex_xu: no worries, we can do the conversation on your patches. | 13:41 |
*** slaweq has joined #openstack-nova | 13:42 | |
Dinesh_Bhor | cdent: Hi, do you have 5 min? I have a question: http://paste.openstack.org/show/624973/ | 13:43 |
*** brault has joined #openstack-nova | 13:44 | |
cdent | hi Dinesh_Bhor, reading that | 13:44 |
edleafe | Scheduler subteam meeting in 15 minutes in #openstack-meeting-alt | 13:45 |
cdent | Dinesh_Bhor: let’s take it to #openstack-sdks | 13:45 |
*** efried_brb is now known as efried | 13:46 | |
*** awaugama has joined #openstack-nova | 13:48 | |
*** brault has quit IRC | 13:48 | |
*** amodi has quit IRC | 13:49 | |
*** tbachman has quit IRC | 13:51 | |
*** ratailor has quit IRC | 13:51 | |
jaypipes | efried: luckily, the SQL language has not changed too significantly since 2003. :) | 13:56 |
efried | jaypipes And we seem to be using a subset of it anyway, in case different backends. | 13:57 |
*** jpena|lunch is now known as jpena | 13:57 | |
jaypipes | efried: ack | 13:57 |
*** armax has joined #openstack-nova | 13:58 | |
*** jmlowe has joined #openstack-nova | 13:59 | |
edleafe | efried: and sqla makes it *so* easy!! | 13:59 |
efried | edleafe 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-nova | 14:00 | |
*** alexchadin has quit IRC | 14:00 | |
*** hongbin has joined #openstack-nova | 14:00 | |
edleafe | Because ORMs insulate us from the rough edges of SQL, and make life easy! | 14:00 |
*** amodi has joined #openstack-nova | 14:01 | |
edleafe | Scheduler subteam meeting running now in #openstack-meeting-alt | 14:01 |
*** salv-orlando has quit IRC | 14:02 | |
*** Eran_Kuris has joined #openstack-nova | 14:02 | |
*** smatzek has joined #openstack-nova | 14:03 | |
*** smatzek has quit IRC | 14:03 | |
*** smatzek has joined #openstack-nova | 14:04 | |
*** smatzek has quit IRC | 14:04 | |
*** brault has joined #openstack-nova | 14:05 | |
*** smatzek has joined #openstack-nova | 14:05 | |
jaypipes | jsheeren: feel free to provide color commentary on https://bugs.launchpad.net/nova/+bug/1728603 :) | 14:06 |
openstack | Launchpad bug 1728603 in OpenStack Compute (nova) "Resize a boot-from-volume instance with NFS destroys instance" [High,Confirmed] | 14:06 |
*** coreywright has quit IRC | 14:08 | |
mriedem | jaypipes: guh, and that change was backported to ocata https://review.openstack.org/#/c/441037/ | 14:09 |
*** brault has quit IRC | 14:09 | |
mriedem | we do have an NFS job in the nova experimental queue, not sure if it tests resize though - but probably should | 14:10 |
*** marst has joined #openstack-nova | 14:11 | |
*** alexchadin has joined #openstack-nova | 14:13 | |
*** lbragstad has quit IRC | 14:13 | |
*** lbragstad has joined #openstack-nova | 14:13 | |
*** tbachman has joined #openstack-nova | 14:17 | |
*** lbragstad has quit IRC | 14:18 | |
*** coreywright has joined #openstack-nova | 14:21 | |
*** lbragstad has joined #openstack-nova | 14:22 | |
*** salv-orlando has joined #openstack-nova | 14:24 | |
*** felipemonteiro has quit IRC | 14:30 | |
*** thorst has joined #openstack-nova | 14:30 | |
*** thorst has quit IRC | 14:31 | |
*** thorst has joined #openstack-nova | 14:31 | |
*** amodi has quit IRC | 14:33 | |
*** Apoorva has joined #openstack-nova | 14:33 | |
cdent | dtantsur: tomorrow’s fine for traits for me too | 14:34 |
dtantsur | nice! sorry all | 14:34 |
dtantsur | moving to a new country and new flat means sudden appointments here and there.. | 14:34 |
*** vishwanathj has joined #openstack-nova | 14:34 | |
*** udesale has quit IRC | 14:37 | |
*** READ10 has joined #openstack-nova | 14:37 | |
*** artom has joined #openstack-nova | 14:42 | |
jaypipes | dtantsur: not a problem, man. :) | 14:42 |
edleafe | dtantsur: I can make it tomorrow, too | 14:43 |
jaypipes | dtantsur: it's 3pm UTC yeah? | 14:44 |
bhagyashris | Hi All, | 14:45 |
dtantsur | jaypipes: I think so, yeah | 14:45 |
bhagyashris | I am checking the below scenario by enabling the AggregateImagePropertiesIsolation filter | 14:45 |
bhagyashris | Created two host aggregate groups and add one compute node to each | 14:45 |
bhagyashris | Host aggregate group A ->add 'os_type=license_ubuntu' host aggregate metadata | 14:45 |
bhagyashris | Host aggregate group B-> add 'k1=v1' host aggregate metadata | 14:45 |
*** amodi has joined #openstack-nova | 14:45 | |
bhagyashris | created the two images by setting the properties | 14:45 |
jaypipes | mriedem: lol. forever in bluejeans. | 14:45 |
bhagyashris | Image1 -> add 'os_type=license_ubuntu' | 14:46 |
bhagyashris | Image2 -> add 'k1=v1' | 14:46 |
bhagyashris | created the inatence using the Image2 so it should boot on Host aggregate group B but it's booting on Host aggregate group A | 14:46 |
*** slaweq has quit IRC | 14:46 | |
bhagyashris | I have also set the aggregate_image_properties_isolation_namespace = os_type_1 | 14:47 |
bhagyashris | aggregate_image_properties_isolation_separator = = in /etc/nova/nova.conf on both the Host | 14:47 |
*** zzzeek has joined #openstack-nova | 14:47 | |
bhagyashris | s/inatence/instance | 14:48 |
jsheeren | jaypipes: :) | 14:49 |
mriedem | jaypipes: i assume dtantsur will get any and all neil diamond references | 14:49 |
jaypipes | heh | 14:49 |
*** slaweq has joined #openstack-nova | 14:49 | |
dtantsur | :) | 14:50 |
*** dikonoor has joined #openstack-nova | 14:50 | |
jsheeren | jaypipes: fixed my name, rest looks good and as we experienced it | 14:50 |
*** mvk has quit IRC | 14:51 | |
jaypipes | jsheeren: oops, sorry about that! | 14:51 |
bauzas | wow, just found http://45.55.105.55:3000/dashboard/db/openstack-bugs | 14:56 |
* bauzas loves markus_z | 14:56 | |
jsheeren | no problem! i'm used to fixing my name, it's that ' that messes with most people though :) | 14:59 |
*** lyan has quit IRC | 14:59 | |
*** gcb has quit IRC | 15:00 | |
*** yamahata has joined #openstack-nova | 15:01 | |
*** lyan has joined #openstack-nova | 15:03 | |
*** salv-orlando has quit IRC | 15:04 | |
*** alexchadin has quit IRC | 15:04 | |
efried | jsheeren What's the provenance of that name? (I'm a collector.) | 15:04 |
efried | Flemish? | 15:05 |
*** salv-orlando has joined #openstack-nova | 15:06 | |
*** esberglu has joined #openstack-nova | 15:07 | |
*** esberglu has quit IRC | 15:07 | |
*** esberglu has joined #openstack-nova | 15:07 | |
*** takashin has left #openstack-nova | 15:12 | |
*** mvk has joined #openstack-nova | 15:13 | |
jsheeren | Flemish indeed | 15:14 |
jsheeren | we're a rare breed: http://forebears.io/surnames/s-heeren | 15:15 |
jsheeren | hah apparently i have family in Canada.. hmm | 15:16 |
bauzas | mriedem: any patch up for https://specs.openstack.org/openstack/nova-specs/priorities/queens-priorities.html ? | 15:16 |
mriedem | bauzas: not yet but i will | 15:16 |
bauzas | mriedem: I'm just writing a presentation and I'd love to speak about priorities | 15:16 |
mriedem | for sydney? | 15:16 |
bauzas | yup | 15:16 |
mriedem | which presentation is that? | 15:16 |
bauzas | bug triage | 15:17 |
mriedem | what does this have to do with that? | 15:17 |
bauzas | nothing really related | 15:17 |
bauzas | but just explaining about our tags | 15:17 |
mriedem | umm, ok | 15:17 |
bauzas | but meh | 15:17 |
jaypipes | efried: what happened to the patch about parsing numbered request groups that I commented on? | 15:17 |
bauzas | mriedem: no worries, I'll just pass a list of tags | 15:17 |
*** itlinux has joined #openstack-nova | 15:18 | |
efried | jaypipes 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 |
mriedem | bauzas: were you going to drop your -1 on this? https://review.openstack.org/#/c/515530 | 15:19 |
jaypipes | efried: k, thanks, yeah, that's the one. I was looking at https://review.openstack.org/#/c/514091/ and wondering where it was... | 15:20 |
efried | jaypipes Yeah, that one's coming from the placement side. | 15:20 |
mriedem | bauzas: also fyi in case you didn't notice this from the weekend https://review.openstack.org/#/c/515530/ | 15:20 |
jaypipes | efried: k | 15:20 |
efried | jaypipes Also needs a review, if you please :) | 15:20 |
bauzas | mriedem: yup, just forgot to remove it | 15:20 |
jaypipes | efried: yup, I'm on it. just finished reviewing cdent's series on POST /allocations. Moving on to yours. | 15:20 |
efried | jaypipes Thanks. | 15:21 |
jaypipes | edleafe is still reworking the Selection object series, right? | 15:21 |
mriedem | bauzas: i think we should try and get https://review.openstack.org/#/c/515530/ into newton before eol | 15:21 |
mriedem | since we shipped a broken feature basically | 15:21 |
edleafe | jaypipes: the object is fine. It's the RPC change that's being re-worked | 15:22 |
*** slaweq has quit IRC | 15:22 | |
*** salv-orl_ has joined #openstack-nova | 15:23 | |
jaypipes | edleafe: gotcha. | 15:24 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 15:25 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 15:25 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 15:25 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 15:25 |
jaypipes | dansmith, 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 IRC | 15:27 | |
mriedem | you can't tempt me with halloween candy | 15:29 |
*** AlexeyAbashkin has quit IRC | 15:30 | |
*** AlexeyAbashkin has joined #openstack-nova | 15:31 | |
*** fragatina has joined #openstack-nova | 15:31 | |
*** lajoskatona has left #openstack-nova | 15:33 | |
*** stvnoyes has joined #openstack-nova | 15:33 | |
*** jsheeren has quit IRC | 15:34 | |
*** jsheeren has joined #openstack-nova | 15:34 | |
efried | alex_xu Still awake? | 15:34 |
*** AlexeyAbashkin has quit IRC | 15:35 | |
*** jsheeren has left #openstack-nova | 15:35 | |
cdent | jaypipes: I don’t think we want zero value resources? See my response on https://review.openstack.org/#/c/500073/27 | 15:35 |
*** felipemonteiro has joined #openstack-nova | 15:36 | |
*** catintheroof has quit IRC | 15:37 | |
*** felipemonteiro_ has joined #openstack-nova | 15:38 | |
*** gyee has joined #openstack-nova | 15:38 | |
*** yamamoto has quit IRC | 15:40 | |
jaypipes | cdent: yeah, guess I was confused. | 15:40 |
namnh | jaypipes: 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 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add microversion to allow setting flavor description https://review.openstack.org/515213 | 15:41 |
jaypipes | cdent: the magic zeroed allocations are only internal to the AllocationList.create_all() method then... | 15:41 |
mriedem | namnh: that's a question for lyarwood probably | 15:41 |
*** yamamoto has joined #openstack-nova | 15:41 | |
*** felipemonteiro has quit IRC | 15:42 | |
jaypipes | namnh: yeah, especially since I won't be in Sydney :) | 15:42 |
cdent | yes, 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-nova | 15:42 | |
jaypipes | cdent: k, cool with me then. | 15:43 |
cdent | ✔\ | 15:43 |
mriedem | namnh: should eventually show up here https://wiki.openstack.org/wiki/Forum/Sydney2017 | 15:43 |
namnh | mriedem: I asked him, but he does not know :) I've just asked Arkady, he suggest me to ask Jay :( | 15:44 |
mriedem | who is the actual session leader? | 15:44 |
mriedem | https://www.openstack.org/summit/sydney-2017/summit-schedule/events/20451/fast-forward-upgrades | 15:45 |
mriedem | eric mccormick | 15:45 |
mriedem | find him | 15:45 |
*** yamamoto has quit IRC | 15:45 | |
*** yamamoto has joined #openstack-nova | 15:45 | |
* mriedem asks in -operators | 15:45 | |
namnh | mriedem: according to this mailling-list http://lists.openstack.org/pipermail/openstack-dev/2017-September/122361.html . that is Arkady | 15:46 |
mriedem | great... | 15:47 |
mriedem | dansmith: is lyarwood not going to be at the summit? | 15:47 |
dansmith | I don't think so | 15:47 |
mdbooth | melwitt: mriedem Any chance we can merge this one: https://review.openstack.org/#/c/507202/ | 15:47 |
mriedem | ok and the person moderating the session is sending people to jay...who won't be at the summit, nor is involved in fast forward upgrades | 15:48 |
namnh | mriedem: i asked both of them, it seems there is no a list of members for now | 15:48 |
mdbooth | Not sure what the current hold up is, but it's still a data corruptor | 15:48 |
mriedem | mdbooth: i believe the hold up is getting a run with https://review.openstack.org/#/c/508163/2 | 15:49 |
mriedem | in the live migration job to make sure nothing is regressed | 15:49 |
mriedem | namnh: ok i'll ask in the ML then | 15:49 |
namnh | mriedem: thanks in advance, because I'd like to know a list of members to prepare something for discussion :) | 15:50 |
mdbooth | mriedem: Thanks. I see melwitt spammed it a few times already. I've spammed it again. | 15:51 |
namnh | mriedem: are you joining FFU-session? | 15:51 |
mdbooth | Eurgh, gonna have to rebase it | 15:52 |
mriedem | namnh: idk | 15:52 |
mriedem | FFU isn't really a focus area of mine | 15:52 |
*** slaweq has joined #openstack-nova | 15:53 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 15:53 |
namnh | mriedem: i see, thanks for your time | 15:53 |
openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202 | 15:55 |
openstackgerrit | Matthew Booth proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163 | 15:55 |
namnh | jaypipes: :)) i see | 15:55 |
*** slaweq has quit IRC | 15:57 | |
*** spectr has quit IRC | 16:02 | |
*** ijw has joined #openstack-nova | 16:03 | |
*** catintheroof has joined #openstack-nova | 16:07 | |
*** fragatina has quit IRC | 16:07 | |
*** fragatina has joined #openstack-nova | 16:08 | |
*** Oku_OS is now known as Oku_OS-away | 16:09 | |
mriedem | dansmith: fyi added starter etherpads for the nova sessions i'm aware of https://wiki.openstack.org/wiki/Forum/Sydney2017#List_of_Sessions_.26_Etherpads | 16:10 |
dansmith | mriedem: cool | 16:10 |
*** itlinux has quit IRC | 16:12 | |
*** catintheroof has quit IRC | 16:12 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 16:14 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 16:14 |
*** ircuser-1 has quit IRC | 16:15 | |
*** fragatina has quit IRC | 16:17 | |
*** jamesden_ has quit IRC | 16:22 | |
*** jamesdenton has joined #openstack-nova | 16:23 | |
*** sridharg has quit IRC | 16:26 | |
*** slaweq has joined #openstack-nova | 16:27 | |
*** rmcallis has joined #openstack-nova | 16:28 | |
*** fragatina has joined #openstack-nova | 16:30 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: rp: break functions out of _set_traits() https://review.openstack.org/509908 | 16:32 |
*** fragatina has quit IRC | 16:33 | |
*** fragatina has joined #openstack-nova | 16:33 | |
*** yamahata has quit IRC | 16:34 | |
openstackgerrit | Doug Hellmann proposed openstack/nova master: add whereto for testing redirect rules https://review.openstack.org/516385 | 16:36 |
openstackgerrit | Doug Hellmann proposed openstack/nova master: add whereto for testing redirect rules https://review.openstack.org/516385 | 16:37 |
*** ragiman has quit IRC | 16:38 | |
efried | jaypipes 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 |
efried | I'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-nova | 16:41 | |
jaypipes | efried: no, the request group just means "in the same provider tree". | 16:41 |
efried | jaypipes This was the whole thing the spec was about. | 16:41 |
efried | jaypipes Same provider tree doesn't work for e.g. a VF and its bandwidth. | 16:41 |
*** Swami has joined #openstack-nova | 16:41 | |
efried | jaypipes See use cases in the spec. | 16:42 |
jaypipes | efried: Sorry, by "same provider tree" I mean "same provider or its descendants" | 16:42 |
jaypipes | efried: and each request group means exactly the same thing. | 16:42 |
efried | jaypipes Whoah. That's totally not what happened in the spec. To be continued... | 16:43 |
*** efried is now known as efried_rollin | 16:43 | |
*** jpena is now known as jpena|brb | 16:44 | |
*** markvoelker_ has joined #openstack-nova | 16:47 | |
*** markvoelker has quit IRC | 16:49 | |
*** fragatina has quit IRC | 16:49 | |
*** rmcallis has quit IRC | 16:49 | |
*** namnh has quit IRC | 16:49 | |
*** slaweq has quit IRC | 16:50 | |
*** namnh has joined #openstack-nova | 16:50 | |
*** slaweq has joined #openstack-nova | 16:50 | |
*** markvoelker_ has quit IRC | 16:51 | |
*** catintheroof has joined #openstack-nova | 16:52 | |
*** rmcallis has joined #openstack-nova | 16:53 | |
*** tesseract has quit IRC | 16:55 | |
*** slaweq has quit IRC | 16:55 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: do not remove inst_base when volume-backed during resize https://review.openstack.org/516395 | 16:55 |
mriedem | jaypipes: this should allow reproducing that bug in the NFS job ^ | 16:55 |
*** gszasz has quit IRC | 16:56 | |
*** catintheroof has quit IRC | 16:56 | |
*** markvoelker has joined #openstack-nova | 17:00 | |
*** markvoelker has quit IRC | 17:05 | |
*** yamahata has joined #openstack-nova | 17:07 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Pass the correct image to build_request_spec in conductor.rebuild_instance https://review.openstack.org/516400 | 17:07 |
*** Apoorva has quit IRC | 17:10 | |
*** stvnoyes has quit IRC | 17:11 | |
*** namnh has quit IRC | 17:11 | |
*** harlowja has joined #openstack-nova | 17:19 | |
*** mvk has quit IRC | 17:20 | |
*** kfarr has joined #openstack-nova | 17:21 | |
*** jpena|brb is now known as jpena | 17:21 | |
*** lpetrut has quit IRC | 17:22 | |
*** fragatina has joined #openstack-nova | 17:22 | |
*** markvoelker has joined #openstack-nova | 17:23 | |
*** catintheroof has joined #openstack-nova | 17:27 | |
*** markvoelker has quit IRC | 17:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Pass the correct image to build_request_spec in conductor.rebuild_instance https://review.openstack.org/516404 | 17:29 |
*** markvoelker has joined #openstack-nova | 17:30 | |
*** AlexeyAbashkin has joined #openstack-nova | 17:30 | |
*** nikhil has joined #openstack-nova | 17:31 | |
*** itlinux has joined #openstack-nova | 17:31 | |
*** Apoorva has joined #openstack-nova | 17:32 | |
*** Apoorva has quit IRC | 17:32 | |
*** catintheroof has quit IRC | 17:32 | |
*** Apoorva has joined #openstack-nova | 17:32 | |
*** vks1 has quit IRC | 17:34 | |
*** markvoelker has quit IRC | 17:34 | |
*** slaweq has joined #openstack-nova | 17:34 | |
*** AlexeyAbashkin has quit IRC | 17:35 | |
*** catintheroof has joined #openstack-nova | 17:37 | |
*** slaweq has quit IRC | 17:39 | |
*** catintheroof has quit IRC | 17:42 | |
*** slaweq has joined #openstack-nova | 17:45 | |
*** sahid has quit IRC | 17:45 | |
*** jmlowe has quit IRC | 17:47 | |
*** sshwarts has quit IRC | 17:49 | |
*** purplerbot has quit IRC | 17:50 | |
*** purplerbot has joined #openstack-nova | 17:51 | |
*** lpetrut has joined #openstack-nova | 17:56 | |
openstackgerrit | Merged openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036 | 17:59 |
*** ijw has quit IRC | 18:00 | |
*** avolkov` has joined #openstack-nova | 18:01 | |
*** ijw has joined #openstack-nova | 18:01 | |
*** dikonoor has quit IRC | 18:02 | |
*** avolkov has quit IRC | 18:02 | |
*** jmlowe has joined #openstack-nova | 18:04 | |
*** markvoelker has joined #openstack-nova | 18:05 | |
*** nicolasbock has quit IRC | 18:05 | |
*** slaweq has quit IRC | 18:08 | |
*** markvoelker has quit IRC | 18:09 | |
*** markvoelker has joined #openstack-nova | 18:10 | |
*** namnh has joined #openstack-nova | 18:12 | |
*** slaweq has joined #openstack-nova | 18:13 | |
*** namnh has quit IRC | 18:17 | |
*** slaweq has quit IRC | 18:18 | |
*** jpena is now known as jpena|off | 18:18 | |
*** markvoelker has quit IRC | 18:19 | |
*** markvoelker has joined #openstack-nova | 18:19 | |
openstackgerrit | David Rabel proposed openstack/nova master: Fix format in live-migration-usage.rst https://review.openstack.org/516416 | 18:19 |
*** markvoelker_ has joined #openstack-nova | 18:20 | |
*** markvoelker has quit IRC | 18:23 | |
*** markvoelker has joined #openstack-nova | 18:32 | |
*** markvoelker_ has quit IRC | 18:33 | |
*** markvoelker has quit IRC | 18:37 | |
*** efried_rollin is now known as efried | 18:37 | |
*** dtantsur is now known as dtantsur|afk | 18:40 | |
*** felipemonteiro_ has quit IRC | 18:41 | |
*** markvoelker has joined #openstack-nova | 18:41 | |
*** catintheroof has joined #openstack-nova | 18:43 | |
*** mikal has quit IRC | 18:43 | |
*** felipemonteiro has joined #openstack-nova | 18:44 | |
*** felipemonteiro_ has joined #openstack-nova | 18:45 | |
*** catintheroof has quit IRC | 18:48 | |
*** felipemonteiro has quit IRC | 18:49 | |
*** catintheroof has joined #openstack-nova | 18:49 | |
*** salv-orlando has joined #openstack-nova | 18:52 | |
*** mvk has joined #openstack-nova | 18:53 | |
*** pchavva has quit IRC | 18:53 | |
*** ijw has quit IRC | 18:54 | |
*** mikal has joined #openstack-nova | 18:55 | |
*** salv-orl_ has quit IRC | 18:55 | |
*** baoli has quit IRC | 18:59 | |
efried | jaypipes Now a good time? | 18:59 |
*** jdurgin has joined #openstack-nova | 19:00 | |
*** ijw has joined #openstack-nova | 19:06 | |
*** baoli has joined #openstack-nova | 19:09 | |
*** AlexeyAbashkin has joined #openstack-nova | 19:10 | |
jaypipes | efried: sure | 19:13 |
efried | jaypipes 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 |
efried | jaypipes Then we can go into whether the spec is wrong, and what to do about that. | 19:14 |
efried | jaypipes I was just going over the comments on the spec myself... | 19:14 |
efried | https://review.openstack.org/#/c/510244/ | 19:14 |
efried | jaypipes 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 IRC | 19:16 | |
*** READ10 has quit IRC | 19:17 | |
jaypipes | efried: 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 |
efried | jaypipes In which case we nevertheless need to keep them semantically separate when processing them, nah? | 19:21 |
efried | jaypipes 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 |
jaypipes | efried: sure, but I'd prefer we just annotate the RequestGroup with a "can_use_shared_resources" attribute or similar | 19:22 |
efried | jaypipes Oh, rather than calling it 'numbered' vs 'unnumbered', sure I'm open to better ways of describing. | 19:22 |
jaypipes | right | 19:22 |
efried | jaypipes But IIUC, we can't share an object between the scheduler side and the placement side, can we? | 19:22 |
jaypipes | cuz numbered and unnumbered doesn't indicate what the diff is. | 19:23 |
jaypipes | that was my complaint. | 19:23 |
jaypipes | efried: 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 |
jaypipes | that's what I was trying to say | 19:24 |
efried | jaypipes Cool, I'm good with that for sure. | 19:24 |
efried | jaypipes Back to the other issue... | 19:24 |
jaypipes | efried: 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 |
efried | jaypipes Dig | 19:25 |
jaypipes | k, cool. | 19:25 |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM - testing check-requirements https://review.openstack.org/516195 | 19:25 |
efried | jaypipes 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 |
efried | jaypipes Bah, I meant the "sharing" group. | 19:25 |
efried | Cause this wasn't confusing enough. | 19:25 |
efried | jaypipes Per the spec, the "non-sharing" group is limited to getting its resources from exactly one RP. No trees, sub-branches, etc. | 19:26 |
jaypipes | efried: ok, if that's what was agreed to in the spec, cool. | 19:27 |
efried | jaypipes 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 |
efried | Cleaner to keep it same-RP. | 19:27 |
jaypipes | right | 19:28 |
jaypipes | efried: 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 |
efried | jaypipes Dig. | 19:29 |
jaypipes | coolio. | 19:29 |
efried | jaypipes So do you want me to have a (separately-defined) RequestGroup class on the placement side of the world as well? | 19:29 |
efried | Not a DB object, obviously; just a util/helper. | 19:30 |
mriedem | sdague: want to send this through? https://review.openstack.org/#/c/508271/ | 19:33 |
mriedem | otherwise i'm just going to self approve | 19:33 |
*** markvoelker has quit IRC | 19:34 | |
*** markvoelker has joined #openstack-nova | 19:34 | |
dansmith | mriedem: I got it | 19:35 |
mriedem | thanks | 19:35 |
dansmith | sorry I wasn't paying attention to that | 19:35 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163 | 19:36 |
jaypipes | efried: yeah, prolly. | 19:38 |
efried | jaypipes ight. | 19:39 |
jaypipes | efried: you can put it in nova/api/openstack/placement/ | 19:39 |
efried | ack | 19:39 |
jaypipes | efried: and the other in /nova/scheduler/ somewhere | 19:39 |
efried | jaypipes It's in utils at the moment, will leave it there. | 19:39 |
jaypipes | efried: should just be a POD object (plain ol' data) | 19:39 |
jaypipes | efried: sure, that's cool | 19:39 |
efried | yuh | 19:39 |
*** ijw has quit IRC | 19:40 | |
sdague | mriedem: is ceph going to eventually work on the super conductor stuff? | 19:40 |
mriedem | sdague: the problem isn't superconductor + ceph, | 19:40 |
sdague | oh, I guess the issue is grenade doesn't do that yet | 19:40 |
mriedem | the problem there is grenade | 19:40 |
*** ociuhandu has quit IRC | 19:40 | |
mriedem | yup | 19:40 |
mriedem | sdague: which i half heartedly posted https://review.openstack.org/#/c/509803/ just to see how bad things blew up | 19:41 |
mriedem | but didn't really dig into it | 19:41 |
mriedem | appears to be a discover_hosts issue | 19:41 |
mriedem | http://logs.openstack.org/03/509803/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/d8414a5/logs/grenade.sh.txt.gz | 19:41 |
*** rmcallis has quit IRC | 19:42 | |
*** awaugama has quit IRC | 19:42 | |
mriedem | yeah 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.gz | 19:44 |
openstackgerrit | Merged openstack/nova master: Pass the correct image to build_request_spec in conductor.rebuild_instance https://review.openstack.org/515530 | 19:46 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | 19:52 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 19:52 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 19:52 |
openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | 19:52 |
jaypipes | cdent: turned out that you identified some useless code that should be removed. so I did. ^ | 19:52 |
*** jmlowe has quit IRC | 19:57 | |
*** kylek3h has quit IRC | 20:00 | |
*** penick has quit IRC | 20:00 | |
sdague | mriedem: yeh, let me try a thing with that | 20:02 |
cdent | jaypipes: yay less code! | 20:02 |
*** baoli has quit IRC | 20:04 | |
*** liverpooler has quit IRC | 20:06 | |
*** oanson has quit IRC | 20:08 | |
cfriesen | when 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 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Add _get_providers_with_any_resource method https://review.openstack.org/514197 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: _get_usages_by_provider_and_rc not filter on RC ID https://review.openstack.org/514649 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add _get_traits_by_rp_ids method https://review.openstack.org/511184 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add ProviderSummaryList object https://review.openstack.org/514198 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: More caches: ProviderSummary, ProviderSummaryList https://review.openstack.org/514199 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add AggregatedProviderSummary[List] objects https://review.openstack.org/514200 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: Add has_resources to AggregatedProviderSummary https://review.openstack.org/514650 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: AggregatedProvierSummary.as_allocation_request_obj https://review.openstack.org/514651 | 20:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters https://review.openstack.org/479766 | 20:10 |
efried | jaypipes 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 |
efried | jaypipes This one: https://review.openstack.org/514197 | 20:11 |
jaypipes | efried: wasn't that alex_xu's? | 20:11 |
efried | jaypipes alex_xu and I have been pair programming it. | 20:12 |
jaypipes | cfriesen: mriedem would know best. | 20:12 |
jaypipes | efried: k. | 20:12 |
mriedem | ummm | 20:12 |
efried | jaypipes Gah, pair "developing", per edleafe | 20:12 |
*** rabel has quit IRC | 20:12 | |
efried | which is different. | 20:12 |
mriedem | cfriesen: you're going to have to check the code | 20:13 |
jaypipes | mriedem: you're all about that flag. | 20:13 |
* jaypipes runs for the hills | 20:14 | |
*** kylek3h has joined #openstack-nova | 20:14 | |
*** slaweq has joined #openstack-nova | 20:14 | |
cfriesen | jaypipes: mriedem: heh...I think I found it. git history is my friend | 20:14 |
*** penick has joined #openstack-nova | 20:14 | |
*** kylek3h_ has joined #openstack-nova | 20:14 | |
cfriesen | efried: someone's developing a pair? | 20:14 |
*** baoli has joined #openstack-nova | 20:15 | |
efried | cfriesen It's a slow process. But actually more efficient than one at a time. | 20:16 |
*** salv-orlando has quit IRC | 20:17 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: document caveats with scheduler hints https://review.openstack.org/516442 | 20:17 |
mriedem | sdague: how is this for documented ambiguity? :) ^ | 20:17 |
*** yamamoto has joined #openstack-nova | 20:17 | |
*** salv-orlando has joined #openstack-nova | 20:17 | |
sdague | mriedem: yeh, pretty much | 20:18 |
*** eharney has quit IRC | 20:18 | |
mriedem | mordred: do you ever have to rely on scheduler hints? | 20:18 |
*** kylek3h has quit IRC | 20:18 | |
clarkb | mriedem: 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 nodes | 20:19 |
mriedem | clarkb: 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 wild | 20:20 |
clarkb | I know ovh schedules us to dedicate resources so we don't noisy neighbor their customers | 20:21 |
clarkb | they do this with a flavor just for us not sure if that involves scheduler hints | 20:21 |
clarkb | but ya we don't directly interact with scheduler hints at least | 20:21 |
*** salv-orlando has quit IRC | 20:21 | |
mriedem | clarkb: yeah that wouldn't be scheduler hints, just extra specs and host aggregates | 20:21 |
openstackgerrit | Eric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151 | 20:22 |
openstackgerrit | Eric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223 | 20:22 |
openstackgerrit | Eric Fried proposed openstack/nova master: ResourceRequest.to_querystring() https://review.openstack.org/515811 | 20:22 |
mriedem | i need to know if cloud x has a scheduler hint so i can tell it which lpar i want my powervm instance on | 20:23 |
mriedem | because lpar 44 is the one i want | 20:23 |
*** larainema has quit IRC | 20:26 | |
jaypipes | mriedem: and how would a cloud user know that lpar 44 is the one they want? :) | 20:26 |
*** yamamoto has quit IRC | 20:26 | |
sdague | jaypipes: probably the same way they'd know which az they wanted, it was documented somewhere. | 20:27 |
jaypipes | sdague: heh | 20:28 |
jaypipes | sdague: my point is that the AZ is a publicly-visible grouping construct that OpenStack exposes. LPARs are not publicly-visible constructs... | 20:29 |
jaypipes | sdague: 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-nova | 20:30 | |
jaypipes | sdague: but maybe you're just pulling my leg... I can't tell :) | 20:30 |
mriedem | i'm just using a ridiculous example | 20:31 |
mriedem | but yeah, just saying, one cloud/product docs could have any number of weird things tailored for their users | 20:32 |
mriedem | if 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 discoverable | 20:34 |
*** markvoelker has quit IRC | 20:35 | |
jaypipes | mriedem: sure, agreed. discoverable, not interoperable... | 20:35 |
mriedem | right, neither are flavors | 20:35 |
jaypipes | mriedem: ack | 20:35 |
jaypipes | mriedem: but standard resource classes and os-traits are getting us much closer in that respect. | 20:35 |
*** stvnoyes has quit IRC | 20:35 | |
*** penick has quit IRC | 20:36 | |
*** AlexeyAbashkin has quit IRC | 20:37 | |
*** penick has joined #openstack-nova | 20:38 | |
*** penick has quit IRC | 20:39 | |
*** ijw has joined #openstack-nova | 20:40 | |
*** cdent has quit IRC | 20:42 | |
*** penick has joined #openstack-nova | 20:44 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514 | 20:44 |
*** penick has quit IRC | 20:45 | |
*** markvoelker has joined #openstack-nova | 20:45 | |
*** ijw has quit IRC | 20:46 | |
*** ijw has joined #openstack-nova | 20:46 | |
*** markvoelker has quit IRC | 20:49 | |
*** markvoelker has joined #openstack-nova | 20:49 | |
*** jmlowe has joined #openstack-nova | 20:55 | |
*** rmcallis has joined #openstack-nova | 20:58 | |
*** penick has joined #openstack-nova | 20:59 | |
mriedem | this 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_122380 | 21:00 |
mriedem | can't find that request in the scheduler logs to see why NoValidHost | 21:00 |
*** markvoelker_ has joined #openstack-nova | 21:02 | |
*** markvoelker has quit IRC | 21:04 | |
mriedem | ah because we overwrite the NoValidHost message before returning to the api | 21:04 |
mriedem | dansmith: 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_003148 | 21:04 |
*** stvnoyes has joined #openstack-nova | 21: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 IRC | 21:07 | |
*** smatzek has quit IRC | 21:09 | |
*** smatzek has joined #openstack-nova | 21:10 | |
*** slaweq has quit IRC | 21:11 | |
*** esberglu has quit IRC | 21:11 | |
*** slaweq has joined #openstack-nova | 21:12 | |
*** esberglu has joined #openstack-nova | 21:12 | |
*** smatzek has quit IRC | 21:14 | |
*** pcaruana has quit IRC | 21:15 | |
*** slaweq has quit IRC | 21:16 | |
*** esberglu has quit IRC | 21:16 | |
*** slaweq has joined #openstack-nova | 21:16 | |
*** slaweq has quit IRC | 21:16 | |
mriedem | e-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-nova | 21:18 | |
*** kylek3h_ has quit IRC | 21:20 | |
*** dave-mccowan has quit IRC | 21:20 | |
*** kylek3h has joined #openstack-nova | 21:21 | |
mriedem | https://bugs.launchpad.net/nova/+bug/1728722 | 21:21 |
openstack | Launchpad 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 |
mriedem | ok, 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 scheduler | 21:22 |
*** salv-orlando has quit IRC | 21:22 | |
*** esberglu has joined #openstack-nova | 21:23 | |
openstackgerrit | Merged openstack/nova master: rp: break functions out of _set_traits() https://review.openstack.org/509908 | 21:25 |
*** kylek3h has quit IRC | 21:25 | |
openstackgerrit | Merged openstack/nova master: Fix live migration grenade ceph setup https://review.openstack.org/508271 | 21:25 |
openstackgerrit | Merged openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | 21:25 |
*** avolkov` has quit IRC | 21:27 | |
cfriesen | mriedem: 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 |
openstack | Launchpad bug 1482040 in OpenStack Compute (nova) "rebuild volume-backed instance with image does not replace image in root disk" [Undecided,Confirmed] | 21:31 |
mriedem | just tracking that under that bug | 21:31 |
*** esberglu has quit IRC | 21:31 | |
*** esberglu has joined #openstack-nova | 21:32 | |
cfriesen | is there any concensus on how to handle that? if not, should we bring it up at the Forum? | 21:32 |
*** esberglu has quit IRC | 21:32 | |
*** esberglu has joined #openstack-nova | 21:32 | |
mriedem | we should fail fast in the api if it's a volume-backed instance | 21:32 |
cfriesen | sounds reasonable | 21:33 |
*** jmlowe has quit IRC | 21:35 | |
*** smatzek has joined #openstack-nova | 21:36 | |
*** thorst has quit IRC | 21:38 | |
kukacz | hi, 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 IRC | 21:39 | |
kukacz | I'm thinking of doing it based eg. on some pre-defined metadata value presence | 21:39 |
*** smatzek has quit IRC | 21:40 | |
*** jmlowe has joined #openstack-nova | 21:42 | |
*** baoli has quit IRC | 21:42 | |
mriedem | kukacz: check this out https://docs.openstack.org/nova/latest/user/vendordata.html | 21:43 |
mriedem | configure 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 guest | 21:43 |
mriedem | jaypipes: 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#L1887 | 21:46 |
mriedem | jaypipes: 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 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | 21:47 |
mriedem | or allow the user to request a certain number of retries but have a configurable max in placement, defaulted to 3? | 21:47 |
kukacz | mriedem: thanks! looking into that | 21:47 |
*** Sukhdev has joined #openstack-nova | 21:48 | |
dansmith | mriedem: is that a generation mismatch thing? | 21:48 |
mriedem | dansmith: yup | 21:48 |
cfriesen | mriedem: 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 image | 21:48 |
dansmith | mriedem: so we're not retrying those in the reportclient I guess? | 21:48 |
mriedem | dansmith: not in put_allocations | 21:48 |
mriedem | which is what the migrate task uses | 21:48 |
dansmith | ack okay | 21:49 |
mriedem | the claim_resources method in the reportclient is the thing the scheduler calls and that does the double up and retry logic | 21:49 |
dansmith | okay | 21:49 |
dansmith | mriedem: you gots a bug yet? | 21:49 |
mriedem | so we could hacky sack a bit and have migrate call claim_resources but pass a "don't double things" | 21:49 |
mriedem | dansmith: yeah https://bugs.launchpad.net/nova/+bug/1728722 | 21:49 |
openstack | Launchpad 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 |
dansmith | mriedem: nah, we can just make a wrapper for put that does it I think and refactor a bit | 21:50 |
mriedem | that's what i started doing, | 21:50 |
dansmith | mriedem: I'll work on that tomorrow | 21:50 |
mriedem | but then you end up copying most of the put_allocations method from placement, and have to get the response back for the status_code | 21:50 |
kukacz | mriedem: which release was this vendordata feature introduced in? Pike? | 21:50 |
mriedem | and reportclient.put doesn't have the safe_connect on it | 21:50 |
mriedem | but i think we said we could remove that... | 21:50 |
dansmith | mriedem: I think we want the wrapper in reportclient though | 21:51 |
mriedem | kukacz: newton i think, and enhanced in ocata | 21:51 |
mriedem | dansmith: ah yeah that would work | 21:51 |
dansmith | mriedem: I would think we could make claim_resources use the wrapped version too | 21:51 |
dansmith | to avoid that duplication | 21:51 |
dansmith | anything calling put_allocations shouldn't have to worry about generation retries | 21:51 |
*** takashin has joined #openstack-nova | 21:51 | |
*** rcernin has joined #openstack-nova | 21:51 | |
openstackgerrit | Merged openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | 21:51 |
kukacz | mriedem: great, seems this could help our usecase. thanks a lot! | 21:52 |
mriedem | kukacz: yw | 21:52 |
melwitt | mriedem: I'm not seeing where https://review.openstack.org/#/c/508163 ran the test_volume_backed_live_migration test in the results | 21:54 |
mriedem | melwitt: 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_842520 | 21:55 |
melwitt | mriedem: 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 much | 21:55 |
mriedem | nope | 21:55 |
jaypipes | mriedem: I think a hard-coded server-side retry is what I would go with. | 21:55 |
mriedem | these are post test hook driven | 21:55 |
melwitt | oh :\ | 21:56 |
mriedem | melwitt: looks like i didn't uncomment the part to run it with ceph as the ephemeral backend, but not sure that would make any difference | 21:56 |
jaypipes | mriedem: but right now, dinner... | 21:56 |
*** jmlowe has quit IRC | 21:56 | |
mriedem | https://review.openstack.org/#/c/508163/4/nova/tests/live_migration/hooks/run_tests.sh@67 | 21:56 |
melwitt | oh, I see | 21:56 |
melwitt | I *think* that doesn't make a difference but we could do it just to be sure | 21:57 |
*** thorst has joined #openstack-nova | 21:58 | |
mriedem | melwitt: yeah it might given the comment for VIR_MIGRATE_NON_SHARED_INC says "Migration with non-shared storage with incremental disk copy" | 22:00 |
mriedem | so i guess i'll rev that test patch to see what happens | 22:00 |
melwitt | yeah, I think that'd be best. I'll keep an eye out for the result | 22:00 |
*** penick has quit IRC | 22:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163 | 22:01 |
*** penick has joined #openstack-nova | 22:01 | |
*** thorst has quit IRC | 22:02 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202 | 22:02 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163 | 22:02 |
mriedem | i don't know why it says it can't merge | 22:03 |
melwitt | I never figured out why it says that | 22:05 |
*** jmlowe has joined #openstack-nova | 22:06 | |
*** penick has quit IRC | 22:06 | |
*** oanson has joined #openstack-nova | 22:09 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | 22:11 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | 22:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | 22:15 |
*** marst has quit IRC | 22:15 | |
*** salv-orlando has joined #openstack-nova | 22:19 | |
*** salv-orlando has quit IRC | 22:23 | |
*** felipemonteiro_ has quit IRC | 22:23 | |
*** rmcallis has quit IRC | 22:25 | |
*** mlavalle has quit IRC | 22:25 | |
*** slaweq has joined #openstack-nova | 22:25 | |
*** salv-orlando has joined #openstack-nova | 22:26 | |
*** AlexeyAbashkin has joined #openstack-nova | 22:29 | |
*** AlexeyAbashkin has quit IRC | 22:33 | |
*** lpetrut has quit IRC | 22:33 | |
*** cdent has joined #openstack-nova | 22:43 | |
cdent | efried, jaypipes you see https://twitter.com/amitry/status/925112698102255616 ? | 22:44 |
efried | cdent I don't twit. | 22:44 |
efried | cdent How do I "ping" this guy? | 22:44 |
cdent | one moment please | 22:45 |
efried | (cdent I mean, without joining twitter) | 22:45 |
cdent | you can use me as a sock puppet if you like | 22:45 |
cdent | I can give him your email address | 22:46 |
cdent | if that works for yhou | 22:46 |
efried | cdent For sure. | 22:46 |
*** catintheroof has quit IRC | 22:48 | |
*** lyan has quit IRC | 22:49 | |
mikal | mriedem: you still around? | 22:52 |
*** slaweq has quit IRC | 22:52 | |
cdent | efried: he’s already responded, says he’ll start an email thread | 22:54 |
efried | noyce, thanks. | 22:55 |
efried | That twitter thing sure is responsive | 22:55 |
cdent | do not be tempted | 22:56 |
*** cdent has quit IRC | 22:59 | |
*** jaypipes has quit IRC | 23:06 | |
*** liverpooler has joined #openstack-nova | 23:09 | |
melwitt | mriedem: I tried rebasing your tempest change and rechecking and it didn't work. sorry about that :( | 23:15 |
melwitt | it needs the -W reapplied https://review.openstack.org/#/c/481346/ | 23:15 |
*** sdague has quit IRC | 23:16 | |
*** yamamoto has joined #openstack-nova | 23:17 | |
*** hongbin has quit IRC | 23:23 | |
*** baoli has joined #openstack-nova | 23:25 | |
*** dave-mccowan has joined #openstack-nova | 23:28 | |
*** AlexeyAbashkin has joined #openstack-nova | 23:29 | |
*** Swami has quit IRC | 23:30 | |
*** lbragstad has quit IRC | 23:32 | |
*** AlexeyAbashkin has quit IRC | 23:33 | |
*** gyee has quit IRC | 23:33 | |
*** takashin has quit IRC | 23:37 | |
*** baoli has quit IRC | 23:38 | |
*** thorst has joined #openstack-nova | 23:39 | |
*** jmlowe has quit IRC | 23:45 | |
*** vladikr has quit IRC | 23:45 | |
*** vladikr has joined #openstack-nova | 23:45 | |
*** esberglu_ has joined #openstack-nova | 23:51 | |
openstackgerrit | melanie witt proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202 | 23:51 |
openstackgerrit | melanie witt proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163 | 23:51 |
melwitt | mriedem: there was a real merge conflict from the other ceph test improvement that got merged earlier today. oops | 23:52 |
*** ijw has quit IRC | 23:53 | |
*** ijw has joined #openstack-nova | 23:54 | |
*** esberglu has quit IRC | 23:54 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151 | 23:57 |
openstackgerrit | Eric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223 | 23:57 |
openstackgerrit | Eric Fried proposed openstack/nova master: ResourceRequest.to_querystring() https://review.openstack.org/515811 | 23:57 |
efried | gibi ^ | 23:57 |
*** aloga has quit IRC | 23:59 | |
*** aloga has joined #openstack-nova | 23:59 | |
*** jmlowe has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!