Thursday, 2018-06-07

*** tbachman has joined #openstack-nova00:03
*** heyongli has quit IRC00:05
*** heyongli has joined #openstack-nova00:05
*** shaohe_feng has quit IRC00:06
*** shaohe_feng has joined #openstack-nova00:10
*** heyongli has quit IRC00:15
*** heyongli has joined #openstack-nova00:15
*** bigdogstl has joined #openstack-nova00:15
*** shaohe_feng has quit IRC00:16
*** shaohe_feng has joined #openstack-nova00:17
*** itlinux has joined #openstack-nova00:20
*** fragatina has quit IRC00:21
*** bigdogstl has quit IRC00:23
*** threestrands has quit IRC00:25
*** heyongli has quit IRC00:25
*** heyongli has joined #openstack-nova00:25
*** jmlowe has quit IRC00:26
*** shaohe_feng has quit IRC00:27
*** jmlowe has joined #openstack-nova00:27
*** threestrands has joined #openstack-nova00:28
*** shaohe_feng has joined #openstack-nova00:29
*** tetsuro has joined #openstack-nova00:30
*** heyongli has quit IRC00:35
*** heyongli has joined #openstack-nova00:36
*** shaohe_feng has quit IRC00:37
*** shaohe_feng has joined #openstack-nova00:38
*** heyongli has quit IRC00:46
*** heyongli has joined #openstack-nova00:46
*** shaohe_feng has quit IRC00:47
*** shaohe_feng has joined #openstack-nova00:48
*** hshiina has quit IRC00:52
openstackgerritSteve Kowalik proposed openstack/nova master: Switch to oslo_messaging.ConfFixture.transport_url  https://review.openstack.org/57304600:53
*** heyongli has quit IRC00:56
*** heyongli has joined #openstack-nova00:56
*** shaohe_feng has quit IRC00:57
*** itlinux has quit IRC00:58
*** shaohe_feng has joined #openstack-nova00:59
*** liuzz has joined #openstack-nova01:00
*** itlinux has joined #openstack-nova01:00
*** Nel1x has joined #openstack-nova01:02
*** zhaochao has joined #openstack-nova01:02
*** Dinesh_Bhor has joined #openstack-nova01:04
*** heyongli has quit IRC01:06
*** heyongli has joined #openstack-nova01:06
*** shaohe_feng has quit IRC01:08
*** shaohe_feng has joined #openstack-nova01:08
*** chyka has joined #openstack-nova01:09
*** felipemonteiro has joined #openstack-nova01:12
*** chyka has quit IRC01:14
*** felipemonteiro_ has joined #openstack-nova01:14
*** Tom-Tom has joined #openstack-nova01:15
*** hongbin has joined #openstack-nova01:16
*** heyongli has quit IRC01:16
*** hshiina has joined #openstack-nova01:17
*** heyongli has joined #openstack-nova01:17
*** phuongnh has joined #openstack-nova01:17
*** felipemonteiro has quit IRC01:18
*** shaohe_feng has quit IRC01:18
*** jangutter has quit IRC01:19
*** Tom-Tom has quit IRC01:20
*** mdnadeem has joined #openstack-nova01:20
*** suresh12 has quit IRC01:21
*** suresh12 has joined #openstack-nova01:22
*** shaohe_feng has joined #openstack-nova01:22
*** felipemonteiro_ has quit IRC01:24
*** suresh12 has quit IRC01:26
*** heyongli has quit IRC01:27
*** heyongli has joined #openstack-nova01:27
*** shaohe_feng has quit IRC01:28
*** shaohe_feng has joined #openstack-nova01:29
*** threestrands has quit IRC01:35
*** threestrands has joined #openstack-nova01:35
*** threestrands has quit IRC01:36
*** heyongli has quit IRC01:37
*** threestrands has joined #openstack-nova01:37
*** threestrands has quit IRC01:37
*** threestrands has joined #openstack-nova01:37
*** heyongli has joined #openstack-nova01:37
*** threestrands has quit IRC01:38
*** shaohe_feng has quit IRC01:38
*** threestrands has joined #openstack-nova01:38
*** threestrands has quit IRC01:38
*** threestrands has joined #openstack-nova01:38
*** threestrands has quit IRC01:39
*** threestrands has joined #openstack-nova01:40
*** threestrands has quit IRC01:40
*** threestrands has joined #openstack-nova01:40
*** shaohe_feng has joined #openstack-nova01:40
*** threestrands has quit IRC01:41
*** edmondsw has joined #openstack-nova01:42
*** jichen has joined #openstack-nova01:43
*** edmondsw has quit IRC01:47
*** heyongli has quit IRC01:47
*** heyongli has joined #openstack-nova01:47
*** shaohe_feng has quit IRC01:49
*** dpawlik has joined #openstack-nova01:49
*** shaohe_feng has joined #openstack-nova01:49
openstackgerritTakashi NATSUME proposed openstack/nova master: Update admin/flavors document  https://review.openstack.org/57306301:52
*** dpawlik has quit IRC01:53
*** Tom-Tom has joined #openstack-nova01:54
openstackgerritMerged openstack/nova master: Re-base placement object unit tests on NoDBTestCase  https://review.openstack.org/57227901:55
*** r-daneel has joined #openstack-nova01:56
*** heyongli has quit IRC01:57
*** heyongli has joined #openstack-nova01:58
openstackgerritTsuyoshi Nagata proposed openstack/nova master: Fix nova can handle device name length more widly for universally.  https://review.openstack.org/57306601:58
*** shaohe_feng has quit IRC01:59
openstackgerritTsuyoshi Nagata proposed openstack/nova master: Fix nova can handle device name length more widly for universally.  https://review.openstack.org/57306602:02
*** shaohe_feng has joined #openstack-nova02:04
*** tbachman has quit IRC02:04
*** heyongli has quit IRC02:08
*** heyongli has joined #openstack-nova02:08
*** shaohe_feng has quit IRC02:09
*** shaohe_feng has joined #openstack-nova02:10
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver  https://review.openstack.org/52338702:10
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765802:10
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424002:10
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334002:10
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334402:11
*** threestrands has joined #openstack-nova02:11
*** threestrands has quit IRC02:11
*** threestrands has joined #openstack-nova02:11
mriedemwe have a gate failure at a pretty high rate now, see bug 177549102:11
openstackbug 1775491 in tempest "test_tagged_attachment randomly fails with: sequence or collection is not empty: [{u'type': u'disk', u'bus': u'pci', u'address': u'0000:00:07.0', u'serial': u'56e9a4a0-51fc-4319-845b-02c6a21d5b16', u'tags': [u'volume-tag']}]" [Undecided,New] https://launchpad.net/bugs/177549102:11
mriedemneed to hit artom in the morning02:12
openstackgerritTsuyoshi Nagata proposed openstack/nova master: Define nova's max number of devices by each bus-type.  https://review.openstack.org/56747202:15
*** heyongli has quit IRC02:18
*** heyongli has joined #openstack-nova02:18
*** shaohe_feng has quit IRC02:19
*** shaohe_feng has joined #openstack-nova02:20
*** slaweq has joined #openstack-nova02:21
openstackgerritMerged openstack/nova master: Fix nits in nested provider allocation candidates  https://review.openstack.org/57267302:24
*** slaweq has quit IRC02:26
*** heyongli has quit IRC02:28
*** heyongli has joined #openstack-nova02:28
*** shaohe_feng has quit IRC02:30
*** shaohe_feng has joined #openstack-nova02:30
*** tbachman has joined #openstack-nova02:36
openstackgerritBrin Zhang proposed openstack/nova master: Fix a logic error in network_request  https://review.openstack.org/57307302:38
*** heyongli has quit IRC02:38
*** Kevin_Zheng has joined #openstack-nova02:38
*** heyongli has joined #openstack-nova02:39
*** shaohe_feng has quit IRC02:40
*** shaohe_feng has joined #openstack-nova02:42
openstackgerritMatt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment  https://review.openstack.org/57279002:42
*** trungnv_ has quit IRC02:43
*** trungnv has joined #openstack-nova02:43
*** tiendc has joined #openstack-nova02:47
*** heyongli has quit IRC02:49
*** heyongli has joined #openstack-nova02:49
*** shaohe_feng has quit IRC02:50
*** shaohe_feng has joined #openstack-nova02:51
*** mriedem has quit IRC02:52
*** psachin has joined #openstack-nova02:52
*** heyongli has quit IRC02:59
*** heyongli has joined #openstack-nova02:59
*** yamahata has quit IRC02:59
*** shaohe_feng has quit IRC03:00
*** tbachman has quit IRC03:01
*** hiro-kobayashi has joined #openstack-nova03:03
*** lifeless_ has joined #openstack-nova03:03
*** shaohe_feng has joined #openstack-nova03:03
*** lifeless has quit IRC03:03
*** BrinZhang has joined #openstack-nova03:04
*** boris_42_ has quit IRC03:05
openstackgerritHironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648703:08
*** heyongli has quit IRC03:09
*** heyongli has joined #openstack-nova03:09
*** shaohe_feng has quit IRC03:11
*** threestrands has quit IRC03:13
*** shaohe_feng has joined #openstack-nova03:13
*** tbachman has joined #openstack-nova03:17
*** fragatina has joined #openstack-nova03:18
*** heyongli has quit IRC03:19
*** heyongli has joined #openstack-nova03:20
*** jangutter has joined #openstack-nova03:20
openstackgerritMerged openstack/nova stable/ocata: libvirt: Skip fetching the virtual size of block devices  https://review.openstack.org/57143303:21
*** shaohe_feng has quit IRC03:21
*** fragatina has quit IRC03:24
*** Nel1x has quit IRC03:26
*** shaohe_feng has joined #openstack-nova03:28
*** Bhujay has joined #openstack-nova03:28
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854203:29
*** heyongli has quit IRC03:30
*** heyongli has joined #openstack-nova03:30
*** edmondsw has joined #openstack-nova03:30
*** shaohe_feng has quit IRC03:31
*** shaohe_feng has joined #openstack-nova03:32
*** edmondsw has quit IRC03:35
*** germs has quit IRC03:37
openstackgerritChen proposed openstack/nova master: Fix issues in nova-show-usage-statistics-for-hosts-instances.rst  https://review.openstack.org/57307703:37
*** germs has joined #openstack-nova03:38
*** heyongli has quit IRC03:40
*** heyongli has joined #openstack-nova03:40
*** shaohe_feng has quit IRC03:41
*** shaohe_feng has joined #openstack-nova03:42
*** lpetrut has joined #openstack-nova03:42
openstackgerritChen proposed openstack/nova master: Fix issues in nova-show-usage-statistics-for-hosts-instances.rst  https://review.openstack.org/57307703:48
*** dpawlik has joined #openstack-nova03:49
*** heyongli has quit IRC03:50
*** heyongli has joined #openstack-nova03:50
*** shaohe_feng has quit IRC03:52
*** shaohe_feng has joined #openstack-nova03:52
*** fragatina has joined #openstack-nova03:53
*** fragatina has quit IRC03:54
*** dpawlik has quit IRC03:54
*** fragatina has joined #openstack-nova04:00
*** yamamoto has joined #openstack-nova04:00
*** heyongli has quit IRC04:00
*** heyongli has joined #openstack-nova04:01
*** fragatina has quit IRC04:01
*** fragatina has joined #openstack-nova04:02
*** suresh12 has joined #openstack-nova04:02
*** shaohe_feng has quit IRC04:02
*** udesale has joined #openstack-nova04:02
*** shaohe_feng has joined #openstack-nova04:05
*** ccamacho has quit IRC04:05
*** tbachman has quit IRC04:08
*** vladikr has quit IRC04:10
*** heyongli has quit IRC04:11
*** vladikr has joined #openstack-nova04:11
*** heyongli has joined #openstack-nova04:11
*** shaohe_feng has quit IRC04:12
*** shaohe_feng has joined #openstack-nova04:13
*** lpetrut has quit IRC04:14
*** germs has quit IRC04:16
*** Nel1x has joined #openstack-nova04:18
*** heyongli has quit IRC04:21
*** heyongli has joined #openstack-nova04:21
*** shaohe_feng has quit IRC04:22
*** suresh12 has quit IRC04:23
*** shaohe_feng has joined #openstack-nova04:23
*** fragatina has quit IRC04:24
*** Nel1x has quit IRC04:25
*** hongbin has quit IRC04:26
*** links has joined #openstack-nova04:28
*** annp has joined #openstack-nova04:29
*** heyongli has quit IRC04:31
*** heyongli has joined #openstack-nova04:31
*** takashin has quit IRC04:32
*** shaohe_feng has quit IRC04:33
*** shaohe_feng has joined #openstack-nova04:36
*** heyongli has quit IRC04:41
*** heyongli has joined #openstack-nova04:42
*** masayukig_ has quit IRC04:43
*** shaohe_feng has quit IRC04:43
*** brault has joined #openstack-nova04:44
*** shaohe_feng has joined #openstack-nova04:46
*** brault has quit IRC04:48
*** heyongli has quit IRC04:52
*** heyongli has joined #openstack-nova04:52
*** shaohe_feng has quit IRC04:53
*** mvk has joined #openstack-nova04:54
*** shaohe_feng has joined #openstack-nova04:55
*** Dinesh_Bhor has quit IRC05:00
*** heyongli has quit IRC05:02
*** heyongli has joined #openstack-nova05:02
*** shaohe_feng has quit IRC05:03
*** chyka has joined #openstack-nova05:04
*** shaohe_feng has joined #openstack-nova05:04
*** myrat has joined #openstack-nova05:06
*** chyka has quit IRC05:08
*** slaweq has joined #openstack-nova05:11
*** hiro-kobayashi has quit IRC05:11
*** heyongli has quit IRC05:12
*** heyongli has joined #openstack-nova05:12
*** shaohe_feng has quit IRC05:14
*** shaohe_feng has joined #openstack-nova05:15
*** slaweq has quit IRC05:15
*** edmondsw has joined #openstack-nova05:19
*** vivsoni has quit IRC05:19
*** vivsoni has joined #openstack-nova05:19
openstackgerritMerged openstack/nova master: Change consecutive build failure limit to a weigher  https://review.openstack.org/57219505:22
*** heyongli has quit IRC05:22
*** heyongli has joined #openstack-nova05:23
*** edmondsw has quit IRC05:23
*** shaohe_feng has quit IRC05:24
*** shaohe_feng has joined #openstack-nova05:25
openstackgerritjichenjc proposed openstack/nova-specs master: Add additional information for z/VM spec.  https://review.openstack.org/56215405:26
*** slaweq has joined #openstack-nova05:28
*** Bhujay has quit IRC05:28
openstackgerritMerged openstack/nova master: Add granular policy rules for allocation candidates  https://review.openstack.org/57065605:29
*** Dinesh_Bhor has joined #openstack-nova05:30
*** slaweq has quit IRC05:32
*** heyongli has quit IRC05:33
*** heyongli has joined #openstack-nova05:33
*** shaohe_feng has quit IRC05:34
*** shaohe_feng has joined #openstack-nova05:35
openstackgerritTsuyoshi Nagata proposed openstack/nova master: Fix nova can handle device name length more widely for universally.  https://review.openstack.org/57306605:36
openstackgerritBhagyashri Shewale proposed openstack/python-novaclient master: Modify novaclient to support basic attributes  https://review.openstack.org/57228505:40
*** fragatina has joined #openstack-nova05:43
*** heyongli has quit IRC05:43
*** heyongli has joined #openstack-nova05:43
*** shaohe_feng has quit IRC05:44
*** shaohe_feng has joined #openstack-nova05:45
*** suresh12 has joined #openstack-nova05:49
*** dpawlik has joined #openstack-nova05:50
*** Luzi has joined #openstack-nova05:51
*** Dinesh__Bhor has joined #openstack-nova05:52
*** Dinesh_Bhor has quit IRC05:53
*** gyee has quit IRC05:53
*** heyongli has quit IRC05:53
*** myrat has quit IRC05:53
*** heyongli has joined #openstack-nova05:53
*** suresh12 has quit IRC05:54
*** shaohe_feng has quit IRC05:55
*** dpawlik has quit IRC05:55
*** shaohe_feng has joined #openstack-nova05:57
*** takashin has joined #openstack-nova06:03
*** heyongli has quit IRC06:03
*** heyongli has joined #openstack-nova06:04
*** shaohe_feng has quit IRC06:05
*** hamdyk has joined #openstack-nova06:05
*** shaohe_feng has joined #openstack-nova06:06
openstackgerritjichenjc proposed openstack/nova master: doc follow https://review.openstack.org/#/c/572195  https://review.openstack.org/57309106:10
*** sar has joined #openstack-nova06:10
*** heyongli has quit IRC06:14
*** heyongli has joined #openstack-nova06:14
*** shaohe_feng has quit IRC06:15
*** shaohe_feng has joined #openstack-nova06:17
*** Dinesh__Bhor has quit IRC06:23
*** lpetrut has joined #openstack-nova06:23
*** heyongli has quit IRC06:24
*** heyongli has joined #openstack-nova06:24
*** shaohe_feng has quit IRC06:25
*** Dinesh_Bhor has joined #openstack-nova06:26
*** shaohe_feng has joined #openstack-nova06:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix the file name of development-environment.rst  https://review.openstack.org/57309606:26
*** bigdogstl has joined #openstack-nova06:29
*** dklyle has quit IRC06:29
*** namnh has joined #openstack-nova06:31
openstackgerritjichenjc proposed openstack/nova master: doc follow https://review.openstack.org/#/c/572195  https://review.openstack.org/57309106:31
*** bigdogstl has quit IRC06:33
*** heyongli has quit IRC06:34
*** heyongli has joined #openstack-nova06:35
*** pcaruana has joined #openstack-nova06:35
*** shaohe_feng has quit IRC06:36
*** shaohe_feng has joined #openstack-nova06:36
*** abhishekk has joined #openstack-nova06:36
*** abhishekk is now known as akekane|sick06:36
*** akekane|sick is now known as akekane|away06:37
*** akekane|away is now known as abhishekk|away06:37
*** eric_yang has joined #openstack-nova06:40
*** heyongli has quit IRC06:44
*** heyongli has joined #openstack-nova06:45
*** shaohe_feng has quit IRC06:46
*** shaohe_feng has joined #openstack-nova06:46
*** phuongnh has quit IRC06:47
*** tssurya has joined #openstack-nova06:47
*** phuongnh has joined #openstack-nova06:47
*** ccamacho has joined #openstack-nova06:52
openstackgerritZhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations  https://review.openstack.org/56350506:53
*** heyongli has quit IRC06:55
*** heyongli has joined #openstack-nova06:55
*** shaohe_feng has quit IRC06:56
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854206:59
*** slaweq has joined #openstack-nova07:00
*** slaweq has quit IRC07:00
*** slaweq has joined #openstack-nova07:00
*** shaohe_feng has joined #openstack-nova07:01
*** AlexeyAbashkin has joined #openstack-nova07:04
*** moshele has joined #openstack-nova07:04
*** heyongli has quit IRC07:05
*** heyongli has joined #openstack-nova07:05
*** shaohe_feng has quit IRC07:06
*** edmondsw has joined #openstack-nova07:07
*** shaohe_feng has joined #openstack-nova07:07
*** armaan has joined #openstack-nova07:10
*** edmondsw has quit IRC07:11
*** amoralej|off is now known as amoralej07:14
*** heyongli has quit IRC07:15
*** heyongli has joined #openstack-nova07:15
*** shaohe_feng has quit IRC07:17
*** shaohe_feng has joined #openstack-nova07:18
*** jistr|mtgs is now known as jistr07:22
*** yamahata has joined #openstack-nova07:23
*** heyongli has quit IRC07:25
*** nmathew has joined #openstack-nova07:26
*** heyongli has joined #openstack-nova07:26
*** Dinesh_Bhor has quit IRC07:26
*** ktibi has joined #openstack-nova07:26
openstackgerritTetsuro Nakamura proposed openstack/nova master: placement: Use INNER JOIN for requied traits  https://review.openstack.org/57311307:27
*** shaohe_feng has quit IRC07:27
*** sar has quit IRC07:28
*** shaohe_feng has joined #openstack-nova07:29
*** ttsiouts has joined #openstack-nova07:30
*** Dinesh_Bhor has joined #openstack-nova07:31
*** tssurya_ has joined #openstack-nova07:32
*** dklyle has joined #openstack-nova07:32
*** Alexey_Abashkin has joined #openstack-nova07:34
*** ttsiouts has quit IRC07:35
*** AlexeyAbashkin has quit IRC07:35
*** Alexey_Abashkin is now known as AlexeyAbashkin07:35
*** heyongli has quit IRC07:36
*** jpena|off is now known as jpena07:36
*** heyongli has joined #openstack-nova07:36
*** tssurya_ has quit IRC07:37
*** shaohe_feng has quit IRC07:37
*** shaohe_feng has joined #openstack-nova07:38
*** heyongli has quit IRC07:46
*** AlexeyAbashkin has quit IRC07:46
*** heyongli has joined #openstack-nova07:46
*** brault has joined #openstack-nova07:47
*** shaohe_feng has quit IRC07:47
*** shaohe_feng has joined #openstack-nova07:48
*** dpawlik has joined #openstack-nova07:49
*** jaosorior has joined #openstack-nova07:52
*** mgoddard has joined #openstack-nova07:52
*** dpawlik has quit IRC07:54
*** ttsiouts has joined #openstack-nova07:54
*** ttsiouts has quit IRC07:55
*** ttsiouts has joined #openstack-nova07:56
*** AlexeyAbashkin has joined #openstack-nova07:56
*** heyongli has quit IRC07:56
*** heyongli has joined #openstack-nova07:56
*** shaohe_feng has quit IRC07:58
*** shaohe_feng has joined #openstack-nova07:59
*** ttsiouts has quit IRC08:00
*** sahid has joined #openstack-nova08:02
*** alexchadin has joined #openstack-nova08:04
*** heyongli has quit IRC08:06
*** heyongli has joined #openstack-nova08:07
*** shaohe_feng has quit IRC08:08
*** shaohe_feng has joined #openstack-nova08:09
*** moshele has quit IRC08:09
*** jaosorior has quit IRC08:10
*** moshele has joined #openstack-nova08:10
*** heyongli has quit IRC08:17
*** heyongli has joined #openstack-nova08:17
*** shaohe_feng has quit IRC08:18
*** shaohe_feng has joined #openstack-nova08:19
tssuryagood morning gibi: was wondering what is the status of this: https://review.openstack.org/#/c/529194/08:20
tssuryado we have a work around ?08:20
*** AlexeyAbashkin has quit IRC08:23
tssuryaor is it to set notification_format=unversioned ?08:25
*** dpawlik has joined #openstack-nova08:25
*** AlexeyAbashkin has joined #openstack-nova08:26
*** heyongli has quit IRC08:27
*** heyongli has joined #openstack-nova08:27
*** ratailor has joined #openstack-nova08:28
*** shaohe_feng has quit IRC08:28
*** shaohe_feng has joined #openstack-nova08:30
*** dpawlik has quit IRC08:30
*** moshele has quit IRC08:33
*** AlexeyAbashkin has quit IRC08:33
*** sridharg has joined #openstack-nova08:34
openstackgerritAndrey Volkov proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation  https://review.openstack.org/38285308:36
*** heyongli has quit IRC08:37
*** heyongli has joined #openstack-nova08:37
*** AlexeyAbashkin has joined #openstack-nova08:38
*** shaohe_feng has quit IRC08:39
*** Bhujay has joined #openstack-nova08:39
*** chyka has joined #openstack-nova08:39
*** ttsiouts has joined #openstack-nova08:41
*** shaohe_feng has joined #openstack-nova08:42
*** chyka has quit IRC08:44
*** AlexeyAbashkin has quit IRC08:45
*** heyongli has quit IRC08:47
*** heyongli has joined #openstack-nova08:48
*** shaohe_feng has quit IRC08:49
*** shaohe_feng has joined #openstack-nova08:50
openstackgerritZhenyu Zheng proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.63  https://review.openstack.org/57313608:53
*** jaosorior has joined #openstack-nova08:54
*** edmondsw has joined #openstack-nova08:55
*** janki has joined #openstack-nova08:57
*** heyongli has quit IRC08:58
*** heyongli has joined #openstack-nova08:58
*** hiro-kobayashi has joined #openstack-nova08:59
*** shaohe_feng has quit IRC08:59
*** edmondsw has quit IRC09:00
*** shaohe_feng has joined #openstack-nova09:00
*** takashin has left #openstack-nova09:01
*** Zames has joined #openstack-nova09:01
*** Zames has quit IRC09:04
*** Bhujay has quit IRC09:06
*** heyongli has quit IRC09:08
*** heyongli has joined #openstack-nova09:08
openstackgerritChen Hanxiao proposed openstack/nova master: sync_guest_time: use the proper errno  https://review.openstack.org/57234609:09
*** dtantsur|afk is now known as dtantsur09:09
*** shaohe_feng has quit IRC09:09
*** derekh has joined #openstack-nova09:09
*** shaohe_feng has joined #openstack-nova09:10
*** eric_yang has quit IRC09:11
*** pooja-jadhav is now known as pooja_jadhav09:14
openstackgerrittianhui proposed openstack/nova master: Fix bug to doc:nova-status  https://review.openstack.org/57314909:16
*** lifeless_ has quit IRC09:17
*** lifeless has joined #openstack-nova09:18
*** heyongli has quit IRC09:18
*** heyongli has joined #openstack-nova09:18
*** Tom-Tom has quit IRC09:19
*** shaohe_feng has quit IRC09:20
*** shaohe_feng has joined #openstack-nova09:20
*** nmathew has quit IRC09:22
*** armaan has quit IRC09:27
*** armaan has joined #openstack-nova09:28
gibitssurya: the only workaround right now is to set notification_format=unversioned09:28
*** heyongli has quit IRC09:28
*** heyongli has joined #openstack-nova09:29
gibitssurya: unfortunately I have no idea how the problem reported in the bug can be reproduced09:29
*** moshele has joined #openstack-nova09:30
*** shaohe_feng has quit IRC09:30
gibitssurya: we suspect that some of the past migration failed to add disable field09:30
*** shaohe_feng has joined #openstack-nova09:31
*** AlexeyAbashkin has joined #openstack-nova09:31
gmannmriedem, artom: fix for device tagging bug- https://review.openstack.org/#/c/573107/09:32
*** sambetts|afk is now known as sambetts09:33
tssuryagibi: thanks, we also face the same issue, yea it worked with setting notification_format to unversioned09:34
openstackgerritStephen Finucane proposed openstack/nova master: fix tox python3 overrides  https://review.openstack.org/57297409:35
*** hamdykhader has joined #openstack-nova09:37
openstackgerritStephen Finucane proposed openstack/nova master: fix tox python3 overrides  https://review.openstack.org/57297409:37
*** heyongli has quit IRC09:39
*** heyongli has joined #openstack-nova09:39
*** hamdyk has quit IRC09:39
*** shaohe_feng has quit IRC09:40
*** shaohe_feng has joined #openstack-nova09:41
*** tetsuro has quit IRC09:44
*** hiro-kobayashi has quit IRC09:45
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh'  https://review.openstack.org/53438309:45
openstackgerritStephen Finucane proposed openstack/nova master: tox: Speed things up and document them  https://review.openstack.org/53438209:45
*** links has quit IRC09:48
*** Tom-Tom has joined #openstack-nova09:49
*** heyongli has quit IRC09:49
*** heyongli has joined #openstack-nova09:49
*** dklyle has quit IRC09:50
*** david-lyle has joined #openstack-nova09:50
*** AlexeyAbashkin has quit IRC09:50
*** shaohe_feng has quit IRC09:50
*** shaohe_feng has joined #openstack-nova09:52
*** david-lyle has quit IRC09:52
*** dklyle has joined #openstack-nova09:52
*** dklyle has quit IRC09:53
*** david-lyle has joined #openstack-nova09:53
*** Tom-Tom has quit IRC09:53
*** david-lyle has quit IRC09:53
*** BrinZhang has quit IRC09:54
*** heyongli has quit IRC09:59
*** heyongli has joined #openstack-nova09:59
*** shaohe_feng has quit IRC10:01
*** shaohe_feng has joined #openstack-nova10:01
*** AlexeyAbashkin has joined #openstack-nova10:02
*** jichen has quit IRC10:02
*** tbachman has joined #openstack-nova10:03
*** dklyle has joined #openstack-nova10:04
*** links has joined #openstack-nova10:05
*** ttsiouts has quit IRC10:08
*** heyongli has quit IRC10:09
openstackgerrittianhui proposed openstack/nova master: Fix bug to doc:nova-status  https://review.openstack.org/57314910:09
*** heyongli has joined #openstack-nova10:10
*** myrat has joined #openstack-nova10:10
*** shaohe_feng has quit IRC10:11
*** shaohe_feng has joined #openstack-nova10:11
openstackgerritMerged openstack/osc-placement master: fix tox python3 overrides  https://review.openstack.org/57291710:12
*** AlexeyAbashkin has quit IRC10:13
*** Alexey_Abashkin has joined #openstack-nova10:13
openstackgerritMerged openstack/os-vif master: fix tox python3 overrides  https://review.openstack.org/57292210:14
*** namnh has quit IRC10:15
*** Alexey_Abashkin is now known as AlexeyAbashkin10:16
*** sridharg has quit IRC10:16
alexchadinhi nova team. how can I see which scheduler hints have been used during instance creation?10:16
*** sridharg has joined #openstack-nova10:16
openstackgerrittianhui proposed openstack/nova master: Fix bug to doc:nova-status  https://review.openstack.org/57314910:16
*** Dinesh_Bhor has quit IRC10:17
openstackgerrittianhui proposed openstack/nova master: Fix bug to doc:nova-status  https://review.openstack.org/57314910:18
gibitssurya: I added a small update to the bug https://bugs.launchpad.net/nova/+bug/1739325/comments/810:19
openstackLaunchpad bug 1739325 in OpenStack Compute (nova) "Server operations fail to complete with versioned notifications if payload contains unset non-nullable fields" [Medium,In progress] - Assigned to Balazs Gibizer (balazs-gibizer)10:19
gibitssurya: and I will try to talk to dansmith and mriedem about it10:19
*** sahid has quit IRC10:19
*** heyongli has quit IRC10:20
*** heyongli has joined #openstack-nova10:20
*** shaohe_feng has quit IRC10:21
tssuryagibi: thanks for the efforts;10:22
*** AlexeyAbashkin has quit IRC10:23
*** AlexeyAbashkin has joined #openstack-nova10:25
*** tbachman_ has joined #openstack-nova10:25
*** shaohe_feng has joined #openstack-nova10:25
*** ircuser-1 has quit IRC10:27
*** tbachman has quit IRC10:27
*** tbachman_ is now known as tbachman10:28
*** ircuser-1 has joined #openstack-nova10:28
*** bigdogstl has joined #openstack-nova10:29
*** heyongli has quit IRC10:30
openstackgerritTetsuro Nakamura proposed openstack/nova master: placement: Use INNER JOIN for requied traits  https://review.openstack.org/57311310:30
*** heyongli has joined #openstack-nova10:30
tssuryagibi: also I totally understand your opinion on https://review.openstack.org/#/c/557369/, I am not against a new API bump10:30
tssuryawill think more about this and maybe like mriedem was suggesting, we should have a call sometime soon like early next week maybe10:31
tssuryato discuss the details10:31
*** shaohe_feng has quit IRC10:31
*** shaohe_feng has joined #openstack-nova10:32
*** mdbooth_ has joined #openstack-nova10:34
*** bigdogstl has quit IRC10:34
*** tbachman has quit IRC10:34
*** tbachman has joined #openstack-nova10:36
*** mdbooth has quit IRC10:37
*** Alexey_Abashkin has joined #openstack-nova10:38
*** AlexeyAbashkin has quit IRC10:39
*** Alexey_Abashkin is now known as AlexeyAbashkin10:39
*** links has quit IRC10:39
*** heyongli has quit IRC10:40
*** heyongli has joined #openstack-nova10:40
*** shaohe_feng has quit IRC10:42
*** abhishekk|away has quit IRC10:42
*** edmondsw has joined #openstack-nova10:43
*** shaohe_feng has joined #openstack-nova10:43
*** mdnadeem has quit IRC10:44
*** edmondsw has quit IRC10:47
*** heyongli has quit IRC10:50
*** heyongli has joined #openstack-nova10:51
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers  https://review.openstack.org/56750810:52
openstackgerritTetsuro Nakamura proposed openstack/nova master: Use list instead of set for duplicate check  https://review.openstack.org/56972010:52
*** shaohe_feng has quit IRC10:52
*** shaohe_feng has joined #openstack-nova10:53
*** links has joined #openstack-nova10:56
openstackgerrittianhui proposed openstack/nova master: Fix bug to doc:nova-status  https://review.openstack.org/57314910:59
*** dpawlik has joined #openstack-nova11:00
*** ttsiouts has joined #openstack-nova11:01
*** heyongli has quit IRC11:01
*** AlexeyAbashkin has quit IRC11:01
*** heyongli has joined #openstack-nova11:01
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries  https://review.openstack.org/55804511:02
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree  https://review.openstack.org/55948011:02
openstackgerritTetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate  https://review.openstack.org/56548711:02
*** shaohe_feng has quit IRC11:02
*** shaohe_feng has joined #openstack-nova11:02
*** dpawlik has quit IRC11:05
*** udesale has quit IRC11:07
*** phuongnh has quit IRC11:07
*** heyongli has quit IRC11:11
*** heyongli has joined #openstack-nova11:11
*** shaohe_feng has quit IRC11:12
*** shaohe_feng has joined #openstack-nova11:14
*** yamamoto has quit IRC11:14
*** tiendc has quit IRC11:15
openstackgerritTetsuro Nakamura proposed openstack/nova master: Delete duplicate functions in placement test  https://review.openstack.org/57319311:16
gibitssurya: I'm happy to discuss this on call as well11:17
*** yamahata has quit IRC11:19
tssuryagibi: ack :)11:19
*** sahid has joined #openstack-nova11:19
*** jpena is now known as jpena|off11:20
*** heyongli has quit IRC11:21
*** heyongli has joined #openstack-nova11:21
*** edmondsw has joined #openstack-nova11:22
*** shaohe_feng has quit IRC11:23
*** shaohe_feng has joined #openstack-nova11:23
*** ttsiouts has quit IRC11:26
*** heyongli has quit IRC11:31
*** heyongli has joined #openstack-nova11:32
*** chyka has joined #openstack-nova11:32
*** shaohe_feng has quit IRC11:33
*** shaohe_feng has joined #openstack-nova11:34
*** tbachman has quit IRC11:35
*** tbachman has joined #openstack-nova11:37
*** chyka has quit IRC11:37
openstackgerrittianhui proposed openstack/nova master: Fix bug to doc:nova-status  https://review.openstack.org/57314911:37
*** heyongli has quit IRC11:42
*** brault has quit IRC11:42
*** heyongli has joined #openstack-nova11:42
*** shaohe_feng has quit IRC11:43
*** shaohe_feng has joined #openstack-nova11:44
*** ttsiouts has joined #openstack-nova11:44
*** AlexeyAbashkin has joined #openstack-nova11:51
*** heyongli has quit IRC11:52
*** heyongli has joined #openstack-nova11:52
*** dpawlik has joined #openstack-nova11:52
*** shaohe_feng has quit IRC11:53
openstackgerritTetsuro Nakamura proposed openstack/nova master: WIP: Optimize member_of check for nested providers  https://review.openstack.org/57320311:54
*** shaohe_feng has joined #openstack-nova11:55
*** alexchadin has quit IRC11:56
*** alexchadin has joined #openstack-nova11:57
*** dpawlik has quit IRC11:57
*** alexchadin has quit IRC11:57
*** alexchadin has joined #openstack-nova11:57
*** alexchadin has quit IRC11:58
*** amoralej is now known as amoralej|lunch11:59
*** jpena|off is now known as jpena12:01
*** heyongli has quit IRC12:02
*** heyongli has joined #openstack-nova12:02
*** shaohe_feng has quit IRC12:04
*** eharney has quit IRC12:08
*** shaohe_feng has joined #openstack-nova12:08
*** yamamoto has joined #openstack-nova12:10
*** nicolasbock has joined #openstack-nova12:11
*** hamdykhader has quit IRC12:11
*** armaan has quit IRC12:12
*** heyongli has quit IRC12:12
*** armaan has joined #openstack-nova12:13
*** brault has joined #openstack-nova12:13
*** heyongli has joined #openstack-nova12:13
*** dave-mccowan has joined #openstack-nova12:14
*** shaohe_feng has quit IRC12:14
*** dave-mcc_ has joined #openstack-nova12:16
*** shaohe_feng has joined #openstack-nova12:16
*** armaan has quit IRC12:17
*** dave-mccowan has quit IRC12:18
*** yamamoto has quit IRC12:21
*** cdent has joined #openstack-nova12:21
*** heyongli has quit IRC12:23
*** heyongli has joined #openstack-nova12:23
*** shaohe_feng has quit IRC12:24
bauzasjaypipes: thanks for not holding https://review.openstack.org/#/c/557065/612:24
bauzasjaypipes: I agree with you about the fact that the vGPU type names are just a vendor silly12:25
bauzasbut meh12:25
*** shaohe_feng has joined #openstack-nova12:29
*** liverpooler has joined #openstack-nova12:30
*** hemna_ has joined #openstack-nova12:31
*** heyongli has quit IRC12:33
*** heyongli has joined #openstack-nova12:33
*** alexchadin has joined #openstack-nova12:34
*** shaohe_feng has quit IRC12:34
cdent"a vendor silly" is a good phrase, will need to use it often12:36
*** shaohe_feng has joined #openstack-nova12:37
*** mriedem has joined #openstack-nova12:38
*** alexchadin has quit IRC12:39
openstackgerritMultipleCrashes proposed openstack/nova master: Retry decorator fix for autoscale delete  https://review.openstack.org/56341812:39
*** READ10 has joined #openstack-nova12:41
*** heyongli has quit IRC12:43
*** heyongli has joined #openstack-nova12:43
bauzascdent: sorry, I don't know the english subsantive for the "silly" adjective12:44
bauzassubstantive*12:44
*** shaohe_feng has quit IRC12:45
*** shaohe_feng has joined #openstack-nova12:46
kashyapbauzas: No, don't worry.  It is still a fun phrase12:46
*** Eran_Kuris has quit IRC12:46
bauzaslemme try : silliness12:46
*** Tom-Tom has joined #openstack-nova12:47
bauzas(english is sometimes far easier than french to guess)12:47
*** yamamoto has joined #openstack-nova12:49
openstackgerritBrianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths  https://review.openstack.org/56126212:49
openstackgerritBrianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620412:49
openstackgerritBrianna Poulos proposed openstack/nova master: Add notification support for trusted_certs  https://review.openstack.org/56326912:49
openstackgerritBrianna Poulos proposed openstack/nova master: Add certificate validation docs  https://review.openstack.org/56015812:49
mriedemmdbooth_: see https://review.openstack.org/#/c/572790/ - you can't attach the source volume to another server while we're swapping because nova changes the volume's status to 'detaching': https://review.openstack.org/#/c/572790/712:50
mriedemoops12:50
mriedemhttp://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py#n72312:50
*** pchavva has joined #openstack-nova12:51
mdbooth_mriedem: That would be convenient12:51
mdbooth_mriedem: And completely remove the need for the notification thing12:52
mriedemright, so if i remove the libvirt stuff from the commit message are you good?12:53
*** heyongli has quit IRC12:53
*** heyongli has joined #openstack-nova12:54
mdbooth_mriedem: I wasn't that bothered about the commit message, tbh.12:54
*** mdbooth_ is now known as mdbooth12:54
mriedemso what are we going to do to get you into this beautiful car today12:54
mdboothHehe12:54
mriedemi'll throw in the undercoat for free12:54
*** tbachman has quit IRC12:54
mdboothCan you gimme a minute to punt the other ping I just got to somebody else?12:55
*** shaohe_feng has quit IRC12:55
mriedemsure, i've heard artom is available12:55
mriedemspeaking of artom, http://status.openstack.org/elastic-recheck/#177549112:55
*** rcernin has quit IRC12:56
*** shaohe_feng has joined #openstack-nova12:56
*** esberglu has joined #openstack-nova12:57
openstackgerritJan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads  https://review.openstack.org/56714812:58
*** tbachman has joined #openstack-nova12:58
*** MultipleCrashes has joined #openstack-nova12:59
*** lyan has joined #openstack-nova13:00
*** jmccarthy has joined #openstack-nova13:00
mriedemdansmith: are you going to propose a backport for this? https://review.openstack.org/#/c/572195/13:01
*** Tom-Tom has quit IRC13:01
jmccarthyNot sure what is going on here, but I'm seeing this issue at the moment https://bugs.launchpad.net/kolla/+bug/177440913:02
openstackLaunchpad bug 1774409 in kolla "neutron_dhcp_agent accumulates zombie processes " [Undecided,New]13:02
jmccarthyi.e. creating and deleting networks accumulates zombies .. any ideas ?13:02
mriedemjmccarthy: better to ask in the #openstack-neutron channel?13:02
jmccarthymriedem: Oh ! I'll do that :) Thanks !13:03
*** heyongli has quit IRC13:04
*** heyongli has joined #openstack-nova13:04
*** jmccarthy has left #openstack-nova13:05
openstackgerritJan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads  https://review.openstack.org/56714813:05
*** shaohe_feng has quit IRC13:05
*** shaohe_feng has joined #openstack-nova13:05
*** eharney has joined #openstack-nova13:07
*** AlexeyAbashkin has quit IRC13:08
*** Alexey_Abashkin has joined #openstack-nova13:08
*** udesale has joined #openstack-nova13:09
*** Alexey_Abashkin is now known as AlexeyAbashkin13:10
*** MultipleCrashes has quit IRC13:10
*** hshiina has quit IRC13:11
*** alexchadin has joined #openstack-nova13:13
*** ratailor has quit IRC13:13
*** janki has quit IRC13:13
*** heyongli has quit IRC13:14
*** heyongli has joined #openstack-nova13:14
*** shaohe_feng has quit IRC13:15
*** shaohe_feng has joined #openstack-nova13:18
*** abhishekk has joined #openstack-nova13:18
openstackgerritChris Dent proposed openstack/nova master: Use nova.db.api directly  https://review.openstack.org/54326213:23
*** alexchad_ has joined #openstack-nova13:23
*** alexchad_ is now known as alexchadin_13:23
*** alexchadin has quit IRC13:24
*** heyongli has quit IRC13:24
*** heyongli has joined #openstack-nova13:25
*** shaohe_feng has quit IRC13:26
*** shaohe_feng has joined #openstack-nova13:27
mdboothmriedem: Related, but not strictly the same thing I think. I was checking how we end up unsetting the 'detaching' state and I came across this:13:28
mdboothhttp://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py#n156713:28
dansmithmriedem: yep13:29
mdboothIn the tempest test, we swap one of the volumes, then we check if Nova still thinks that the other server is still attached to the original volume, which it apparently does13:29
mdboothHowever, I think this code might mean that *cinder* thinks it's not13:30
*** suresh12 has joined #openstack-nova13:30
mdboothIt looks like cinder is doing a detach(old)/attach(new) for all attachments, even though we only migrated one of them.13:30
*** ttsiouts has quit IRC13:32
*** openstackgerrit has quit IRC13:34
*** heyongli has quit IRC13:34
*** heyongli has joined #openstack-nova13:35
*** shaohe_feng has quit IRC13:36
*** shaohe_feng has joined #openstack-nova13:37
*** ttsiouts has joined #openstack-nova13:37
mdboothHowever, the above would only be in the case of the direct nova call, because if it went via cinder there would be a migration record13:38
*** mlavalle has joined #openstack-nova13:39
*** tssurya has quit IRC13:40
*** dave-mcc_ has quit IRC13:41
*** dave-mccowan has joined #openstack-nova13:43
*** AlexeyAbashkin has quit IRC13:43
mriedemmdbooth: that's only if nova calls that migration completion code13:43
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L562613:44
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L566113:44
mriedemif the swap isn't initiated from cinder for a retype or volume migration, we don't call that callback method in cinder13:44
*** heyongli has quit IRC13:45
*** heyongli has joined #openstack-nova13:45
*** dave-mccowan has quit IRC13:46
*** shaohe_feng has quit IRC13:46
*** shaohe_feng has joined #openstack-nova13:47
*** dave-mccowan has joined #openstack-nova13:48
*** dave-mcc_ has joined #openstack-nova13:50
*** AlexeyAbashkin has joined #openstack-nova13:50
*** armaan has joined #openstack-nova13:52
*** moshele has quit IRC13:52
*** brault has quit IRC13:52
*** dave-mccowan has quit IRC13:53
mdboothmriedem: Good point. I wonder where we unset that detaching state, then.13:53
*** dpawlik has joined #openstack-nova13:53
mdbooth..we explicitly delete the attachment13:53
*** throwsb1 has joined #openstack-nova13:54
*** heyongli has quit IRC13:55
*** heyongli has joined #openstack-nova13:55
*** openstackgerrit has joined #openstack-nova13:56
openstackgerritDan Smith proposed openstack/nova stable/queens: Change consecutive build failure limit to a weigher  https://review.openstack.org/57323913:56
mriedemmdbooth: i think when the attachment record is deleted http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py#n218813:56
*** shaohe_feng has quit IRC13:56
mriedemif there are no more attachments, the volume is available13:57
mriedemif there are other attached attachments, the volume goes back to in-use13:57
*** shaohe_feng has joined #openstack-nova13:57
mriedemmordred: it's time that we need some API super users to peruse this spec https://review.openstack.org/#/c/557369/ - i'm going to put it on the ML too13:57
mriedemmordred: what nova api should do when listing servers and we have down cells13:58
*** fragatina has quit IRC13:58
*** fragatina has joined #openstack-nova13:58
*** Bhujay has joined #openstack-nova13:58
*** dpawlik has quit IRC13:58
*** Alexey_Abashkin has joined #openstack-nova14:02
mordredmriedem: I have it open to read14:02
mriedem<314:02
*** AlexeyAbashkin has quit IRC14:02
*** Alexey_Abashkin is now known as AlexeyAbashkin14:02
*** udesale has quit IRC14:03
*** udesale has joined #openstack-nova14:04
*** heyongli has quit IRC14:05
*** heyongli has joined #openstack-nova14:05
openstackgerritDan Smith proposed openstack/nova stable/pike: Change consecutive build failure limit to a weigher  https://review.openstack.org/57324814:06
*** shaohe_feng has quit IRC14:07
*** shaohe_feng has joined #openstack-nova14:07
*** amoralej|lunch is now known as amoralej14:07
mriedemdansmith: we should backport https://review.openstack.org/#/c/569127/ to queens probably14:09
mriedemit would help with your backport https://review.openstack.org/#/c/573239/ too14:09
*** awaugama has joined #openstack-nova14:09
dansmithwell I already fixed the conflict, but okay14:09
dansmithI guess that means I have to do both of these over14:10
mriedemthe pike one shouldn't have any impact from that max attempts thing14:11
mriedemsince that was queens only14:11
dansmiththe hash will change14:11
dansmithon the queens one14:11
mriedemsure14:11
mriedemyou can use this for that max_attempts backport https://bugs.launchpad.net/nova/+bug/177562514:14
openstackLaunchpad bug 1775625 in OpenStack Compute (nova) queens "nova-scheduler calculates alternative hosts incorrectly" [Medium,Triaged]14:14
mriedemfor the paperwork14:14
efriedwtf dansmith this is like the second time this week you're online before 8.14:14
*** myrat has quit IRC14:14
dansmithefried: up at 6am every.single.day.dude.14:15
*** felipemonteiro has joined #openstack-nova14:15
*** heyongli has quit IRC14:15
*** felipemonteiro_ has joined #openstack-nova14:15
*** heyongli has joined #openstack-nova14:16
*** alexchadin has joined #openstack-nova14:17
mdboothDoes anything ever create RequestContext except n-api?14:17
*** shaohe_feng has quit IRC14:17
* mdbooth is asking in the context of https://review.openstack.org/#/c/572213/14:18
openstackgerritDan Smith proposed openstack/nova stable/queens: Change consecutive build failure limit to a weigher  https://review.openstack.org/57323914:18
*** shaohe_feng has joined #openstack-nova14:18
openstackgerritDan Smith proposed openstack/nova stable/queens: Fix interpretation of max_attempts for scheduling alternates  https://review.openstack.org/57325414:18
dansmithmriedem: ^ does that make you happy?14:18
*** alexchadin_ has quit IRC14:18
mdboothdansmith: happy seems like a stretch goal :)14:18
*** antosh has joined #openstack-nova14:19
dansmithmdbooth: it was a setup for him to say nothing makes him happy14:19
mdboothdansmith: And I ruined it... Sorry dude.14:19
mriedemi'm currently unable to stand up in polite company14:19
dansmithmriedem: lol14:19
*** felipemonteiro has quit IRC14:20
dansmithmriedem: too far.14:20
mdboothhaha14:20
mriedemhey, you threw the softball14:20
openstackgerritDan Smith proposed openstack/nova stable/pike: Change consecutive build failure limit to a weigher  https://review.openstack.org/57324814:21
*** links has quit IRC14:23
*** heyongli has quit IRC14:26
*** heyongli has joined #openstack-nova14:26
*** itlinux has quit IRC14:26
*** fragatina has quit IRC14:26
*** shaohe_feng has quit IRC14:27
*** tidwellr has joined #openstack-nova14:28
*** tidwellr has quit IRC14:28
*** tidwellr has joined #openstack-nova14:28
*** shaohe_feng has joined #openstack-nova14:28
*** ttsiouts has quit IRC14:30
*** ttsiouts has joined #openstack-nova14:33
kashyapHi folks, when someone gets time, can anyone else comment on the 'params' dict thing here: https://review.openstack.org/#/c/567258/14:33
kashyapThe "tests pass", etc.  But I'm undecided on that single point.14:34
*** cfriesen has quit IRC14:36
*** heyongli has quit IRC14:36
*** cfriesen has joined #openstack-nova14:36
*** heyongli has joined #openstack-nova14:36
stephenfinkashyap: I'd be inclined to agree too. However, if you want to keep it to a separate cleanup patch, I imagine that would be ok (and maybe better - it's beefy enough as-is)14:37
*** shaohe_feng has quit IRC14:37
*** Alexey_Abashkin has joined #openstack-nova14:38
*** shaohe_feng has joined #openstack-nova14:39
kashyapstephenfin: Okay, good to know.  Making an additional patch on top of it means, removing what I just added in this one.  That doesn't sound so clean.14:39
kashyapBut yeah, I also didn't wanted to make this existing patch already too clumsy and big.14:39
stephenfinkashyap: Your call :)14:39
kashyapBut I also don't want to forget the "keep changes relatively small and split up patches" to make reviewing easy. :-)14:40
kashyapOkido, I'll work something out tomm. and post it.14:40
kashyapThanks for the comment.14:40
*** AlexeyAbashkin has quit IRC14:41
*** Alexey_Abashkin is now known as AlexeyAbashkin14:41
*** ccamacho has quit IRC14:42
*** rajinir has joined #openstack-nova14:43
openstackgerritChen proposed openstack/python-novaclient master: Remove PyPI downloads  https://review.openstack.org/57327814:44
mriedembooya https://review.openstack.org/#/q/topic:remove_pypi_downloads+(status:open+OR+status:merged)14:45
*** heyongli has quit IRC14:46
*** moshele has joined #openstack-nova14:46
*** heyongli has joined #openstack-nova14:46
openstackgerritMerged openstack/nova master: add mtu to libvirt xml for ethernet and bridge types  https://review.openstack.org/55307214:47
*** shaohe_feng has quit IRC14:48
*** shaohe_feng has joined #openstack-nova14:48
openstackgerritJohn Garbutt proposed openstack/nova-specs master: Add PENDING vm state  https://review.openstack.org/55421214:52
*** lifeless_ has joined #openstack-nova14:53
*** lifeless has quit IRC14:53
*** heyongli has quit IRC14:56
*** alexchadin has quit IRC14:56
*** heyongli has joined #openstack-nova14:57
*** sapcc-bot9 has quit IRC14:57
*** sapcc-bot10 has joined #openstack-nova14:58
*** shaohe_feng has quit IRC14:58
*** shaohe_feng has joined #openstack-nova15:00
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945915:00
*** abhishekk has quit IRC15:00
*** dpawlik has joined #openstack-nova15:01
*** Luzi has quit IRC15:01
*** zhaochao has quit IRC15:01
*** moshele has quit IRC15:05
*** dpawlik has quit IRC15:05
*** tssurya has joined #openstack-nova15:06
*** heyongli has quit IRC15:07
*** heyongli has joined #openstack-nova15:07
*** shaohe_feng has quit IRC15:08
*** rodolof has joined #openstack-nova15:09
*** shaohe_feng has joined #openstack-nova15:10
*** tssurya has quit IRC15:12
*** itlinux has joined #openstack-nova15:14
*** moshele has joined #openstack-nova15:14
*** heyongli has quit IRC15:17
*** heyongli has joined #openstack-nova15:17
*** shaohe_feng has quit IRC15:18
*** shaohe_feng has joined #openstack-nova15:20
*** imacdonn has quit IRC15:20
*** imacdonn has joined #openstack-nova15:20
*** Alexey_Abashkin has joined #openstack-nova15:20
*** tssurya has joined #openstack-nova15:21
*** AlexeyAbashkin has quit IRC15:22
*** Alexey_Abashkin is now known as AlexeyAbashkin15:22
*** AlexeyAbashkin has quit IRC15:22
*** AlexeyAbashkin has joined #openstack-nova15:23
*** tssurya has quit IRC15:24
*** heyongli has quit IRC15:27
*** heyongli has joined #openstack-nova15:27
johnthetubaguybauzas looking at the comments you made on the rebuild cell0 instance spec: https://review.openstack.org/#/c/55421815:28
*** shaohe_feng has quit IRC15:29
johnthetubaguyand gibi: ^15:29
bauzasyup ?15:29
johnthetubaguyyou make some great points, but not sure how we move things forward15:29
*** ttsiouts has quit IRC15:29
johnthetubaguyI was wondering about data migrations, after adding a new field, is that needed? probably not till we want to bump the object version I guess?15:29
bauzasjohnthetubaguy: about the persisted networks you mean ?15:30
*** ktibi has quit IRC15:30
*** suresh12 has quit IRC15:30
*** chyka has joined #openstack-nova15:31
-openstackstatus- NOTICE: Zuul update for Ansible 2.5 in progress. Scheduler crashed as unexpected side effect of pip upgrade. Will be back and running shortly.15:31
*** shaohe_feng has joined #openstack-nova15:31
*** moshele has quit IRC15:31
*** slaweq_ has joined #openstack-nova15:32
bauzasjohnthetubaguy: the online data migration you ask is for setting the field for the existing req_spec records, right?15:32
*** slaweq has quit IRC15:33
bauzasjohnthetubaguy: that's why I thought about a lazy-loadable attribute15:33
bauzasbecause there are two things15:33
bauzas1/ the instance is being created but then not getting a compute, then going into cell015:34
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Change consecutive build failure limit to a weigher  https://review.openstack.org/57324815:34
bauzasso, then, we're sure, the requested_networks is good15:34
mriedemdansmith: ^ updated the commit message to note the other difference in that backport15:34
bauzas2/ the instance is now rebuilded, so there are some new possible networks15:34
bauzasthen, the field could be stale15:35
*** tbachman has quit IRC15:35
bauzasjohnthetubaguy: but for the data migration just because of a new field, we don't need it15:35
johnthetubaguybauzas: yeah, I just went through a similar though process, I think its fine without the migration15:35
dansmithmriedem: oh actually I didn't notice that got faulted in15:36
dansmithnot sure that's going to work as-is15:36
mriedemwe don't have requested_networks on request spec today, and gibi is adding them in but not persisting them for the bw based scheduling stuff15:36
johnthetubaguybauzas: so block_device_mapping, isn't it the same deal, its just the initial one in there?15:36
mriedembdms aren't in reqspec eithre15:36
mriedemhowever,15:37
johnthetubaguyso I keep getting build_request and request_spec mixed up maybe15:37
mriedemi believe bdms get stored in cell0 o nfailure15:37
*** Alexey_Abashkin has joined #openstack-nova15:37
mriedembuild_request is gone once the instance is in a cell15:37
gibijohnthetubaguy, mriedem: yes, as I want to avoid the situation when a later operation reads the requested_networks from the RequestSpec taht is stale15:37
*** AlexeyAbashkin has quit IRC15:37
*** Alexey_Abashkin is now known as AlexeyAbashkin15:37
johnthetubaguyyeah, maybe build request is where we want this15:37
*** heyongli has quit IRC15:37
mriedemhttps://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L107415:38
*** heyongli has joined #openstack-nova15:38
mriedembdms get put into cell0 on failure so we can detach volumes when you delete the failed server15:38
mriedems/detach/unreserve/15:38
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L197115:38
bauzasthe question is, can we discover the requested networks for existing req_spec records ?15:38
bauzaswe have the instance field, right?15:39
mriedemrequested networks are not persisted15:39
bauzasoops15:39
bauzasthe instance record15:39
*** shaohe_feng has quit IRC15:39
bauzasmriedem: aren't we persisting the requested networks into the instance record ?15:39
*** shaohe_feng has joined #openstack-nova15:40
mriedemno15:40
mriedemthe info cache is in the instance15:40
mriedemwhich is after you've allocated netowrks15:40
mriedemo na host15:40
bauzasif we aren't able to discover the networks, then https://review.openstack.org/#/c/554218/ needs to explain that for existing records, we should just rebuild the instance like if the user wasn't asking for networks15:40
bauzasmriedem: gotcha15:40
mriedemif we rebuild from cell0 ,we have to persist requested networks15:41
mriedemotherwise you've changed the original user request15:41
mriedemwhich you might as well just not support the feature15:41
mriedemespecially if i'm creating a server with 20 ports for NFV megaawesome15:42
bauzasmriedem: sure15:42
gibimriedem: could we keep the BuildRequest in the db if the instance lands in cell0 and persist the networks there?15:42
bauzasmriedem: but for existing instances, how can we know what the user asked ?15:42
bauzasmriedem: if we haven't persisted that15:42
gibibauzas: port.device_id == instance.uuid15:43
bauzasthat's my question15:43
bauzasand then a possible data migration15:43
mriedemgibi: that fucks up a lot of logic in the api15:45
mriedembuild request needs to remain temporary until we pick a cell15:45
openstackgerritCurt Moore proposed openstack/nova-specs master: Add spec for downloading images via RBD  https://review.openstack.org/57280515:45
mriedembauzas: you'd likely have to persist the requested networks in the request spec15:45
mriedemlike everything else that comes in on the POST /servers request15:45
mriedemexcept bdms and config drive and the other 50 things you can request when creating a server...15:46
mriedembut that stuff gets stored on the instance, where as requested_networks don't15:46
*** gyee has joined #openstack-nova15:46
mriedemi'll also say, for the record, that i don't think given the timing that pre-emptible instances / rebuild from cell0 are priorities for rocky15:47
mriedemit's way too big at this point15:47
mriedemimo15:47
*** abhishekk has joined #openstack-nova15:47
mriedemdansmith: you were right, that pike backport failed http://logs.openstack.org/48/573248/2/check/nova-tox-functional/fd4d7a5/testr_results.html.gz15:47
dansmithmriedem: yeah I'm running tests now to fi15:48
dansmith*fix15:48
*** heyongli has quit IRC15:48
mriedemack15:48
johnthetubaguymriedem: yeah, that's fair, I mean its been in review for months, but we only just getting close on the details now15:48
*** heyongli has joined #openstack-nova15:48
bauzasmriedem: don't get me wrong, I agree with persisting requested_networks in the ReqSpec15:48
melwittmriedem: +1 on pre-emptible instances, I was wondering similar yesterday when I was looking over the spec to add the PENDING state15:48
*** wwriverrat has quit IRC15:48
bauzasmriedem: that's exactly the reason why we have this object15:49
melwittif it's too early to be starting on those? I dunno15:49
*** shaohe_feng has quit IRC15:49
bauzasmriedem: but my concern is not about persisting the field for the new instances that are created once we start persisting this info15:49
melwittmriedem: unrelated but just updated the hash for the ocata release https://review.openstack.org/57152215:49
bauzasmriedem: my wonders come from the situation where we also need to discover the networks for the instances that are *already* created and having a request_spec record15:50
*** shaohe_feng has joined #openstack-nova15:50
mriedembauzas: so something already in cell0 that doesn't have requested networks in cell015:50
mriedemin that case, you can't rebuild it15:50
bauzasthat's why I'm wondering how we can get this info, and populate the req_spec record accordingly thru a data migration or something like that15:50
johnthetubaguybauzas: I think we just fail the API call for old instances15:50
mriedem40915:50
johnthetubaguyyeah, +115:50
johnthetubaguyjust added that comment on the spec15:50
bauzasmriedem: johnthetubaguy: I don't disagree, I just mean we need to say this in the specd15:51
johnthetubaguyyup15:51
mriedemeasy workaround - delete the ERROR server and re-create it15:51
bauzasmriedem: johnthetubaguy: ie. either we find it, or we fail15:51
*** cdent has quit IRC15:51
johnthetubaguymriedem: +115:51
johnthetubaguygiven gibi's concerns, I wonder about calling it initial_requested_networks?15:52
johnthetubaguyi.e. its not updated when you add ports later15:52
bauzasjohnthetubaguy: no15:52
gibijohnthetubaguy: or update it whenever it changes :)15:52
bauzasjohnthetubaguy: requested_networks looks to me the exact naming15:52
bauzasyou *requested* a network15:53
*** AlexeyAbashkin has quit IRC15:53
gibibauzas: interface_attach is used to request additional networks15:53
bauzasthat doesn't imply you'll get *this* network15:53
johnthetubaguydo we update the security_groups list that is in request_spec today?15:53
bauzasgibi: sure, but then when rebuilding, we only need to know about what the user asked15:53
mriedemjohnthetubaguy: yes15:53
bauzasjohnthetubaguy: nope, AFAIK15:53
bauzasshit15:53
bauzaswe do ?15:54
gibibauzas: boot, interface_attach, rebuild => you need to know both the originally requested and the later attached network15:54
bauzaswe update the instance group info15:54
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L87615:54
bauzasmriedem: sure, we persist those, but do we update them ?15:54
bauzaswell... nevermind, silly question15:55
johnthetubaguygibi: bauzas: my distraction is we don't need this for regular rebuild (as we need the port_ids, not the network_ids) but we do need it for rebuild from cell0, but agreed we don't what it to be "special"15:55
bauzasI guess johnthetubaguy's point (and gibi's) is to know whether we should persist *both* what the user asked and what the instance got eventually15:55
bauzasgibi: but if the instance got to cell0, it didn't got other networks, right?15:56
gibijohnthetubaguy: I agree, if the persisted field will be used for cell0 only then we have to make it explicity that it is for that case only15:56
johnthetubaguyright, I know we have secuirty_groups from the initial request, but if you change security groups do we update that later on15:56
* johnthetubaguy goes reads code15:56
gibibauzas: right. How do you make sure that the new RequestSpec field only used for the cell0 case as in any other case that info is potentially stale15:57
*** esberglu has quit IRC15:57
*** janki has joined #openstack-nova15:57
johnthetubaguygibi: my argument is its not useful in any other case, but I suspect I would get proved wrong by something15:57
*** cdent has joined #openstack-nova15:58
*** heyongli has quit IRC15:58
melwittmriedem: do we have anything else queued up for ocata that I should wait on for release?15:58
*** dpawlik has joined #openstack-nova15:58
*** heyongli has joined #openstack-nova15:58
*** fragatina has joined #openstack-nova15:58
bauzasgibi: if we consider requested_networks as just a picture of what the user asked, I don't see the problem15:58
gibijohnthetubaguy: in my bandwidth case it could be tempting to store the oringally bandwidth request in the RequestSpec to use it later, but then I will have to ensure it is up to date. So I decided not to persist it to force myself to refresh it every time I need it15:59
bauzasgibi: because if other callers want to check it, then they should make sure that whatever the instance now has as networks, that field is just the *original ask*15:59
*** shaohe_feng has quit IRC15:59
mriedemmelwitt: sorry, in -tc meeting,15:59
mriedemwill look in a bit15:59
bauzasgibi: since I incepted this RequestSpec object, I refrained myself to make it look like "here is what the instance is", but rather "here is what the user asked"16:00
johnthetubaguybauzas: I certainly assumed it was that, the initial state16:00
bauzasif you wanna lookup any field that's related to a current status (like the instance), req_spec is not the good place to look at16:00
bauzasjohnthetubaguy: well, I hitted bumps on the road a number of times16:01
bauzasjohnthetubaguy: like the forced_hosts thing16:01
*** shaohe_feng has joined #openstack-nova16:01
bauzasbut I think we're now in a pretty good shape16:01
*** Tom-Tom has joined #openstack-nova16:02
gibibauzas: but RequestSpec is used and updated later in the lifecycle of the instance e.g. https://github.com/openstack/nova/blob/master/nova/conductor/tasks/migrate.py#L18416:02
*** dpawlik has quit IRC16:02
bauzasgibi: that's what I called 'bumps on the road'16:02
*** cdent has quit IRC16:03
openstackgerritJohn Garbutt proposed openstack/nova-specs master: Enable rebuild for instances in cell0  https://review.openstack.org/55421816:03
bauzaseither way, I think johnthetubaguy only needs the 'requested networks', because he only cares of cell016:03
gibibauzas: I call it precedent16:03
*** cdent has joined #openstack-nova16:03
bauzaslet's just assume that this field will only contain the 'requested_networks', and if someone wants to use this field for anything other than rebuild from cell0, then we could rathole on it later16:04
johnthetubaguybauzas: I just added this note: https://review.openstack.org/#/c/554218/4/specs/rocky/approved/enable-rebuild-for-instances-in-cell0.rst@11216:04
bauzasgibi: people could even create a new field called 'actual_networks' in opposition to 'requested_networks' :)16:04
gibibauzas: OK, I rest my case. We can add big fat comment on the field and handle the issue latear16:04
gibibauzas: :)16:05
bauzasgibi: but again, take the 'requested_destination' field16:05
johnthetubaguygibi: I am +1 a big fat comment to warn about it, can't think of a better way to help that right now16:05
bauzasor the 'availability_zone' field rather16:06
*** moshele has joined #openstack-nova16:06
bauzasreq_spec.availability_zone doesn't get you the *actual* AZ16:06
*** Tom-Tom has quit IRC16:06
*** abhishekk is now known as abhishekk|dinner16:06
bauzasit gets you the AZ the user asked16:06
johnthetubaguythat matches to port_ids pefectly actually16:06
johnthetubaguyyou request "give me a network" you get port uuid 902873408723894716:07
gibiOK, I think I see the difference16:07
*** moshele has quit IRC16:08
*** heyongli has quit IRC16:08
*** heyongli has joined #openstack-nova16:08
gibiso my endeavour to add the bandwidth resource request from the port to the RequestSpec is yet another bump on the road, as the user asked for a port in the boot request and not for bandwidth16:09
*** shaohe_feng has quit IRC16:10
gibibut eventually he will get bandwidth16:10
-openstackstatus- NOTICE: The zuul upgrade to ansible 2.5 is complete and zuul is running again. Changes uploaded or approved between 15:25 and 15:45 will need to be rechecked. Please report any problems in #openstack-infra16:10
gibimaybe the bandwidth request (and actually all implicit resource request) should live in a totally separate object outside of the RequestSpec16:11
*** shaohe_feng has joined #openstack-nova16:11
*** janki has quit IRC16:11
*** yamamoto has quit IRC16:14
*** lifeless has joined #openstack-nova16:16
mriedemmelwitt: given https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata i think we're ok, nothing else is ready16:16
*** lifeless_ has quit IRC16:16
melwittmriedem: ack, just making double sure there was nothing yet-to-be-proposed bugs coming down the pipe. since this release has been up longer than intended16:17
melwittthanks16:17
mriedemmelwitt: i think your hash in the release patch is wrong though16:17
johnthetubaguygibi: bandwidth we get from the port details?16:17
mriedemjohnthetubaguy: https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/bandwidth-resource-provider.html16:17
gibijohnthetubaguy: yes, there will be a new resource_request field in the port16:17
mriedemenjoy!16:18
gibijohnthetubaguy: https://review.openstack.org/#/c/567268/13/nova/tests/fixtures.py@1153 this is how it will look like16:18
johnthetubaguygibi: yeah, its so why we want to create the ports before calling placement, sigh.16:18
*** sahid has quit IRC16:18
melwittmriedem: really? did you think I should use the merge hash?16:18
*** heyongli has quit IRC16:18
*** heyongli has joined #openstack-nova16:19
*** _alastor_ has joined #openstack-nova16:19
gibijohnthetubaguy: yes, the is another reason to eventually move the port creation to the conductor16:19
mriedemmelwitt: i have always used the head commit on the branch16:19
gibijohnthetubaguy: s/the/this/16:19
mriedemmerge or not16:19
mriedemwhich is always what we'd use for tagging16:19
*** bigdogstl has joined #openstack-nova16:20
*** shaohe_feng has quit IRC16:20
melwittmriedem: okay, I keep getting confused about that. I'll change it16:20
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945916:20
*** yamamoto has joined #openstack-nova16:21
*** mrjk_ has joined #openstack-nova16:23
*** shaohe_feng has joined #openstack-nova16:24
*** yamamoto has quit IRC16:26
openstackgerritMerged openstack/nova master: Fix the file name of development-environment.rst  https://review.openstack.org/57309616:26
*** tbachman has joined #openstack-nova16:28
*** heyongli has quit IRC16:29
*** heyongli has joined #openstack-nova16:29
*** shaohe_feng has quit IRC16:30
*** shaohe_feng has joined #openstack-nova16:31
*** READ10 has quit IRC16:32
openstackgerritJay Pipes proposed openstack/nova master: placement: always create consumer records  https://review.openstack.org/56767816:33
openstackgerritJay Pipes proposed openstack/nova master: add consumers generation field  https://review.openstack.org/55795816:33
openstackgerritJay Pipes proposed openstack/nova master: placement: Allocation.consumer field  https://review.openstack.org/56540516:33
openstackgerritJay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict()  https://review.openstack.org/56540716:33
openstackgerritJay Pipes proposed openstack/nova master: Add a microversion for consumer generation support  https://review.openstack.org/56560416:33
mriedemso, does anyone know if lyarwood has been hit by a mini bus because today is spec freeze day and i've got a +2 on https://review.openstack.org/#/c/552078/ and a -1 from lee but i don't think i need to make any changes16:35
dansmithmriedem: he's sick16:35
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331716:35
*** READ10 has joined #openstack-nova16:35
*** lifeless_ has joined #openstack-nova16:35
mriedemok16:35
*** lifeless has quit IRC16:36
*** dklyle has quit IRC16:36
mriedemso, any other core care to take a look at that? we agreed on it in dublin and it came up again in vancouver16:36
dansmithI mean, yeah we could approve with expected addendum if needed, or just trivially excuse it for a few days for clairifcation16:37
dansmithwhichever16:37
*** tbachman has quit IRC16:37
mriedemtbc, i suspect there are some bugs in how cinder manages read only volumes/attachments,16:38
mriedembut those are dependencies, not really blockers on the general idea16:38
*** heyongli has quit IRC16:39
*** heyongli has joined #openstack-nova16:39
openstackgerritMerged openstack/nova-specs master: Add additional information for z/VM spec.  https://review.openstack.org/56215416:40
openstackgerritMerged openstack/nova-specs master: Amend allow abort live migrations in queued status spec  https://review.openstack.org/56087216:40
*** shaohe_feng has quit IRC16:40
*** fragatina has quit IRC16:41
*** shaohe_feng has joined #openstack-nova16:41
*** felipemonteiro_ has quit IRC16:42
*** felipemonteiro_ has joined #openstack-nova16:42
dansmithmriedem: aight, well, I don't really have any complaints about it, having skimmed16:45
dansmithmriedem: I guess you just prefer to have it merged and resolve any issues lee may have later yeah?16:45
dansmithI guess I don't really care either way16:45
openstackgerritDoug Hellmann proposed openstack/python-novaclient master: fix tox python3 overrides  https://review.openstack.org/57334716:46
*** fragatina has joined #openstack-nova16:49
*** lpetrut has quit IRC16:49
*** heyongli has quit IRC16:49
*** heyongli has joined #openstack-nova16:49
*** derekh has quit IRC16:50
*** shaohe_feng has quit IRC16:51
*** shaohe_feng has joined #openstack-nova16:51
mriedemdansmith: yeah16:57
mriedembut i looked into lee's comments and don't consider them blocking, unless i'm missing something16:57
dansmithyeah16:57
*** ccamacho has joined #openstack-nova16:57
*** tbachman has joined #openstack-nova16:58
*** cdent has quit IRC16:58
*** fragatina has quit IRC16:59
*** heyongli has quit IRC16:59
*** heyongli has joined #openstack-nova17:00
*** chyka has quit IRC17:01
*** shaohe_feng has quit IRC17:01
*** shaohe_feng has joined #openstack-nova17:02
mriedemlow hanging fruit opportunity https://bugs.launchpad.net/nova/+bug/177566517:03
openstackLaunchpad bug 1775665 in OpenStack Compute (nova) "api-ref: rebuild server does not mention pre-conditions" [Low,Confirmed]17:03
*** dtantsur is now known as dtantsur|afk17:04
*** lpetrut has joined #openstack-nova17:08
*** ccamacho has quit IRC17:08
*** ccamacho has joined #openstack-nova17:08
*** ccamacho has quit IRC17:09
*** ccamacho has joined #openstack-nova17:09
*** heyongli has quit IRC17:10
*** heyongli has joined #openstack-nova17:10
*** r-daneel has quit IRC17:10
*** shaohe_feng has quit IRC17:11
*** shaohe_feng has joined #openstack-nova17:12
openstackgerritMerged openstack/nova-specs master: Spec for volume multiattach enhancements  https://review.openstack.org/55207817:16
*** abhishekk|dinner is now known as abhishekk17:16
*** jpena is now known as jpena|off17:17
*** chyka has joined #openstack-nova17:18
*** fragatina has joined #openstack-nova17:18
*** heyongli has quit IRC17:20
*** heyongli has joined #openstack-nova17:20
*** ccamacho has quit IRC17:21
*** ccamacho has joined #openstack-nova17:21
*** shaohe_feng has quit IRC17:21
*** lpetrut has quit IRC17:22
*** yamahata has joined #openstack-nova17:23
*** yamamoto has joined #openstack-nova17:23
*** sambetts is now known as sambetts|afk17:24
*** shaohe_feng has joined #openstack-nova17:25
*** Bhujay has quit IRC17:27
*** udesale has quit IRC17:28
*** heyongli has quit IRC17:30
*** heyongli has joined #openstack-nova17:30
*** itlinux has quit IRC17:30
*** yamamoto has quit IRC17:31
*** boris_42_ has joined #openstack-nova17:32
*** shaohe_feng has quit IRC17:32
*** psachin has quit IRC17:32
*** shaohe_feng has joined #openstack-nova17:32
*** amoralej is now known as amoralej|off17:33
*** chyka has quit IRC17:33
*** chyka has joined #openstack-nova17:33
*** openstackgerrit has quit IRC17:34
*** lpetrut has joined #openstack-nova17:36
*** suresh12 has joined #openstack-nova17:36
mriedemmelwitt: i'm going to go through some very obvious things targeted against rocky and defer them out https://blueprints.launchpad.net/nova/rocky17:37
mriedemstuff that doesn't have an owner, etc17:37
*** bigdogstl has quit IRC17:37
melwittmriedem: okay, sounds good. thanks for doing that17:38
*** heyongli has quit IRC17:40
*** heyongli has joined #openstack-nova17:41
*** shaohe_feng has quit IRC17:42
*** shaohe_feng has joined #openstack-nova17:43
karimullLooking for a review on https://review.openstack.org/#/c/565620/ from a core member17:47
*** felipemonteiro__ has joined #openstack-nova17:48
mriedemjroll: ^?17:48
jrollmriedem: looks fine17:50
jrollpretty straightforward :)17:50
*** tssurya has joined #openstack-nova17:50
*** heyongli has quit IRC17:51
*** heyongli has joined #openstack-nova17:51
*** felipemonteiro_ has quit IRC17:52
*** shaohe_feng has quit IRC17:52
*** tssurya has quit IRC17:53
*** shaohe_feng has joined #openstack-nova17:56
*** tssurya has joined #openstack-nova17:57
*** openstackgerrit has joined #openstack-nova17:57
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Add spec for downloading images via RBD  https://review.openstack.org/57280517:57
*** zcorneli__ has joined #openstack-nova17:57
*** armaan_ has joined #openstack-nova17:58
*** suresh12 has quit IRC17:58
mriedemdansmith: +2 on this pretty simple spec about using rbd for getting images from local storage even when you're not running the rbd image backend https://review.openstack.org/#/c/572805/17:58
*** suresh12 has joined #openstack-nova17:58
*** armaan has quit IRC17:58
mriedemrelated to that thread in the ML about the image handler download extension point deprecation17:59
*** suresh12 has quit IRC17:59
*** suresh12_ has joined #openstack-nova17:59
mriedemi have a feeling CERN has also said they wanted something like this for windows images17:59
*** sridharg has quit IRC17:59
*** tssurya has quit IRC18:00
*** zcorneli_ has quit IRC18:00
*** heyongli has quit IRC18:01
*** heyongli has joined #openstack-nova18:01
*** shaohe_feng has quit IRC18:02
*** shaohe_feng has joined #openstack-nova18:05
dansmithhmm18:05
dansmithI didn't realize we still had that18:05
dansmithit's hidden in __init__18:05
melwittmriedem: thanks for following that btw, I had meant to ask mnaser to chime in about it too because I thought maybe it was something he also needs18:05
mriedemi felt compelled since i deprecated it and sent out the call for input in the ML18:08
mriedemand actually caught a bite18:08
melwitt:)18:08
*** r-daneel has joined #openstack-nova18:09
dansmithwe don't have to undeprecate the modules thing for this,18:10
dansmithwe can just convert it to a be_smart=True sort of flag I think18:11
*** heyongli has quit IRC18:11
dansmithapproving this makes me worried that someone that understands the image cache is going to show up and cry foul,18:11
*** heyongli has joined #openstack-nova18:11
dansmithand it's not very helpful to look at a havana implementation18:12
dansmithI also feel like there are things in that spec you would call out as needing more detail if it wasn't spec deadline day and you weren't flustered by getting a bite on the mailing list18:13
*** shaohe_feng has quit IRC18:13
*** shaohe_feng has joined #openstack-nova18:13
dansmithobviously the idea seems sane18:14
mriedemthere are things i thought about like, how would one disable this if they wanted, but i expect that's just (1) if you're using rbd and you want it for both ephemeral and persistent disks, you use images_type=rbd, and (2) if you want ceph for volumes and qcow2 for local, then you don't configure the libvirt ceph stuff for local18:17
mriedemso that doesn't seem like a problem18:17
mriedemotherwise as i said in the review, devil is in the details when the patch shows up, but otherwise yeah spec deadline and such18:17
dansmithI just don't have all that image cache plumbing in my head to know if it's legit to do it like this or not18:18
mriedemi could very obviously be missing stuff and sort of kind of feel bad about this, so if there are issues you see then please don't hold back18:18
dansmithjust would feel better if someone else that kinda understands that had even looked at it I guess18:18
mriedemmdbooth18:18
mriedemto the rescue18:18
mriedemalso, my house is unfortunately now overrun with children since the school system decided it won't hold school year round, and i must evacuate to a coffee shop, so bbiab18:19
dansmithmelwitt: do you know more about this process and/or feel like due diligence has been done on this elsewhere?18:20
*** mriedem has quit IRC18:20
dansmithonce it is approved, it's candidate for a runway and subsequent auto-approval next cycle. the spec has been up for <24 hours and the ML post didn't get much discussion at all18:21
melwittdansmith: no, I don't have that much knowledge about what the extension point is providing18:21
dansmithas long as the opt stays deprecated and not removed we're not screwing this guy over (yet)18:21
*** heyongli has quit IRC18:21
melwittthat is, I don't want us to lose the functionality but I don't know what the right way to provide it is18:22
dansmithand/or we could just give it a spec exception until next week in hopes that we could get a quick read from anyone who knows anything about this to trivially thumb it up or something18:22
dansmiththe spec is about adding a feature,18:22
dansmithit's not about the extension point itself18:22
*** heyongli has joined #openstack-nova18:22
dansmiththe guy is offering to upstream his thing so that we could remove the extension point basically18:22
dansmithbut I dunno if his thing is a total hack or not18:22
melwittbut it's about adding a feature that provides the same functionality that the deprecated extension point is currently taking care of?18:22
dansmithno18:22
dansmithit uses the extension point to add the feature out of tree currently18:23
*** shaohe_feng has quit IRC18:23
dansmithit's coming up because if the extension point goes away, he can't sideload his extra thing18:23
dansmiththe confusion may be that he asserted that bringing it into tree means we keep the extension point, but that's not really a thing18:23
*** shaohe_feng has joined #openstack-nova18:24
melwitthm, okay. yeah, I would be okay with a freeze exception to review this since it showed up only yesterday and if it's an important operator issue18:25
dansmithI'm not sure it is an important operator issue, because the thread on the ML was a week ago and nobody has really jumped on it18:26
melwittI just haven't connected the dots between it and the ML post, I had assumed it was proposed in reaction to the deprecation of the extension point and as a way to keep the fast download behavior18:26
dansmithwe don't have the fast download behavior now18:26
dansmiththis guy hacks it in with an out of tree extension,18:26
dansmithwhich uses this extension point18:26
dansmithhe wants to upstream it so he can keep the functionality after we remove the extension point18:26
melwittoh, I see18:26
dansmithwhich is awesome and stuff,18:26
dansmithI just don't know if what he's describing that he's doing is a gross hack or legit, because Idon't know the image cache maze of caves very wekk18:27
dansmith*well18:27
melwittyeah, gotcha18:27
*** abhishekk has quit IRC18:28
*** munimeha1 has joined #openstack-nova18:29
melwittyeah, I dunno. that's a tough one, bad timing. mdbooth might be able to look at it tomorrow18:29
dansmiththe spec also kinda describes this as just bringing his extension into tree, which will still use the extension point, which is not what I think we should do18:29
dansmithwhich is also why I think this needs another go-round at least :)18:30
dansmithyeah18:30
*** lpetrut has quit IRC18:31
*** claudiub has joined #openstack-nova18:31
*** heyongli has quit IRC18:32
*** heyongli has joined #openstack-nova18:32
*** shaohe_feng has quit IRC18:33
*** shaohe_feng has joined #openstack-nova18:34
*** lpetrut has joined #openstack-nova18:36
melwittthe concept makes a lot of sense, similar to the ceph fast clone stuff we have, I just don't yet understand why the extension point is needed to grab the image if we didn't need it for fast clone18:37
melwittI think the feature is definitely legit but the implementation details are the potential snag, but I think everyone said that already18:39
*** lpetrut has quit IRC18:40
*** lpetrut has joined #openstack-nova18:40
*** mriedem has joined #openstack-nova18:41
*** heyongli has quit IRC18:42
*** heyongli has joined #openstack-nova18:42
*** shaohe_feng has quit IRC18:43
mriedemdansmith: melwitt: i'm happy to sit on that until mdbooth can look it over, i'll drop the +218:44
*** spotz has quit IRC18:45
mriedemsounds like, if image isn't in cache, and images_type!=rbd but CONF.libvirt.rbd_user is configured, then look for the image in ceph, else download18:46
*** spotz has joined #openstack-nova18:46
mriedemwhich is presumably how this works today with the extension download handler18:46
melwittyeah, agreed sounds like it would be something like that18:47
*** shaohe_feng has joined #openstack-nova18:47
*** armaan_ has quit IRC18:48
melwittthe only catch might be that you have to get it from the glance ceph pool. I don't think that's the case for the fast clone snapshot18:48
melwittbut I guess it must be ... thinking about the glance direct urls18:49
dansmithagreed that it sounds like the right thing to do, I just am unsure on the mechanics18:49
dansmithif we can wait for mdbooth to look, I'd feel a lot better18:49
melwittyeah, that's fair enough18:50
*** heyongli has quit IRC18:52
*** heyongli has joined #openstack-nova18:52
*** shaohe_feng has quit IRC18:54
*** shaohe_feng has joined #openstack-nova18:54
mnasermelwitt, mriedem: that spec is nice but it doesn't figure out our issue because we have 2 ceph clusters18:55
mnaserfor me the value would be if the image cache would use rbd instead of local storage18:55
mnaserso we download the image once and then it remains cached inside the cluster18:55
mnaserwhich is a problem i'm about to run into next week that i haven't fully thought out what/how i go about this :(18:56
mriedemso maybe that's an alternative for the spec18:56
mriedemmake image cache ceph aware18:56
mriedemmnaser: there are nova guidance counselors here to help :)18:56
mnaserimage cache and backend are decoupled.. or not?18:57
* dansmith groans18:57
mnasermriedem: https://wiki.openstack.org/wiki/Technical_Committee_Tracker#Project_Teams i'm signed up ha18:58
mnaserso if they are decoupled, that would solve that problem for that use case in that spec18:58
mriedemha18:58
mnaserit would involve an extra import/export18:58
mnaserbecause you'd be exporting it from glance once slowly and importing it to ceph, storing it potentially twice if its the same cluster18:58
mnaserin my case i'd be storing it twice but in different clusters18:59
openstackgerritJay Pipes proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing  https://review.openstack.org/56045919:01
melwittyeah, I dunno if I'm missing something but it sounds like we could do this by adding an extra step in the process, like instead of cow cloning from the glance pool, we take an extra step first to copy it and flatten to the nova pool (if does not exist) and then cow clone off of that instead. it would also decouple instances from the images in glance (problem of being unable to delete a glance image if any instance is in existing cow19:01
melwitt cloned from it)19:01
*** lpetrut has quit IRC19:02
melwitt*instance is existing that was cow cloned from it19:02
mnasermelwitt: thats a pretty good way of going about things and it does remove that weird19:02
mnaser"i cant delete my image!!!" complaints with rbd19:02
melwittyeah19:02
*** heyongli has quit IRC19:02
*** heyongli has joined #openstack-nova19:03
mnaserthe only thing is that does potentially slow down first boots19:03
mnaserand i'm not sure how that would be handled if someone uploads an image and boots 10 VMs of it right away19:03
melwittyeah19:03
*** shaohe_feng has quit IRC19:04
melwittheh, maybe this ties into that spec, rbd export, is that faster than clone + flatten or the same I would think?19:04
mnaseri think flatten is faster because flatten doesnt involve the host that does it19:05
*** shaohe_feng has joined #openstack-nova19:05
mnaserexport means you download as fast as your compute node, and upload as fast as your compute node in import19:05
melwittoh, okay19:05
mnaserwhere as flatten is this cluster operation where osds talk to themselves19:06
*** munimeha1 has quit IRC19:06
melwittgot it19:06
mnaserbut maybe this is what you're talking about but if we decouple the 'export' and 'import' then both my and this users use case can be solved19:06
mnaserif we can do cow, export is noop and import is create cow volume19:07
mnaserif compute node uses local storage and image is on glance, export using rbd export, import using <write-on-disk>19:07
mnaserif compute node uses ceph, image in ceph, but not directly accessible, export by glance-api download, import to ceph cluster19:07
mnasernova/nova/virt/libvirt/imagecache.py havent been touched since mid-2016 other than privsep stuff, i imagine its flawless :)19:08
melwitttotally flawless19:08
mnaser"This cleanup code will delete all info files the first time it runs in Ocata, which means we can delete this block entirely in P."19:09
* mnaser looks at his release clock19:09
melwittheh19:10
mnaserthis doesn't look too crazy, ImageCacheManager could be changed up to have multiple interfaces without too much crazy work scrolling through it19:11
melwittwanna do it? :)19:11
mnaseri can give it a shot, the only thing that is a concern is locking19:12
mnaseri think ceph allows you to lock an image19:12
mriedemi know someone in the UK that loves locks and the image cache manager19:12
melwittI was gonna say, maybe you can write your ideas on the spec review https://review.openstack.org/#/c/572805 if that helps. if these two use cases can be solved with one stone19:12
melwitthah, good point19:12
dansmithyes, mnaser please comment19:13
*** heyongli has quit IRC19:13
dansmithsounds like a good reason to let this spec live for more than 24 hours :)19:13
*** heyongli has joined #openstack-nova19:13
*** shaohe_feng has quit IRC19:14
openstackgerritSimon Dodsley proposed openstack/nova-specs master: Enhanced KVM Storage QoS  https://review.openstack.org/34016819:16
*** shaohe_feng has joined #openstack-nova19:17
*** mvk has quit IRC19:18
mnaserdansmith, melwitt, mriedem: i left comments19:19
mriedemthanks19:19
mnaseri wonder how much pain i'm signing myself up for, but this is something we'll eventually need19:20
melwitt++19:20
dansmithmnaser: you know you like the pain19:20
mnaserha19:20
dansmithhard drive until it megahertz19:20
mnaseri'm going to start on some wip giant patch just to see what i'm dealing with19:21
jmloweQuick question, should this log line indicate that an allocation is being deleted?  DEBUG nova.compute.resource_tracker [Instance XXXX has been moved to another host XXX. There are allocations remaining against the source host that might need to be removed: {u'resources': {u'VCPU': 1, u'MEMORY_MB': 2048, u'DISK_GB': 8}}. _remove_deleted_instances_allocations /usr/lib/python2.7/site-packages/nova/compute/resource_tracker.py:1219:21
jmlowe8619:21
*** eharney has quit IRC19:22
openstackgerritSimon Dodsley proposed openstack/nova master: Add enhanced KVM storage QoS quotas  https://review.openstack.org/55853019:23
*** heyongli has quit IRC19:23
melwittmnaser: I think lots of people need it and I think I could help with some of it, our customers run into things with the glance clone to nova part like this https://review.openstack.org/45788619:23
*** heyongli has joined #openstack-nova19:23
openstackgerritMatt Riedemann proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment  https://review.openstack.org/57279019:23
*** mgoddard has quit IRC19:23
melwittmnaser: and like you've said before, we could see people wanting to use separate ceph clusters per cell and it would help that too. and then the issue this other person has in the spec. etc19:24
*** shaohe_feng has quit IRC19:24
mnasermelwitt: cool, i'll start reading through code and pushing something preliminary, but yeah this is something that has to happen at some point19:24
* melwitt nods19:25
melwittjmlowe: maybe, probably. dansmith did we have a bug or doc that explains/helps with knowing whether allocations need to be cleaned up via the osc-placement cli?19:26
mriedemjmlowe: the code has a big comment with more context on that message http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/resource_tracker.py#n132919:26
*** felipemonteiro__ has quit IRC19:26
dansmithit means it would have been cleaned up in ocata, but not later I think19:26
mriedemthis is a mixed ocata compute / auto delete allocation thing from pike19:26
*** READ10 has quit IRC19:26
jmlowelooking at the placement logs I don't see a delete, it's allocated on two hosts now following a host-evacuate-live19:27
dansmithmelwitt: no doc that I know of19:27
*** lpetrut has joined #openstack-nova19:27
melwittjmlowe: yeah, sorry, misread your question. it doesn't mean it deleted them but is saying it found allocations for an instance that is no longer on the host19:27
mriedemif the source host is dead (evacuated from) then those allocations aren't causing any problems19:27
dansmithhost-evacuate-live is live migration19:27
dansmithnot evacuation19:27
mriedemoh wait host-evacuate-live is not evac19:28
mriedemyeah19:28
mriedemlink me!19:28
mriedemjmlowe: is the instance running properly on the dest host?19:28
mriedemand is the source host pike or later?19:28
* dansmith lunches19:28
jmloweall pike19:28
mriedemok if the instance is happy on the dest host then you can manually remove the allocations for that instance on the source host19:29
*** shaohe_feng has joined #openstack-nova19:29
*** moshele has joined #openstack-nova19:29
mriedemmayhap we need a placement FAQ like our cells v2 FAQ19:29
melwittyeah. we had someone run into this same problem internally but I don't have the bz handy19:29
mriedemi bet i do19:30
melwittor if there was one or if this happened during a testing situation19:30
melwittcan't remember19:30
mriedemhttps://github.com/openstack/nova/commit/12a3b698d17672ec0277dff4e96c44a92186aab1#diff-afb9c0c0ca5276c7eacd987bbf51d8e619:30
*** moshele has quit IRC19:31
mnaserok the flood of questions may start sorry19:31
mnaserimagebackend and imagecache api's are nothing exposed right?19:32
mnaseras in, we don't expect anyone to be extending an image backend as a consumer of nova19:32
melwittnot that I know of, but that would be an mdbooth question19:32
mriedemnone of that stuff is exposed via rest api19:33
mriedemnot pluggable either19:33
*** heyongli has quit IRC19:33
*** heyongli has joined #openstack-nova19:33
*** shaohe_feng has quit IRC19:35
*** shaohe_feng has joined #openstack-nova19:35
*** boris_42_ has quit IRC19:36
*** ssurana has joined #openstack-nova19:36
jmlowemriedem: that instance seems happy on the dest, also it seems that after a round of host-evacuate-live I now have 4 instances that are double allocated19:38
*** slaweq_ has quit IRC19:38
jmlowemore accurately 3 double allocated and one triple allocated19:38
mriedemerrors in the logs?19:39
mriedemjmlowe: btw, the allocation tracking for migrations all got smarter in queens https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/migration-allocations.html19:39
mriedemhint hint19:40
melwittjmlowe: do you have the latest release of pike?19:40
melwittthere were some bugs around allocations that got fixed in subsequent point releases of pike, I thought19:40
jmloweooh, I did find an error, not in placement-api where I expected but in nova-compute19:40
jmlowe"Failed to clean allocation of migrated instance on the source node"19:41
jmlowenot especially helpful19:41
openstackgerritEric Fried proposed openstack/nova master: Return all nested providers in tree  https://review.openstack.org/55948019:42
*** tbachman has quit IRC19:42
jmlowebah, multiline html error confusing my grep19:42
mriedemjmlowe: there should be something before that error in the logs19:42
jmloweit's a 409 unable to allocated would exceed vcpu capacity19:43
*** slaweq has joined #openstack-nova19:43
jmlowewhich begs the question, how did it get put there in the first place?19:43
*** heyongli has quit IRC19:43
*** heyongli has joined #openstack-nova19:44
jmlowemelwitt: I should be running 16.1.119:44
*** shaohe_feng has quit IRC19:45
mriedemif you have debug logging enabled in the compute service, you'd see this before it fails http://git.openstack.org/cgit/openstack/nova/tree/nova/scheduler/client/report.py#n164419:45
mriedemand http://git.openstack.org/cgit/openstack/nova/tree/nova/scheduler/client/report.py#n168119:46
*** ssurana has quit IRC19:46
mriedemwhat it's trying to do is take the existing 'doubled up' allocations that the instance has on both the source and dest, and remove the source node allocations while maintaining the dest host allocations19:46
mriedemsince the instance already has those dest host allocations, i'm not sure why it would complain about that...unless it's considering it a new allocation rather than a replacement19:47
jmlowehttp://paste.openstack.org/show/722922/19:47
mriedemi bet that's where you're running into double/triple allocations19:47
mriedemjaypipes: ^19:48
mriedemefried: ^19:48
*** awaugama has quit IRC19:48
efriedmriedem: tl;dr?19:48
*** shaohe_feng has joined #openstack-nova19:49
jmloweget 409 post live migration with placement causing old allocation to not be deleted, wind up with allocations for one instance on 2+ resource providers19:49
mriedemefried: jmlowe is doing live migrations in pike, and at the end when we go to cleanup allocations from the source host, the PUT /allocations is failing because *i think* placement is considering this like a new allocation request against a provider19:49
mriedemyeah in here http://git.openstack.org/cgit/openstack/nova/tree/nova/scheduler/client/report.py#n159419:50
mriedemin pike we have allocations against the source and dest node during a live migratoin,19:50
jmloweall fixed in queens?19:50
mriedempost live migration, we try to remove the source node allocation whilst maintaining the dest node allocation by munging the allocations19:50
mriedemyes fixed in queens19:50
openstackgerritSimon Dodsley proposed openstack/nova-specs master: Enhanced KVM Storage QoS  https://review.openstack.org/34016819:50
jmlowelol, rolling cluster to get to centos 7.5 in preparation for upgrade to queens19:51
mriedemwhat i don't get is https://developer.openstack.org/api-ref/placement/#update-allocations says " If allocations already exist for this consumer, they are replaced."19:51
mriedemso it shouldn't consider it a new allocation19:51
mriedemwe're doing a PUT /allocatoins here but really it's a PATCH19:51
melwittis the fix not backportable?19:52
mriedemno19:52
*** pchavva has quit IRC19:52
melwittk, that explains that19:52
mriedemthere are rpc version cahnges19:52
mriedemis why19:52
mriedemthings that require specs usually aren't backportable19:53
jmloweI can probably suffer through now that I know what I'm looking for19:53
melwittmriedem: I didn't know what the change was part of, sorry19:53
mriedemi'm worried that we might not be doing an allocation replacement19:53
mriedemtrying to think of a recreate19:53
efriedmriedem: I can't understand it either.  We ought to be replacing allocs with a subset of same allocs.19:53
mriedemright19:54
*** heyongli has quit IRC19:54
* efried pulls pike placement code...19:54
*** heyongli has joined #openstack-nova19:54
mriedemi think if you had a functional test with 2 compute nodes with 1 VCPU each for inventory, create a 1 VCPU instance, live migrate it, that would tell us if we are not replacing19:54
*** pchavva has joined #openstack-nova19:54
efriedAnd by "you" you mean you.19:55
mriedemserver create would claim the 1 VCPU from the source node, live migrate would claim the 1 VCPU from the dest node (doubled up allocs), and post-live migration should remove the 1 VCPU allocation from the source node and maintain the 1 VCPU allocatoin against the dest node19:55
mriedem:(19:55
*** shaohe_feng has quit IRC19:55
efriedinterestingly, we're using 1.10 in that method, which seems unnecessary.19:56
mriedemhttps://github.com/openstack/nova/blob/stable/pike/nova/tests/functional/test_servers.py#L134219:56
*** shaohe_feng has joined #openstack-nova19:57
mriedemi think this is the test we'd want, but it's not using the right fake virt driver https://github.com/openstack/nova/blob/stable/pike/nova/tests/functional/test_servers.py#L231319:58
mriedemalthough it does check the allocations that the server has on both the source and dest after the live migration completes19:58
mriedemhttps://github.com/openstack/nova/blob/stable/pike/nova/tests/functional/test_servers.py#L233719:58
mriedemhttps://github.com/openstack/nova/blob/stable/pike/nova/tests/functional/test_servers.py#L234119:59
mriedemso we're not doubling19:59
*** pkoniszewski has joined #openstack-nova19:59
mriedemi also seem to remember cases where "Unable to allocate inventory: Unable to create allocation for 'VCPU' on resource provider '3f288dee-8183-491e-b488-ad4698141927'. The requested amount would exceed the capacity." is misleading20:00
*** Tom-Tom has joined #openstack-nova20:02
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/placement/objects/resource_provider.py#n208620:03
efriedmriedem: I just walked through that code quickish and it seems sane.20:04
*** eharney has joined #openstack-nova20:04
*** heyongli has quit IRC20:04
mnaserdoes anyone know what imagebackend means when it says 'template'20:04
mriedemjmlowe: do you see this warning in the placement-api logs for that same resource provider? http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/placement/objects/resource_provider.py#n207820:04
*** heyongli has joined #openstack-nova20:04
mnaserthe cache function for backends says: "Creates image from template." and i've gathered image really a 'storage device'20:04
mriedemfor this provider 3f288dee-8183-491e-b488-ad469814192720:05
*** shaohe_feng has quit IRC20:05
*** vladikr has quit IRC20:06
*** shaohe_feng has joined #openstack-nova20:06
*** Tom-Tom has quit IRC20:07
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Enhanced KVM Storage QoS  https://review.openstack.org/34016820:08
*** felipemonteiro has joined #openstack-nova20:09
melwittmnaser: looks like that's just supposed to be the function object that will fetch the image, example fetch_raw_image20:10
melwitt(by looking at the only place create_image is called)20:10
mnasermelwitt: thanks.. i've switched between "rip this apart and implement it cleanly and sanely" and "... just make it work" a few times now20:10
melwittmnaser: heh. yeah.20:11
melwittI have a feeling that "prepare_template" might have been something else at first and morphed into the fetch function. I dunno20:11
mnaseri was thinking of creating a concept of 'storage pools' in nova, storage pool has many volumes, all the operations for the pool are there depending on the driver20:12
mnaserwith a common function for downloading images from glance into said storage pool20:12
mnaserlike download_from_glance (using whatever best efficent way to download), with it calling write_to_backend with the data20:13
mriedemmnaser: what you're saying has all been said by mdbooth at least a dozen times20:13
mriedemand a few others re: "re-implement"20:13
mnaseri guess they havent had the time to get aronud it20:13
jmlowemriedem: chock a block full of those errors20:13
mriedemi don't know if we have the time to tell the tale20:13
openstackgerritMerged openstack/nova master: Follow up changes to granular placement policy reviews  https://review.openstack.org/57058820:14
melwittmnaser: if that could be a subclass of backend and then have rbd derive from that, that might work? unless you think the storage pool thing could apply to all drivers?20:14
*** heyongli has quit IRC20:14
melwittI know I'm majorly oversimplifying this20:14
mriedemjmlowe: hmm, not really sure; we have a test case that does this same scenario and asserts the allocations before and after the live migration across 2 computes are what we'd expect20:14
*** itlinux has joined #openstack-nova20:15
*** lpetrut has quit IRC20:15
mriedemjmlowe: if you have 4 bad instances, maybe we just need to get those cleaned up, upgrade to queens and forget this ever happened20:15
*** heyongli has joined #openstack-nova20:15
mriedemw/o a clear recreate it's hard to say what the fix is20:15
jmlowemriedem: I'm down with that20:15
mnasermelwitt: something that can be applied to all drivers, leveraging the 'Backend' class that exists in imagebackend.py and the existing implemented backends20:15
melwittk20:15
openstackgerritMerged openstack/nova master: Support nested alloc cands with sharing providers  https://review.openstack.org/56750820:15
openstackgerritMerged openstack/nova master: Use list instead of set for duplicate check  https://review.openstack.org/56972020:15
mnaserbut again that's like the 6th solution i've thought of so far20:16
*** shaohe_feng has quit IRC20:16
melwittjmlowe: you'll need to install osc-placement (it's a plugin for openstackclient) and use the CLI to clean up the stale allocations after confirming the instances are all good on the destination20:16
melwittif you want to go that route20:16
jmlowemelwitt: way ahead of you there20:16
melwitthah, cool20:16
*** shaohe_feng has joined #openstack-nova20:18
*** felipemonteiro_ has joined #openstack-nova20:22
melwittmnaser: fwiw, if there's a minimalistic way we could add the stuff for ceph that isn't fugly, I'd go that route. and maybe try to work backward to make it nicer and more general for other drivers. just MHO20:23
mnasermelwitt: that's what i'm thinking.. like adding some extra code before `clone` and `create_image` for caching..20:23
*** heyongli has quit IRC20:24
melwittyeah, something. maybe an intermediate layer that does the caching part20:24
*** heyongli has joined #openstack-nova20:25
*** felipemonteiro has quit IRC20:25
*** shaohe_feng has quit IRC20:26
*** tbachman has joined #openstack-nova20:28
*** shaohe_feng has joined #openstack-nova20:28
*** zcorneli has joined #openstack-nova20:30
*** zcorneli__ has quit IRC20:31
*** heyongli has quit IRC20:35
efriedmriedem, jmlowe: It's not consistent?20:35
*** heyongli has joined #openstack-nova20:35
*** brault has joined #openstack-nova20:35
*** shaohe_feng has quit IRC20:36
*** pcaruana has quit IRC20:39
*** brault has quit IRC20:40
*** shaohe_feng has joined #openstack-nova20:41
*** awaugama has joined #openstack-nova20:43
*** suresh12_ has quit IRC20:43
*** suresh12 has joined #openstack-nova20:44
*** tianhui has joined #openstack-nova20:44
*** tianhui_ has quit IRC20:45
*** heyongli has quit IRC20:45
*** heyongli has joined #openstack-nova20:45
*** shaohe_feng has quit IRC20:46
*** mriedem1 has joined #openstack-nova20:47
*** shaohe_feng has joined #openstack-nova20:47
*** armaan has joined #openstack-nova20:47
*** mriedem has quit IRC20:48
*** suresh12 has quit IRC20:48
melwittnova meeting in 10 min20:50
dansmithmelwitt: I have a thing distracting me right now - can you just summarize the cells meeting for me?20:51
melwittsure20:51
mnasermelwitt: thinking this through a bit more.. i'm thinking maybe glance image cache could be a cleaner way... if glance adds locations when it caches things.. this little bit here .. https://github.com/openstack/nova/blob/master/nova/virt/libvirt/imagebackend.py#L923-L92620:51
*** pkoniszewski has quit IRC20:51
mnaserhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/storage/rbd_utils.py#L199-L22520:52
mnaserwould just skip the ones with different fsid and do the normal cow for the one matching20:52
*** takashin has joined #openstack-nova20:53
melwittoh, hm20:54
*** suresh12 has joined #openstack-nova20:54
melwitthave to think about it more, I'm not _that_ familiar with these20:54
mnaseryeah, i'm still kinda hoping from one possible solutino to another20:55
*** heyongli has quit IRC20:55
*** tidwellr has quit IRC20:55
*** heyongli has joined #openstack-nova20:55
mnaserhttps://bugzilla.redhat.com/show_bug.cgi?id=122577520:56
openstackbugzilla.redhat.com bug 1225775 in openstack-glance "[RFE] Glance should be able to manage multiple Ceph clusters as backends" [Urgent,Assigned] - Assigned to akekane20:56
*** shaohe_feng has quit IRC20:57
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Enhanced KVM Storage QoS  https://review.openstack.org/34016820:57
*** throwsb1 has quit IRC20:57
mriedem1another pretty trivial spec ^20:57
mriedem1pass through more volume qos specs to the guest20:58
*** mriedem1 is now known as mriedem20:58
mriedemprobably didn't need a spec actually20:58
*** shaohe_feng has joined #openstack-nova20:58
*** germs has joined #openstack-nova20:59
*** germs has quit IRC21:01
*** germs has joined #openstack-nova21:03
*** germs has quit IRC21:04
*** heyongli has quit IRC21:05
*** heyongli has joined #openstack-nova21:06
*** shaohe_feng has quit IRC21:07
*** shaohe_feng has joined #openstack-nova21:08
*** suresh12 has quit IRC21:08
*** suresh12 has joined #openstack-nova21:09
*** itlinux has quit IRC21:15
*** heyongli has quit IRC21:16
*** bigdogstl has joined #openstack-nova21:16
*** heyongli has joined #openstack-nova21:16
*** shaohe_feng has quit IRC21:17
*** vladikr has joined #openstack-nova21:18
*** shaohe_feng has joined #openstack-nova21:19
*** awaugama has quit IRC21:21
*** nicolasbock has quit IRC21:22
*** antosh has quit IRC21:22
*** lyan has quit IRC21:23
*** moshele has joined #openstack-nova21:24
*** armaan has quit IRC21:25
*** heyongli has quit IRC21:26
openstackgerritMerged openstack/nova master: Fix issues in nova-show-usage-statistics-for-hosts-instances.rst  https://review.openstack.org/57307721:26
openstackgerritMerged openstack/nova master: Fix bug to doc:nova-status  https://review.openstack.org/57314921:26
*** heyongli has joined #openstack-nova21:26
openstackgerritMerged openstack/nova master: Delete duplicate functions in placement test  https://review.openstack.org/57319321:26
openstackgerritMerged openstack/nova master: Enhance api-guide general info some updates  https://review.openstack.org/56177321:26
*** shaohe_feng has quit IRC21:27
*** brault has joined #openstack-nova21:27
*** moshele has quit IRC21:28
*** felipemonteiro_ has quit IRC21:28
*** felipemonteiro_ has joined #openstack-nova21:29
*** shaohe_feng has joined #openstack-nova21:29
*** pchavva has quit IRC21:32
*** rajinir has quit IRC21:34
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (3)  https://review.openstack.org/56818421:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4)  https://review.openstack.org/56846221:36
*** heyongli has quit IRC21:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (4)  https://review.openstack.org/57075021:36
*** heyongli has joined #openstack-nova21:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5)  https://review.openstack.org/57084221:37
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6)  https://review.openstack.org/57133021:37
*** shaohe_feng has quit IRC21:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7)  https://review.openstack.org/57199221:38
*** bigdogstl has quit IRC21:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8)  https://review.openstack.org/57199321:38
*** edmondsw has quit IRC21:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in nova/tests/unit/virt/xenapi/stubs.py  https://review.openstack.org/56841221:39
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.openstack.org/56615321:39
openstackgerritMerged openstack/nova-specs master: Proposes Multiple GPU types  https://review.openstack.org/55706521:39
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1)  https://review.openstack.org/56465821:39
*** shaohe_feng has joined #openstack-nova21:39
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2)  https://review.openstack.org/56465921:39
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751421:40
openstackgerritTakashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller  https://review.openstack.org/34728921:40
*** tbachman has quit IRC21:41
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (1)  https://review.openstack.org/57201821:41
jangutterApologies for a final spec review request: https://review.openstack.org/#/c/567148 It's a network $thingy, hopefully low impact, and will rip out some legacy VIF code.21:44
*** heyongli has quit IRC21:46
*** heyongli has joined #openstack-nova21:47
*** shaohe_feng has quit IRC21:48
*** vladikr has quit IRC21:48
*** shaohe_feng has joined #openstack-nova21:49
*** janki has joined #openstack-nova21:55
*** heyongli has quit IRC21:57
*** heyongli has joined #openstack-nova21:57
*** bigdogstl has joined #openstack-nova21:57
*** throwsb1 has joined #openstack-nova21:57
*** shaohe_feng has quit IRC21:58
*** shaohe_feng has joined #openstack-nova21:59
*** zcorneli has quit IRC22:00
*** zcorneli has joined #openstack-nova22:00
*** bigdogstl has quit IRC22:03
*** burt has quit IRC22:03
*** lifeless_ has quit IRC22:03
*** lifeless has joined #openstack-nova22:04
*** hamzy has quit IRC22:04
*** hamzy has joined #openstack-nova22:05
*** heyongli has quit IRC22:07
*** heyongli has joined #openstack-nova22:07
melwittmriedem: novaclient release proposed https://review.openstack.org/573453 bumped minor version for microversion 2.62 support22:08
*** shaohe_feng has quit IRC22:08
*** shaohe_feng has joined #openstack-nova22:09
melwittlooks like adding traits to os-traits isn't considered adding a feature, so that one will be a patch release22:12
melwitthttps://review.openstack.org/48098922:12
*** slaweq has quit IRC22:13
mriedemefried: thanks for https://review.openstack.org/#/c/572583/ - some comments/questions inline22:14
*** bigdogstl has joined #openstack-nova22:14
*** slaweq has joined #openstack-nova22:14
efriedmriedem: ack.  Ima wait until cdent, dansmith, and jaypipes have had an initial crack at it, then address everything together.22:15
melwittmriedem: os-traits release proposed https://review.openstack.org/57345422:15
openstackgerritMerged openstack/nova-specs master: Enhanced KVM Storage QoS  https://review.openstack.org/34016822:16
*** heyongli has quit IRC22:17
*** heyongli has joined #openstack-nova22:17
*** shaohe_feng has quit IRC22:19
*** bigdogstl has quit IRC22:19
*** slaweq has quit IRC22:19
*** eharney has quit IRC22:19
*** shaohe_feng has joined #openstack-nova22:20
melwittmriedem: for os-traits I would have thought adding new traits would be a feature thus a minor version bump but saw the last release that did that only did a patch bump22:21
melwittso beware that22:21
mriedemefried: yup makes sense22:21
mriedemmelwitt: ok22:21
efriedI'll throw a couple responses in there anyway.22:22
mriedemmelwitt: os-traits is really just a collection of enums22:23
mriedemso semver rules on that are...weird22:23
mriedemi guess we'd do minor or major if we ever REMOVED traits22:24
mriedembut idk22:24
melwittyeah, I was seeing that22:24
*** rcernin has joined #openstack-nova22:24
mriedemefried: if you are able to actually birth this migrator,22:25
mriedemare you going to consider it your 5th and most beautiful child?22:25
efriedooo ooo, another shiny nickel?22:25
efriedoh22:26
efriedWell, I guess that would be an acceptable alternative.22:26
mriedemi still have that nickel in my backpack, i forgot in dublin *and* in vancouver22:26
mriedemand every time i get home and i'm cleaning my bag out i'm wondering "wtf is this nickel doing in .... oh"22:26
melwittmriedem: osc-placement https://review.openstack.org/57345622:27
*** heyongli has quit IRC22:27
*** heyongli has joined #openstack-nova22:28
*** shaohe_feng has quit IRC22:29
*** shaohe_feng has joined #openstack-nova22:30
*** chyka_ has joined #openstack-nova22:33
*** mchlumsky has quit IRC22:34
mriedemETOOMANYRELEASES22:35
melwittoof, sounds like monday might be "fun", files and irrelevant-files behavior changing22:36
melwittyeah, we have too many libs22:36
*** edmondsw has joined #openstack-nova22:36
*** chyka has quit IRC22:36
mriedemdon't forget os-vif!22:37
melwittgonna wait till later to propose the nova release since at least one thing is in the gate right now https://review.openstack.org/55804522:37
melwittI didn't! it again didn't have any release-able changes22:37
melwittjust some tox changes and such22:37
mriedemhttps://github.com/openstack/os-vif/compare/1.10.0...master22:37
mriedemah22:37
melwittunless I missed something22:37
mriedemthis is like the 100th change i've seen now like this in the last week or so https://github.com/openstack/os-vif/commit/dc17cd1fabc8bfdb84fde7bd87091d48b31a578122:38
*** heyongli has quit IRC22:38
mriedemthe pypi patches are a boon for the stat padders22:38
melwittyeah, heh22:38
*** heyongli has joined #openstack-nova22:38
*** shaohe_feng has quit IRC22:39
*** shaohe_feng has joined #openstack-nova22:40
*** edmondsw has quit IRC22:41
*** rodolof has quit IRC22:45
*** bigdogstl has joined #openstack-nova22:47
*** heyongli has quit IRC22:48
*** heyongli has joined #openstack-nova22:48
*** felipemonteiro_ has quit IRC22:48
*** felipemonteiro has joined #openstack-nova22:49
*** shaohe_feng has quit IRC22:49
*** shaohe_feng has joined #openstack-nova22:51
*** bigdogstl has quit IRC22:52
* mriedem goes to mow the lawn22:52
*** mriedem is now known as mriedem_afk22:52
*** janki has quit IRC22:52
*** dklyle has joined #openstack-nova22:56
*** heyongli has quit IRC22:58
*** heyongli has joined #openstack-nova22:59
*** shaohe_feng has quit IRC23:00
*** bigdogstl has joined #openstack-nova23:01
*** shaohe_feng has joined #openstack-nova23:01
*** adreznec has quit IRC23:04
*** adreznec has joined #openstack-nova23:06
*** tbachman has joined #openstack-nova23:08
*** heyongli has quit IRC23:08
*** heyongli has joined #openstack-nova23:09
*** bigdogstl has quit IRC23:09
*** bigdogstl has joined #openstack-nova23:10
*** shaohe_feng has quit IRC23:10
*** shaohe_feng has joined #openstack-nova23:11
*** zcorneli has quit IRC23:12
*** bigdogstl has quit IRC23:14
*** heyongli has quit IRC23:19
*** heyongli has joined #openstack-nova23:19
openstackgerritTsuyoshi Nagata proposed openstack/nova master: [nova] increasing the number of allowed volumes attached per instance > 26  https://review.openstack.org/56747223:20
*** shaohe_feng has quit IRC23:20
*** felipemonteiro_ has joined #openstack-nova23:21
*** tbachman has quit IRC23:22
*** felipemonteiro has quit IRC23:22
*** shaohe_feng has joined #openstack-nova23:22
*** throwsb1 has quit IRC23:24
*** heyongli has quit IRC23:29
*** heyongli has joined #openstack-nova23:29
*** bigdogstl has joined #openstack-nova23:30
*** shaohe_feng has quit IRC23:30
*** chyka has joined #openstack-nova23:31
*** shaohe_feng has joined #openstack-nova23:34
alex_xujaypipes: Can I use trait to pass down baremetal hardware configuration to the ironic? it's like there a, b ,c BIOS confiugrations, then I have three traits mapping to this three configurations. And attach this three traits to the node, and placement can ensure anyone of this three traits schedule to the right node. Then ironic configure the BM based on this trait.23:34
*** chyka_ has quit IRC23:35
*** bigdogstl has quit IRC23:35
*** chyka has quit IRC23:36
*** hemna_ has quit IRC23:38
*** heyongli has quit IRC23:39
*** heyongli has joined #openstack-nova23:39
*** shaohe_feng has quit IRC23:41
*** mlavalle has quit IRC23:43
*** shaohe_feng has joined #openstack-nova23:44
*** lyan has joined #openstack-nova23:47
*** bigdogstl has joined #openstack-nova23:49
*** heyongli has quit IRC23:49
*** heyongli has joined #openstack-nova23:50
*** shaohe_feng has quit IRC23:51
*** shaohe_feng has joined #openstack-nova23:52
*** tbachman has joined #openstack-nova23:57
*** chyka has joined #openstack-nova23:58
*** suresh12 has quit IRC23:59

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