Tuesday, 2018-07-03

*** shaohe_feng has quit IRC00:00
*** phuongnh has joined #openstack-nova00:01
*** shaohe_feng has joined #openstack-nova00:02
*** threestrands_ has joined #openstack-nova00:07
*** threestrands_ has quit IRC00:07
*** threestrands_ has joined #openstack-nova00:07
*** heyongli has quit IRC00:08
*** heyongli has joined #openstack-nova00:08
*** threestrands_ has quit IRC00:08
*** threestrands_ has joined #openstack-nova00:09
*** threestrands_ has quit IRC00:09
*** threestrands_ has joined #openstack-nova00:09
*** threestrands has quit IRC00:10
*** threestrands_ has quit IRC00:10
*** shaohe_feng has quit IRC00:10
*** threestrands_ has joined #openstack-nova00:10
*** shaohe_feng has joined #openstack-nova00:11
*** threestrands_ has quit IRC00:11
*** threestrands_ has joined #openstack-nova00:12
*** threestrands_ has quit IRC00:13
*** threestrands_ has joined #openstack-nova00:13
*** threestrands_ has quit IRC00:13
*** threestrands_ has joined #openstack-nova00:13
*** threestrands_ has quit IRC00:14
*** threestrands_ has joined #openstack-nova00:15
*** threestrands_ has quit IRC00:15
*** threestrands_ has joined #openstack-nova00:15
*** threestrands_ has quit IRC00:16
*** threestrands_ has joined #openstack-nova00:16
*** heyongli has quit IRC00:18
*** heyongli has joined #openstack-nova00:18
*** shaohe_feng has quit IRC00:20
*** shaohe_feng has joined #openstack-nova00:21
*** med_ has quit IRC00:21
*** antosh has quit IRC00:23
*** edmondsw has joined #openstack-nova00:23
*** edmondsw has quit IRC00:28
*** heyongli has quit IRC00:28
*** heyongli has joined #openstack-nova00:28
*** shaohe_feng has quit IRC00:30
*** shaohe_feng has joined #openstack-nova00:36
*** heyongli has quit IRC00:38
*** aarents has quit IRC00:38
*** heyongli has joined #openstack-nova00:39
*** aarents has joined #openstack-nova00:39
*** shaohe_feng has quit IRC00:41
*** shaohe_feng has joined #openstack-nova00:42
*** moshele has joined #openstack-nova00:45
*** korean101 has quit IRC00:48
*** heyongli has quit IRC00:49
*** heyongli has joined #openstack-nova00:49
*** shaohe_feng has quit IRC00:51
*** shaohe_feng has joined #openstack-nova00:53
*** phuongnh has quit IRC00:53
*** phuongnh has joined #openstack-nova00:54
*** yedongcan has joined #openstack-nova00:54
*** dklyle has quit IRC00:55
*** hongbin has joined #openstack-nova00:57
*** Sundar has quit IRC00:58
*** heyongli has quit IRC00:59
*** heyongli has joined #openstack-nova00:59
*** shaohe_feng has quit IRC01:01
*** shaohe_feng has joined #openstack-nova01:02
*** yingjun has joined #openstack-nova01:03
naichuans_efried: Thank you, will wait until reshape finished :)01:08
*** hiro-kobayashi has joined #openstack-nova01:09
*** heyongli has quit IRC01:09
*** heyongli has joined #openstack-nova01:09
*** shaohe_feng has quit IRC01:11
*** shaohe_feng has joined #openstack-nova01:13
*** masber has joined #openstack-nova01:13
*** mrsoul` has joined #openstack-nova01:14
*** mrsoul_ has joined #openstack-nova01:14
*** mrsoul has quit IRC01:17
*** mschuppert has quit IRC01:17
*** tetsuro has joined #openstack-nova01:18
*** heyongli has quit IRC01:19
*** heyongli has joined #openstack-nova01:20
*** tiendc has joined #openstack-nova01:21
*** shaohe_feng has quit IRC01:22
*** shaohe_feng has joined #openstack-nova01:23
*** heyongli has quit IRC01:30
*** heyongli has joined #openstack-nova01:30
openstackgerritBrin Zhang proposed openstack/nova-specs master: Add support specify volume type when boot instance  https://review.openstack.org/57952001:31
*** shaohe_feng has quit IRC01:32
*** shaohe_feng has joined #openstack-nova01:33
*** antosh has joined #openstack-nova01:39
*** heyongli has quit IRC01:40
*** heyongli has joined #openstack-nova01:40
*** shaohe_feng has quit IRC01:42
*** yangyapeng has quit IRC01:42
*** shaohe_feng has joined #openstack-nova01:43
*** yangyapeng has joined #openstack-nova01:44
gmanndeepak_mourya: i think natsume replied on bug. 'id' is in ignored sort_keys - https://github.com/openstack/nova/blob/9382b7854286bba21235c4bfe39c585a6b894419/nova/api/openstack/compute/schemas/servers.py#L42901:49
*** heyongli has quit IRC01:50
*** heyongli has joined #openstack-nova01:50
*** annp has joined #openstack-nova01:52
*** shaohe_feng has quit IRC01:53
*** shaohe_feng has joined #openstack-nova01:53
*** mylu has quit IRC01:57
*** jichen has joined #openstack-nova01:57
*** heyongli has quit IRC02:00
*** heyongli has joined #openstack-nova02:01
*** Guest53213 has quit IRC02:01
*** shaohe_feng has quit IRC02:03
*** shaohe_feng has joined #openstack-nova02:04
*** heyongli has quit IRC02:11
*** heyongli has joined #openstack-nova02:11
*** shaohe_feng has quit IRC02:13
*** shaohe_feng has joined #openstack-nova02:14
*** heyongli has quit IRC02:21
*** heyongli has joined #openstack-nova02:21
*** mylu has joined #openstack-nova02:21
*** shaohe_feng has quit IRC02:23
*** shaohe_feng has joined #openstack-nova02:24
*** yingjun has quit IRC02:31
*** heyongli has quit IRC02:31
*** heyongli has joined #openstack-nova02:31
*** shaohe_feng has quit IRC02:34
*** shaohe_feng has joined #openstack-nova02:34
*** trungnv has quit IRC02:38
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337502:38
*** trungnv has joined #openstack-nova02:38
*** heyongli has quit IRC02:41
*** heyongli has joined #openstack-nova02:42
*** yingjun has joined #openstack-nova02:43
*** mylu has quit IRC02:43
*** shaohe_feng has quit IRC02:44
*** psachin has joined #openstack-nova02:44
*** shaohe_feng has joined #openstack-nova02:44
*** Nel1x has quit IRC02:48
*** moshele has quit IRC02:49
*** heyongli has quit IRC02:52
*** heyongli has joined #openstack-nova02:52
*** shaohe_feng has quit IRC02:54
*** shaohe_feng has joined #openstack-nova02:55
*** heyongli has quit IRC03:02
*** heyongli has joined #openstack-nova03:02
*** shaohe_feng has quit IRC03:04
*** shaohe_feng has joined #openstack-nova03:05
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340103:06
*** yingjun has quit IRC03:09
*** heyongli has quit IRC03:12
openstackgerritBrin Zhang proposed openstack/nova master: Add fault info when unshelve instance failed  https://review.openstack.org/57974703:12
*** heyongli has joined #openstack-nova03:12
*** shaohe_feng has quit IRC03:15
*** yingjun has joined #openstack-nova03:15
*** shaohe_feng has joined #openstack-nova03:16
*** heyongli has quit IRC03:22
*** heyongli has joined #openstack-nova03:23
*** yingjun has quit IRC03:23
*** shaohe_feng has quit IRC03:25
*** shaohe_feng has joined #openstack-nova03:26
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy  https://review.openstack.org/57116603:31
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy  https://review.openstack.org/57116603:32
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model  https://review.openstack.org/57146503:32
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753403:33
*** heyongli has quit IRC03:33
*** heyongli has joined #openstack-nova03:33
*** shaohe_feng has quit IRC03:35
*** shaohe_feng has joined #openstack-nova03:35
*** udesale has joined #openstack-nova03:42
*** heyongli has quit IRC03:43
*** heyongli has joined #openstack-nova03:43
*** lpetrut has joined #openstack-nova03:45
*** shaohe_feng has quit IRC03:45
*** mdnadeem has joined #openstack-nova03:47
*** shaohe_feng has joined #openstack-nova03:48
*** heyongli has quit IRC03:53
*** heyongli has joined #openstack-nova03:53
*** shaohe_feng has quit IRC03:56
openstackgerritMerged openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667503:56
openstackgerritMerged openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804303:56
openstackgerritMerged openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832603:56
*** shaohe_feng has joined #openstack-nova03:56
*** heyongli has quit IRC04:03
*** heyongli has joined #openstack-nova04:04
*** shaohe_feng has quit IRC04:06
*** shaohe_feng has joined #openstack-nova04:06
*** hongbin has quit IRC04:07
*** heyongli has quit IRC04:14
*** heyongli has joined #openstack-nova04:14
*** shaohe_feng has quit IRC04:16
*** shaohe_feng has joined #openstack-nova04:20
*** heyongli has quit IRC04:24
*** heyongli has joined #openstack-nova04:25
*** threestrands_ has quit IRC04:25
*** shaohe_feng has quit IRC04:26
*** shaohe_feng has joined #openstack-nova04:28
*** antosh has quit IRC04:28
*** lpetrut has quit IRC04:32
*** abhishekk has joined #openstack-nova04:33
*** andreas_s has joined #openstack-nova04:33
*** heyongli has quit IRC04:34
*** heyongli has joined #openstack-nova04:34
*** sridharg has joined #openstack-nova04:36
openstackgerritDeepak Mourya proposed openstack/nova master: Resource_provider API handler does not return specific error codes  https://review.openstack.org/57942304:36
*** shaohe_feng has quit IRC04:37
*** shaohe_feng has joined #openstack-nova04:37
*** andreas_s has quit IRC04:38
*** heyongli has quit IRC04:44
*** heyongli has joined #openstack-nova04:45
*** shaohe_feng has quit IRC04:47
*** shaohe_feng has joined #openstack-nova04:48
*** ratailor has joined #openstack-nova04:50
*** bkopilov has joined #openstack-nova04:51
*** heyongli has quit IRC04:55
*** heyongli has joined #openstack-nova04:55
*** shaohe_feng has quit IRC04:57
*** shaohe_feng has joined #openstack-nova04:59
*** heyongli has quit IRC05:05
*** mdnadeem has quit IRC05:05
*** heyongli has joined #openstack-nova05:05
*** shaohe_feng has quit IRC05:07
*** shaohe_feng has joined #openstack-nova05:10
*** heyongli has quit IRC05:15
*** heyongli has joined #openstack-nova05:15
*** swamireddy has quit IRC05:16
*** mdnadeem has joined #openstack-nova05:17
*** shaohe_feng has quit IRC05:18
*** shaohe_feng has joined #openstack-nova05:19
*** heyongli has quit IRC05:25
*** heyongli has joined #openstack-nova05:26
*** shaohe_feng has quit IRC05:28
*** shaohe_feng has joined #openstack-nova05:29
*** takashin has joined #openstack-nova05:29
*** swamireddy has joined #openstack-nova05:31
*** heyongli has quit IRC05:36
*** heyongli has joined #openstack-nova05:36
*** shaohe_feng has quit IRC05:38
*** shaohe_feng has joined #openstack-nova05:40
*** heyongli has quit IRC05:46
*** heyongli has joined #openstack-nova05:46
*** edmondsw has joined #openstack-nova05:48
*** shaohe_feng has quit IRC05:48
*** quiquell|off is now known as quiquell|rover05:49
*** armaan has quit IRC05:49
*** moshele has joined #openstack-nova05:49
*** armaan has joined #openstack-nova05:49
*** shaohe_feng has joined #openstack-nova05:49
*** janki has joined #openstack-nova05:51
*** edmondsw has quit IRC05:53
*** heyongli has quit IRC05:56
*** heyongli has joined #openstack-nova05:56
*** sahid has joined #openstack-nova05:57
*** shaohe_feng has quit IRC05:59
*** mrsoul_ is now known as mschuppert05:59
*** shaohe_feng has joined #openstack-nova05:59
*** Bhujay has joined #openstack-nova06:00
*** Luzi has joined #openstack-nova06:00
*** andreas_s has joined #openstack-nova06:01
*** heyongli has quit IRC06:06
*** heyongli has joined #openstack-nova06:07
*** shaohe_feng has quit IRC06:09
*** shaohe_feng has joined #openstack-nova06:09
*** lpetrut has joined #openstack-nova06:10
*** heyongli has quit IRC06:17
*** heyongli has joined #openstack-nova06:17
*** mdnadeem_ has joined #openstack-nova06:18
*** mdnadeem has quit IRC06:19
*** shaohe_feng has quit IRC06:19
*** shaohe_feng has joined #openstack-nova06:22
*** phuongnh has quit IRC06:23
*** heyongli has quit IRC06:27
*** heyongli has joined #openstack-nova06:27
*** shaohe_feng has quit IRC06:29
*** shaohe_feng has joined #openstack-nova06:30
*** heyongli has quit IRC06:37
*** heyongli has joined #openstack-nova06:37
*** shaohe_feng has quit IRC06:40
*** shaohe_feng has joined #openstack-nova06:40
*** nicolasbock has joined #openstack-nova06:42
*** dpawlik has joined #openstack-nova06:42
*** dpawlik has quit IRC06:43
*** dpawlik has joined #openstack-nova06:43
*** gongysh has joined #openstack-nova06:43
*** heyongli has quit IRC06:47
*** heyongli has joined #openstack-nova06:48
*** openstackgerrit has quit IRC06:49
*** andreas_s has quit IRC06:49
*** andreas_s has joined #openstack-nova06:49
*** shaohe_feng has quit IRC06:50
*** shaohe_feng has joined #openstack-nova06:50
*** andreas_s has quit IRC06:54
*** openstackgerrit has joined #openstack-nova06:54
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/57801906:54
*** rcernin has quit IRC06:55
*** heyongli has quit IRC06:58
*** heyongli has joined #openstack-nova06:58
*** quiquell|rover is now known as quique|rover|bbl06:59
*** shaohe_feng has quit IRC07:00
*** shaohe_feng has joined #openstack-nova07:01
*** mdnadeem has joined #openstack-nova07:02
*** mdnadeem_ has quit IRC07:03
*** tesseract has joined #openstack-nova07:06
*** mdnadeem has quit IRC07:07
*** heyongli has quit IRC07:08
*** heyongli has joined #openstack-nova07:08
*** damien_r has joined #openstack-nova07:09
*** shaohe_feng has quit IRC07:10
*** shaohe_feng has joined #openstack-nova07:12
*** ispp has joined #openstack-nova07:15
*** josecastroleon has joined #openstack-nova07:18
*** heyongli has quit IRC07:18
*** heyongli has joined #openstack-nova07:19
*** amoralej|off is now known as amoralej07:19
*** shaohe_feng has quit IRC07:21
*** peereb has joined #openstack-nova07:21
*** shaohe_feng has joined #openstack-nova07:23
*** alexchadin has joined #openstack-nova07:24
*** ispp has quit IRC07:27
*** heyongli has quit IRC07:28
*** heyongli has joined #openstack-nova07:29
*** jangutter_ has joined #openstack-nova07:29
*** jangutter has quit IRC07:29
*** andreas_s has joined #openstack-nova07:29
*** shaohe_feng has quit IRC07:31
*** andreas_s_ has joined #openstack-nova07:31
*** tssurya has joined #openstack-nova07:31
*** lpetrut has quit IRC07:31
openstackgerritTuan Do Anh proposed openstack/nova-specs master: fix tox python3 overrides  https://review.openstack.org/57979307:31
*** shaohe_feng has joined #openstack-nova07:32
*** andreas__ has joined #openstack-nova07:33
*** quique|rover|bbl is now known as quiquell|rover07:34
*** andreas_s has quit IRC07:34
*** andreas_s_ has quit IRC07:35
*** edmondsw has joined #openstack-nova07:36
*** heyongli has quit IRC07:39
*** heyongli has joined #openstack-nova07:39
*** BrinZhang has joined #openstack-nova07:39
*** oanson has quit IRC07:39
*** BrinZhang has quit IRC07:40
*** BrinZhang has joined #openstack-nova07:40
*** oanson has joined #openstack-nova07:41
*** edmondsw has quit IRC07:41
*** shaohe_feng has quit IRC07:41
*** BrinZhang has quit IRC07:41
*** shaohe_feng has joined #openstack-nova07:41
*** BrinZhang has joined #openstack-nova07:42
*** ispp has joined #openstack-nova07:42
*** ragiman has joined #openstack-nova07:43
*** lpetrut has joined #openstack-nova07:44
openstackgerritLei Zhang proposed openstack/nova master: Add method to get cpu traits  https://review.openstack.org/56031707:47
*** BrinZhang has quit IRC07:48
*** BrinZhang has joined #openstack-nova07:49
*** heyongli has quit IRC07:49
*** heyongli has joined #openstack-nova07:49
*** shaohe_feng has quit IRC07:51
*** avolkov has joined #openstack-nova07:51
*** shaohe_feng has joined #openstack-nova07:52
*** jpena|off is now known as jpena07:52
openstackgerritClaudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages  https://review.openstack.org/47077507:56
openstackgerritClaudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated  https://review.openstack.org/34221107:56
*** heyongli has quit IRC07:59
*** heyongli has joined #openstack-nova07:59
*** armaan has quit IRC08:00
*** armaan has joined #openstack-nova08:00
openstackgerritClaudiu Belu proposed openstack/nova master: WIP: replace spec with autospec  https://review.openstack.org/55729908:01
*** shaohe_feng has quit IRC08:02
*** rpittau has quit IRC08:02
*** shaohe_feng has joined #openstack-nova08:02
*** armaan has quit IRC08:07
*** armaan has joined #openstack-nova08:07
openstackgerritBalazs Gibizer proposed openstack/nova master: Prevent updating an RP's parent to form a loop  https://review.openstack.org/57961808:08
*** ccamacho has joined #openstack-nova08:08
*** heyongli has quit IRC08:09
*** heyongli has joined #openstack-nova08:10
*** rpittau has joined #openstack-nova08:10
*** shaohe_feng has quit IRC08:12
*** shaohe_feng has joined #openstack-nova08:13
*** cz2 has quit IRC08:14
*** vivsoni has quit IRC08:14
*** cz2 has joined #openstack-nova08:16
*** heyongli has quit IRC08:20
*** heyongli has joined #openstack-nova08:20
*** armaan has quit IRC08:21
*** yassine_ has quit IRC08:21
*** armaan has joined #openstack-nova08:22
*** shaohe_feng has quit IRC08:22
*** hyunsikyang has joined #openstack-nova08:23
*** shaohe_feng has joined #openstack-nova08:23
*** armaan has quit IRC08:25
*** armaan has joined #openstack-nova08:26
*** mvk has quit IRC08:28
*** heyongli has quit IRC08:30
*** ralonsoh has joined #openstack-nova08:30
*** heyongli has joined #openstack-nova08:30
*** yassine_ has joined #openstack-nova08:31
openstackgerritMerged openstack/nova master: Merge server create schema for keypair extension  https://review.openstack.org/57833208:31
*** shaohe_feng has quit IRC08:32
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object  https://review.openstack.org/57362808:33
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Refactor the policies to policy  https://review.openstack.org/57911308:33
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337508:33
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340108:33
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy  https://review.openstack.org/57116608:33
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model  https://review.openstack.org/57146508:33
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753408:34
*** shaohe_feng has joined #openstack-nova08:34
*** vivsoni has joined #openstack-nova08:34
*** armaan has quit IRC08:35
*** priteau has joined #openstack-nova08:36
*** armaan has joined #openstack-nova08:36
*** armaan has quit IRC08:37
*** armaan has joined #openstack-nova08:38
*** heyongli has quit IRC08:40
*** trungnv has quit IRC08:40
*** heyongli has joined #openstack-nova08:41
*** shaohe_feng has quit IRC08:43
*** Bhujay has quit IRC08:44
*** shaohe_feng has joined #openstack-nova08:45
*** Cardoe has quit IRC08:49
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform missing delete notifications  https://review.openstack.org/41029708:50
*** heyongli has quit IRC08:50
*** heyongli has joined #openstack-nova08:51
*** shaohe_feng has quit IRC08:53
*** shaohe_feng has joined #openstack-nova08:53
*** mvk has joined #openstack-nova08:55
*** Cardoe has joined #openstack-nova08:55
*** cdent has joined #openstack-nova08:56
*** armaan has quit IRC08:57
*** armaan has joined #openstack-nova08:57
*** derekh has joined #openstack-nova08:58
*** heyongli has quit IRC09:01
*** heyongli has joined #openstack-nova09:01
*** shaohe_feng has quit IRC09:03
*** tianhui_ is now known as tianhui09:03
*** gongysh has quit IRC09:03
*** shaohe_feng has joined #openstack-nova09:04
*** cdent_ has joined #openstack-nova09:04
*** cdent has quit IRC09:05
*** cdent_ is now known as cdent09:05
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform instance-live_migration_post notification  https://review.openstack.org/48011909:06
*** mdnadeem has joined #openstack-nova09:07
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the instance payload  https://review.openstack.org/53624309:10
*** heyongli has quit IRC09:11
*** heyongli has joined #openstack-nova09:11
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix unbound local when saving an unchanged RequestSpec  https://review.openstack.org/57376809:12
*** shaohe_feng has quit IRC09:13
*** alexchadin has quit IRC09:13
*** shaohe_feng has joined #openstack-nova09:14
*** alexchadin has joined #openstack-nova09:16
*** vivsoni has quit IRC09:17
*** heyongli has quit IRC09:21
*** heyongli has joined #openstack-nova09:21
*** shaohe_feng has quit IRC09:24
*** ratailor has quit IRC09:24
*** shaohe_feng has joined #openstack-nova09:24
*** edmondsw has joined #openstack-nova09:25
*** MultipleCrashes has joined #openstack-nova09:25
openstackgerritSilvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver  https://review.openstack.org/55419509:25
*** ratailor has joined #openstack-nova09:25
openstackgerrityuqian proposed openstack/nova master: vmware does't support VirtualSriovEthernetCard  https://review.openstack.org/57982809:26
*** edmondsw has quit IRC09:29
*** Bhujay has joined #openstack-nova09:30
*** takashin has left #openstack-nova09:30
lyarwoodkashyap / stephenfin ; https://review.openstack.org/#/c/579125/ - would you mind taking a swing at this today?09:31
*** heyongli has quit IRC09:31
* kashyap waves09:32
*** heyongli has joined #openstack-nova09:32
kashyaplyarwood: Oh, this one.  Yes, I saw your change.  Thanks for addressing09:32
*** sahid has quit IRC09:32
*** aloga has joined #openstack-nova09:32
*** sahid has joined #openstack-nova09:33
*** sambetts|afk is now known as sambetts09:33
*** tojuvone has quit IRC09:33
*** tojuvone has joined #openstack-nova09:33
*** shaohe_feng has quit IRC09:34
*** shaohe_feng has joined #openstack-nova09:35
*** hiro-kobayashi has quit IRC09:37
*** lpetrut has quit IRC09:38
*** alexchadin has quit IRC09:39
*** heyongli has quit IRC09:42
*** heyongli has joined #openstack-nova09:42
*** shaohe_feng has quit IRC09:44
*** lpetrut has joined #openstack-nova09:44
*** jangutter has joined #openstack-nova09:45
openstackgerritBrin Zhang proposed openstack/nova master: Add unshelve instance error info to fault table  https://review.openstack.org/57974709:45
*** jangutter has quit IRC09:45
*** shaohe_feng has joined #openstack-nova09:45
*** jangutter_ has quit IRC09:46
*** jangutter has joined #openstack-nova09:46
*** aloga has quit IRC09:46
*** tetsuro has quit IRC09:49
*** trungnv has joined #openstack-nova09:50
*** MultipleCrashes has quit IRC09:50
*** heyongli has quit IRC09:52
*** heyongli has joined #openstack-nova09:52
*** shaohe_feng has quit IRC09:54
*** nicolasbock has quit IRC09:55
*** rmart04 has joined #openstack-nova09:55
*** shaohe_feng has joined #openstack-nova09:55
*** gongysh has joined #openstack-nova09:56
*** ratailor has quit IRC09:57
*** yassine_ has quit IRC09:58
*** ratailor has joined #openstack-nova09:58
*** heyongli has quit IRC10:02
*** heyongli has joined #openstack-nova10:03
*** shaohe_feng has quit IRC10:05
*** tssurya has quit IRC10:06
*** shaohe_feng has joined #openstack-nova10:07
*** alexchadin has joined #openstack-nova10:11
*** heyongli has quit IRC10:12
*** heyongli has joined #openstack-nova10:13
*** dtantsur|afk is now known as dtantsur10:14
*** shaohe_feng has quit IRC10:15
*** shaohe_feng has joined #openstack-nova10:15
*** jichen has quit IRC10:16
pooja_jadhavefried, jaypipes: Actually I want to know the scenario when this method will get called? Ref: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py#L391410:22
*** heyongli has quit IRC10:23
*** heyongli has joined #openstack-nova10:23
*** cdent has quit IRC10:25
*** owalsh_pto has quit IRC10:25
*** shaohe_feng has quit IRC10:25
*** shaohe_feng has joined #openstack-nova10:26
*** abhishekk has quit IRC10:32
*** heyongli has quit IRC10:33
*** heyongli has joined #openstack-nova10:33
*** shaohe_feng has quit IRC10:35
*** mvk has quit IRC10:35
*** shaohe_feng has joined #openstack-nova10:36
*** heyongli has quit IRC10:43
*** heyongli has joined #openstack-nova10:43
*** shaohe_feng has quit IRC10:46
*** heyongli has quit IRC10:53
hyunsikyangHello all11:01
hyunsikyangI am Hyunsik Yang at Soongsil Unviersity, South Korea.11:01
hyunsikyangIn the OPNFV project, especially, infra WG, i am trying to provide vBaremetal environment for testing the OPNFV platform.11:01
hyunsikyangFor that, IPMI and PXE should be supported well like as a physical server.11:01
hyunsikyangHowever, in the current OpenStack environment, NOVA manages the lifecycle of the VM, so IPMI operation based on KVM does not work normally.11:01
hyunsikyangTo support this, NOVA should support IPMI and PXE. Therefore, this proposal intends to add a function for normal control ofIPMI and PXE in NOVA.11:01
hyunsikyangSo, i uploaded spec first. If anyone who interested in this proposal, Please review it.11:01
hyunsikyangThanks.11:01
hyunsikyangThe link is below.11:01
hyunsikyanghttps://review.openstack.org/#/c/577048/11:01
lyarwoodhyunsikyang: https://docs.openstack.org/tripleo-docs/latest/install/environments/virtualbmc.html - you might want to look at how TripleO already does this with the vbmc project11:03
lyarwoodhyunsikyang: https://github.com/openstack/virtualbmc11:04
*** Bhujay has quit IRC11:08
hyunsikyangI knew it. vBMC which is based on KVM is now working for openstack instance. so, we also make a code for it too.11:08
*** heyongli has joined #openstack-nova11:12
*** shaohe_feng has joined #openstack-nova11:12
*** claudiub has joined #openstack-nova11:13
*** heyongli has quit IRC11:14
*** heyongli has joined #openstack-nova11:14
*** BrinZhang has quit IRC11:15
hyunsikyanglyarwood: I knew it. vBMC which is based on KVM is now working for openstack instance. so, we also make a code for it too.11:16
*** shaohe_feng has quit IRC11:16
*** shaohe_feng has joined #openstack-nova11:18
openstackgerritBalazs Gibizer proposed openstack/nova master: Escalate UUID validation warning to error in test  https://review.openstack.org/54038611:21
lyarwoodhyunsikyang: I don't follow, doesn't Ironic provide what you need?11:22
*** gongysh has quit IRC11:24
*** heyongli has quit IRC11:24
*** heyongli has joined #openstack-nova11:24
*** yedongcan has left #openstack-nova11:26
hyunsikyanglyarwood: YES. Ironic used VM Which is based on KVM.11:26
*** shaohe_feng has quit IRC11:27
hyunsikyanglyarwood: I want to use openstack instance as a baremetal.11:27
lyarwoodhyunsikyang: then just use vbmc to provide the ipmi layer to Ironic?11:27
lyarwoodhyunsikyang: nova is never going to directly support ipmi or pxe11:27
*** tiendc has quit IRC11:28
*** shaohe_feng has joined #openstack-nova11:28
*** jpena is now known as jpena|lunch11:29
*** pchavva has joined #openstack-nova11:30
hyunsikyanglyarwood: i can't understand what is the meaning of 'then just use vbmc to provide the ipmi layer to Ironic?'11:31
*** Bhujay has joined #openstack-nova11:32
hyunsikyanglyarwood: current nova doesn't support ipmi pxe for openstack instance.11:32
lyarwoodhyunsikyang: right, Ironic however does, so the stack is QEMU/KVM instance <-> Nova <-> VBMC <-> Ironic11:33
lyarwoodhyunsikyang: VBMC just provides an IPMI layer around the traditional Nova virtual instances that Ironic can use.11:34
*** heyongli has quit IRC11:34
*** heyongli has joined #openstack-nova11:35
*** udesale has quit IRC11:36
*** shaohe_feng has quit IRC11:37
*** shaohe_feng has joined #openstack-nova11:38
hyunsikyanglyarwood:  vBMC provides IPMI for VM which is based on KVM.  you mean that when user send the command to the vBMC, vBMC will forward this command to the openstack instance? i don't think so. we can control VM which is based on KVM. but we can't controll openstack instance.11:41
hyunsikyanglyarwood:  I checked code and tested it. but, it didn't work.11:42
*** yingjun has joined #openstack-nova11:42
*** derekh has quit IRC11:43
*** heyongli has quit IRC11:45
*** heyongli has joined #openstack-nova11:45
*** yassine has joined #openstack-nova11:45
*** mvk has joined #openstack-nova11:45
*** yassine is now known as Guest6659311:46
*** amoralej is now known as amoralej|lunch11:46
*** sridharg has quit IRC11:46
*** shaohe_feng has quit IRC11:47
*** sridharg has joined #openstack-nova11:48
*** shaohe_feng has joined #openstack-nova11:48
lyarwoodhyunsikyang: ah wait, my apologies I mixed up vbmc and ovb - https://openstack-virtual-baremetal.readthedocs.io/en/latest/introduction.html11:49
lyarwoodhyunsikyang: vbmc talks to libvirt directly11:49
*** owalsh has joined #openstack-nova11:49
*** tssurya has joined #openstack-nova11:49
*** armaan has quit IRC11:50
hyunsikyangYes  I knew it. too:)11:50
*** armaan has joined #openstack-nova11:50
openstackgerritBalazs Gibizer proposed openstack/nova master: Call generate_image_url only for legacy notification  https://review.openstack.org/56452811:50
hyunsikyangbut that code also need a modification of nova. nova should support Pxe / IPMI for instance. so i am working on it.11:51
openstackgerritBalazs Gibizer proposed openstack/nova master: Call generate_image_url only for legacy notification  https://review.openstack.org/56452811:51
hyunsikyangi uploaded two part. one is nova another is vBMC.11:52
*** yingjun has quit IRC11:52
hyunsikyanglyarwood: but that code also need a modification of nova. nova should support Pxe / IPMI for instance. so i am working on it. i uploaded two part. one is nova another is vBMC.11:52
hyunsikyanglyarwood: https://review.openstack.org/#/c/570737/ this is for vBMC11:53
*** heyongli has quit IRC11:55
*** quiquell|rover is now known as quique|rover|lch11:55
*** armaan has quit IRC11:55
*** heyongli has joined #openstack-nova11:55
*** ratailor has quit IRC11:56
*** mchlumsky has joined #openstack-nova11:57
*** shaohe_feng has quit IRC11:57
*** shaohe_feng has joined #openstack-nova11:59
*** voelzmo has joined #openstack-nova11:59
*** heyongli has quit IRC12:05
*** heyongli has joined #openstack-nova12:05
*** janki has quit IRC12:06
*** vladikr has joined #openstack-nova12:07
*** lyan has joined #openstack-nova12:07
*** lyan is now known as Guest3616512:07
*** shaohe_feng has quit IRC12:08
*** shaohe_feng has joined #openstack-nova12:09
*** moshele has quit IRC12:09
*** mchlumsky has quit IRC12:10
*** med_ has joined #openstack-nova12:10
*** med_ has quit IRC12:10
*** med_ has joined #openstack-nova12:10
*** moshele has joined #openstack-nova12:10
*** mchlumsky has joined #openstack-nova12:11
*** Guest36165 has quit IRC12:14
*** alexchadin has quit IRC12:15
*** heyongli has quit IRC12:15
*** heyongli has joined #openstack-nova12:16
*** shaohe_feng has quit IRC12:18
openstackgerritBalazs Gibizer proposed openstack/nova master: Enhance service restart in functional env  https://review.openstack.org/51255212:19
openstackgerritBalazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env  https://review.openstack.org/51255312:19
openstackbug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer)12:19
openstackgerritBalazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor  https://review.openstack.org/51262312:19
*** shaohe_feng has joined #openstack-nova12:20
*** heyongli has quit IRC12:26
*** heyongli has joined #openstack-nova12:26
*** shaohe_feng has quit IRC12:28
*** kaisers has quit IRC12:28
*** jpena|lunch is now known as jpena12:28
*** shaohe_feng has joined #openstack-nova12:32
*** mdnadeem has quit IRC12:33
*** kaisers has joined #openstack-nova12:34
*** heyongli has quit IRC12:36
*** alexchadin has joined #openstack-nova12:36
*** heyongli has joined #openstack-nova12:36
*** yamamoto_ has quit IRC12:37
*** lyan_ has joined #openstack-nova12:37
*** shaohe_feng has quit IRC12:38
*** shaohe_feng has joined #openstack-nova12:40
*** armaan has joined #openstack-nova12:45
*** derekh has joined #openstack-nova12:45
*** d063130_ has quit IRC12:45
*** sapcc-bot has quit IRC12:45
*** d063130_ has joined #openstack-nova12:46
*** sapcc-bot has joined #openstack-nova12:46
*** heyongli has quit IRC12:46
*** heyongli has joined #openstack-nova12:47
*** shaohe_feng has quit IRC12:49
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the instance payload  https://review.openstack.org/53624312:49
*** shaohe_feng has joined #openstack-nova12:50
*** armaan has quit IRC12:50
*** armaan has joined #openstack-nova12:50
*** yangyapeng has quit IRC12:51
*** mriedem has joined #openstack-nova12:51
*** yamamoto has joined #openstack-nova12:53
openstackgerritMerged openstack/nova master: Handle nested serialized json entries in assertJsonEqual  https://review.openstack.org/57961512:54
*** armaan has quit IRC12:55
*** heyongli has quit IRC12:56
*** heyongli has joined #openstack-nova12:57
*** quique|rover|lch is now known as quiquell|rover12:57
*** yamamoto has quit IRC12:58
*** shaohe_feng has quit IRC12:59
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Use ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa in tree  https://review.openstack.org/57986213:01
*** shaohe_feng has joined #openstack-nova13:01
*** edmondsw has joined #openstack-nova13:01
*** ispp has quit IRC13:02
*** artom has quit IRC13:02
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix unbound local when saving an unchanged RequestSpec  https://review.openstack.org/57376813:02
*** andreas__ has quit IRC13:03
bauzasmriedem: so, do we have a gate issue ?13:03
*** janki has joined #openstack-nova13:03
*** andreas_s has joined #openstack-nova13:03
bauzasmriedem: I saw your nvm13:03
mriedemyou can ignore, it wasn't specific to rescue13:05
mriedemjust libvirt randomly crapping out13:05
*** ispp has joined #openstack-nova13:06
*** edmondsw has quit IRC13:06
*** heyongli has quit IRC13:07
*** heyongli has joined #openstack-nova13:07
*** yamamoto has joined #openstack-nova13:08
*** shaohe_feng has quit IRC13:09
*** mriedem is now known as mriedem_afk13:09
*** lyan_ has quit IRC13:10
*** yamamoto has quit IRC13:13
*** shaohe_feng has joined #openstack-nova13:13
*** heyongli has quit IRC13:17
*** heyongli has joined #openstack-nova13:17
openstackgerritBalazs Gibizer proposed openstack/nova master: cover migration cases with functional tests  https://review.openstack.org/49386513:18
*** lyan has joined #openstack-nova13:19
*** lyan is now known as Guest4109413:19
*** mdbooth_ has joined #openstack-nova13:19
*** shaohe_feng has quit IRC13:19
*** shaohe_feng has joined #openstack-nova13:20
*** tssurya has quit IRC13:21
*** andreas_s has quit IRC13:22
*** mdbooth has quit IRC13:22
*** yamamoto has joined #openstack-nova13:23
*** amoralej|lunch is now known as amoralej13:25
*** hongbin has joined #openstack-nova13:25
*** andreas_s has joined #openstack-nova13:26
efriedpooja_jadhav: I can explain the circumstances under which https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py#L3914 gets called, but it would help me to know why you're interested in that specific code path.13:26
*** janki has quit IRC13:26
*** heyongli has quit IRC13:27
*** heyongli has joined #openstack-nova13:27
*** yamamoto has quit IRC13:28
*** shaohe_feng has quit IRC13:30
*** shaohe_feng has joined #openstack-nova13:30
pooja_jadhavefried: I am eager to learn what DB query is generated when we call method https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py#L2691.13:31
*** abhishekk has joined #openstack-nova13:31
pooja_jadhavefied: so want to know under which scenarios this method will get called?13:31
*** jogo has quit IRC13:33
*** andreas_s has quit IRC13:36
*** Bhujay has quit IRC13:37
openstackgerritMargarita Shakhova proposed openstack/nova master: Do not skip migrations in _destroy_evacuated_instances()  https://review.openstack.org/56362313:37
*** heyongli has quit IRC13:37
*** heyongli has joined #openstack-nova13:38
*** shaohe_feng has quit IRC13:40
*** shaohe_feng has joined #openstack-nova13:41
*** yamamoto has joined #openstack-nova13:41
*** yamamoto has quit IRC13:41
efriedpooja_jadhav: Okay.  If you create a query with a granular request group, you should hit it.13:42
efriedpooja_jadhav: Take a look at nova/tests/functional/api/openstack/placement/gabbits/granular.yaml13:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Send soft_delete from context manager  https://review.openstack.org/47645913:43
efriedpooja_jadhav: Anywhere you see `resources<Number>` is a query that should hit that code path.13:43
*** mriedem_afk is now known as mriedem13:44
*** andreas_s has joined #openstack-nova13:44
pooja_jadhavefried: ok, will check and if any doubts will let u know. thanks for your time :)13:44
efriedGood luck pooja_jadhav13:45
jrollefried: does this ring a bell? sambetts is hitting it in third party CI: http://192.133.158.227/12/514312/12/experimental/ironic-dsvm-tempest-ironic-cimc-current-centos-7-cimc/f7c455a/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jul_03_12_53_48_63870413:45
jrolland/or do you have tips on debugging that? or do we need to add more logging13:45
*** burt has joined #openstack-nova13:46
efriedjroll: Is it causing a problem?13:46
pooja_jadhavefried: :)13:46
efriedjroll: It's actually a semi-expected thing, supposed to self-heal on the next periodic hit.13:46
*** janki has joined #openstack-nova13:46
jrollefried: yes, causes a reschedule (and maybe a failure due to lack of hardware?)13:46
jrollyes, failure13:46
jrollhttp://192.133.158.227/12/514312/12/experimental/ironic-dsvm-tempest-ironic-cimc-current-centos-7-cimc/f7c455a/logs/screen-n-cond-cell1.txt.gz#_Jul_03_12_53_47_51758813:46
*** awaugama has joined #openstack-nova13:46
sambettsyeah, we only have one node, so the retry filter kills the instance13:47
jrollnot sure if it's a constant failure or occassional, sam can clarify13:47
*** andreas_s has quit IRC13:47
sambettsall my CI failures seem to be this at the moment13:47
efriedThat's weird that it would be affecting a sched operation.  At least the one you're showing me ought to be part of periodic, not part of spawn.13:47
*** heyongli has quit IRC13:48
efriedbut in any case, yeah, we have to close some of these gaps in rocky13:48
*** heyongli has joined #openstack-nova13:48
efriedIs this rocky?13:48
jrolloh, that is a periodic, odd13:48
efriedWhat we need to do is put a retry loop around this chunk in the resource tracker: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L883-L91413:49
*** mlavalle has joined #openstack-nova13:49
efriedThe devil is in the details, of course.13:50
*** shaohe_feng has quit IRC13:50
*** shaohe_feng has joined #openstack-nova13:50
*** jmlowe has quit IRC13:51
sambettslooking at the n-cond log that placement sync error is bubbling up in the _build_and_run_instance function13:51
*** andreas_s has joined #openstack-nova13:51
sambettsalthough its been wrapped in a ReschduledException :/13:52
sambettssomwhere along the way13:52
openstackgerritMatt Riedemann proposed openstack/nova master: Wait for vif plugging during live migration job  https://review.openstack.org/57855113:52
*** andreas_s has quit IRC13:54
*** andreas_s has joined #openstack-nova13:55
*** moshele has quit IRC13:55
openstackgerritMatt Riedemann proposed openstack/nova master: Stop setting glance_api_version in cinder.conf in nova-live-migration  https://review.openstack.org/57987113:56
efriedsambetts: Are you sure it's the same ResourceProviderSyncFailed exception?  (Though that's the only one I'm seeing in your compute log.)13:57
*** namnh has joined #openstack-nova13:58
*** heyongli has quit IRC13:58
*** heyongli has joined #openstack-nova13:58
sambettsefried: what interesting is that the node UUID in the exeception in n-cpu isn't the node UUID in the n-cond logs13:59
*** andreas_s has quit IRC13:59
sambettsso I don't know if this is being caused by other nodes async in the background being created and deleted by the API tests13:59
sambettsjroll: ^14:00
*** psachin has quit IRC14:00
efriedsambetts: It isn't?  It looks the same to me14:00
*** shaohe_feng has quit IRC14:00
sambettsoh acually may bad I had two different cases of this happening up *face plam*14:00
jrollyeah, it is the same :)14:01
efriedn-cond: node b5bb2c51-3960-4d9c-b47e-223d83cb613914:01
efriedn-cpu: Error updating resources for node b5bb2c51-3960-4d9c-b47e-223d83cb613914:01
*** andreas_s has joined #openstack-nova14:01
*** andreas_s has quit IRC14:01
efriedokay phew14:01
sambettsI had two jobs for two different ironic drivers and I'm seeing the same error in both so I have like 10 tabs open with all the different logs XD14:01
efriedStill mysterious why a failed periodic in n-cpu would cause a spawn to fail.  Unless it's not periodic.14:01
efriedor unless sambetts is looking at two different sets of logs :P14:01
*** andreas_s has joined #openstack-nova14:02
*** andreas_s has quit IRC14:02
efriedBut yeah, I see the resched failure and I'm sure I'm only looking at one :)14:02
sambettsyeah, its definately the same, double checked14:02
*** shaohe_feng has joined #openstack-nova14:02
*** andreas_s has joined #openstack-nova14:03
jrollhrm, it's the same request id in the reschedule log as the request id in the periodics logs14:04
*** mriedem has quit IRC14:04
*** abhishekk has quit IRC14:04
*** mriedem has joined #openstack-nova14:04
*** lpetrut has quit IRC14:05
sambettsthe trace isn't quite deep enough in the n-cpu logs to know what called _update_available_resource_for_node14:05
*** tidwellr has joined #openstack-nova14:06
sambettsonly place I can see it being called is here: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L747914:06
jrollsambetts: it's only from the periodic task: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L750014:06
jrollodd14:07
*** yingjun has joined #openstack-nova14:08
efriedI'm pretty sure there's a code path that hits that guy during spawn.14:08
*** heyongli has quit IRC14:08
*** tssurya has joined #openstack-nova14:08
*** heyongli has joined #openstack-nova14:09
efriedI don't remember where in the stack it happens.  We're, what, five layers deep in methods called update-something?14:09
efriedbut I would have expected to see it in this stack trace.14:09
jrollefried: calls which guy, specifically?14:09
*** jogo has joined #openstack-nova14:09
efriedthat's what I don't remember.14:10
*** yangyapeng has joined #openstack-nova14:10
efriedjroll: I'm saying I think there's a code path during spawn that winds up here in _update, but I don't remember where in the call stack it's initiated.14:10
efriedthat didn't make sense, gimme a sec to find what I'm talking about.14:10
jrollhm, ok14:10
*** shaohe_feng has quit IRC14:11
jrollthe compute traceback is pretty clearly from the periodic task14:11
jrollper http://192.133.158.227/12/514312/12/experimental/ironic-dsvm-tempest-ironic-cimc-current-centos-7-cimc/f7c455a/logs/screen-n-cpu.txt.gz?level=DEBUG#_Jul_03_12_53_48_62494414:11
jroller wait, that's later14:11
jrollignore me14:11
jroll(also, I see what you're saying)14:12
*** shaohe_feng has joined #openstack-nova14:12
*** Guest41094 has quit IRC14:12
sambettsits a same that because a new exepcetion is raised in the n-cond code the line that through the original exeception is lost14:13
sambettss/thought/threw14:14
*** ircuser-1 has joined #openstack-nova14:14
*** yangyapeng has quit IRC14:15
*** tidwellr has quit IRC14:15
*** tidwellr has joined #openstack-nova14:16
efriedI am now unable to find the code path that calls this from not-a-periodic.14:18
jrollinstance_claim() calls into _update()14:18
*** heyongli has quit IRC14:18
*** jmlowe has joined #openstack-nova14:18
*** heyongli has joined #openstack-nova14:19
jrollwhich is in _build_and_run_instance14:19
jrollso that's probably where the build is bombing14:19
jrolland the n-cpu traceback is an unrelated red herring14:19
sambettssame exception on both sides though :/14:20
efriedjroll: Then why is that exception message in the resched... yeah14:20
jrollyes14:20
*** jibby has joined #openstack-nova14:20
jrollsame exception, different call stack14:20
sambettsmaybe the reason they are both raising that execetion is because of a collision?14:21
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform instance-live_migration_post notification  https://review.openstack.org/48011914:21
jrollnote that the conductor exception has _do_build_and_run_instance as the source14:21
*** shaohe_feng has quit IRC14:21
*** tidwellr has quit IRC14:21
*** andreas_s has quit IRC14:21
jroll_do_build_and_run_instance calls _build_and_run_instance calls rt.instance_claim() calls rt._update() calls.... down to where that exception happens14:21
*** lpetrut has joined #openstack-nova14:23
*** shaohe_feng has joined #openstack-nova14:24
*** alexchadin has quit IRC14:25
dansmithmriedem: does this mean you don't need to have the hangout this morning? https://review.openstack.org/#/c/564445/11/nova/network/neutronv2/api.py14:25
*** lyan has joined #openstack-nova14:25
sambettsjroll, efried: I can't see anything in the placement logs to indicate why it could sync :/14:26
*** lyan is now known as Guest7532614:26
*** andreas_s has joined #openstack-nova14:26
*** andreas_s has quit IRC14:26
sambettsand I would have thought if this was racey it would be so consistently failing :/14:26
*** jibby has left #openstack-nova14:26
efriedsambetts: Which call is 409ing?14:26
dansmithI'm still not clear on why we just need to know if any of the networks have the attributes specified, because that doesn't seem like nearly enough information for us to do something differently that we couldn't do by default, but..14:26
*** andreas_s has joined #openstack-nova14:27
*** alexchadin has joined #openstack-nova14:27
sambettsefried: I don't see any14:27
efriedwell wtf14:28
mriedemdansmith: if it's satisfactory for you then maybe not, but as you said, i'm still unclear on how this is going to be used and i haven't gotten that far in the series yet to find out - at least i'd like to hold off on approving the object changes at the bottom of the series until i'm more comfortable with how they are being used in the rest of the series14:28
dansmithokay I'm not sure what revelation you've had that makes you okay with it now, if not "why"14:29
*** heyongli has quit IRC14:29
mriedembut i think it answers my question about physnet and tunneled being mutually exclusive for a given network14:29
*** heyongli has joined #openstack-nova14:29
dansmithokay, yeah I get that although it's on faith about the larger thing14:29
*** mvk has quit IRC14:29
*** shaohe_feng has quit IRC14:31
*** shaohe_feng has joined #openstack-nova14:32
tssuryadansmith: do you have some time now to discuss about aggregates behaviour ?14:33
dansmithtssurya: yeah, I was going to hassle mriedem about one other thing, but it can wait14:33
tssuryaso, since we have aggregate "hosts" concept, the thing doesn't work well with ironic14:34
tssuryasince in ironic 1 host = many compute nodes14:34
tssuryaand we would need something like aggregate "compute bodes"14:34
tssuryanodes*14:34
*** tidwellr has joined #openstack-nova14:34
tssuryabecause now we have the pre-placement filters which allow scheduling14:34
efriedsambetts, jroll: Let me spin up a quick and dirty fix and we can see if it clears this up.  Need a couple hours tho.14:35
tssuryabased on aggregates14:35
sambettsefried: thanks that would be really useful :)14:35
*** Luzi has quit IRC14:35
tssuryaI guess what I am trying to say is, does it make sense to expand aggregate add hosts to actually be aggregate add compute nodes ?14:35
dansmithtssurya: well, that's a sticky situation of course14:37
mriedemtssurya: there is a whole series i just abandoned for that last week14:37
mriedemand a spec14:37
dansmithtssurya: you can create aggregates in placement and add compute nodes to them because the resource providers are actually the ironic nodes14:37
tssuryamriedem: oh really ?14:37
jrollefried: <314:37
mriedemand we said at the ptg you can do aggregates with pre-placement request filters and placement aggregates14:37
dansmithtssurya: but in terms of making it automatic we'd need to change nova's api14:37
tssuryadansmith: right, I mean nova api change14:38
tssuryabecause it is a pain to add this manually on the nova side14:38
dansmithyeah, that's a problem14:38
tssuryafor the ironic nodes14:38
*** lpetrut has quit IRC14:38
*** eharney has joined #openstack-nova14:38
mriedemtssurya: https://review.openstack.org/#/q/topic:bp/aggregate-affinity14:38
tssuryamriedem: thanks looking14:39
*** tidwellr has quit IRC14:39
*** heyongli has quit IRC14:39
*** heyongli has joined #openstack-nova14:39
dansmithmriedem: https://review.openstack.org/#/c/559032/14:41
*** shaohe_feng has quit IRC14:41
*** shaohe_feng has joined #openstack-nova14:42
*** jmlowe has quit IRC14:43
tssuryamriedem, dansmith: oh right so the work around was to just have them in the placement aggregate!14:44
tssuryathanks14:44
dansmithyeah14:44
tssuryaI guess I totally missed/forgot14:44
tssuryathat14:44
*** jmlowe has joined #openstack-nova14:45
*** rmart04_ has joined #openstack-nova14:48
*** andreas_s has quit IRC14:49
*** heyongli has quit IRC14:49
*** heyongli has joined #openstack-nova14:49
*** andreas_s has joined #openstack-nova14:50
*** yamamoto has joined #openstack-nova14:50
*** rmart04 has quit IRC14:50
*** rmart04_ is now known as rmart0414:50
*** edmondsw has joined #openstack-nova14:51
*** ragiman has quit IRC14:51
*** shaohe_feng has quit IRC14:52
*** shaohe_feng has joined #openstack-nova14:53
*** BlackDex has quit IRC14:54
*** mvk has joined #openstack-nova14:55
*** edmondsw has quit IRC14:55
*** yamamoto has quit IRC14:58
*** heyongli has quit IRC14:59
*** heyongli has joined #openstack-nova15:00
sahidstephenfin: do you want reconsider your vote for https://review.openstack.org/#/c/579611/ ?15:00
sahidmriedem: ^15:00
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the instance payload  https://review.openstack.org/53624315:01
stephenfinsahid: In a meeting but I can take a look once I'm done15:01
*** dpawlik has quit IRC15:02
*** shaohe_feng has quit IRC15:02
dansmithmriedem: coming? it's awkward with just stephenfin and myself15:02
mriedem4ever in bluejeans15:02
mriedemthis isn't hangouts at all15:02
openstackgerritKonstantinos Samaras-Tsakiris proposed openstack/nova master: Hide hypervisor id on windows guests  https://review.openstack.org/57989715:02
*** shaohe_feng has joined #openstack-nova15:02
sahidstephenfin: ok np, nothing hurry actually15:03
*** quiquell|rover is now known as quiquell|off15:05
*** peereb has quit IRC15:05
*** gyee has joined #openstack-nova15:05
*** voelzmo has quit IRC15:07
*** rmart04 has quit IRC15:07
*** lpetrut has joined #openstack-nova15:09
*** felipemonteiro_ has joined #openstack-nova15:09
*** heyongli has quit IRC15:10
*** heyongli has joined #openstack-nova15:10
*** felipemonteiro__ has joined #openstack-nova15:10
*** shaohe_feng has quit IRC15:12
*** alexchadin has quit IRC15:13
*** shaohe_feng has joined #openstack-nova15:14
*** felipemonteiro_ has quit IRC15:14
*** aarents has quit IRC15:20
*** heyongli has quit IRC15:20
*** heyongli has joined #openstack-nova15:20
*** aarents has joined #openstack-nova15:21
*** shaohe_feng has quit IRC15:22
*** yingjun has quit IRC15:23
*** shaohe_feng has joined #openstack-nova15:23
efriedsambetts, jroll: Can you identify whether there are/were/should be nodes with the following UUIDs in the environment for15:28
efriedhttp://192.133.158.227/12/514312/12/experimental/ironic-dsvm-tempest-ironic-cimc-current-centos-7-cimc/f7c455a/logs/screen-n-cpu.txt.gz15:28
efried=> fbd2e932-ea59-4b57-a206-a839f40fa18215:28
efried=> 7e1567e7-4d7c-4ad1-9563-3d277f4fb04d15:28
efried=> 72b0a2e0-277b-4698-bfd6-aa15623e44bd15:29
*** weshay|ruck is now known as weshay15:29
jrollefried: those are likely fake nodes created by other tempest tests15:29
jrollthat said, this is a third party CI environment that isn't mine15:30
jrollso I'll let sam speak from here on :)15:30
*** heyongli has quit IRC15:30
*** heyongli has joined #openstack-nova15:30
efriedahhh.  So, that appears to be what's actually causing the failure here.  Those guys show up in the provider tree, so we try to update them, but by the time we get to hitting placement for them, they're gone.15:31
sambettsefried: those nodes were created by the ironic tempest api tests and then deleted after, the only real node in the test environment is b5bb2c51-3960-4d9c-b47e-223d83cb613915:31
jrolloh, we update everything at build time?15:31
jrollor everything for that compute service15:31
efriedjroll: Yes, update_from_provider_tree is supposed to make the host's placement presence right.15:32
jrollmmmm15:32
stephenfinsahid: Yup, makes sense. +2 now15:32
*** shaohe_feng has quit IRC15:33
*** shaohe_feng has joined #openstack-nova15:34
openstackgerritMerged openstack/nova master: Update xenapi_disable_agent config option usage in docs  https://review.openstack.org/57884815:36
*** heyongli has quit IRC15:40
*** heyongli has joined #openstack-nova15:41
*** dklyle has joined #openstack-nova15:41
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration  https://review.openstack.org/55137015:43
openstackgerritMatt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager  https://review.openstack.org/55137115:43
openstackgerritMatt Riedemann proposed openstack/nova master: Port binding based on events during live migration  https://review.openstack.org/43487015:43
openstackgerritMatt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate  https://review.openstack.org/52253715:43
*** shaohe_feng has quit IRC15:43
*** namnh has quit IRC15:44
*** shaohe_feng has joined #openstack-nova15:44
*** namnh has joined #openstack-nova15:44
efriedsambetts: jroll: So I think this is probably the problem here, which a retry isn't going to fix: ironic's update_provider_tree isn't getting rid of nodes when they disappear.15:45
*** med_ has quit IRC15:45
efriedsambetts, jroll: How (via what code path) are these fake nodes "created by the ironic tempest api tests and then deleted after" ?15:45
jrollefried: right, and it races with the thing that cleans up orphan nodes15:46
jrollefried: via api tests in our tempest plugin being run (in parallel?) with the scenario tests15:46
efriedjroll: Oh, we have a thing that cleans up orphan nodes?  Does that thing actually remove them from placement?  Where does that code live?15:46
jrollefried: not sure where the code is offhand, but see e.g. http://192.133.158.227/12/514312/12/experimental/ironic-dsvm-tempest-ironic-cimc-current-centos-7-cimc/f7c455a/logs/screen-n-cpu.txt.gz#_Jul_03_12_53_45_41403615:47
efriedjroll: Okay, but the tempest tests presumably use some kind of API to create/delete the nodes.  Because they're ending up registered in placement somehow.15:47
jrollI think it's handled in update_available_resource or something15:47
*** tssurya has quit IRC15:47
sambettsthey are just created in ironic, then nova syncs with ironic and picks up the new nodes15:47
jrollefried: yes, POST ironic-host/v1/nodes, then the ..... yes that15:47
efriedoho15:48
efriedit's happening *after* _update_available_resource_for_node15:48
efriedsambetts: How reproducible is this thang?15:49
*** chyka has joined #openstack-nova15:50
openstackgerritStephen Finucane proposed openstack/nova master: network: Always retrieve network information if available  https://review.openstack.org/56444415:51
openstackgerritStephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_physnet_info'  https://review.openstack.org/56444515:51
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMACell.network_metadata  https://review.openstack.org/56443915:51
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMATopologyLimits.networks  https://review.openstack.org/57548615:51
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Start accounting for networks in NUMA placement  https://review.openstack.org/56444815:51
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add RequestSpec.network_metadata  https://review.openstack.org/56444215:51
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445215:51
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444015:51
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field  https://review.openstack.org/56444115:51
sambettsevery single one of my CI runs hits it at the moment, I expect the reason the upstream ironic CI isn't hitting it is because setting tempest concurrency=115:51
*** heyongli has quit IRC15:51
*** heyongli has joined #openstack-nova15:51
*** nicolasbock has joined #openstack-nova15:52
*** shaohe_feng has quit IRC15:53
*** ccamacho has quit IRC15:54
*** shaohe_feng has joined #openstack-nova15:55
*** moshele has joined #openstack-nova15:57
*** Guest66593 has quit IRC15:58
*** namnh has quit IRC15:58
*** wwriverrat has quit IRC15:59
*** heyongli has quit IRC16:01
*** heyongli has joined #openstack-nova16:01
*** shaohe_feng has quit IRC16:03
*** shaohe_feng has joined #openstack-nova16:05
*** namnh has joined #openstack-nova16:05
*** sahid has quit IRC16:08
mriedemgibi: so i'm looking at https://bugs.launchpad.net/nova/+bug/1739325 again and Flavor.disabled isn't something you can set from the API as far as i can tell16:08
openstackLaunchpad bug 1739325 in OpenStack Compute (nova) "Server operations fail to complete with versioned notifications if payload contains unset non-nullable fields" [Medium,Confirmed] - Assigned to Balazs Gibizer (balazs-gibizer)16:08
*** wwriverrat has joined #openstack-nova16:09
*** wwriverr_ has joined #openstack-nova16:10
*** namnh has quit IRC16:10
gibimriedem: but then why we even have a disabled field?16:11
*** heyongli has quit IRC16:11
mriedemidk, i'm not seeing in git history where it was added, or if there is a way to ever set it via cli or something16:11
*** felipemonteiro__ has quit IRC16:11
mriedemdansmith: do you remember?16:11
*** heyongli has joined #openstack-nova16:11
gibimriedem: https://bugs.launchpad.net/nova/+bug/125926216:13
openstackLaunchpad bug 1259262 in python-novaclient "there is no api or cli to enable/disable a flavor" [Wishlist,Opinion]16:13
*** wwriverrat has quit IRC16:13
gibipretty old whishlist bug to make the disabled flag changeable16:13
*** shaohe_feng has quit IRC16:14
mriedemhttps://github.com/openstack/nova/commit/f37119816:14
*** shaohe_feng has joined #openstack-nova16:15
gibimriedem: nice finding. Even this patch uses direct db access to created a disabled instance_type https://github.com/openstack/nova/commit/f371198#diff-59404890aec6423ece734730c153addbR75216:16
gibiI think we can conclude that there is no way to set it outside of the API16:17
mriedemso i think the issue might be related to https://github.com/openstack/nova/blob/master/nova/compute/flavors.py#L5216:17
mriedemand how we used to store the embedded flavor in the instance based on the instance system_metadata16:17
mriedemwhich didn't include the 'disabled' key16:17
mriedemso likely any really old instance didn't have the instance_type_disabled key in it's system_metadata16:18
mriedemwhich might explain why we're hitting "Field `disabled' cannot be None" when extracting these things and loading them into Flavor objects16:18
*** dpawlik has joined #openstack-nova16:19
gibithis very well be the explanation16:19
gibidansmith: would this ^^ be enough explanation to revive https://review.openstack.org/#/c/529194/ ?16:20
*** tesseract has quit IRC16:20
gibidansmith: you were -1 on that patch as we did not know why can be the instance.flavor.disabled undefined16:21
gibimriedem: thanks for the investigation I was not able to find this by myself16:21
*** heyongli has quit IRC16:21
*** heyongli has joined #openstack-nova16:22
*** andreas_s has quit IRC16:22
*** andreas_s has joined #openstack-nova16:22
*** andreas_s has quit IRC16:22
dansmithgibi: I'd really rather not embed that behavior in the flavor object.. either we should set those properties when we load an older flavor from the instance_extra, or (better) just be graceful about it when we're emitting the notification16:22
*** shaohe_feng has quit IRC16:24
*** artom has joined #openstack-nova16:24
*** lpetrut has quit IRC16:24
*** shaohe_feng has joined #openstack-nova16:25
mriedemleft a comment here about how i think it all tied together https://bugs.launchpad.net/nova/+bug/1739325/comments/916:26
openstackLaunchpad bug 1739325 in OpenStack Compute (nova) "Server operations fail to complete with versioned notifications if payload contains unset non-nullable fields" [Medium,Confirmed] - Assigned to Balazs Gibizer (balazs-gibizer)16:26
*** ispp has quit IRC16:26
gibimriedem: thanks16:26
gibidansmith: do we still load those old flavors from the system_metadata or we loaded once and copied into the instance without the disabled field?16:27
dansmithno, we don't, they've all been migrated to extra now16:28
gibidansmith: then the first option is not a real solution16:29
dansmithgibi: no, the first option I meant was when we're in the context of the instance, loading a flavor from instance_extra16:29
mriedemwouldn't we just implement obj_load_attr (lazy-load) for Flavor.disabled?16:29
dansmithwe could, but that also seems obscure to me,16:30
dansmithbecause the flavor could be coming from the instance, or from the actual db16:30
dansmithand we wouldn't want the disabled default to affect the latter16:30
mriedemso then add default=False to 'disabled': fields.BooleanField(), ?16:30
dansmithit seems like we should patch it up if/when we're loading from extra and notice that field is gone16:30
dansmiththat's basically the same as the lazy-load, although slightly less icky, and one of the things I suggested in the original patch16:31
*** heyongli has quit IRC16:32
mriedemok so maybe handle in _flavor_from_db16:32
mriedemInstance._flavor_from_db16:32
*** heyongli has joined #openstack-nova16:32
mriedemif 'disabled' not in flavor_info16:32
mriedemflavor_info['disabled'] = False16:32
dansmithexactly16:32
mriedemok i'll push that up16:32
gibimriedem, dansmith I got it16:32
dansmithbecause there we know what we're dealing with16:32
*** dpawlik has quit IRC16:32
dansmithand a comment about old instance flavors will be quite relevant16:33
dansmith_load_flavor will get it implicitly if we're actually lazy-loading instance.flavor too, so that is good16:33
*** shaohe_feng has quit IRC16:34
zzzeeknova peeps, dansmith  / mriedem , what happens when someone in Newton sets "workers=0" in nova.conf, does that mean default to number of CPUs ?16:34
gibimriedem, dansmith: thanks for the help16:35
*** ralonsoh has quit IRC16:35
*** shaohe_feng has joined #openstack-nova16:35
mriedemdansmith: gibi: pre-tests https://gist.github.com/mriedem/3f1560a7e9697ade633ab11b6aab8c6216:36
mriedemzzzeek: yes16:37
zzzeekmriedem: thanks!16:37
dansmithmriedem: sure16:37
gibimriedem: LGTM16:37
*** heyongli has quit IRC16:42
*** heyongli has joined #openstack-nova16:42
*** sapd has joined #openstack-nova16:43
*** shaohe_feng has quit IRC16:44
*** amoralej is now known as amoralej|off16:45
*** shaohe_feng has joined #openstack-nova16:46
*** claudiub has quit IRC16:49
efriedsambetts: Sorry, it's been a day.  Can you tell me where/how to run this thing?16:49
efriedIf I push a nova patch, do I have to push something in ironic with a Depends-On?  Looks like an experimental job, needs special incantation?16:50
*** yamamoto has joined #openstack-nova16:50
efriedsambetts: Can I mod the commit message on https://review.openstack.org/#/c/514312/ to add the Depends-On?16:51
openstackgerritJay Pipes proposed openstack/nova master: move lookup of provider from _new_allocations()  https://review.openstack.org/57992016:52
openstackgerritJay Pipes proposed openstack/nova master: placement: delete auto-created consumers on fail  https://review.openstack.org/57992116:52
*** heyongli has quit IRC16:52
*** heyongli has joined #openstack-nova16:52
*** shaohe_feng has quit IRC16:55
openstackgerritEric Fried proposed openstack/nova master: WIP: Delete orphan nodes before updating resources  https://review.openstack.org/57992216:55
*** yamamoto has quit IRC16:55
*** shaohe_feng has joined #openstack-nova16:56
sambettsefried: you can mod the commit message to add the Depends-On, then you need to leave a "cisco-experimental" comment to trigger our experimental job queue which has some other fixes for our ironic job16:57
efriedsambetts: Roger wilco, it's on the way...16:57
sambettsefried: thanks so much for looking into this!16:57
efriedsambetts: the experimental fix is ---^ btw16:57
*** derekh has quit IRC16:58
sambettsefried: oh nice thats quite an eligant fix16:58
efriedsambetts: I think it's probably going to work, but I don't think that's the end of it.  We're seeing similar races in other places as well (though not as consistently), so I think I really need to implement that retry loop I mentioned earlier.16:59
*** janki has quit IRC16:59
sambettsyeah I feel like the whole compute claiming thing (particularly with ironic) is really racey, I still think we have the race to do with a claim being released on an ironic node that then goes into cleaning, doesn't get removed from the resource tracker before a new claim can be made on it17:01
sambettsbut thats another story17:01
dansmithsambetts: there's no compute claiming anymore17:01
*** heyongli has quit IRC17:02
*** heyongli has joined #openstack-nova17:03
sambettsmight be using the wrong words, still a race between the scheduler and the resource tracker17:03
efrieddansmith: TL;DR we're running into races now because rt._update is run both during periodic and from instance_claim.  Both of them hit update_[from_]provider_tree asynchronously, and freak each other out.17:03
efriedYeah, what sambetts said.17:03
dansmiththere should be no changes coming from resource tracker17:04
efriedThe periodic is designed to be self-healing, but if the race makes the claim side fail, that's bad.17:04
dansmithor do you mean a race between an allocation and inventory changes/17:04
*** shaohe_feng has quit IRC17:05
efrieddansmith: Yeah, or aggregate changes, which are happening a relatively lot now that we're mirroring host aggs to placement.17:05
openstackgerritMatt Riedemann proposed openstack/nova master: Default embedded instance.flavor.disabled attribute  https://review.openstack.org/57992517:05
mriedemdansmith: gibi: ^17:06
dansmithwell, I guess I see that as a major improvement over the races we used to have between scheduling and claiming (on the compute)17:06
dansmithbut yeah, "compute claiming" is the wrong term since that's not what happens now17:06
efrieddansmith: What we're seeing in tempest envs is aggs being added/removed as hosts are brought up & down in concurrent tests.17:06
dansmithhosts aren't brought up and down in tempest17:06
efrieddansmith: I've been thinking the real solution will be a retry loop around this chunk: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L883-L91417:06
dansmithbut tempest may be creating/destroying aggregates17:06
*** shaohe_feng has joined #openstack-nova17:07
efrieddansmith: Here's an example: http://logs.openstack.org/69/556669/11/check/nova-multiattach/b3195af/job-output.txt.gz#_2018-06-22_14_50_02_09550617:07
efriedAdding agg generation checking to the report client exposes it pretty badly.17:08
dansmiththat message means you didn't remove all the hosts from the agg before trying to delete the agg17:08
dansmiththat shouldn't have anything to do with inventory or instances being booted17:08
*** dtantsur is now known as dtantsur|afk17:09
dansmithperhaps means we failed to remove a host because a call to placement (for mirroring) failed or something like that17:09
efriedActually, that message doesn't make sense to me.17:09
efriedcan't remove X from {X, ...} because {X, ...} isn't empty?  Well of course it isn't - it contains X.  Or am I misinterpreting that?  Is the first aggregate in that sentence the placement aggregate?17:10
dansmithit's saying "can't delete aggregate "foo" because "foo" still contains some host "bar""17:10
*** Guest75326 has quit IRC17:11
*** sambetts is now known as sambetts|afk17:11
sambetts|afkefried: thanks again for looking into that stuff, I'll check the results in the morning, night all o/17:12
efriedo/17:12
efrieddansmith: That's not how it's worded, perhaps we could stand to fix thta.17:12
*** cfriesen has joined #openstack-nova17:12
dansmithI'm not sure why it's confusing.. it makes sense to me17:13
*** heyongli has quit IRC17:13
dansmithyou did a delete of aggregate 217:13
dansmithcan't remove aggregate 2 because it's not empty17:13
dansmithoh, I see the confusing part "cannot remove host"17:13
dansmithfair enough17:13
*** heyongli has joined #openstack-nova17:13
*** shaohe_feng has quit IRC17:15
mriedemhttp://logs.openstack.org/69/556669/11/check/nova-multiattach/b3195af/logs/screen-n-api.txt.gz#_Jun_22_14_30_55_67714117:15
mriedemthat's the failure17:15
mriedemUnexpected exception in API method: ResourceProviderUpdateConflict: A conflict was encountered attempting to update resource provider b9342d94-3665-4da1-90c8-e48ab77e181e (generation 6): {"errors": [{"status": 409, "request_id": "req-fe8eea9d-ec9d-4128-9117-73a68e462394", "detail": "There was a conflict when trying to complete your request.\n\n Update conflict: Another thread concurrently updated the data. Please retry your u17:15
mriedeme  ", "title": "Conflict"}]}17:15
efriedjust so17:16
mriedemwho wants to open the bug17:16
efriedWell, it's not a bug yet17:16
dansmithso no retry around te mirror?17:16
mriedemnope17:16
efriedit doesn't show up until we start gen checking aggs.17:16
*** shaohe_feng has joined #openstack-nova17:16
mriedemi read that as 'chicken eggs'17:16
mriedemmmm17:17
efriedA rat snake has been eating our eggs in the coop.17:17
efriedI mean, the race is still there, we're just blowing away whatever the first one did with the second.17:17
efriedI hear you're supposed to put a light bulb in the nest box.  They eat the bulb and... ouch.17:17
mriedemadd_host_to_aggregate does handle several other exceptions and log a warning and return17:17
efriedmm, we could add UpdateConflict in there, but that would just mask the problem (I had that objection to that try/except when this code went in iirc).17:18
mriedemhmm, wtf17:19
mriedemwe don't pass a generatoin when updating the aggregates17:19
efriedmriedem: That patch does.17:19
mriedemah ok17:19
efriedSwhat I was saying earlier.  This isn't causing failures except in that patch.  (The ironic thing is different.)17:19
efried(though related, and probably fixable by the same retry loop)17:20
* mriedem backs away and gets lunch17:20
*** med_ has joined #openstack-nova17:21
*** med_ has quit IRC17:21
*** med_ has joined #openstack-nova17:21
*** felipemonteiro_ has joined #openstack-nova17:21
dansmithI'm not sure why being obsessive about the generation in reportclient would cause the api side to fail,17:21
dansmithunless it's just that it generates a little more load and opportunity to collide with the api mirror17:22
*** felipemonteiro__ has joined #openstack-nova17:23
*** heyongli has quit IRC17:23
*** heyongli has joined #openstack-nova17:23
*** lyan has joined #openstack-nova17:24
*** lyan is now known as Guest5309517:24
efrieddansmith: Every time _update runs (so periodic and from instance_claim) it tries to sync placement.  Part of that is hitting the method to push aggregates for the compute host, which is the same method the host agg sync runs.  If _update thinks it needs to push *any* change associated with the compute node RP, and the host agg sync happens between when we retrieve the data and when we push it, we'll 409.17:25
efried...once we're checking gens on aggs, which is what this patch introduces.17:25
*** shaohe_feng has quit IRC17:25
dansmithokay so just a lot of bumping the agg generation yeah?17:25
*** shaohe_feng has joined #openstack-nova17:26
efrieddansmith: update_from_provider_tree is smart enough to invalidate the cache entry for the provider that 409s, so the next time around it'll re-GET and have the new generation.  But up to this point, we're not doing that retry until the next periodic hits.17:26
efriedSo from the pov of periodic, we're fine, it heals itself.  But if the 409 happens in the instance_claim path, it can cause a resched/fail.17:27
*** felipemonteiro_ has quit IRC17:27
dansmithokay but the failure that breaks a test is the agg mirroring in the api17:27
dansmithwhich is presumably because it doesn't retry on 40917:27
dansmithwhat I'm wondering is,17:27
efriedright17:27
dansmithwhy is the compute causing any update churn on the agg generation17:27
dansmith?17:27
efriedDon't we bump the RP gen whenever we allocate?17:28
efriedThe agg doesn't have a generation.  The RP does.17:28
*** BlackDex has joined #openstack-nova17:29
dansmithokay, so the 409 that the api is getting is that the RP it's trying to add to the agg has changed generations (and it doesn't retry)?17:29
efriedright17:29
dansmithso what extra churn on the RP is the patch causing?17:30
efriedIt's not causing extra churn.  Before this patch, we weren't checking generations on agg updates - we were just blasting them in.17:30
efriedSo we never got 409 - just whoever went second won.17:30
dansmithright, but that's my point,17:30
efriedwhich is a bug17:30
efriedbut it would be hidden17:30
dansmithwe're doing the same number of RP updates as before this patch?17:31
efriedchecking gens exposes the bug.17:31
efriedyes17:31
dansmith...so what did it change that causes it to race and conflict with the api mirroring?17:31
efriedThe change that caused the race was the introduction of host agg sync.17:31
dansmithor are we potentially doing one more update if we got a 409 on the compute node and do a retry?17:31
efriedBut the race stayed hidden because we weren't checking rp gen on agg updates.17:31
efriedUntil this patch.17:31
efriedNothing does a retry (yet)17:32
dansmithno, that doesn't make sense17:32
dansmithyou checking aggs on the compute node doesn't make something else get a 40917:32
efriedI'm suggesting that as the fix.17:32
efriedOh, the generation checking is in the method that's common to both the compute side and the host agg sync.17:32
efriedsorry that wasn't clear17:32
dansmithoh set_aggregates_for_provider()17:33
dansmith?17:33
efriedhttps://review.openstack.org/#/c/556669/11/nova/scheduler/client/report.py@1250 yeah17:33
*** heyongli has quit IRC17:33
dansmithapi is using that for the mirroring, which never would fail before because it didn't check, but now is failing because it does, in the patch17:33
*** heyongli has joined #openstack-nova17:33
dansmithokay, I got it.. I thought you were saying lots of things other than this17:33
* dansmith cancels the hit17:34
efried:)17:34
*** Sundar has joined #openstack-nova17:34
efriedYou were too late.  I already killed your hit man.17:34
dansmithI requested NFO17:34
dansmithso that must have been jaypipes' hit man17:34
efriedwhile typing with the other hand17:34
efrieddansmith: So do you agree adding a retry on both sides of this thing would be a viable solution?17:35
*** jpena is now known as jpena|off17:35
*** shaohe_feng has quit IRC17:36
dansmithwell, I agree that (a) mirroring should be checking generation and (b) that it should retry if generation conflicts, so .. yes?17:36
efrieddig17:36
*** shaohe_feng has joined #openstack-nova17:36
efriedI'll retract what I said earlier about there not being a bug, mriedem.  There is, we were just hiding it before.  I'll open it and *try* to explain it.  Then fix it via ^17:38
dansmithagree17:38
efriedDoes the Big Lock in the rt mutex the _update periodic with the instance_claim path?17:39
*** sridharg has quit IRC17:39
efriedI guess it would have to.17:39
dansmiththat's the point of it I think17:41
efriedconfirmed.17:41
*** heyongli has quit IRC17:43
*** heyongli has joined #openstack-nova17:44
*** shaohe_feng has quit IRC17:46
*** shaohe_feng has joined #openstack-nova17:47
*** Sundar has quit IRC17:49
*** yamamoto has joined #openstack-nova17:51
*** dklyle has quit IRC17:53
*** namnh has joined #openstack-nova17:54
*** heyongli has quit IRC17:54
*** heyongli has joined #openstack-nova17:54
*** yamamoto has quit IRC17:56
*** shaohe_feng has quit IRC17:56
*** shaohe_feng has joined #openstack-nova17:57
*** namnh has quit IRC17:59
*** heyongli has quit IRC18:04
*** heyongli has joined #openstack-nova18:04
*** shaohe_feng has quit IRC18:06
*** shaohe_feng has joined #openstack-nova18:09
*** moshele has quit IRC18:11
openstackgerritMerged openstack/nova master: Merge server create schema for multiple create extension  https://review.openstack.org/57833818:11
openstackgerritMerged openstack/nova master: Merge server create schema for scheduler hint extension  https://review.openstack.org/57834318:11
*** cdent has joined #openstack-nova18:13
*** heyongli has quit IRC18:14
*** heyongli has joined #openstack-nova18:14
*** shaohe_feng has quit IRC18:17
*** shaohe_feng has joined #openstack-nova18:19
*** heyongli has quit IRC18:24
*** heyongli has joined #openstack-nova18:25
*** shaohe_feng has quit IRC18:27
*** sapd has quit IRC18:27
*** edmondsw has joined #openstack-nova18:27
*** shaohe_feng has joined #openstack-nova18:28
*** mriedem has quit IRC18:31
*** edmondsw has quit IRC18:32
*** pchavva has quit IRC18:32
*** heyongli has quit IRC18:35
*** heyongli has joined #openstack-nova18:35
*** shaohe_feng has quit IRC18:37
*** shaohe_feng has joined #openstack-nova18:38
*** mriedem has joined #openstack-nova18:39
*** imacdonn has joined #openstack-nova18:40
karimullEric: I have one issue with a functional test case in nova on shared RP, can you help me understand where the issue is? here is the link18:42
karimullEric: http://paste.openstack.org/show/i9a6MRtJ1IUxF9xlI8to/18:43
*** eharney has quit IRC18:43
*** mgoddard has joined #openstack-nova18:44
*** heyongli has quit IRC18:45
*** heyongli has joined #openstack-nova18:45
openstackgerritMatt Riedemann proposed openstack/nova master: Add action initiator attribute to the instance payload  https://review.openstack.org/53624318:45
*** mgoddard has quit IRC18:47
*** shaohe_feng has quit IRC18:47
*** shaohe_feng has joined #openstack-nova18:48
*** itlinux has joined #openstack-nova18:50
karimullefried: oops! pasted with wrong handle18:52
efriedkarimull: looking...18:52
*** yamamoto has joined #openstack-nova18:52
karimullefried:thank you..18:52
efriedkarimull: What patch are you working on this test for?18:53
*** heyongli has quit IRC18:55
*** med_ has quit IRC18:55
*** heyongli has joined #openstack-nova18:55
karimullefried:https://review.openstack.org/#/c/569498/918:57
efriedkarimull: Is the rebuild succeeding18:57
efried?18:57
*** shaohe_feng has quit IRC18:58
*** yamamoto has quit IRC18:58
efriedkarimull: This is trying to rebuild to the same host?18:58
*** shaohe_feng has joined #openstack-nova18:58
openstackgerritMerged openstack/nova master: conf: Correct documentation for '[pci] passthrough_whitelist'  https://review.openstack.org/55287418:59
openstackgerritMerged openstack/nova master: [doc] enhance admin/configuration/api.rst  https://review.openstack.org/56600818:59
openstackgerritMerged openstack/nova master: Remove mox in sec group test and functional tests  https://review.openstack.org/57675118:59
karimullEfried:server state is Active after rebuild, but the DISK_GB usage is not accounted?19:00
openstackgerritMatt Riedemann proposed openstack/nova master: Time how long pre_live_migration() takes  https://review.openstack.org/57448119:00
efriedkarimull: or it got the disk from the compute node.19:00
efriedkarimull: Where is the flavor defined?19:00
openstackgerritMerged openstack/nova master: conf: Resolve Sphinx errors  https://review.openstack.org/57954519:02
karimullefried:I commented out "self.assertEqual({'DISK_GB': 1}, shared_rp_usages)" and the test passed19:02
karimullefried: one min19:03
efriedkarimull: I'm not completely sure how rebuild works, but I can't see where you're checking that the rebuild happened at all.19:03
efriedkarimull: The other thing I'm confused about is how the rebuild could be succeeding - assuming the flavor asks for MEMORY_MB - because afaict you've *replaced* the inventory with just VCPU:1 (L92 of your paste).  That's why I want to see the flavor...19:04
efriedoh, I guess it doesn't reallocate.19:04
efriedokay, I think I understand what's going on.19:05
*** claudiub has joined #openstack-nova19:05
*** heyongli has quit IRC19:05
*** heyongli has joined #openstack-nova19:06
*** felipemonteiro__ has quit IRC19:06
*** felipemonteiro__ has joined #openstack-nova19:06
efriedWhen you initially create the VM, it's getting its DISK_GB from the compute node.  If you want to force it to get the DISK_GB from the sharing provider, you'll have to remove the DISK_GB inventory from the compute node.  I think the libvirt driver does this for you at this point; try copying L76 to after L80 and see if it blows up.19:06
karimullefried:ok..let me check19:07
*** shaohe_feng has quit IRC19:08
efriedI think the other thing to know here is that I kinda doubt we're reallocating properly on rebuild with sharing providers.  I don't think we've gone through and scrubbed the code for this.19:08
*** shaohe_feng has joined #openstack-nova19:10
*** dgonzalez has quit IRC19:14
*** MultipleCrashes has joined #openstack-nova19:15
*** tssurya has joined #openstack-nova19:15
cdentefried: there's a fixme at that part of the code for any kind of migration-like thing, if I remember right19:15
*** heyongli has quit IRC19:16
efriedcdent: Yes, I believe you are correct.19:16
*** heyongli has joined #openstack-nova19:16
*** dgonzalez has joined #openstack-nova19:16
jmloweI've been chipping away at getting erasure coding support for ceph, patches in review for glance and cinder.  I hit a wall with nova.19:16
efriedcdent, karimull: It's not clear where we're messing up in this flow, though.  Is the initial build allocating from the sharing provider or the compute node provider?  Is the rebuild changing allocations at all?  No combination of answers fully makes sense with what karimull is seeing, though.19:17
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:18
*** shaohe_feng has quit IRC19:18
efriedcdent: Do I tag a bug `placement` if it's a bug in nova's use of placement?19:18
karimullefried: failing at the same place...looks like libvirt (mock?) is not setup correctly for this test case... compared to other test cases similar to this one ..19:19
jmloweBasically all you need to do is give the name of an erasure coded pool when you do create or clone operations.  I saw some soft of inferred pool name for cloning back into glance and I have no idea how to infer the data pool option.19:19
*** shaohe_feng has joined #openstack-nova19:19
mriedemdansmith: see what you think about what yikun is doing here during this refactor https://review.openstack.org/#/c/579113/19:19
jmloweAny ideas or am I just talking nonsense here?19:19
efriedkarimull: Oh, yeah, I missed that, you're using SmallFakeDriver, you should be using FakeLibvirtDriver.19:20
cdentefried: for now, yes19:20
mriedemi understand why he's removing the ability to save changes to InstanceGroup.policies (it saves him some work for a thing that isn't used today and isn't supported anyway), but still19:20
efriedmriedem, dansmith: FYI https://bugs.launchpad.net/nova/+bug/177993119:20
openstackLaunchpad bug 1779931 in OpenStack Compute (nova) "Provider update race between host aggregate sync and resource tracker" [Undecided,New]19:20
*** MultipleCrashes has quit IRC19:20
*** dgonzalez has quit IRC19:21
karimullefried:good catch..let me try with FakeLibvirtDriver..thanks ..19:21
mriedemjmlowe: no idea, jbernard or mnaser or cfriesen might have ideas19:22
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:22
efriedkarimull: Actually, I don't think you want to use FakeLibvirtDriver, sorry.19:23
efriedI think you want to use the *actual* LibvirtDriver, with the FakeLibvirtFixture.19:23
efriedkarimull: I think you should rebase on top of 579201 and set up your test case like https://review.openstack.org/#/c/579201/1/nova/tests/functional/libvirt/test_report_cpu_traits.py19:24
dansmithmriedem: what does the uselist=False do on the join?19:24
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:24
mriedemdansmith: i think enforces the one-to-one mapping19:24
karimullefried:sure will do that..19:24
efrieddansmith, mriedem: Can I please get your eyes on https://review.openstack.org/#/c/579201/ ?19:24
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:24
dansmithmriedem: what happens if there's actually one to many in the DB?19:25
mriedemdansmith: http://docs.sqlalchemy.org/en/latest/orm/relationship_api.html#sqlalchemy.orm.relationship.params.uselist19:25
*** heyongli has quit IRC19:26
dansmithmriedem: that doesn't really tell me19:26
mriedemhttp://docs.sqlalchemy.org/en/latest/orm/basic_relationships.html#relationships-one-to-one19:26
*** heyongli has joined #openstack-nova19:26
mriedemthat was the answer to your 'what does uselist=False do'19:26
*** dgonzalez has joined #openstack-nova19:26
dansmithunless.. is there a unique constraint on policy now?19:26
*** eharney has joined #openstack-nova19:26
dansmithnope19:27
mriedemno19:27
dansmithI just don't see anything about what happens if there are more than one in the DB currently19:27
mriedemwould have to have a unique constraint in instance_group_policy over group_id and policy columns19:28
dansmithbecause the old code _will_ let you set multiple policies at the db/object level right?19:28
mriedembut even then that doesn't fix the uselist worry19:28
mriedemdansmith: yes19:28
mriedemeven though that's not possible from our rest api19:28
dansmithseems like it would be safer to just keep the list, and pick off the first one19:28
dansmithright19:28
*** shaohe_feng has quit IRC19:28
mriedemi think he's trying to avoid having to make this work with the new policy field in the model https://review.openstack.org/#/c/579113/4/nova/objects/instance_group.py@a7219:29
mriedemhe also needs to add the policy field to the InstanceGroup model in the next change in the series (the one we told him to split this out of)19:30
mriedembecause that's the backref to the InstanceGroupPolicy table19:31
*** shaohe_feng has joined #openstack-nova19:31
dansmithyou mean he needs to add it to this one right?19:31
*** felipemonteiro_ has joined #openstack-nova19:31
mriedemsorry yeah he's replacing _policies with policy19:31
mriedemi'm ok with his db model changes, i just wasn't sure about dropping support for saving InstanceGroup.policies in InstanceGroup.save() w/o a version bump - even though nothing does that today19:33
mnaserjmlowe: isnt an erasure coded pool just a normal ceph pool?19:33
mriedemi'd also rather not continue humping around the 1:M data model that is really 1:119:33
dansmithI guess I'm confused here19:34
*** felipemonteiro__ has quit IRC19:34
efriedcdent: This one: https://github.com/openstack/nova/blob/master/nova/conductor/tasks/migrate.py#L5619:36
*** heyongli has quit IRC19:36
cdentefried: that's the one19:36
*** heyongli has joined #openstack-nova19:37
mriedemwe don't hit that on rebuild19:37
mriedemfellas19:37
*** felipemonteiro_ has quit IRC19:37
*** felipemonteiro_ has joined #openstack-nova19:37
cdentmriedem: do we have a conceptually similar thing?19:37
mriedemno, rebuild is on the same host, no new resource allocations / claims19:38
mriedemno new taxes19:38
mriedemread my lips19:38
mriedemzzzeek: what happens with an orm.relationship(..., uselist=False) if the backref has >1 related record? kablammo? https://review.openstack.org/#/c/579113/4/nova/db/sqlalchemy/api_models.py19:38
cdentin my recent experience, there will always be new taxes^wmethods every time we do a placement thing19:38
*** shaohe_feng has quit IRC19:39
*** shaohe_feng has joined #openstack-nova19:39
cdentmriedem: is there never a doubling of resource use during a rebuild?19:39
cdentwhat magic is this?19:39
mriedemno19:39
dansmithcdent: rebuild is in place19:39
mriedemrebuild = destroy the guest and re-spawn it with optionally a new image, but re-use the same volumes and ports and uuid and all that jazz19:40
zzzeekmriedem: that will raise yse19:40
* cdent needs a cheat sheet for resize, rebuild, evacuate, shelf, etc. They all blend together19:40
zzzeekmriedem: means you did a mistake19:40
cdentthanks dansmith, mriedem19:41
* dansmith does a fistpump19:41
mriedemcdent: don't forget unshelve19:41
cdentsee!?19:41
mriedemoh you said shelf19:41
*** awaugama has quit IRC19:41
zzzeekmriedem: e.g. one-to-one is not always simple to enforce needs to have correct unique constraints to make sure u dont blow it19:41
cdentefried: so what's going on with your thing?19:41
mriedemdansmith: so are you worried that we're going to blow up someone's db if they have >1 policy record per group https://review.openstack.org/#/c/579113/4/nova/db/sqlalchemy/api_models.py ?19:41
efriedcdent: I have several things.19:41
* cdent is not sure he should be paying any attention to this now19:41
*** namnh has joined #openstack-nova19:42
zzzeekmriedem: oh the backref....umm, backref is an independent relationship.  depends on if it has uselist=False or not19:42
dansmithmriedem: I think if we don't have the schema to protect us, and we've had interfaces to let us create those, we might as well just do the safe thing and use list[0] everywhere, until we can check for and enforce 1:119:42
dansmithmriedem: just like that instance uuid dupe thing.. shouldn't have ever happened, but there were dupes in people's dbs19:43
mriedemzzzeek: tl;dr is we have 2 tables in a 1:1 relationship from a REST API standpoint, but modeled them as 1:M years ago for whatever reason, idk - future requirements or something19:43
mriedemthis change is trying to reconcile the 1:1ness19:43
zzzeekmriedem: yeah i am looking19:43
mriedemdansmith: instance uuid dupe thing? you mean when i added that unique constraint?19:43
mriedemthose were the days....19:43
dansmithyeah19:44
dansmithand remember, we added a check, cleanup, and then the enforce patch19:44
zzzeekmriedem: commented19:45
zzzeekmriedem: w/ workaroun19:45
mriedemok i'm not sure what that does for the rest of his series and the object interaction, maybe his policies property on the InstanceGroup db model just needs to change to return the backref list rather than a list of policy names19:45
mriedemand the object code would have to handle that accordingly19:45
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:45
dansmithjust always use _policies[0]19:45
dansmithhe already has that on create19:45
dansmithand an @property on the model for policy makes sense19:46
mriedemyeah i think you and zzzeek said the same thing :)19:46
mriedemworks for me19:46
dansmithokay19:46
*** heyongli has quit IRC19:46
*** heyongli has joined #openstack-nova19:47
*** namnh has quit IRC19:47
*** shaohe_feng has quit IRC19:49
mriedemdansmith: i left a summary comment, can you confirm?19:50
openstackgerritEric Fried proposed openstack/nova master: WIP: Tighten up ReportClient use of generation  https://review.openstack.org/55666919:51
efriedmriedem, cdent, dansmith, jroll, sambetts|afk: Let's try ^ that19:51
*** shaohe_feng has joined #openstack-nova19:52
efriedThere wound up being no reasonable way to do the retry in its own patch - because we weren't raising the exception to retry on.19:52
*** yamamoto has joined #openstack-nova19:54
jmlowemnaser: an erasure coded pool is normal'ish, cannot do OMAP so it cannot hold the metadata of an rbd device, only need to reference an erasure coded pool for rbd during create or clone operations19:54
jmlowemy concern is this, you use an erasure coded pool for glance, everything works ok until you go to clone back into glance during image-create then you need to know what pool to put the data in, currently infers the meta data pool just fine19:56
*** heyongli has quit IRC19:57
mriedemefried: questions in https://review.openstack.org/#/c/579201/19:57
*** heyongli has joined #openstack-nova19:57
jmloweif you don't specify a data pool all the image data will go back into the metadata pool which may not be sized properly and will probably be a nasty surprise for the operator19:58
*** priteau has quit IRC19:59
mnaserjmlowe: i don't know EC pools enough to help answer your questions19:59
mnaserim sorry19:59
*** shaohe_feng has quit IRC19:59
*** yamamoto has quit IRC20:00
jmlowemnaser: np, I'll have to play around and see if there is some way to discover from the parent image20:00
*** shaohe_feng has joined #openstack-nova20:00
*** heyongli has quit IRC20:07
*** heyongli has joined #openstack-nova20:07
*** felipemonteiro_ has quit IRC20:08
*** shaohe_feng has quit IRC20:09
*** shaohe_feng has joined #openstack-nova20:12
*** moshele has joined #openstack-nova20:16
*** edmondsw has joined #openstack-nova20:16
*** priteau has joined #openstack-nova20:16
*** priteau has quit IRC20:16
*** heyongli has quit IRC20:17
*** mgoddard has joined #openstack-nova20:17
*** heyongli has joined #openstack-nova20:18
*** shaohe_feng has quit IRC20:20
*** shaohe_feng has joined #openstack-nova20:20
dansmithmriedem: yeah I think that's fine20:21
*** edmondsw has quit IRC20:21
*** mgoddard has quit IRC20:25
*** jmlowe has quit IRC20:26
*** heyongli has quit IRC20:27
*** heyongli has joined #openstack-nova20:28
*** shaohe_feng has quit IRC20:30
*** gcb has joined #openstack-nova20:31
*** shaohe_feng has joined #openstack-nova20:31
*** MultipleCrashes has joined #openstack-nova20:35
*** heyongli has quit IRC20:38
*** heyongli has joined #openstack-nova20:38
*** shaohe_feng has quit IRC20:40
*** shaohe_feng has joined #openstack-nova20:42
efriedmriedem: Replied20:46
*** MultipleCrashes has quit IRC20:47
*** heyongli has quit IRC20:48
*** heyongli has joined #openstack-nova20:48
*** Sundar has joined #openstack-nova20:49
*** eharney has quit IRC20:50
mriedemalright +220:50
mriedemwho wants a +2, i'm in a giving mood20:50
*** shaohe_feng has quit IRC20:50
*** shaohe_feng has joined #openstack-nova20:51
efriedmriedem: You could +2/+A the one below it :)20:52
mriedemw/o looking20:52
efriedmriedem: I think that lets us close another bp20:52
mriedemyeah yeah i'm looking20:53
mriedemyou know what turns me on, process20:53
efriedKnew you couldn't resist that carrot20:54
Sundarefried: Please take a look at my unicast message about os-acc plugin design. Thanks.20:54
efriedSundar: Unicast where?20:54
SundarYour email20:55
efriedokay, I see it.20:55
efriedSundar: I plough through like 200-300 emails a day, sometimes with hours of lag.20:56
*** yamamoto has joined #openstack-nova20:56
*** heyongli has quit IRC20:58
*** heyongli has joined #openstack-nova20:58
*** yamamoto has quit IRC21:00
*** shaohe_feng has quit IRC21:01
*** shaohe_feng has joined #openstack-nova21:01
Sundarefried: NP.21:03
efriedSundar: I'm responding.21:03
efriedSundar: Though this might be a good discussion for the dev mailing list...21:03
SundarSure, NP. Let us take it there then21:07
*** heyongli has quit IRC21:08
*** heyongli has joined #openstack-nova21:09
*** shaohe_feng has quit IRC21:11
*** shaohe_feng has joined #openstack-nova21:13
*** priteau has joined #openstack-nova21:17
*** heyongli has quit IRC21:19
*** Guest53095 has quit IRC21:19
*** heyongli has joined #openstack-nova21:19
mriedemefried: comments in that cpu features traits patch21:20
mriedemkashyap should also look at that b/c of the extra cpu flags thing21:20
mriedemkashyap: https://review.openstack.org/#/c/560317/21:20
mriedemalso, that patch fails the pkvm ci21:20
*** shaohe_feng has quit IRC21:21
*** shaohe_feng has joined #openstack-nova21:22
*** priteau has quit IRC21:22
*** priteau has joined #openstack-nova21:23
mriedemppc64le?!?!21:23
mriedemwhere is tonyb when we need him21:23
efriedThanks Matt.21:26
*** heyongli has quit IRC21:29
*** heyongli has joined #openstack-nova21:29
*** namnh has joined #openstack-nova21:30
*** shaohe_feng has quit IRC21:31
*** shaohe_feng has joined #openstack-nova21:33
*** priteau has quit IRC21:34
*** namnh has quit IRC21:34
mriedembeyond that it seems ok21:35
mriedemefried: i'm surprised the zkvm ci doesn't blow up from that as well21:36
mriedemsince that ci job sets cpu_mode=none21:36
*** heyongli has quit IRC21:39
*** heyongli has joined #openstack-nova21:40
*** moshele has quit IRC21:41
*** shaohe_feng has quit IRC21:42
*** shaohe_feng has joined #openstack-nova21:43
*** heyongli has quit IRC21:49
*** heyongli has joined #openstack-nova21:50
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995621:50
*** med_ has joined #openstack-nova21:51
*** med_ has quit IRC21:51
*** med_ has joined #openstack-nova21:51
*** shaohe_feng has quit IRC21:52
*** yamamoto has joined #openstack-nova21:57
*** shaohe_feng has joined #openstack-nova21:57
*** tssurya has quit IRC21:58
*** nicolasbock has quit IRC22:00
*** heyongli has quit IRC22:00
*** heyongli has joined #openstack-nova22:00
*** burt has quit IRC22:01
*** yamamoto has quit IRC22:01
*** shaohe_feng has quit IRC22:02
*** lbragstad is now known as lbragstad_50322:02
*** avolkov has quit IRC22:03
*** shaohe_feng has joined #openstack-nova22:04
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995622:04
*** edmondsw has joined #openstack-nova22:05
*** rcernin has joined #openstack-nova22:09
*** edmondsw has quit IRC22:09
*** heyongli has quit IRC22:10
*** heyongli has joined #openstack-nova22:10
*** shaohe_feng has quit IRC22:12
*** mriedem is now known as mriedem_explosio22:14
*** mriedem_explosio is now known as mriedem_burnmark22:14
*** shaohe_feng has joined #openstack-nova22:14
*** Sundar has quit IRC22:18
*** heyongli has quit IRC22:20
*** heyongli has joined #openstack-nova22:21
*** shaohe_feng has quit IRC22:23
*** cdent has quit IRC22:23
*** hongbin has quit IRC22:25
*** shaohe_feng has joined #openstack-nova22:26
*** dtruong has quit IRC22:30
*** heyongli has quit IRC22:30
*** heyongli has joined #openstack-nova22:31
*** shaohe_feng has quit IRC22:33
*** shaohe_feng has joined #openstack-nova22:34
*** jmlowe has joined #openstack-nova22:36
*** vladikr has quit IRC22:37
*** vladikr has joined #openstack-nova22:37
*** heyongli has quit IRC22:41
*** heyongli has joined #openstack-nova22:41
openstackgerritJay Pipes proposed openstack/nova master: move lookup of provider from _new_allocations()  https://review.openstack.org/57992022:41
openstackgerritJay Pipes proposed openstack/nova master: placement: delete auto-created consumers on fail  https://review.openstack.org/57992122:41
*** namnh has joined #openstack-nova22:41
*** shaohe_feng has quit IRC22:43
*** shaohe_feng has joined #openstack-nova22:44
*** namnh_ has joined #openstack-nova22:46
*** namnh has quit IRC22:48
*** rcernin has quit IRC22:49
*** heyongli has quit IRC22:51
*** heyongli has joined #openstack-nova22:51
*** shaohe_feng has quit IRC22:53
*** shaohe_feng has joined #openstack-nova22:55
*** yamamoto has joined #openstack-nova22:57
*** heyongli has quit IRC23:01
*** heyongli has joined #openstack-nova23:01
*** rcernin has joined #openstack-nova23:02
*** yamamoto has quit IRC23:02
*** shaohe_feng has quit IRC23:04
*** shaohe_feng has joined #openstack-nova23:04
*** heyongli has quit IRC23:11
*** heyongli has joined #openstack-nova23:12
*** shaohe_feng has quit IRC23:14
*** shaohe_feng has joined #openstack-nova23:14
*** namnh_ has quit IRC23:15
*** heyongli has quit IRC23:22
*** heyongli has joined #openstack-nova23:22
*** antosh has joined #openstack-nova23:22
*** rcernin_ has joined #openstack-nova23:23
*** yassine has joined #openstack-nova23:23
*** yassine is now known as Guest3844423:24
*** shaohe_feng has quit IRC23:24
*** rcernin has quit IRC23:24
*** shaohe_feng has joined #openstack-nova23:25
*** chyka has quit IRC23:27
*** mriedem_burnmark has quit IRC23:30
*** heyongli has quit IRC23:32
*** heyongli has joined #openstack-nova23:32
*** shaohe_feng has quit IRC23:34
*** shaohe_feng has joined #openstack-nova23:35
*** mlavalle has quit IRC23:38
*** heyongli has quit IRC23:42
*** heyongli has joined #openstack-nova23:42
*** shaohe_feng has quit IRC23:45
*** shaohe_feng has joined #openstack-nova23:45
*** heyongli has quit IRC23:52
*** heyongli has joined #openstack-nova23:53
*** edmondsw has joined #openstack-nova23:53
*** shaohe_feng has quit IRC23:55
*** shaohe_feng has joined #openstack-nova23:57
*** edmondsw has quit IRC23:57
*** yamamoto has joined #openstack-nova23:58

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