Wednesday, 2018-05-30

*** geekinutah has quit IRC00:04
*** jamespage has quit IRC00:04
*** zul has quit IRC00:04
*** auggy_ has joined #openstack-nova00:05
*** shaohe_feng has quit IRC00:05
*** ildikov_ has joined #openstack-nova00:05
*** ameade_ has joined #openstack-nova00:05
*** sergek__ has joined #openstack-nova00:05
*** shaohe_feng has joined #openstack-nova00:05
*** patrickeast_ has joined #openstack-nova00:05
*** awestin1_ has joined #openstack-nova00:05
*** patrickeast has quit IRC00:05
*** auggy has quit IRC00:05
*** patrickeast_ is now known as patrickeast00:05
*** auggy_ is now known as auggy00:05
*** csatari_ has joined #openstack-nova00:05
*** catmando_ has joined #openstack-nova00:05
*** awestin1 has quit IRC00:05
*** csatari has quit IRC00:05
*** ildikov has quit IRC00:05
*** ameade has quit IRC00:05
*** catmando has quit IRC00:05
*** sergek_ has quit IRC00:05
*** csatari_ is now known as csatari00:05
*** odyssey4me has quit IRC00:05
*** awestin1_ is now known as awestin100:05
*** ameade_ is now known as ameade00:05
*** ildikov_ is now known as ildikov00:05
*** sergek__ is now known as sergek_00:05
*** test222____ has joined #openstack-nova00:06
*** nicolasbock has quit IRC00:06
*** MasterofJOKers has quit IRC00:06
*** mordred has quit IRC00:06
*** test222___ has quit IRC00:06
*** test222____ is now known as test222___00:06
*** munimeha1_ has quit IRC00:06
*** zigo has quit IRC00:06
*** Roamer` has quit IRC00:06
*** afazekas has quit IRC00:06
*** melwitt has quit IRC00:06
*** zigo_ has joined #openstack-nova00:06
*** r-daneel has quit IRC00:07
*** Roamer` has joined #openstack-nova00:08
*** mdnadeem has joined #openstack-nova00:09
*** cz2 has quit IRC00:10
*** MasterofJOKers has joined #openstack-nova00:10
*** odyssey4me has joined #openstack-nova00:11
*** melwitt has joined #openstack-nova00:11
*** afazekas has joined #openstack-nova00:11
*** GonZo2000 has quit IRC00:12
*** ChanServ has quit IRC00:12
*** cz2 has joined #openstack-nova00:13
*** mdnadeem has quit IRC00:13
*** GonZo2000 has joined #openstack-nova00:15
*** shaohe_feng has quit IRC00:15
*** mordred has joined #openstack-nova00:15
*** jamespage has joined #openstack-nova00:15
*** geekinutah has joined #openstack-nova00:15
*** zul has joined #openstack-nova00:16
*** shaohe_feng has joined #openstack-nova00:17
*** takashin has joined #openstack-nova00:23
*** Dinesh_Bhor has joined #openstack-nova00:24
*** shaohe_feng has quit IRC00:25
*** shaohe_feng has joined #openstack-nova00:26
*** ChanServ has joined #openstack-nova00:33
*** barjavel.freenode.net sets mode: +o ChanServ00:33
*** shaohe_feng has quit IRC00:35
*** shaohe_feng has joined #openstack-nova00:36
*** lbragstad[m] has quit IRC00:37
*** sean-k-mooney[m] has quit IRC00:37
*** ttsiouts has quit IRC00:39
*** melwitt is now known as Guest4815300:39
*** awestin1 is now known as Guest5425100:39
*** GonZo2000 is now known as Guest7769400:39
*** nicolasbock has joined #openstack-nova00:40
*** ttsiouts has joined #openstack-nova00:40
*** dpawlik has joined #openstack-nova00:40
*** ttsiouts has quit IRC00:44
*** dpawlik has quit IRC00:45
*** shaohe_feng has quit IRC00:46
*** shaohe_feng has joined #openstack-nova00:47
*** tetsuro has joined #openstack-nova00:48
*** trungnv_ has quit IRC00:53
*** annp has quit IRC00:53
*** annp has joined #openstack-nova00:54
*** trungnv_ has joined #openstack-nova00:54
*** yikun has joined #openstack-nova00:54
*** shaohe_feng has quit IRC00:56
*** Guest77694 has quit IRC00:56
*** shaohe_feng has joined #openstack-nova00:57
*** harlowja has quit IRC01:00
*** jichen has joined #openstack-nova01:00
*** chyka has joined #openstack-nova01:03
*** tetsuro has quit IRC01:05
*** lbragstad[m] has joined #openstack-nova01:05
*** shaohe_feng has quit IRC01:06
*** phuongnh has joined #openstack-nova01:06
*** phuongnh has quit IRC01:08
*** chyka has quit IRC01:08
*** liuyulong_ has joined #openstack-nova01:09
*** liuyulong_ has quit IRC01:10
*** shaohe_feng has joined #openstack-nova01:10
*** phuongnh has joined #openstack-nova01:10
*** liuyulong has joined #openstack-nova01:11
*** tiendc has joined #openstack-nova01:11
openstackgerritMerged openstack/nova master: Fix doc mistakes  https://review.openstack.org/57089401:14
*** Tom-Tom has joined #openstack-nova01:16
*** shaohe_feng has quit IRC01:16
*** gongysh has joined #openstack-nova01:17
*** tetsuro has joined #openstack-nova01:17
*** shaohe_feng has joined #openstack-nova01:18
*** liuyulong_ has joined #openstack-nova01:18
*** takashin has quit IRC01:19
*** Tom-Tom has quit IRC01:21
*** liuyulong has quit IRC01:22
*** mdnadeem has joined #openstack-nova01:23
*** shaohe_feng has quit IRC01:27
*** shaohe_feng has joined #openstack-nova01:28
*** gyankum has joined #openstack-nova01:29
*** yikun_ has joined #openstack-nova01:29
*** jamespage has quit IRC01:30
*** yikun has quit IRC01:30
*** yikun_ is now known as yikun01:30
*** awaugama has joined #openstack-nova01:31
*** mordred has quit IRC01:31
*** gyee has quit IRC01:31
*** sree has joined #openstack-nova01:34
*** shaohe_feng has quit IRC01:37
*** shaohe_feng has joined #openstack-nova01:37
*** hongbin has joined #openstack-nova01:38
*** sree has quit IRC01:38
*** gongysh has quit IRC01:41
*** jogo has quit IRC01:43
*** jamespage has joined #openstack-nova01:44
*** mordred has joined #openstack-nova01:44
*** dr_gogeta86 has quit IRC01:45
*** shaohe_feng has quit IRC01:47
*** shaohe_feng has joined #openstack-nova01:48
*** armaan has quit IRC01:48
*** armaan has joined #openstack-nova01:49
*** sean-k-mooney[m] has joined #openstack-nova01:54
*** zhaochao has joined #openstack-nova01:55
*** shaohe_feng has quit IRC01:57
*** shaohe_feng has joined #openstack-nova02:03
*** tetsuro has quit IRC02:07
*** shaohe_feng has quit IRC02:08
*** bkopilov_ has quit IRC02:08
*** shaohe_feng has joined #openstack-nova02:08
*** rajinir has joined #openstack-nova02:17
*** shaohe_feng has quit IRC02:18
*** yamahata has quit IRC02:20
*** shaohe_feng has joined #openstack-nova02:21
*** takashin has joined #openstack-nova02:26
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table.  https://review.openstack.org/56083202:28
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337502:28
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340102:28
*** shaohe_feng has quit IRC02:28
*** shaohe_feng has joined #openstack-nova02:29
*** Tom-Tom has joined #openstack-nova02:30
*** test_ has joined #openstack-nova02:31
*** awaugama has quit IRC02:36
*** shaohe_feng has quit IRC02:38
*** shaohe_feng has joined #openstack-nova02:39
*** dklyle has joined #openstack-nova02:40
*** jogo has joined #openstack-nova02:41
*** psachin has joined #openstack-nova02:42
*** markvoelker_ has quit IRC02:45
*** zhaochao has quit IRC02:47
*** zhaochao has joined #openstack-nova02:48
*** namnh has joined #openstack-nova02:48
*** shaohe_feng has quit IRC02:49
*** shaohe_feng has joined #openstack-nova02:49
*** xinliang has quit IRC02:52
*** dklyle has quit IRC02:54
*** xinliang has joined #openstack-nova02:58
*** shaohe_feng has quit IRC02:59
*** jangutter has quit IRC02:59
*** shaohe_feng has joined #openstack-nova03:00
*** moshele has quit IRC03:04
*** gbarros has joined #openstack-nova03:07
*** masayuki- is now known as masayukig_03:08
*** shaohe_feng has quit IRC03:09
*** masayukig is now known as masayukig__03:11
*** test_ has quit IRC03:11
*** test_ has joined #openstack-nova03:12
*** masayukig_ is now known as masayukig03:12
*** shaohe_feng has joined #openstack-nova03:12
*** janki has joined #openstack-nova03:12
*** masayukig__ is now known as masayukig_03:13
*** tetsuro has joined #openstack-nova03:13
*** vladikr has quit IRC03:19
*** shaohe_feng has quit IRC03:19
*** vladikr has joined #openstack-nova03:19
*** shaohe_feng has joined #openstack-nova03:21
*** takashin has quit IRC03:24
*** bkopilov_ has joined #openstack-nova03:24
*** Tom-Tom has quit IRC03:29
*** dikonoor has joined #openstack-nova03:29
*** harlowja has joined #openstack-nova03:29
*** shaohe_feng has quit IRC03:30
*** shaohe_feng has joined #openstack-nova03:31
*** tetsuro has quit IRC03:32
*** vladikr has quit IRC03:34
*** vladikr has joined #openstack-nova03:35
*** markvoelker has joined #openstack-nova03:35
*** markvoelker has quit IRC03:39
*** shaohe_feng has quit IRC03:40
*** tetsuro has joined #openstack-nova03:40
*** shaohe_feng has joined #openstack-nova03:41
openstackgerrittianhui proposed openstack/nova master: Fix doc mistakes  https://review.openstack.org/57108203:42
*** Tom-Tom has joined #openstack-nova03:43
*** lpetrut has joined #openstack-nova03:46
*** Tom-Tom has quit IRC03:48
*** shaohe_feng has quit IRC03:50
*** shaohe_feng has joined #openstack-nova03:52
*** hongbin has quit IRC03:53
*** nicolasbock has quit IRC03:55
*** udesale has joined #openstack-nova03:58
*** shaohe_feng has quit IRC04:00
*** shaohe_feng has joined #openstack-nova04:01
*** takashin has joined #openstack-nova04:03
*** calebb has quit IRC04:05
*** calebb has joined #openstack-nova04:05
*** gbarros has quit IRC04:06
*** shaohe_feng has quit IRC04:11
*** links has joined #openstack-nova04:11
*** shaohe_feng has joined #openstack-nova04:12
*** annp has quit IRC04:13
*** sridharg has joined #openstack-nova04:14
*** harlowja has quit IRC04:14
*** abhishekk has joined #openstack-nova04:14
*** lpetrut has quit IRC04:19
*** shaohe_feng has quit IRC04:21
*** shaohe_feng has joined #openstack-nova04:23
*** vladikr has quit IRC04:28
*** vladikr has joined #openstack-nova04:28
*** markvoelker has joined #openstack-nova04:29
*** shaohe_feng has quit IRC04:31
*** sree has joined #openstack-nova04:31
*** fragatina has quit IRC04:31
*** shaohe_feng has joined #openstack-nova04:32
*** fragatina has joined #openstack-nova04:33
*** vladikr has quit IRC04:34
*** vladikr has joined #openstack-nova04:35
*** fragatina has quit IRC04:37
*** hshiina2 has joined #openstack-nova04:41
*** hshiina has quit IRC04:41
*** shaohe_feng has quit IRC04:41
*** shaohe_feng has joined #openstack-nova04:42
*** dpawlik has joined #openstack-nova04:42
*** psachin has quit IRC04:46
*** dpawlik has quit IRC04:47
*** GonZo2000 has joined #openstack-nova04:49
*** GonZo2000 has joined #openstack-nova04:49
*** shaohe_feng has quit IRC04:52
*** shaohe_feng has joined #openstack-nova04:54
*** jangutter has joined #openstack-nova04:58
*** mvk has joined #openstack-nova04:58
*** shaohe_feng has quit IRC05:02
*** markvoelker has quit IRC05:03
*** shaohe_feng has joined #openstack-nova05:04
*** dikonoor has quit IRC05:05
*** vladikr has quit IRC05:06
*** vladikr has joined #openstack-nova05:07
*** GonZo2000 has quit IRC05:07
*** bhujay has joined #openstack-nova05:10
*** hshiina2 is now known as hshiina05:10
*** eandersson has joined #openstack-nova05:10
*** dikonoor has joined #openstack-nova05:11
*** shaohe_feng has quit IRC05:12
*** jdillaman has quit IRC05:13
*** shaohe_feng has joined #openstack-nova05:14
*** jdillaman has joined #openstack-nova05:16
*** fragatina has joined #openstack-nova05:18
*** lifeless_ has quit IRC05:19
*** lifeless has joined #openstack-nova05:19
*** GonZo2000 has joined #openstack-nova05:21
*** GonZo2000 has joined #openstack-nova05:21
*** shaohe_feng has quit IRC05:22
*** dikonoor has quit IRC05:23
*** shaohe_feng has joined #openstack-nova05:24
*** lyan has quit IRC05:24
*** lyan has joined #openstack-nova05:24
*** Tom-Tom has joined #openstack-nova05:25
*** dikonoor has joined #openstack-nova05:26
*** dikonoor has quit IRC05:29
*** shaohe_feng has quit IRC05:33
*** shaohe_feng has joined #openstack-nova05:34
*** bhujay has quit IRC05:41
*** shaohe_feng has quit IRC05:43
*** Eran_Kuris has joined #openstack-nova05:43
*** shaohe_feng has joined #openstack-nova05:44
*** kholkina has joined #openstack-nova05:45
*** GonZo2000 has quit IRC05:46
*** eandersson has quit IRC05:48
*** eandersson has joined #openstack-nova05:49
*** sar_ has quit IRC05:52
*** shaohe_feng has quit IRC05:53
*** shaohe_feng has joined #openstack-nova05:54
*** bhujay has joined #openstack-nova05:54
openstackgerritNguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build  https://review.openstack.org/55180205:54
*** moshele has joined #openstack-nova05:59
*** Dinesh_Bhor has quit IRC06:01
*** hamdyk has joined #openstack-nova06:02
*** shaohe_feng has quit IRC06:03
*** mdnadeem_ has joined #openstack-nova06:05
*** abhishekk has quit IRC06:06
*** Dinesh_Bhor has joined #openstack-nova06:07
*** shaohe_feng has joined #openstack-nova06:08
*** mdnadeem has quit IRC06:08
*** abhishekk has joined #openstack-nova06:08
*** armaan has quit IRC06:09
*** armaan has joined #openstack-nova06:09
*** sar_ has joined #openstack-nova06:12
*** shaohe_feng has quit IRC06:14
*** shaohe_feng has joined #openstack-nova06:14
*** imacdonn has quit IRC06:16
*** imacdonn has joined #openstack-nova06:16
*** Luzi has joined #openstack-nova06:19
*** markvoelker has joined #openstack-nova06:20
*** alexchadin has joined #openstack-nova06:20
*** tetsuro has quit IRC06:21
*** lpetrut has joined #openstack-nova06:22
*** lifeless has quit IRC06:23
*** shaohe_feng has quit IRC06:24
*** lifeless has joined #openstack-nova06:25
*** shaohe_feng has joined #openstack-nova06:26
*** rajinir has quit IRC06:27
*** AlexeyAbashkin has joined #openstack-nova06:32
*** pcaruana has joined #openstack-nova06:33
*** ratailor has joined #openstack-nova06:33
*** shaohe_feng has quit IRC06:34
*** shaohe_feng has joined #openstack-nova06:35
*** dpawlik has joined #openstack-nova06:41
*** sree has quit IRC06:44
*** shaohe_feng has quit IRC06:44
*** dpawlik has quit IRC06:45
*** shaohe_feng has joined #openstack-nova06:46
*** annp has joined #openstack-nova06:52
*** sree has joined #openstack-nova06:52
*** markvoelker has quit IRC06:54
*** apple01 has joined #openstack-nova06:54
*** shaohe_feng has quit IRC06:55
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340106:55
*** shaohe_feng has joined #openstack-nova06:56
*** avolkov has joined #openstack-nova06:59
*** takashin has left #openstack-nova07:00
*** shaohe_feng has quit IRC07:05
*** shaohe_feng has joined #openstack-nova07:06
*** gongysh has joined #openstack-nova07:06
*** bhagyashris_ has quit IRC07:07
*** bhagyashris has joined #openstack-nova07:09
*** tetsuro has joined #openstack-nova07:10
*** ccamacho has joined #openstack-nova07:11
openstackgerritChris Friesen proposed openstack/nova-specs master: Add support for emulated virtual TPM  https://review.openstack.org/57111107:11
*** jaosorior has joined #openstack-nova07:15
*** shaohe_feng has quit IRC07:15
*** apple01 has quit IRC07:17
*** shaohe_feng has joined #openstack-nova07:17
openstackgerritzhaodan7597 proposed openstack/nova master: fix a bug, when creating a vmware instance from a volume,and it goes to error state, the volume still in  "in use" state.  https://review.openstack.org/57111207:18
*** tesseract has joined #openstack-nova07:20
*** damien_r has joined #openstack-nova07:21
*** abhishekk has quit IRC07:21
*** shaohe_feng has quit IRC07:25
*** rcernin has quit IRC07:27
*** amoralej|off is now known as amoralej07:27
*** shaohe_feng has joined #openstack-nova07:27
*** mdnadeem_ has quit IRC07:28
*** alexchadin has quit IRC07:28
*** ociuhandu has joined #openstack-nova07:29
*** sahid has joined #openstack-nova07:33
*** zhaochao has quit IRC07:33
*** alexchadin has joined #openstack-nova07:34
*** dpawlik has joined #openstack-nova07:34
*** zhaochao has joined #openstack-nova07:35
*** shaohe_feng has quit IRC07:36
*** bhujay has quit IRC07:37
*** abhishekk has joined #openstack-nova07:38
*** shaohe_feng has joined #openstack-nova07:39
*** dpawlik has quit IRC07:39
*** mdnadeem_ has joined #openstack-nova07:41
*** gongysh has quit IRC07:41
*** dpawlik has joined #openstack-nova07:41
*** markvoelker has joined #openstack-nova07:45
*** dpawlik has quit IRC07:45
*** dpawlik_ has joined #openstack-nova07:45
*** jpena|off is now known as jpena07:45
*** ociuhandu has quit IRC07:45
*** shaohe_feng has quit IRC07:46
*** apple01 has joined #openstack-nova07:46
*** Tom-Tom has quit IRC07:46
*** shaohe_feng has joined #openstack-nova07:47
*** ttsiouts has joined #openstack-nova07:47
*** armaan has quit IRC07:48
*** armaan has joined #openstack-nova07:49
*** markvoelker has quit IRC07:49
*** Tom-Tom has joined #openstack-nova07:49
*** Tom-Tom_ has joined #openstack-nova07:51
*** apple01 has quit IRC07:51
*** ttsiouts has quit IRC07:53
*** gongysh has joined #openstack-nova07:53
*** Tom-Tom has quit IRC07:54
*** shaohe_feng has quit IRC07:56
*** gibi_ is now known as gibi07:56
*** shaohe_feng has joined #openstack-nova07:57
*** gibi has quit IRC07:57
*** gibi has joined #openstack-nova07:57
*** chyka has joined #openstack-nova07:58
*** bhujay has joined #openstack-nova07:59
*** gibi has quit IRC07:59
*** gibi has joined #openstack-nova07:59
*** gibi has quit IRC08:01
*** gibi has joined #openstack-nova08:01
*** chyka has quit IRC08:02
*** armaan has quit IRC08:04
*** armaan has joined #openstack-nova08:04
*** gibi has quit IRC08:06
*** gibi has joined #openstack-nova08:06
*** shaohe_feng has quit IRC08:06
*** shaohe_feng has joined #openstack-nova08:07
*** gibi has quit IRC08:09
*** mgoddard has joined #openstack-nova08:11
*** bhujay has quit IRC08:12
*** shaohe_feng has quit IRC08:17
*** shaohe_feng has joined #openstack-nova08:20
*** gibi has joined #openstack-nova08:20
*** gibi has quit IRC08:21
*** gibi has joined #openstack-nova08:21
*** vivsoni has quit IRC08:25
*** shaohe_feng has quit IRC08:27
*** shaohe_feng has joined #openstack-nova08:28
*** mdbooth has joined #openstack-nova08:33
*** shaohe_feng has quit IRC08:37
*** markvoelker has joined #openstack-nova08:38
*** shaohe_feng has joined #openstack-nova08:39
*** GonZo2000 has joined #openstack-nova08:40
*** lifeless_ has joined #openstack-nova08:44
*** lifeless has quit IRC08:44
*** shaohe_feng has quit IRC08:47
*** shaohe_feng has joined #openstack-nova08:48
*** vivsoni has joined #openstack-nova08:50
*** Alexey_Abashkin has joined #openstack-nova08:50
*** damien_r has quit IRC08:51
*** alexchadin has quit IRC08:51
*** Alexey_Abashkin has quit IRC08:51
*** AlexeyAbashkin has quit IRC08:51
*** AlexeyAbashkin has joined #openstack-nova08:52
*** brault has joined #openstack-nova08:55
*** catmando_ is now known as catmando08:55
*** ttsiouts has joined #openstack-nova08:55
*** apple01 has joined #openstack-nova08:56
openstackgerritzhaodan7597 proposed openstack/nova master: fix a bug, when creating a vmware instance from a volume,and it goes to error state, the volume still in  "in use" state. Closes-Bug:#1774137 Change-Id: I987d9b46c82b66ecc0ef2453bdd7fce6d3a7810d  https://review.openstack.org/57111208:56
*** bhujay has joined #openstack-nova08:56
*** shaohe_feng has quit IRC08:58
*** alexchadin has joined #openstack-nova08:58
*** shaohe_feng has joined #openstack-nova08:58
*** apple01 has quit IRC09:01
*** salv-orlando has joined #openstack-nova09:02
*** gongysh has quit IRC09:04
*** ttsiouts has quit IRC09:04
*** armaan has quit IRC09:05
*** fragatina has quit IRC09:05
*** armaan has joined #openstack-nova09:05
*** fragatina has joined #openstack-nova09:06
*** tetsuro has quit IRC09:06
*** ttsiouts has joined #openstack-nova09:08
*** shaohe_feng has quit IRC09:08
*** dpawlik_ has quit IRC09:09
*** dpawlik has joined #openstack-nova09:09
*** armaan has quit IRC09:09
*** shaohe_feng has joined #openstack-nova09:10
*** alexchadin has quit IRC09:11
*** markvoelker has quit IRC09:12
*** armaan has joined #openstack-nova09:13
*** dpawlik has quit IRC09:13
*** shaohe_feng has quit IRC09:18
*** dpawlik has joined #openstack-nova09:18
*** gcb has joined #openstack-nova09:19
*** shaohe_feng has joined #openstack-nova09:21
*** yamamoto has quit IRC09:21
openstackgerrittianhui proposed openstack/nova master: Fix doc mistakes  https://review.openstack.org/57114709:23
*** dr_gogeta86 has joined #openstack-nova09:25
*** dr_gogeta86 has quit IRC09:25
*** dr_gogeta86 has joined #openstack-nova09:25
openstackgerritzhaodan7597 proposed openstack/nova master: when creating a vmware instance from volume failed ,detach the volume Closes-Bug:#1774137 Change-Id: I987d9b46c82b66ecc0ef2453bdd7fce6d3a7810d  https://review.openstack.org/57111209:26
*** shaohe_feng has quit IRC09:28
*** shaohe_feng has joined #openstack-nova09:30
*** Dinesh_Bhor has quit IRC09:30
*** udesale_ has joined #openstack-nova09:31
*** udesale_ has quit IRC09:32
*** udesale_ has joined #openstack-nova09:32
*** udesale_ has quit IRC09:32
openstackgerrithyunsik Yang proposed openstack/nova master: support pxe boot between the VM  https://review.openstack.org/57068509:32
*** udesale_ has joined #openstack-nova09:33
*** udesale has quit IRC09:33
*** dtantsur|afk is now known as dtantsur09:38
*** shaohe_feng has quit IRC09:39
*** shaohe_feng has joined #openstack-nova09:39
*** shaohe_feng has quit IRC09:49
*** shaohe_feng has joined #openstack-nova09:49
*** blkart has quit IRC09:50
*** yamamoto has joined #openstack-nova09:50
*** blkart has joined #openstack-nova09:51
*** zigo_ is now known as zigo09:56
*** sree has quit IRC09:57
*** shaohe_feng has quit IRC09:59
*** GonZo2000 has quit IRC10:00
openstackgerritMerged openstack/nova stable/queens: Avoid unnecessary joins in HostManager._get_instances_by_host  https://review.openstack.org/57008310:00
*** shaohe_feng has joined #openstack-nova10:01
*** annp has quit IRC10:01
openstackgerritzhaodan7597 proposed openstack/nova master: Unable to delete volume when a vmware instance bfv is failed.  https://review.openstack.org/57111210:01
*** udesale__ has joined #openstack-nova10:02
*** udesale__ has quit IRC10:02
*** udesale has joined #openstack-nova10:03
*** GonZo2000 has joined #openstack-nova10:04
*** GonZo2000 has quit IRC10:04
*** GonZo2000 has joined #openstack-nova10:04
*** udesale_ has quit IRC10:05
*** apple01 has joined #openstack-nova10:06
*** Tom-Tom_ has quit IRC10:07
*** ttsiouts has quit IRC10:07
*** ttsiouts has joined #openstack-nova10:07
*** namnh has quit IRC10:08
*** shaohe_feng has quit IRC10:09
*** shaohe_feng has joined #openstack-nova10:10
*** ttsiouts has quit IRC10:12
*** GonZo2000 has quit IRC10:14
*** jaosorior has quit IRC10:14
*** apple01 has quit IRC10:14
*** bkopilov_ has quit IRC10:18
*** shaohe_feng has quit IRC10:20
*** psachin has joined #openstack-nova10:21
*** sambetts|afk is now known as sambetts10:23
*** shaohe_feng has joined #openstack-nova10:24
*** salv-orlando has quit IRC10:26
*** Tom-Tom has joined #openstack-nova10:27
*** shaohe_feng has quit IRC10:30
*** AlexeyAbashkin has quit IRC10:30
*** AlexeyAbashkin has joined #openstack-nova10:31
*** jaosorior has joined #openstack-nova10:31
*** Tom-Tom has quit IRC10:32
*** lifeless_ has quit IRC10:32
*** shaohe_feng has joined #openstack-nova10:33
openstackgerritsahid proposed openstack/nova master: pci: don't consider case when match tags specs  https://review.openstack.org/56580810:34
openstackgerritsahid proposed openstack/nova master: libvirt: configure trust mode for vfs  https://review.openstack.org/45851410:34
openstackgerritsahid proposed openstack/nova master: libvirt:  add vf_trusted field for network metadata  https://review.openstack.org/56634310:34
openstackgerritsahid proposed openstack/nova master: metadata: add vf_trusted field to device metadata  https://review.openstack.org/56634410:34
openstackgerritsahid proposed openstack/nova master: network: update pci request spec to handle trusted tags  https://review.openstack.org/45882010:34
*** markvoelker has joined #openstack-nova10:35
*** AlexeyAbashkin has quit IRC10:36
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy  https://review.openstack.org/57116610:39
*** ttsiouts has joined #openstack-nova10:40
*** shaohe_feng has quit IRC10:40
*** shaohe_feng has joined #openstack-nova10:41
*** gyankum has quit IRC10:41
*** abhishekk has quit IRC10:44
*** nicolasbock has joined #openstack-nova10:50
*** shaohe_feng has quit IRC10:50
openstackgerritMerged openstack/nova stable/pike: Document how to disable notifications  https://review.openstack.org/56325410:51
*** mdnadeem_ has quit IRC10:51
*** shaohe_feng has joined #openstack-nova10:52
*** hyunsikyang has joined #openstack-nova10:56
*** tbachman has quit IRC10:57
*** adreznec has quit IRC11:00
*** shaohe_feng has quit IRC11:01
*** lifeless has joined #openstack-nova11:01
*** adreznec has joined #openstack-nova11:01
*** shaohe_feng has joined #openstack-nova11:01
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337511:03
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340111:03
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy  https://review.openstack.org/57116611:03
*** apple01 has joined #openstack-nova11:04
*** markvoelker has quit IRC11:06
*** apple01 has quit IRC11:08
*** phuongnh has quit IRC11:09
*** shaohe_feng has quit IRC11:11
*** shaohe_feng has joined #openstack-nova11:12
*** armaan has quit IRC11:16
*** armaan has joined #openstack-nova11:17
*** ociuhandu has joined #openstack-nova11:19
*** apple01 has joined #openstack-nova11:20
*** dave-mccowan has joined #openstack-nova11:21
*** AlexeyAbashkin has joined #openstack-nova11:21
*** shaohe_feng has quit IRC11:21
*** shaohe_feng has joined #openstack-nova11:22
*** liuzz_ has quit IRC11:22
*** liuzz has joined #openstack-nova11:22
*** mdbooth has quit IRC11:24
*** derekh has joined #openstack-nova11:25
*** mdbooth has joined #openstack-nova11:26
*** dave-mccowan has quit IRC11:26
*** dave-mcc_ has joined #openstack-nova11:26
*** dpawlik has quit IRC11:27
*** udesale_ has joined #openstack-nova11:28
*** damien_r has joined #openstack-nova11:28
*** yamamoto has quit IRC11:29
*** sar_ has quit IRC11:29
*** udesale has quit IRC11:31
*** sar_ has joined #openstack-nova11:31
*** shaohe_feng has quit IRC11:31
*** shaohe_feng has joined #openstack-nova11:32
*** udesale_ has quit IRC11:33
*** armaan has quit IRC11:33
*** armaan_ has joined #openstack-nova11:33
*** jpena is now known as jpena|lunch11:33
*** chyka has joined #openstack-nova11:35
*** lifeless_ has joined #openstack-nova11:37
*** lifeless has quit IRC11:38
*** ociuhandu has quit IRC11:38
*** amoralej is now known as amoralej|lunch11:38
*** chyka has quit IRC11:40
*** dpawlik has joined #openstack-nova11:41
*** shaohe_feng has quit IRC11:42
*** apple01 has quit IRC11:42
*** shaohe_feng has joined #openstack-nova11:43
*** alexchadin has joined #openstack-nova11:44
*** nguyenhai_ has joined #openstack-nova11:46
nguyenhai_hi guys, how can I enable again nova-compute after it is disable by "Auto-disabled due to 10 build failures"11:47
*** apple01 has joined #openstack-nova11:49
*** armaan_ has quit IRC11:49
*** damien_r has quit IRC11:50
*** shaohe_feng has quit IRC11:52
*** shaohe_feng has joined #openstack-nova11:53
*** do3meli has joined #openstack-nova11:53
*** tiendc has quit IRC11:53
*** sar has joined #openstack-nova11:54
*** sar_ has quit IRC11:54
*** gyankum has joined #openstack-nova11:54
*** BrinZhang has joined #openstack-nova11:55
*** mikal has joined #openstack-nova11:57
*** mikal_ has quit IRC11:59
*** shaohe_feng has quit IRC12:02
*** shaohe_feng has joined #openstack-nova12:04
TahvokHey guys! On Ocata here. I've enable auto-convergence on all my nova computes, and since then, doing a live migration will shutdown the instance and start it on the destination node12:04
*** hamdyk has quit IRC12:04
TahvokI'm pretty sure that's not what should be happening. How should I investigate such issue?12:04
*** liuyulong_ has quit IRC12:06
*** yamamoto has joined #openstack-nova12:07
*** liuyulong_ has joined #openstack-nova12:07
*** damien_r has joined #openstack-nova12:08
*** tbachman has joined #openstack-nova12:08
*** ratailor has quit IRC12:11
*** shaohe_feng has quit IRC12:12
*** rha has joined #openstack-nova12:13
*** rha has quit IRC12:13
*** rha has joined #openstack-nova12:13
*** Tahvok has quit IRC12:15
*** shaohe_feng has joined #openstack-nova12:15
*** sar has quit IRC12:19
*** sree_ has joined #openstack-nova12:19
*** sar has joined #openstack-nova12:19
*** sree_ is now known as Guest7572812:20
openstackgerritMerged openstack/nova master: Remove mox in tests/unit/api/openstack/compute  https://review.openstack.org/56652812:22
*** shaohe_feng has quit IRC12:23
*** shaohe_feng has joined #openstack-nova12:23
*** BrinZhang has quit IRC12:24
*** Guest75728 has quit IRC12:24
*** GonZo2000 has joined #openstack-nova12:25
*** GonZo2000 has quit IRC12:25
*** GonZo2000 has joined #openstack-nova12:25
*** pchavva has joined #openstack-nova12:25
*** jpena|lunch is now known as jpena12:28
*** udesale has joined #openstack-nova12:29
*** cz2 has quit IRC12:29
*** cz2 has joined #openstack-nova12:30
*** armaan has joined #openstack-nova12:32
*** shaohe_feng has quit IRC12:33
*** alexchadin has quit IRC12:33
*** shaohe_feng has joined #openstack-nova12:33
*** mvenesio has joined #openstack-nova12:37
*** lifeless_ has quit IRC12:41
*** amoralej|lunch is now known as amoralej12:41
*** salv-orlando has joined #openstack-nova12:43
*** salv-orlando has quit IRC12:43
*** shaohe_feng has quit IRC12:43
*** salv-orlando has joined #openstack-nova12:44
*** salv-orlando has quit IRC12:44
*** armaan has quit IRC12:44
*** salv-orlando has joined #openstack-nova12:44
*** salv-orlando has quit IRC12:44
*** salv-orlando has joined #openstack-nova12:44
*** apple01 has quit IRC12:44
*** salv-orlando has quit IRC12:45
*** shaohe_feng has joined #openstack-nova12:45
*** salv-orlando has joined #openstack-nova12:45
*** hshiina is now known as hshiina|afk12:45
*** salv-orlando has quit IRC12:45
*** salv-orlando has joined #openstack-nova12:46
*** eharney has quit IRC12:46
*** salv-orlando has quit IRC12:46
*** udesale has quit IRC12:47
*** udesale has joined #openstack-nova12:47
*** edmondsw has joined #openstack-nova12:50
*** shaohe_feng has quit IRC12:53
*** markvoelker has joined #openstack-nova12:54
*** shaohe_feng has joined #openstack-nova12:58
*** mriedem has joined #openstack-nova12:59
*** mvenesio has quit IRC13:02
*** do3meli has quit IRC13:03
*** shaohe_feng has quit IRC13:04
*** yamamoto has quit IRC13:04
*** READ10 has joined #openstack-nova13:04
*** do3meli has joined #openstack-nova13:04
*** shaohe_feng has joined #openstack-nova13:05
*** GonZo2000 has quit IRC13:07
*** GonZo2000 has joined #openstack-nova13:08
*** armaan has joined #openstack-nova13:08
*** throwsb1 has joined #openstack-nova13:09
*** bpoulos has joined #openstack-nova13:12
*** ttsiouts has quit IRC13:13
*** tbachman has quit IRC13:14
*** shaohe_feng has quit IRC13:14
*** GonZo2000 has quit IRC13:14
*** shaohe_feng has joined #openstack-nova13:16
*** wolverineav has joined #openstack-nova13:18
*** tbachman has joined #openstack-nova13:20
*** ttsiouts has joined #openstack-nova13:22
openstackgerritMatt Riedemann proposed openstack/nova master: Set scope for remaining placement policy rules  https://review.openstack.org/57120113:22
*** GonZo2000 has joined #openstack-nova13:22
*** GonZo2000 has quit IRC13:22
*** GonZo2000 has joined #openstack-nova13:23
*** GonZo2000 has quit IRC13:23
*** GonZo2000 has joined #openstack-nova13:23
*** shaohe_feng has quit IRC13:24
*** markvoelker has quit IRC13:24
openstackgerrithyunsik Yang proposed openstack/nova master: support pxe boot between the VM  https://review.openstack.org/57068513:25
stephenfinbauzas: Friendly reminder to take a look at https://review.openstack.org/#/c/540887/, if you can :)13:25
stephenfinAlso, no API meeting today?13:25
*** esberglu has joined #openstack-nova13:26
bauzasstephenfin: LOL13:26
bauzasstephenfin: I should give you a t-shirt saying 'I love RST'13:26
*** liuzz_ has joined #openstack-nova13:26
stephenfinbauzas: And I would wear that t-shirt :)13:27
*** shaohe_feng has joined #openstack-nova13:27
*** links has quit IRC13:29
*** liuzz has quit IRC13:30
*** links has joined #openstack-nova13:30
*** tbachman has quit IRC13:30
efriedstephenfin: Does :program: render the same as ``double backticks`` ?  (It appears to under sphinx build, anyway.)13:31
efriedJust wondering why that was used vs using double backticks.13:31
stephenfinefried: Sure does, but it also has the benefits of rendering in the glossary13:31
stephenfinWhich we currently ignore :(13:31
efriedoh, a glossary...13:31
stephenfinBut I'm planning to fix that13:32
efriedas in we don't build the glossary?13:32
*** tbachman has joined #openstack-nova13:32
*** eharney has joined #openstack-nova13:32
stephenfinWe do but we don't include it anywhere, to the best of my knowledge13:33
efriedLooking forward to seeing what that looks like.  And wondering how e.g. https://review.openstack.org/#/c/540887/3/doc/source/cli/nova-rootwrap.rst@42 will fold into it.13:33
*** shaohe_feng has quit IRC13:34
*** yamamoto has joined #openstack-nova13:35
*** burt has joined #openstack-nova13:36
*** shaohe_feng has joined #openstack-nova13:36
*** GonZo2000 has quit IRC13:36
*** yamamoto has quit IRC13:39
*** nicolasbock has quit IRC13:40
openstackgerritMerged openstack/nova master: doc: Don't use single backticks in man pages  https://review.openstack.org/54088713:41
stephenfinbauzas: Ta!13:43
stephenfinThat was a fast merge. Guess they fixed that ignore-files rule thing for zuul.conf13:44
*** bpoulos has quit IRC13:44
*** links has quit IRC13:44
*** bkopilov_ has joined #openstack-nova13:44
*** shaohe_feng has quit IRC13:45
bauzasstephenfin: you know my opinion on docs reviews13:45
bauzasefried: have you seen my thread on upgrade concerns for n-rp ?13:45
*** shaohe_feng has joined #openstack-nova13:45
bauzasprobably only VGPU resource class is impacted, but I guess the upgrade concern can be there for any RC once we work on NUMA13:46
bauzasat least VCPU and MEMORY_MB13:46
*** tbachman has quit IRC13:49
bhagyashrisefried: Hi,13:49
bhagyashrisefried: I am still struggling to write the that functional test case as per our discussion I have made the FakeLibvirtDriver class as *subclass* of LibvirtDriver but still i am gertting the issue as File "nova/virt/fake.py", line 754, in __init__13:51
bhagyashris super(FakeLibvirtDriver, self).__init__(virtapi, read_only)13:51
bhagyashrisFile "nova/virt/libvirt/driver.py", line 314, in __init__13:51
bhagyashrislibvirt = importutils.import_module('libvirt')13:51
bhagyashris       File "/opt/stack/nova/.tox/functional/local/lib/python2.7/site-packages/oslo_utils/importutils.py", line 73, in import_module13:51
bhagyashris__import__(import_str)13:51
bhagyashrisImportError: No module named libvirt13:51
*** awaugama has joined #openstack-nova13:52
*** nicolasbock has joined #openstack-nova13:53
*** shaohe_feng has quit IRC13:55
jichenhi dansmith sorry to disturb , could you please help to approve this spec https://review.openstack.org/#/c/562154/  as it's mostly address the questions in the ML?13:55
bhagyashrisefried: I have moved the same functional test case under the https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_rt_servers.py#L27 but there at the time of start of compute service it's not creating the compute node resource provider in placement db13:55
*** shaohe_feng has joined #openstack-nova13:56
dansmithjichen: I will leave that to someone else, but I'm replying to your question on the ML right now13:56
jichendansmith: ok, then I will ask someone else to review and approve it , as it got some +1 and one +2 already13:57
dansmithack13:57
*** gbarros has joined #openstack-nova13:58
*** sar has quit IRC13:59
jichendansmith: and could you please check this -2 on https://review.openstack.org/#/c/523387/ ? it's been reviewed by some core and non-cores, will there be some criteria to remove the -2 so I can get more core to help (efried has reviewed the patch and it's dependency patches previously thoroughly)13:59
*** sree has joined #openstack-nova13:59
dansmithjichen: that -2 is just to hold the base patch until that patch and the rest have +2+W on them so we don't land a driver with no function14:00
dansmithjichen: so yes, I'll drop that procedural -2 when the approvals are stacked up behind it14:00
*** lpetrut has quit IRC14:00
dansmithI said that on the patch, so it shouldn't be preventing anyone from reviewing it14:00
jichendansmith: ok, I will get more people to review and have +2 +w on them then ping you later, thanks  a lot14:00
dansmithyep14:00
*** Tahvok has joined #openstack-nova14:00
*** AlexeyAbashkin has quit IRC14:02
*** AlexeyAbashkin has joined #openstack-nova14:04
mriedemsahid: some comments in your neutron docs patch for trusted vf https://review.openstack.org/#/c/571167/14:05
*** shaohe_feng has quit IRC14:05
*** tbachman has joined #openstack-nova14:05
*** shaohe_feng has joined #openstack-nova14:06
mriedemdansmith: just noticed this https://review.openstack.org/#/c/570685/ above, but wasn't there something similar that bnemec did for tripleo?14:06
dansmithI think he proposed something14:07
*** bhujay has quit IRC14:08
*** EmilienM_PTO is now known as EmilienM14:08
bhagyashriskashyap: any idea about ^14:08
kashyapbhagyashris: Sorry, which one?14:09
openstackgerritMoshe Levi proposed openstack/nova stable/pike: Don't overwrite binding-profile  https://review.openstack.org/56350114:11
mriedemso something that's been bothering me about the FFU sessions which i don't think anyone has brought up,14:11
bhagyashriskashyap: actually i have written functional test here: https://review.openstack.org/#/c/560459/11/nova/tests/functional/test_servers.py the patch is having libvirt related changes but in the functional test case i have not initialized the actual libvirt driver14:12
mriedembesides just saying, the config options churn a lot,14:12
mriedemis that for the upgrade support tag, we can't drop config options within a release, but we can across boundaries,14:12
mriedemwhich includes dropping support for deprecated config option names,14:12
*** mlavalle has joined #openstack-nova14:12
mriedemsometimes we leave those deprecated config option aliases around for much more than 1 release, like pci_passthrough_whitelist and the scheduler_default_filters14:13
bhagyashriskashyap: so as per the review comments I am trying to initialized the actual libvirt driver so that the change in code will be tested but i am facing some issues14:13
mriedemi'm kind of surprised the FFU folk haven't pushed for config option drop/renames to last more than 1 release14:13
mriedemlyarwood: have you heard anything like ^ in your FFU travels?14:13
kashyapbhagyashris: If you don't mind, I'm in the middle of something.  This requires me to see all the context, will get to it tonight.14:14
bhagyashriskashyap: ok np but i am not sure will be present because i am working in IST time zone so here it's almost EOD14:15
*** janki has quit IRC14:15
mriedeme.g. pci_passthrough_whitelist was deprecated in ocata https://review.openstack.org/#/c/356604/ and renamed to [pci]/passthrough_whitelist, certainly that's a fair enough amount of time to drop the old deprecated name14:15
kashyapbhagyashris: I'll comment on the review tomm.  I need to be AFK soon14:15
*** shaohe_feng has quit IRC14:15
*** shaohe_feng has joined #openstack-nova14:16
bhagyashriskashyap: ok thank you :)14:16
* lyarwood reads14:17
*** armaan has quit IRC14:18
lyarwoodmriedem: yeah some ops/diy folks have asked for that but we (TripleO) don't need it, heat/puppet takes care of everything for us under the covers.14:18
*** r-daneel has joined #openstack-nova14:18
*** armaan has joined #openstack-nova14:19
mriedemi imagine the pain is really just needing to read through the release notes to see which config options are deprecated/dropped each release14:20
mriedemand digesting that across all projects per release you're FFing through14:20
lyarwoodmriedem: yeah, something this group is trying to address for ops/diy folks AFAIK - https://www.openstack.org/videos/sydney-2017/making-fast-forward-upgrade-openstack-possible-with-configuration-mapping-tool14:23
* lyarwood forgets the topic this is landing under on oslo.config14:23
lyarwoodhttps://review.openstack.org/#/c/526314/ maybe?14:25
*** rajinir has joined #openstack-nova14:25
*** shaohe_feng has quit IRC14:26
*** kholkina has quit IRC14:27
*** shaohe_feng has joined #openstack-nova14:28
*** jmccarthy has joined #openstack-nova14:30
*** Luzi has quit IRC14:31
mriedemyup i heard about that for the first time at the summit14:31
*** hongbin has joined #openstack-nova14:33
*** dklyle has joined #openstack-nova14:33
*** tbachman has quit IRC14:34
*** shaohe_feng has quit IRC14:36
*** r-daneel_ has joined #openstack-nova14:36
*** shaohe_feng has joined #openstack-nova14:37
*** r-daneel has quit IRC14:37
*** r-daneel_ is now known as r-daneel14:37
*** munimeha1 has joined #openstack-nova14:38
efriedbhagyashris: Looks like the libvirt library isn't a [test-]requirement.  If you install it into your functional venv, you should be able to get around that import error.14:38
openstackgerritJay Pipes proposed openstack/nova master: mirror nova host aggregate members to placement  https://review.openstack.org/55359714:39
mriedemjaypipes: figured out massimo's cold migrate issue in the ML14:39
efriedbhagyashris: As for the second thing, not sure I can help without seeing the code.  Can you upload your patch set?14:39
bhagyashrisefried: here is my second approach test case http://paste.openstack.org/show/722352/14:41
*** moshele has quit IRC14:41
bhagyashrisefried: under /nova/tests/functional/libvirt/test_rt_servers.py14:42
jaypipesmriedem: I saw that. :)14:42
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs  https://review.openstack.org/56725814:43
*** bpoulos has joined #openstack-nova14:43
bhagyashrisefried: can you help me how i should install  libvirt library in my  functional venv14:43
efriedbhagyashris: Are you still getting that import error?14:43
stephenfinbhagyashris: IIRC, you need to install the libvirt and python-libvirt packages using your package manager (apt, dnf, yum...)14:44
efriedbhagyashris: You would do it like this:14:45
efried  bash  # enter a new subshell14:45
efried  source .tox/functional/bin/activate  # enter the `functional` venv14:45
efried  pip install libvirt  # install the lib into the venv14:45
efried  exit  # leave the venv by leaving the subshell14:45
*** sree has quit IRC14:45
efriedstephenfin: Oh, they're apt, not pip?14:45
openstackgerritStephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec  https://review.openstack.org/54129014:45
efriedbhagyashris: Listen to stephenfin, not me.  He would know better.14:45
*** sree has joined #openstack-nova14:45
stephenfinefried: I think so, yeah. It's installed by default on Fedora, anyway14:46
dansmithbhagyashris: what stephenfin said14:46
*** shaohe_feng has quit IRC14:46
efriedstephenfin: If you're in a helpful mood, would you take a look at bhagyashris's paste above? What we're going for here is a functional test setup that uses the actual libvirt driver as much as possible, so that the compute node RP is created in placement, and update_provider_tree gets called by the RT on compute startup and periodics.14:47
*** shaohe_feng has joined #openstack-nova14:48
mriedemall of this request spec hydration code just needs to be burned out with acid/fire; i've had a todo on my list for awhile to actually do that, but...14:48
*** GonZo2000 has joined #openstack-nova14:48
*** GonZo2000 has joined #openstack-nova14:48
efriedand I don't know from libvirt, so don't really have a concept of how/where stuff needs to be mocked or whatever.14:48
stephenfinefried: I thought functional tests were supposed to use fakelibvirt everywhere?14:48
*** armaan has quit IRC14:49
stephenfini.e. you're testing everything in nova but nothing outside it, if possible. Integration tests (Tempest) are for testing the whole system14:49
efriedstephenfin: You mean nova.tests.unit.virt.libvirt.fakelibvirt.FakeLibvirtFixture ?14:49
stephenfinYup14:49
sahiddansmith: what was the agreement regarding incrementing objects version for os-vif?14:49
stephenfinWait, lemme double check14:49
sahidhttps://review.openstack.org/#/c/570959/1/os_vif/objects/vif.py14:49
*** sree has quit IRC14:50
stephenfinefried: See nova/tests/functional/libvirt/test_numa_servers.py14:50
dansmithsahid: what do you mean? that patch looks correct to me (wrt the version)14:50
dansmithsahid: if it's an os-vif-specific thing, then I guess ask jaypipes14:51
sahiddansmith: no it's not it's really about objects verion. I think you did not want that we incrmeent the version because that is not going to be shared by rpc14:51
sahidor something like that, no?14:51
sahidat least it's not right now14:52
openstackgerritElod Illes proposed openstack/nova stable/pike: placement: Fix HTTP error generation  https://review.openstack.org/57121814:52
mriedemi thought we tried to follow the normal object versioning in os-vif even if it's not going over the wire yet14:52
dansmithsahid: but it could be serialized somewhere else at some point, like between neutron and nova14:52
dansmithmriedem: right, I'm not sure why not, but maybe I'm missing something14:52
sahidmriedem, dansmith ok so let's do that, seems reasonable yes14:53
stephenfinmriedem, dansmith: Reworked the numa-aware-vswitch spec per our discussion in Vancouver. Lemme know if I missed anything14:53
stephenfinWill start reworking the patches themselves shortly14:53
bhagyashrisefried, stephenfin: thanks checking for the same.14:53
mriedemstephenfin: ok thanks,14:54
mriedemgot a couple of patches ahead of me yet14:54
stephenfinack14:54
dansmithstephenfin: cool. given the complexity here, I think it probably makes sense to evaluate the spec and the code somewhat in parallel14:54
*** tbachman has joined #openstack-nova14:55
*** dave-mcc_ is now known as dave-mccowan14:55
stephenfindansmith: Just to make sure, there isn't a deadline I need to wary of, in that case?14:55
dansmithoh, maybe14:55
stephenfinI'm also nervous of forgetting our learnings about not solving everything in the spec14:55
stephenfinI think the code that I already have proves out the idea. Now I just need to shuffle about where I'm storing stuff14:56
stephenfin(also, no spec = no runway slot)14:56
*** shaohe_feng has quit IRC14:56
*** shaohe_feng has joined #openstack-nova14:57
bauzasgentle reminder for folks that the spec about multiple vGPU types has been updated https://review.openstack.org/#/c/557065/15:01
*** armaan has joined #openstack-nova15:01
bauzasjaypipes: dansmith: ^15:01
mriedemstephenfin: spec freeze is june 715:02
mriedemso you've got about a week15:02
stephenfinto...?15:02
mriedemget the spec approved15:02
stephenfinThat's in your hands then, I guess? I'll work on updating the code in the interim15:03
mriedemprobably not a bad idea to do another spec review sprint/flush in the coming week to get anything we want into rocky through before spec freeze15:03
mriedemmy volume multiattach enhancement spec has been waiting too15:03
*** test_ has quit IRC15:04
bauzasmriedem: you can review this easy spec for just a conf opt https://review.openstack.org/#/c/557065/15:04
mriedemi can add it to the queue15:05
cfriesenmriedem: what are your thoughts on getting https://review.openstack.org/#/c/465787/ merged?15:05
bauzasmriedem: fair enough15:05
cfriesenmriedem: I guess you've got a todo in there still15:05
*** gyee has joined #openstack-nova15:05
openstackgerritVishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers  https://review.openstack.org/56909015:05
stephenfinmriedem: I don't know how much use I'd be reviewing that. I've only reviewed patches for that stuff because ildikov wouldn't stop bugging me :) I can try though15:05
bauzasmriedem: stephenfin: FWIW, I'm tempted to punt https://review.openstack.org/#/c/552924/ from Rocky and defer to Stein15:06
bauzasjaypipes: ^15:06
bauzasbecause nested RPs is still in early bits, and because I shit have no time to further work on implementing it for Rocky anyway15:06
bauzasso I'll work on providing a new rev' of course, but not prioritary15:06
*** jichen has quit IRC15:07
*** shaohe_feng has quit IRC15:07
stephenfinbauzas: Wait, how does that compare with jaypipes' spec?15:07
ildikovstephenfin: I can bug you again if that helps :)15:07
jaypipesbauzas: I have a feeling tetsuro will have made significant progress on much of that in Rocky.15:07
jaypipesstephenfin: the cpu-resources spec is about dedicated cores, not NUMA.15:07
*** apple01 has joined #openstack-nova15:07
jaypipesdedicated and shared cores.15:07
mriedemcfriesen: i would like to see it merged, so i don't have thoughts on my own patch :)15:07
mriedemcfriesen: you should ask other cores15:07
*** shaohe_feng has joined #openstack-nova15:07
bauzasjaypipes: I looked at tetsuro's patches and those are still under review, right?15:07
jaypipesbauzas: of course, yes15:07
bauzasso, it's in my pipe15:08
mriedemcfriesen: just throw it in the starlingx mega patch-o-doom and it will get reviews from at&t for sure!15:08
stephenfinjaypipes: But you're using NUMA, no?15:08
jaypipesbauzas: not just tetsuro, though. efried and Citrix guys are pushing ahead.15:08
bauzasbut my point is, nested resource providers is just incepted, and possibly vGPUs are the first usecases that'll use those15:08
*** markvoelker has joined #openstack-nova15:08
mriedemcfriesen: the todo is for future refactor15:08
stephenfinjaypipes: e.g. https://review.openstack.org/#/c/555081/10/specs/rocky/approved/cpu-resources.rst@21715:08
bauzasjaypipes: I'm working on that too, remember15:08
jaypipesstephenfin: that latest revision I tried to remove as much ass possible any mentions of NUMA in so much as extra specs or anything about the NUMATopologyFilter.15:08
mriedemcfriesen: if i meant to hold up the patch on the todo, i would have wip'ed it15:08
cfriesenmriedem: the plan is to submit individual small patches...   Actually, we're already using your patch so it's already in the list.15:09
bauzasjaypipes: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130783.html15:09
mriedemcfriesen: heh i know15:09
cfriesenmriedem: yeah, I caught that on re-read15:09
bauzasjaypipes: you possibly missed it15:09
jaypipesbauzas: I'm not holding up progress on n-r-p's various efforts ongoing.15:09
mriedemcfriesen: i just need to stab the starlingx thing at every opportunity15:09
janguttersahid: Apologies for side-tracking, but I've got a question regarding the thorny issue of multiqueue and os-vif, if you have a few seconds.15:09
cfriesenmriedem: bring it on, I can take it15:09
jaypipesbauzas: I didn't miss it. just haven't had a chance to go through it.15:09
*** lpetrut has joined #openstack-nova15:09
jaypipesbauzas: will do today.15:09
bauzasack, thanks15:09
jaypipesnp15:09
mriedemcfriesen: also, poutine, wtf?!15:10
bauzasbecause I'm just litterally working on the implementation15:10
cfriesenmriedem: good poutine is awesome.  crappy poutine sucks15:10
stephenfinjaypipes: I'm guessing you can't ignore NUMA completely though, if you want to support use case 4 and the likes? https://review.openstack.org/#/c/555081/10/specs/rocky/approved/cpu-resources.rst@42515:10
bauzasjaypipes: and I'm hold until we get consensus on the right approach15:10
bauzasjaypipes: and FWIW, naichuan's implementation for Xen is impacted too15:10
*** GonZo2000 has quit IRC15:11
jaypipesstephenfin: no, I specifically *don't* want to include the realtime extra specs or any of the numa_nodes/numa_cells goo.15:11
jaypipesstephenfin: because the purpose of that spec is to handle requests for dedicated and shared CPU resources. nothing to do with NUMA really.15:12
sahidjangutter: i imagine what you are going to do it's add in nova a new condition for vif detail vrouter, which will enable the multiqueue=True15:12
mriedemstephenfin: wasn't trying to bug you for reviews on my spec, just saying i'm in the same boat15:12
sahidthat under the section vhostuser15:12
jaypipesstephenfin: and I'm deliberately trying to not bring in the awful coupling of NUMATopology and InstanceNUMATopology and the horrible extra specs that serve to construct that mess.15:12
*** bhujay has joined #openstack-nova15:12
mriedemstephenfin: we can happily drown together15:12
mriedemyou can be leo15:12
mriedemi'll be kate15:12
mriedem / who didn't drown /15:12
jaypipesmriedem: wow, how chivalrous of you :P15:12
janguttersahid: actually, the idea is to migrate the 'vrouter' vif which is not vhostuser...15:13
openstackgerritMerged openstack/nova-specs master: Libvirt file backed memory  https://review.openstack.org/56370415:13
stephenfinjaypipes: I've missed something so. I thought requesting 'resources1:PCPU=8 resources2:PCPU=8 group_policy=isolate' means host NUMA topology would need to be accounted for15:13
janguttersahid: I did some checking, and it looks like multiqueue is an instance-wide setting, not a vif-specific one.15:13
jaypipesstephenfin: no. it has nothing explicitly to do with NUMA. that just means "I need 2 groups of 8 dedicated CPU resources. they need to be provided by two distinct resource providers"15:14
janguttersahid: ref: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L70815:14
jaypipesstephenfin: the fact that the compute node resource provider might be a parent to two other child providers that *represent* different sockets or NUMA cells is really not relevant to the request that two different providers of dedicated CPU supply the requested resource groups.15:15
sahidjangutter: i'm not sure i understand what you want do?15:16
jaypipesstephenfin: in other words, if some non-NUMA devices *happened* to expose PCPU inventory under a provider tree, they would meet the requirements...15:16
bauzasjaypipes: stephenfin: just a point, we could have the same upgrade impact with VCPU resources being in child RPs than the one I described with http://lists.openstack.org/pipermail/openstack-dev/2018-May/130783.html15:16
stephenfinjaypipes: Practically speaking so, requesting 'resources1:PCPU=8 resources2:PCPU=8 group_policy=isolate' will always fail until NUMA is modelled in placement?15:16
stephenfinAh, gotcha15:16
jaypipesstephenfin: again, we are trying desperately to free ourselves of the NUMA coupling that has plagued all of this for years.15:16
*** bhujay has quit IRC15:16
*** r-daneel has quit IRC15:16
*** do3meli has quit IRC15:16
janguttersahid: vrouter kernel mode plugging needs to create a tap with multiqueue enabled, if libvirt starts the VM up looking for it.15:17
*** shaohe_feng has quit IRC15:17
stephenfinjaypipes: But we're still coupled, no? I mean, we're going to need NUMA for the above request to work15:17
bhagyashrisefried: any suggestion about second approach15:17
jaypipesstephenfin: I'm trying my best to treat NUMA as just the model of distances to various memory (and memory page) caches to a processor/bus, instead of coupling all that mess with CPU pinning.15:17
*** bhujay has joined #openstack-nova15:17
janguttersahid: the VIFGeneric portion can set the libvirt config correctly for multiqueue, but, it looks like it's going to be difficult to pass "multiqueue" on a per-vif basis without modding large portions of the os-vif plugging code.15:18
sahidjangutter: ok so you plugin needs to know whether multiqueue is enable, which is via the os-vif object nova is filling then passing the the os-vif plugin15:18
openstackgerritMatt Riedemann proposed openstack/nova master: Add osprofiler config options to generated reference  https://review.openstack.org/57122515:18
janguttersahid: yep, so, either we can pass "multiqueue=True" via the vif os-vif object, or via the instance os-vif object.15:18
*** shaohe_feng has joined #openstack-nova15:19
*** zcorneli has joined #openstack-nova15:19
*** dpawlik has quit IRC15:19
bauzasjaypipes: unfortunately from what I understood from operators, pinning is there because NUMA affinity misses for PCI devices15:19
bauzasjaypipes: so both are somehow related15:20
stephenfinjaypipes: To be clear though, use case 4 in that spec cannot be achieved with modelling NUMA in placement, right?15:20
janguttersahid: the trick is, passing it via the VIF looks very tricky. A lot of code has to be added there, since, paradoxically, the choice whether to enable multiqueue depends on the instance and flavor settings, not VIF settings.15:20
stephenfinjaypipes: Use case 4 = https://review.openstack.org/#/c/555081/10/specs/rocky/approved/cpu-resources.rst@42515:20
openstackgerritMatt Riedemann proposed openstack/nova master: Add osprofiler config options to generated reference  https://review.openstack.org/57122515:20
bauzasjaypipes: in terms of priority, I'd consider NUMA placement and NUMA affinity be prioritary to CPU pinning, because folks care more about proximity and throughput than real placement decisions15:21
openstackgerritMerged openstack/nova master: doc: Start using openstackdoctheme's extlink extension  https://review.openstack.org/54088815:21
sahidjangutter: ok i see, I could see it live in instance actually15:21
bauzasCPU pinning is just a hack15:21
stephenfinHurrah! The extlink patch merged at last15:21
janguttersahid: for example, you can spawn an instance asking for multiqueue, but only 1 cpu, and multiqueue will magically disable :-p15:21
jaypipesbauzas: I'm referring to the poor software design of coupling the topology objects together with memory pages, usage of instance virtual NUMA topology on a host NUMA topology, and all that.15:21
bauzasjaypipes: sure, I understood that point15:22
stephenfinbauzas: I think jaypipes is suggesting we can do CPU pinning without _needing_ to think about NUMA affinity. They don't have to be tied15:22
bauzasstephenfin: my own take is that CPU pinning isn't required if we go with NUMA affinity15:22
sahidjangutter: yes i that is not what you want?15:22
stephenfinBut anyone that is using this will want to couple them, given the performance implications of not doing so15:22
stephenfinbauzas: It isn't currently, no15:22
janguttersahid: That's perfectly acceptable, as long as both sides agree.15:22
bauzasstephenfin: I don't disagree with your point :)15:23
jaypipesstephenfin: use case 4 can be implemented, sure, but until sockets are modeled as child providers or a compute node parent provider, the scheduler would always return NoValidHosts for a request like use case 4. that clear?15:23
bauzasstephenfin: I mean, CPU pinning is tied to NUMA resources because people care of throughput15:23
stephenfinjaypipes: Sure, if s/sockets/NUMA nodes/15:23
janguttersahid: another reason for keeping it as a parameter of the instance...15:23
stephenfinbecause of Cluster-on-Die etc.15:23
jaypipesstephenfin: sigh...15:23
stephenfinSorry :)15:24
janguttersahid: thanks, I'll respin. The nova bit is much easier to do with it being part of the instance.15:24
jaypipesstephenfin: quick RST doc question for you..15:24
stephenfinOooh, yay15:25
*** r-daneel has joined #openstack-nova15:25
stephenfinjaypipes: shoot15:25
jaypipesstephenfin: how can I make the text of an anchor different between two calling points to that anchor?15:25
*** Guest48153 is now known as melwitt15:25
stephenfin:ref:`text you want to display <text-of-anchor>`15:25
jaypipesstephenfin: you rightfully pointed out that the granular request spec is linked in two places.15:25
jaypipesstephenfin: ah..15:25
sahidok i don't really see why it's reflecting better the state of the instance using multiqueue or not, but i'm do not disagree with you to have that attribute with instance15:25
jaypipesstephenfin: ty sir15:25
sahidjangutter: please ping me when you send a new version15:26
jaypipesstephenfin: and no need for the trailing _ char when using :ref?15:27
*** shaohe_feng has quit IRC15:27
melwittdansmith, mriedem: are we cool to approve this follow up specless blueprint for https://blueprints.launchpad.net/nova/+spec/memory-backend-file-discard that depends on the libvirt-file-backed-memory blueprint approved today?15:27
stephenfinNope. :ref:`x` is a Sphinx thing that lets you cross-reference between docs etc., if you need to. `x`_ is plain old rST (docutils)15:28
jaypipesstephenfin: ack. and the < > brackets are required, yes?15:28
dansmithmelwitt: um, not sure why we need that extra blueprint, tbh15:28
dansmithwe're just waiting for libvirt 4.4.0 and then could tag the tweak to nova against the original blueprint, IMHO15:28
*** shaohe_feng has joined #openstack-nova15:28
stephenfinjaypipes: If you wish to use a different display text, yes15:29
mriedemmelwitt: had never seen it, but seems it could be part of the original bp15:29
dansmithmriedem: melwitt yeah, it will just be a small tweak to the xml we generate if libvirt is new enough15:29
melwittokay, I saw the link in the review comments on the spec for the original15:29
*** sar has joined #openstack-nova15:30
mriedemwe could amend the spec to add it as a separate work item15:30
jaypipesstephenfin: hmm, getting a doc error...15:30
mriedemi've been meaning to go over the spec anyway15:30
dansmithmriedem: it was commented on the spec, just left out because it's not critical and isn't in released libvirt yet15:31
melwittdiscussed here https://review.openstack.org/#/c/563704/8/specs/rocky/approved/libvirt-file-backed-memory.rst@17015:31
jaypipesstephenfin: any ideas? http://paste.openstack.org/show/722356/15:31
efriedbhagyashris: Second approach?15:32
dansmitha tiny spec revision when it becomes available would be plenty, and/or just an extra patch tagged against the bp15:32
mriedemwhen is libvirt 4.4.0 going to get released?15:32
mriedemwithin the next month or so?15:32
*** janki has joined #openstack-nova15:33
stephenfinjaypipes: Oops. I sent you the wrong way. That's for headers :(15:33
*** pcaruana has quit IRC15:33
melwittI was thinking maybe we should just approve the follow on bp since it's already been created?15:33
stephenfinjaypipes: Try `foo <bar>`_15:33
mriedemit's just as easy to obsolete the blueprint15:34
mriedemif it doesn't happen in rocky, then yeah i'd make it a separate blueprint in stein15:34
dansmithmriedem: I dunno, 4.4.0 is out, so the version might not be right15:36
*** tianhui has quit IRC15:36
stephenfinjaypipes: Hmm, that doesn't work either. Maybe ignore that one for now. I was sure :ref: would do it15:36
dansmithmriedem: but they do monthly releases I think, so should be soon if not now15:36
*** udesale has quit IRC15:36
mriedemjust say that kubevirt really needs this like yesterday15:37
stephenfinjaypipes: Got it. `foo <bar_>`_ (the underscore after 'bar' is the important bit)15:37
*** shaohe_feng has quit IRC15:37
jaypipesstephenfin: bingo, that got it. thx for your help! :)15:37
*** tianhui has joined #openstack-nova15:38
*** shaohe_feng has joined #openstack-nova15:39
melwittthis page says libvirt v4.4.0 is unreleased https://libvirt.org/news.html15:39
dansmithI found a package for it, hang on15:40
dansmithhttps://launchpad.net/ubuntu/+source/linux/4.4.0-121.14515:40
dansmithmaybe that's built from master or something15:40
dansmithoh danh15:40
dansmiththat's the kernel, google turned it up instead15:40
dansmithso yeah nevermind :D15:40
melwittah okay. last version was released v4.3.0 (2018-05-02) so if monthly then it will be any day now15:41
*** tidwellr has joined #openstack-nova15:41
dansmithhttps://imgur.com/a/effQ3Gn15:41
dansmiththanks a lot google15:41
melwittyou got googled15:41
dansmithapparently15:41
*** markvoelker has quit IRC15:41
melwittokay, I'll mark the bp as Obsolete with a note that libvirt 4.4.0 is soon and that the addition of the discard feature can be rolled into the original bp15:42
dansmithcool15:42
*** udesale has joined #openstack-nova15:43
openstackgerritChris Friesen proposed openstack/nova master: add parameter docstring for 'params' to libvirt.guest.Guest.migrate()  https://review.openstack.org/57123615:43
cfriesenkashyap: ^15:43
zcorneliWorks for me. I'll look into what it'll take to add that to the code (particularly the migration segments)15:43
kashyapcfriesen: Thanks; will review tomm; heading to my Dutch class :-)15:44
cfriesenkashyap: no rush15:45
*** zhaochao has quit IRC15:45
*** shaohe_feng has quit IRC15:48
*** tianhui has quit IRC15:49
*** tianhui has joined #openstack-nova15:50
melwittzcorneli: cool, thanks. to be clear, it's also fine for sahid to work on it if you want, authors can work together on one blueprint, just link the blueprint in the commit message of the new patches when libvirt 4.4.0 is out15:50
melwittwe're just organizing the patches on one blueprint as one feature altogether15:51
*** apple01 has quit IRC15:51
openstackgerritChris Friesen proposed openstack/nova master: add parameter docstring for 'params' to libvirt.guest.Guest.migrate()  https://review.openstack.org/57123615:51
sahidmelwitt: ok i saw you comment on the bp as the one from matt, it makes sense to have only one bp yes15:52
*** shaohe_feng has joined #openstack-nova15:52
melwittcoolness15:52
sahidi was not sure if it was something we wanted to do, so i tried to be proactive ;)15:53
melwittyeah, I wasn't sure either, so no worries there :)15:53
zcorneliWorks for me either way. Definitely agreed on having it all under the one bp.15:53
*** lyan has quit IRC15:54
*** lpetrut has quit IRC15:55
*** mriedem has quit IRC15:56
*** mriedem has joined #openstack-nova15:56
*** shaohe_feng has quit IRC15:58
*** shaohe_feng has joined #openstack-nova15:59
*** chyka has joined #openstack-nova16:01
cfriesendansmith: any chance you could take a look at mriedem's patch at https://review.openstack.org/#/c/465787/ ?  It's a noticeable improvement for instances that have many vifs, we've been using it for a year now.16:03
* dansmith looks for the appropriate StarlingX snarky comment16:04
dansmithcfriesen: I'll add it to my queue, but in the middle of something else atm16:04
cfriesenno worries, it's waited this long. :)16:04
*** moshele has joined #openstack-nova16:08
*** shaohe_feng has quit IRC16:08
*** shaohe_feng has joined #openstack-nova16:09
openstackgerritMatt Riedemann proposed openstack/nova master: Use instance project/user when creating RequestSpec during resize reschedule  https://review.openstack.org/57124516:13
*** sahid has quit IRC16:14
*** moshele has quit IRC16:14
melwittI saw that mriedem suggested another spec review day earlier and thought it sounds like a good idea. what does everyone think of a spec review day next tuesday jun 5 ahead of the spec freeze jun 7?16:18
*** tianhui has quit IRC16:18
*** jmccarthy has left #openstack-nova16:18
*** tianhui has joined #openstack-nova16:18
*** shaohe_feng has quit IRC16:18
mriedemwfm16:21
*** shaohe_feng has joined #openstack-nova16:23
*** pcaruana has joined #openstack-nova16:23
*** lpetrut has joined #openstack-nova16:24
*** udesale has quit IRC16:26
mriedemjaypipes: https://review.openstack.org/#/c/571245/ is part 1 in my multi-part novella about request spec + AggregateMultiTenancyIsolation + cold migrate16:26
mriedem"it was a dark and stormy night, and we had an incomplete request spec causing much gastric distress"16:27
openstackgerritJay Pipes proposed openstack/nova-specs master: Standardize CPU resource tracking  https://review.openstack.org/55508116:27
jaypipesstephenfin: ^^16:27
jaypipesmriedem: worst. plotline. ever.16:27
jaypipes:P16:27
mriedemha16:28
stephenfinjaypipes: Sweet. It's nearly 5:30 and super sunny outside so I'm going to go roll around in the grass or something. I'll tackle that in the morning though16:28
* mriedem imagines stephenfin marking trees and such16:28
*** shaohe_feng has quit IRC16:29
stephenfinjaypipes: In the interim, the NUMA aware vSwitch spec has been updated per comments from dansmith and mriedem (plus gibi). Would appreciate another look over16:29
jaypipesstephenfin: rock on.16:29
jaypipesstephenfin: will do.16:30
*** shaohe_feng has joined #openstack-nova16:30
openstackgerritZack Cornelius proposed openstack/nova master: Refactor libvirt get_memory_used_mb()  https://review.openstack.org/57103016:33
openstackgerritZack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt  https://review.openstack.org/56787616:33
dansmithmriedem: wassup with this? https://bugs.launchpad.net/nova/+bug/167560716:36
openstackLaunchpad bug 1675607 in OpenStack Compute (nova) "test_list_migrations_in_flavor_resize_situation fails with NoValidHost - AvailabilityZoneFilter returned 0 hosts" [High,Fix released] - Assigned to Dan Smith (danms)16:36
mriedemdansmith: that i changed the severity16:38
mriedem?16:38
mriedemsee https://bugs.launchpad.net/nova/+bug/177420516:38
openstackLaunchpad bug 1774205 in OpenStack Compute (nova) "AggregateMultiTenancyIsolation uses wrong tenant_id during cold migrate" [High,In progress] - Assigned to Matt Riedemann (mriedem)16:38
dansmithmriedem: yeah16:38
mriedemthe problem that was reported in the ops list was fixed by you via that bug in pike,16:38
mriedembut there are still some issues like the patch above, plus i'm going to add a functional test for the AggregateMultiTenancyIsolation scenario in the ops list16:39
dansmithoh I see16:39
*** shaohe_feng has quit IRC16:39
dansmithokay16:39
*** shaohe_feng has joined #openstack-nova16:39
*** jpena is now known as jpena|off16:41
openstackgerritChris Friesen proposed openstack/nova-specs master: Add support for emulated virtual TPM  https://review.openstack.org/57111116:44
openstackgerritJay Pipes proposed openstack/nova master: mirror nova host aggregate members to placement  https://review.openstack.org/55359716:45
jaypipesefried: ^16:45
*** moshele has joined #openstack-nova16:47
*** salv-orlando has joined #openstack-nova16:47
*** salv-orlando has quit IRC16:47
*** salv-orlando has joined #openstack-nova16:48
*** salv-orlando has quit IRC16:48
*** salv-orlando has joined #openstack-nova16:48
*** mgoddard has quit IRC16:49
*** salv-orlando has quit IRC16:49
*** salv-orlando has joined #openstack-nova16:49
*** shaohe_feng has quit IRC16:49
*** salv-orlando has quit IRC16:50
*** shaohe_feng has joined #openstack-nova16:50
*** salv-orlando has joined #openstack-nova16:50
*** armaan has quit IRC16:50
*** salv-orlando has quit IRC16:50
*** damien_r has quit IRC16:51
*** salv-orlando has joined #openstack-nova16:51
*** salv-orlando has quit IRC16:51
*** moshele has quit IRC16:51
*** jpena|off is now known as jpena16:52
*** gbarros has quit IRC16:53
dansmithmriedem: are you intentionally keeping this unified for a backport or something? https://review.openstack.org/#/c/465787/7/nova/network/neutronv2/api.py@240616:56
*** gbarros has joined #openstack-nova16:56
dansmiththat does seem a bit like a "if special case, do thing, then bail" hack in the middle of a function16:56
mriedemnot necessarily for a backport, but to keep the scope of the change down16:57
mriedemsplitting it out into a separate function means it has to pass back a sentinel to tell the caller what to do16:57
mriedemi can do that, i just figured it was easier in a follow up so it doesn't confuse the original fix16:58
*** sambetts is now known as sambetts|afk16:59
*** shaohe_feng has quit IRC16:59
*** dtantsur is now known as dtantsur|afk17:00
*** derekh has quit IRC17:01
*** shaohe_feng has joined #openstack-nova17:02
*** amoralej is now known as amoralej|off17:07
mriedemi can never remember if a cold migrate puts the server in VERIFY_RESIZE status17:07
mriedemit's all the same code so i assume it does17:07
dansmithafaik it does17:08
melwittlast time I used it in devstack it does17:08
mriedemour api ref doesn't mention any post-conditions on cold migrate17:08
melwitthave to confirm or revert the resize after that17:08
mriedemso i always have to look17:08
*** damien_r has joined #openstack-nova17:08
*** shaohe_feng has quit IRC17:10
*** tesseract has quit IRC17:10
mriedemlow-hanging-fruit for anyone that cares https://bugs.launchpad.net/nova/+bug/177423417:11
openstackLaunchpad bug 1774234 in OpenStack Compute (nova) "api-ref: cold migrate reference doesn't mention asynchronous post conditions" [Medium,Triaged]17:11
*** yamahata has joined #openstack-nova17:11
melwittnoyce17:11
*** shaohe_feng has joined #openstack-nova17:12
*** ttsiouts has quit IRC17:15
*** ttsiouts has joined #openstack-nova17:16
*** damien_r has quit IRC17:17
*** jpena is now known as jpena|off17:18
*** shaohe_feng has quit IRC17:20
*** ttsiouts has quit IRC17:20
*** shaohe_feng has joined #openstack-nova17:21
*** moshele has joined #openstack-nova17:21
*** gyankum has quit IRC17:23
*** AlexeyAbashkin has quit IRC17:29
*** markvoelker has joined #openstack-nova17:30
*** shaohe_feng has quit IRC17:30
openstackgerritBrianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620417:31
openstackgerritBrianna Poulos proposed openstack/nova master: Add notification support for trusted_certs  https://review.openstack.org/56326917:31
openstackgerritBrianna Poulos proposed openstack/nova master: Add certificate validation docs  https://review.openstack.org/56015817:31
*** shaohe_feng has joined #openstack-nova17:31
*** AlexeyAbashkin has joined #openstack-nova17:31
*** moshele has quit IRC17:32
*** AlexeyAbashkin has quit IRC17:36
*** sridharg has quit IRC17:38
*** shaohe_feng has quit IRC17:40
*** shaohe_feng has joined #openstack-nova17:41
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test for AggregateMultiTenancyIsolation + migrate  https://review.openstack.org/57126517:43
*** cz2 has quit IRC17:43
*** moshele has joined #openstack-nova17:45
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test for AggregateMultiTenancyIsolation + migrate  https://review.openstack.org/57126517:46
*** gbarros has quit IRC17:50
*** gbarros has joined #openstack-nova17:50
*** shaohe_feng has quit IRC17:51
*** sree has joined #openstack-nova17:52
*** lpetrut has quit IRC17:53
*** shaohe_feng has joined #openstack-nova17:53
*** zcorneli_ has joined #openstack-nova17:54
*** zcorneli has quit IRC17:57
*** AlexeyAbashkin has joined #openstack-nova17:57
*** sree has quit IRC17:57
*** psachin has quit IRC17:57
*** markvoelker has quit IRC18:00
*** Alexey_Abashkin has joined #openstack-nova18:00
*** shaohe_feng has quit IRC18:01
*** AlexeyAbashkin has quit IRC18:02
*** Alexey_Abashkin is now known as AlexeyAbashkin18:02
*** shaohe_feng has joined #openstack-nova18:04
*** mvenesio has joined #openstack-nova18:05
*** zcorneli_ is now known as zcorneli18:07
mdboothdansmith: Hey, looking at https://bugzilla.redhat.com/show_bug.cgi?id=1584315 at first glance I think that would affect master18:09
openstackbugzilla.redhat.com bug 1584315 in openstack-nova "DiskNotFound error during instance resize-confirm after restarting nova compute" [Unspecified,New] - Assigned to nova-maint18:09
dansmithmdbooth: if so you should be able to find it in gate logs during a resize, right?18:09
*** pcaruana has quit IRC18:09
mdboothdansmith: Depends how long it goes unconfirmed18:10
mdboothBut specifically the issue after resize18:10
mdboothSorry, *restart* of the service18:10
*** shaohe_feng has quit IRC18:11
mdboothIt looks like ResourceTracker still initially populates the compute_nodes dict in update_available_resource18:11
*** ccamacho has quit IRC18:11
mdboothWhich calls driver.get_available_resource() first18:11
mdboothwhich I think would still fail18:11
mdbooth... so it would never be populated18:12
*** shaohe_feng has joined #openstack-nova18:12
mdboothI should.. you know... actually test it :)18:12
dansmithif you have to restart the service while something is in resize then we wouldn't hit it in the gate,18:12
dansmithbut you could write a functional test for it18:12
mdboothThat's part 2 of the failure. Part 1 we could potentially see in the gate18:13
mdboothThat's the DiskNotFound error18:13
dansmithyeah18:13
mdboothIf an instance stayed unconfirmed for more than a minute, we should see that18:13
dansmithor we got lucky18:13
* mdbooth tries to work out logstash18:14
*** lpetrut has joined #openstack-nova18:14
dansmithmdbooth: mriedem <3's helping people with logstash18:14
dansmithmdbooth: http://logs.openstack.org/96/569296/1/check/legacy-tempest-dsvm-neutron-scenario-linuxbridge/518dd97/logs/screen-n-cpu.txt#_May_30_17_38_39_17639618:20
*** mriedem1 has joined #openstack-nova18:21
mriedem1mdbooth: dansmith: logstash taint gonna help ye18:21
dansmithmriedem1: no?18:21
mriedem1because we don't have tests in tempest that wait for periodics to run in the services18:21
*** shaohe_feng has quit IRC18:21
mriedem1the tempest tests would be non-deterministic and likely timeout18:21
dansmithmriedem1: right, luck18:21
mdboothYeah18:21
dansmithI found a bunch of cases where it happens18:22
dansmithhere's another: http://logs.openstack.org/82/566182/5/check/neutron-tempest-ovsfw/30a3077/logs/screen-n-cpu.txt#_May_30_15_33_14_68662418:22
mdboothWhat did you type in the query box, btw?18:22
mriedem1i've seen things where we get DiskNotFound races while the RT update periodic runs while we're concurrently deleting a server18:22
* mdbooth got as far as trying to find out which manual to read :)18:22
* dansmith checks the pound/dollar conversion rate18:22
dansmithmdbooth: message:"DiskNotFound"18:22
mriedem1dansmith: yup same call stack i was thinking of18:23
*** shaohe_feng has joined #openstack-nova18:23
*** mriedem has quit IRC18:23
dansmithmriedem1: sounds like if you restart the service in that scenario, you get really screwed18:23
dansmithanyway, mdbooth is going to fix it up :)18:23
*** Tom-Tom has joined #openstack-nova18:23
mriedem1what we need is HA nova-compute18:24
*** mriedem1 is now known as mriedem18:24
* mdbooth is going to go home, have dinner, and go to Legoland tomorrow18:24
* mdbooth will transcribe the BZ into launchpad first, though :)18:24
*** moshele has quit IRC18:24
dansmithmriedem: mo' computes, mo' pro'lems18:24
mriedemthis is the recent thing i was thinking of https://github.com/openstack/nova/commit/5f16e714f58336344752305f94451e7c7c55742c18:25
*** lpetrut has quit IRC18:26
dansmithah, but we shouldn't stop going to the next instance and finishing update_available_resource()18:26
mriedemwe don't after that patch18:26
mriedemdepending on the state of the instance18:26
dansmithright, but this will happen on resize, before confirm,18:26
dansmithwhich means you will re-raise18:26
dansmithsince we don't undefine the domain until confirm18:26
mriedemsure, i was just handling the race with delete bug i was seeing in the gate all the time18:27
dansmithyup18:27
mriedemi figured there would be additional scenarios that need to be dealt with18:27
mriedemonce mdbooth smelt what was dealt18:28
mdboothI don't think that patch will fix this, because it only ignores DiskNotFound if task state is set18:30
mdboothWhich it looks like it's not between resize and confirm18:30
dansmithmdbooth: rightI just said that :)18:30
mdboothdansmith: Well I wasn't looking! I was reading code.18:30
dansmithmdbooth: that patch was for the deleting task state basically18:30
dansmithmdbooth: keep up!18:30
* mdbooth creates an LP18:31
*** Tom-Tom has quit IRC18:31
*** shaohe_feng has quit IRC18:32
*** shaohe_feng has joined #openstack-nova18:35
*** lyan has joined #openstack-nova18:35
*** AlexeyAbashkin has quit IRC18:40
*** shaohe_feng has quit IRC18:42
*** shaohe_feng has joined #openstack-nova18:43
*** markvoelker has joined #openstack-nova18:50
melwittdansmith, mriedem: are we cells meeting today? looks like the CERN peeps aren't around today18:50
dansmithoh, I have something on my calendar that covers it up so I wasn't paying attention18:51
dansmithso yeah, cancel if we can18:51
*** READ10 has quit IRC18:51
dansmithwe just talked last week anyway18:51
melwitt+118:51
*** shaohe_feng has quit IRC18:52
mriedemdon't need to have a meeting, but tssurya has 2 bug fixes that i'm +2 on which you should probably look at18:53
melwittk18:53
dansmithlinks18:53
mriedemhttps://review.openstack.org/#/c/568271/ and https://review.openstack.org/#/c/567878/18:53
dansmithI'll get the second18:54
dansmithsince online migration18:54
melwittk18:54
*** markvoelker has quit IRC18:55
*** fragatina has quit IRC18:56
*** shaohe_feng has joined #openstack-nova18:57
*** moshele has joined #openstack-nova18:58
*** rosmaita has joined #openstack-nova19:01
*** shaohe_feng has quit IRC19:02
*** rosmaita has left #openstack-nova19:02
*** shaohe_feng has joined #openstack-nova19:04
*** moshele has quit IRC19:05
mriedemdansmith: we have backported online data migrations to fix bugs before https://review.openstack.org/#/c/408725/19:05
*** ttsiouts has joined #openstack-nova19:06
dansmithokay, probably want to do a special reno with it though right? so people (if paying attention) will know to re-run them19:06
*** janki has quit IRC19:08
mriedemyeah i think so,19:08
mriedemhttps://review.openstack.org/#/c/529389/ was similar but not as explicit,19:08
mriedemthe idea just being, if we can get the fix into the stable branches before people roll up to them, then great, else we tried19:09
dansmithsure, that's a fair point19:09
dansmithanyway, wasn't saying we couldn't or shouldn't, just that we should highlight it19:09
mriedemshould we put a simple 'fixes' reno on that patch now before backports?19:09
mriedemthen we don't need to make it weird on stable19:09
dansmithI think a dedicated reno for the backports will make it more clear, right? "If you are applying this update after already being on the current release, you'll need to re-run online migrations" or something19:10
*** moshele has joined #openstack-nova19:10
*** ttsiouts has quit IRC19:10
mriedemsure that works for me as well19:11
dansmithif people see that it's fixed in the stable they're running, but it's still broken for them and they don't realize they would need to re-run online migrations at a peri-release boundary they might be confused19:11
* dansmith lunches19:12
mriedemi'll push the backport to queens once i get done with reviewing sahid's trusted vf series again19:12
*** shaohe_feng has quit IRC19:13
*** ttsiouts has joined #openstack-nova19:13
*** shaohe_feng has joined #openstack-nova19:14
*** moshele has quit IRC19:19
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: configure trust mode for vfs  https://review.openstack.org/45851419:19
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt:  add vf_trusted field for network metadata  https://review.openstack.org/56634319:19
openstackgerritMatt Riedemann proposed openstack/nova master: metadata: add vf_trusted field to device metadata  https://review.openstack.org/56634419:19
openstackgerritMatt Riedemann proposed openstack/nova master: network: update pci request spec to handle trusted tags  https://review.openstack.org/45882019:20
*** shaohe_feng has quit IRC19:23
*** shaohe_feng has joined #openstack-nova19:24
*** MikeG451 has joined #openstack-nova19:25
*** tianhui has quit IRC19:28
*** tianhui has joined #openstack-nova19:28
*** ttsiouts has quit IRC19:31
*** lifeless has joined #openstack-nova19:31
*** ttsiouts has joined #openstack-nova19:32
*** nicolasbock has quit IRC19:33
*** shaohe_feng has quit IRC19:33
*** shaohe_feng has joined #openstack-nova19:34
*** wolverineav has quit IRC19:35
*** wolverineav has joined #openstack-nova19:35
*** wolverineav has quit IRC19:40
*** shaohe_feng has quit IRC19:43
*** moshele has joined #openstack-nova19:44
efriedjaypipes: Done.  mriedem: I'm +2 on https://review.openstack.org/#/c/553597/ now.19:44
*** shaohe_feng has joined #openstack-nova19:44
*** markvoelker has joined #openstack-nova19:45
mriedemartom: you might want to take a quick run through this https://review.openstack.org/#/c/566344/19:46
mriedemefried: ok19:46
*** moshele has quit IRC19:46
*** tssurya has joined #openstack-nova19:47
mriedem"Snarklepuss is actually my middle name, not my surname." blows my mind19:47
mriedemthat's professional snark at a PhD level19:47
*** slaweq_ has joined #openstack-nova19:47
*** slaweq has quit IRC19:48
*** nicolasbock has joined #openstack-nova19:48
artommriedem, indeed, thanks for motivating me19:48
artomThat series has been sitting on my review list since forever19:48
*** eharney has quit IRC19:49
mriedemartom: besides that one patch i'm +2 on the rest19:49
mriedemartom: if you're so inclined, you could even address my comments in that one and we could get the entire thing +2ed today19:49
artommriedem, not a bad idea, sahid's done for the day anyways19:49
artomI don't think he'll mind19:50
artomWill we lose all your +2s if I push/rebaes tho?19:50
mriedemi've been addressing my own nits to keep it going19:50
mriedemso yeah he won't mind19:50
mriedemno19:50
artom"rebaes"19:50
mriedemif you rebase like you spell, we might have problems19:50
artomWhen you come back to your bae19:50
tssuryamriedem, dansmith: thanks for the reviews, I just got back from a (2 weeks long) vacation and also don't have anything for the meeting. Will catch up on the summit stuff from tomorrow.19:51
dansmithtssurya: cool19:51
*** avolkov has quit IRC19:51
mriedemtssurya: rest assured we talked you up big time to belmiro and tim in vancouver19:51
tssuryamriedem: haha thanks a ton!!19:51
mriedemartom: just: git review -d 458820 && git rebase -i HEAD~219:52
mriedemfix the one, then git rebase --continue && git review -R -y19:52
artomAh, yeah, rebasing on the one below it instead of master isn't a bad idea19:53
artomTiny bit patronizing tho ;)19:53
*** tssurya has quit IRC19:54
*** shaohe_feng has quit IRC19:54
mriedemright don't rebase on master19:54
mriedemthat makes the diffs between patches hard to follow19:54
*** markvoelker has quit IRC19:54
mriedemi only rebase on master if i have a merge conflict19:54
*** shaohe_feng has joined #openstack-nova19:54
*** mdbooth has quit IRC19:54
artomAt some point, someone somewhere will make a diff of diffs tool19:54
artomI remember Diana talking about it before she left Red hat19:55
dansmithgit review can do that for ou19:55
mriedemtonyb probably already has one19:55
artomdansmith, yeah? Will need to look into that19:55
dansmithgit review -m XXX,4,519:55
mriedemi'm currently explaining the difference, over text message, between bratwurst, kielbasa and summer sausage19:56
mriedemand can't tell if it's some cruel joke19:56
dansmithgit review can't do that19:56
dansmithgit review will do anything for love, but it won't ... doo... thaaat.19:56
mriedemgit review --sausage-index19:56
dansmithhah19:56
* artom tries very hard to cram a sausage party joke in here19:57
dansmithI think you just did19:57
artomSo meta19:57
*** awaugama has quit IRC20:00
artommriedem, so is 'vf_trusted' always going to show in device metadata from now on? And set to True/False?20:01
artomOr it'll only appear if the VF is trusted?20:01
artom(Yeah, I could read, but... :/ )20:01
mriedemhe's always putting it in,20:01
mriedemdefault to False for old devices, which is correct20:01
mriedemthe new field shows up if the requested version on the metadata api is new enough20:02
artomI'm cool with whatever, just wanted to be sure20:02
mriedemi tend to like adding new fields like we do in the compute rest api, which is we add the new field always, we don't expose it conditionally based on it's value20:02
dansmithdefinitely20:02
dansmithnot all languages have a __contains__ type operator, which makes checking for the presence before the value annoying20:03
*** efried has quit IRC20:03
*** shaohe_feng has quit IRC20:04
*** efried has joined #openstack-nova20:04
*** mvenesio has quit IRC20:08
*** shaohe_feng has joined #openstack-nova20:08
*** mvenesio has joined #openstack-nova20:08
*** throwsb1 has quit IRC20:09
mriedemefried: jaypipes: +W on the mirror patch20:09
efriedsüsse20:09
*** mvenesio has quit IRC20:13
*** wolverineav has joined #openstack-nova20:13
*** shaohe_feng has quit IRC20:14
*** shaohe_feng has joined #openstack-nova20:14
*** armaan has joined #openstack-nova20:15
*** moshele has joined #openstack-nova20:15
*** wolverin_ has joined #openstack-nova20:16
*** wolverineav has quit IRC20:16
*** moshele has quit IRC20:17
openstackgerritMatt Riedemann proposed openstack/nova master: Skip ServerShowV254Test.test_rebuild_server in cells v1 job  https://review.openstack.org/56945420:18
artommriedem, https://review.openstack.org/#/c/566344/7/nova/tests/unit/test_metadata.py@24420:21
artomwe need to include it though, no? We're asserting that the metadata will be equal to those dicts we define, so we need to add vf_trusted=False because we expect the code to generate that20:22
mriedemoh is that the thing we're asserting against in the response?20:23
mriedemif so, then sure20:23
mriedemit wasn't obvious20:23
artomYeah, it's giving me a headache too20:24
mriedemwithout reading it yet, is it fair to say the ML thread on review nit picking has itself succumbed to nit picking?20:24
*** shaohe_feng has quit IRC20:24
artomDuh20:25
artomReally accessible topic, easy visibility points20:25
artomEven *I* contributed20:25
*** markvoelker has joined #openstack-nova20:25
*** shaohe_feng has joined #openstack-nova20:27
*** Tom-Tom has joined #openstack-nova20:32
*** shaohe_feng has quit IRC20:35
*** shaohe_feng has joined #openstack-nova20:35
*** Tom-Tom has quit IRC20:36
*** fragatina has joined #openstack-nova20:38
*** s1061123 has quit IRC20:42
*** s1061123 has joined #openstack-nova20:43
*** itlinux has joined #openstack-nova20:44
artommriedem, need to relocate home (via daycare) will try to finish the metadata thing tonight20:45
artomYou east or west coast?20:45
*** shaohe_feng has quit IRC20:45
artom(Locality-wise, not rap)20:45
artomThough I guess we already have your +2, it's jaypipes or whoever else we need to care about20:45
jaypipesartom: hmm?20:46
*** shaohe_feng has joined #openstack-nova20:46
artomjaypipes, oh, apologies, for some reason I was sure mriedem roped you into reviewing https://review.openstack.org/#/q/topic:bp/sriov-trusted-vfs+(status:open+OR+status:merged)20:47
artomHe's +2 except for 1 patch with nits20:47
artomWhich I'm trying to fix20:47
mriedemartom: MN20:47
mriedemno coast20:47
jaypipesartom: ok. can try tomorrow morning (there's a bunch of stuff ahead of that right now)20:47
artomjaypipes, heh, it wasn't a request, but thank you!20:48
mriedemwe can throw it in the runways queue20:48
mriedemsahid should have already done that anyway20:48
mriedemi'll add it20:48
mriedemartom: east coast rap-wise20:48
*** moshele has joined #openstack-nova20:48
jaypipesmriedem: definitely west coast.20:49
artommriedem, ok, you have an hour or so on me in terms of time left in day, you might still be around to +2 it if I succeed in making those damn tests pass20:49
* artom really needs to run now20:49
*** pchavva has quit IRC20:51
*** artom has quit IRC20:53
*** shaohe_feng has quit IRC20:55
*** moshele has quit IRC20:56
*** shaohe_feng has joined #openstack-nova20:56
*** salv-orlando has joined #openstack-nova21:00
*** salv-orlando has quit IRC21:01
*** moshele has joined #openstack-nova21:01
*** salv-orlando has joined #openstack-nova21:01
*** sapd_ has joined #openstack-nova21:03
*** shaohe_feng has quit IRC21:05
*** bpoulos has quit IRC21:06
*** shaohe_feng has joined #openstack-nova21:06
*** sapd has quit IRC21:07
*** tbachman has quit IRC21:12
mriedemdansmith: before Kevin_Zheng goes off and adds some backoff looping call logic here, you might want to make sure we're not missing something on the problem for which he's trying to account https://review.openstack.org/#/c/568542/1/nova/compute/manager.py@622221:12
mriedembasically a race window during abort of a live migration21:13
*** salv-orlando has quit IRC21:13
* dansmith looks21:13
*** markvoelker has quit IRC21:14
*** sree has joined #openstack-nova21:14
*** markvoelker has joined #openstack-nova21:14
*** shaohe_feng has quit IRC21:16
*** moshele has quit IRC21:16
*** shaohe_feng has joined #openstack-nova21:17
*** markvoelker has quit IRC21:18
*** sree has quit IRC21:19
*** shaohe_feng has quit IRC21:26
*** lifeless has quit IRC21:26
mriedemhmm, also, on surya's patch, she already has a release note https://review.openstack.org/#/c/567878/5/releasenotes/notes/migration-tool-to-populate-inst.avz-29fed2fe57a9764d.yaml21:26
mriedemso i'm not sure what to add to that in the backport21:26
*** lifeless_ has joined #openstack-nova21:26
*** shaohe_feng has joined #openstack-nova21:27
dansmithmriedem: right which is why I suggested a new one21:27
dansmithmriedem: commented on that live migration abort thing21:28
*** tidwellr has quit IRC21:28
dansmithI think proper ordering will make that mostly unracy for our purposes without a polling loop21:28
*** tidwellr has joined #openstack-nova21:29
*** tidwellr has quit IRC21:29
*** tidwellr has joined #openstack-nova21:34
mriedemdansmith: alright this is what i had added, but it overlaps pretty heavily http://paste.openstack.org/show/722381/21:35
*** tidwellr has quit IRC21:36
*** shaohe_feng has quit IRC21:36
*** tidwellr has joined #openstack-nova21:36
dansmithmriedem: I just wanted something that indicates that they have to run them again, if you think what is there now is enough, then that's fine21:37
*** shaohe_feng has joined #openstack-nova21:37
dansmithI agree your paste adds nothing to what is there,21:37
dansmithI was just thinking about calling it out specifically21:37
dansmithbut it's cool21:37
dansmitha couple days ago I was working an internal bug where someone updated the code, reported the fix didn't work21:38
dansmithand turns out they didn't even restart nova-compute to get the new code21:38
dansmithso maybe I'm overly sensitive at the moment21:38
*** lifeless has joined #openstack-nova21:38
*** lifeless_ has quit IRC21:38
*** armaan has quit IRC21:39
mriedemalright, i'll remove my extra reno in the backport. i figured that most deployment tools will run nova-manage db sync and the online data migrations during an update as well as a major upgrade, since it shouldn't hurt if there is nothing new, but can hurt if you don't run it and there is21:39
mriedemalso, thanks for the comments on kevin's patch, makes sense when i read it21:40
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Metadata-API fails to retrieve avz for instances created before Pike  https://review.openstack.org/57131721:40
*** tidwellr has quit IRC21:41
dansmithI don't think they do actuall21:41
dansmithfwiw21:41
dansmithat least our process for rolling out updates (not upgrades) is basically just yum21:42
*** esberglu has quit IRC21:43
*** itlinux has quit IRC21:43
cfriesenyeah, we normally wouldn't re-run online data migrations unless it's specifically highlighted.21:43
cfriesenprevious fixes have generally fixed things up on accessing the data, and the online data migrations fixed up ones that hadn't been accessed "live"21:44
*** fragatina has quit IRC21:46
*** shaohe_feng has quit IRC21:46
*** fragatina has joined #openstack-nova21:46
*** shaohe_feng has joined #openstack-nova21:48
mriedemalright, well, meh21:50
mriedemi guess people doing updates can run them if needed,21:50
mriedemotherwise people rolling up to those during FFU can get it21:50
openstackgerritMerged openstack/nova master: Metadata-API fails to retrieve avz for instances created before Pike  https://review.openstack.org/56787821:52
openstackgerritMerged openstack/nova master: Make nova service-list use scatter-gather routine  https://review.openstack.org/56827121:52
openstackgerritMerged openstack/nova master: doc: cleanup API guide about instance faults  https://review.openstack.org/56017821:52
*** harlowja has joined #openstack-nova21:52
melwittinteresting, looking at a recent fail of a VolumesBackupTest in the ceph job, it looks like if you issue a delete request twice for the same volume, the second one can fail with a 400 for having not allowed volume status http://logs.openstack.org/53/570653/1/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/c5d03e7/logs/screen-c-api.txt.gz#_2018-05-30_04_41_07_09521:54
melwittanyone seen that before? is that usual?21:54
*** tidwellr has joined #openstack-nova21:55
*** takashin has joined #openstack-nova21:55
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Metadata-API fails to retrieve avz for instances created before Pike  https://review.openstack.org/57132021:56
melwittsmcginnis ^21:56
*** shaohe_feng has quit IRC21:57
mriedemmelwitt: this? http://logs.openstack.org/53/570653/1/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/c5d03e7/logs/screen-c-api.txt.gz#_2018-05-30_04_41_07_30621:57
*** shaohe_feng has joined #openstack-nova21:58
mriedemtoo bad it didn't tell you what the volume status was when it puked21:58
mriedemmust be 'deleting' or something?21:58
melwittyes. that was a second request to delete the same volume. and yeah, also wish it said, I can only assume it was 'deleting' or the like21:58
mriedemis tempest doing the 2nd delete during a cleanup?21:59
melwittI don't yet know why the teardown of the test is trying to delete an already deleted volume21:59
melwittyeah, it's the cleanup step that fails21:59
mriedemi.e. first delete is explicit in the test, but not waiting for the volume to actually be gone21:59
melwittthat's my guess but haven't confirmed yet21:59
*** tidwellr has quit IRC22:00
mriedembecause most of tempest adds an automatic cleanup if you create the resource in one of the base test methods22:00
mriedemso the test likely just needs to wait for the volume to be gone,22:00
mriedemthen on cleanup, it will handle the 404 and ignore it22:00
melwittah, got it22:00
melwitthm, none of the tests seem to be doing an explicit delete in the test https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_backup.py22:01
*** lifeless has quit IRC22:03
melwittohhh, wait, this is a cinder in-tree tempest test from ocata, so different than what I just linked22:04
*** lifeless has joined #openstack-nova22:04
*** shaohe_feng has quit IRC22:07
openstackgerritMerged openstack/nova master: Expand tests for multiple shared resources case  https://review.openstack.org/56963722:09
*** shaohe_feng has joined #openstack-nova22:10
melwittah, indeed we have redundant cleanups because the test is adding a cleanup and the base class is also adding a cleanup during self.create_volume https://github.com/openstack/cinder/blob/stable/ocata/cinder/tests/tempest/api/volume/test_volume_backup.py#L7722:17
*** shaohe_feng has quit IRC22:17
*** shaohe_feng has joined #openstack-nova22:18
eanderssonIs numa pinning exposed in any api calls?22:18
*** takedakn has joined #openstack-nova22:19
*** mlavalle has quit IRC22:19
openstackgerritJan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif  https://review.openstack.org/57132522:21
openstackgerritJan Gutter proposed openstack/os-vif master: Add multiqueue field to InstanceInfo  https://review.openstack.org/57095922:22
*** takedakn has quit IRC22:22
*** burt has quit IRC22:22
melwitteandersson: do you mean the flavor extra specs that show what numa pinning was requested? it will show that in the server detail GET call and recently will also show it via the flavors API as of this recent bp https://blueprints.launchpad.net/nova/+spec/add-extra-specs-to-flavor-list22:22
*** rcernin has joined #openstack-nova22:22
*** munimeha1 has quit IRC22:23
*** dave-mccowan has quit IRC22:23
eanderssonI think case I actually wanted to see the pinning on the compute itself.22:24
eanderssonFor each VM22:25
*** tbachman has joined #openstack-nova22:25
eanderssonWe wrote an ansible playbook for this, and checked each host using that, but would have been nice to be able to get that kind of data from the api as well.22:26
eandersson(as an admin at least)22:26
melwittI see. I have limited knowledge about it, but maybe cfriesen might know if he's still around today22:27
eanderssonIn this particular case we figured out that most of our hosts had the network interrupt rx queue pinned to a specific core22:27
eandersson(well the ones that didn't have multiqueue enabled anyway)22:27
*** shaohe_feng has quit IRC22:27
*** shaohe_feng has joined #openstack-nova22:28
mriedemeandersson: there isn't anything in the server details that say where an instance is pinned22:29
*** slaweq_ has quit IRC22:30
mriedemstephenfin: dansmith: went over the numa aware vswitch spec again, i think it's pretty close, just missed some cleanups and other small details https://review.openstack.org/#/c/541290/22:30
*** slaweq has joined #openstack-nova22:30
mriedemeandersson: there isn't anything in the os-hypervisors API either22:31
eanderssonfigured as much, but wanted to double check (as the data is available in the db at least)22:31
mriedemNUMA and CPU pinning stuff will eventually be expressed in the placement API22:31
eanderssonNice22:31
mriedemthere used to be an os-pci API added back in juno and removed later in i think pike (or maybe older)22:31
mriedemsince it never did anything22:31
*** salv-orlando has joined #openstack-nova22:32
mriedemeandersson: btw, was it you or someone else from your company that asked about the TypeAffinityFilter being removed during the nova project update session in vancouver?22:32
mriedemor just totally coincidental?22:32
eanderssonOne of my guys asked about it, but heard that someone else also asked about it22:33
*** lifeless has quit IRC22:33
eanderssonas well22:33
mriedemi heard someone named their first born "TypeAffinityFilter" they loved it so much22:34
eanderssonhaha22:34
eanderssonTo be fair the version in nova stinks, especially since it's based on uuid22:34
*** slaweq has quit IRC22:34
mriedemuuid? the one that was in tree was based on the flavor.id primary key22:35
mriedemwhich is pretty much the reason it was removed22:35
eanderssonYea22:35
eanderssonOur version was always based on name, not id22:35
mriedembecause i can create flavorid=foo with id=1, delete it and recreate it with the same flavorid but now it's id=222:35
mriedemand your affinity stuff is all busted22:35
*** lifeless has joined #openstack-nova22:35
mriedemok22:36
mriedemwell, fyi http://lists.openstack.org/pipermail/openstack-operators/2018-May/015277.html22:36
mriedemhttps://review.openstack.org/#/c/569247/ will cause a perf penalty on out of tree filters that need to access any fields on HostState.instances values22:36
mriedemso if people (ops at large) are needing a type affinity filter, one should probably be proposed upstream22:37
*** salv-orlando has quit IRC22:37
*** shaohe_feng has quit IRC22:38
*** shaohe_feng has joined #openstack-nova22:38
*** hongbin has quit IRC22:38
eanderssonbtw unrelated but was a feature to allow rebuild to move the VM to a new compute?22:40
eanderssonor even restart (similar to how Amazon does it)22:40
eanderssonever discussed?22:40
eanderssonWe are trying to find a strategy to make it easier for our customers when we need to bring a compute down for maintenance22:41
mriedemeandersson: that's the evacuate API22:42
*** r-daneel has quit IRC22:42
eanderssonYea - I guess it's the same thing.22:43
mriedemthat's after nova-compute is down though22:43
mriedemhttps://docs.openstack.org/nova/latest/admin/evacuate.html22:43
eanderssonOur ideal scenario would be that if we want to do maintenance on compute X, we mark it as "going down in 7 days", and anyone that hits rebuild on their VM on that host gets moved to a new compute22:44
eanderssonand after 7 days they would just automatically get rebuilt and moved22:44
mriedemi think you could build that on top of existing APIs in an external service22:44
eanderssonYea - that is our general plan of action for now22:45
mriedemyou can force a compute service down so new things can't go to it but the existing VMs continue to run22:45
*** wolverin_ has quit IRC22:45
mriedemhttps://developer.openstack.org/api-ref/compute/#update-forced-down22:45
*** wolverineav has joined #openstack-nova22:45
mriedemso you can force it down to evacuate things off it,22:45
mriedemor disable it so new things can't go to it22:45
eanderssonNice22:45
mriedemrebuild to the same host will work while the service is disabled, it won't auto-migrate it22:46
mriedemnon-admins can only move during things like shelve_offload/unshelve and resize22:46
mriedemyour external thing could mark the compute as disabled/down, and then after 7 days, trigger a cold migration22:47
mriedemor live if you support that22:47
*** nicolasbock has quit IRC22:47
mriedemactually if you disable the compute, shelve all the instances, unshelve them, you get the same thing you're asking for22:48
*** shaohe_feng has quit IRC22:48
mriedemwith some additional glance snapshot traffic, but otherwise the same as rebuild22:48
*** mriedem is now known as mriedem_afk22:48
*** shaohe_feng has joined #openstack-nova22:48
*** tbachman has quit IRC22:49
*** wolverineav has quit IRC22:50
*** tianhui_ has joined #openstack-nova22:50
*** tianhui has quit IRC22:51
*** edmondsw has quit IRC22:52
*** edmondsw has joined #openstack-nova22:53
*** edmondsw has quit IRC22:57
*** shaohe_feng has quit IRC22:58
*** shaohe_feng has joined #openstack-nova22:59
*** tidwellr has joined #openstack-nova23:00
*** tidwellr has quit IRC23:04
*** shaohe_feng has quit IRC23:08
*** dpawlik has joined #openstack-nova23:10
*** shaohe_feng has joined #openstack-nova23:10
*** lifeless has quit IRC23:11
*** dpawlik has quit IRC23:14
*** wolverineav has joined #openstack-nova23:18
*** shaohe_feng has quit IRC23:19
*** shaohe_feng has joined #openstack-nova23:19
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6)  https://review.openstack.org/57133023:20
*** markvoelker has joined #openstack-nova23:25
*** shaohe_feng has quit IRC23:29
*** shaohe_feng has joined #openstack-nova23:30
*** chyka has quit IRC23:31
*** salv-orlando has joined #openstack-nova23:33
*** salv-orlando has quit IRC23:34
*** salv-orlando has joined #openstack-nova23:34
*** shaohe_feng has quit IRC23:39
*** salv-orlando has quit IRC23:40
*** lifeless has joined #openstack-nova23:41
*** shaohe_feng has joined #openstack-nova23:41
*** markvoelker has quit IRC23:43
*** markvoelker has joined #openstack-nova23:44
*** markvoelker has quit IRC23:48
*** shaohe_feng has quit IRC23:49
*** shaohe_feng has joined #openstack-nova23:50
openstackgerritkarim proposed openstack/nova master: Handle rebuild of instances with image traits  https://review.openstack.org/56949823:52
*** ttsiouts has quit IRC23:52
*** tbachman has joined #openstack-nova23:56
*** lifeless has quit IRC23:58
*** eharney has joined #openstack-nova23:58
*** lifeless has joined #openstack-nova23:59
*** artom has joined #openstack-nova23:59
openstackgerritKen'ichi Ohmichi proposed openstack/nova master: Add "must" to automatic names for multiple servers  https://review.openstack.org/57133523:59

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