Thursday, 2016-02-25

*** Jeffrey4l has joined #openstack-nova00:00
*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-nova00:01
*** Sukhdev has quit IRC00:03
*** rk4n has quit IRC00:04
*** mylu has joined #openstack-nova00:06
*** yamahata has joined #openstack-nova00:07
*** y_sawai has quit IRC00:07
*** john51 has quit IRC00:10
*** diogogmt has quit IRC00:10
*** john51 has joined #openstack-nova00:11
*** yonglihe has quit IRC00:12
*** sdague has quit IRC00:15
*** eil397 has joined #openstack-nova00:16
*** annasort has joined #openstack-nova00:18
*** ljxiash has joined #openstack-nova00:18
*** nic has quit IRC00:21
*** ljxiash has quit IRC00:23
*** zenoway has joined #openstack-nova00:23
*** Sukhdev has joined #openstack-nova00:24
*** ivase_ has quit IRC00:24
*** ivase has joined #openstack-nova00:25
*** Sukhdev has quit IRC00:27
*** zenoway has quit IRC00:28
openstackgerritShaoHe Feng proposed openstack/nova: Add new APIs and deprecate old API for migrations  https://review.openstack.org/25877100:29
*** ivase_ has joined #openstack-nova00:31
*** qeelee has joined #openstack-nova00:31
*** ivase has quit IRC00:31
*** bjornar__ has quit IRC00:32
*** Sukhdev has joined #openstack-nova00:35
*** ivase has joined #openstack-nova00:36
*** ivase_ has quit IRC00:37
openstackgerritJay Pipes proposed openstack/nova: Use new inventory schema in compute_node_get_all()  https://review.openstack.org/28448400:38
jaypipesbauzas, dansmith, alaski: ^^00:38
*** mylu has quit IRC00:42
*** ivase_ has joined #openstack-nova00:42
*** ivase has quit IRC00:43
*** Sukhdev has quit IRC00:44
*** zz_dimtruck is now known as dimtruck00:45
*** zenoway has joined #openstack-nova00:48
*** ivase has joined #openstack-nova00:48
*** ildikov has quit IRC00:48
*** y_sawai has joined #openstack-nova00:48
*** ivase_ has quit IRC00:49
*** y_sawai_ has joined #openstack-nova00:51
*** ildikov has joined #openstack-nova00:52
*** zenoway has quit IRC00:52
*** y_sawai has quit IRC00:53
*** ivase_ has joined #openstack-nova00:54
*** mylu has joined #openstack-nova00:54
*** ivase has quit IRC00:55
*** y_sawai_ has quit IRC00:56
*** delatte has joined #openstack-nova00:58
*** ivase has joined #openstack-nova01:00
*** dimtruck is now known as zz_dimtruck01:00
*** ivase_ has quit IRC01:00
*** delattec has quit IRC01:00
*** haomaiwang has quit IRC01:01
*** Sukhdev has joined #openstack-nova01:01
*** haomaiwang has joined #openstack-nova01:01
*** eil397 has quit IRC01:03
*** ctrath has joined #openstack-nova01:03
*** ivase_ has joined #openstack-nova01:05
*** ivase has quit IRC01:06
*** jdillaman has quit IRC01:08
*** Sukhdev has quit IRC01:10
*** ivase has joined #openstack-nova01:11
*** ivase_ has quit IRC01:12
*** mylu has quit IRC01:13
*** y_sawai has joined #openstack-nova01:14
*** ivase_ has joined #openstack-nova01:17
*** aorourke is now known as aorourke-afk01:17
openstackgerritSean Dague proposed openstack/nova: complete the removal of api_version from rest client parameters  https://review.openstack.org/28425701:18
*** ivase has quit IRC01:18
*** EinstCrazy has joined #openstack-nova01:20
*** ducttape_ has joined #openstack-nova01:20
*** jdillaman has joined #openstack-nova01:21
openstackgerritSean Dague proposed openstack/nova: fix usage of opportunistic test cases with enginefacade  https://review.openstack.org/28372801:22
openstackgerritSean Dague proposed openstack/nova: add regression test for bug #1538011  https://review.openstack.org/28336401:22
openstackbug 1538011 in OpenStack Compute (nova) "Aggregate metadata keys are not case sensitive" [High,In progress] https://launchpad.net/bugs/1538011 - Assigned to Augustina Ragwitz (auggy)01:22
openstackgerritSean Dague proposed openstack/nova: add regression test for bug #1541691  https://review.openstack.org/28326101:22
openstackbug 1541691 in OpenStack Compute (nova) "server boot with leading and trailing white spaces in name, displays weird error message" [High,In progress] https://launchpad.net/bugs/1541691 - Assigned to Alex Xu (xuhj)01:22
*** haomaiwang has quit IRC01:22
*** ivase has joined #openstack-nova01:23
openstackgerritSean Dague proposed openstack/nova: reduce pep8 requirements to just hacking  https://review.openstack.org/26709701:23
*** ivase_ has quit IRC01:23
*** jaypipes has quit IRC01:24
*** vladikr has quit IRC01:27
*** ivase_ has joined #openstack-nova01:28
*** thorst has joined #openstack-nova01:28
*** ivase has quit IRC01:29
*** ducttape_ has quit IRC01:30
*** mtanino has quit IRC01:32
*** zhangjn has quit IRC01:32
*** abitha has quit IRC01:32
*** Guest35189 has quit IRC01:32
*** zhangyufei has joined #openstack-nova01:33
*** mutiwu has joined #openstack-nova01:33
*** thorst has quit IRC01:34
*** mutiwu is now known as Guest398101:34
*** ivase has joined #openstack-nova01:34
*** thorst has joined #openstack-nova01:34
*** ivase_ has quit IRC01:35
*** edtubill has joined #openstack-nova01:36
*** zhangjn has joined #openstack-nova01:37
*** ctrath1 has joined #openstack-nova01:37
*** gongysh has joined #openstack-nova01:38
*** thorst has quit IRC01:39
*** ivase_ has joined #openstack-nova01:40
*** ctrath has quit IRC01:41
*** ivase has quit IRC01:41
*** ljxiash has joined #openstack-nova01:44
*** vladikr has joined #openstack-nova01:44
*** zzzeek has quit IRC01:45
*** zzzeek has joined #openstack-nova01:45
*** ivase has joined #openstack-nova01:46
*** ivase_ has quit IRC01:46
*** smatzek__ has joined #openstack-nova01:47
*** tangchen_ has joined #openstack-nova01:47
*** zzzeek has quit IRC01:47
tangchen_Hi alex_xu, would you please help to take a look at this patch ?  https://review.openstack.org/#/c/280120/01:47
*** Guest75 has quit IRC01:47
*** Guest3981 has quit IRC01:48
*** mutiwu_ has joined #openstack-nova01:48
*** zzzeek has joined #openstack-nova01:50
*** ivase_ has joined #openstack-nova01:51
*** dims has quit IRC01:52
*** zzzeek has quit IRC01:52
*** ivase has quit IRC01:52
*** zzzeek has joined #openstack-nova01:54
*** tangchen_ has quit IRC01:57
*** ivase has joined #openstack-nova01:57
*** ivase_ has quit IRC01:58
*** haomaiwang has joined #openstack-nova01:59
*** haomaiwang has quit IRC02:01
*** haomaiwang has joined #openstack-nova02:01
*** zhangyufei has quit IRC02:01
*** ijw has quit IRC02:02
*** zenoway has joined #openstack-nova02:02
*** ducttape_ has joined #openstack-nova02:03
*** ivase_ has joined #openstack-nova02:03
*** ivase has quit IRC02:04
*** Sukhdev has joined #openstack-nova02:05
*** diogogmt has joined #openstack-nova02:05
*** Guest75 has joined #openstack-nova02:06
*** zenoway has quit IRC02:08
*** Sukhdev has quit IRC02:09
*** ivase has joined #openstack-nova02:09
*** mutiwu_ has quit IRC02:09
*** ivase_ has quit IRC02:10
*** lyanchih has joined #openstack-nova02:11
openstackgerritShaoHe Feng proposed openstack/nova: Add new APIs and deprecate old API for migrations  https://review.openstack.org/25877102:12
*** andreykurilin__ has quit IRC02:13
*** tangchen has joined #openstack-nova02:15
*** ivase_ has joined #openstack-nova02:15
*** tangchen has quit IRC02:15
*** ivase has quit IRC02:15
*** erlon has quit IRC02:16
claudiubmikal: hello. Sorry to bug you, but if you have one free moment to review a 13 LOC patch, it would be awesome. :D https://review.openstack.org/#/c/258614/502:17
*** gongysh has quit IRC02:18
*** penick has quit IRC02:19
*** ivase has joined #openstack-nova02:21
*** ivase_ has quit IRC02:21
*** Guest75 has quit IRC02:22
*** vilobhmm11 has quit IRC02:24
*** hoangcx has joined #openstack-nova02:25
*** vilobhmm11 has joined #openstack-nova02:25
*** houming has joined #openstack-nova02:26
*** ivase_ has joined #openstack-nova02:26
*** ivase has quit IRC02:27
*** Bjoern_ has joined #openstack-nova02:27
*** armax has joined #openstack-nova02:28
*** amotoki has joined #openstack-nova02:29
*** zenoway has joined #openstack-nova02:30
*** shaohe_feng_ has quit IRC02:30
*** zz_dimtruck is now known as dimtruck02:32
*** ivase has joined #openstack-nova02:32
*** ivase_ has quit IRC02:33
*** vilobhmm11 has quit IRC02:35
*** zenoway has quit IRC02:35
*** vilobhmm11 has joined #openstack-nova02:36
*** Guest75 has joined #openstack-nova02:36
*** vilobhmm11 has quit IRC02:37
*** tojuvone_ has quit IRC02:37
*** ivase_ has joined #openstack-nova02:38
*** ivase has quit IRC02:38
mikalclaudiub: I'm in a meeting, but I'll try to get to it today02:39
claudiubsure, thanks. :)02:39
*** zhangyufei has joined #openstack-nova02:42
*** ducttape_ has quit IRC02:43
*** ducttape_ has joined #openstack-nova02:43
*** ivase has joined #openstack-nova02:43
*** ivase_ has quit IRC02:44
*** tojuvone has joined #openstack-nova02:44
*** yamahata has quit IRC02:46
*** links has joined #openstack-nova02:47
*** smatzek__ has quit IRC02:49
*** ivase_ has joined #openstack-nova02:49
*** ivase has quit IRC02:51
*** apoorvad has quit IRC02:52
*** apoorvad has joined #openstack-nova02:53
*** zenoway has joined #openstack-nova02:54
*** ivase has joined #openstack-nova02:55
*** ivase_ has quit IRC02:55
*** gongysh has joined #openstack-nova02:56
*** apoorvad has quit IRC02:57
*** y_sawai has quit IRC02:57
*** y_sawai has joined #openstack-nova02:58
*** edtubill has quit IRC02:59
*** zenoway has quit IRC02:59
*** haomaiwang has quit IRC03:01
*** ivase_ has joined #openstack-nova03:01
*** haomaiwang has joined #openstack-nova03:01
*** ivase has quit IRC03:02
*** gcb has quit IRC03:02
*** lyanchih has quit IRC03:03
*** gcb has joined #openstack-nova03:03
*** liusheng has left #openstack-nova03:04
*** boris-42 has quit IRC03:04
*** ctrath1 has quit IRC03:06
*** ducttape_ has quit IRC03:06
*** ivase has joined #openstack-nova03:07
*** edmondsw has quit IRC03:07
*** gcb has quit IRC03:07
*** ivase_ has quit IRC03:07
*** gcb has joined #openstack-nova03:08
*** lyanchih has joined #openstack-nova03:10
*** ducttape_ has joined #openstack-nova03:12
*** ivase_ has joined #openstack-nova03:12
*** ivase has quit IRC03:13
*** Guest75 has quit IRC03:16
*** lyanchih has quit IRC03:16
*** ivase has joined #openstack-nova03:18
*** zenoway has joined #openstack-nova03:18
*** ivase_ has quit IRC03:19
*** amotoki has quit IRC03:19
*** lyanchih has joined #openstack-nova03:23
*** zenoway has quit IRC03:23
*** diogogmt has quit IRC03:23
*** ivase_ has joined #openstack-nova03:25
*** ivase has quit IRC03:25
*** diogogmt has joined #openstack-nova03:26
*** ducttape_ has quit IRC03:26
*** salv-orlando has joined #openstack-nova03:27
*** salv-orlando has quit IRC03:28
*** amotoki has joined #openstack-nova03:30
*** ivase has joined #openstack-nova03:30
*** salv-orl_ has quit IRC03:30
*** baoli_ has quit IRC03:31
*** ivase_ has quit IRC03:31
*** links has quit IRC03:34
*** amotoki has quit IRC03:35
*** ivase_ has joined #openstack-nova03:36
*** ivase has quit IRC03:36
*** ljxiash has quit IRC03:38
*** ljxiash has joined #openstack-nova03:38
*** vladikr has quit IRC03:38
*** baoli has joined #openstack-nova03:40
*** amotoki has joined #openstack-nova03:40
*** ducttape_ has joined #openstack-nova03:41
*** ivase has joined #openstack-nova03:41
*** ivase_ has quit IRC03:42
*** baoli has quit IRC03:42
*** zenoway has joined #openstack-nova03:43
*** baoli has joined #openstack-nova03:43
*** ljxiash has quit IRC03:43
*** toure_ is now known as toure03:43
*** diogogmt has quit IRC03:43
*** y_sawai has quit IRC03:45
*** y_sawai has joined #openstack-nova03:45
*** zenoway has quit IRC03:47
*** baoli has quit IRC03:48
*** Bjoern_ has quit IRC03:51
*** vladikr has joined #openstack-nova03:51
*** ducttape_ has quit IRC03:53
*** mingyu has quit IRC03:53
*** mingyu has joined #openstack-nova03:53
*** claudiub has quit IRC03:54
*** ducttape_ has joined #openstack-nova03:56
*** zenoway has joined #openstack-nova03:56
*** qeelee has quit IRC04:00
*** zenoway has quit IRC04:00
*** haomaiwang has quit IRC04:01
*** amotoki has quit IRC04:01
*** haomaiwa_ has joined #openstack-nova04:01
*** hoangcx has quit IRC04:02
*** mingyu has quit IRC04:02
*** hoangcx has joined #openstack-nova04:03
*** mingyu has joined #openstack-nova04:04
*** mubirru has joined #openstack-nova04:05
*** shyama has joined #openstack-nova04:10
*** zenoway has joined #openstack-nova04:10
*** amotoki has joined #openstack-nova04:13
*** zenoway has quit IRC04:15
*** ducttape_ has quit IRC04:24
openstackgerritDongcan Ye proposed openstack/nova: VMware: Fix getting wrong image cahe folder  https://review.openstack.org/28452904:25
*** gcb has quit IRC04:28
*** gcb has joined #openstack-nova04:28
*** ljxiash has joined #openstack-nova04:28
*** salv-orlando has joined #openstack-nova04:29
*** links has joined #openstack-nova04:29
*** ducttape_ has joined #openstack-nova04:29
*** haomaiwa_ has quit IRC04:30
*** _RuiChen has quit IRC04:32
*** RuiChen has joined #openstack-nova04:33
*** ljxiash has quit IRC04:34
*** ljxiash has joined #openstack-nova04:34
*** achanda has quit IRC04:35
*** zenoway has joined #openstack-nova04:36
*** ducttape_ has quit IRC04:37
*** lyanchih has quit IRC04:37
*** ljxiash has quit IRC04:39
*** salv-orlando has quit IRC04:39
*** haomaiwa_ has joined #openstack-nova04:39
*** Marga__ has joined #openstack-nova04:40
*** zenoway has quit IRC04:41
*** Marga__ has quit IRC04:44
*** tangchen has joined #openstack-nova04:44
*** Marga_ has quit IRC04:44
*** mingyu has quit IRC04:49
*** mingyu has joined #openstack-nova04:51
*** armax has quit IRC04:51
*** mingyu has quit IRC04:52
*** y_sawai_ has joined #openstack-nova04:53
*** y_sawai has quit IRC04:54
*** achanda has joined #openstack-nova04:55
*** Marga_ has joined #openstack-nova04:56
*** qeelee has joined #openstack-nova04:59
*** Marga_ has quit IRC05:00
*** haomaiwa_ has quit IRC05:01
*** zenoway has joined #openstack-nova05:01
*** Marga_ has joined #openstack-nova05:01
*** haomaiwang has joined #openstack-nova05:01
*** Marga_ has quit IRC05:01
*** Marga_ has joined #openstack-nova05:01
*** Marga_ has quit IRC05:02
*** Marga_ has joined #openstack-nova05:02
*** y_sawai_ has quit IRC05:02
*** dimtruck is now known as zz_dimtruck05:03
*** y_sawai has joined #openstack-nova05:04
*** garyk has joined #openstack-nova05:05
*** zenoway has quit IRC05:05
*** yonglihe has joined #openstack-nova05:05
*** qeelee has quit IRC05:10
*** belmoreira has joined #openstack-nova05:12
*** ljxiash has joined #openstack-nova05:14
*** Fdaisuke has quit IRC05:15
*** mingyu has joined #openstack-nova05:16
*** mingyu has quit IRC05:16
*** Fdaisuke has joined #openstack-nova05:17
*** shyama has quit IRC05:24
*** shyama_ has joined #openstack-nova05:24
*** shyama_ is now known as shyama05:24
*** zenoway has joined #openstack-nova05:25
*** lyanchih has joined #openstack-nova05:26
*** xiexs has quit IRC05:28
*** zenoway has quit IRC05:29
*** mingyu has joined #openstack-nova05:30
*** mingyu has quit IRC05:31
*** mingyu has joined #openstack-nova05:31
*** tangchen has quit IRC05:31
*** y_sawai_ has joined #openstack-nova05:32
*** y_sawai has quit IRC05:33
*** irenab has quit IRC05:43
*** y_sawai_ has quit IRC05:44
*** liverpooler has quit IRC05:44
*** y_sawai has joined #openstack-nova05:45
*** zenoway has joined #openstack-nova05:49
*** qeelee has joined #openstack-nova05:49
*** tangchen has joined #openstack-nova05:52
*** dims has joined #openstack-nova05:52
*** zenoway has quit IRC05:53
openstackgerritShaoHe Feng proposed openstack/nova: Add new APIs and deprecate old API for migrations  https://review.openstack.org/25877105:57
*** nkrinner has joined #openstack-nova05:58
*** salv-orlando has joined #openstack-nova05:58
*** doug-fish has joined #openstack-nova06:00
*** haomaiwang has quit IRC06:01
*** haomaiwang has joined #openstack-nova06:01
*** edtubill has joined #openstack-nova06:03
*** gongysh has quit IRC06:03
*** salv-orlando has quit IRC06:03
*** ekuris has joined #openstack-nova06:03
*** harlowja_at_home has quit IRC06:11
*** achinm has joined #openstack-nova06:11
*** harlowja_at_home has joined #openstack-nova06:13
*** tangchen has quit IRC06:14
*** zenoway has joined #openstack-nova06:15
*** n0ano has quit IRC06:15
*** y_sawai has quit IRC06:16
*** y_sawai has joined #openstack-nova06:17
*** gcb has quit IRC06:18
*** zenoway has quit IRC06:21
*** y_sawai has quit IRC06:25
*** y_sawai has joined #openstack-nova06:26
*** gcb has joined #openstack-nova06:30
*** harlowja_at_home has quit IRC06:33
*** harlowja_at_home has joined #openstack-nova06:33
*** aagrawal has joined #openstack-nova06:37
*** irenab has joined #openstack-nova06:37
*** edtubill has quit IRC06:38
*** tyagiprince has joined #openstack-nova06:41
*** zenoway has joined #openstack-nova06:43
*** dims has quit IRC06:43
*** mrhillsman has quit IRC06:45
*** achanda_ has joined #openstack-nova06:45
*** dave-mccowan has quit IRC06:45
*** achanda has quit IRC06:46
*** zenoway has quit IRC06:47
*** ljxiash has quit IRC06:47
*** ivase_ has joined #openstack-nova06:48
*** josecastroleon has joined #openstack-nova06:49
*** ivase has quit IRC06:49
*** fawadkhaliq has joined #openstack-nova06:50
*** qeelee_ has joined #openstack-nova06:50
*** qeelee has quit IRC06:50
*** qeelee_ is now known as qeelee06:50
*** hoangcx_ has joined #openstack-nova06:51
*** zenoway has joined #openstack-nova06:52
*** hoangcx has quit IRC06:52
*** belmoreira has quit IRC06:53
*** doug-fish has quit IRC06:54
*** ivase has joined #openstack-nova06:54
openstackgerritChris Friesen proposed openstack/nova: check max_net_count against min_count when booting  https://review.openstack.org/28386106:54
*** harlowja_at_home has quit IRC06:54
*** ljxiash has joined #openstack-nova06:55
*** ivase_ has quit IRC06:55
*** gongysh has joined #openstack-nova06:56
*** zzzeek has quit IRC06:56
*** zzzeek has joined #openstack-nova06:59
*** ivase_ has joined #openstack-nova07:00
*** haomaiwang has quit IRC07:01
*** ivase has quit IRC07:01
*** haomaiwang has joined #openstack-nova07:01
*** salv-orlando has joined #openstack-nova07:02
openstackgerritBartosz Górski proposed openstack/python-novaclient: Adds missing internationalization for help message  https://review.openstack.org/28445807:05
*** codebauss has joined #openstack-nova07:06
*** ivase has joined #openstack-nova07:06
*** ivase_ has quit IRC07:07
*** salv-orlando has quit IRC07:07
*** codebauss is now known as mrhillsman07:07
*** mjura has joined #openstack-nova07:10
*** zzzeek has quit IRC07:10
openstackgerritFeng Xi Yan proposed openstack/nova: <VMWARE> Add power state cache for host init and power state sync.  https://review.openstack.org/24200607:11
*** ivase_ has joined #openstack-nova07:12
*** ivase has quit IRC07:13
*** paul-carlton1 has joined #openstack-nova07:14
*** tomoiaga has joined #openstack-nova07:16
openstackgerritTakashi NATSUME proposed openstack/nova: Remove APIRouterV3  https://review.openstack.org/28455907:17
*** ivase has joined #openstack-nova07:17
openstackgerritgaryk proposed openstack/nova: VMware: remove deprecation warnings from oslo_versionedobjects  https://review.openstack.org/28428607:18
*** ivase_ has quit IRC07:18
*** zenoway has quit IRC07:19
*** lpetrut has joined #openstack-nova07:19
*** zenoway has joined #openstack-nova07:19
*** tomoiaga has quit IRC07:21
*** ivase_ has joined #openstack-nova07:23
*** zenoway has quit IRC07:24
*** scheuran has joined #openstack-nova07:24
*** ivase has quit IRC07:24
*** zenoway has joined #openstack-nova07:24
*** salv-orlando has joined #openstack-nova07:25
*** salv-orl_ has joined #openstack-nova07:26
*** salv-orlando has quit IRC07:26
*** zhangyufei has quit IRC07:27
*** mingyu has quit IRC07:28
*** ivase_ has quit IRC07:29
*** ivase has joined #openstack-nova07:30
*** ociuhandu has joined #openstack-nova07:30
*** ociuhandu has quit IRC07:31
*** zhangyufei has joined #openstack-nova07:32
*** ociuhandu has joined #openstack-nova07:32
*** cfriesen has quit IRC07:32
*** liverpooler has joined #openstack-nova07:32
*** ivase_ has joined #openstack-nova07:35
*** ljxiash_ has joined #openstack-nova07:35
*** ljxiash has quit IRC07:36
*** ivase has quit IRC07:36
*** tyagiprince1 has joined #openstack-nova07:36
*** tyagiprince has quit IRC07:38
*** tyagiprince1 is now known as tyagiprince07:38
*** ivase has joined #openstack-nova07:41
*** ivase_ has quit IRC07:41
*** zul has quit IRC07:41
*** ivase_ has joined #openstack-nova07:46
*** ivase has quit IRC07:47
*** zul has joined #openstack-nova07:47
*** zhangyufei has quit IRC07:48
*** ociuhandu has quit IRC07:49
*** mpavone has joined #openstack-nova07:49
*** ljxiash_ has quit IRC07:49
*** belmoreira has joined #openstack-nova07:51
*** fawadkhaliq has quit IRC07:51
*** ivase has joined #openstack-nova07:52
*** ljxiash has joined #openstack-nova07:52
*** ivase_ has quit IRC07:54
*** moshele has joined #openstack-nova07:57
*** belmoreira has quit IRC07:58
*** ivase_ has joined #openstack-nova07:58
*** ivase has quit IRC07:58
*** belmoreira has joined #openstack-nova08:00
*** haomaiwang has quit IRC08:01
*** haomaiwa_ has joined #openstack-nova08:01
*** ivase_ has quit IRC08:04
*** ivase has joined #openstack-nova08:04
*** rdopiera has joined #openstack-nova08:05
*** belmoreira has quit IRC08:06
*** matrohon has joined #openstack-nova08:08
*** ivase_ has joined #openstack-nova08:09
*** ivase has quit IRC08:10
*** rk4n has joined #openstack-nova08:11
openstackgerritTakashi NATSUME proposed openstack/nova: Remove unused methods in nova/utils.py  https://review.openstack.org/28458008:13
*** lpetrut has quit IRC08:13
*** ivase has joined #openstack-nova08:15
*** ivase_ has quit IRC08:16
*** erno has joined #openstack-nova08:17
*** Oku_OS has quit IRC08:18
*** browne has quit IRC08:19
*** y_sawai has quit IRC08:19
*** tyagiprince has quit IRC08:19
*** rk4n has quit IRC08:20
*** y_sawai has joined #openstack-nova08:20
*** ivase_ has joined #openstack-nova08:21
*** ivase has quit IRC08:21
*** rk4n has joined #openstack-nova08:22
*** jichen has joined #openstack-nova08:23
*** y_sawai has quit IRC08:24
*** ivase has joined #openstack-nova08:27
*** ivase_ has quit IRC08:28
*** lyanchih has quit IRC08:29
openstackgerritEli Qiao proposed openstack/nova: Allow block_migration and disk_over_commit to be None  https://review.openstack.org/25410208:31
openstackgerritEli Qiao proposed openstack/nova: API: Improve os-migrateLive input parameters  https://review.openstack.org/25931908:31
*** ivase_ has joined #openstack-nova08:32
*** ivase has quit IRC08:33
*** PaulMurray has joined #openstack-nova08:34
PaulMurraygood morning08:35
*** mingyu has joined #openstack-nova08:35
*** ivase_ has quit IRC08:38
*** ivase has joined #openstack-nova08:39
*** lyanchih has joined #openstack-nova08:40
bauzasgood morning Nova08:42
*** ivase_ has joined #openstack-nova08:44
*** ivase has quit IRC08:44
*** Mackie2k has joined #openstack-nova08:44
*** rk4n has quit IRC08:47
*** ivase has joined #openstack-nova08:50
*** ivase_ has quit IRC08:50
*** ihrachys has joined #openstack-nova08:51
*** danpb has joined #openstack-nova08:51
*** boris-42 has joined #openstack-nova08:52
*** amotoki has quit IRC08:53
*** tomoiaga has joined #openstack-nova08:53
*** ivase has quit IRC08:56
*** ivase_ has joined #openstack-nova08:56
*** hoangcx_ has quit IRC08:59
*** mnestratov has joined #openstack-nova09:00
*** hoangcx has joined #openstack-nova09:00
*** achanda_ has quit IRC09:00
*** haomaiwa_ has quit IRC09:01
*** haomaiwang has joined #openstack-nova09:01
*** jistr has joined #openstack-nova09:01
*** ivase has joined #openstack-nova09:01
*** claudiub has joined #openstack-nova09:02
*** ivase_ has quit IRC09:03
*** achanda has joined #openstack-nova09:04
bauzasoh snap, I used a wrong channel...09:05
*** paul-carlton1 has quit IRC09:07
*** ivase_ has joined #openstack-nova09:07
*** ivase has quit IRC09:08
*** mvk has joined #openstack-nova09:09
*** yassine__ has joined #openstack-nova09:10
*** dedery has joined #openstack-nova09:10
*** amotoki has joined #openstack-nova09:10
*** Oku_OS has joined #openstack-nova09:10
*** samueldmq1 has joined #openstack-nova09:11
*** ivase has joined #openstack-nova09:13
*** ivase_ has quit IRC09:13
PaulMurrayshaohe_feng, hi09:15
*** samueldmq1 has quit IRC09:15
*** ildikov has quit IRC09:15
PaulMurrayshaohe_feng, I'm going off line for half an hour, but will be back. Wanted to talk about the in progress migrations09:16
* PaulMurray AFK 30 mins09:16
*** fawadkhaliq has joined #openstack-nova09:17
*** achanda has quit IRC09:19
*** ivase_ has joined #openstack-nova09:19
*** ivase has quit IRC09:20
*** claudiub|2 has joined #openstack-nova09:20
*** ndipanov has joined #openstack-nova09:21
*** achanda has joined #openstack-nova09:24
*** claudiub has quit IRC09:24
*** ivase has joined #openstack-nova09:24
*** ivase_ has quit IRC09:25
*** claudiub has joined #openstack-nova09:26
*** lucas-hungry is now known as lucasagomes09:26
*** paul-carlton1 has joined #openstack-nova09:26
*** salv-orlando has joined #openstack-nova09:27
*** sudipto has joined #openstack-nova09:27
*** PaulMurray has quit IRC09:28
*** claudiub|2 has quit IRC09:29
*** fawadkhaliq has quit IRC09:29
*** achanda has quit IRC09:29
*** ivase has quit IRC09:30
*** salv-orl_ has quit IRC09:30
*** ivase has joined #openstack-nova09:30
*** ivase_ has joined #openstack-nova09:36
*** ivase has quit IRC09:37
jed56hello nova team09:37
jed56can you tell me if it is possible to retrieve from "nova" if a particular  "nova-compute " is configured with shared storage or block migration ?09:37
danpbthat's not something we expose right now09:38
danpbin fact nova itself is pretty dumb and doesn't really know if it has shared storage or not09:38
jed56so if we want to migrate a VM we need to try first in shared storage and if it failed retry we block migration09:39
*** ljxiash has quit IRC09:40
*** ljxiash has joined #openstack-nova09:40
danpbyeah its pretty sucky situation - basically we expect the cloud admin to remember/know whether they've used shared storage or not and pick the right migraiton option09:41
*** liverpooler has quit IRC09:41
jed56do you know if we can have two configurations at the same time : - shared storage and block migration ?09:42
danpbin the future we're planning to fix this so nova knows it has shared storage or not, and will automatically do block migration if it needs it09:42
*** liverpooler has joined #openstack-nova09:42
*** ivase has joined #openstack-nova09:42
*** mingyu has quit IRC09:42
danpbjed56: sure you can, but it makes life harder for you to predict whether you need block migration or not for any given vm09:42
danpbso i'd tend to recommend you pick or the other right now09:42
*** salv-orlando has quit IRC09:43
jed56hmm okay09:43
*** ivase_ has quit IRC09:43
johnthetubaguyjed56: have you seen this spec we are working on: http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/making_live_migration_api_friendly.html09:43
johnthetubaguythe idea is to add a block_migration = "auto" flag this release09:43
*** ljxiash_ has joined #openstack-nova09:44
*** tyagiprince has joined #openstack-nova09:44
jed56ahh this is a great news09:44
jed56:)09:44
*** efoley has joined #openstack-nova09:44
johnthetubaguyjed56: so the initial version might not do exactly what you want, depends on your deployment scenrio09:44
johnthetubaguybut as danpb said, we are trying to fix up in Newton our tracking of shared storage, with generic-resource-pools spec09:45
*** ljxiash has quit IRC09:45
*** Mackie2k has quit IRC09:45
johnthetubaguyjed56: for our admins, we only use block_migration=true, so this stops them having to remember the correct value (if the pass false thy just see a failed live-migration, becuase nova says false was a bad value, which is dumb...)09:46
*** lyanchih has quit IRC09:46
johnthetubaguybut if you have several pools of storage, and you don't pick your destination host, it can all go a bit funny right now, but baby steps forward here09:46
*** vincentfrancoise has joined #openstack-nova09:46
*** ivase_ has joined #openstack-nova09:47
jed56woah...09:48
pkholkin_dims: hi Dims, what was the question?09:48
*** ivase has quit IRC09:48
*** haomaiw__ has joined #openstack-nova09:48
*** haomaiwang has quit IRC09:49
jed56thanks have a nice day09:49
*** PaulMurray has joined #openstack-nova09:50
* PaulMurray is back09:51
*** qeelee has quit IRC09:51
*** grassy is now known as davechen_afk09:53
*** ivase has joined #openstack-nova09:54
*** sfinucan has joined #openstack-nova09:54
johnthetubaguypkholkin_: I have a feeling he is in the west coast of the US this week09:54
*** ivase_ has quit IRC09:55
johnthetubaguyPaulMurray: hows live-migrate shaping up?09:55
pkholkin_johnthetubaguy: yes, John, I feel the same)09:55
pkholkin_we had some question, will wait09:55
pkholkin_he*09:55
PaulMurrayjohnthetubaguy, need to get https://review.openstack.org/#/c/258771/32 sorted out today if possible09:56
PaulMurrayjohnthetubaguy, that has abort blocked behind it09:56
PaulMurrayjohnthetubaguy, then a chain of 4 patches for novaclient has just been unblocked by the first patch - needs approval09:57
johnthetubaguyThats is shaohe_feng's patch I guess, just replying to the ML post09:57
PaulMurrayyes, I was looking at that - but go aheaad - I think there is actually little to do there09:58
PaulMurrayBTW - the CI failures seem to be trouble in setup - is something going on at the moment?09:58
PaulMurraylots of failures in the check queue09:58
*** ivase_ has joined #openstack-nova09:59
openstackgerritStephen Finucane proposed openstack/nova-specs: Rework policies for virt-driver CPU thread pinning  https://review.openstack.org/24419810:00
openstackgerritStephen Finucane proposed openstack/nova-specs: Move thread policy features between specs  https://review.openstack.org/24485710:00
*** ivase has quit IRC10:00
johnthetubaguyPaulMurray: good question, not sure, bauzas usually knows about those issues before I spot them10:00
*** derekh has joined #openstack-nova10:00
*** haomaiw__ has quit IRC10:01
*** ildikov has joined #openstack-nova10:01
bauzasjohnthetubaguy: PaulMurray: I haven't seen any CI problem AFAIK for today10:01
bauzasPaulMurray: link ?10:01
*** haomaiwang has joined #openstack-nova10:01
*** mvk has quit IRC10:01
*** lyanchih has joined #openstack-nova10:01
* bauzas is just rushing to deliver the last Mitaka bits for check-dests (coldmigrate/resize and rebuild)10:02
sfinucandanpb: mikal: ndipanov: Appreciate that you're all busy today, but it would be awesome if ye could get this bp rework in today. It's technically a doc change, and a change in the admin guide is blocked by it. Please and thank you :) https://review.openstack.org/24485710:02
ndipanovsfinucan, I happen to be rather free today, but no +2 on specs... tough luck :)10:04
ndipanovsfinucan, just kidding I am not free at all :(10:04
ndipanovsfinucan, btw did you ever get a chance to try the shelve fixes?10:04
sfinucanndipanov: I'm doing that today, funnily enough :)10:04
*** ivase has joined #openstack-nova10:05
sfinucanspent the week triaging numa/pinning bugs10:05
*** ivase_ has quit IRC10:05
*** efoley has quit IRC10:06
johnthetubaguysfinucan: did you only drop those "s" bits since mikal added his +2?10:06
*** efoley_ has joined #openstack-nova10:06
*** vincentfrancoise has left #openstack-nova10:06
*** hoangcx has quit IRC10:06
sfinucanjohnthetubaguy: That, and removed a dependency on a bug that's already been fixed by ndipanov10:06
*** EinstCrazy has quit IRC10:07
* johnthetubaguy scrolls down10:07
johnthetubaguyah yes10:07
*** daemontool has joined #openstack-nova10:07
*** EinstCrazy has joined #openstack-nova10:07
*** ljxiash has joined #openstack-nova10:08
*** ljxiash_ has quit IRC10:10
*** ivase has quit IRC10:11
*** ivase has joined #openstack-nova10:11
johnthetubaguysfinucan: so that should unblock the doc work, but could you do a follow up to that last one, we probably should add a redirect for that rename: https://review.openstack.org/#/c/24485710:12
*** EinstCrazy has quit IRC10:12
wznoinskhi all10:13
wznoinskcould someone have a look at my  https://review.openstack.org/#/c/280299/ , it's  a fix do api-guide in nova repository, I'm wondering does anybody care or not (cause docs are planned to be somewhere else or else)10:14
*** rk4n has joined #openstack-nova10:16
sfinucanjohnthetubaguy: Sure10:16
sfinucanOr should I revert the file rename?10:16
*** ivase_ has joined #openstack-nova10:16
johnthetubaguysfinucan: a redirect is fine, I quite like the rename10:17
johnthetubaguysfinucan: we have a file of redirects in place already, so should just be adding an extra one10:17
*** ivase has quit IRC10:18
johnthetubaguynormally used for the approved->implemented kind of thing10:18
openstackgerritStephen Finucane proposed openstack/nova-specs: Add redirect for renamed CPU thread policy spec  https://review.openstack.org/28463210:18
sfinucanjohnthetubaguy: Sweet. Done10:19
*** qeelee has joined #openstack-nova10:19
*** lyanchih has quit IRC10:19
johnthetubaguysfinucan: sorry, I was meaning put it in this file: https://github.com/openstack/nova-specs/blob/master/specs/mitaka/redirects10:20
*** daemontool has quit IRC10:21
johnthetubaguythis one has examples in it already: https://github.com/openstack/nova-specs/blob/master/specs/liberty/redirects10:21
pkoniszewskialex_xu: hi, there?10:21
*** ljxiash has quit IRC10:21
*** ivase has joined #openstack-nova10:22
*** ivase_ has quit IRC10:22
PaulMurraybauzas, this is the one with the failures: https://review.openstack.org/#/c/258771/32 don't want to waste your time, I was just asking because I saw a lot of red in the check queue this monring10:23
bauzasPaulMurray: well, that's all the tempest tests failing10:24
bauzasPaulMurray: in general, it's only one job which fails10:24
bauzasPaulMurray: so my first thought would that be a PEBKAC10:24
*** daemontool has joined #openstack-nova10:24
bauzaschecking logs tho10:25
openstackgerritStephen Finucane proposed openstack/nova-specs: Add redirect for renamed CPU thread policy spec  https://review.openstack.org/28463210:25
openstackgerritStephen Finucane proposed openstack/nova-specs: Add redirect for renamed CPU thread policy spec  https://review.openstack.org/28463210:25
*** tyagiprince1 has joined #openstack-nova10:26
*** tyagiprince has quit IRC10:26
*** tyagiprince1 is now known as tyagiprince10:26
openstackgerritAndrey Pavlov proposed openstack/nova: remove EC2 legacy code  https://review.openstack.org/26642510:27
openstackgerritStephen Finucane proposed openstack/nova-specs: Mark CPU thread pinning as implemented  https://review.openstack.org/28463310:27
*** ljxiash has joined #openstack-nova10:27
openstackgerritSergey Nikitin proposed openstack/nova: DO_NOT_MERGE  https://review.openstack.org/28463410:28
*** ivase_ has joined #openstack-nova10:28
*** ivase has quit IRC10:28
*** achanda has joined #openstack-nova10:29
openstackgerritStephen Finucane proposed openstack/nova-specs: Mark CPU thread pinning as implemented  https://review.openstack.org/28463310:30
openstackgerritStephen Finucane proposed openstack/nova-specs: Add redirect for renamed CPU thread policy spec  https://review.openstack.org/28463210:30
wznoinskalex_xu  - would you ahve a moment to review https://review.openstack.org/#/c/280299/ ?10:31
sfinucanjohnthetubaguy: Third time lucky, I hope :fingerscrossed:10:31
*** qeelee has quit IRC10:31
bauzasPaulMurray: mmm, interesting http://graphite.openstack.org/render/?from=-200hours&height=500&until=now&width=800&bgcolor=ffffff&fgcolor=000000&yMax=100&yMin=0&vtitle=Failure%20Rate%20in%20Percent&title=Test%20failure%20rates%20over%20last%20200%20hours&&target=lineWidth%28color%28alias%28movingAverage%28asPercent%28stats.zuul.pipeline.check.job.gate-tempest-dsvm-cells.FAILURE,sum%28stats.zuul.pipeline.check.job.gate-tempest-dsv10:32
bauzasI probably missed a big problem10:32
*** ljxiash has quit IRC10:32
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Use constant_time_compare from oslo.utils  https://review.openstack.org/28463510:33
*** ivase has joined #openstack-nova10:34
*** achanda has quit IRC10:35
*** ivase_ has quit IRC10:35
*** mjura has quit IRC10:36
*** lyanchih has joined #openstack-nova10:38
*** zenoway has quit IRC10:39
*** lpetrut has joined #openstack-nova10:39
*** zenoway has joined #openstack-nova10:39
*** ivase_ has joined #openstack-nova10:39
*** ivase has quit IRC10:40
PaulMurraybauzas, do you know what this is:10:41
PaulMurray2016-02-25 06:22:46.660 | + /tmp/ansible/bin/ansible subnodes -f 5 -i /home/jenkins/workspace/gate-tempest-dsvm-full/inventory -m synchronize -a 'mode=pull src='\''/opt/stack/logs/'\'' dest='\''/opt/stack/logs/subnode-{{ host_counter }}'\'''10:41
PaulMurray2016-02-25 06:22:46.660 | ERROR! Specified hosts options do not match any hosts10:41
bauzasPaulMurray: that's for collecting the logs AFAIK10:41
*** pcaruana has joined #openstack-nova10:42
bauzasbut the change you pointed me had an issue with devstack not returning correctly10:42
bauzasPaulMurray: http://logs.openstack.org/71/258771/32/check/gate-tempest-dsvm-full/30e1b7a/console.html#_2016-02-25_06_22_33_13610:43
PaulMurraybauzas, yes, saw that - been trying to find where it fails10:43
*** zenoway has quit IRC10:44
*** cdent has joined #openstack-nova10:45
*** ivase has joined #openstack-nova10:45
*** ivase_ has quit IRC10:46
bauzasPaulMurray: that's a $100M question10:48
PaulMurraybauzas, the only two patches in the check queue failing in the same way now10:48
PaulMurrayhttps://review.openstack.org/25931910:48
PaulMurrayhttps://review.openstack.org/25410210:49
*** belmoreira has joined #openstack-nova10:49
bauzasfun10:49
PaulMurrayI suspect this is happening to all the nova jobs10:49
bauzaslemme see logstash10:49
* bauzas doing 3 things at the same time, so sorry about the log10:50
bauzaslag*10:50
*** tyagiprince has quit IRC10:50
*** tyagiprince has joined #openstack-nova10:50
bauzasI don't remember but we also have a nice dashboard10:50
johnthetubaguybelmoreira: did you get chance to review that neutron routed network spec, given the discussion on the ops lists about cells and networking10:50
kashyappkoniszewski: Hi, when you get a moment, might want to rebase this? -- https://review.openstack.org/#/c/168916/10:50
bauzasPaulMurray: yeah I remember now http://status.openstack.org/openstack-health/#/10:51
*** ivase_ has joined #openstack-nova10:51
*** ivase has quit IRC10:51
bauzasinteresting, the last data is from one week...10:52
bauzasso back to logstash :D10:52
*** claudiub has quit IRC10:52
pkoniszewskikashyap: sure, thanks for reminder! :)10:53
kashyappkoniszewski: Thanks for working on it! :-)10:53
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Use constant_time_compare from oslo.utils  https://review.openstack.org/28463510:54
PaulMurraybauzas, don't worry - I'm juggling too10:56
lyarwoodndipanov: any chance you could take another look at https://review.openstack.org/#/c/266095/ if you have time today?10:56
bauzasPaulMurray: so, hard to tell10:56
bauzasPaulMurray: I'd say, do a recheck10:56
PaulMurray:)10:56
PaulMurrayif in doubt.....10:56
*** drankis has joined #openstack-nova10:56
PaulMurrayswitch it off and on again10:56
*** ivase has joined #openstack-nova10:57
*** ivase_ has quit IRC10:57
*** gszasz has joined #openstack-nova10:58
*** shaohe_feng_ has joined #openstack-nova10:58
*** salv-orlando has joined #openstack-nova10:59
*** salv-orlando has quit IRC10:59
*** salv-orlando has joined #openstack-nova10:59
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-nova11:01
PaulMurrayalex_xu, johnthetubaguy bauzas looking for a +W on https://review.openstack.org/#/c/278454 to get a chain moving - simple novaclient change11:01
kaisersHi Novas! Can somebody pls. point to an explanation what exactly is the difference between nova migrate vs nova live-migration? I'm testing a Cinder impl and just realized i can use nova migrate on a running instance. So what is the difference to live-migration. Only the ability to select the target host?11:01
johnthetubaguyPaulMurray: I am just struggling to really understand that stuff11:02
johnthetubaguykaisers: we have a doc on that, let me find the link11:02
*** efoley__ has joined #openstack-nova11:02
kaisersjohnthetubaguy: that would be great. Google found me tons of pages on how to configure things but nothing about the semantics... :)11:02
johnthetubaguykaisers: http://developer.openstack.org/api-guide/compute/server_concepts.html#moving-servers11:03
kaisersjohnthetubaguy: thanks!11:03
alex_xuPaulMurray: added to my review list, will try to review asap11:03
*** zenoway has joined #openstack-nova11:03
johnthetubaguykaisers: also this one is a bit shorter and specific about the API: http://developer.openstack.org/api-ref-compute-v2.1.html#os-admin-actions-v2.111:03
PaulMurrayalex_xu, thanks11:03
pkoniszewskikaisers: also you can check this http://docs.openstack.org/admin-guide-cloud/compute-configuring-migrations.html to see what are the limitations of both types of migrations11:03
kashyappkoniszewski: BTW, is there a simple way to make an RPC request hang?11:03
alex_xuPaulMurray: np11:04
*** efoley_ has quit IRC11:04
kaisersthanks11:04
kaisers<--- digs into things11:04
shaohe_feng_johnthetubaguy: ping.11:04
johnthetubaguykaisers: basically we have cold migrate and live-migrate, one turns the VM off when you move it, the other tries to keep it running11:04
johnthetubaguyshaohe_feng_: how can i help?11:04
bauzasjohnthetubaguy: the nova internals are so lovely between resize/cold-migrate and rebuild/evacuate...11:06
*** sdague has joined #openstack-nova11:06
shaohe_feng_johnthetubaguy:   sdague : do we need to add migrate_type?11:07
* alex_xu nod to bauzas11:07
bauzasjohnthetubaguy: conceptually, I wonder if we shouldn't big-bang the definitions and provide a clean split between the ops11:07
*** omnipresent has joined #openstack-nova11:07
*** zenoway has quit IRC11:07
pkoniszewskikashyap: you mean simple way to reproduce it? you can slow down conductor before it starts async, it will timeout on the API side11:07
shaohe_feng_johnthetubaguy: how we define in progress list.  still negative logic ?11:07
johnthetubaguybauzas: yeah, the history here is xenapi implemented migrate/resize, libvirt did resize, then they each added the other one, at least that what I think happened11:07
johnthetubaguyshaohe_feng_: I hate the negative logic myself11:08
bauzasjohnthetubaguy: so, followin the RPC calls just limbo and there is high error rate11:08
bauzasbut anyway11:08
bauzasthat's a long story11:08
kaisersjohnthetubaguy: So at the core, 'nova migrate' does a cold migration where the VM is in fact shut down for a short time, which 'nova live-migration' does not. But both commands can be run on an active instance which pops up/transfers on/to the new host running and active. Is that right?11:08
kashyappkoniszewski: Yeah, a reproducer.  Slowing down Conductor - never done that, how do you go about it?11:08
bauzasthat's just because I'm right on that :-)11:08
*** ivase has quit IRC11:08
shaohe_feng_johnthetubaguy: so let me update it  in the coming version.11:09
johnthetubaguybauzas: yeah, its stuff we are bad at testing really, and glad to see us fixing that now11:09
*** ivase has joined #openstack-nova11:09
johnthetubaguykaisers: yes, its cold vs live, basically11:09
bauzashopefully, things are improving11:09
kaisersjohnthetubaguy: great, got that now. :)11:09
kaisersthanks again!11:09
johnthetubaguykaisers: np11:10
shaohe_feng_johnthetubaguy:  and filter [queued, preparing, running, post-migrating]  for in progress.11:10
johnthetubaguyshaohe_feng_: I am not sure we need the migration_type, if we only return live-migration things, not sure what sdague and alex_xu think though, I am curious11:11
PaulMurraybauzas, johnthetubaguy we pushed out refactoring the code for migrations from the last cycle to get some more immediate stuff done11:11
PaulMurraybauzas, johnthetubaguy there is plenty of interest in cleaning this stuff up now11:11
alex_xujohnthetubaguy: we return all types migration in shaohe_feng_'s patch, shaohe_feng_ right?11:11
sdaguejohnthetubaguy: I don't think we need it11:11
shaohe_feng_johnthetubaguy: yes.11:11
johnthetubaguyPaulMurray: agreed, its something I wrote up in a blueprint 2.5 years ago, and still haven't have time, so happy to see it happening11:11
kashyappkoniszewski: I'll try to find out, don't bother here.11:12
sdaguealex_xu: I thought it was only returning live migrations11:12
bauzasPaulMurray: honestly, I'm not talking of refactoring, just cleaning out the definitions and have clear actions11:12
johnthetubaguysdague: I am not sure it does that yet...11:12
sdaguejohnthetubaguy: oh, so we should do that11:12
johnthetubaguysdague: +111:12
pkoniszewskikashyap: so the easiest way is to make it sleep (conductor, source compute or destination compute during prechecks), another way is to reduce number of file descriptors for rabbitmq on, e.g., source compute node11:12
PaulMurraybauzas, oh, right, something we can document you mean?11:12
sdaguebecause that was the plan11:12
sdagueand why we dropped all the flavor stuff11:12
johnthetubaguysdague: the spec was a little vague and contradictory when I checked it11:12
johnthetubaguysdague: yeah11:12
alex_xusdague: ok, if only return live-migration...agree to we needn't that11:12
kashyappkoniszewski: Ah, reducing FDs sound interesting11:12
bauzasPaulMurray: no, touching the REST API11:12
pkoniszewskikashyap: it will timeout on different layers due to chain of sync calls11:12
alex_xusdague: johnthetubaguy one more question, we didn't return all types, just for first step?11:13
PaulMurraybauzas, restructuing the API then?11:13
alex_xus/we/why/11:13
bauzasPaulMurray: and the RPC bits so that one REST action corresponds straight to one RPC call11:13
kashyappkoniszewski: Yep, noted.  Thank you11:13
sdaguealex_xu: because we're only implementing DELETE for one type11:13
sdagueif we expose all types here, the DELETE patch gets a ton more complicated11:13
johnthetubaguysdague: yeah, I just -1ed the patch for it not checking its live-migrate11:13
sdaguethis was always supposed to be just live-migration to start11:14
alex_xusdague: ok, got it11:14
shaohe_feng_sdague: johnthetubaguy: OK, only return live migrations.11:14
bauzasand not for example cold_migrate on the REST side being a compute.api resize which is a migrate_server conductor method which ends up to tasks.migrate11:14
johnthetubaguyshaohe_feng_: so I think we need to make sure we only return live-migrations, so that should help with the inprogress, we just need that states for live-migrations11:14
*** ivase_ has joined #openstack-nova11:14
sdagueI'm not even sure what progress you could return on a resize, for instance11:14
PaulMurraybauzas, oh, well, I don't get that exactly11:14
bauzasPaulMurray: because we factored some code11:14
johnthetubaguysdague: you could do disk copied, but lets not try get into that11:14
PaulMurraybauzas, but it does come into the proposals that have been around11:14
alex_xushaohe_feng_: whether /os-migration return all type migrations?11:15
alex_xuif yes, what about ref-link in /os-migration?11:15
pkoniszewskijohnthetubaguy: isn't decorator enough? Im talking about checking migration type in 'abort live migration'11:15
bauzasI mean that if that's a cold move, we should provide only one operation for the REST API11:15
bauzasthat can be asking for resize or not11:15
johnthetubaguypkoniszewski: no, migrating state is not just live-migrate11:15
*** ivase has quit IRC11:15
shaohe_feng_alex_xu: it should return all type. let me check it.11:15
alex_xulooks like same as non active migration, just 404, sounds ok11:15
bauzasif that's a live move, then there is only one thing to do11:15
sdaguepkoniszewski: no, the representation we have doesn't make any sense for non live migration11:16
*** cdent has quit IRC11:16
pkoniszewskijohnthetubaguy: thought that cold-migrate/resize has other task state11:16
bauzasso, evacuate, resize, rebuild and cold-migrate are pretty identical to me11:16
sdaguereally, explain memory_remaining_bytes field for resize11:16
bauzasthey're just a cold move operation with a different set of constraints11:16
bauzasPaulMurray: ^11:16
*** derekh has quit IRC11:16
johnthetubaguybauzas: right, thats what I wrote up in my blueprint 2.5 years ago, totally agree, although admittedly thats before we added evacuate, possibly, not sure11:17
pkoniszewskijohnthetubaguy: sdague: but okay, we already have the data so we can check it, so we will avoid any troubles, probably i will need to add it to force_complete too11:17
bauzasevacuate is just a cold-migrate where the added constraint is that we can't check the source host11:17
PaulMurraypkoniszewski, johnthetubaguy sdague the user issues are different for non-live-migration11:17
PaulMurrayonlly live-migration tends to hang11:18
johnthetubaguybauzas: actually I made live=true/false an option in there, or something like that, so a single admin "move server", "resize"/"rebuild" is a bit different though, anyways, different convo11:18
PaulMurrayor take time at least11:18
johnthetubaguypkoniszewski: yeah, we should add that to force_complete11:18
bauzasjohnthetubaguy: conceptually, I was seeing from an UX PoV :11:18
pkoniszewskijohnthetubaguy: got it, will work on it11:19
*** smatzek__ has joined #openstack-nova11:19
sdaguepkoniszewski: the entire issue that I understood we were trying to solve was the live migation black hole11:19
bauzas#1 as an user, I accept downtime and I want to recreate my instance either locally or remotely11:19
bauzaswell11:19
bauzasit's rather11:19
bauzasdepending on whether I'm a user, I want to recreate my instance11:19
bauzaslocally or remotely is not something I should know unless I'm an admin11:20
*** ivase has joined #openstack-nova11:20
bauzasjohnthetubaguy: but anyway, we can think a lot of that, but we should maybe resurrect your spec and discuss it as a backlog one11:20
PaulMurraybauzas, I think the four examples in the concepts guide are reasonable11:20
*** erlon has joined #openstack-nova11:20
*** ivase_ has quit IRC11:20
bauzasPaulMurray: why not, I'm not opiniated11:21
johnthetubaguybauzas: so I think there are a list of use cases, probably a separate API for each concept, but probably a single set of "move" primitives underneath that implement those API, I think this was pre-specs, I should go find that...11:21
bauzas++11:21
bauzasexactly *that*11:21
johnthetubaguyyeah, the docs covers this quite well now11:21
johnthetubaguythe use cases that is11:21
PaulMurrayi agree11:21
johnthetubaguypkoniszewski: shaohe_feng_: do you think you are both unblocked on these changes now? happy about the next steps?11:22
PaulMurraymost admin use cases are about moving vms11:22
pkoniszewskijohnthetubaguy: I have one more question actually before I start work11:22
johnthetubaguypkoniszewski: fire away11:22
PaulMurraymost user use cases are about doing something to a vm that may require it to move behind the scenes11:22
johnthetubaguyPaulMurray: +111:22
PaulMurraybauzas, e.g. anythig a user can do to a vm that would affect the way it was scheduled implies a potential move11:23
pkoniszewskiso we mark migration as running before we actually start migration: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L521211:24
pkoniszewskithere is a lot of code after this line executed in driver11:24
pkoniszewskiwonder if I should ask libvirt if is there any job undergoing so we won't end up with a paused VM, instead of forced live migration11:24
pkoniszewskiim talking about another check at driver level: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L650911:25
*** sudipto has quit IRC11:25
pkoniszewskijohnthetubaguy: ^^11:25
johnthetubaguypkoniszewski: I think we do need that11:25
*** ivase_ has joined #openstack-nova11:26
johnthetubaguypkoniszewski: well, I think lets just check we are aborting a live-migrate job?11:26
shaohe_feng_johnthetubaguy: another question.  the instance db  API level or cell level in cell v2?11:26
*** ivase has quit IRC11:26
johnthetubaguypkoniszewski: oh, sorry, I see, you mean for the pause11:26
pkoniszewskijohnthetubaguy: exactly, we won't pause during abort because of check at compute-manager level i believe11:26
johnthetubaguypkoniszewski: yeah, checking there is an active live-migrate job, before calling pause might do the trick here? and probably double checking there is still one going after we call pause11:27
*** fawadkhaliq has joined #openstack-nova11:27
johnthetubaguypkoniszewski: not sure we change the task state, thinking about it... hmm11:28
*** gongysh has quit IRC11:28
johnthetubaguypkoniszewski: lets leave that issue for now, and just fix these other ones first11:28
pkoniszewskijohnthetubaguy: depends what is more important, that VM is paused or that it is being migrated11:28
pkoniszewskijohnthetubaguy: asking libvirt should avoid potential race condition, yes11:28
johnthetubaguypkoniszewski: right now I think we need to get the new features merged11:29
pkoniszewskijohnthetubaguy: sure, I just want to clarify it at least a bit before I start work, let's focus on important stuff before freeze11:29
pkoniszewskijohnthetubaguy: thanks!11:29
johnthetubaguypkoniszewski: np11:30
johnthetubaguypkoniszewski: these are new tools for admins, so I think its better we add the tool, rather than make it perfect, in this particular case11:30
shaohe_feng_johnthetubaguy: I read cell doc. the migration is API level, and the instance is cell level in V2.11:30
bauzasshaohe_feng: not sure I understood your questions, but in cells V2 world, instances will be kept on the cell db, not in the api db11:30
*** houming has quit IRC11:31
johnthetubaguyshaohe_feng_: oh, so passing down the migration will be quite important, eventually...11:31
bauzasshaohe_feng: so we have an InstanceMapping table in the API DB for knowing which instance is in which cell11:31
johnthetubaguyshaohe_feng_: yeah, where is this question?11:31
*** ivase has joined #openstack-nova11:31
shaohe_feng_bauzas: johnthetubaguy: so we can get migration by object directly. but we still get instance by compute_api . right?11:31
bauzasjohnthetubaguy: that's quite identical as my RequestSpec object for check-dests, if we get any object in the API DB, we need to pass it as payload to the conductor11:32
*** ivase_ has quit IRC11:32
bauzasjohnthetubaguy: shaohe_feng: although it would still be possible to get some object from the api db within a cell, but we'd like to avoid that as much as possible (because of races)11:32
bauzasbut I don't have much context, shaohe_feng could you please clarify your question ?11:33
*** wanghao has quit IRC11:33
johnthetubaguyso don't worry about it, the object layer will be hiding most of that detail, I suspect11:34
johnthetubaguylets just do the simple thing now, thats consistent, and worry about those fix ups later11:34
shaohe_feng_bauzas: in nova API, can we access the migration object directly? pass_through computer_api or cell_api11:35
shaohe_feng_johnthetubaguy: ^11:35
*** wanghao has joined #openstack-nova11:35
bauzaswhich migration object ?11:35
shaohe_feng_that's my question.11:35
bauzasI'm missing context11:35
alex_xujohnthetubaguy: so that is a problem for https://review.openstack.org/#/c/277971/ ? we need pass migration object down11:36
PaulMurrayjohnthetubaguy, bauzas, sdague  meanwhile, back in the check queue.... looks like tempest jobs are failing for everyone11:36
PaulMurraysdague, who should we ask about this?11:36
shaohe_feng_the migration object which accesses migration db table.11:36
bauzasshaohe_feng: you mean the Migration nova object ?11:36
sdaguePaulMurray: yeh... I think I found the issue there11:36
johnthetubaguyso I think we should pass the migration object down, really11:36
sdaguehttps://review.openstack.org/28465211:36
bauzassdague: which was ? I was looking at the issue but hard to tell11:36
bauzasit's not super explicit11:36
sdagueansible release11:36
bauzasokay11:37
*** ankit_ag has quit IRC11:37
*** ivase_ has joined #openstack-nova11:37
PaulMurraysdague, good job11:37
shaohe_feng_bauzas: yes.11:37
*** ankit_ag has joined #openstack-nova11:37
alex_xujohnthetubaguy: ok, got it11:37
*** ivase has quit IRC11:37
johnthetubaguyalex_xu: hmm, I am still on the fence, actually maybe its odd we put the migration objects in the API11:39
alex_xujohnthetubaguy: why?11:39
alex_xujohnthetubaguy: if we have complex logic, that should be in the comptue_api11:40
johnthetubaguyalex_xu: oh, sorry, I was talking about something different, totally the complex logic should live in compute_api11:40
alex_xujohnthetubaguy: sorry, misunderstand that11:41
*** aagrawal has quit IRC11:41
PaulMurraysdague, :( your patch failed11:41
*** gcb has quit IRC11:42
sdaguePaulMurray: oh, right 2 =11:42
*** mingyu has joined #openstack-nova11:43
*** EinstCrazy has joined #openstack-nova11:43
sdaguehey, I'm only 1/2 a cup of coffee in. :)11:43
sdagueI get to make some mistakes11:43
*** sridharg has joined #openstack-nova11:43
*** gcb has joined #openstack-nova11:43
*** ivase_ has quit IRC11:43
PaulMurraysdague, don't worry, on half a coffee you are still two steps ahead of us11:44
*** josecastroleon has quit IRC11:45
PaulMurraysdague, speaking of coffee, I have to walk to another floor now...11:45
*** efoley__ has quit IRC11:46
*** mingyu has quit IRC11:47
shaohe_feng_bauzas: Can rest API access Migration nova object directly, without computer_api or cell_api?11:47
bauzasshaohe_feng: again, sorry, not sure I get your question11:49
bauzasshaohe_feng: you want to know if a WSGI resource module can fetch the object if it's in the API DB ? then yes11:50
bauzasalthough I tend to prefer using the compute.api module for that11:50
*** mdnadeem has quit IRC11:50
*** mdnadeem has joined #openstack-nova11:51
bauzasin cell v2, there will be no "compute_cell" proxy11:51
bauzasI mean, the nova.compute.cells_api module, sortu11:51
bauzassorry11:51
bauzasbecause by default, the regular API will be there11:52
*** rfolco has joined #openstack-nova11:53
*** cdent has joined #openstack-nova11:53
*** lyanchih has joined #openstack-nova11:54
johnthetubaguyshaohe_feng: everywhere can access both datastores, if they have to, but for scalability reasons, the api accessing the child cell is fine, but the reverse should be avoided where possible11:55
johnthetubaguydoes that help?11:55
alex_xujohnthetubaguy: so...will you -1 this patch https://review.openstack.org/#/c/277971/ for pass migration down, or I help you -1 it?11:56
johnthetubaguyalex_xu: so I think its OK for now, mostly because its consistent with the other one, and really we do need to double check the migration state11:57
*** Guest75 has joined #openstack-nova11:57
*** ekuris has quit IRC11:57
alex_xujohnthetubaguy: ok, got it11:57
johnthetubaguyalex_xu: thats my way of saying, I don't think the migration object should live in the API database, where did we say that would be the case, btw?11:57
alex_xujohnthetubaguy: oops, I guess I misunderstand, I thought migraiton object should in the API database11:59
johnthetubaguyI think shaohe_feng mentioned that, I am just wondering where we said that, it doesn't feel quite right to me11:59
johnthetubaguyas its so related to the instance, it feels like a child cell thing11:59
johnthetubaguybut I could be missing something11:59
*** haomaiwang has quit IRC12:01
alex_xujohnthetubaguy: ok12:01
*** lyanchih has quit IRC12:01
*** thorst has joined #openstack-nova12:01
*** haomaiwang has joined #openstack-nova12:01
* johnthetubaguy runs away for a little bit12:02
*** mpavone has quit IRC12:03
*** MCoLo has quit IRC12:04
openstackgerritGleb Stepanov proposed openstack/nova: Console log copy for block live migratiob  https://review.openstack.org/28467412:04
*** MCoLo has joined #openstack-nova12:04
sdagueI think we probably need a write up about what goes into the API database, and is strict about it. Otherwise we're going to walk ourselves back into a single db solution one table at a time.12:04
*** aysyd has joined #openstack-nova12:05
bauzassdague: that's on-going12:06
bauzassdague: lemme find the ref12:06
bauzassdague: yeah this http://docs.openstack.org/developer/nova/cells.html#database-split12:06
*** smatzek__ is now known as smatzek12:06
bauzassdague: but the formatting is awful12:06
bauzasI can fix that12:07
*** Guest75 has quit IRC12:07
shaohe_feng_bauzas:  johnthetubaguy : got it. thank you.12:17
shaohe_feng_bauzas:  WSGI resource module can fetch the object if it's in the child cell DB ?12:19
bauzaslike johnthetubaguy said12:20
shaohe_feng_bauzas: seen it.  the answer is yes.12:22
*** mjura has joined #openstack-nova12:25
*** haomaiwang has quit IRC12:31
*** shaohe_feng_ has quit IRC12:31
*** achanda has joined #openstack-nova12:33
*** raildo-afk is now known as raildo12:37
belmoreirajohnthetubaguy https://review.openstack.org/#/c/225384 yes but not https://review.openstack.org/#/c/263898. I will have a look12:37
belmoreirajohnthetubaguy thanks for ping me12:37
*** lucasagomes is now known as lucas-hungry12:39
sdaguewhile we are waiting for devstack jobs to work again, might I suggest some test only fixes to review - https://review.openstack.org/#/c/267097/ - https://review.openstack.org/#/c/283261/12:39
*** ljxiash has joined #openstack-nova12:40
*** achanda has quit IRC12:40
*** rtheis has joined #openstack-nova12:42
*** fawadkhaliq has quit IRC12:42
*** daemontool_ has joined #openstack-nova12:43
*** mingyu has joined #openstack-nova12:43
*** fawadkhaliq has joined #openstack-nova12:43
*** zul has quit IRC12:43
*** zul has joined #openstack-nova12:44
*** daemontool has quit IRC12:46
*** diogogmt has joined #openstack-nova12:46
*** fawadkhaliq has quit IRC12:47
*** daemontool_ has quit IRC12:47
openstackgerritDongcan Ye proposed openstack/nova: VMware: Add warning log when creating folder already exists  https://review.openstack.org/28469412:47
*** mingyu has quit IRC12:47
sdaguealex_xu: https://review.openstack.org/#/c/283261/ is actually a regression test for the regex bug you are working on12:53
*** pcaruana has quit IRC12:55
*** mpavone has joined #openstack-nova12:55
*** baoli has joined #openstack-nova12:55
*** salv-orlando has quit IRC12:56
*** dave-mccowan has joined #openstack-nova12:56
*** efoley__ has joined #openstack-nova12:57
*** Daisy has joined #openstack-nova12:58
*** daemontool has joined #openstack-nova13:01
*** takedakn has joined #openstack-nova13:02
openstackgerritZhengguang Ou proposed openstack/nova: libvirt: fix race condition in detach volume and create image  https://review.openstack.org/16615213:02
*** Daisy has quit IRC13:02
*** mpavone has quit IRC13:03
*** ildikov has quit IRC13:03
*** edmondsw has joined #openstack-nova13:04
thorstIf someone has a chance, could they take a peak at https://review.openstack.org/#/c/264157/  Its been sitting for a few weeks, but solves a long standing bug for cloud-init & ipv6 only environments13:07
*** efoley_ has joined #openstack-nova13:08
*** pcaruana has joined #openstack-nova13:09
*** diogogmt has quit IRC13:10
*** pradk has joined #openstack-nova13:10
*** pradk has quit IRC13:10
*** efoley__ has quit IRC13:11
*** mpavone has joined #openstack-nova13:11
*** derekh has joined #openstack-nova13:13
*** ekuris has joined #openstack-nova13:13
*** tyagiprince has quit IRC13:15
*** takedakn has quit IRC13:15
*** zz_dimtruck is now known as dimtruck13:16
kaisersHi Folks! Can anybody here give me a 'minute' to find out why i encounter a migration issue with console.log beeing chown'ed from nova:nova to root:root?13:16
*** ducttape_ has joined #openstack-nova13:17
sdaguePaulMurray / bauzas / et all13:18
sdaguethe gate is unwedged13:18
sdagueso if we've got patches that need a recheck13:18
sdaguenow is the time13:18
*** lucas-hungry is now known as lucasagomes13:19
*** rotbeard has joined #openstack-nova13:20
*** claudiub has joined #openstack-nova13:23
*** mubirru has quit IRC13:23
*** baoli_ has joined #openstack-nova13:26
*** links has quit IRC13:26
*** kylek3h__away is now known as kylek3h13:28
*** baoli has quit IRC13:28
*** mpavone has quit IRC13:30
*** josecastroleon has joined #openstack-nova13:31
*** ivase has joined #openstack-nova13:34
openstackgerritGleb Stepanov proposed openstack/nova: Console log copy for block live migratiob  https://review.openstack.org/28467413:35
*** ivase has quit IRC13:36
*** ducttape_ has quit IRC13:36
*** ivase has joined #openstack-nova13:36
*** rtheis has quit IRC13:36
*** ankit_ag has quit IRC13:37
*** achanda has joined #openstack-nova13:37
*** toure is now known as toure_13:39
*** hamzy has quit IRC13:41
*** achanda has quit IRC13:41
*** ivase has quit IRC13:41
*** rtheis has joined #openstack-nova13:42
*** ccarmack has joined #openstack-nova13:43
*** shyama has quit IRC13:44
*** rtheis has quit IRC13:44
*** rtheis has joined #openstack-nova13:45
openstackgerritPaul Murray proposed openstack/python-novaclient: Support for abort an ongoing live migration  https://review.openstack.org/28429613:45
*** burgerk has joined #openstack-nova13:46
*** armax has joined #openstack-nova13:47
*** dims has joined #openstack-nova13:47
*** yassine__ has quit IRC13:49
*** shaohe_feng_ has joined #openstack-nova13:49
*** ildikov has joined #openstack-nova13:50
*** jaypipes has joined #openstack-nova13:51
kaisersCan anybody give me a quick hint on a libvirt/migration related question?: what is the file console.log for and when/where is it created?13:51
kaisersI'm hitting issues with that file beeing owned by root:root and nova beeing unable to access it because of that.13:51
*** xyang1 has joined #openstack-nova13:52
pkoniszewskiandreykurilin: hi, there?13:52
andreykurilinpkoniszewski: hi! yes13:52
jaypipescdent: good point on the context/get_engine() thing. /me needs to ask rpodolyaka how to do that..13:53
jaypipesrpodolyaka: how do I do that? :) See cdent's comment on https://review.openstack.org/#/c/284484/1/nova/db/sqlalchemy/api.py13:53
*** fawadkhaliq has joined #openstack-nova13:53
cdentjaypipes: I was pretty surprised I could still see past the blood in my eyes by the time I got to that line.13:54
jaypipescdent: :( /me cries13:54
cdentActually that's not quite right: it is very well written for what it is and I did understand it13:54
cdentIt's just ... a lot13:54
jaypipescdent: luckily, that won't be there for long...13:54
pkoniszewskiandreykurilin: i wonder if we should chain new test classes when adding support for new microversion in novaclient13:55
jaypipescdent: once the resource-providers schema is the only thing used for inventory and usage stuff, it will be a lot simpler and more efficient.13:55
pkoniszewskiandreykurilin: i'm talking about this https://review.openstack.org/#/c/284296/2/novaclient/tests/unit/v2/test_server_migrations.py13:55
cdentI had to write a similar many-aliases thing in some other context otherwise it would have been a bit unfamliar13:55
jaypipescdent: i.e. less use of outer joins. more ability to winnow results on the DB side.13:55
cdent\o/13:55
johnthetubaguybelmoreira: cool, just thinking its a good time to double check it will work for you :)13:56
*** Guest75 has joined #openstack-nova13:56
*** zzzeek has joined #openstack-nova13:57
andreykurilinpkoniszewski: imo, we should not put them in the chain13:57
*** zzzeek has quit IRC13:57
pkoniszewskiandreykurilin: okay, we chain tests in nova api, thought we follow it there, but that's fine too, thanks! :)13:58
*** BobBall_ has joined #openstack-nova13:58
*** salv-orlando has joined #openstack-nova13:58
*** zzzeek has joined #openstack-nova13:58
*** BobBall_ is now known as BobBall13:58
andreykurilinpkoniszewski: we do the same in novaclient( https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/unit/v2/test_servers.py#L1050 ) lol, but, I want to fix this:)13:59
*** daemontool_ has joined #openstack-nova13:59
openstackgerritPaul Murray proposed openstack/python-novaclient: Support for forcing live migration to complete  https://review.openstack.org/27964713:59
alaskicdent: jaypipes I think you can just pass the context in to get engine to preserve the context_manager semantics, by kwarg since it's not the first arg13:59
jaypipesalaski: ah, I see. thank you sir.14:01
pkoniszewskiandreykurilin: ok, thanks for clarifying this! :)14:01
jaypipesI need to keep up with the new oslo.db enginefacade stuff.14:01
*** efoley_ has quit IRC14:01
kashyappkoniszewski: I only noticed your question just now on this bug, this was discussed on #virt on OFTC14:01
kashyaphttps://bugzilla.redhat.com/show_bug.cgi?id=994882#c314:01
openstackbugzilla.redhat.com bug 994882 in libvirt "Live block migration results in a zero console log" [High,New] - Assigned to libvirt-maint14:01
*** shyama has joined #openstack-nova14:01
*** daemontool has quit IRC14:02
kashyapI added the comment (#3) from that discussion there.14:02
openstackgerritPaul Murray proposed openstack/python-novaclient: Support for abort an ongoing live migration  https://review.openstack.org/28429614:02
*** edtubill has joined #openstack-nova14:03
*** apearson has joined #openstack-nova14:03
*** salv-orlando has quit IRC14:03
*** edtubill has quit IRC14:04
*** edtubill has joined #openstack-nova14:04
*** whenry has quit IRC14:05
johnthetubaguymikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas xyang tpatil med_ nic scottda nagyz belliott dguitarbite sdague jroll diana_clarke14:06
johnthetubaguy _diana_ raildo jichen gjayavelu mdorman klindgren sorrison belmoreira mrda auggy PaulMurray gcb thorst duncant macsz thomasem tpatzig bobball lbelivea14:06
johnthetubaguynova meeting is currently happening, in the usual place14:06
johnthetubaguyoops, sorry, for the late ping14:06
*** rotbeard has quit IRC14:06
kashyapWhat is the usual place?14:06
* kashyap forgets the IRC channel, goes to look up14:06
pkoniszewskikashyap: sorry if i send it twice, but just got DC and dunno if it went out14:06
pkoniszewskikashyap: sounds fair that higher-layer should handle this, yes14:06
kashyappkoniszewski: Didn't receive your first message, okay, I'll add the relevant comment in the Nova LP bug as well.14:07
*** links has joined #openstack-nova14:07
*** efoley_ has joined #openstack-nova14:07
*** ctrath has joined #openstack-nova14:08
*** andymaier has joined #openstack-nova14:08
*** apearson has quit IRC14:12
*** apearson has joined #openstack-nova14:13
garykBobBall: is the CI broken? I got a citrox -1 for a VMware test update :)14:14
BobBallprobably just a race...14:14
*** rlrossit_ has joined #openstack-nova14:15
BobBallnope14:15
BobBalldefinitely broken :D14:15
BobBallI'll check it out14:15
garykgracias14:15
garykBobBall: https://review.openstack.org/28428614:15
*** jichen has quit IRC14:17
BobBallinteresting - not seen this failure from devstack before http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/86/284286/2/7309/logs/devstacklog.txt.gz14:17
*** irenab has quit IRC14:18
BobBallLooks like a general failure garyk - see http://osdir.com/ml/openstack-dev/2016-02/msg02021.html for others who have hit this today14:19
*** rlrossit_ has quit IRC14:19
*** rotbeard has joined #openstack-nova14:19
*** rlrossit has joined #openstack-nova14:19
garykBobBall: ok, thanks14:20
BobBallLooks as though it killed the powerkvm CI and most of jenkins at the same time14:20
BobBallSo it's a waiting game I imagine14:20
*** fawadkhaliq has quit IRC14:20
johnthetubaguyBobBall: is that the ansible issue?14:21
johnthetubaguyoh, right, I see now14:21
thomasemhey14:21
BobBallPossibly johnthetubaguy - is it? :)14:21
johnthetubaguyBobBall: not sure now, that might be different14:21
johnthetubaguythomasem: howdy14:22
johnthetubaguysdague: I just writing a response to that os-brick thread on the ML14:22
BobBallgaryk: I'm on vacation this afternoon so I'll check in later - when jenkins is fixed I'll check if the fix worked for the citrix CI14:22
johnthetubaguysdague: I kinda thought we agreed to the sudo like privsep, to save us reverting out os-brick14:22
garykBobBall: no problem. enjoy the vacation. its not urgent14:23
*** yassine__ has joined #openstack-nova14:24
*** dimtruck is now known as zz_dimtruck14:25
*** lyanchih has joined #openstack-nova14:25
*** sudipto has joined #openstack-nova14:26
*** mgoddard_ has joined #openstack-nova14:27
*** rotbeard has quit IRC14:29
PaulMurrayjohnthetubaguy, I'm going to fix up the abort migration patch: https://review.openstack.org/#/c/277971/1514:30
*** mgoddard has quit IRC14:30
PaulMurrayjohnthetubaguy, I've got a couple of questions to clear up14:30
PaulMurrayjohnthetubaguy, comments https://review.openstack.org/#/c/277971/15/nova/compute/instance_actions.py14:31
*** smatzek has quit IRC14:31
*** erno has quit IRC14:31
*** lyanchih has quit IRC14:31
*** gongysh_ has joined #openstack-nova14:31
PaulMurrayjohnthetubaguy, first it was changed from past tense to present, the comments seem to say change it back to past tense....?14:32
PaulMurrayalso there are - vs _ so I'm a bit lost now14:32
PaulMurraysdague, you commented on the above ^^14:33
*** Jeffrey4l has quit IRC14:33
*** claudiub|2 has joined #openstack-nova14:33
*** sridharg has quit IRC14:33
*** mdrabe has joined #openstack-nova14:33
*** claudiub has quit IRC14:34
johnthetubaguyPaulMurray: oh good questions14:35
johnthetubaguyPaulMurray: oh my, we have revertResize live-migration and changePassword...14:35
johnthetubaguyPaulMurray: I think it should really match the API action14:35
johnthetubaguybut this isn't one14:36
*** rk4n has quit IRC14:36
*** rk4n has joined #openstack-nova14:36
*** amotoki has quit IRC14:37
PaulMurraythe compute_api or the external rest api? the compute_api action is live_migration_abort14:38
*** rk4n has quit IRC14:38
*** burgerk has quit IRC14:39
*** zz_dimtruck is now known as dimtruck14:40
*** mriedem has joined #openstack-nova14:40
johnthetubaguyPaulMurray: I was meaning rest api14:41
PaulMurraythe reset api is a delete on the migration resource14:41
*** jwcroppe has joined #openstack-nova14:44
johnthetubaguyyeah, that doesn't help really14:44
johnthetubaguyPaulMurray: I would +1 live-migration-cancel if that helps14:44
johnthetubaguynot sure if others would agree14:44
*** armax has quit IRC14:44
PaulMurrayI was just typing that - I'll do that14:44
johnthetubaguycool14:45
*** rotbeard has joined #openstack-nova14:45
PaulMurrayAlso I think we decided to stick with the 400 HTTPBadRequest for InvalidMigrationState14:46
PaulMurrayPersonally I don't like that, but can go with the flow?14:46
PaulMurrayhttps://review.openstack.org/#/c/277971/15/nova/api/openstack/compute/server_migrations.py14:46
*** cfriesen has joined #openstack-nova14:46
*** Jeffrey4l has joined #openstack-nova14:47
johnthetubaguyPaulMurray: I guess we did settle on that, and conflict for the instance in a bad state, hmm, that does seem odd14:47
johnthetubaguyPaulMurray: I like the consistency though14:48
*** apearson has quit IRC14:49
PaulMurrayok - I can wizz through this now. I may also add a patch to check its live-migration that we pause in force-complete as well as fixing this - but that's less urgent14:49
*** apearson has joined #openstack-nova14:49
* dansmith bows to jaypipes' fu14:50
*** dimtruck is now known as zz_dimtruck14:51
*** hamzy has joined #openstack-nova14:52
*** mrkz has joined #openstack-nova14:53
dansmithjaypipes: failed for legitness on postgres: http://logs.openstack.org/84/284484/1/check/gate-tempest-dsvm-postgres-full/585d0bf/logs/screen-n-api.txt.gz?level=TRACE#_2016-02-25_01_07_21_73714:53
jaypipesdansmith: ya, saw that. fixing it up.14:53
dansmithokay cool14:53
*** mingyu has joined #openstack-nova14:54
*** sc68cal has joined #openstack-nova14:54
*** erno has joined #openstack-nova14:58
mriedemgaryk: this is a pretty old bug but someone internally it hitting something that sounds similar, have you seen this? https://bugs.launchpad.net/nova/+bug/137305414:58
openstackLaunchpad bug 1373054 in OpenStack Compute (nova) "VMWare compute driver incorrectly attaches volumes on iSCSI for target with multiple LUNs" [Low,Confirmed]14:58
mriedem*is hitting14:58
*** diogogmt has joined #openstack-nova14:59
jaypipesedleafe: yes, dansmith answered correctly your question about None.14:59
*** smatzek has joined #openstack-nova14:59
*** eharney has joined #openstack-nova15:00
*** pushkaru has joined #openstack-nova15:02
openstackgerritGleb Stepanov proposed openstack/nova: Fix backing file copying  https://review.openstack.org/28227515:02
*** ducttape_ has joined #openstack-nova15:03
*** PsionTheory has joined #openstack-nova15:04
*** Swami has joined #openstack-nova15:05
*** davideagle_ has quit IRC15:06
*** burgerk has joined #openstack-nova15:07
*** salv-orlando has joined #openstack-nova15:08
openstackgerritMatt Riedemann proposed openstack/nova: Don't lazy-load instance.services if the instance is deleted  https://review.openstack.org/28382015:08
openstackgerritMatt Riedemann proposed openstack/nova: Add functional regression test for list deleted instances on v2.16  https://review.openstack.org/28383715:08
*** rotbeard has quit IRC15:09
*** achinm has quit IRC15:09
claudiub|2hello. if anyone can review this tiny bug fix, it would be great. :) https://review.openstack.org/#/c/258614/15:10
*** daemontool__ has joined #openstack-nova15:10
*** links has quit IRC15:12
*** rotbeard has joined #openstack-nova15:13
garykmriedem: i have not hit that one. to be honest i only work with the cinder dirver that supports vmdks.15:13
garykso when it comes to iscsi i have no time int he trenches. sorry15:13
*** davideagle_ has joined #openstack-nova15:13
openstackgerritJay Pipes proposed openstack/nova: Use new inventory schema in compute_node_get_all()  https://review.openstack.org/28448415:13
*** e0ne has joined #openstack-nova15:14
*** daemontool_ has quit IRC15:14
jaypipesdansmith, edleafe, alaski, cdent: ^^ addressed your concerns (and the PostgreSQL thing)15:14
mriedemgaryk: yeah, what's worse is the one this team hit internally was using the ibm xiv ds8k cinder driver, which is basically closed source15:14
tjonesmriedem: w.r.t. https://bugs.launchpad.net/nova/+bug/1373054 if they come back with info i can get someone to look into it15:14
openstackLaunchpad bug 1373054 in OpenStack Compute (nova) "VMWare compute driver incorrectly attaches volumes on iSCSI for target with multiple LUNs" [Low,Incomplete]15:14
mriedemgaryk: they were trying to figure out what sets the disk label, wasn't sure if that was cinder or nova's vmware volumeops15:15
mriedemtjones: i asked the people here to report their bug upstream with details15:15
tjonesyes15:15
*** salv-orlando has quit IRC15:15
tjonessaw that15:15
*** knikolla has joined #openstack-nova15:16
*** nkrinner has quit IRC15:16
PaulMurraydanpb, I have a quick question about cancelling a migration15:16
danpbok15:16
garykmriedem: i would think that it is the cinder side.15:17
PaulMurraydanpb, looking at https://review.openstack.org/#/c/277971/15/nova/virt/libvirt/driver.py15:17
smcginnisVipin Balachandran might be able to help there.15:17
smcginnisHe's not online right now, but he's our Cinder VMware guy.15:17
smcginnisOr at least one of them.15:17
PaulMurraydanpb, what's the best way to check a live migration is running ?15:18
mriedemunrelated, we hit this warning A LOT in n-cpu logs: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Trying%20to%20apply%20a%20migration%20context%20that%20does%20not%20seem%20to%20be%20set%20for%20this%20instance%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d15:18
*** zz_dimtruck is now known as dimtruck15:18
danpbPaulMurray: err, i commented on that yesterday i thought15:18
danpbyes, see https://review.openstack.org/#/c/277971/14/nova/virt/libvirt/driver.py15:18
PaulMurraydanpb, so you did, sorry I missed that15:18
edleafejaypipes: dansmith: got it. Thanks.15:19
*** rk4n has joined #openstack-nova15:19
danpbbasically if its in the migrating state in nova's DB, you can assume the job is related to migration15:19
openstackgerritDmitry Stepanenko proposed openstack/nova: Deprecate ComputeNode pci_device_pools field  https://review.openstack.org/24769215:19
danpbbecause we don't support other non-nova apps directly messing with guests behind nova's back15:19
*** kfarr has joined #openstack-nova15:19
PaulMurraydanpb, I have changed the API layer to check it actually is doing a live migration15:20
*** rk4n has quit IRC15:20
PaulMurraydanpb, I'll follow through the code to check, but that should make it safe then15:20
*** rk4n has joined #openstack-nova15:20
mriedemsmcginnis: yeah, but this isn't the cinder vmware vmdk backend :(15:21
mriedemit's the ibm xiv dsk815:22
danpbPaulMurray: you have  @check_instance_state(task_state=[task_states.MIGRATING])  in the api entry point which is fine imho15:22
*** gongysh_ has quit IRC15:22
*** mdrabe has quit IRC15:22
*** su_zhang has joined #openstack-nova15:22
dansmithjaypipes: that seems to work15:23
*** ekuris has quit IRC15:23
*** penick has joined #openstack-nova15:23
*** efoley_ has quit IRC15:24
*** apearson has quit IRC15:25
*** PsionTheory has quit IRC15:26
*** mrsoul has quit IRC15:26
*** mrsoul has joined #openstack-nova15:26
*** mrsoul has quit IRC15:26
*** mrsoul has joined #openstack-nova15:26
*** ekuris_ has joined #openstack-nova15:27
cfriesenis this a known tempest issue?  "ERROR! Specified hosts options do not match any hosts"15:27
*** davideagle_ has quit IRC15:27
mriedemjbernard: a thing! https://bugs.launchpad.net/nova/+bug/154985415:28
openstackLaunchpad bug 1549854 in OpenStack Compute (nova) "compute host "has not been heard from in a while" failing ceph jobs" [Undecided,New]15:28
*** rtheis has quit IRC15:28
*** rtheis has joined #openstack-nova15:29
*** shaohe_feng_ has quit IRC15:30
*** Jeffrey4l has quit IRC15:31
*** omnipresent has quit IRC15:31
*** rtheis has quit IRC15:31
*** rtheis has joined #openstack-nova15:32
*** sdague has quit IRC15:32
*** davideagle_ has joined #openstack-nova15:33
*** armax has joined #openstack-nova15:33
*** ekuris_ has quit IRC15:33
*** mgoddard_ has quit IRC15:33
*** rtheis has quit IRC15:35
openstackgerritThomas Maddox proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate  https://review.openstack.org/27479215:35
*** liverpooler has quit IRC15:36
johnthetubaguycfriesen: yeah, I think the fix for that happening, unless I got my wires crossed15:36
*** penick has quit IRC15:36
openstackgerritSean M. Collins proposed openstack/os-vif: Add mtu attribute to Network object  https://review.openstack.org/28420915:36
*** mgoddard has joined #openstack-nova15:37
*** rtheis has joined #openstack-nova15:37
*** mdrabe has joined #openstack-nova15:39
*** achanda has joined #openstack-nova15:39
*** cleong has joined #openstack-nova15:40
dansmithjaypipes: so you're going to have to convert the singular compute_node_get calls as well15:40
dansmithjaypipes: otherwise we only see the new values when we query them all15:40
*** e0ne has quit IRC15:40
*** e0ne_ has joined #openstack-nova15:41
lbeliveauIs there another core that could review this fix simple please, already got a +2 but need workflow: https://review.openstack.org/#/c/21604915:41
*** sudipto has quit IRC15:41
jaypipesdansmith: ah, yes. Can I do that in a followup patch, or will it need to be in this same one?15:41
mriedemxenserver ci seems borked15:41
mriedemBobBall: ^15:41
jaypipesdansmith: trying to keep the amount of code down15:41
dansmithjaypipes: it has to be underneath mine, regardless of how you want to do it15:41
dansmithjaypipes: seems like it might be most contextually-relevant in this one, but as long as it's in front of mine, I don't care15:42
PaulMurraydanpb, I guess strictly speaking asynchrony between two api servers could invalidate that check15:42
*** zenoway has joined #openstack-nova15:43
*** mtanino has joined #openstack-nova15:43
*** knikolla has quit IRC15:43
edleafejaypipes: want me to work on the singular patch?15:44
*** achanda has quit IRC15:45
mriedemVek: are you around to review https://review.openstack.org/#/c/278454/ ?15:45
mriedemjohnthetubaguy: might be worth sending the rax ping to Vek for ^15:45
mriedemsince we're approaching the freeze15:46
dansmithjaypipes: so I will base my current patch on your current one and put it up so people can start looking at it, even though it fails because it's not getting the latest stuff out of the db15:46
dansmithjaypipes: the save/create migrations work nice though15:46
*** tomoiaga has quit IRC15:46
mriedemjohnthetubaguy: btw, when did you plan on doing the final novaclient release for mitaka? i see the schedule says it's 2/29-3/4 for client FF15:46
mriedem*client release freeze15:46
*** zenoway has quit IRC15:47
*** rotbeard has quit IRC15:48
johnthetubaguymriedem: I was really hoping PaulMurray would say when all the live-migrate stuff lands, but that might be a bit tight15:48
mriedemjohnthetubaguy: that's blocked on ^15:48
mriedemhence the Vek ping15:49
johnthetubaguyah..15:49
mriedemhttp://zuul.openstack.xenproject.org/scoreboard/ is also down, so i think i'm going to block xenserver ci15:49
mriedemBobBall: ^ fyi15:49
*** y_sawai has joined #openstack-nova15:49
*** aorourke-afk is now known as aorourke15:49
PaulMurraymriedem, johnthetubaguy this patch is blocking others - needs approval https://review.openstack.org/#/c/278454/1315:49
PaulMurraymriedem, johnthetubaguy then next was previously approved, just rebased - needs +W again15:51
PaulMurrayThe next two need reviews but can be done quick enough15:52
*** e0ne_ has quit IRC15:52
PaulMurrayby reviews I mean core reviews15:52
*** mtanino has quit IRC15:52
PaulMurrayjohnthetubaguy, mriedem when is the freeze exactly?15:52
mriedemPaulMurray: release freeze for clients is 2/29-3/415:53
PaulMurraymriedem, I think we should be able to get these four patches in for that15:53
johnthetubaguyyeah, probably thursday morning15:53
PaulMurrayI panicked for a minute then, I thought you were going to say tomorrow15:54
openstackgerritAndrew Laski proposed openstack/nova: Create BuildRequest object during boot process  https://review.openstack.org/27812515:54
openstackgerritAndrew Laski proposed openstack/nova: Add build_requests database table and model  https://review.openstack.org/26392615:54
openstackgerritAndrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell  https://review.openstack.org/26392715:54
openstackgerritAndrew Laski proposed openstack/nova: Add BuildRequest object  https://review.openstack.org/27812415:54
openstackgerritAndrew Laski proposed openstack/nova: Make InstanceMappings.cell_id nullable  https://review.openstack.org/27859815:54
openstackgerritAndrew Laski proposed openstack/nova: Populate instance_mappings during boot  https://review.openstack.org/26392515:54
mriedemPaulMurray: +W on the 2.22 patch15:55
mriedemheh, 2.23 and 2.22 seem to be in the wrong order,15:56
mriedem2.22 allows you to cancel a live migration,15:56
mriedembut requires a migration id15:56
mriedem2.23 provides the API to get the migrations15:56
mriedemto use in 2.2215:56
mriedemoops15:56
*** angdraug has joined #openstack-nova15:58
PaulMurraymriedem, that's the way they were landed15:58
PaulMurrayin nova15:58
PaulMurrayactually you can get the id from the old migrations api15:58
PaulMurrayso from novaclient point of view it is wrong, but you can do it15:59
mriedemPaulMurray: yeah true http://developer.openstack.org/api-ref-compute-v2.1.html#os-migrations-v2.115:59
*** knikolla has joined #openstack-nova15:59
*** mgoddard_ has joined #openstack-nova15:59
mriedemso now we have 2 APIs to list migrations?15:59
johnthetubaguymriedem: almost yes16:00
PaulMurraythe old one is being deprecated16:00
PaulMurraywith adding the new one16:00
mriedemis there a simple way to explain why?16:00
mriedemwe can't ever remove the old one unless we bump the minimum required nova microversion16:00
PaulMurrayno16:00
PaulMurraythe new one is treating a migration as a sub resource of an instance16:01
PaulMurraythe old one lists all migrations in the system16:01
*** burt has joined #openstack-nova16:01
PaulMurrayit was 'decided' that the new one is inline with the direction we are taking in nova as a whole16:01
*** apearson has joined #openstack-nova16:02
bauzasmriedem: just a question coming out of mind now that we're close to FF, are you planning to provide a dot release for the liberty branch ?16:02
*** mgoddard has quit IRC16:03
mriedembauzas: yeah, it's on my todo list16:03
*** mjura has quit IRC16:03
bauzasmriedem: because I remember you claimed for review help on the stable branch16:03
openstackgerritPaul Murray proposed openstack/nova: Abort an ongoing live migration  https://review.openstack.org/27797116:03
*** mtanino has joined #openstack-nova16:03
mriedembauzas: yeah https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/liberty16:04
mriedemthere isn't much16:04
*** garyk has quit IRC16:04
bauzask16:05
bauzasalaski: I saw you having some merge conflicts16:05
bauzasalaski: for the cells patches16:05
bauzasalaski: fancy rebasing those ?16:05
openstackgerritDan Smith proposed openstack/nova: WIP: Migrate compute node stats to Inventory objects  https://review.openstack.org/27931316:06
openstackgerritDan Smith proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries  https://review.openstack.org/27931216:06
alaskibauzas: yeah, I just pushed up new ones about 10 minutes ago16:06
dansmithjaypipes: cdent ^16:06
*** harlowja_at_home has joined #openstack-nova16:06
*** dims has quit IRC16:06
bauzasalaski: ooooh16:06
mriedemalaski: i'll get on https://review.openstack.org/#/c/263926/ today16:06
bauzasalaski: so https://review.openstack.org/#/c/263926/ as a top notch ?16:07
alaskimriedem: thanks16:07
alaskibauzas: yeah, that's the starting point for that series16:07
*** moshele has quit IRC16:07
bauzask16:07
PaulMurraymelwitt, are you there? I'm looking for someone to review this patch on python-novaclient and I haven't tried you yet: https://review.openstack.org/#/c/278454/16:08
* bauzas refueling his review pipe16:08
PaulMurraymelwitt, it has a +216:08
*** mylu has joined #openstack-nova16:08
*** diogogmt has quit IRC16:08
*** drankis has quit IRC16:11
*** y_sawai has quit IRC16:13
*** sdague has joined #openstack-nova16:14
*** y_sawai has joined #openstack-nova16:14
openstackgerritShaoHe Feng proposed openstack/nova: Add new APIs and deprecate old API for migrations  https://review.openstack.org/25877116:15
*** daemontool__ has quit IRC16:16
openstackgerritPaul Murray proposed openstack/nova: Abort an ongoing live migration  https://review.openstack.org/27797116:17
*** y_sawai has quit IRC16:18
cdentpe16:18
mriedemalaski: at some point we might want to amend http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/cells-scheduling-interaction.html to point out there are data model impacts16:20
sdaguejohnthetubaguy: we completely agreed to the sudo like privsep16:20
sdagueduring the midcycle, with all the cinder folks plugged in16:20
sdagueI have no idea why this got raised again16:20
*** jlanoux has joined #openstack-nova16:21
*** y_sawai has joined #openstack-nova16:22
openstackgerritSwaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron networks  https://review.openstack.org/27507316:22
alaskimriedem: the data model impact is described in https://review.openstack.org/#/c/169901/4/specs/liberty/approved/add-buildrequest-obj.rst, which apparently didn't get carried over to M16:23
alaskimriedem: but since I've been assigning patches to the scheduling-interaction one I can update it16:23
*** jlanoux_ has joined #openstack-nova16:24
mriedemso http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/add-buildrequest-obj.html#data-model-impact16:24
*** haomaiwa_ has joined #openstack-nova16:24
openstackgerritShaoHe Feng proposed openstack/nova: Abort an ongoing live migration  https://review.openstack.org/27797116:24
openstackgerritShaoHe Feng proposed openstack/nova: Add new APIs and deprecate old API for migrations  https://review.openstack.org/25877116:24
alaskimriedem: yep16:24
*** jlanoux has quit IRC16:24
*** rtheis_ has joined #openstack-nova16:25
*** rtheis has quit IRC16:25
*** drankis has joined #openstack-nova16:26
*** belmoreira has quit IRC16:27
bauzasalaski: mriedem: yeah I had some concerns about the fields16:28
bauzasI had no time yet to ask why we needed both an instance_uuid again but also an Enum for some RBAC policy rules16:29
johnthetubaguysdague: I am glad you said that, I was thinking I was imagining things, I have VC links between rooms16:29
johnthetubaguys/have VC/hate VC/16:29
PaulMurrayshaohe_feng, I just saw you post a change - are you around?16:29
sdagueyeh, and I was pretty sure DuncanT spoke up during that conversation16:29
alaskibauzas: ultimately everything is in there in order to have the method in https://review.openstack.org/#/c/263927/4/nova/objects/build_request.py16:29
*** gszasz has quit IRC16:30
*** jlanoux_ has quit IRC16:30
*** claudiub|2 has quit IRC16:30
*** mylu has quit IRC16:30
bauzasalaski: oh the locked_by thing ?16:30
bauzasman, I hate that16:30
*** zenoway has joined #openstack-nova16:30
sdaguejohnthetubaguy / PaulMurray - was offline for a bit, how do the live migration API patches stand?16:31
*** jlanoux has joined #openstack-nova16:31
alaskibauzas: it's just copying what's on the instance16:31
*** mylu has joined #openstack-nova16:31
bauzasalaski: that said, https://review.openstack.org/#/c/263927/4/nova/objects/build_request.py can still get instance_uuid from the spec obj ;)16:31
bauzasalaski: yeah, I'm just double-checking that16:31
bauzasalaski: if so, I apologize for the Enum16:31
PaulMurraysdague, shaohe_feng and I just pushed new versions of https://review.openstack.org/25877 and https://review.openstack.org/27797116:32
*** davideagle_ has quit IRC16:32
alaskibauzas: it can. it's there in order to look up the BuildRequest by uuid. I don't love the duplication but it greatly simplifies querying16:32
PaulMurraysdague, oops, wrong patch16:32
bauzasalaski: right, I see16:32
bauzasalaski: you plan to have some BuildRequest.get_by_instance_uuid ?16:33
*** MCoLo has quit IRC16:33
PaulMurraysdague, https://review.openstack.org/277971 and https://review.openstack.org/25877116:33
bauzasalaski: if that, I was wondering why not joining with the ReqSpec16:33
alaskibauzas: https://review.openstack.org/#/c/278124/8/nova/objects/build_request.py16:33
*** silos has joined #openstack-nova16:33
mriedembauzas: alaski: i'm still going through it16:33
johnthetubaguysdague: yeah, there might be a few more open questions, but closer I hope16:34
johnthetubaguyI need to hit those16:34
bauzassince we agreed to use the RequestSpec initially, but provide a BuildRequest for any external items that weren't in the Spec object16:34
*** MCoLo has joined #openstack-nova16:34
*** Bjoern_ has joined #openstack-nova16:35
*** Bjoern_ is now known as Bjoern_zZzZzZzZ16:35
*** zenoway has quit IRC16:35
alaskibauzas: I can join on request_spec, I just took the easy route first16:35
*** claudiub has joined #openstack-nova16:36
alaskiI mean, I do join on that. But I can lookup by request_spec.instance_uuid rather than on build_request16:36
*** y_sawai has quit IRC16:36
bauzasalaski: I just want to make sure that RequestSpec is the initial main object, and BuildRequest is just an extending object for Cell V216:37
*** y_sawai has joined #openstack-nova16:37
*** Bjoern_zZzZzZzZ is now known as Bjoern_16:37
bauzasalaski: but I should think more of that16:37
openstackgerritGleb Stepanov proposed openstack/nova: Fix wrong watermark state during live migration  https://review.openstack.org/27961616:37
mriedemalaski: bauzas: where is the request_specs model defined?16:38
openstackgerritClaudiu Belu proposed openstack/nova: Adds HostCapabilities object model and updates ComputeNode object  https://review.openstack.org/22828016:38
openstackgerritClaudiu Belu proposed openstack/nova: Adds host_capabilities to HostState  https://review.openstack.org/25474716:38
openstackgerritClaudiu Belu proposed openstack/nova: scheduler: Adds HostCapabilitiesFilter filter  https://review.openstack.org/25507316:38
openstackgerritClaudiu Belu proposed openstack/nova: Adds ResourceTracker host capabilities update  https://review.openstack.org/25016616:38
openstackgerritClaudiu Belu proposed openstack/nova: driver: Adds get_hypervisor_capabilities method  https://review.openstack.org/25412416:38
openstackgerritClaudiu Belu proposed openstack/nova: db: Adds host_capabilities column in compute_nodes  https://review.openstack.org/22827916:38
*** salv-orlando has joined #openstack-nova16:38
alaskimriedem: db/sqlalchemy/api_models.py16:38
*** sheel has joined #openstack-nova16:39
mriedemoh gdi16:39
mriedemyeah,16:39
mriedemmy github was stuck on an old hash16:39
bauzasmriedem: so the Spec object is pretty small in the DB, just a blob and an index :)16:39
*** y_sawai_ has joined #openstack-nova16:39
bauzasthe index being the instance uuid16:39
*** silos has left #openstack-nova16:39
*** y_sawai has quit IRC16:41
*** salv-orlando has quit IRC16:41
sdaguePaulMurray: these seem good to me16:41
*** paul-carlton1 has quit IRC16:41
*** salv-orlando has joined #openstack-nova16:41
PaulMurraysdague, good - thanks - there is a novaclient patch that needs +W sitting in our way: https://review.openstack.org/#/c/278454/1316:42
PaulMurraysdague, I feel really bad because I've been bleating on about it to anyone on this channel - any chance you could look at that please ?16:43
sdaguePaulMurray: that patch is approved?16:43
sdaguePaulMurray: yeh, Vek got you 5 minutes ago16:44
PaulMurraysdague, ah - Vek got there - thanks Vek16:44
PaulMurraysdague, and I didn't even ask him !16:45
PaulMurraymaybe I should take something away from that....16:45
sdague:)16:45
*** y_sawai_ has quit IRC16:45
*** EinstCrazy has quit IRC16:46
mriedemooo boy gerrit is slow16:46
mriedemalaski: so on this build_requests object, with things like info_cache and instance_metadata, we're just avoiding new tables and joins on other tables and just storing serialized results of those in the build_requests entry?16:47
alaskimriedem: exactly. this is just short term storage until a cell is picked by the scheduler at which point that will all be written normally into the cell16:48
mriedemand if it is banished to cell0...16:49
mriedemthen we need enough to show details about the instance...16:49
*** y_sawai has joined #openstack-nova16:49
alaskicell0 is a normal cell db, so it can still write it all out properly16:49
*** y_sawai has quit IRC16:49
*** josecastroleon has quit IRC16:49
*** rdopiera has quit IRC16:50
mriedemwhat about the flavor?16:50
johnthetubaguyflavor only lives at the api_db layer, and cached in the instance record I thought?16:50
alaskimriedem: that's in the request_spec16:51
alaskimriedem: https://review.openstack.org/#/c/263927/4/nova/objects/build_request.py16:51
mriedemhttps://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/004_add_request_spec.py#L33 ?16:51
alaskithat generates the instance for the API16:51
alaskijohnthetubaguy: right. this is just clarifying where that exists before an instance record exists16:52
ababichsdague: Hi! could you please look at https://review.openstack.org/#/c/280147 &16:52
*** ivase has joined #openstack-nova16:53
*** y_sawai has joined #openstack-nova16:53
mriedemalaski: sorry, i'm looking at the request_specs table in the api db and i'm only see the spec blob16:53
mriedemhttps://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/004_add_request_spec.py#L3316:53
mriedemso i'm assuming the flavor info is in there,16:53
mriedembut where is that pulled out to populate the RequestSpec object to be used in https://review.openstack.org/#/c/263927/4/nova/objects/build_request.py16:53
*** mylu has quit IRC16:53
alaskimriedem: https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py16:54
alaskithat's a better model to look at16:54
mriedemso https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L36916:54
*** dims has joined #openstack-nova16:54
alaskiyep. the db just stores the primitive object16:54
*** amotoki has joined #openstack-nova16:54
*** mylu has joined #openstack-nova16:54
*** y_sawai has quit IRC16:55
*** browne has joined #openstack-nova16:55
mriedemok16:55
ababichsdague: Thanks!16:55
openstackgerritGleb Stepanov proposed openstack/nova: Console log copy for block live migratiob  https://review.openstack.org/28467416:56
mriedemsome of this code could use some documentation :)16:56
mriedem'wondering wtf this is?! go look here!'16:56
*** rfolco has quit IRC16:56
mriedembut meh16:56
mriedemonce it's in, if anyone ever asks i'll just direct them to alaski and bauzas16:56
alaskimriedem: heh. I do comment a lot in later patches when this gets used. But if you point out those bits I'm happy to document stuff16:57
*** yamahata has joined #openstack-nova16:58
johnthetubaguyare we close enough to have a "how your instance gets created in a cells v2 world" sort of document?16:58
johnthetubaguycovering the mapping, cell0, etc, would be fairly cool16:58
*** rfolco has joined #openstack-nova16:59
*** rfolco has quit IRC16:59
alaskijohnthetubaguy: the specifics are still spread out in specs, but the high level hasn't changed from https://github.com/openstack/nova/blob/master/doc/source/cells.rst16:59
alaskibut a lower level writeup would be good to have16:59
*** haomaiwa_ has quit IRC17:01
*** haomaiwang has joined #openstack-nova17:01
johnthetubaguyalaski: yeah, I guess the low level ones are the ones that quickly go out of date, maybe its all normal in a month or two17:01
*** y_sawai has joined #openstack-nova17:02
mriedemalaski: alright, -1 on https://review.openstack.org/#/c/263926/ with comments inline17:04
mriedembauzas: fyi http://docs.sqlalchemy.org/en/rel_1_0/core/metadata.html?highlight=nullable#sqlalchemy.schema.Column.params.nullable17:04
*** thorst is now known as thorst_afk17:04
mriedemnullable=True is the default17:04
cdentdansmith: Is an objects.Aggregate.get_by_uuid already in progress?17:04
*** sfinucan has quit IRC17:06
mriedembauzas: you might be good to review this backport https://review.openstack.org/#/c/276191/17:06
*** zenoway has joined #openstack-nova17:06
alaskimriedem: bauzas that's why I don't use the default, because it's hard to remember what it is :)17:07
*** mgoddard_ has quit IRC17:07
mriedemthere was an epic bikeshed in a cinder meeting a few weeks ago for a similar discussion17:07
*** mgoddard has joined #openstack-nova17:07
smcginnisWe like bikes.17:07
openstackgerritKevin Benton proposed openstack/nova: Mark 'network_device_mtu' as deprecated  https://review.openstack.org/28481817:08
*** jistr has quit IRC17:08
*** cdent has quit IRC17:09
alaskiif it was clear what the default would be then I would be all for it, but in this case one isn't more clearly right17:09
alaskithough if I had to pick I would expect the default to be False17:10
mriedemi just remember jogo going through the model once and dropping all of the default=True kwargs17:10
mriedemprobably b/c we had default=False, default=True, and some with neither specified17:10
mriedemso i've tended to try and keep that enforced17:10
*** zenoway has quit IRC17:10
mriedem*RIP jogo*17:11
mriedem*sniff*17:11
alaskiheh17:11
alaskiI'm fine dropping the nullable=True17:11
dansmithI hate the assumption that nullable is true, fwiw17:12
wznoinskalex_xu and/or other cores, could someone have a look at my doc fixing and let me know is it GTG or not? (https://review.openstack.org/#/c/280299/)17:12
bauzasmriedem: man, that happens when you review when doing 2 things at the same time... forgetting the default value :)17:12
*** scheuran has quit IRC17:12
* bauzas bbiab17:12
*** thorst_afk is now known as thorst17:12
*** paul-carlton1 has joined #openstack-nova17:16
*** drankis has quit IRC17:17
*** RuiChen has quit IRC17:19
*** penick has joined #openstack-nova17:20
*** RuiChen has joined #openstack-nova17:20
*** krtaylor has quit IRC17:21
*** READ10 has joined #openstack-nova17:22
*** mylu has quit IRC17:22
*** Swami has quit IRC17:25
*** haplo37 has joined #openstack-nova17:25
*** su_zhang has quit IRC17:26
*** su_zhang has joined #openstack-nova17:26
mriedemdansmith: you had a bug open for nixing lazy-load of things from the instance object right?17:27
dansmithmriedem: things in general17:28
mriedemnvm i'll find it17:28
mriedembug 154052617:28
openstackbug 1540526 in OpenStack Compute (nova) "Too many lazy-loads in predictable situations" [Undecided,In progress] https://launchpad.net/bugs/1540526 - Assigned to Dan Smith (danms)17:28
dansmithcool17:28
sheelDear Core members:17:28
sheelneed some eyes on https://review.openstack.org/#/c/282862/17:28
sheelplease find some spare time to review this small patch17:29
sheelthank you!!17:29
*** nic has joined #openstack-nova17:29
mriedemsheel: didn't that come up in the cross-project meeting last week?17:30
mriedemdamn cdent isn't around17:30
mriedemjaypipes: ^17:30
dansmitharen't we -2ing that?17:30
dansmithsdague: ?17:30
sheelmriedem: actually this is not related to db changes17:30
*** yassine__ has quit IRC17:30
*** su_zhang has quit IRC17:30
*** rtheis_ is now known as rtheis17:31
mriedemsheel: yeah, it's changing the 500 to a 40017:31
sheelmriedem: yep17:31
openstackgerritGleb Stepanov proposed openstack/nova: Remove an unnecessary variable in a unit test  https://review.openstack.org/28357517:31
*** yassine__ has joined #openstack-nova17:31
dansmithoh is this just checking for and handling the fail gracefully?17:32
mriedemi'd like to see that handled more generically17:32
mriedemyeah17:32
mriedemrather than copying that around every getter in the db api17:32
*** krtaylor has joined #openstack-nova17:33
*** y_sawai has quit IRC17:33
jaypipessheel: I can try to do a review of that later today. right now I'm working on a patch for dstepanenko.17:33
sheeljaypipes: thank you17:34
mriedemsheel: more comments inline17:34
mriedemhandling in the REST API would be cool if possible, barring that i'd think an error handler wrapper in the db api would be at least mroe re-usable17:34
sheeldansmith: yes, its regarding handling fail17:35
*** y_sawai has joined #openstack-nova17:35
mriedemhonestly, this seems like something we could push down into oslo.db17:35
*** cdent has joined #openstack-nova17:35
mriedembut we'd still have to handle the oslo.db error and return a 40017:35
sheelmriedem: yep, we already had pushed this in oslo db17:35
*** zenoway has joined #openstack-nova17:35
*** achanda has joined #openstack-nova17:36
mriedempatch?17:36
openstackgerritMark Doffman proposed openstack/nova: Add manage command for cell0.  https://review.openstack.org/27526617:36
openstackgerritMark Doffman proposed openstack/nova: WIP: Map instances to cell0 on failed schedule  https://review.openstack.org/26782717:36
openstackgerritMark Doffman proposed openstack/nova: Add a cell and host mapping utility to nova-manage  https://review.openstack.org/27056517:36
*** cdent has quit IRC17:36
openstackgerritMark Doffman proposed openstack/nova: Manage db sync command for cell0  https://review.openstack.org/27488517:36
*** davideagle has joined #openstack-nova17:36
sheelmriedem: https://review.openstack.org/#/c/265921/4/oslo_db/tests/sqlalchemy/test_exc_filters.py17:36
sheelhttps://review.openstack.org/#/c/265921/417:36
doffmalaski: Could you take a quick look at https://review.openstack.org/26782717:36
*** cdent has joined #openstack-nova17:36
*** y_sawai has quit IRC17:37
doffmJust the code is up, i'm fighting with the tests for that patch. The conductor unit tests are messing with my mind. :/17:37
openstackgerritEd Leafe proposed openstack/nova: Use new inventory schema in all compute_node gets  https://review.openstack.org/28483317:37
doffmWant to make sure the idea is right before I keep fighting with the tests.17:37
edleafejaypipes: dansmith: first crack at genericizing the compute_node query changes17:37
edleafejaypipes: dansmith: ^^17:38
*** mrkz has quit IRC17:38
*** mrkz has joined #openstack-nova17:38
*** jwcroppe has quit IRC17:38
mriedemsheel: so we need that oslo.db change to properly detect this for mysql and raise the DBDataError exception, right?17:39
*** jwcroppe has joined #openstack-nova17:39
sheelmriedem: right17:39
mriedemsheel: that went into oslo.db 4.4.017:39
mriedemglobal-requirements requires oslo.db>=4.1.017:39
sheelmriedem: oops17:39
mriedemso you need to raise the minimum required version of oslo.db in global-requirements to be sure we're getting that change from oslo.db17:39
sheelmriedem: yes, right17:40
sheelmriedem: good point17:40
johnthetubaguywe might be very close to the deadline for changing that I guess17:40
alaskidoffm: commented17:41
*** Swami has joined #openstack-nova17:42
doffmalaski: Thanks.17:42
sheeljohnthetubaguy: hmm, deadline is much closer..17:42
*** mylu has joined #openstack-nova17:43
*** jwcroppe has quit IRC17:43
claudiubhello. if anyone can take a look at tiny bugfix, that already has a +2, i'd be greatful. :) https://review.openstack.org/#/c/258614/17:43
openstackgerritMatt Riedemann proposed openstack/nova: DNM: see where instance.pci_devices is getting lazy-loaded  https://review.openstack.org/28483917:45
jaypipesedleafe: reviewed. right track, can be limited though to a lot fewer lines of code changed.17:46
jaypipesok, back to dstepanenko's patch :)17:46
*** y_sawai has joined #openstack-nova17:46
* jogo waves to mriedem17:46
edleafejaypipes: ok, will take a look17:46
mriedemhe's alive!17:46
cdentjaypipes: before you go, also dansmith your input desired too17:46
mriedemclaudiub: hyper-v ci?17:46
*** _ducttape_ has joined #openstack-nova17:46
mriedemi guess it was ok on 2/517:46
claudiubmriedem: PS5: Build succeeded.17:46
DuncanTsdague: It was me who spoke up. I underestimated how like the current model the new privsep temporary thing was going to be, but having seen a bit more where it is going I'm happy again17:47
cdentjaypipes, dansmith: Working on ResourcePool it seems I either need to add get_by_uuid to Aggregate or just bypass using the object and go straight to the db. This is a question I hit often when working on object stuff. When do you chose to skip objects for related pieces.17:47
dansmithcdent: you never bypass17:47
dansmithcdent: and yes, you should add a get_by_uuid() I just didn't because I didn't need it17:48
jaypipesI thought dansmith had a patch up that added uuid to aggregate?17:48
cdent^^17:48
dansmithjaypipes: just no query method for it17:48
cdentI'll do that now and then put what I'm working on underneath it17:48
dansmithyou mean above17:48
* dansmith has strong feelings on patch-queue-endianess17:48
cdentI hear what you are saying but that just not how my mind works dansmith17:48
*** dedery has quit IRC17:49
*** ljxiash has quit IRC17:49
alaskigerrit display is canonical IMO17:49
*** ducttape_ has quit IRC17:49
dansmithalaski: which means I'm right, right?17:49
cdentgerrit's UI should never be considered canonical for anything. to say otherwise is pure stockhold syndrome17:49
dansmithstockholm?17:50
cdentstockhold, huh, that's about right17:50
cdentI type by syllables apparently17:50
DuncanTsdague: It came round again because the patch looked *nothing* like privsep was supposed to, rather than a bit like it but with lax validation, which is what I was expecting17:50
alaskidansmith: depends, I'm not sure what you meant. to me below=before, above=after17:51
sdagueDuncanT: ok.17:51
johnthetubaguysheel: yeah, I was just checking it wasn't yesterday really :)17:51
dansmithalaski: correct17:51
mriedemclaudiub: +W17:51
*** apoorvad has joined #openstack-nova17:52
sheeljohnthetubaguy: so we have time for now for this...17:52
dansmithedleafe: are you swizzling that patch now?17:52
mriedemsdague: i think you will totally be in love with this now https://review.openstack.org/#/c/283837/17:52
*** y_sawai has quit IRC17:53
*** paul-carlton1 has quit IRC17:53
*** sc68cal has quit IRC17:53
sdaguemriedem: ideally you'd do this as an expectFailure17:54
*** y_sawai has joined #openstack-nova17:54
sdaguemriedem: https://review.openstack.org/#/c/283261/3/nova/tests/functional/regressions/test_bug_1541691.py17:54
*** matrohon has quit IRC17:54
*** browne has quit IRC17:54
sdagueself.expectFailure("some message", ... )17:55
sdaguethen the test you expect to work17:55
*** claudiub has quit IRC17:55
*** abitha has joined #openstack-nova17:55
*** rmart04 has joined #openstack-nova17:55
*** amotoki has quit IRC17:56
openstackgerritEd Leafe proposed openstack/nova: Use new inventory schema in all compute_node gets  https://review.openstack.org/28483317:56
sdaguealthough, because of the asserts off the client, that might not work here17:56
sdaguenever mind, good with it17:57
cdentIs this worth fixing: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L549417:57
*** jlanoux has quit IRC17:57
cdent(writer being used where reader should be)17:57
sheelmriedem: thanks for  your comments, quite qualitative comments17:57
openstackgerritNikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448917:57
* sdague adds a non fatal mode for the client as a thing we should do 17:57
ndipanovPaulMurray, bauzas ^^17:57
sheelmriedem: response to your comments, (mentioning here to fasten up the process :) )17:57
sdaguecdent: yes17:57
ndipanovfinally got around to resurrecting this one17:57
sheelhttps://www.irccloud.com/pastebin/oAbNytAl/17:57
ndipanovplease take a look guys17:57
cdentsdague: should I make a bug or just fix it?17:57
sdaguecdent: just fix it17:58
cdentroger17:58
sdaguethere was a blueprint for all the engine facade work, you could tag it with that17:58
*** paul-carlton1 has joined #openstack-nova17:58
*** jwcroppe has joined #openstack-nova17:58
sdaguesheel: shouldn't we catch this at the json schema layer?17:58
PaulMurrayndipanov, thanks, will do17:59
*** y_sawai has quit IRC17:59
sheelsdague: yes, but for now there is some bug in that17:59
sheelsdague: extra characters around about  of 1000 lines17:59
sheelin output if 4 byte unicode is provided18:00
sheelI could see same in keystone and nova18:00
sheelin json schema validation18:00
sdaguesheel: this bug - https://bugs.launchpad.net/nova/+bug/1541691 ?18:00
openstackLaunchpad bug 1541691 in OpenStack Compute (nova) "server boot with leading and trailing white spaces in name, displays weird error message" [High,In progress] - Assigned to Alex Xu (xuhj)18:01
*** haomaiwang has quit IRC18:01
sdagueor something else18:01
*** hemna has joined #openstack-nova18:01
PaulMurraymriedem, did I see you say that xenserver CI is playing up?18:01
sheelyes this as well as one other(I raised that)18:01
*** haomaiwang has joined #openstack-nova18:01
sheelsdague: but both represent same18:01
dansmithedleafe: we need the result of compute_node_update to do it as well :/18:01
sdaguethe fixes for 1541691 are all in flight and will probably land this week18:01
*** mylu has quit IRC18:02
*** bjornar__ has joined #openstack-nova18:02
sdagueonce that is fixed... what more is missing?18:02
sheelsdague: I think we can use that then18:02
sheelsdague: once fixed18:02
sdagueok, it seems like security groups don't get jsonschema validated today18:03
*** derekh has quit IRC18:03
sdagueso I guess we'd need schema for that as well?18:03
sheelsdague: yes18:03
sdagueI'd *much* rather catch this as the API then bounce it off the db18:03
openstackgerritChris Dent proposed openstack/nova: Make db.aggregate_get a reader not a writer  https://review.openstack.org/28484618:04
sheelsdague: agree o nit18:04
cdentsdague: I reckon the amont of effort that ^^ change required justifies my wages18:04
sheelsdague: agree on it18:04
sdaguecdent: heh18:05
sheelsdague: but I need to see wat needs to fix in schema way18:05
sheelsdague: may be require more time for that18:05
sdagueok, how about you investigate a schema fix for this18:06
sdaguejsonschema that is18:06
sdagueto enforce at the API layer18:06
sdagueand see if we can get a suitable fix there18:06
sheelsdague: yes, that seems acceptable way of fixing18:06
*** ndipanov has quit IRC18:07
*** lucasagomes is now known as lucas-afk18:08
sdagueBobBall: xenserver-ci needs to update their devstack-gate pull18:08
*** lpetrut has quit IRC18:08
*** mingyu has quit IRC18:09
*** mingyu has joined #openstack-nova18:09
*** READ10 has quit IRC18:10
sheelsdague: so, you want me to abandon this and work on jsonschema thing.... or go with this one in this version and update with jsonshema in next release?18:12
*** Bjoern_ is now known as Bjoern_zZzZzZzZ18:12
sdaguesheel: I want to see what a json schema fix looks like18:12
sdagueto see if it can fully address the issue18:12
sheelsdague: ok, sure18:12
sdaguebecause if it can, I don't think we want the db patch18:12
*** rmart04 has quit IRC18:12
sdagueif it can't, we can discuss the db patch18:12
sheelsdague: sounds good18:13
sdaguesheel: thanks18:13
sheelsdague: thank you for discussion..18:13
dansmithedleafe: actually, scratch that, based on how compute_node_update() works18:13
*** su_zhang has joined #openstack-nova18:15
*** mingyu has quit IRC18:16
*** mingyu has joined #openstack-nova18:16
edleafedansmith: what about the compute_node_statistics method?18:16
dansmithedleafe: I dunno18:16
edleafedansmith: it didn't break any tests, so I had my fingers crossed :)18:17
*** wanghao has quit IRC18:17
dansmithyeah, let's ignore for now18:17
*** Bjoern_zZzZzZzZ is now known as Bjoern_18:17
*** wanghao has joined #openstack-nova18:17
*** annasort has quit IRC18:19
*** penick has quit IRC18:20
*** rk4n has quit IRC18:20
* edleafe heads off to lunch18:21
dansmithedleafe: thanks for doing that18:22
*** ihrachys has quit IRC18:24
sdagueso.... why are we doing that _do_test() pattern instead of context manager mock?18:25
*** penick has joined #openstack-nova18:25
sdaguebecause - https://review.openstack.org/#/c/281915/1/nova/tests/unit/compute/test_compute_mgr.py shows the definite danger of it18:25
alaskifor one mock it shouldn't be used18:26
alaskiwhen you have multiple that pattern is much cleaner, though admittedly dangerous18:26
sdagueyeh, the danger seems pretty bad18:27
sdagueit's getting cargo culted all over the place18:27
dansmithit's infinitely better, IMHO18:27
alaskiI seem to recall there was some issue with using multiple context managers and line breaks18:27
dansmithwe have cases where we assert mock properties that don't exist too18:28
sdaguedansmith: except for the part where people forget the _dot_test18:28
dansmithyeah, the context mock is just terrrrrible18:28
dansmithsdague: yeah, I get it.. I'm saying we assert mock properties that don't exist too18:28
sdaguedansmith: sorry, explain?18:28
dansmithself.assertTrue(mymockything.is_called)18:28
dansmithpasses because is_called is truthy but we didn't actually call the thing18:29
dansmithbecause it's .called not .is_called18:29
sdagueok18:29
sdaguemock protects us from a bunch of that now18:29
dansmithit's gotten a little better yeah18:29
dansmithbut the nature of the thing doesn't really make it possible to protect it all18:30
dansmithI'm just saying, please don't make use use the terrible context mock just because you can forget to call your thing18:30
sdaguesure, ok, but that's sort of a different issue18:30
dansmithsame class of thing to me18:30
sdaguedansmith: why is it terrible?18:30
dansmithbecause it's so messy and the with statement doesn't allow for line breaks in certain confusing cases18:31
dansmithand it manifests really strangely18:31
dansmithespecially when you just get the usual no tests ran, subunit explosion, or whatever because it's a parse error18:31
dansmithand multi-line with statements even where it's allowed is a lot uglier and harder to read than a nice neat stack of decorators18:31
dansmithI'm surprised flake8 doesn't notice that a thing was defined and never called or referenced18:32
sdagueok, so why not have that decorator stack on top of the test_ function?18:32
dansmithI bet pylint would catch it18:32
*** PaulMurray has quit IRC18:32
dansmithsdague: because you can't patch objects that way18:32
dansmithbecause they don't exist18:32
alaskisdague: you can't when you're patching 'self'18:32
*** salv-orlando has quit IRC18:32
openstackgerritGleb Stepanov proposed openstack/nova: Fix backing file copying  https://review.openstack.org/28227518:33
dansmithI mean, you can patch objects, but not things you have to create first18:33
*** hemna has quit IRC18:33
*** maishsk has joined #openstack-nova18:33
*** maishsk has left #openstack-nova18:34
dansmithedleafe: jaypipes: I think something is broken in that get patch.. I'm setting a column and then getting the thing back and it's different18:35
sdaguedansmith: ok, but you also missed it here - https://github.com/openstack/nova/commit/2d7a851199b12355b36a3982ea0fc7bb5a7cd11f18:35
sdagueit just seems like the current pattern is pretty error prone18:35
sdagueand hard to catch correctly in review18:36
dansmithsdague: well, I suck, but seriously the alternative is not better, IMHO18:36
dansmithwhy is it hard to catch?18:36
dansmithdefining a function with no call or reference to it seems pretty obvious to me18:36
sdaguedansmith: ok, if you want to build somethign for it. But we're clearly landing tests that don't actually run right now18:37
dansmithsdague: where else did I miss it in that patch?18:37
sdaguein that patch, that's just the one instance18:37
dansmithah, 4333 O see18:37
dansmithoh, no, nevermind18:38
sdaguebut that's actually not fixed by https://review.openstack.org/#/c/281915/1/nova/tests/unit/compute/test_compute_mgr.py18:38
sdagueI just decided to read the rest of the file to see if there were other instances18:38
sdagueand there were18:38
*** salv-orlando has joined #openstack-nova18:39
dansmithyep, and you know why? I remember this now18:39
*** haomaiwang has quit IRC18:39
dansmiththis was such a disaster of a thing to maintain18:39
dansmithI consistently had git merge out those lines because it thought they were the same18:39
*** rodrigods has quit IRC18:40
*** rodrigods has joined #openstack-nova18:40
dansmithwhich can happen to anything where you have some repetition and git thinks it is smart18:40
dansmithif you look at that diff you see why it thinks it's right18:40
*** hemna has joined #openstack-nova18:40
sdaguesure, I get it, I'm just questioning a pattern where there are many failure modes, including git trying to delete it out fromus18:41
dansmithwell, I question who would ever prefer the alternative.. so there :)18:41
dansmithare you going to comment on that one patch to fix up the rest or should I do it separately?18:42
*** paul-carlton1 has quit IRC18:42
sdagueI have an outstanding draft, I was just trying to understand more18:43
dansmithokay18:43
sdagueI left a comment on it18:43
sdagueI'm still +2, because it fixes a real thing18:44
*** haomaiwang has joined #openstack-nova18:44
sdaguebut there are clearly other things to be fixed18:44
dansmithokay, I will keep hacking on this rpools thing, but if it doesn't get fixed in this patch I'll gladly do it18:44
*** haomaiwang has quit IRC18:49
openstackgerritPavel Kholkin proposed openstack/nova: WIP: keypairs pagination bp  https://review.openstack.org/28486818:49
*** hemna has quit IRC18:52
mriedemgdi, now xenproject ci is hosed too18:53
mriedemi'm going to turn off both i think18:53
openstackgerritVladik Romanovsky proposed openstack/nova: Clean up networks with SR-IOV binding on reschedule  https://review.openstack.org/28319818:53
mriedemthe stakes are high and so am i ..18:53
sdaguemriedem: right, because of the ansible thing18:53
sdagueapparently they don't take latest devstack-gate18:54
*** mvk has joined #openstack-nova18:54
mriedemthe xenserver ci log i looked at this morning was taking that actually18:54
mriedembut still failed18:54
sdaguemriedem: really?18:54
sdaguenot the one I just looked at18:54
sdaguehttps://review.openstack.org/#/c/284846/18:54
*** y_sawai has joined #openstack-nova18:55
sdague2016-02-25 18:11:01.009 | ++ sudo -H /tmp/ansible/bin/pip install ansible18:55
sdaguethey think they have that code, but they aren't using it?18:55
mriedemalright, xenproject ci also removed18:56
sdagueyeh, they have a fork18:56
sdague2016-02-25 18:10:48.407 | + git clone https://github.com/citrix-openstack/devstack-gate -b master18:56
*** lpetrut has joined #openstack-nova18:56
sdaguewhich, you know, kind of pointless :)18:56
*** hemna has joined #openstack-nova18:58
mriedemsdague: damn, that missing call to _test() in that one test you pointed out, adding it makes the test fail - so the test is busted or sometihng :(18:59
mriedemi'll see if i can figure it out18:59
*** Marga_ has quit IRC19:00
openstackgerritSean Dague proposed openstack/nova: Deprecate conductor manager option  https://review.openstack.org/28433219:00
*** y_sawai has quit IRC19:00
*** davideagle has quit IRC19:00
*** yassine__ has quit IRC19:01
sdaguemriedem: or the code is busted19:01
*** mingyu has quit IRC19:01
sdaguemriedem: on the fixing tests front - https://review.openstack.org/#/c/282983/1 is another good one19:02
*** davideagle_ has joined #openstack-nova19:02
sdagueif you want to approve that19:02
*** mingyu has joined #openstack-nova19:02
cdentdansmith: I've done aggregate_get_by_uuid but I'm not sure where to put it. I need it for the resourcepool stuff, which will slot "above" this: https://review.openstack.org/#/c/281945/ but that stack is stalled pending approval of the modification to the resource providers spec. Should I put the aggregate stuff in that stack, or by itself on the hope/assumption that it will get merged sooner?19:04
*** josecastroleon has joined #openstack-nova19:06
*** vilobhmm11 has joined #openstack-nova19:06
*** inc0 has joined #openstack-nova19:07
openstackgerritThomas Maddox proposed openstack/nova: Try udevadm settle between lvremove attempts  https://review.openstack.org/24061119:07
SamYapleinc0: ping19:08
inc0so guys, question about upgrade_levels19:08
inc0in kolla we do upgrades of nova in a way that we upgrade conductors first and rest of it later19:08
openstackgerritAndrew Laski proposed openstack/nova: Create BuildRequest object during boot process  https://review.openstack.org/27812519:09
openstackgerritAndrew Laski proposed openstack/nova: Add build_requests database table and model  https://review.openstack.org/26392619:09
openstackgerritAndrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell  https://review.openstack.org/26392719:09
openstackgerritAndrew Laski proposed openstack/nova: Add BuildRequest object  https://review.openstack.org/27812419:09
openstackgerritAndrew Laski proposed openstack/nova: Make InstanceMappings.cell_id nullable  https://review.openstack.org/27859819:09
openstackgerritAndrew Laski proposed openstack/nova: Populate instance_mappings during boot  https://review.openstack.org/26392519:09
inc0but sighupping nova services aftes whole process is done seems to be breaking (at least inside docker)19:09
*** penick has quit IRC19:09
inc0do we really need it with upgrade_levels=auto?19:09
alaskimriedem: bauzas I updated the cells patch series19:09
inc0dansmith ^19:10
sdaguejaypipes: https://review.openstack.org/#/c/284257 - responded to your question19:10
sdaguemissed it when I was rebasing last night19:10
dansmithinc0: =auto if you want to avoid doing manual pins, and I hope to remove manual pins at some point19:11
jaypipessdague: ah, gotcha. cool, thanks for the explanation19:11
*** baoli_ has quit IRC19:11
*** angdraug has quit IRC19:12
inc0dansmith, and all of nova services should handle sighup for it right?19:12
dansmithyes19:12
inc0or do we need to restart19:12
*** _ducttape_ has quit IRC19:12
SamYapledansmith: the question is actually do we _need_ to sighup or can we restart (is restarting going to give the same effect)?19:16
SamYaplethe issue here is with docker and sighup and we dont have alot of time to track it down atm19:16
dansmithSamYaple: yeah same effect19:17
SamYaplei thought so. thanks dansmith !19:17
dansmithyep19:18
*** skoude has joined #openstack-nova19:18
*** jichen has joined #openstack-nova19:19
skoudehmm.. I have a problem that instances try to access http://169.254.169.254/2009-04-04/meta-data/instance-id when instead they should access https://169.254.169.254/2009-04-04/meta-data/instance-id.. Any idea where to change the address?19:20
SamYapledansmith: to clear up some confusion. we would need to restart/sighup all nova services after the last nova-compute node is upgraded. does that mean _also_ the nova-compute services again?19:20
skoudeTried to find solution, but haven't still found one..19:20
dansmithSamYaple: yeah19:20
SamYapleok we can work with that. thanks again dansmith19:21
skoudeI have defined nova_metadata_protocol = https, but still it does not connect as https19:21
*** fawadkhaliq has joined #openstack-nova19:22
openstackgerritChris Dent proposed openstack/nova: Add name to ResourceProvider object  https://review.openstack.org/28194519:24
*** markusz has quit IRC19:25
*** ducttape_ has joined #openstack-nova19:27
sdagueskoude: it's cloud init19:28
*** Sukhdev has joined #openstack-nova19:29
*** salv-orlando has quit IRC19:29
*** andymaier has quit IRC19:30
*** redhat034 has joined #openstack-nova19:30
mriedem'at least you can't complain about good american spooning'19:30
mriedemnice quote from the neutron midcycle19:30
skoudesdague: yeap, I know, but where do I define it globally? It worked before, but suse upgraded some packages and now it tries to connect http instead of https.. And I'm trying to figure out where is that config.19:31
openstackgerritChris Dent proposed openstack/nova: Add a get_by_uuid for aggregates  https://review.openstack.org/28489519:31
sdaguemriedem: heh19:32
mriedempeople complaining about sharing a hotel room19:32
*** redhat034 has quit IRC19:33
*** su_zhang has quit IRC19:35
*** josecastroleon has quit IRC19:35
*** su_zhang has joined #openstack-nova19:36
*** Marga_ has joined #openstack-nova19:36
mriedemsdague: https://review.openstack.org/#/c/282983/ scares me19:36
*** salv-orlando has joined #openstack-nova19:36
sdaguemriedem: I'm definitely partially to blame on that one19:37
sdaguebecause I was reviewing that code, but we were doing so many iterations at the end I missed those19:37
sdagueand was focussed on other issues with those patches19:38
sdagueat least on the libosinfo bits19:38
sdaguemriedem: also - because you loved this patch so much the first time - https://review.openstack.org/#/c/284257/ jaypipes +2ed it19:39
bauzasalaski: got a wonder here https://review.openstack.org/#/c/263926/13/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/006_build_request.py@6519:39
sdaguebauzas: what is your magic to get @ into the urls?19:39
bauzassdague: thanks to gerrit19:40
mriedemsdague: i had it open in a tab,19:40
bauzassdague: just see the hyperlink19:40
mriedembut then saw you and dan fighting like an old married couple19:40
*** su_zhang has quit IRC19:40
bauzaswhen commenting19:40
mriedemand now i'm fixing that other test thing19:40
bauzasalaski: btw. FWIW will be on PTO tomorrow19:41
sdaguebauzas: nope, I don't see how to do it.19:41
*** su_zhang has joined #openstack-nova19:42
openstackgerritJay Pipes proposed openstack/nova: Added supports_requests method to PciDeviceList  https://review.openstack.org/24208719:42
bauzassdague: for example, look at a comment19:42
bauzassdague: you'll see every line having like line <nr> : my_comment19:42
alaskibauzas: I'm out tomorrow as well19:43
alaskibauzas: are you suggesting pulling the request_spec into the build_request?19:43
*** toure_ is now known as toure19:43
bauzasalaski: no, just removing the FK19:43
sdagueoh...19:43
bauzaswe're already constrained by the UC19:43
sdagueyou have to leave a comment, then get it back at the top page19:43
bauzassdague: well, that's an hyperlink, but you can still provide any line number19:44
sdaguesure19:44
sdagueI just wanted it in the diff view19:44
alaskibauzas: right, but that makes no guarantee that there is a request_spec19:44
bauzasalaski: so the FK is useful for 3 points :19:44
bauzas#1 the UC it creates19:44
*** markusz has joined #openstack-nova19:44
bauzas#2 the parallel removal it could have when deleting the parent key19:45
*** haomaiwang has joined #openstack-nova19:45
bauzas#3 for making sure that we have a correct id19:45
alaskiyeah, I'm interested in #319:46
bauzasalaski: like I said, #1 is made by having an UC, and I think we can have #2 and #2 just by coding that19:46
mriedembut if you bork up coding that,19:46
mriedemthe fkey is there to save you19:46
mriedemif you delete the request_spec before the build_request19:46
mriedemor attempt to19:46
alaskiI'm not really sure what I gain by not having it19:46
Swamihttps://review.openstack.org/#/c/275073/ could I get some reviews on this patch - this for handling live migration in neutron.19:46
bauzasalaski: I need to remember some old convo I saw about that one19:47
bauzasmriedem: sure, but since we have a DB Facade, I thought it's just okay19:47
alaskiI know that some people really don't like them. But I think they add value19:47
bauzaslike the host/service lazy relationship we have19:47
bauzasalaski: honestly, it's not a strong objection to having a FK19:48
*** lpetrut has quit IRC19:48
bauzasalaski: I'm not a DB guru so I'm taking both mriedem and your's points19:48
bauzasalaski: but I just feel it's not really needed for our use, that's it :)19:49
mriedemshall we summon zzzeek and morded since we're talking about foreign keys? :)19:49
mriedemlet's get ready to rumble!!!!19:49
bauzasheh19:49
alaskioh boy19:49
doffmmriedem: Is there some special summoning dance?19:49
mriedemno19:49
bauzaswho is calling the kracken ?19:49
*** baoli has joined #openstack-nova19:49
mriedemdifferent epic19:50
doffmWith the DB split there are going to be a number of relationships that arn't enforced by the DB schema.19:50
bauzass/calling/awakening19:50
*** shyama has quit IRC19:50
alaskibauzas: the likelihood of a foreign key catching an issue is low right now because we're being careful adding all of this. But as we start messing with request_specs later I prefer to have the sanity check19:50
*** krotscheck is now known as krotscheck_dcm19:50
*** baoli has quit IRC19:51
bauzasalaski: what I'm concerned is by keeping a loose relationship between both19:51
bauzasalaski: because the Spec object is aimed to go down to the compute ideallyu19:51
doffmBut if the tables are in the same db... why not have the extra enforcement?19:52
mriedemsdague: got that test passing, pushing up a rev soon19:52
mriedemi'll just +W myself19:52
bauzasthat's fine to say that it's just an instance and that the persisted object will still be in the API DB, but I feel that conceptually, we only need to have this partnership for the initial call19:52
sdaguemriedem: sure, or show me the link and I'll wallaby19:53
bauzasdoffm: well, the point is that it doesn't scale great19:53
*** gmurphy has joined #openstack-nova19:53
*** haomaiwang has quit IRC19:53
bauzasbut since it's the API DB, I can say it's probably a nitpick19:53
bauzasbut I dunno19:53
alaskibauzas: the buildrequest is short lived though. so the partnership goes away quickly19:53
bauzasagreed19:54
*** lpetrut has joined #openstack-nova19:54
bauzasokay, it seems there is a consensus of 3 people having good opinions and me just picky with my +119:54
bauzas:)19:54
bauzasalaski: okay, will turn my comment19:55
alaskithere are definitely times that they don't make sense. I just don't see a downside for this usage and I like the extra validation, since we absolutely have to have a request_spec or the build_request is useless19:55
*** hemna has quit IRC19:56
*** eil397 has joined #openstack-nova19:56
*** sc68cal has joined #openstack-nova19:57
bauzasalaski: tbc, my concerns are pretty well explained in https://www.percona.com/blog/2006/12/12/innodb-locking-and-foreign-keys/19:57
*** y_sawai has joined #openstack-nova19:57
bauzasI'm not saying it's the case here, just me trying to avoid FKS whenever it's possible19:57
bauzas(that's an old blogpost I read a couple of years before, probably the mysql engine is performing better now)19:59
alaskilooks like it's more a concern for write heavy tables, which these are not20:00
*** danpb has quit IRC20:00
bauzasagreed20:00
bauzasit just scales per request :)20:00
*** ljxiash has joined #openstack-nova20:01
*** rk4n has joined #openstack-nova20:01
openstackgerritJay Pipes proposed openstack/nova: Added supports_requests method to PciDeviceList  https://review.openstack.org/24208720:02
*** y_sawai has quit IRC20:02
bauzasanyway, that's not big deal20:02
bauzasthat's something we can modify later on20:02
openstackgerritChris Dent proposed openstack/nova: Add a get_by_uuid for aggregates  https://review.openstack.org/28489520:02
alaskibauzas: I appreciate the thoroughness of the review. I'm always happy to get constructive feedback.20:03
openstackgerritMatt Riedemann proposed openstack/nova: Added missing execution of the test  https://review.openstack.org/28191520:03
mriedemsdague: ^20:03
*** eil397 has quit IRC20:04
*** eil397 has joined #openstack-nova20:05
*** ljxiash has quit IRC20:05
openstackgerritDan Smith proposed openstack/nova: WIP: Hacking check to detect uncalled closures in tests  https://review.openstack.org/28490520:09
dansmithalaski: sdague ^20:09
dansmithI only found those two examples of this pattern failing,20:09
*** ducttape_ has quit IRC20:09
dansmithbut lots of other things we've defined and orphaned in a test that can just go away20:09
dansmithif that looks good to you then I'll fix those other issues20:09
dansmithbut wanted first impressions ... first20:10
*** su_zhang has quit IRC20:12
*** su_zhang has joined #openstack-nova20:13
*** fawadkhaliq has quit IRC20:13
*** fawadkhaliq has joined #openstack-nova20:13
alaskiyou went a little more generic than my attempt so far. I have been trying to narrow down to closures decorated with mocks and checking for a specific call20:14
alaskibut the general check might be more useful20:14
*** su_zhang has quit IRC20:15
*** penick has joined #openstack-nova20:15
*** Marga_ has quit IRC20:15
*** su_zhang has joined #openstack-nova20:15
*** annasort has joined #openstack-nova20:15
mriedemsdague: do you remember anything recently where nova service-list was listing downed API services?20:15
*** Marga_ has joined #openstack-nova20:15
dansmithalaski: okay it seemed like the safest thing to go general, but it also hits a lot of things20:16
dansmithalaski: interested to see what you come up with though for sure20:16
mriedemSam-I-Am was seeing something where nova service-list was showing downed API services but nova-api was running,20:16
dansmithmaybe both are useful20:16
alaskiyeah, I'll keep pushing on this20:16
mriedemon my newish devstack those API services don't show up20:16
*** ducttape_ has joined #openstack-nova20:16
*** inc0 has quit IRC20:16
alaskimriedem: I recall reviewing something about that not too long ago20:17
dansmithmriedem: api services don't update their records20:17
alaskimriedem: https://review.openstack.org/#/c/277881/20:17
dansmiththe patch was to filter them out of service list Ithink20:17
dansmithyeah that :)20:17
mriedemnice, that's exactly it20:18
mriedemthanks20:18
mriedemi wonder if that's an issue in liberty20:18
*** salv-orlando has quit IRC20:18
mriedemor something regressed that in mitaka20:18
dansmithnope just mitaka I think20:19
mriedemcool20:19
dansmithedleafe: you back?20:19
edleafedansmith: yup - just got back20:19
dansmithedleafe: I assert that your patch is broken20:20
dansmithbut I don't know why20:20
*** salv-orlando has joined #openstack-nova20:20
edleafedansmith: just because you know my work? :)20:20
dansmithedleafe: I do a compute_node_update({'memory_mb': 2000}), get back a matching thing, then do another compute_node_get(id) and get back the original un-updated value20:20
dansmithedleafe: heh, no :)20:20
edleafedansmith: hmmm - I haven't checked out compute_node_update20:21
dansmithedleafe: thought I'd point it out in case you had any ideas or could write some quick tests to prove it's me and no you20:21
dansmithand got distracted on something else before I could do it myself20:22
edleafedansmith: ok - just started working on the tests, so I'll add that to the list20:22
dansmithokay thanks20:22
mriedemsdague: shouldn't v2.1 also be in here? step 4 when creating compute endpoints: http://docs.openstack.org/liberty/install-guide-ubuntu/nova-controller-install.html#prerequisites20:23
*** kfarr has quit IRC20:23
mriedemor use v2.1 instead of v2, period20:23
*** Marga_ has quit IRC20:25
*** Marga_ has joined #openstack-nova20:26
*** Marga_ has quit IRC20:27
*** sheel has quit IRC20:27
*** su_zhang has quit IRC20:30
dansmithedleafe: http://pastebin.com/NDw6NZqL20:30
dansmithwas hoping I would be pasting "edsstuffisbroken: PASS" but .. :D20:30
*** su_zhang has joined #openstack-nova20:30
dansmithtesttools.matchers._impl.MismatchError: 1024 != 51220:30
dansmithooooh20:31
dansmithI know what it is20:31
edleafedansmith: is update updating the right thing?20:31
dansmithyou're returning a dict now and compute_node_get is updating that dict, not the model20:31
dansmithsorry, compute_node_update is updating the dict from _get20:32
edleafeah20:32
dansmithbut,20:32
edleafenot the object20:32
dansmithwe need to keep some of that behavior20:32
dansmithlemme make a change and paste you a diff20:32
edleafek20:32
*** su_zhang has quit IRC20:35
*** shawnmaten has joined #openstack-nova20:37
*** thorst is now known as thorst_afk20:38
*** angdraug has joined #openstack-nova20:39
*** toure is now known as toure_20:39
dansmithHmm, I dunno how to get sqla not to do a create20:39
*** jwcroppe has quit IRC20:40
dansmithedleafe: this, but working: http://pastebin.com/MW6zv4h220:40
dansmiththat .save() ends up doing an INSERT, but I just want it to UPDATE20:40
*** jwcroppe has joined #openstack-nova20:40
dansmithi.e. it would be nice to not have to query out the new way and the old way, update the old one and return the new one20:41
*** huats has quit IRC20:41
openstackgerritMatt Riedemann proposed openstack/nova: Update the help for deprecated glance host/port/protocol options  https://review.openstack.org/28492020:42
edleafedansmith: instead of creating a new ComputeNode object, can you do a get for the given id?20:42
*** matrohon has joined #openstack-nova20:42
edleafeor are the objects smart enough to update if the id exists?20:42
*** huats has joined #openstack-nova20:42
dansmithedleafe: that's a sqla model20:43
*** haplo37 has quit IRC20:43
dansmithedleafe: objects just call the db_api method (this one in this case) and pass the dict of values and expect it to be handled, however that may be20:43
edleafedansmith: doh! yeah, I was thinking Nova objects20:44
dansmithzzzeek: if I want to just create a model, set the id and a few new values and get sqla to do an UPDATE instead of INSERT.. is there some trick?20:44
*** jwcroppe has quit IRC20:45
dansmithor jaypipes ^20:45
jaypipesdansmith: session.add(model)?20:46
dansmithoh?20:46
dansmithnope.20:47
*** porunov has joined #openstack-nova20:48
mriedemSwami: comments in https://review.openstack.org/#/c/275073/20:49
mriedemSwami: is that migration_to binding:host stuff new for neutron in mitaka?20:49
*** shawnmaten has left #openstack-nova20:49
*** browne has joined #openstack-nova20:49
*** rk4n has quit IRC20:50
*** haomaiwang has joined #openstack-nova20:50
sdaguemriedem: yeh20:51
sdaguethe install guide should say v2.120:51
mriedemsdague: ok, that's what i told Sam-I-Am20:52
*** ducttape_ has quit IRC20:52
Swamimriedem: yes it is new, it has not been handled so far.20:53
*** ducttape_ has joined #openstack-nova20:53
*** thorst_afk is now known as thorst20:53
mriedemSwami: i see https://review.openstack.org/#/c/275420/20:54
Swamimriedem: since we already had the binding:profile support in ports, we just mad use of it to address the "migrating_to"20:54
bauzasdansmith: jaypipes: where are we with resource-providers ? which change shall I review now ? I can see most of changes are dependent on https://review.openstack.org/#/c/281837/18 right?20:54
mriedemyeah, binding:* is apparently just turning into a wild west of stuff to describe a port20:54
Swamimriedem: yes you got the right patch.20:55
Swamimriedem: I agree20:55
mriedemis there a ci job that runs live migration with neutron? sdague?20:55
*** Marga_ has joined #openstack-nova20:55
mriedemeven though this is specific to dvr...20:55
*** rtheis has quit IRC20:55
sdaguemriedem: nothing voting yet20:55
sdaguethere might be experimental jobs20:56
openstackgerritRonald Bradford proposed openstack/nova: Define context.roles using base class  https://review.openstack.org/28492520:56
jaypipesbauzas: https://review.openstack.org/#/c/242087/ could use your eyes. same with https://review.openstack.org/#/c/284484/20:56
mriedemwas someone driving those? sc68cal?20:56
sc68calhello20:56
*** mylu has joined #openstack-nova20:57
mriedemsc68cal: neutron live migration + dvr - go!20:57
bauzasjaypipes: ack20:57
mriedemsc68cal:  + testing!20:57
*** y_sawai has joined #openstack-nova20:57
*** haomaiwang has quit IRC20:58
openstackgerritMark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule  https://review.openstack.org/26782720:58
sc68calI think we test live migrations in the neutron-dsvm-multinode-dvr job...20:58
sc68caland I think the neutron-dsvm-multinode - but not 100% sure20:58
sc68calproblem is - dvr job has like 50/50 failure rate :-\20:59
sc68calso yeah long way of saying what sdague said succintly - "not voting yet"20:59
*** Marga_ has quit IRC20:59
sc68caldoes that sound accurate Swami or am I giving you the short shrift?21:00
*** gyee has joined #openstack-nova21:00
sc68calre live migrate21:00
Swamisc68cal: the most recent failure that we have been seeing is the "test_volume_backend_live_migration"21:01
*** y_sawai has quit IRC21:02
_gryfsdague, remember our conversation about tox running different python interpreter from yesterday? turns out that my setup.py is doing things, which convinced tox to use certain interpreter, even if it doesn't make sense. so the case is closed, and thanks anyway :)21:03
Swamisc68cal: Yes our multinode job is still experimental and not voting yet21:03
mriedemtest_volume_backend_live_migration wouldn't use block migrate though right21:04
Swamimriedem: yes the neutron ci multinode jobs are currently testing the live-migration21:04
mriedemSwami: sc68cal: basically, it'd be cool if we had a job (probably on experimental queue) that shows live block migration working with neutron + dvr for https://review.openstack.org/#/c/275073/21:04
*** vishwanathj has quit IRC21:04
mriedemor the neutron changes dependent on https://review.openstack.org/#/c/275073/21:04
*** eil397 has quit IRC21:04
Swamimriedem: we have been seeing some inconsistency in the live migration tests.21:04
*** raildo is now known as raildo-afk21:04
mriedemSwami: so i guess we should see the multinode dvr job passing live block migration with https://review.openstack.org/#/c/260738/ ? although i don't know the live migration tests have a floating IP attached...21:05
*** daemontool has joined #openstack-nova21:05
*** eil397 has joined #openstack-nova21:06
*** haplo37 has joined #openstack-nova21:06
mriedemhttp://logs.openstack.org/38/260738/18/check/gate-tempest-dsvm-neutron-dvr-multinode-full/8493f58/console.html#_2016-02-25_06_54_23_63521:06
mriedemtempest.api.compute.admin.test_live_migration.LiveBlockMigrationTestJSON.test_live_block_migration [19.204524s] ... ok21:06
Swamimriedem: Yes I had a doubt about it, but this patch should work for both non floatingip case and for floatingip case with respect to dvr, since we pre-create the routers in the agent.21:06
openstackgerritMark Doffman proposed openstack/nova: Move logging outside of LibvirtConfigObject.to_xml  https://review.openstack.org/22421621:06
*** rtheis has joined #openstack-nova21:06
bauzas alaski: around ? I have a question about cells v121:08
alaskibauzas: yep21:08
bauzasalaski: so I'm modifying resize_instance() for providing the Spec obj21:08
mriedemSwami: sc68cal: so i guess i can't trust those job results b/c that test passes on other neutron changes in the same job21:09
mriedemhttp://logs.openstack.org/76/282876/13/check/gate-tempest-dsvm-neutron-dvr-multinode-full/2879370/console.html#_2016-02-24_20_15_35_43821:09
mriedemto see the failure, we'd need to probably update the test to attach a floating ip to the server that gets migrated21:09
bauzasalaski: by modifying https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2632 to add an extra request_spec21:09
*** aysyd has quit IRC21:10
mriedemwe could do that as a one-time change to tempest (do not merge it, just use it for testing)21:10
Swamimriedem: make sense21:10
mriedemoh, well....to really verify it i'd think we'd have to ssh into the guest before and after it's live migrated21:11
bauzasalaski: so, tbc, if we're in cells v1, compute.api.ComputeTaskAPI.resize() will be called but the RPC API will go to the cells ?21:11
mriedemi see gate-tempest-dsvm-neutron-dvr-multinode-full already uses floating IPs21:11
mriedemhttp://logs.openstack.org/76/282876/13/check/gate-tempest-dsvm-neutron-dvr-multinode-full/2879370/logs/tempest_conf.txt.gz21:11
mriedemssh_connect_method = floating21:11
Swamimriedem: we need to ssh and see if the ssh session does not break.21:11
*** eil397 has quit IRC21:11
mriedemso the test is passing but networking could be broken21:11
mriedemright21:11
bauzasalaski: speaking of https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L7221:12
alaskibauzas: was just about to link that :)  because it's set there the call will go through cells/rpcapi to the cell21:12
bauzasalaski: so I'm a bit worried noxw21:13
alaskibauzas: https://github.com/openstack/nova/blob/master/nova/cells/rpcapi.py#L55721:13
bauzasalaski: because I would need to also modify the cells RPC API to add this new arg, right?21:13
*** ducttape_ has quit IRC21:13
alaskiyes21:13
bauzasoh man, it was merged21:14
bauzasI mean the ConductorTaskAPI, without adding the cells RPC API too21:14
bauzasalaski: ^21:14
*** gmurphy has left #openstack-nova21:15
*** davideagle_ has quit IRC21:15
bauzasalaski: meaning https://github.com/openstack/nova/blob/master/nova/conductor/rpcapi.py#L270-L27221:15
bauzassec, I'm puzzled21:15
mriedemhmm, during driver.spawn, libvirt eventually tries to get pci_devices from the instance, but i don't think we have them at that point...21:15
mriedemnot sure where those get created21:16
alaskibauzas: I don't see resize_instance in there21:16
bauzasalaski: because compute.api.resize() calls conductor.rebuild()21:16
*** ildikov has quit IRC21:16
bauzasbut ooooh21:16
alaskibauzas: but you may have broken rebuild and migrate21:17
alaskiand unshelve21:17
bauzasalaski: that's what I wonder21:17
bauzasoh man, those levels of indirection are getting me crazy21:17
alaskibauzas: the easiest fix might be to update cells/rpcapi to accept those fields and then not pass the request_spec over the wire21:18
openstackgerritjichenjc proposed openstack/nova: Replace fake_utils by using Fixture  https://review.openstack.org/23095421:18
*** smatzek has quit IRC21:19
alaskibauzas: and within each cell it will need to fall back to treating instances like older ones with no req_spec21:19
openstackgerritMark Doffman proposed openstack/nova: Cleanup rescue images on unrescue when using RBD.  https://review.openstack.org/24177721:19
*** fawadkhaliq has quit IRC21:20
*** Sukhdev has quit IRC21:21
bauzasalaski: okay, I just double-checked, and I see why I didn't modified resize_instance() for the conductor RPC API, it's because it's named migrate_server21:22
bauzasalaski: yeah, was thinking of that21:22
mriedemdansmith: man, figuring out where instance.pci_devices gets set during the instance create, that's not going to be fun21:22
openstackgerritMatt Riedemann proposed openstack/nova: DNM: see where instance.pci_devices is getting lazy-loaded  https://review.openstack.org/28483921:22
*** jichen has quit IRC21:22
*** Marga_ has joined #openstack-nova21:22
bauzasfor cells v1, it would mean that this feature wouldn't be available but that's a trade-off21:22
mriedemdansmith: but i think we're round-tripping to lazy-load that field on instance spawn and it's empty21:22
mriedemso basically pointless21:23
alaskibauzas: yeah. you could pass it over the wire as well, there just isn't a good way to test that21:23
dansmithmriedem: yep, that was on my list, but I have nothing done for it, because it wasn't obvious21:23
*** achanda has quit IRC21:23
mriedemdansmith: ^ test patch21:23
mriedemtrying to find out21:23
dansmithmriedem: oh, I have a generic one up21:23
*** su_zhang has joined #openstack-nova21:23
bauzasalaski: given that we're pretty frozen on cells v1 with a minimal coverage (because we missed that with our CI), I'm honestly fine to just add the arg and not pass it21:24
dansmithmriedem: https://review.openstack.org/#/c/274714/8/nova/objects/instance.py21:24
mriedem https://review.openstack.org/#/c/211288/?21:24
alaskibauzas: sounds good21:24
bauzasalaski: so, just writing the fix21:24
mriedemdansmith: ok, i already did get a trace on pci_devices21:24
mriedemhttp://logs.openstack.org/39/284839/1/check/gate-tempest-dsvm-full/433bb96//logs/screen-n-cpu.txt.gz?level=TRACE21:24
bauzasalaski: thanks for your help, really sorry to have borked that :(21:24
mriedemdefinitely from spawn, i just wanted to confirm it's an empty list21:24
bauzasbork, bork21:24
alaskiheh21:24
dansmithyep cool21:24
*** matrohon has quit IRC21:25
*** annasort has quit IRC21:25
*** rtheis has quit IRC21:26
*** rtheis has joined #openstack-nova21:26
*** browne has quit IRC21:27
*** Marga_ has quit IRC21:27
*** su_zhang has quit IRC21:28
*** annasort has joined #openstack-nova21:28
*** matrohon has joined #openstack-nova21:29
*** apearson has quit IRC21:30
*** achanda has joined #openstack-nova21:31
*** rk4n has joined #openstack-nova21:32
*** apearson has joined #openstack-nova21:32
*** ijw has joined #openstack-nova21:33
*** achanda has quit IRC21:33
bauzasalaski: last point, I don't see unshelve in the RPC proxy for cells, I suppose it's not supported for cells v1 ?21:34
alaskibauzas: it's done differently https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L34121:35
bauzasBOOOOM21:35
bauzas<head exploded>21:35
*** ducttape_ has joined #openstack-nova21:35
alaskiyeah, cells seemed too simple at first so multiple ways of doing things were added21:36
alaskireally we just didn't migrate everything to the new ways21:36
melwitthah, that's great21:37
*** achanda has joined #openstack-nova21:37
openstackgerritEd Leafe proposed openstack/nova: Use new inventory schema in all compute_node gets  https://review.openstack.org/28483321:38
*** thorst is now known as thorst_afk21:39
bauzasalaski: okay, so IIRC, it just cuts off compute_task_api but still calls compute_api.unshevel and use instead cast_to_cells to propagate thru cells,right?21:39
*** kaisers has quit IRC21:39
alaskiyep21:39
bauzasin that case, it's fine21:39
bauzasokay, writing a few tests for covering that hack, and go21:40
bauzasalaski: thanks21:40
*** hamzy has quit IRC21:40
*** sc68cal has quit IRC21:41
*** ijw has quit IRC21:41
*** kaisers has joined #openstack-nova21:41
*** ijw has joined #openstack-nova21:42
*** thorst_afk has quit IRC21:43
*** thorst_afk has joined #openstack-nova21:43
openstackgerritMatt Riedemann proposed openstack/nova: Add specific method to lazy-load instance.pci_devices  https://review.openstack.org/28494521:44
*** annasort has quit IRC21:44
mriedemdansmith: ^ easy one for you21:44
*** yamahata has quit IRC21:44
dansmithmriedem: looks okay but lemme scrape through the intel pci logs if it runs21:45
mriedemi almost thought that was a joke21:46
dansmithit might be, we'll see :)21:47
*** daemontool has quit IRC21:48
*** eil397 has joined #openstack-nova21:49
*** annasort has joined #openstack-nova21:50
*** Marga_ has joined #openstack-nova21:51
*** thorst_afk has quit IRC21:52
*** shawnmaten has joined #openstack-nova21:54
*** haomaiwang has joined #openstack-nova21:55
*** eil397 has quit IRC21:57
*** eil397 has joined #openstack-nova21:57
*** ZZelle_ has joined #openstack-nova21:58
*** y_sawai has joined #openstack-nova21:58
*** penick_ has joined #openstack-nova21:59
*** penick has quit IRC21:59
*** penick_ is now known as penick21:59
*** delatte has quit IRC22:00
mriedemoooo i think intel nfv ci found a bug22:00
*** haomaiwang has quit IRC22:01
sdagueshut the front door22:01
*** porunov has quit IRC22:01
*** asselin has left #openstack-nova22:01
*** ivase has quit IRC22:01
*** browne has joined #openstack-nova22:01
openstackgerritSylvain Bauza proposed openstack/nova: Fix Cells RPC API by accepting a RequestSpec arg  https://review.openstack.org/28495922:02
bauzasalaski: melwitt: (anyone who wants to understand cells v1) ^22:02
*** apearson has quit IRC22:02
mriedemhttps://bugs.launchpad.net/nova/+bug/155001422:02
openstackLaunchpad bug 1550014 in OpenStack Compute (nova) "test_shelve_shelved_server fails in Intel NFV CI for api-compute-on-nfv-flavors scenario" [Undecided,New]22:02
mriedemsdague: unless it's an env issue22:02
*** y_sawai has quit IRC22:02
sdaguemriedem: log ?22:03
*** shawnmaten has quit IRC22:03
mriedemhttp://intel-openstack-ci-logs.ovh/compute-ci/refs/changes/20/283820/2/compute-nfv-flavors/20160225_183949/screen/n-cpu.log.gz22:03
mriedem2016-02-25 19:15:26.874 ERROR nova.compute.manager [req-078b2da5-cb6e-41ed-925c-01f3acb52264 tempest-ServersNegativeTestJSON-975508431 tempest-ServersNegativeTestJSON-1886110650] [instance: e6395df6-5393-4391-aa8e-6f3790268f8f] Instance failed to spawn22:03
*** shawnmaten has joined #openstack-nova22:05
*** ivase has joined #openstack-nova22:05
mriedemcfriesen: don't you care about cpu pinning? ^22:06
*** apearson has joined #openstack-nova22:09
*** achanda has quit IRC22:09
*** ivase has quit IRC22:10
*** eharney has quit IRC22:10
*** apearson has quit IRC22:10
*** mylu has quit IRC22:10
*** dedery has joined #openstack-nova22:11
bauzasmriedem: heh, just created https://bugs.launchpad.net/nova/+bug/155001822:12
openstackLaunchpad bug 1550018 in OpenStack Compute (nova) "Cells v1 is borked for live-migrate and evacuate" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)22:12
bauzasmriedem: so yeah will be attaching the bug to the change22:12
*** knikolla has quit IRC22:13
openstackgerritSylvain Bauza proposed openstack/nova: Fix Cells RPC API by accepting a RequestSpec arg  https://review.openstack.org/28495922:13
*** cleong has quit IRC22:15
*** mylu has joined #openstack-nova22:17
*** daemontool has joined #openstack-nova22:17
*** fawadkhaliq has joined #openstack-nova22:19
openstackgerritEd Leafe proposed openstack/nova: Use new inventory schema in all compute_node gets  https://review.openstack.org/28483322:19
edleafedansmith: ^^ Let me know what you think about this approach22:19
*** shawnmaten has quit IRC22:22
*** su_zhang has joined #openstack-nova22:23
*** shawnmaten has joined #openstack-nova22:24
*** delatte has joined #openstack-nova22:25
openstackgerritChris Dent proposed openstack/nova: WIP Add ResourcePool object  https://review.openstack.org/28496322:26
openstackgerritChris Dent proposed openstack/nova: Adjust resource-providers models for resource-pools  https://review.openstack.org/28183722:28
*** Sukhdev has joined #openstack-nova22:28
openstackgerritChris Dent proposed openstack/nova: Add name to ResourceProvider object  https://review.openstack.org/28194522:28
openstackgerritChris Dent proposed openstack/nova: Add a get_by_uuid for aggregates  https://review.openstack.org/28489522:29
openstackgerritChris Dent proposed openstack/nova: WIP Add ResourcePool object  https://review.openstack.org/28496322:29
*** apearson has joined #openstack-nova22:29
*** hemna has joined #openstack-nova22:30
*** burgerk has quit IRC22:33
*** delattec has joined #openstack-nova22:33
*** john51_ has joined #openstack-nova22:34
openstackgerritAndrew Laski proposed openstack/nova: WIP Add a hacking check for test method closures  https://review.openstack.org/28497022:35
*** mylu has quit IRC22:35
alaskidansmith: ^ my initial attempt22:35
alaskiit fails on self.assertRaises though22:35
*** delatte has quit IRC22:36
alaskiI'm not walking all nodes though and I may need to convert to doing that22:36
*** john51 has quit IRC22:38
*** ccarmack has quit IRC22:40
*** mylu has joined #openstack-nova22:42
*** armax has quit IRC22:43
*** annasort has quit IRC22:43
*** armax has joined #openstack-nova22:44
*** mylu has quit IRC22:44
*** mylu has joined #openstack-nova22:44
bauzasmriedem: see the commit msg for https://review.openstack.org/#/c/284959/2/nova/cells/rpcapi.py22:45
bauzasmriedem: as discussed with alaski we agreed to not modify the RPC API22:46
mriedemwhat would adding a version break?22:46
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Cold migrate using the RequestSpec object  https://review.openstack.org/28497422:47
bauzasmriedem: so bumping a version for that would mean that we touch the RPC API for something we don't really want22:48
bauzasmriedem: here, we just want to make sure that the cells api accepts the signature, not really needing to pass it down22:48
bauzasmriedem: hence the UTs I added https://review.openstack.org/#/c/284959/2/nova/tests/unit/cells/test_cells_rpcapi.py22:49
*** thorst_afk has joined #openstack-nova22:49
mriedemunrelated22:51
mriedemhttp://logs.openstack.org/89/283689/3/gate/gate-tempest-dsvm-full-ceph-src-glance_store/f23f9eb/logs/screen-n-sch.txt.gz#_2016-02-25_01_47_37_42922:51
mriedemSeems service is down. Last heartbeat was 2016-02-25 01:43:59. Elapsed time is 218.428975 is_up22:51
mriedemthat elapsed time seems wrong22:51
dansmithedleafe: that doesn't fix compute_node_update()22:51
mriedembut i guess it's not22:51
openstackgerritSwaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron networks  https://review.openstack.org/27507322:53
* bauzas heading off to bed, good week-end folks (me on PTO tomorrow)22:53
*** mdrabe has quit IRC22:55
*** hemna has quit IRC22:55
openstackgerritMark Doffman proposed openstack/nova: Add manage command for cell0.  https://review.openstack.org/27526622:56
openstackgerritMark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule  https://review.openstack.org/26782722:56
openstackgerritMark Doffman proposed openstack/nova: Manage db sync command for cell0  https://review.openstack.org/27488522:56
*** thorst_afk has quit IRC22:57
*** armax has quit IRC22:57
*** y_sawai has joined #openstack-nova22:59
*** haomaiwang has joined #openstack-nova22:59
*** eil397 has quit IRC23:00
*** haomaiwang has quit IRC23:01
*** delatte has joined #openstack-nova23:01
*** vilobhmm11 has quit IRC23:01
*** vilobhmm11 has joined #openstack-nova23:02
*** rlrossit has quit IRC23:02
*** vilobhmm11 has quit IRC23:03
*** shawnmaten has quit IRC23:03
*** delattec has quit IRC23:03
*** y_sawai has quit IRC23:04
*** shawnmaten has joined #openstack-nova23:04
*** shawnmaten has quit IRC23:04
*** achanda has joined #openstack-nova23:04
*** zenoway has quit IRC23:05
*** vilobhmm11 has joined #openstack-nova23:05
*** hemna has joined #openstack-nova23:07
*** salv-orl_ has joined #openstack-nova23:08
*** cdent has quit IRC23:09
*** edmondsw has quit IRC23:11
*** lpetrut has quit IRC23:11
*** edtubill has quit IRC23:11
*** ctrath has quit IRC23:12
*** salv-orlando has quit IRC23:12
*** xyang1 has quit IRC23:14
*** matrohon has quit IRC23:15
*** dedery has quit IRC23:16
*** zenoway has joined #openstack-nova23:17
*** Daisy has joined #openstack-nova23:18
*** xek_ has joined #openstack-nova23:18
*** xek has quit IRC23:19
*** zenoway has quit IRC23:21
openstackgerritMatt Riedemann proposed openstack/nova: Add service binary/host to service is down log for context  https://review.openstack.org/28498023:22
*** Daisy has quit IRC23:22
*** mriedem has quit IRC23:23
*** ducttape_ has quit IRC23:24
*** burt has quit IRC23:24
*** haplo37 has quit IRC23:25
edleafe:w23:26
* edleafe sighs23:26
*** bjornar__ has quit IRC23:26
*** dimtruck is now known as zz_dimtruck23:27
*** hemna has quit IRC23:28
*** davideagle has joined #openstack-nova23:29
*** Sukhdev has quit IRC23:29
*** y_sawai has joined #openstack-nova23:30
edleafedansmith: this seems to work for me: http://pastebin.com/WK1P9Lrx23:34
*** rtheis has quit IRC23:34
edleafedansmith: let me know what issues you're still seeing23:34
* edleafe leaves for a school meeting (yay!)23:35
dansmithedleafe: I think I got hung up scrolling and didn't see that you updated cmopute_node_update23:36
*** yamahata has joined #openstack-nova23:36
*** Bjoern_ has quit IRC23:36
*** davideagle has quit IRC23:38
*** Swami has quit IRC23:38
*** achanda has quit IRC23:38
*** salv-orl_ has quit IRC23:39
*** RichardRaseley has joined #openstack-nova23:39
*** y_sawai has quit IRC23:42
*** angdraug has quit IRC23:43
*** fawadkhaliq has quit IRC23:47
*** achanda has joined #openstack-nova23:49
*** edtubill has joined #openstack-nova23:49
*** zenoway has joined #openstack-nova23:52
*** shawnmaten has joined #openstack-nova23:54
*** thorst_afk has joined #openstack-nova23:55
*** su_zhang has quit IRC23:57
*** zenoway has quit IRC23:57
*** su_zhang has joined #openstack-nova23:57
*** krtaylor has quit IRC23:58

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