| *** tbachman has joined #openstack-nova | 00:02 | |
| *** felipemonteiro has joined #openstack-nova | 00:05 | |
| *** baoli has quit IRC | 00:08 | |
| *** baoli has joined #openstack-nova | 00:09 | |
| *** tetsuro has joined #openstack-nova | 00:10 | |
| openstackgerrit | David Bingham proposed openstack/nova master: Adding apply_cells to nova-manage to enable automated cells_v2 configuration. https://review.openstack.org/568987 | 00:11 |
|---|---|---|
| *** yamamoto has joined #openstack-nova | 00:12 | |
| *** baoli has quit IRC | 00:14 | |
| *** felipemonteiro has quit IRC | 00:14 | |
| *** mriedem_afk has quit IRC | 00:15 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM snapshot cleanup https://review.openstack.org/568988 | 00:15 |
| *** yamamoto has quit IRC | 00:16 | |
| *** liuzz has quit IRC | 00:17 | |
| *** liuzz has joined #openstack-nova | 00:19 | |
| *** gjayavel_ has quit IRC | 00:28 | |
| *** r-daneel has quit IRC | 00:29 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:31 | |
| *** yamamoto has joined #openstack-nova | 00:33 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:34 | |
| *** david-lyle has quit IRC | 00:34 | |
| openstackgerrit | David Bingham proposed openstack/nova master: Adding apply_cells to nova-manage to enable automated cells_v2 configuration. https://review.openstack.org/568987 | 00:34 |
| *** dpawlik has joined #openstack-nova | 00:35 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 00:35 |
| *** yamamoto has quit IRC | 00:37 | |
| *** dpawlik has quit IRC | 00:40 | |
| openstackgerrit | Merged openstack/nova master: Add tests for alloc_cands with member_of https://review.openstack.org/561399 | 00:49 |
| naichuans | melwitt: It OK for me to have a review. Thanks you very much | 00:52 |
| *** annp has quit IRC | 00:53 | |
| *** trungnv has quit IRC | 00:53 | |
| *** annp has joined #openstack-nova | 00:53 | |
| *** yamamoto has joined #openstack-nova | 00:54 | |
| *** trungnv has joined #openstack-nova | 00:54 | |
| openstackgerrit | Merged openstack/nova master: Fix member_of with sharing providers https://review.openstack.org/561400 | 00:55 |
| openstackgerrit | Merged openstack/nova master: Expand member_of functional test cases https://review.openstack.org/566011 | 00:56 |
| melwitt | naichuans: cool, thanks | 00:56 |
| naichuans | Np | 00:57 |
| *** yamamoto has quit IRC | 01:00 | |
| *** phuongnh has joined #openstack-nova | 01:02 | |
| *** baoli has joined #openstack-nova | 01:05 | |
| *** tiendc has joined #openstack-nova | 01:06 | |
| openstackgerrit | Nakanishi Tomotaka proposed openstack/nova master: Test Compute API in multiple cells https://review.openstack.org/534116 | 01:08 |
| *** hongbin has joined #openstack-nova | 01:09 | |
| *** zhaochao has joined #openstack-nova | 01:10 | |
| *** slaweq has joined #openstack-nova | 01:11 | |
| *** slaweq has quit IRC | 01:15 | |
| *** fragatina has quit IRC | 01:16 | |
| *** yamamoto has joined #openstack-nova | 01:16 | |
| *** fragatina has joined #openstack-nova | 01:18 | |
| *** Nel1x has joined #openstack-nova | 01:19 | |
| *** fragatina has quit IRC | 01:19 | |
| *** liverpooler has joined #openstack-nova | 01:19 | |
| *** edmondsw has quit IRC | 01:19 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | 01:20 |
| *** yamamoto has quit IRC | 01:20 | |
| *** tbachman has quit IRC | 01:21 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | 01:23 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | 01:25 |
| *** hemna_ has quit IRC | 01:32 | |
| *** markvoelker_ has quit IRC | 01:34 | |
| *** markvoelker has joined #openstack-nova | 01:38 | |
| *** yamamoto has joined #openstack-nova | 01:38 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 01:40 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider candidates https://review.openstack.org/567150 | 01:40 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | 01:40 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 01:40 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 01:40 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487 | 01:40 |
| *** namnh has joined #openstack-nova | 01:40 | |
| *** yamamoto has quit IRC | 01:44 | |
| *** lei-zh has joined #openstack-nova | 01:49 | |
| *** abalutoiu__ has joined #openstack-nova | 01:51 | |
| *** fragatin_ has joined #openstack-nova | 01:52 | |
| *** abalutoiu_ has quit IRC | 01:54 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add tests for _get_trees_matching_all() function https://review.openstack.org/568737 | 01:56 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 01:56 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider candidates https://review.openstack.org/567150 | 01:56 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | 01:56 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 01:56 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 01:57 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487 | 01:57 |
| *** abalutoiu_ has joined #openstack-nova | 01:57 | |
| *** owalsh_ has joined #openstack-nova | 01:58 | |
| *** yamamoto has joined #openstack-nova | 02:00 | |
| *** abalutoiu__ has quit IRC | 02:00 | |
| *** owalsh has quit IRC | 02:02 | |
| *** itlinux has joined #openstack-nova | 02:02 | |
| *** yamamoto has quit IRC | 02:05 | |
| *** Dinesh_Bhor has quit IRC | 02:09 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:11 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: update the document related to vdi streaming https://review.openstack.org/568444 | 02:19 |
| *** yamamoto has joined #openstack-nova | 02:21 | |
| *** moshele has quit IRC | 02:24 | |
| *** yamamoto has quit IRC | 02:25 | |
| *** takashin has left #openstack-nova | 02:30 | |
| *** itlinux has quit IRC | 02:30 | |
| *** moshele has joined #openstack-nova | 02:32 | |
| *** Tom-Tom has quit IRC | 02:36 | |
| *** Tom-Tom has joined #openstack-nova | 02:37 | |
| *** moshele has quit IRC | 02:37 | |
| *** itlinux has joined #openstack-nova | 02:38 | |
| *** lbragstad has quit IRC | 02:39 | |
| *** Tom-Tom has quit IRC | 02:41 | |
| *** yamamoto has joined #openstack-nova | 02:41 | |
| *** Nel1x has quit IRC | 02:45 | |
| *** itlinux has quit IRC | 02:46 | |
| *** liverpooler has quit IRC | 02:46 | |
| *** yamamoto has quit IRC | 02:48 | |
| *** masber has quit IRC | 02:49 | |
| *** baoli has quit IRC | 02:51 | |
| *** Zames has joined #openstack-nova | 02:51 | |
| *** baoli has joined #openstack-nova | 02:51 | |
| *** dave-mcc_ has quit IRC | 02:52 | |
| *** masber has joined #openstack-nova | 02:52 | |
| *** Zames has quit IRC | 02:54 | |
| *** awaugama has quit IRC | 02:54 | |
| *** gbarros has joined #openstack-nova | 02:55 | |
| *** baoli has quit IRC | 02:56 | |
| *** hongbin has quit IRC | 02:56 | |
| *** bkopilov has quit IRC | 02:57 | |
| *** Tom-Tom has joined #openstack-nova | 02:58 | |
| *** Tom-Tom_ has joined #openstack-nova | 02:58 | |
| *** Tom-Tom has quit IRC | 02:58 | |
| *** psachin has joined #openstack-nova | 02:59 | |
| *** edmondsw has joined #openstack-nova | 02:59 | |
| *** yamamoto has joined #openstack-nova | 03:04 | |
| *** edmondsw has quit IRC | 03:04 | |
| *** Zames has joined #openstack-nova | 03:05 | |
| *** itlinux has joined #openstack-nova | 03:08 | |
| *** Zames has quit IRC | 03:08 | |
| *** yamamoto has quit IRC | 03:08 | |
| *** bkopilov has joined #openstack-nova | 03:10 | |
| *** slaweq has joined #openstack-nova | 03:11 | |
| *** eharney has quit IRC | 03:14 | |
| *** slaweq has quit IRC | 03:16 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 03:17 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 03:17 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 03:17 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 03:17 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 03:17 |
| *** hongbin has joined #openstack-nova | 03:17 | |
| *** zcorneli has quit IRC | 03:20 | |
| *** baoli has joined #openstack-nova | 03:22 | |
| *** nicolasbock has quit IRC | 03:23 | |
| openstackgerrit | JiaJunsu proposed openstack/nova master: Remove args(os=False) in monkey_patch https://review.openstack.org/568999 | 03:23 |
| *** yamamoto has joined #openstack-nova | 03:25 | |
| *** bkopilov has quit IRC | 03:29 | |
| *** yamamoto has quit IRC | 03:29 | |
| *** baoli has quit IRC | 03:30 | |
| *** itlinux has quit IRC | 03:34 | |
| *** sapcc-bot6 has joined #openstack-nova | 03:40 | |
| *** sapcc-bot5 has quit IRC | 03:41 | |
| openstackgerrit | JiaJunsu proposed openstack/nova master: Remove args(os=False) in monkey_patch https://review.openstack.org/568999 | 03:45 |
| *** hongbin has quit IRC | 03:47 | |
| *** yamamoto has joined #openstack-nova | 03:47 | |
| *** udesale has joined #openstack-nova | 03:49 | |
| *** gbarros has quit IRC | 03:51 | |
| *** yamamoto has quit IRC | 03:53 | |
| *** vladikr has quit IRC | 03:56 | |
| *** masuberu has joined #openstack-nova | 03:56 | |
| *** vladikr has joined #openstack-nova | 03:56 | |
| *** gyankum has joined #openstack-nova | 03:59 | |
| *** masber has quit IRC | 04:00 | |
| *** masber has joined #openstack-nova | 04:02 | |
| *** masuberu has quit IRC | 04:05 | |
| *** yamamoto has joined #openstack-nova | 04:08 | |
| *** udesale has quit IRC | 04:10 | |
| *** lei-zh has quit IRC | 04:10 | |
| *** germs has quit IRC | 04:10 | |
| *** janki has joined #openstack-nova | 04:12 | |
| *** yamamoto has quit IRC | 04:13 | |
| *** udesale has joined #openstack-nova | 04:13 | |
| *** yamamoto has joined #openstack-nova | 04:14 | |
| *** links has joined #openstack-nova | 04:24 | |
| *** gyee has quit IRC | 04:26 | |
| *** gongysh has joined #openstack-nova | 04:32 | |
| *** udesale has quit IRC | 04:36 | |
| *** udesale has joined #openstack-nova | 04:37 | |
| *** tbachman_ has joined #openstack-nova | 04:37 | |
| *** sridharg has joined #openstack-nova | 04:38 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs master: Count quota based on resource class https://review.openstack.org/569011 | 04:46 |
| alex_xu | melwitt: jaypipes jroll, here is a start for quota on rc https://review.openstack.org/#/c/569011/ | 04:47 |
| *** edmondsw has joined #openstack-nova | 04:48 | |
| *** vladikr has quit IRC | 04:50 | |
| *** vladikr has joined #openstack-nova | 04:50 | |
| alex_xu | Kevin_Zheng: ^ also for you guys care about keystone limits I guess | 04:51 |
| *** edmondsw has quit IRC | 04:53 | |
| *** Dinesh__Bhor has joined #openstack-nova | 04:59 | |
| *** Dinesh_Bhor has quit IRC | 04:59 | |
| *** ratailor has joined #openstack-nova | 05:01 | |
| *** slaweq has joined #openstack-nova | 05:11 | |
| *** bkopilov has joined #openstack-nova | 05:13 | |
| *** vladikr has quit IRC | 05:14 | |
| *** vladikr has joined #openstack-nova | 05:14 | |
| *** slaweq has quit IRC | 05:15 | |
| *** udesale has quit IRC | 05:20 | |
| *** udesale has joined #openstack-nova | 05:21 | |
| *** lei-zh has joined #openstack-nova | 05:22 | |
| *** yamamoto has quit IRC | 05:24 | |
| *** yamamoto has joined #openstack-nova | 05:25 | |
| *** udesale has quit IRC | 05:26 | |
| *** yamamoto_ has joined #openstack-nova | 05:28 | |
| *** tbachman_ has quit IRC | 05:28 | |
| *** yamamoto has quit IRC | 05:32 | |
| *** blkart has joined #openstack-nova | 05:36 | |
| *** Tom-Tom_ has quit IRC | 05:36 | |
| *** udesale has joined #openstack-nova | 05:43 | |
| *** slaweq has joined #openstack-nova | 05:47 | |
| *** liuzz has quit IRC | 05:47 | |
| *** liuzz has joined #openstack-nova | 05:48 | |
| *** vivsoni has joined #openstack-nova | 05:51 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider candidates https://review.openstack.org/567150 | 05:51 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | 05:51 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 05:51 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 05:51 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487 | 05:51 |
| *** slaweq has quit IRC | 05:51 | |
| *** penick has quit IRC | 05:53 | |
| *** bhujay has joined #openstack-nova | 05:55 | |
| *** penick has joined #openstack-nova | 05:58 | |
| *** vladikr has quit IRC | 06:02 | |
| *** vladikr has joined #openstack-nova | 06:03 | |
| *** slaweq has joined #openstack-nova | 06:11 | |
| *** armaan has joined #openstack-nova | 06:12 | |
| *** liuzz_ has joined #openstack-nova | 06:15 | |
| *** slaweq has quit IRC | 06:16 | |
| *** liuzz has quit IRC | 06:16 | |
| *** Dinesh__Bhor has quit IRC | 06:17 | |
| *** Dinesh__Bhor has joined #openstack-nova | 06:17 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: raise exception early https://review.openstack.org/569024 | 06:18 |
| *** trinaths has joined #openstack-nova | 06:19 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: raise correct exception https://review.openstack.org/569026 | 06:19 |
| *** liuzz_ has quit IRC | 06:20 | |
| *** vladikr has quit IRC | 06:20 | |
| *** vladikr has joined #openstack-nova | 06:21 | |
| *** liuzz has joined #openstack-nova | 06:21 | |
| *** trinaths has quit IRC | 06:22 | |
| *** liuzz has quit IRC | 06:23 | |
| *** liuzz has joined #openstack-nova | 06:25 | |
| *** gongysh has quit IRC | 06:25 | |
| *** vladikr has quit IRC | 06:31 | |
| *** gjayavelu has joined #openstack-nova | 06:31 | |
| *** sidx64 has joined #openstack-nova | 06:31 | |
| *** vladikr has joined #openstack-nova | 06:31 | |
| *** liuzz_ has joined #openstack-nova | 06:32 | |
| *** liuzz has quit IRC | 06:33 | |
| *** sidx64_ has joined #openstack-nova | 06:39 | |
| *** sidx64 has quit IRC | 06:41 | |
| *** sidx64 has joined #openstack-nova | 06:42 | |
| *** do3meli has joined #openstack-nova | 06:43 | |
| *** sidx64_ has quit IRC | 06:44 | |
| *** mdnadeem has joined #openstack-nova | 06:48 | |
| *** tssurya has joined #openstack-nova | 06:49 | |
| *** hiro-kobayashi has quit IRC | 06:54 | |
| *** gongysh has joined #openstack-nova | 06:55 | |
| *** ispp has joined #openstack-nova | 06:59 | |
| *** tesseract has joined #openstack-nova | 07:01 | |
| *** sahid has joined #openstack-nova | 07:01 | |
| *** vladikr has quit IRC | 07:01 | |
| *** vladikr has joined #openstack-nova | 07:02 | |
| *** Dinesh__Bhor has quit IRC | 07:02 | |
| *** ccamacho has joined #openstack-nova | 07:03 | |
| *** slaweq has joined #openstack-nova | 07:08 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 07:15 |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 07:15 |
| *** ispp has quit IRC | 07:17 | |
| *** dpawlik has joined #openstack-nova | 07:24 | |
| *** sridharg has quit IRC | 07:25 | |
| *** hamdyk has joined #openstack-nova | 07:27 | |
| *** gongysh has quit IRC | 07:27 | |
| *** ispp has joined #openstack-nova | 07:30 | |
| *** damien_r has joined #openstack-nova | 07:37 | |
| *** lei-zh has quit IRC | 07:39 | |
| *** lei-zh has joined #openstack-nova | 07:40 | |
| Kevin_Zheng | alex_xu: Thanks for the info, we were talk about it this morning, but I didn't have time to check the details yet. | 07:41 |
| alex_xu | Kevin_Zheng: cool | 07:42 |
| *** alexchadin has joined #openstack-nova | 07:44 | |
| *** jpena|off is now known as jpena | 07:46 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:47 | |
| *** ispp has quit IRC | 07:54 | |
| openstackgerrit | JiaJunsu proposed openstack/nova master: Fakelibvirt migrateToURI3 should provide args according to libvirt doc https://review.openstack.org/569047 | 07:54 |
| *** amoralej|off is now known as amoralej | 07:54 | |
| hamdyk | Hi, I need an os-brick release with few commits related to NVMeOF, how can I do that ? | 07:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message asserts in functional test https://review.openstack.org/569048 | 07:55 |
| *** ragiman has joined #openstack-nova | 07:58 | |
| *** mgoddard has joined #openstack-nova | 08:05 | |
| *** yamamoto_ has quit IRC | 08:10 | |
| *** avolkov has joined #openstack-nova | 08:10 | |
| *** yamamoto has joined #openstack-nova | 08:10 | |
| *** rcernin has quit IRC | 08:12 | |
| *** alexchad_ has joined #openstack-nova | 08:16 | |
| *** gongysh has joined #openstack-nova | 08:18 | |
| *** alexchadin has quit IRC | 08:18 | |
| *** ispp has joined #openstack-nova | 08:20 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Make _instances_cores_ram_count() be smart about cells https://review.openstack.org/569055 | 08:21 |
| *** edmondsw has joined #openstack-nova | 08:24 | |
| *** gjayavelu has quit IRC | 08:26 | |
| *** dtantsur|afk is now known as dtantsur | 08:26 | |
| *** edmondsw has quit IRC | 08:29 | |
| *** tianhui_ has joined #openstack-nova | 08:30 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add the guideline to write API reference https://review.openstack.org/569058 | 08:30 |
| *** tianhui has quit IRC | 08:32 | |
| *** links has quit IRC | 08:32 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 08:33 |
| *** gyankum has quit IRC | 08:33 | |
| *** bhujay has quit IRC | 08:36 | |
| *** Guest37211 has quit IRC | 08:37 | |
| *** mdnadeem has quit IRC | 08:37 | |
| *** lei-zh has quit IRC | 08:38 | |
| *** mdnadeem has joined #openstack-nova | 08:40 | |
| *** wolverineav has joined #openstack-nova | 08:40 | |
| *** yamamoto_ has joined #openstack-nova | 08:41 | |
| *** alexchadin has joined #openstack-nova | 08:42 | |
| *** tianhui has joined #openstack-nova | 08:44 | |
| *** alexchad_ has quit IRC | 08:44 | |
| *** tianhui_ has quit IRC | 08:45 | |
| *** yamamoto has quit IRC | 08:45 | |
| *** owalsh_ is now known as owalsh | 08:48 | |
| *** links has joined #openstack-nova | 08:49 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Metadata-API fails to retrieve avz for instances created before Pike https://review.openstack.org/567878 | 08:50 |
| *** Guest37211 has joined #openstack-nova | 08:50 | |
| *** derekh has joined #openstack-nova | 08:51 | |
| gibi | Does somebody has an idea why the ovo hash differs for the same ovo between py27 and py35 env? https://review.openstack.org/#/c/568840 | 08:58 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | 08:58 |
| *** Zames has joined #openstack-nova | 08:58 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | 08:59 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 09:00 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 09:00 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 09:00 |
| *** bhujay has joined #openstack-nova | 09:01 | |
| *** Zames has quit IRC | 09:02 | |
| *** mdbooth has joined #openstack-nova | 09:04 | |
| *** gyankum has joined #openstack-nova | 09:04 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | 09:05 |
| *** lei-zh has joined #openstack-nova | 09:05 | |
| *** alexchad_ has joined #openstack-nova | 09:08 | |
| *** wolverineav has quit IRC | 09:08 | |
| *** wolverineav has joined #openstack-nova | 09:08 | |
| *** gyankum has quit IRC | 09:08 | |
| *** gongysh has quit IRC | 09:09 | |
| *** alexchadin has quit IRC | 09:10 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 09:11 |
| *** wolverineav has quit IRC | 09:13 | |
| openstackgerrit | sahid proposed openstack/nova master: pci: don't consider case when match tags specs https://review.openstack.org/565808 | 09:14 |
| openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 09:14 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | 09:14 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add vf_trusted field for network metadata https://review.openstack.org/566343 | 09:14 |
| openstackgerrit | sahid proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | 09:14 |
| *** sidx64 has quit IRC | 09:14 | |
| *** ispp has quit IRC | 09:15 | |
| *** gongysh has joined #openstack-nova | 09:17 | |
| *** sidx64 has joined #openstack-nova | 09:17 | |
| *** alexchadin has joined #openstack-nova | 09:19 | |
| *** alexcha__ has joined #openstack-nova | 09:20 | |
| *** dims has quit IRC | 09:20 | |
| *** alexchad_ has quit IRC | 09:22 | |
| *** gongysh has quit IRC | 09:24 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: Improve 'qemu-img convert' performance https://review.openstack.org/522067 | 09:24 |
| *** alexchadin has quit IRC | 09:24 | |
| *** sahid has quit IRC | 09:35 | |
| *** gyankum has joined #openstack-nova | 09:46 | |
| *** xinliang has quit IRC | 09:46 | |
| *** Zames has joined #openstack-nova | 09:46 | |
| *** Zames has quit IRC | 09:48 | |
| *** moshele has joined #openstack-nova | 09:50 | |
| *** gyankum has quit IRC | 09:50 | |
| *** mdnadeem has quit IRC | 09:52 | |
| *** AlexeyAbashkin has quit IRC | 09:53 | |
| *** moshele has quit IRC | 09:54 | |
| *** gyankum has joined #openstack-nova | 09:56 | |
| *** Zames has joined #openstack-nova | 09:58 | |
| *** xinliang has joined #openstack-nova | 09:59 | |
| *** namnh has quit IRC | 10:00 | |
| *** ispp has joined #openstack-nova | 10:01 | |
| *** Zames has quit IRC | 10:02 | |
| *** dims has joined #openstack-nova | 10:07 | |
| *** lei-zh has quit IRC | 10:12 | |
| *** edmondsw has joined #openstack-nova | 10:13 | |
| *** sahid has joined #openstack-nova | 10:14 | |
| *** annp has quit IRC | 10:15 | |
| *** edmondsw has quit IRC | 10:17 | |
| *** dims has quit IRC | 10:17 | |
| *** nicolasbock has joined #openstack-nova | 10:30 | |
| *** ispp has quit IRC | 10:40 | |
| *** zhaochao has quit IRC | 10:41 | |
| *** hoonetorg has quit IRC | 10:43 | |
| *** tetsuro has quit IRC | 10:44 | |
| *** tetsuro has joined #openstack-nova | 10:45 | |
| *** tetsuro has quit IRC | 10:45 | |
| *** Zames has joined #openstack-nova | 10:46 | |
| *** zhaochao has joined #openstack-nova | 10:47 | |
| *** armaan has quit IRC | 10:48 | |
| *** armaan has joined #openstack-nova | 10:49 | |
| *** ykarel has joined #openstack-nova | 10:49 | |
| *** Zames has quit IRC | 10:50 | |
| *** edmondsw has joined #openstack-nova | 10:52 | |
| *** bkopilov has quit IRC | 10:54 | |
| *** AlexeyAbashkin has joined #openstack-nova | 10:54 | |
| *** jpena is now known as jpena|lunch | 10:59 | |
| *** mdbooth has quit IRC | 11:00 | |
| *** hoonetorg has joined #openstack-nova | 11:00 | |
| *** jaosorior has quit IRC | 11:01 | |
| *** cdent has joined #openstack-nova | 11:01 | |
| *** jaosorior has joined #openstack-nova | 11:05 | |
| *** mdbooth has joined #openstack-nova | 11:07 | |
| *** dave-mccowan has joined #openstack-nova | 11:09 | |
| *** gyan_ has joined #openstack-nova | 11:14 | |
| *** gyankum has quit IRC | 11:16 | |
| *** links has quit IRC | 11:16 | |
| *** alexcha__ has quit IRC | 11:17 | |
| *** alexchadin has joined #openstack-nova | 11:17 | |
| *** alexchadin has quit IRC | 11:18 | |
| *** alexchadin has joined #openstack-nova | 11:18 | |
| *** alexchadin has quit IRC | 11:18 | |
| *** alexchadin has joined #openstack-nova | 11:19 | |
| *** tssurya has quit IRC | 11:19 | |
| *** alexchadin has quit IRC | 11:19 | |
| *** alexchadin has joined #openstack-nova | 11:20 | |
| *** mdnadeem has joined #openstack-nova | 11:20 | |
| *** alexchadin has quit IRC | 11:20 | |
| *** armaan has quit IRC | 11:23 | |
| *** armaan has joined #openstack-nova | 11:23 | |
| *** phuongnh has quit IRC | 11:28 | |
| *** psachin has quit IRC | 11:28 | |
| *** links has joined #openstack-nova | 11:29 | |
| *** ispp has joined #openstack-nova | 11:30 | |
| *** yamamoto_ has quit IRC | 11:30 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090 | 11:31 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | 11:32 |
| openstackgerrit | Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090 | 11:32 |
| *** lbragstad has joined #openstack-nova | 11:33 | |
| *** wolverineav has joined #openstack-nova | 11:36 | |
| *** psachin has joined #openstack-nova | 11:37 | |
| *** vladikr has quit IRC | 11:38 | |
| *** vladikr has joined #openstack-nova | 11:39 | |
| *** sidx64 has quit IRC | 11:40 | |
| *** moshele has joined #openstack-nova | 11:40 | |
| *** stakeda has quit IRC | 11:42 | |
| *** bhagyashri_s has joined #openstack-nova | 11:42 | |
| *** bhagyashri_s has quit IRC | 11:43 | |
| *** tssurya has joined #openstack-nova | 11:44 | |
| *** tiendc has quit IRC | 11:44 | |
| *** bhagyashri_s has joined #openstack-nova | 11:44 | |
| gibi | found the reason of the different hash in https://review.openstack.org/#/c/568840 opened a bug https://bugs.launchpad.net/oslo.versionedobjects/+bug/1771804 | 11:45 |
| openstack | Launchpad bug 1771804 in oslo.versionedobjects "ObjectVersionChecker.get_hash generates unstable has for set() default value between py27 and py35" [Undecided,New] | 11:45 |
| *** bhagyashri_s has quit IRC | 11:45 | |
| *** bhagyashris has quit IRC | 11:46 | |
| *** bhagyashri_s has joined #openstack-nova | 11:46 | |
| *** yamamoto has joined #openstack-nova | 11:47 | |
| *** amoralej is now known as amoralej|lunch | 11:47 | |
| *** sidx64 has joined #openstack-nova | 11:47 | |
| *** pooja-jadhav has joined #openstack-nova | 11:48 | |
| *** pooja_jadhav has quit IRC | 11:50 | |
| *** mmehan has joined #openstack-nova | 11:51 | |
| *** vivsoni has quit IRC | 11:52 | |
| *** udesale has quit IRC | 11:53 | |
| *** vivsoni has joined #openstack-nova | 11:53 | |
| *** mriedem has joined #openstack-nova | 11:57 | |
| *** liuzz_ has quit IRC | 11:57 | |
| *** alexchadin has joined #openstack-nova | 11:58 | |
| *** hamdyk has quit IRC | 12:00 | |
| *** pooja_jadhav has joined #openstack-nova | 12:02 | |
| *** bhagyashris_ has joined #openstack-nova | 12:03 | |
| *** cdent has quit IRC | 12:03 | |
| *** alexchadin has quit IRC | 12:04 | |
| *** pooja-jadhav has quit IRC | 12:06 | |
| *** wolverineav has quit IRC | 12:06 | |
| *** bhagyashri_s has quit IRC | 12:06 | |
| *** wolverineav has joined #openstack-nova | 12:06 | |
| *** liverpooler has joined #openstack-nova | 12:08 | |
| *** moshele has quit IRC | 12:10 | |
| *** jaosorior has quit IRC | 12:12 | |
| *** eharney has joined #openstack-nova | 12:12 | |
| *** bhagyashri_s has joined #openstack-nova | 12:15 | |
| *** abhishekk has joined #openstack-nova | 12:16 | |
| *** bhagyashris_ has quit IRC | 12:18 | |
| *** alexchadin has joined #openstack-nova | 12:19 | |
| *** jaosorior has joined #openstack-nova | 12:19 | |
| *** ispp has quit IRC | 12:23 | |
| *** belmoreira has joined #openstack-nova | 12:24 | |
| *** moshele has joined #openstack-nova | 12:24 | |
| *** jpena|lunch is now known as jpena | 12:24 | |
| mriedem | efried: mostly small things inline https://review.openstack.org/#/c/549300/ | 12:25 |
| *** ispp has joined #openstack-nova | 12:25 | |
| *** bhagyashris has joined #openstack-nova | 12:25 | |
| mriedem | tell esberglu to get up | 12:25 |
| *** bkopilov has joined #openstack-nova | 12:26 | |
| *** bhagyashri_s has quit IRC | 12:28 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | 12:30 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | 12:30 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 12:30 |
| mriedem | gibi: is this now the 2nd changed error message in the placement api in about a week that has broken osc-placement functional tests? https://review.openstack.org/#/c/569048 | 12:30 |
| *** lyan has joined #openstack-nova | 12:31 | |
| belmoreira | Hi mriedem. I'm trying to find how to use a custom resource class for a compute node (not ironic). Can't find any info. Do you have any pointer? | 12:31 |
| gibi | mriedem: yes, unfortunately | 12:32 |
| mriedem | belmoreira: yeah https://docs.openstack.org/osc-placement/latest/index.html | 12:33 |
| *** mdnadeem has quit IRC | 12:33 | |
| *** pchavva has joined #openstack-nova | 12:33 | |
| mriedem | openstack resource class create - to create your custom resource class | 12:33 |
| gibi | mriedem: and it seems that the fix https://review.openstack.org/#/c/569048 shows a race condition as well | 12:33 |
| mriedem | openstack resource provider list - to find your compute node resource provider | 12:33 |
| mriedem | belmoreira: and then you want to set that custom resource class inventory on the compute node resource provider? use https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-inventory-set | 12:33 |
| mriedem | or actually https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-inventory-class-set is probably easier | 12:34 |
| *** sapd_ has quit IRC | 12:34 | |
| mriedem | openstack resource provider inventory set - makes you specify the full inventory with the changes, since it's a full overwrite | 12:34 |
| mriedem | openstack resource provider class set - just modifies the inventory for the specified class | 12:34 |
| *** sapd_ has joined #openstack-nova | 12:34 | |
| mriedem | gibi: yeah i was wondering why the osc-placement functional test was still failng | 12:34 |
| *** cdent has joined #openstack-nova | 12:35 | |
| *** sapd_ has quit IRC | 12:35 | |
| *** sapd_ has joined #openstack-nova | 12:35 | |
| *** sapd_ has quit IRC | 12:36 | |
| *** sapd_ has joined #openstack-nova | 12:36 | |
| mriedem | gibi: are you sure that https://review.openstack.org/#/c/569048/ ran against the merged nova change? | 12:36 |
| belmoreira | mriedem: thanks. This will be always defined with placement API. There isn't the option for the compute_node advertise the custom resource classes that it supports? | 12:36 |
| gibi | mriedem: the nova patch merged yesterday, I pushed the fix for osc_placement today | 12:37 |
| *** mmehan has quit IRC | 12:37 | |
| gibi | mriedem: so I assume | 12:37 |
| mriedem | belmoreira: the compute node doesn't make up custom resource classes on the fly randomly :) | 12:38 |
| mriedem | the compute node will report standard resource classes | 12:38 |
| gibi | mriedem: and the osc_placement test fails differently with the fix than before | 12:38 |
| mriedem | gibi: it looks like assertCommandFailed might just be doing a prefix vs assertIn check? | 12:38 |
| mriedem | because you got HTTP 404 back | 12:39 |
| mriedem | http://logs.openstack.org/48/569048/1/check/osc-placement-dsvm-functional/2800b3e/job-output.txt.gz#_2018-05-17_08_20_04_900926 | 12:39 |
| mriedem | belmoreira: we plan on reporting compute driver capabilities as standard traits on the compute node provider, see https://review.openstack.org/#/c/538498/ | 12:39 |
| mriedem | belmoreira: so we can report things like, "does this compute node support device tags, does this compute note support multiattach volumes, etc" | 12:39 |
| *** armaan has quit IRC | 12:40 | |
| mriedem | those are being defined here https://github.com/openstack/os-traits/tree/master/os_traits/compute | 12:40 |
| gibi | mriedem: ahh. I've just realized that it fails in cleanup not in the test itself http://logs.openstack.org/48/569048/1/check/osc-placement-dsvm-functional/2800b3e/job-output.txt.gz#_2018-05-17_08_20_04_880092 | 12:40 |
| mriedem | i just need to cleanup that patch and get on the update_provider_tree() train | 12:40 |
| mriedem | gibi: aha | 12:40 |
| *** armaan has joined #openstack-nova | 12:41 | |
| gibi | mriedem: so the test deletes the trait, then fails to clean up as that want to delete the trait again | 12:41 |
| mriedem | gibi: ah yes https://review.openstack.org/#/c/569048/1/osc_placement/tests/functional/base.py@212 | 12:41 |
| mriedem | if 'not found' not in err_message: | 12:41 |
| mriedem | just need to fix that | 12:41 |
| *** bhagyashri_s has joined #openstack-nova | 12:41 | |
| gibi | mriedem: good point | 12:41 |
| mriedem | if if 'not found' not in err_message or 'HTTP 404' in err_message: | 12:41 |
| *** pooja_jadhav has quit IRC | 12:41 | |
| *** alexchadin has quit IRC | 12:42 | |
| gibi | mriedem: I missed that | 12:42 |
| mriedem | to support old and new placement | 12:42 |
| mriedem | er or 'HTTP 404' not in err_message: | 12:42 |
| gibi | mriedem: even better point to support old and new placement | 12:42 |
| *** bhagyashris has quit IRC | 12:42 | |
| mriedem | gibi: want to fix that quick and then i'll rebase avolkov's series | 12:42 |
| mriedem | belmoreira: does that help you out? | 12:42 |
| *** pooja_jadhav has joined #openstack-nova | 12:42 | |
| gibi | mriedem: I'm doing it as we speak | 12:42 |
| *** moshele has quit IRC | 12:43 | |
| *** dims has joined #openstack-nova | 12:43 | |
| belmoreira | mriedem traits would do the trick as well. What I was thinking is an option to specify "custom" traits/resource classes per host. | 12:44 |
| belmoreira | report compute_driver capabilities is good but we may want to use different properties as well | 12:45 |
| *** moshele has joined #openstack-nova | 12:48 | |
| *** bhagyashri_s has quit IRC | 12:48 | |
| gibi | mriedem: both the old and the new error message contains HTTP 404, so we dont need the ORed conditional | 12:48 |
| gibi | stack@ubuntu:~/osc-placement$ openstack trait delete ALMA --os-placement-api-version 1.6 | 12:48 |
| gibi | No such trait(s): ALMA. (HTTP 404) | 12:48 |
| mriedem | belmoreira: you can do that yourself | 12:49 |
| mriedem | belmoreira: you mean you want to set a config option for nova to proxy all of that information for you? | 12:49 |
| *** pooja-jadhav has joined #openstack-nova | 12:49 | |
| *** bhagyashri_s has joined #openstack-nova | 12:49 | |
| gibi | stack@ubuntu:~/nova$ openstack trait delete ALMA --os-placement-api-version 1.6 | 12:50 |
| gibi | Not found (HTTP 404) | 12:50 |
| mriedem | belmoreira: the beauty of custom resource classes and traits and placement is that it's external to nova, so you don't need to proxy stuff through nova and add the complexity / bottleneck there | 12:50 |
| mriedem | gibi: cool | 12:50 |
| openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message asserts in functional test https://review.openstack.org/569048 | 12:51 |
| gibi | mriedem: ^^ here it is | 12:51 |
| mriedem | yup, watching zuul | 12:51 |
| *** pooja_jadhav has quit IRC | 12:52 | |
| *** belmoreira has quit IRC | 12:52 | |
| *** yamamoto has quit IRC | 12:52 | |
| *** bhagyashris_ has joined #openstack-nova | 12:53 | |
| mriedem | tssurya: i guess belmiro didn't like that answer? | 12:54 |
| *** gyan_ has quit IRC | 12:54 | |
| tssurya | mriedem: I am sure that's not the case :D | 12:55 |
| mriedem | he hates me now doesn't he | 12:55 |
| tssurya | haha | 12:55 |
| mriedem | I BLEW IT | 12:55 |
| mriedem | tssurya: btw, got any cells v2 upgrade news? | 12:55 |
| *** bhagyashri_s has quit IRC | 12:55 | |
| jroll | alex_xu: thanks | 12:56 |
| *** jmlowe has quit IRC | 12:56 | |
| tssurya | mriedem: its actually okay.. | 12:57 |
| tssurya | we are just trying to understand what is taking time in the scheduler | 12:57 |
| tssurya | I mean what takes most of the time in the path of the scheduling | 12:57 |
| mriedem | my guess would be that it's pulling more compute node records from the db than it used to do with the per-cell scheduler | 12:58 |
| *** ispp has quit IRC | 12:58 | |
| tssurya | and what's more weird is, how do I test exactly this scatter-gather versus normal sequential querying ? | 12:58 |
| gibi | mriedem: the lates ps of the osc_placement fix still failing the functional test locally :/ | 12:58 |
| tssurya | like the time taken | 12:58 |
| mriedem | so if you have 50 cells with 200 computes, before the top level scheduler would pick a cell and that would pull at most 200 compute nodes, right? but now it could be pulling at most 10K compute nodes | 12:59 |
| tssurya | mriedem: we use dan's pre-placement filter to ensure we don't pull that much | 12:59 |
| mriedem | tssurya: hmmm, we could add some debug logging to the scatter gather routine to log how long an operation took per cell... | 12:59 |
| tssurya | mriedem: that is what I did, but it gives weird times, | 13:00 |
| tssurya | not convincing ones | 13:00 |
| mriedem | is the time being spent in pulling the compute nodes from the cells or from the filters? | 13:01 |
| tssurya | mriedem: filters don't seem to take much time, pulling computes (the query directly done on the DB is very fast, however logging the scatter-gather routine gives weird longer times) | 13:02 |
| tssurya | what is taking time is in the select destinations | 13:02 |
| tssurya | I am working on it now to narrow it down further | 13:02 |
| mriedem | also, do you know if you're hitting the same compute nodes for concurrent requests (packing)? just wondering if you're getting 409s from placement when trying to allocate resources against a target node | 13:02 |
| gibi | mriedem: ohh, there is an upper() call on the message somewhere as {{{err_message:no such trait(s): custom_fake_hw_gpu_class_264c6d7df889415ea5d3b428d7a4a2fc. (http 404)}}} | 13:02 |
| mriedem | tssurya: hmm, might be the alternate selection code? | 13:03 |
| tssurya | mriedem: yea I am right at that point now | 13:03 |
| tssurya | will see in prod if that's the one | 13:04 |
| *** moshele has quit IRC | 13:04 | |
| openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message asserts in functional test https://review.openstack.org/569048 | 13:04 |
| mriedem | that will add more time compared to before queens because we are trying to now find up to max_attempts alternate hosts in the same cell, and that means running those alternates through the filters again | 13:05 |
| tssurya | mriedem: oh right! | 13:05 |
| *** amoralej|lunch is now known as amoralej | 13:05 | |
| mriedem | and that's per instance | 13:05 |
| mriedem | so if you do server multicreate in a single request with like 10 instances and max_attempts is 3, we are trying to find i think 30 hosts | 13:06 |
| tssurya | mriedem: hmm yea, will know if this is the one taking time, will check the logs | 13:07 |
| tssurya | meanwhile mriedem I have another issue : http://logs.openstack.org/71/568271/2/check/tempest-full-py3/7142c54/testr_results.html.gz , not sure on how to fix this | 13:07 |
| gibi | mriedem: ps 3 passes the functional locally | 13:08 |
| tssurya | it totally depends on if the host runs both cell and super conductors or runs one of them | 13:08 |
| *** links has quit IRC | 13:09 | |
| * tssurya goes to check if alternate host path is the one taking the time or not. | 13:09 | |
| *** alexchadin has joined #openstack-nova | 13:09 | |
| openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802 | 13:09 |
| mriedem | tssurya: hmm, the cell conductor should be reporting into the cell1 database | 13:11 |
| mriedem | this is n-super-cond: http://logs.openstack.org/71/568271/2/check/tempest-full-py3/7142c54/controller/logs/screen-n-super-cond.txt.gz#_May_17_08_20_09_133873 | 13:12 |
| mriedem | http://logs.openstack.org/71/568271/2/check/tempest-full-py3/7142c54/controller/logs/etc/nova/nova_conf.txt.gz | 13:12 |
| mriedem | which is in cell0 | 13:12 |
| mriedem | [database] connection = mysql+pymysql://root:secretdatabase@127.0.0.1/nova_cell0?charset=utf8 | 13:12 |
| mriedem | the cell conductor is using the nova cell1 config | 13:13 |
| mriedem | http://logs.openstack.org/71/568271/2/check/tempest-full-py3/7142c54/controller/logs/screen-n-cond-cell1.txt.gz#_May_17_08_20_10_145541 | 13:13 |
| mriedem | http://logs.openstack.org/71/568271/2/check/tempest-full-py3/7142c54/controller/logs/etc/nova/nova_cell1_conf.txt.gz | 13:13 |
| mriedem | [database] connection = mysql+pymysql://root:secretdatabase@127.0.0.1/nova_cell1?charset=utf8 | 13:13 |
| *** yamamoto has joined #openstack-nova | 13:13 | |
| mriedem | tssurya: it's a buggy test http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_services.py#n61 | 13:14 |
| mriedem | it randomly picks the first service from the list and then looks that up by host and binary | 13:14 |
| tssurya | mriedem: yea! | 13:15 |
| mriedem | it's a single node devstack, so the host is going to be the same | 13:15 |
| mriedem | so tempest needs to just not ever use conductor here | 13:15 |
| mriedem | i'll report a tempest bug | 13:15 |
| *** armaan has quit IRC | 13:15 | |
| tssurya | mriedem: awesome thanks, | 13:15 |
| *** armaan has joined #openstack-nova | 13:15 | |
| mriedem | https://bugs.launchpad.net/tempest/+bug/1771817 | 13:17 |
| openstack | Launchpad bug 1771817 in tempest "test_get_service_by_service_and_host_name intermittently fails if it tries to use nova-conductor on a single node devstack" [Undecided,New] | 13:17 |
| mriedem | we should just delete that test from tempest, i'll ask the qa team | 13:17 |
| *** mchlumsky has joined #openstack-nova | 13:17 | |
| *** yamamoto has quit IRC | 13:18 | |
| tssurya | mriedem: ack | 13:19 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375 | 13:20 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | 13:21 |
| *** burt has joined #openstack-nova | 13:23 | |
| *** yamamoto has joined #openstack-nova | 13:23 | |
| *** ratailor has quit IRC | 13:26 | |
| *** baoli has joined #openstack-nova | 13:28 | |
| *** yikun__ has joined #openstack-nova | 13:29 | |
| *** gongysh has joined #openstack-nova | 13:29 | |
| *** mdnadeem has joined #openstack-nova | 13:29 | |
| *** sambetts|afk is now known as sambetts | 13:29 | |
| *** belmoreira has joined #openstack-nova | 13:31 | |
| Kevin_Zheng | mriedem: I was thinking about count services matches the condition before assert | 13:31 |
| belmoreira | mriedem sorry I needed to leave for few minutes | 13:31 |
| *** baoli has quit IRC | 13:32 | |
| *** pchavva has quit IRC | 13:32 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: Improve 'qemu-img convert' performance https://review.openstack.org/522067 | 13:32 |
| *** baoli has joined #openstack-nova | 13:32 | |
| *** esberglu has joined #openstack-nova | 13:32 | |
| belmoreira | mriedem If "colour" is a property that we would like to consider during scheduling how this should be defined? | 13:33 |
| *** munishmehan has joined #openstack-nova | 13:36 | |
| dansmith | belmoreira: a property of what? | 13:36 |
| mriedem | Kevin_Zheng: we do'nt need to do that | 13:37 |
| mriedem | Kevin_Zheng: https://review.openstack.org/569112 | 13:37 |
| mriedem | dansmith: of the compute node, | 13:37 |
| mriedem | he wants to put a custom resource class or trait on the compute node resource provider | 13:37 |
| *** takashin has joined #openstack-nova | 13:37 | |
| mriedem | belmoreira: traits-based scheduling | 13:37 |
| *** mmehan has joined #openstack-nova | 13:38 | |
| dansmith | mriedem: ah | 13:38 |
| mriedem | belmoreira: see required traits here https://docs.openstack.org/nova/latest/user/flavors.html#extra-specs | 13:38 |
| mriedem | belmoreira: you put a trait on the compute node resource provider (via osc-placement or REST API or whatever) and then put that required trait in a flavor | 13:38 |
| mriedem | and then that flavor must land on a compute node with that trait | 13:38 |
| *** jmlowe has joined #openstack-nova | 13:38 | |
| *** felipemonteiro has joined #openstack-nova | 13:39 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova service-list use scatter-gather routine https://review.openstack.org/568271 | 13:39 |
| belmoreira | mriedem: I understand that. What I was thinking is if the compute node can it self advertise the trait | 13:39 |
| *** sdake_ is now known as sdake | 13:39 | |
| mriedem | belmoreira: as i said earlier, the compute node can report standard traits that it knows about, | 13:39 |
| mriedem | for custom stuff, you can do that outside of nova-compute needing to report it for you by proxy | 13:40 |
| Kevin_Zheng | mriedem: if that is the case, then most of the tests can be removed? | 13:40 |
| *** ratailor has joined #openstack-nova | 13:40 | |
| *** munishmehan has quit IRC | 13:40 | |
| efried | belmoreira: The virt driver can set any traits it wants. But one would have to make a case for a given trait being a sensible thing for the virt driver to set. | 13:41 |
| mriedem | Kevin_Zheng: i'm starting small | 13:41 |
| openstackgerrit | sahid proposed openstack/os-vif stable/pike: ovs-hybrid: should permanently keep MAC entries https://review.openstack.org/505731 | 13:41 |
| *** ratailor has quit IRC | 13:41 | |
| belmoreira | the use case is when deploying a set of nodes with different custom traits all of this needs to be setup up using tha placement api. It would be easier to include them in a nova config option. This case is much easier to handle with a configuration management tool | 13:41 |
| Kevin_Zheng | mriedem: :) | 13:42 |
| efried | belmoreira: In Dublin we agreed that there would be a certain set of traits that the compute service "owns" - has the authority to set/unset on the providers it owns - and anything outside of that circle could be set externally via direct placement API access (via osc or whatever). | 13:42 |
| belmoreira | efried mriedem thanks for the clarification | 13:43 |
| efried | belmoreira: You're suggesting a config option like | 13:43 |
| efried | compute_provider_additional_traits = CUSTOM_FOO, CUSTOM_BAR | 13:43 |
| efried | ? | 13:43 |
| mriedem | yes, and i don't think we should do that | 13:44 |
| belmoreira | efried yes | 13:44 |
| dansmith | you'd have to have a negative as well so you could ever remove traits you had previously set | 13:44 |
| mriedem | it makes nova a proxy, | 13:44 |
| dansmith | mriedem: agree | 13:44 |
| mriedem | and blocks everyone on nova supporting this thing | 13:44 |
| mriedem | when you can already do it external to nova | 13:44 |
| efried | belmoreira: yes, I agree with mriedem and dansmith. | 13:44 |
| mriedem | i already don't really care for the ironic trait proxy stuff we do | 13:44 |
| belmoreira | my issue is how to manage all this things at scale | 13:45 |
| efried | belmoreira: You would be editing config files, one per host, in any case. | 13:45 |
| mriedem | your config mgmt tool is already doing the logic to put the custom traits in the specific compute host config file yes? | 13:45 |
| efried | belmoreira: Rather than doing that, run a placement CLI once per host. | 13:45 |
| belmoreira | a configuration management tool doing API calls doesn't looks good to me | 13:45 |
| mriedem | why can't that same code just put the traits in placement once nova-compute is running and the rp is registered? | 13:45 |
| *** felipemonteiro has quit IRC | 13:46 | |
| mriedem | belmoreira: how do you create keystone endpoints? | 13:46 |
| mriedem | and users/projects | 13:46 |
| efried | Are you equating "API calls" with invoking CLIs? | 13:46 |
| mriedem | openstack-ansible does all of this quite a bit i think | 13:47 |
| mriedem | that's how they do the cells v2 setup | 13:47 |
| mriedem | it's really the only way since we don't have rest apis for cell and host mappings | 13:47 |
| belmoreira | mriedem is true for keystone. We built our own machinery to create/delete projects. And the experience is not great | 13:47 |
| belmoreira | efried: no. The problem is given credentials to a configuration management tool to populate these properties | 13:49 |
| belmoreira | changing configuration files is much easier and less worrying | 13:50 |
| *** ispp has joined #openstack-nova | 13:50 | |
| *** tetsuro has joined #openstack-nova | 13:51 | |
| *** zhaochao has quit IRC | 13:51 | |
| *** gbarros has joined #openstack-nova | 13:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Fix error message asserts in functional test https://review.openstack.org/569048 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | 13:54 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | 13:54 |
| mriedem | gibi: https://review.openstack.org/#/c/514644/ should be ready to go now | 13:54 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375 | 13:55 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401 | 13:55 |
| *** armaan has quit IRC | 13:56 | |
| *** armaan has joined #openstack-nova | 13:57 | |
| gibi | mriedem: done | 13:57 |
| mriedem | yay | 13:57 |
| melwitt | nova meeting in 3 min | 13:57 |
| *** tbachman has joined #openstack-nova | 13:58 | |
| *** artom has quit IRC | 13:59 | |
| *** mlavalle has joined #openstack-nova | 13:59 | |
| *** armaan has quit IRC | 14:00 | |
| *** belmorei_ has joined #openstack-nova | 14:00 | |
| *** armaan has joined #openstack-nova | 14:00 | |
| *** abhishekk_ has joined #openstack-nova | 14:01 | |
| *** ccamacho has quit IRC | 14:01 | |
| *** belmoreira has quit IRC | 14:01 | |
| *** ccamacho has joined #openstack-nova | 14:02 | |
| *** abhishekk has quit IRC | 14:04 | |
| *** abhishekk_ is now known as abhishekk | 14:04 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Proposes Multiple GPU types https://review.openstack.org/557065 | 14:09 |
| *** armaan has quit IRC | 14:15 | |
| *** armaan has joined #openstack-nova | 14:15 | |
| *** sidx64 has quit IRC | 14:16 | |
| *** johanssone has quit IRC | 14:16 | |
| mriedem | tssurya: check the max_attempts config option, if it's 1 then we shouldn't be calculating alternate hosts in the scheduler | 14:17 |
| mriedem | and belmorei_ said in boston that cern doesn't do retries | 14:17 |
| dansmith | mriedem: well, let me just hand you my thunder then | 14:17 |
| mriedem | tssurya: dansmith reminded me that max_attempts=1 means no retries and dansmith reminded me that belmorei_ said in boston that cern doesn't do retries | 14:18 |
| dansmith | haha | 14:18 |
| *** sidx64 has joined #openstack-nova | 14:20 | |
| belmorei_ | mriedem: correct :) we have max_attempts=1 | 14:20 |
| *** sidx64 has quit IRC | 14:21 | |
| dansmith | so if it's trying to get alternates in that case, it's probably a bug | 14:21 |
| mriedem | yeah need to figure that part out, although i thought edleafe added unit tests for that scenario to assert that we didn't try calculating alternates if max_attempest=1 | 14:22 |
| mriedem | tempest on the brain | 14:22 |
| dansmith | ah, | 14:23 |
| dansmith | max_attempts grabs one alternate it looks like | 14:23 |
| dansmith | instead of max_attempts-1 ? | 14:23 |
| *** sidx64 has joined #openstack-nova | 14:23 | |
| dansmith | https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L161-L161 | 14:24 |
| *** hemna_ has joined #openstack-nova | 14:24 | |
| edleafe | num_alts = CONF.scheduler.max_attempts if return_alternates else 0 | 14:24 |
| edleafe | so yeah, it will return 1 alternate with max_attempts=1 | 14:24 |
| dansmith | which is wrong | 14:24 |
| mriedem | arvindn05: how are you coming on that traits-in-images + rebuild patch? | 14:25 |
| dansmith | we also would do too much extra work if num_alts is zero | 14:25 |
| dansmith | belmorei_: tssurya: if you can confirm that with max_attempts the scheduler is asking for one extra alternate that would be good | 14:26 |
| dansmith | would mean about an exact doubling of placement and filter load I guess | 14:26 |
| mriedem | shouldn't be any more placement load, | 14:26 |
| mriedem | just filtering | 14:26 |
| mriedem | the alternates are pulled from the compute node list we get after placement allocation candidates are retrieved | 14:26 |
| dansmith | it asks placement again doesn't it? | 14:26 |
| mriedem | not for new allocation candidates | 14:27 |
| edleafe | alternates only get re-filtered when building > 1 instance per request | 14:27 |
| mriedem | or it shouldn't... | 14:27 |
| belmorei_ | dansmith: will do | 14:27 |
| dansmith | mriedem: okay | 14:27 |
| *** moshele has joined #openstack-nova | 14:27 | |
| *** hamzy_ has joined #openstack-nova | 14:28 | |
| mriedem | there are some unit tests setting max_attempts to 1 but i'm having a hard time seeing what they assert | 14:29 |
| *** hamzy has quit IRC | 14:29 | |
| mriedem | probably just need a simple tests where max_attempts=1, single instance in the request, and make sure we don't mess with alternate calculation | 14:29 |
| mriedem | *test | 14:29 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Rename spec to match blueprint https://review.openstack.org/568870 | 14:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/569127 | 14:29 |
| mriedem | dansmith: want to keep this backport train a comin https://review.openstack.org/#/c/568369/ | 14:29 |
| dansmith | belmorei_: tssurya ^ is what I expect we'll do | 14:29 |
| dansmith | *we'll need | 14:30 |
| belmorei_ | this is an interesting issue. We get in average 20s for scheduling... We are debugging and let you know when we have more info | 14:30 |
| dansmith | 20s to schedule one? that's.. uh, terrible | 14:30 |
| *** tetsuro has quit IRC | 14:31 | |
| *** ispp has quit IRC | 14:31 | |
| *** ispp has joined #openstack-nova | 14:32 | |
| melwitt | yeah ... | 14:33 |
| *** jaypipes has quit IRC | 14:34 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 14:37 |
| openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 14:37 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Normalize inventory from update_provider_tree https://review.openstack.org/569132 | 14:37 |
| efried | bhagyashris_: ^ | 14:37 |
| dansmith | belmorei_: tssurya: something else to maybe look at.. by default we ask for 1000 placement results on each schedule, which is probably insane | 14:42 |
| dansmith | you might try tweaking down max_placement_results to see if that makes a difference | 14:43 |
| dansmith | depending on where you see the hotspot is | 14:43 |
| *** ykarel is now known as ykarel|away | 14:43 | |
| belmorei_ | in our case that shouldn't happen because we use the request_filter | 14:43 |
| *** hamzy_ is now known as hamzy | 14:43 | |
| dansmith | what do you mean? | 14:44 |
| dansmith | you mean because you're limiting to a cell with only 200 hosts? | 14:44 |
| belmorei_ | the request_filter to use placement_aggregates | 14:44 |
| dansmith | yeah, okay | 14:44 |
| dansmith | well, still, on an empty cell you'd get back the whole cell, but fair enough | 14:45 |
| mriedem | are the compute node resource providers in placement within aggregates? | 14:45 |
| dansmith | mriedem: I think that's the implication :) | 14:45 |
| mriedem | in other words, are you getting back 200 allocation candidates or 10K? | 14:45 |
| mriedem | sure but have they asserted that's all linked up properly | 14:45 |
| dansmith | of they didn't, they would get back zero | 14:45 |
| mriedem | sorry, i mean 1K | 14:45 |
| dansmith | unless they set required=false | 14:45 |
| belmorei_ | depends in the project... but should get in average less than 200 | 14:46 |
| mriedem | because we'd say give me allocation candidates that are a member of this aggregate, and if the rp isn't in that aggregate we'd get back 0, right | 14:46 |
| dansmith | mriedem: yeah | 14:46 |
| *** ykarel|away has quit IRC | 14:49 | |
| *** abhishekk has quit IRC | 14:50 | |
| *** dklyle has joined #openstack-nova | 14:54 | |
| openstackgerrit | Merged openstack/osc-placement master: Fix error message asserts in functional test https://review.openstack.org/569048 | 14:57 |
| *** yamamoto has quit IRC | 14:58 | |
| *** artom has joined #openstack-nova | 15:00 | |
| melwitt | lyarwood: thanks for jumping on https://bugs.launchpad.net/nova/+bug/1771700 | 15:01 |
| openstack | Launchpad bug 1771700 in OpenStack Compute (nova) "nova-lvm tempest job failing with InvalidDiskInfo" [High,In progress] - Assigned to Lee Yarwood (lyarwood) | 15:01 |
| lyarwood | melwitt: well, sorry for breaking nova-lvm ^_^ | 15:01 |
| melwitt | mriedem: lyarwood had an idea of running the nova-lvm job non-voting on nova/virt/libvirt/* changes. what do you think? | 15:02 |
| melwitt | no worries | 15:02 |
| mriedem | why non-voting? | 15:03 |
| mriedem | i think it is stable otherwise | 15:03 |
| *** yikun_ has joined #openstack-nova | 15:04 | |
| mriedem | we just don't gate on it because it's not a highly deployed image backend | 15:04 |
| melwitt | I dunno. voting then? | 15:04 |
| mriedem | we also had a raw imagebackend job at one point | 15:04 |
| *** sidx64 has quit IRC | 15:04 | |
| mriedem | or maybe that was just a dnm patch i had... | 15:04 |
| mriedem | let's see how it looks on lee's fix first | 15:05 |
| *** takashin has left #openstack-nova | 15:05 | |
| mriedem | 1 hour and 20 minutes... | 15:05 |
| mriedem | but it passes | 15:05 |
| mriedem | maybe we should just run the nova-lvm job on compute api tests | 15:05 |
| mriedem | non-voting to start | 15:05 |
| mriedem | and only on libvirt changes | 15:05 |
| melwitt | I think that would be helpful | 15:06 |
| *** yikun__ has quit IRC | 15:07 | |
| openstackgerrit | Merged openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 15:07 |
| *** fragatin_ has quit IRC | 15:08 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | 15:09 |
| mriedem | lyarwood: let's make sure we don't regress the raw + vfat config drive + live migration case :) ^ | 15:09 |
| *** ykarel|away has joined #openstack-nova | 15:09 | |
| *** fragatina has joined #openstack-nova | 15:10 | |
| *** eharney has quit IRC | 15:10 | |
| lyarwood | mriedem: thanks, I'd like to get off of this regression merry-go-round eventually :) | 15:11 |
| *** AlexeyAbashkin has quit IRC | 15:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Deduplicate config/policy reference docs from main index https://review.openstack.org/564846 | 15:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425 | 15:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578 | 15:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for resource providers inventories https://review.openstack.org/568666 | 15:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for usages https://review.openstack.org/568706 | 15:12 |
| mriedem | lyarwood: job security! | 15:12 |
| mriedem | per my comments in https://review.openstack.org/#/c/567860/ i'd also like to diversify the live migration tests we run | 15:13 |
| mriedem | to mix up the image format | 15:13 |
| mriedem | and to use config drive | 15:13 |
| mriedem | we already run the nova-live-migration job and the neutron-tempest-multinode-full job runs live migration, | 15:14 |
| mriedem | so we can make nova-live-migration run with force_config_drive=True, | 15:14 |
| mriedem | and nova-live-migration runs with 2 image formats today, first is default (qcow2) and 2nd is rbd, we could change that first run to be raw | 15:15 |
| *** eharney has joined #openstack-nova | 15:15 | |
| mriedem | and then neutron-tempest-multinode-full would test qcow2 (default) with no config drive (default) | 15:15 |
| *** AlexeyAbashkin has joined #openstack-nova | 15:15 | |
| mriedem | we still wouldn't have vfat config drive format testing, but i suspect we can remove the vfat config_drive_format option/workaround | 15:15 |
| mriedem | i think that was for old versions of libvirt and we now require a minimum that has that fixed | 15:16 |
| *** pchavva has joined #openstack-nova | 15:17 | |
| *** yamamoto has joined #openstack-nova | 15:19 | |
| *** jpena is now known as jpena|off | 15:22 | |
| *** yamamoto has quit IRC | 15:23 | |
| *** cz2 has quit IRC | 15:24 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-lvm run in check on libvirt changes and compute API tests https://review.openstack.org/569149 | 15:25 |
| mriedem | melwitt: lyarwood: ^ | 15:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-lvm run in check on libvirt changes and compute API tests https://review.openstack.org/569149 | 15:25 |
| lyarwood | mriedem: ack thanks! | 15:25 |
| *** ragiman has quit IRC | 15:25 | |
| *** gbarros has quit IRC | 15:29 | |
| *** ispp has quit IRC | 15:31 | |
| *** jpena|off is now known as jpena | 15:33 | |
| *** germs has joined #openstack-nova | 15:34 | |
| *** germs has quit IRC | 15:34 | |
| *** germs has joined #openstack-nova | 15:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | 15:34 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-lvm run in check on libvirt changes and compute API tests https://review.openstack.org/569149 | 15:34 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | 15:35 |
| *** ispp has joined #openstack-nova | 15:37 | |
| *** throwsb1 has joined #openstack-nova | 15:37 | |
| *** damien_r has quit IRC | 15:37 | |
| *** damien_r has joined #openstack-nova | 15:37 | |
| *** germs has quit IRC | 15:39 | |
| *** yamamoto has joined #openstack-nova | 15:40 | |
| *** germs has joined #openstack-nova | 15:40 | |
| *** germs has quit IRC | 15:40 | |
| *** eharney has quit IRC | 15:41 | |
| *** gyee has joined #openstack-nova | 15:41 | |
| *** germs has joined #openstack-nova | 15:41 | |
| *** germs has quit IRC | 15:41 | |
| *** germs has joined #openstack-nova | 15:41 | |
| *** ispp has quit IRC | 15:41 | |
| openstackgerrit | Merged openstack/nova master: Fix auth_url example in hypervisor-hyper-v.rst https://review.openstack.org/568983 | 15:42 |
| *** dklyle has quit IRC | 15:43 | |
| *** gongysh has quit IRC | 15:44 | |
| *** yamamoto has quit IRC | 15:45 | |
| *** gbarros has joined #openstack-nova | 15:46 | |
| mriedem | tssurya: do you need to rebase https://review.openstack.org/#/c/567878/ on top of https://review.openstack.org/#/c/568271/ or just wait for https://review.openstack.org/569112/ ? | 15:47 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 15:47 |
| *** dklyle has joined #openstack-nova | 15:48 | |
| *** lucasagomes is now known as lucas-afk | 15:48 | |
| *** alexchadin has quit IRC | 15:50 | |
| *** armaan has quit IRC | 15:50 | |
| *** armaan has joined #openstack-nova | 15:51 | |
| *** cz2 has joined #openstack-nova | 15:52 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 15:56 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 15:56 |
| *** germs has quit IRC | 15:57 | |
| *** germs has joined #openstack-nova | 15:58 | |
| *** germs has quit IRC | 15:58 | |
| *** germs has joined #openstack-nova | 15:58 | |
| *** germs has quit IRC | 15:58 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | 15:59 |
| *** damien_r has quit IRC | 15:59 | |
| openstackgerrit | Merged openstack/nova master: Deprecate running API services under eventlet https://review.openstack.org/549510 | 15:59 |
| *** germs has joined #openstack-nova | 15:59 | |
| *** germs has quit IRC | 15:59 | |
| *** germs has joined #openstack-nova | 15:59 | |
| openstackgerrit | Merged openstack/nova master: Simplify BDM boot index checking https://review.openstack.org/564480 | 16:01 |
| *** yamamoto has joined #openstack-nova | 16:01 | |
| *** felipemonteiro has joined #openstack-nova | 16:02 | |
| *** yamamoto has quit IRC | 16:06 | |
| *** chyka has joined #openstack-nova | 16:10 | |
| *** felipemonteiro has quit IRC | 16:12 | |
| *** wwriverrat has quit IRC | 16:12 | |
| *** do3meli has quit IRC | 16:13 | |
| melwitt | mriedem: do you see any issue with us increasing the max number of allowed attached volumes per instance from 26 to 1024? I was thinking to approve this specless bp https://blueprints.launchpad.net/nova/+spec/nova-improvement-of-maximum-attach-volumes-more-than-26-vols | 16:13 |
| mriedem | i am not the person to ask about that... | 16:14 |
| mriedem | should ask kvm people | 16:14 |
| mriedem | like kashyap re danpb | 16:14 |
| melwitt | I already talked to danpb about it and he suggested 1024 as the new max | 16:15 |
| mriedem | because of the default fd? | 16:15 |
| melwitt | was just thinking to ask you since you've worked on the multi-attach stuff | 16:15 |
| mriedem | er ulimit used to default to 1024 | 16:15 |
| mriedem | multiattach volumes are >1 instance to the same volume, | 16:16 |
| mriedem | not >1 volumes to the same instance | 16:16 |
| mriedem | default quota on volumes is 10 anyway | 16:16 |
| *** burt has quit IRC | 16:16 | |
| mriedem | so a normal tenant is going to have at most 10 volumes attached to a server | 16:16 |
| mriedem | if you're using powervm, i think you get up to like 256 or something | 16:16 |
| mriedem | efried: ^ | 16:16 |
| melwitt | oh, right. okay | 16:16 |
| mriedem | because powervm just f'ing loves volumes | 16:17 |
| mriedem | also, | 16:17 |
| mriedem | this reminds me, | 16:17 |
| melwitt | picked 1024 arbitrarily based on the field tests described here https://rwmj.wordpress.com/2017/04/25/how-many-disks-can-you-add-to-a-virtual-linux-machine/ | 16:17 |
| mriedem | Ia34d06429c1f8f0a8259616bcba0c349c4c9aa33 | 16:18 |
| melwitt | danpb suggested 1000 | 16:18 |
| mriedem | https://review.openstack.org/#/c/366955/ | 16:18 |
| mriedem | depending on how many volumes you try to attach in a single request, you could blow out the build requests table | 16:18 |
| efried | I'm asking Gerald whether 1024 is enough for us. | 16:18 |
| mriedem | so would 1024 BDMs in a single bfv request get stored in MediumText? | 16:18 |
| melwitt | hm | 16:18 |
| mriedem | MediumText is pretty damn big so i assume yes | 16:19 |
| mriedem | (11:44:41 AM) burgerk: mriedem: broke at ~ 80 volumes, want to go up to 128 | 16:19 |
| mriedem | so powervm was using 128 in that case | 16:19 |
| efried | okay. | 16:20 |
| *** moshele has quit IRC | 16:20 | |
| melwitt | okay, so increasing the max is cool, it might be that 1024 is too high | 16:21 |
| mriedem | MEDIUMTEXT [CHARACTER SET charset_name] [COLLATE collation_name] A TEXT column with a maximum length of 16,777,215 (224 − 1) characters. The effective maximum length is less if the value contains multibyte characters. Each MEDIUMTEXT value is stored using a 3-byte length prefix that indicates the number of | 16:21 |
| mriedem | bytes in the value. | 16:21 |
| mriedem | idk | 16:21 |
| mriedem | someone should test it | 16:21 |
| mriedem | i don't want to guess wtf would happen if someone tries to boot from volume with 1024 volumes | 16:21 |
| *** yamamoto has joined #openstack-nova | 16:21 | |
| mriedem | Tsuyoshi Nagata should test that | 16:22 |
| melwitt | I'll ask | 16:22 |
| mriedem | what is even the practical application of this? | 16:22 |
| * mriedem goes back inside from telling kids to get off his lawn | 16:23 | |
| openstackgerrit | Merged openstack/nova master: Skip ServerActionsTestJSON.test_rebuild_server for cells v1 job https://review.openstack.org/567256 | 16:23 |
| openstackgerrit | Merged openstack/nova master: Remove mox in libvirt/test_driver.py (1) https://review.openstack.org/566555 | 16:23 |
| openstackgerrit | Merged openstack/nova stable/pike: libvirt: Report the virtual size of RAW disks https://review.openstack.org/568369 | 16:23 |
| dansmith | mriedem: I bet it is "because kvm can do it and nova can't" | 16:24 |
| openstackgerrit | Merged openstack/nova master: Remove mox in test_live_migrate.py https://review.openstack.org/564656 | 16:24 |
| dansmith | not for any real reason | 16:24 |
| openstackgerrit | Merged openstack/nova master: PowerVM Driver: DiskAdapter parent class https://review.openstack.org/549053 | 16:24 |
| mriedem | NFV something or other | 16:24 |
| dansmith | or "because kvm can do it and nova won't let you" | 16:24 |
| *** burt has joined #openstack-nova | 16:24 | |
| mriedem | in today's modern high-demand enterprise telco world, 26 volumes is for babies | 16:24 |
| melwitt | I had thought it was something to do with people attaching a lot of volumes for HA or something? | 16:24 |
| mriedem | that's a hell of a lot of HA :) | 16:25 |
| melwitt | I remember it was mentioned in the nova/cinder session at the PTG in the context of the bulk volume create/attach discussion | 16:25 |
| mriedem | this needs to be....super fucking redundant | 16:25 |
| mriedem | all i took out of that discussion was, | 16:25 |
| mriedem | "the storage backend allows it, so we want to make the software do it" | 16:25 |
| mriedem | "because the admin can attach 500 volumes in 2 hours rather than 10, but they'll never touch this thing again after that" | 16:26 |
| mriedem | so i'm kind of -1 on this blueprint just based on the "why?" | 16:26 |
| *** yamamoto has quit IRC | 16:26 | |
| dansmith | yar | 16:26 |
| mriedem | where is ken'ichi when i want to praise him | 16:27 |
| melwitt | okay. I thought it had been discussed in the past that some people attach a lot of volumes for an HA use case | 16:28 |
| mriedem | idk maybe, that's certainly an argument for volume multiattach | 16:28 |
| mriedem | but that's not attaching >1 volume to an instance | 16:28 |
| melwitt | and maybe 1024 is too high but I was thinking it didn't sound unreasonable to increase it > 26 for such a use case | 16:28 |
| mriedem | and i don't see how 1024 is sane HA | 16:28 |
| mriedem | sure, i mean maybe 128 like the powervm thing | 16:29 |
| mriedem | but need to actually justify why we should make this change | 16:29 |
| mriedem | not just because we can | 16:29 |
| melwitt | yeah, I think the point of the blueprint is > 26 and we were trying to find a reasonable max | 16:29 |
| mriedem | sure, that's just not the first question i'd ask | 16:29 |
| mriedem | i want shelve to send me an email when it's done | 16:30 |
| mriedem | not because i need it, just because i can make it happen | 16:30 |
| mriedem | i actually think we've had that blueprint...the email thing anyway | 16:30 |
| melwitt | okay, considering we've raised it to 128 for powervm and 256 for virtio-scsi, I didn't think we needed to ask about > 26 for this case as to why. others are attaching a lot more devices than 26 | 16:30 |
| dansmith | mriedem: melwitt one way to look at it is 1024 is probably more than you could expect reasonable throughput from any of them on, given the number of host threads you'd need for them | 16:31 |
| melwitt | sure ... I'm sorry I picked 1024, I didn't know it would be way too high. they didn't ask for 1024, they actually started with MAX_INT and I wanted to lower it to something more reasonable | 16:33 |
| *** sambetts is now known as sambetts|afk | 16:33 | |
| dansmith | tbh, I don't think it matters that much.. if it's not a big change, then whatever, but the justification can't be "because this is a legit real-world use case" | 16:34 |
| *** tesseract has quit IRC | 16:38 | |
| melwitt | okay, I'll ask them to lower the proposed max and re-ask how many volumes they need to attach for their use case | 16:38 |
| *** bhujay has quit IRC | 16:39 | |
| *** mdnadeem has quit IRC | 16:41 | |
| edleafe | dansmith: your change https://review.openstack.org/#/c/569127/ is failing unit tests due to the one-off change. Want me to fix it? | 16:41 |
| *** dklyle has quit IRC | 16:42 | |
| *** yamamoto has joined #openstack-nova | 16:42 | |
| dansmith | edleafe: ah, I figured it would.. I can do it | 16:42 |
| edleafe | dansmith: ok | 16:42 |
| dansmith | glad we have coverage on that | 16:42 |
| *** mdnadeem has joined #openstack-nova | 16:43 | |
| belmorei_ | dansmith: me and tssurya were debugging the scheduler time. It's related with https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L772 | 16:45 |
| belmorei_ | creating a dict with all instances info per host | 16:46 |
| dansmith | per host we're considering each time we run through yeah? | 16:48 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300 | 16:48 |
| belmorei_ | yeah | 16:48 |
| *** claudiub has joined #openstack-nova | 16:48 | |
| *** yamamoto has quit IRC | 16:49 | |
| dansmith | belmorei_: what is the slow part exactly? it looks like it's hitting a cell db once per host instead of grabbing a bunch at a time maybe? | 16:49 |
| melwitt | I've actually seen that complaint before and I don't understand how compute isn't sending updates to the scheduler. or does that not happen anymore because of placement? | 16:49 |
| mriedem | it does | 16:49 |
| dansmith | melwitt: that would be an upcall | 16:49 |
| mriedem | if configured to do so | 16:49 |
| dansmith | right | 16:50 |
| melwitt | I see | 16:50 |
| mriedem | it's an rpc cast from all computes to the scheduler topic | 16:50 |
| mriedem | it's only used for the affinity filters | 16:50 |
| mriedem | track_instance_changes or something like that | 16:50 |
| dansmith | yeah, and what I'm not getting is why this would be different for them in the new arrangement,m | 16:50 |
| mriedem | https://docs.openstack.org/nova/latest/configuration/config.html#filter_scheduler.track_instance_changes | 16:50 |
| dansmith | because they're only considering hosts from one cell at this point anyway | 16:50 |
| mriedem | dansmith: i'm wondering the same | 16:50 |
| melwitt | yeah. lemme see if I can find the bug I'm thinking of | 16:51 |
| bauzas | is someone able to tell me how to simply just associate an instance and a floating IP, without going thru the list of ports ? | 16:51 |
| bauzas | we had that in the past, but we removed that here | 16:52 |
| melwitt | this one https://bugs.launchpad.net/nova/+bug/1737465 | 16:52 |
| openstack | Launchpad bug 1737465 in OpenStack Compute (nova) "[cellv2] the performance issue of cellv2 when creating 500 instances concurrently" [Undecided,Confirmed] - Assigned to Jiang (jiangpf) | 16:52 |
| belmorei_ | dansmith not following... hosts from one cell? | 16:52 |
| mriedem | bauzas: I CAN | 16:52 |
| mriedem | bauzas: https://docs.openstack.org/python-openstackclient/pike/cli/command-objects/floating-ip.html#floating-ip-set | 16:52 |
| mriedem | get the port id from the instance | 16:52 |
| mriedem | then associate the port to hte floating ip | 16:52 |
| bauzas | mriedem: port is required | 16:52 |
| mriedem | or create the floating ip with the port | 16:52 |
| dansmith | belmorei_: that code should be building a dict of host info, but only from the list of uuids that we got back from placement, which should be only one cell, and only the subset of those that are candidates | 16:52 |
| mriedem | bauzas: yes the port is what links the instance to the floating ip | 16:53 |
| bauzas | mriedem: so I need to openstack port list before | 16:53 |
| bauzas | I was thinking we could somehow magically avoid one roundtrip | 16:53 |
| *** hongbin has joined #openstack-nova | 16:53 | |
| bauzas | with openstack CLI doing that in secret for me | 16:53 |
| mriedem | you can do it on a single command line if you want... | 16:53 |
| bauzas | awk my love | 16:53 |
| mriedem | no | 16:53 |
| mriedem | NO | 16:53 |
| mriedem | PORT1=$(openstack port list --device-id $SERVER1 -c ID -f value) | 16:54 |
| mriedem | FLOATINGIP1=$(openstack floating ip create --port $PORT1 public -c floating_ip_address -f value) | 16:54 |
| mriedem | do that in a single line | 16:54 |
| bauzas | meh | 16:54 |
| belmorei_ | dansmith: yes, but takes a lot of time even for 200 nodes | 16:54 |
| mriedem | then $$$ me | 16:54 |
| mriedem | bauzas: meh? | 16:54 |
| dansmith | belmorei_: but it should have been doing that before in the same way in the cell scheduler I mean | 16:54 |
| bauzas | mriedem: yeah, meh. | 16:54 |
| mriedem | bauzas: see channel topic | 16:54 |
| bauzas | mriedem: hahah | 16:54 |
| mriedem | love my idea or get out | 16:54 |
| dansmith | mriedem: nice | 16:54 |
| bauzas | anyway, you're right | 16:54 |
| bauzas | that's not a dev call :p | 16:55 |
| bauzas | mriedem: FWIW, you were confused with the status of libvirt vGPUs | 16:55 |
| bauzas | mriedem: the branch is up for reviews, and I just updated the spec | 16:55 |
| dansmith | belmorei_: maybe it was taking just as long before in the cell, but it wasn't as obvious because it was spread out? | 16:55 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM snapshot cleanup https://review.openstack.org/568988 | 16:55 |
| melwitt | dansmith: what do you mean by "cell scheduler"? | 16:56 |
| dansmith | belmorei_: or did you have track_instance_changes enabled? I wouldn't think affinity would work with cellsv1 so I would have assumed no | 16:56 |
| bauzas | mriedem: jay had some concerns with traits, but I think we should do that in a separate spec | 16:56 |
| dansmith | melwitt: in cellsv1 there is a scheduler in the cell | 16:56 |
| belmorei_ | dansmith: maybe | 16:56 |
| belmorei_ | dansmith: track_instance_changes is enabled | 16:56 |
| melwitt | okay. well, with cells v1, wouldn't compute be reporting updates to the scheduler and then it wouldn't build that dict? | 16:56 |
| mriedem | belmorei_: was it disabled when you were doing cells v1? | 16:56 |
| dansmith | melwitt: that's what I just said above | 16:56 |
| bauzas | melwitt: dansmith: catching up the convo, what's the problem with computes updating the scheduler ? | 16:56 |
| dansmith | but I thought we still had to hit the db once per host even with that enbabled | 16:57 |
| bauzas | there is a fanout upcall | 16:57 |
| melwitt | ? so doesn't that make it obvious why it was fast with cells v1 and slow with cells v2? | 16:57 |
| melwitt | because with cells v2 it's building the dict every time in the absence of updates? I'm confused | 16:57 |
| dansmith | melwitt: I had asked him if they had it enabled | 16:57 |
| dansmith | melwitt: affinity doesn't work with cellsv1 so I would have expected they didn't have that enabled | 16:57 |
| dansmith | (afaik) | 16:57 |
| melwitt | okay, I see | 16:58 |
| *** sidx64 has joined #openstack-nova | 16:58 | |
| dansmith | and, I thought we still hit the db once per host even with it enabled for the compute info, but not for the instance info part | 16:58 |
| *** amoralej is now known as amoralej|off | 16:58 | |
| dansmith | so I wouldn't expect a lage difference | 16:58 |
| melwitt | it probably works for them by way of patches, maybe | 16:58 |
| dansmith | *large | 16:58 |
| dansmith | well, then they're cheating :) | 16:58 |
| melwitt | what's the compute info part? | 16:59 |
| melwitt | the normal host states stuff? | 17:00 |
| dansmith | yeah | 17:00 |
| *** mdnadeem has quit IRC | 17:00 | |
| belmorei_ | ok, we will continue to dig on this | 17:01 |
| belmorei_ | thanks | 17:01 |
| belmorei_ | I'm leaving now | 17:01 |
| *** jpena is now known as jpena|off | 17:01 | |
| mriedem | random thinking out loud, i think the instance info list only uses the uuid from the instances for the affinity filters, | 17:01 |
| melwitt | belmorei_: seeya next week | 17:02 |
| *** belmorei_ has quit IRC | 17:02 | |
| mriedem | we could probably use placement to just pull uuids of consumers that have allocations on a given node and replace that | 17:02 |
| *** ccamacho has quit IRC | 17:02 | |
| mriedem | would be a lot fast | 17:02 |
| mriedem | *faster | 17:02 |
| dansmith | well, it depends on which piece of his linked function is taking a while | 17:02 |
| dansmith | we do hit the db for the hosts every time, | 17:02 |
| dansmith | but the instance list is the thing we would skip with track_changes | 17:02 |
| dansmith | so yeah, maybe it's just that | 17:02 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L753 could also avoid some default joins... | 17:03 |
| dansmith | still once call to placement per instance would be not great | 17:03 |
| dansmith | er per compute | 17:03 |
| mriedem | i meant per compute | 17:03 |
| mriedem | right now it looks like this does a full instance list per host including default joined tables | 17:03 |
| mriedem | which is i think info_cache and security_groups | 17:03 |
| mriedem | and all the affinity filters care about is the uuid | 17:03 |
| dansmith | yeah probably | 17:03 |
| melwitt | good find | 17:04 |
| *** AlexeyAbashkin has quit IRC | 17:04 | |
| mriedem | https://blueprints.launchpad.net/nova/+spec/put-host-manager-instance-info-on-a-diet | 17:04 |
| *** dtantsur is now known as dtantsur|afk | 17:04 | |
| mriedem | maybe i should have worked on this blueprint some more | 17:04 |
| *** ykarel|away has quit IRC | 17:05 | |
| *** yamamoto has joined #openstack-nova | 17:05 | |
| dansmith | it would be even better to just have a single db call that would take the compute uuids, and return a dict of {host: [uuid, ]} | 17:05 |
| *** derekh has quit IRC | 17:05 | |
| dansmith | although it does have a full instance object in the result there.. are we sure that's not used? | 17:05 |
| mriedem | pretty sure, read that blueprint | 17:05 |
| *** mgoddard has quit IRC | 17:06 | |
| mriedem | it used to also pull the instance.instance_type_id for the TypeAffinityFilter, but we removed that filter | 17:06 |
| *** tssurya has quit IRC | 17:06 | |
| mriedem | s/pull/use/ | 17:06 |
| dansmith | yeah, but any filter out there that expects a full list of instances in the host state will care | 17:06 |
| mriedem | sure, but. | 17:06 |
| mriedem | f'em | 17:06 |
| dansmith | we could put fake things in there that would lazy-load if they touch them :) | 17:07 |
| dansmith | but okay yeah, that bp text makes sense | 17:08 |
| mriedem | so if you have a filter that cares about the network info cache, and we stop auto-joining on that by default, you'd start lazy-loading and see a perf hit during scheduling | 17:08 |
| dansmith | (he says noting he's the approver) | 17:08 |
| dansmith | yeah | 17:08 |
| dansmith | makes it look like their fault though! :) | 17:08 |
| mriedem | but at that point, you either (1) live with it (2) fork the host manager code (3) upstream your filter | 17:08 |
| *** ykarel has joined #openstack-nova | 17:09 | |
| dansmith | it'd be nice to get some clarification from them that this is indeed where their hotspot is (grabbing that large instance list) | 17:09 |
| *** yamamoto has quit IRC | 17:10 | |
| mriedem | yeah and they are both gone now | 17:10 |
| mriedem | tired of our antics | 17:10 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update instance.availability_zone during live migration https://review.openstack.org/567701 | 17:11 |
| melwitt | did y'all look at the bug I linked earlier? https://bugs.launchpad.net/nova/+bug/1737465 | 17:14 |
| openstack | Launchpad bug 1737465 in OpenStack Compute (nova) "[cellv2] the performance issue of cellv2 when creating 500 instances concurrently" [Undecided,Confirmed] - Assigned to Jiang (jiangpf) | 17:14 |
| dansmith | yeah | 17:14 |
| dansmith | I'm sure it's the same thing, but I can't really make sense of most of the text in there | 17:15 |
| melwitt | okay, I was thinking they're hitting the same thing in _get_instance_info and that's doing the instance list, it doesn't do anything else | 17:16 |
| melwitt | yeah, it's not easy to read but they did point out it was the call to _get_instance_info where they get the slowdown | 17:16 |
| dansmith | yeah, that's why I say I'm sure it's the same | 17:17 |
| mriedem | "Because cellv2 have disable instance info cache." - i assume that means they disabled track_instance_changes | 17:17 |
| melwitt | yeah, me too. so you're thinking there could possibly be something else in _get_instance_info that could cause a slowdown other than the instance list, when you said it would be good if they could clarify? | 17:17 |
| dansmith | no, I'm just saying it would be good to clarify, since he linked at a rather high-up spot | 17:18 |
| *** sidx64_ has joined #openstack-nova | 17:18 | |
| melwitt | mriedem: I think it means because the upcall is disallowed by default, yeah? | 17:18 |
| *** mdbooth has quit IRC | 17:18 | |
| melwitt | dansmith: oh, okay | 17:18 |
| mriedem | melwitt: no | 17:18 |
| mriedem | track_instance_changes is on by default, | 17:18 |
| mriedem | if he's using devstack, then yes that upcall is disabled | 17:19 |
| mriedem | https://docs.openstack.org/nova/latest/configuration/config.html#workarounds.disable_group_policy_check_upcall is not disabled by default though | 17:19 |
| melwitt | but with cells v2 it won't cache it, because the upcall is disabled by default, right? oh nevermind | 17:19 |
| mriedem | so out of the box brand new install that's not devstack is going to be doing the rpc blast upcall | 17:19 |
| mriedem | no | 17:19 |
| mriedem | has nothing to do with cells v2 | 17:19 |
| *** sidx64 has quit IRC | 17:20 | |
| melwitt | yeah, I was thinking maybe that upcall was disabled by default because of cells v2. but it's not | 17:21 |
| dansmith | it just won't ever make it to the scheduler | 17:21 |
| mriedem | if you're doing split MQ | 17:21 |
| mriedem | i'd be kind of surprised if someone was following that setup if they didn't know what they were doing | 17:21 |
| melwitt | ohh... | 17:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Metadata-API fails to retrieve avz for instances created before Pike https://review.openstack.org/567878 | 17:24 |
| *** ykarel_ has joined #openstack-nova | 17:25 | |
| *** yamamoto has joined #openstack-nova | 17:26 | |
| *** ykarel has quit IRC | 17:27 | |
| *** janki has quit IRC | 17:29 | |
| *** ykarel_ has quit IRC | 17:30 | |
| *** yamamoto has quit IRC | 17:30 | |
| *** imacdonn has quit IRC | 17:31 | |
| *** imacdonn has joined #openstack-nova | 17:31 | |
| dansmith | edleafe: hmm, so I'm a bit confused actually | 17:32 |
| *** udesale has joined #openstack-nova | 17:32 | |
| *** udesale has quit IRC | 17:33 | |
| *** ykarel has joined #openstack-nova | 17:33 | |
| *** psachin has quit IRC | 17:37 | |
| *** ivve has joined #openstack-nova | 17:41 | |
| *** gjayavelu has joined #openstack-nova | 17:44 | |
| edleafe | dansmith: do tell | 17:45 |
| dansmith | edleafe: something is off by one, one way or the other and I'm not sure I get it | 17:46 |
| *** mgoddard has joined #openstack-nova | 17:46 | |
| dansmith | either the test is wrong or needs some other tweak, or | 17:46 |
| dansmith | something | 17:46 |
| edleafe | I have a few minutes - let me look | 17:46 |
| dansmith | edleafe: anyway, I've been working on it in parallel to other things and I think I've confused myself out of the way | 17:46 |
| dansmith | so yeah, maybe you should look without me telling you what I'm confused about :) | 17:46 |
| *** sidx64_ has quit IRC | 17:46 | |
| edleafe | I'd much rather confuse myself :) | 17:47 |
| *** yamamoto has joined #openstack-nova | 17:48 | |
| *** tomtom001 has quit IRC | 17:52 | |
| *** moshele has joined #openstack-nova | 17:53 | |
| *** yamamoto has quit IRC | 17:53 | |
| *** tomtom001 has joined #openstack-nova | 17:54 | |
| *** felipemonteiro has joined #openstack-nova | 17:54 | |
| edleafe | dansmith: Looks like this needs to be adjusted, too: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L367 | 17:56 |
| dansmith | edleafe: yeah, I did that | 17:56 |
| dansmith | and I can make the tests pass at that point, | 17:56 |
| dansmith | but ...isn't that just subtracting one from A and adding it to B? | 17:56 |
| edleafe | nope | 17:56 |
| dansmith | well, assuming you mean changing >= to > | 17:57 |
| edleafe | before your change, "num_alts" really was the total returned | 17:57 |
| edleafe | not the number of alternates | 17:57 |
| edleafe | change that line to: >= num_alts + 1 | 17:58 |
| edleafe | the '1' is the 'selected' in 'selected_plus_alts' | 17:58 |
| *** links has joined #openstack-nova | 17:58 | |
| *** ykarel has quit IRC | 17:59 | |
| dansmith | ah, heh, that actually makes it click | 17:59 |
| dansmith | changing it to >, which is what I did felt wrong | 17:59 |
| dansmith | but leaving >= and putting the +1 makes it logically make sense to me | 17:59 |
| edleafe | yeah | 17:59 |
| edleafe | I'm re-running the unit tests | 18:00 |
| edleafe | I can push that assuming that the tests pass | 18:00 |
| dansmith | alright | 18:00 |
| mriedem | melwitt: +2 on lyarwood's nova-lvm fix https://review.openstack.org/#/c/569062/ | 18:02 |
| edleafe | dansmith: should that patch still be a WIP? | 18:07 |
| dansmith | edleafe: nah I just WIP'd it because no tests | 18:07 |
| edleafe | k | 18:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fakelibvirt migrateToURI3 should provide args according to libvirt doc https://review.openstack.org/569047 | 18:08 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/569127 | 18:08 |
| edleafe | dansmith: Done ^^ | 18:09 |
| *** tomtom001 has quit IRC | 18:09 | |
| *** yamamoto has joined #openstack-nova | 18:09 | |
| melwitt | mriedem: you don't want to wait for lyarwood's test additions? | 18:09 |
| dansmith | edleafe: cool | 18:10 |
| mriedem | unit test? | 18:10 |
| mriedem | was he going to add one? | 18:10 |
| melwitt | on the review he said "nova-lvm is looking good so I'll respin with tests later today." | 18:11 |
| mriedem | yeah, ok | 18:11 |
| edleafe | mriedem: the tests already covered that | 18:11 |
| edleafe | that's why they broke when he made the change | 18:11 |
| dansmith | edleafe: well, they did, but we probably need one to make sure that we're doing the different thing | 18:12 |
| dansmith | I'm not sure where that should go really though | 18:12 |
| *** tomtom001 has joined #openstack-nova | 18:12 | |
| mriedem | edleafe: ? | 18:13 |
| mriedem | edleafe: the lvm thing? | 18:13 |
| dansmith | mriedem: oh we thought you were talking about the alternates thing | 18:13 |
| *** yamamoto has quit IRC | 18:13 | |
| dansmith | edleafe: we probably just need to validate that max_attempts=1 gets you 1 result | 18:14 |
| mriedem | no i don't talk about that | 18:14 |
| edleafe | dansmith: does this test satisfy? https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/test_filter_scheduler.py#L915-L917 | 18:22 |
| dansmith | edleafe: no, because the essential change is that max_attempts=1 should end up with zero alternates to filter | 18:23 |
| edleafe | ok, let me add that | 18:25 |
| *** eharney has joined #openstack-nova | 18:27 | |
| *** yamamoto has joined #openstack-nova | 18:30 | |
| *** yamamoto has quit IRC | 18:34 | |
| *** links has quit IRC | 18:35 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/569127 | 18:38 |
| edleafe | dansmith: ^^ now with shiny new unit test | 18:38 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in HostManager._get_instances_by_host https://review.openstack.org/569218 | 18:42 |
| dansmith | edleafe: I forgot about how that one test has num_alternates=0 | 18:44 |
| dansmith | which is kiiinda the same thing | 18:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in HostManager._get_instances_by_host https://review.openstack.org/569218 | 18:44 |
| mriedem | dansmith: melwitt: ^ the small optimization noted earlier | 18:44 |
| *** damien_r has joined #openstack-nova | 18:51 | |
| *** yamamoto has joined #openstack-nova | 18:51 | |
| dansmith | edleafe: you know, actually I think what we probably want is to mock out the interface between _schedule() and _get_alternate_hosts() and make sure that num_alts out and in from those (respectively) includes the new offset | 18:55 |
| *** yamamoto has quit IRC | 18:55 | |
| dansmith | edleafe: because testing them together just hides that we made a change, which is why that new unit test is kinda forced | 18:55 |
| dansmith | edleafe: I'm happy to do that if you want | 18:55 |
| dansmith | ...if you agree | 18:55 |
| *** dklyle has joined #openstack-nova | 18:56 | |
| melwitt | mriedem: ack | 18:57 |
| *** chyka has quit IRC | 19:00 | |
| *** eharney has quit IRC | 19:00 | |
| *** felipemonteiro__ has joined #openstack-nova | 19:04 | |
| edleafe | dansmith: go for it. I have meetings for the next two hours | 19:06 |
| edleafe | dansmith: and looking at the code, it *was* working correctly, just poorly named. If we had simply changed 'num_alts' to 'num_returned', it would have continued to work, and made more sense to someone reading it | 19:07 |
| *** felipemonteiro has quit IRC | 19:08 | |
| *** damien_r has quit IRC | 19:09 | |
| *** agomerz has joined #openstack-nova | 19:10 | |
| *** agomerz has quit IRC | 19:11 | |
| *** agomerz has joined #openstack-nova | 19:11 | |
| *** yamamoto has joined #openstack-nova | 19:12 | |
| *** chyka has joined #openstack-nova | 19:13 | |
| *** yamamoto has quit IRC | 19:16 | |
| *** moshele has quit IRC | 19:19 | |
| *** moshele has joined #openstack-nova | 19:21 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Normalize inventory from update_provider_tree https://review.openstack.org/569132 | 19:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 19:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 19:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.has_inventory_changed for new fields https://review.openstack.org/569228 | 19:22 |
| efried | edleafe, cdent, dansmith: y'all may want to take a look at these ^ | 19:22 |
| efried | the bottom two anyway (which are intended to fix tempest on the top two) | 19:22 |
| efried | bhagyashris_: FYI, hoping those bottom two patches clear up the tempest failures. | 19:24 |
| efried | bhagyashris_: We were running out of hosts in tempest because we were no longer setting allocation ratios. I imagine the 16.0 allocation ratio on VCPU was what was killing us. | 19:24 |
| *** eharney has joined #openstack-nova | 19:25 | |
| *** jmlowe has quit IRC | 19:27 | |
| *** dtantsur|afk has quit IRC | 19:29 | |
| *** yamamoto has joined #openstack-nova | 19:32 | |
| cdent | seems sane, efried, left a query on the second one | 19:33 |
| *** slaweq has quit IRC | 19:34 | |
| efried | cdent: Thanks. Seems like we're reworking the allocation ratio thing - but I think it still is (or can be) driven from conf and/or via the compute node record, and the virt still doesn't necessarily have access to the information. | 19:34 |
| efried | cdent: I really wanted to catch Jay to get the skinny on that, but he seems to be out today. | 19:34 |
| *** slaweq has joined #openstack-nova | 19:35 | |
| efried | cdent: As it stands, if the virt driver thinks it knows better, it can set those and _normalize... won't overwrite them. | 19:35 |
| cdent | Yeah, that was the source of my confusion: given _now_, what you've got seems right | 19:35 |
| efried | Glad we agree on that. | 19:36 |
| *** dtantsur has joined #openstack-nova | 19:36 | |
| efried | And I don't know what all the future is supposed to hold for this. So _now_ is about all I can account for... | 19:36 |
| *** yamamoto has quit IRC | 19:37 | |
| cdent | Indeed | 19:37 |
| * cdent goes to pack | 19:37 | |
| efried | See you in 'couver. | 19:38 |
| *** felipemonteiro__ has quit IRC | 19:39 | |
| efried | mriedem: I caught up with Gerald. He cited a customer scenario with 500 volumes on an instance. So 1024 seems like a safe and relatively future-proof number. | 19:39 |
| *** felipemonteiro has joined #openstack-nova | 19:39 | |
| *** dtantsur has quit IRC | 19:40 | |
| *** sidx64 has joined #openstack-nova | 19:48 | |
| *** moshele has quit IRC | 19:50 | |
| *** dtantsur has joined #openstack-nova | 19:51 | |
| dansmith | efried: um, I had already provided a customer scenario with 600, although no other detail, so clearly 1024 makes sense in a real-world situation | 19:52 |
| efried | dansmith: Okay, I didn't follow that conversation to its end. Just got an answer so thought I would post it. | 19:52 |
| *** sidx64 has quit IRC | 19:52 | |
| *** yamamoto has joined #openstack-nova | 19:53 | |
| dansmith | efried: how is "some customer wants to use 500" a reasonable argument? I guess my overt sarcasm wasn't overt enough | 19:53 |
| efried | now you've lost me. | 19:54 |
| dansmith | efried: I want to hear the *reason* for wanting to use 500 volumes on a single guest, not just that someone thinks it's a good idea | 19:55 |
| efried | You mean we should require said customers to give justification that we agree with, as opposed to letting us tell them how many volumes they should be using instead? | 19:55 |
| efried | yeah, got it. | 19:55 |
| dansmith | the bug/bp being filed seems to indicate that someone somewhere wants to do it | 19:56 |
| efried | In this case, the customer is using IBMi and has 16 FC adapters and is doing huge striping to get desired performance. | 19:56 |
| dansmith | hah, I was going to say raid0ing of 1GB volumes | 19:56 |
| *** avolkov has quit IRC | 19:56 | |
| *** jmlowe has joined #openstack-nova | 19:57 | |
| *** yamamoto has quit IRC | 19:59 | |
| openstackgerrit | Merged openstack/nova master: Deduplicate config/policy reference docs from main index https://review.openstack.org/564846 | 20:07 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Trim the fat on HostState.instances https://review.openstack.org/569247 | 20:09 |
| mriedem | i thorougly enjoy what you have to say for this new config option if you enable it | 20:10 |
| mriedem | ^ | 20:10 |
| cdent | striping across 500 volumes sounds kind of fun. Like: whatever they are doing, they must think it is pretty serious. Thus: fun. | 20:10 |
| dansmith | more fun for the operator that gave a tenant 500 volume quota and realized they attached them all to a single instance to hack around performance limits | 20:11 |
| *** tssurya has joined #openstack-nova | 20:11 | |
| efried | mriedem: Fast-approved | 20:14 |
| mriedem | ha | 20:14 |
| *** wolverineav has quit IRC | 20:16 | |
| *** yamamoto has joined #openstack-nova | 20:16 | |
| tssurya | mriedem: sorry was afk, but thanks for fixing the nits in both the patches | 20:17 |
| dansmith | mriedem: you really want to provide a workaround knob for that? | 20:19 |
| *** sahid has quit IRC | 20:20 | |
| mriedem | dansmith: as in just let it break? | 20:20 |
| dansmith | yeah | 20:20 |
| *** yamamoto has quit IRC | 20:20 | |
| dansmith | I mean, I appreciate the care, but.. | 20:20 |
| mriedem | i didn't see any HostInstance.instances usage in the huawei product code but... | 20:21 |
| dansmith | also, | 20:21 |
| dansmith | if we change how we do that query, we could populate the id field, which would make it actually lazy-load the things they want | 20:21 |
| dansmith | right now it will fail with | 20:21 |
| dansmith | "orphaned object" errors | 20:21 |
| dansmith | or make instance lazy-load by uuid if it doesn'talready | 20:22 |
| * dansmith checks | 20:22 | |
| *** jmlowe has quit IRC | 20:22 | |
| artom | Or just outright own the full thing and change the object to be a list of instance UUIDs? | 20:22 |
| artom | Not sure how that would work for backwards compatibility tho | 20:22 |
| mriedem | so first fellas, WIP clearly | 20:23 |
| dansmith | oh it does load by uuid, so make the context right and it will lazy-load | 20:23 |
| dansmith | mriedem: I know, I'm just asking :) | 20:23 |
| mriedem | second, yeah long term i want to do away with the dict and just make HostState.instances a list of uuids | 20:23 |
| artom | mriedem, too bad, you gave me an excuse to procrastinate real work. Fell the full wrath of my nitpicking. | 20:23 |
| mriedem | dansmith: so {uuid: objects.Instance(cctxt, uuid=uuid)}? | 20:23 |
| dansmith | mriedem: yup | 20:23 |
| mriedem | i fell it real good | 20:23 |
| dansmith | mriedem: then you can just not do the conf option, it'll be slow if they touch it, and we can remove it later | 20:24 |
| mriedem | dansmith: yeah that's agood point | 20:24 |
| mriedem | damn good point | 20:24 |
| dansmith | mriedem: snarky comments still required though | 20:24 |
| dansmith | # NOTE(mriedem): Set the timer on this bomb and walk out casually | 20:25 |
| mriedem | dansmith: setting the context on the instance object won't lazy-load all fields | 20:31 |
| mriedem | just those in INSTANCE_OPTIONAL_ATTRS | 20:31 |
| dansmith | should call _load_generic, which will do anything | 20:31 |
| *** fragatina has quit IRC | 20:32 | |
| mriedem | if attrname not in INSTANCE_OPTIONAL_ATTRS: | 20:32 |
| mriedem | raise exception.ObjectActionError( | 20:32 |
| dansmith | oh, heh, | 20:32 |
| dansmith | I dunno why we did that | 20:32 |
| mriedem | so now my filter on instance.default_ephemeral_device no longer works | 20:32 |
| dansmith | because load_generic will do it | 20:32 |
| *** moshele has joined #openstack-nova | 20:33 | |
| dansmith | it's from 2013, nuke it | 20:33 |
| dansmith | it looks like that was just trying to make it not do an empty load for something it wasn't going to be able to satisfy, | 20:34 |
| dansmith | since it was trying to figure out what to put in extra_attrs | 20:34 |
| *** tssurya has quit IRC | 20:35 | |
| dansmith | and that was before we had load_generic | 20:35 |
| dansmith | so when we added the generic thing we probably just didn't remove the thing at the top to let you use it for anything | 20:35 |
| dansmith | we should also have that fill in any field it gets that isn't already set, | 20:36 |
| dansmith | so you don't load the instance twice for two regular fields | 20:36 |
| dansmith | I can make those changes in one patch if you want | 20:36 |
| mriedem | "we should also have that fill in any field it gets that isn't already set," - i believe it will do that for all non-joined fields anyway | 20:36 |
| mriedem | because it just does a normal get and load from db | 20:36 |
| dansmith | load_generic won't, | 20:36 |
| *** yamamoto has joined #openstack-nova | 20:37 | |
| dansmith | it will only copy over the one thing you asked for | 20:37 |
| mriedem | oh i see | 20:37 |
| mriedem | if instance.obj_attr_is_set(attrname): | 20:37 |
| mriedem | self[attrname] = instance[attrname] | 20:37 |
| openstackgerrit | Merged openstack/nova master: Cleanup ugly stub in TestLocalDeleteAllocations https://review.openstack.org/561997 | 20:37 |
| dansmith | yeah | 20:37 |
| dansmith | I mean, | 20:37 |
| dansmith | it nicely punishes them hard, | 20:37 |
| dansmith | but probably not the best thing to do :P | 20:37 |
| mriedem | i just sent to the ML btw | 20:38 |
| mriedem | efried: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130624.html | 20:38 |
| efried | mriedem: ack | 20:38 |
| *** Guest37211 has quit IRC | 20:39 | |
| mriedem | also related, we probably no longer need to default join on security_groups since that would only ever be populated for nova-network | 20:39 |
| *** ccamacho has joined #openstack-nova | 20:40 | |
| mriedem | dansmith: if you want, what i do need right now is to handle the NOTE in InstanceList.v | 20:40 |
| mriedem | *get_uuids_by_host | 20:40 |
| dansmith | hah | 20:41 |
| *** yamamoto has quit IRC | 20:41 | |
| dansmith | yeah I can do that too | 20:41 |
| dansmith | there has to be some animated gif that applies well to that | 20:41 |
| mriedem | i've been meaning to do that since https://review.openstack.org/#/c/563234/1/nova/api/openstack/compute/services.py@222 | 20:42 |
| dansmith | ooh, yessss, I made unit tests fail with infinite recursion | 20:42 |
| *** pchavva has quit IRC | 20:43 | |
| dansmith | heh, instance will trigger a double lazy load on itself apparently, which I uncovered by removing this check | 20:47 |
| dansmith | that's nice | 20:47 |
| mriedem | you also have a note in that exact code | 20:48 |
| mriedem | about recursion | 20:48 |
| edmondsw | any stable cores around to review https://review.openstack.org/#/c/567599 ? | 20:48 |
| dansmith | I know | 20:48 |
| edmondsw | been out there a while, and it's an easy review | 20:49 |
| *** artom has quit IRC | 20:50 | |
| *** throwsb1 has quit IRC | 20:53 | |
| openstackgerrit | Merged openstack/nova master: Remove mox in test_xenapi.py (3) https://review.openstack.org/564645 | 20:56 |
| *** Guest37211 has joined #openstack-nova | 20:56 | |
| *** yamamoto has joined #openstack-nova | 20:59 | |
| *** fragatina has joined #openstack-nova | 20:59 | |
| *** fragatina has quit IRC | 21:01 | |
| *** fragatina has joined #openstack-nova | 21:02 | |
| *** dtantsur has quit IRC | 21:02 | |
| *** dtantsur has joined #openstack-nova | 21:03 | |
| *** fragatina has quit IRC | 21:03 | |
| *** yamamoto has quit IRC | 21:04 | |
| *** fragatina has joined #openstack-nova | 21:04 | |
| *** wolverineav has joined #openstack-nova | 21:07 | |
| dansmith | mriedem: funny thing happened on the way to the forum: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2596-L2607 | 21:11 |
| mriedem | edmondsw: easy for me to -1 | 21:12 |
| mriedem | ha | 21:14 |
| edmondsw | esberglu ^ | 21:14 |
| mriedem | dansmith: one difference with that, | 21:14 |
| mriedem | the project_only=False kwarg to model_query | 21:14 |
| dansmith | aww | 21:14 |
| mriedem | yup | 21:14 |
| dansmith | well, I'll show you | 21:15 |
| dansmith | er, | 21:16 |
| dansmith | project_only=False by default on mode_query | 21:16 |
| *** esberglu has quit IRC | 21:16 | |
| edmondsw | mriedem this fixes a PowerVM CI breakage. If we regressed, the PowerVM CI would blow up. And the regression would have to be a change in powervm code, so reviewers better be checking the powervm ci. Is that not sufficient? | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Trim the fat on HostState.instances https://review.openstack.org/569247 | 21:17 |
| mriedem | damn you smith | 21:18 |
| mriedem | edmondsw: it'll be a cold day in hell before i... | 21:20 |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: update add-consumer-generation to focus on API https://review.openstack.org/565565 | 21:20 |
| efried | mriedem: ^ | 21:20 |
| *** yamamoto has joined #openstack-nova | 21:20 | |
| efried | and dansmith ^ | 21:21 |
| openstackgerrit | Merged openstack/nova master: Remove unnecessary 'to_primitive' call https://review.openstack.org/568532 | 21:21 |
| openstackgerrit | Merged openstack/nova master: Fakelibvirt migrateToURI3 should provide args according to libvirt doc https://review.openstack.org/569047 | 21:21 |
| *** yamamoto has quit IRC | 21:26 | |
| mriedem | efried: left a comment also | 21:27 |
| *** felipemonteiro has quit IRC | 21:28 | |
| *** felipemonteiro has joined #openstack-nova | 21:28 | |
| *** felipemonteiro has quit IRC | 21:28 | |
| *** felipemonteiro has joined #openstack-nova | 21:28 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make instance able to lazy-load almost everything https://review.openstack.org/569267 | 21:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Expose instance_get_all_uuids_by_host() from DB API and use it https://review.openstack.org/569268 | 21:30 |
| *** agomerz has quit IRC | 21:31 | |
| dansmith | mriedem: ^ | 21:31 |
| dansmith | mriedem: we had pretty anal testing of the load behaviors for various special cases, | 21:33 |
| dansmith | so keeping those tests unchanged and allowing the new case makes me fairly confident in that | 21:34 |
| *** hemna_ has quit IRC | 21:34 | |
| efried | dansmith: The "sentinel proj/user conf option" business was your idea, right? You wanna post a delta on the spec accordingly? | 21:35 |
| efried | Maybe if we get enough hands on this spec, there'll be nobody left to approve it. | 21:35 |
| dansmith | efried: (a) I don't like calling it a sentinel, but (b) I really wish we could just update the spec afterwards, because we've already wandered around a bunch of times | 21:35 |
| efried | dansmith: But Matt will make fun of your haircut if we do that. | 21:36 |
| dansmith | if people really need it then I will, but.. | 21:36 |
| efried | mriedem: your call ^ | 21:36 |
| dansmith | maybe I should so I can put "it's not an effing senintel people" in bold | 21:36 |
| dansmith | that spec will be really yelly at that point | 21:37 |
| dansmith | the angst is well codified | 21:37 |
| *** fragatina has quit IRC | 21:39 | |
| *** mgoddard has quit IRC | 21:40 | |
| *** yamamoto has joined #openstack-nova | 21:43 | |
| mriedem | since i haven't gotten into the code yeah i'd like that | 21:43 |
| *** hughsaunders has quit IRC | 21:44 | |
| *** edmondsw has quit IRC | 21:46 | |
| efried | Either of y'all have any idea how allocation_ratio values are ending up 0.0 here? http://logs.openstack.org/32/569132/2/check/openstack-tox-py35/0ad1d4e/testr_results.html.gz | 21:46 |
| *** fragatina has joined #openstack-nova | 21:46 | |
| *** wolverineav has quit IRC | 21:46 | |
| efried | this test passes for me locally. | 21:47 |
| *** hughsaunders has joined #openstack-nova | 21:47 | |
| *** yamamoto has quit IRC | 21:49 | |
| *** takashin has joined #openstack-nova | 21:49 | |
| efried | okay, I think I figured it out. There's another test *somewhere* stomping on those values. Just gotta track it down... | 21:49 |
| *** lyan has quit IRC | 21:51 | |
| *** chyka_ has joined #openstack-nova | 21:52 | |
| *** chyka has quit IRC | 21:56 | |
| mriedem | "gifts-for-mriedem" | 21:56 |
| mriedem | that's nice | 21:56 |
| *** chyka_ has quit IRC | 21:57 | |
| *** eharney has quit IRC | 21:57 | |
| *** burt has quit IRC | 21:58 | |
| dansmith | heh | 21:59 |
| *** wolverineav has joined #openstack-nova | 22:00 | |
| *** agomerz has joined #openstack-nova | 22:01 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/569127 | 22:02 |
| *** rcernin has joined #openstack-nova | 22:04 | |
| *** hemna_ has joined #openstack-nova | 22:05 | |
| mriedem | dansmith: you missed a question i had in the earlier PS in ^ | 22:05 |
| dansmith | apparently I broke unit tests on that base one | 22:05 |
| *** yamamoto has joined #openstack-nova | 22:05 | |
| mriedem | there might still be an optimization to be made in the multicreate case | 22:05 |
| dansmith | mriedem: I thought you were answering your own question below | 22:06 |
| dansmith | let me re-read | 22:06 |
| dansmith | oh, I see because we're weighing there between them | 22:06 |
| dansmith | er, re-weighing I should say | 22:08 |
| *** baoli has quit IRC | 22:09 | |
| dansmith | I guess I'm not sure why we need to re-weigh there | 22:10 |
| mriedem | per the comment, | 22:10 |
| *** baoli has joined #openstack-nova | 22:10 | |
| mriedem | we "claim" from the selected hosts before we start looking for alternates | 22:10 |
| *** yamamoto has quit IRC | 22:10 | |
| dansmith | right | 22:10 |
| mriedem | so we want to cleanup the alternate hosts before selecting them | 22:10 |
| dansmith | does re-filtering dump out the ones we've claimed or something? | 22:10 |
| mriedem | it might | 22:10 |
| mriedem | if they were filled up | 22:10 |
| dansmith | oh, I see, I thought that was handled by consume? | 22:12 |
| dansmith | otherwise it's no different for the primaries if we have a lot of those | 22:13 |
| dansmith | or.. | 22:13 |
| dansmith | is that why we stripe across hosts when doing num_instances>1? | 22:13 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1) https://review.openstack.org/564658 | 22:13 |
| dansmith | oh jeez, I'm failing like all the functional tests | 22:13 |
| mriedem | https://www.youtube.com/watch?v=gGrNAB45CtY | 22:14 |
| *** baoli has quit IRC | 22:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Trim the fat on HostState.instances https://review.openstack.org/569247 | 22:15 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659 | 22:15 |
| *** felipemonteiro has quit IRC | 22:20 | |
| *** hemna_ has quit IRC | 22:24 | |
| *** hemna_ has joined #openstack-nova | 22:24 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make instance able to lazy-load almost everything https://review.openstack.org/569267 | 22:24 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Expose instance_get_all_uuids_by_host() from DB API and use it https://review.openstack.org/569268 | 22:24 |
| *** yamamoto has joined #openstack-nova | 22:26 | |
| *** chyka has joined #openstack-nova | 22:28 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Fix auth_url example in hypervisor-hyper-v.rst https://review.openstack.org/569284 | 22:29 |
| *** wolverineav has quit IRC | 22:29 | |
| *** yamamoto has quit IRC | 22:31 | |
| *** chyka has quit IRC | 22:32 | |
| *** mriedem is now known as mriedem_afk | 22:36 | |
| *** threestrands has joined #openstack-nova | 22:37 | |
| *** mriedem_afk has quit IRC | 22:38 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Normalize inventory from update_provider_tree https://review.openstack.org/569132 | 22:40 |
| openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: get_inventory => update_provider_tree https://review.openstack.org/560444 | 22:40 |
| openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 22:40 |
| *** gjayavelu has quit IRC | 22:40 | |
| *** gjayavelu has joined #openstack-nova | 22:40 | |
| *** hongbin has quit IRC | 22:40 | |
| *** slaweq has quit IRC | 22:44 | |
| *** slaweq has joined #openstack-nova | 22:44 | |
| openstackgerrit | Merged openstack/nova stable/queens: Stringify instance UUID https://review.openstack.org/567599 | 22:46 |
| *** yamamoto has joined #openstack-nova | 22:47 | |
| *** d34dh0r53 has quit IRC | 22:47 | |
| *** d34dh0r53 has joined #openstack-nova | 22:47 | |
| *** yamamoto has quit IRC | 22:51 | |
| *** mmehan has quit IRC | 22:51 | |
| *** claudiub has quit IRC | 22:52 | |
| *** cdent has quit IRC | 22:54 | |
| *** moshele has quit IRC | 22:55 | |
| *** moshele has joined #openstack-nova | 22:59 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/569127 | 23:03 |
| *** edmondsw has joined #openstack-nova | 23:03 | |
| *** yamamoto has joined #openstack-nova | 23:07 | |
| *** edmondsw has quit IRC | 23:08 | |
| *** yamamoto has quit IRC | 23:12 | |
| *** moshele has quit IRC | 23:23 | |
| *** artom has joined #openstack-nova | 23:24 | |
| *** baoli has joined #openstack-nova | 23:25 | |
| *** baoli has quit IRC | 23:25 | |
| *** yamamoto has joined #openstack-nova | 23:28 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid unnecessary joins in HostManager._get_instances_by_host https://review.openstack.org/569218 | 23:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Trim the fat on HostState.instances https://review.openstack.org/569247 | 23:31 |
| *** yamamoto has quit IRC | 23:32 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Trim the fat on HostState.instances https://review.openstack.org/569247 | 23:33 |
| *** mlavalle has quit IRC | 23:35 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Make nova-lvm run in check on libvirt changes and compute API tests https://review.openstack.org/569149 | 23:41 |
| *** edmondsw has joined #openstack-nova | 23:46 | |
| *** tetsuro has joined #openstack-nova | 23:47 | |
| *** yamamoto has joined #openstack-nova | 23:50 | |
| *** yamamoto has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!