Thursday, 2017-12-21

*** wind has quit IRC00:06
*** moshele has joined #openstack-nova00:11
*** yingjun has joined #openstack-nova00:12
*** itlinux__ has quit IRC00:15
*** salv-orlando has joined #openstack-nova00:18
*** itlinux_ has joined #openstack-nova00:19
*** tetsuro has joined #openstack-nova00:24
*** jaypipes has quit IRC00:25
*** yangyape_ has quit IRC00:27
*** yangyapeng has joined #openstack-nova00:27
*** yangyapeng has quit IRC00:32
*** hiro-kobayashi has joined #openstack-nova00:34
*** moshele has quit IRC00:35
*** felipemonteiro__ has quit IRC00:53
*** tbachman has quit IRC00:54
*** itlinux_ has quit IRC00:55
*** jobewan has quit IRC00:55
*** marst has quit IRC00:56
*** hoangcx has quit IRC00:59
*** phuongnh has joined #openstack-nova01:01
*** salv-orlando has quit IRC01:03
*** yamahata has quit IRC01:03
*** trungnv has quit IRC01:03
*** salv-orlando has joined #openstack-nova01:04
*** kumarmn has joined #openstack-nova01:04
*** suresh12 has joined #openstack-nova01:05
*** hoangcx has joined #openstack-nova01:09
*** kumarmn has quit IRC01:09
*** tuanla____ has joined #openstack-nova01:09
*** huanxie has joined #openstack-nova01:11
openstackgerritMr Rambo proposed openstack/nova master: Fix the problems that volume-backed server rebuild  https://review.openstack.org/52874001:12
*** yingjun has quit IRC01:12
*** yangyapeng has joined #openstack-nova01:12
*** psachin has joined #openstack-nova01:20
*** edmondsw has joined #openstack-nova01:21
*** kumarmn has joined #openstack-nova01:22
*** edmondsw has quit IRC01:26
*** tbachman has joined #openstack-nova01:30
*** penick has quit IRC01:32
*** penick has joined #openstack-nova01:32
*** crushil has joined #openstack-nova01:33
*** kumarmn has quit IRC01:36
*** chyka has joined #openstack-nova01:37
*** kumarmn has joined #openstack-nova01:38
*** suresh12 has quit IRC01:41
*** chyka has quit IRC01:42
*** crushil has quit IRC01:42
*** salv-orlando has quit IRC01:46
*** Apoorva_ has joined #openstack-nova01:46
*** Apoorva has quit IRC01:50
*** Apoorva_ has quit IRC01:51
*** suresh12 has joined #openstack-nova01:51
*** threestrands has joined #openstack-nova01:51
*** threestrands has quit IRC01:51
*** threestrands has joined #openstack-nova01:51
*** jappleii__ has quit IRC01:52
*** andreas_s has joined #openstack-nova01:55
*** andreas_s has quit IRC02:00
openstackgerritMerged openstack/nova stable/newton: Patch mkisofs calls  https://review.openstack.org/52928402:02
*** gcb has joined #openstack-nova02:03
*** tommylikehu_ has joined #openstack-nova02:04
*** redondo-mk_ has joined #openstack-nova02:04
*** rajinir_ has joined #openstack-nova02:04
*** takashin_ has joined #openstack-nova02:06
*** Hazelesque_ has joined #openstack-nova02:06
*** mgariepy has quit IRC02:07
*** takashin has quit IRC02:07
*** mvk has quit IRC02:07
*** harlowja has quit IRC02:07
*** obre has quit IRC02:07
*** tommylikehu has quit IRC02:07
*** redondo-mk has quit IRC02:07
*** Hazelesque has quit IRC02:07
*** Gorian has quit IRC02:07
*** ujjain has quit IRC02:07
*** dansmith has quit IRC02:07
*** evrardjp has quit IRC02:07
*** comstud has quit IRC02:07
*** leifz has quit IRC02:07
*** rajinir has quit IRC02:07
*** redondo-mk_ is now known as redondo-mk02:07
*** obre_ has joined #openstack-nova02:07
*** tommylikehu_ is now known as tommylikehu02:07
*** rajinir_ is now known as rajinir02:07
*** leifz has joined #openstack-nova02:07
*** Gorian has joined #openstack-nova02:07
*** kumarmn has quit IRC02:08
*** mvk has joined #openstack-nova02:08
*** ujjain has joined #openstack-nova02:08
*** ujjain has quit IRC02:08
*** ujjain has joined #openstack-nova02:08
*** mgariepy has joined #openstack-nova02:08
*** dansmith has joined #openstack-nova02:08
*** dansmith is now known as Guest9968002:08
*** comstud has joined #openstack-nova02:08
*** evrardjp has joined #openstack-nova02:09
*** yamamoto has joined #openstack-nova02:09
*** dave-mccowan has joined #openstack-nova02:09
*** andreykurilin has quit IRC02:10
*** mriedem has quit IRC02:10
*** sambetts|afk has quit IRC02:10
*** andreykurilin has joined #openstack-nova02:12
*** sambetts_ has joined #openstack-nova02:13
*** yangyapeng has quit IRC02:14
*** yangyapeng has joined #openstack-nova02:15
*** Sandy619 has joined #openstack-nova02:15
Sandy619Hello Guys, Need some information.. How do nova collect console logs of instances? Some of my instances logs are not coming, i need to troubleshoot this.02:17
Sandy619ping #openstack-nova , can you please help me with above query :) if anybody have an idea.02:19
*** suresh12 has quit IRC02:20
*** gouthamr has quit IRC02:29
*** hui_ has quit IRC02:31
openstackgerritChen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance  https://review.openstack.org/52634802:33
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Fix functional tests for USE_NEUTRON  https://review.openstack.org/52945602:34
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Fix functional tests for USE_NEUTRON  https://review.openstack.org/52945602:35
*** trungnv has joined #openstack-nova02:36
*** esberglu has quit IRC02:37
*** huanxie has quit IRC02:38
*** huanxie has joined #openstack-nova02:44
*** _pewp_ has quit IRC02:46
*** _pewp_ has joined #openstack-nova02:46
*** salv-orlando has joined #openstack-nova02:47
*** salv-orlando has quit IRC02:47
openstackgerritJianle He proposed openstack/nova master: Fix nova backup for volume-backed instance  https://review.openstack.org/52945802:54
openstackgerritTetsuro Nakamura proposed openstack/nova master: doc: update supported drivers for cpu topology  https://review.openstack.org/52929403:00
*** Apoorva has joined #openstack-nova03:01
*** masuberu has joined #openstack-nova03:03
*** yamahata has joined #openstack-nova03:03
*** masber has quit IRC03:04
*** Sandy619_opensta has joined #openstack-nova03:04
*** Sandy619_opensta has quit IRC03:05
*** suresh12 has joined #openstack-nova03:07
*** Sandy619 has quit IRC03:07
*** edmondsw has joined #openstack-nova03:09
*** suresh12 has quit IRC03:12
*** edmondsw has quit IRC03:14
*** masuberu has quit IRC03:17
*** dave-mccowan has quit IRC03:20
*** gyee has quit IRC03:21
*** Apoorva has quit IRC03:22
*** Apoorva has joined #openstack-nova03:23
*** Apoorva has quit IRC03:27
*** kumarmn has joined #openstack-nova03:28
*** kumarmn has quit IRC03:37
*** pooja-jadhav has joined #openstack-nova03:38
*** pooja_jadhav has joined #openstack-nova03:38
*** kumarmn has joined #openstack-nova03:38
*** threestrands_ has joined #openstack-nova03:39
*** threestrands_ has quit IRC03:39
*** threestrands_ has joined #openstack-nova03:39
*** threestrands has quit IRC03:41
*** namnh has joined #openstack-nova03:42
*** kumarmn has quit IRC03:43
*** tiendc has joined #openstack-nova03:49
*** kumarmn has joined #openstack-nova03:50
*** yamamoto_ has joined #openstack-nova03:53
*** yamamoto has quit IRC03:56
*** kumarmn has quit IRC03:58
*** kumarmn has joined #openstack-nova03:58
*** kumarmn has quit IRC03:59
*** hiro-kobayashi_ has joined #openstack-nova04:07
*** esberglu has joined #openstack-nova04:08
*** hiro-kobayashi has quit IRC04:09
*** esberglu has quit IRC04:11
*** namnh has quit IRC04:12
*** salv-orlando has joined #openstack-nova04:15
*** pooja-jadhav has quit IRC04:15
*** masber has joined #openstack-nova04:17
*** andreas_s has joined #openstack-nova04:20
*** salv-orlando has quit IRC04:20
*** sridharg has joined #openstack-nova04:22
*** huanxie has quit IRC04:23
*** andreas_s has quit IRC04:24
*** huanxie has joined #openstack-nova04:29
*** fragatina has quit IRC04:33
*** fragatina has joined #openstack-nova04:34
*** links has joined #openstack-nova04:35
*** links has quit IRC04:35
*** udesale has joined #openstack-nova04:45
*** tiendc has quit IRC04:46
openstackgerritMerged openstack/nova master: Delete the TypeAffinityFilter  https://review.openstack.org/52809904:51
*** psachin has quit IRC04:55
*** suresh12 has joined #openstack-nova04:56
*** edmondsw has joined #openstack-nova04:57
*** huanxie has quit IRC04:59
*** huanxie has joined #openstack-nova05:00
*** ratailor has joined #openstack-nova05:01
*** edmondsw has quit IRC05:02
*** gmann has joined #openstack-nova05:09
*** aignatov2 has joined #openstack-nova05:12
*** mdrabe_ has joined #openstack-nova05:12
*** mdrabe has quit IRC05:13
*** aignatov has quit IRC05:13
*** aignatov2 is now known as aignatov05:13
*** chyka has joined #openstack-nova05:13
*** salv-orlando has joined #openstack-nova05:16
*** chyka has quit IRC05:18
*** salv-orlando has quit IRC05:21
*** Rambo has joined #openstack-nova05:26
*** crushil has joined #openstack-nova05:30
*** psachin has joined #openstack-nova05:43
*** moshele has joined #openstack-nova05:59
*** psachin has quit IRC06:02
openstackgerritMerged openstack/nova master: Use instance.project_id when creating request specs for old instances  https://review.openstack.org/52918406:03
*** janki has joined #openstack-nova06:03
openstackgerritMerged openstack/nova master: Workaround missing RequestSpec.project_id when moving an instance  https://review.openstack.org/52918506:03
openstackgerritMerged openstack/nova master: Implement set_and_clear_allocations in report client  https://review.openstack.org/52808906:03
RamboFix the problems that volume-backed server rebuild06:04
Rambo.Could you please review it?  https://review.openstack.org/#/c/528740/06:04
*** moshele has quit IRC06:07
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: implement NotImplementedError for _set_quiesced  https://review.openstack.org/52927806:07
*** armax has quit IRC06:12
*** hui has joined #openstack-nova06:12
*** armax has joined #openstack-nova06:12
*** armax has quit IRC06:13
*** armax has joined #openstack-nova06:13
*** armax has quit IRC06:14
*** armax has joined #openstack-nova06:14
openstackgerritMr Rambo proposed openstack/nova master: Fix the problems that volume-backed server rebuild  https://review.openstack.org/52874006:14
*** salv-orlando has joined #openstack-nova06:14
*** armax has quit IRC06:14
*** armax has joined #openstack-nova06:15
*** armax has quit IRC06:15
*** salv-orl_ has joined #openstack-nova06:17
*** DinaBelova2 has joined #openstack-nova06:19
*** threestrands_ has quit IRC06:21
*** tristanC_ has joined #openstack-nova06:21
*** suresh12 has quit IRC06:24
*** salv-orlando has quit IRC06:26
*** crushil has quit IRC06:26
*** rajinir has quit IRC06:26
*** redondo-mk has quit IRC06:26
*** tommylikehu has quit IRC06:26
*** krtaylor has quit IRC06:26
*** edleafe has quit IRC06:26
*** itlinux has quit IRC06:26
*** afazekas has quit IRC06:26
*** Dave has quit IRC06:26
*** amodi has quit IRC06:26
*** robcresswell has quit IRC06:26
*** betherly has quit IRC06:26
*** ameade has quit IRC06:26
*** TheJulia has quit IRC06:26
*** DinaBelova has quit IRC06:26
*** jroll has quit IRC06:26
*** Roamer` has quit IRC06:26
*** kencjohnston has quit IRC06:26
*** knikolla has quit IRC06:26
*** auggy has quit IRC06:26
*** tristanC has quit IRC06:26
*** DinaBelova2 is now known as DinaBelova06:26
*** Rambo has quit IRC06:28
*** itlinux has joined #openstack-nova06:29
*** rajinir has joined #openstack-nova06:31
*** redondo-mk has joined #openstack-nova06:31
*** tommylikehu has joined #openstack-nova06:31
*** afazekas has joined #openstack-nova06:31
*** Dave has joined #openstack-nova06:31
*** amodi has joined #openstack-nova06:31
*** robcresswell has joined #openstack-nova06:31
*** betherly has joined #openstack-nova06:31
*** ameade has joined #openstack-nova06:31
*** TheJulia has joined #openstack-nova06:31
*** jroll has joined #openstack-nova06:31
*** Roamer` has joined #openstack-nova06:31
*** kencjohnston has joined #openstack-nova06:31
*** knikolla has joined #openstack-nova06:31
*** auggy has joined #openstack-nova06:31
*** edleafe has joined #openstack-nova06:32
*** krtaylor has joined #openstack-nova06:33
openstackgerritMerged openstack/nova master: trivial: conf: libvirt: fix a typo  https://review.openstack.org/52922906:34
openstackgerritMerged openstack/nova master: [placement] Separate API schemas (allocation)  https://review.openstack.org/52863006:41
*** edmondsw has joined #openstack-nova06:45
*** rajinir has quit IRC06:48
*** redondo-mk has quit IRC06:48
*** tommylikehu has quit IRC06:48
*** afazekas has quit IRC06:48
*** Dave has quit IRC06:48
*** amodi has quit IRC06:48
*** robcresswell has quit IRC06:48
*** betherly has quit IRC06:48
*** ameade has quit IRC06:48
*** TheJulia has quit IRC06:48
*** jroll has quit IRC06:48
*** Roamer` has quit IRC06:48
*** kencjohnston has quit IRC06:48
*** knikolla has quit IRC06:48
*** auggy has quit IRC06:48
*** yamamoto_ has quit IRC06:49
*** edmondsw has quit IRC06:50
*** aignatov has quit IRC06:50
*** aignatov has joined #openstack-nova06:53
*** zhenguo has quit IRC07:04
*** hiro-kobayashi_ has quit IRC07:09
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/52479507:09
*** karthiks has joined #openstack-nova07:15
*** ameeda has quit IRC07:16
*** yamamoto has joined #openstack-nova07:17
*** psachin has joined #openstack-nova07:19
*** ameeda has joined #openstack-nova07:20
ameedaHello, Can you please review my code here https://review.openstack.org/#/c/528385/ ?07:21
*** Roamer` has joined #openstack-nova07:24
*** afazekas has joined #openstack-nova07:24
*** Lingwu has quit IRC07:24
*** TheJulia has joined #openstack-nova07:24
*** kencjohnston has joined #openstack-nova07:25
*** Dave__ has joined #openstack-nova07:25
*** robcresswell has joined #openstack-nova07:25
*** amodi has joined #openstack-nova07:25
*** knikolla has joined #openstack-nova07:25
*** betherly has joined #openstack-nova07:25
*** jroll has joined #openstack-nova07:25
*** ameade has joined #openstack-nova07:26
*** tommylikehu has joined #openstack-nova07:26
*** redondo-mk has joined #openstack-nova07:26
*** auggy has joined #openstack-nova07:26
*** rajinir has joined #openstack-nova07:26
*** Lingwu has joined #openstack-nova07:27
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model  https://review.openstack.org/52505507:29
*** rcernin has quit IRC07:37
*** andreas_s has joined #openstack-nova07:38
*** syjulian has quit IRC07:39
*** AlexeyAbashkin has joined #openstack-nova07:41
*** ianw_pto is now known as ianw07:43
*** ianw is now known as ianw_pto07:43
*** jpena|off is now known as jpena07:56
*** pcaruana has joined #openstack-nova07:58
*** annp has joined #openstack-nova07:59
*** sshwarts has joined #openstack-nova08:01
*** syjulian has joined #openstack-nova08:11
*** elias_ has joined #openstack-nova08:12
ameedaCan you please review my code here https://review.openstack.org/#/c/526900/ ?08:13
*** damien_r has joined #openstack-nova08:16
*** gcb has quit IRC08:16
*** xinliang has quit IRC08:17
openstackgerritxhzhf proposed openstack/nova master: [cellv2] performance issue when querying compute nodes in a cell  https://review.openstack.org/52950408:23
*** Rambo has joined #openstack-nova08:26
*** gmann has quit IRC08:27
*** sahid has joined #openstack-nova08:27
*** xinliang has joined #openstack-nova08:29
*** edmondsw has joined #openstack-nova08:33
*** salv-orl_ has quit IRC08:35
*** salv-orlando has joined #openstack-nova08:35
*** edmondsw has quit IRC08:38
*** linkmark has joined #openstack-nova08:39
*** alexchadin has joined #openstack-nova08:39
*** salv-orlando has quit IRC08:40
*** namnh has joined #openstack-nova08:41
*** claudiub has joined #openstack-nova08:41
*** andreas__ has joined #openstack-nova08:42
*** salv-orlando has joined #openstack-nova08:44
*** mdnadeem has joined #openstack-nova08:45
*** andreas_s has quit IRC08:45
tetsurosahid: I have updated this document revision patch about cpu topology on qemu TCG https://review.openstack.org/#/c/529294/08:47
*** kumarmn has joined #openstack-nova08:48
*** kumarmn has quit IRC08:52
*** tristanC_ is now known as tristanC08:53
*** ragiman has joined #openstack-nova08:57
sahidtetsuro: oh, thanks a lot. I will see that :)08:59
*** rm_work has quit IRC08:59
*** alexchadin has quit IRC09:00
*** yamahata has quit IRC09:00
*** gcb has joined #openstack-nova09:03
*** alexchadin has joined #openstack-nova09:04
*** rm_work has joined #openstack-nova09:07
*** yikun_jiang is now known as yikun09:09
*** Brin has joined #openstack-nova09:10
*** syjulian has quit IRC09:14
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Clean up foreign key before root resource provider deleting  https://review.openstack.org/52951909:15
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc  https://review.openstack.org/52952009:16
*** rmart04 has joined #openstack-nova09:24
*** udesale__ has joined #openstack-nova09:26
*** gmann has joined #openstack-nova09:27
*** derekh has joined #openstack-nova09:28
*** udesale has quit IRC09:28
*** udesale has joined #openstack-nova09:28
*** ralonsoh has joined #openstack-nova09:30
*** udesale__ has quit IRC09:31
bauzasgood morning Nova09:32
*** udesale has quit IRC09:33
*** udesale has joined #openstack-nova09:33
*** udesale has quit IRC09:35
*** udesale has joined #openstack-nova09:35
*** dtantsur|afk is now known as dtantsur09:36
*** salv-orlando has quit IRC09:44
*** yamamoto_ has joined #openstack-nova09:45
*** salv-orlando has joined #openstack-nova09:45
*** udesale has quit IRC09:46
*** lucas-afk is now known as ucasagomes09:47
*** takashin_ has left #openstack-nova09:47
*** yamamoto has quit IRC09:48
*** salv-orlando has quit IRC09:49
*** gongysh has joined #openstack-nova09:51
*** trungnv has quit IRC09:56
*** penick has quit IRC10:04
*** udesale has joined #openstack-nova10:05
*** ucasagomes is now known as lucasagomes10:09
*** annp has quit IRC10:16
*** lpetrut has joined #openstack-nova10:18
*** namnh has quit IRC10:20
*** edmondsw has joined #openstack-nova10:22
*** edmondsw has quit IRC10:26
*** dtantsur is now known as dtantsur|brb10:27
openstackgerritIlya Shakhat proposed openstack/nova master: Initialize osprofiler in WSGI application  https://review.openstack.org/51966410:28
*** alexchadin has quit IRC10:31
*** sambetts_ is now known as sambetts10:31
stephenfinNot sure who's about between now and new year, but I'd appreciate some eyes on this series to get it over the line https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/websocket-proxy-to-host-security10:31
stephenfinIt's been ongoing since 2013. Would be great to get the damn thing finished, heh10:32
*** fragatina has quit IRC10:35
*** fragatina has joined #openstack-nova10:36
*** alexchadin has joined #openstack-nova10:37
*** alexchadin has quit IRC10:41
*** gongysh has quit IRC10:41
*** pcaruana has quit IRC10:43
openstackgerritChen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance  https://review.openstack.org/52634810:44
*** salv-orlando has joined #openstack-nova10:45
*** kumarmn has joined #openstack-nova10:50
*** salv-orlando has quit IRC10:50
*** huanxie has quit IRC10:51
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model  https://review.openstack.org/52505510:53
*** huanxie has joined #openstack-nova10:54
*** kumarmn has quit IRC10:54
*** tuanla____ has quit IRC10:55
*** priteau has joined #openstack-nova10:59
*** udesale has quit IRC11:01
*** AlexeyAbashkin has quit IRC11:01
*** bhagyashris has quit IRC11:07
*** alexchadin has joined #openstack-nova11:08
openstackgerritAlex Xu proposed openstack/nova master: Remove the LoadedExtensionInfo object  https://review.openstack.org/52920711:08
openstackgerritAlex Xu proposed openstack/nova master: Merge ResourceV21 obj into Resource obj  https://review.openstack.org/52921711:08
openstackgerritAlex Xu proposed openstack/nova master: Remove the inherits parameter for the Resource object  https://review.openstack.org/52921811:08
*** AlexeyAbashkin has joined #openstack-nova11:08
alex_xuoomichi: ^ thanks for the review, just updated11:09
*** fragatina has quit IRC11:10
*** fragatina has joined #openstack-nova11:11
*** andreas__ has quit IRC11:12
*** andreas_s has joined #openstack-nova11:17
*** hoonetorg has quit IRC11:22
*** salv-orlando has joined #openstack-nova11:24
*** tetsuro has quit IRC11:25
*** andreas_s has quit IRC11:26
*** andreas_s has joined #openstack-nova11:27
*** bhagyashris has joined #openstack-nova11:34
*** hoonetorg has joined #openstack-nova11:34
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Fix error when deleting any root resource provider  https://review.openstack.org/52951911:35
*** andreas_s has quit IRC11:37
*** andreas_s has joined #openstack-nova11:37
openstackgerritxhzhf proposed openstack/nova master: [cellv2] performance issue when querying compute nodes in a cell  https://review.openstack.org/52950411:41
*** andreas_s has quit IRC11:41
*** Brin has quit IRC11:44
openstackgerritMatthew Booth proposed openstack/nova master: Removed unused argument from LibvirtDriver._connect_volume  https://review.openstack.org/52955611:47
*** andreas_s has joined #openstack-nova11:47
mdboothlyarwood: Cleanup related to our earlier discussion ^^^11:47
lyarwoodmdbooth: ack nice!11:49
lyarwoodmdbooth: I wonder if we can do the same with disk_dev on _disconnect_volume11:50
mdboothlyarwood: Don't. I've been actively trying to limit the quantity of ocean I'm boiling lately.11:50
mdboothCleanup as you go is a wonderful concept which I fully support, but in practice it seems to be where my patch series end up getting buried.11:52
lyarwoodmeh, I'll do this as a standalone, if it lands great :)11:52
mdboothI wanted to do ^^^ because disk_info is relevant to my other series.11:53
mdboothI may actually incorporate it in there, as it probably reduces some test noise.11:53
lyarwoodkk11:53
*** andreas_s has quit IRC11:54
*** andreas_s has joined #openstack-nova11:54
*** liverpooler has quit IRC11:54
mdboothlyarwood: FWIW, looks like disk_dev can also be removed. It's only used for logging.11:56
mdboothAnd the log messages aren't really required given the context.11:57
mdboothMeh, I'll do it anyway as I have it open11:57
lyarwoodwonderful11:57
mdboothlyarwood: Remind me, can connection_info contain secrets?11:59
* mdbooth asks cinder folks12:00
lyarwoodmdbooth: yeah I think so12:00
mdboothI'd have thought so. I'll just confirm.12:00
*** udesale has joined #openstack-nova12:01
*** MikeG451 has joined #openstack-nova12:03
*** cdent has joined #openstack-nova12:03
*** jpena is now known as jpena|lunch12:06
*** yamamoto_ has quit IRC12:06
*** dave-mccowan has joined #openstack-nova12:09
*** edmondsw has joined #openstack-nova12:10
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls  https://review.openstack.org/46024312:11
*** yamamoto has joined #openstack-nova12:11
*** dtantsur|brb is now known as dtantsur12:12
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Introduce disk encryption config classes  https://review.openstack.org/46400812:12
lyarwoodstephenfin: sorry, was about to do that, just respinning the refactor one final time12:13
*** lucasagomes is now known as lucas-hungry12:13
stephenfinlyarwood: No problem :) I didn't realize you'd pushed a new revision12:14
stephenfinComments on said revision incoming though12:14
lyarwoodstephenfin: thanks12:14
*** edmondsw has quit IRC12:14
* lyarwood holds fire12:14
stephenfinlyarwood: and done12:16
lyarwoodstephenfin: cheers, yeah that's why I was respinning12:17
stephenfin:) Cool12:17
openstackgerritMatthew Booth proposed openstack/nova master: Fix unit test failures when direct IO not supported  https://review.openstack.org/52956812:19
mdboothIf we can get ^^^ in sharpish that would be awesome. It's really annoying.12:20
stephenfinmdbooth: (y)12:21
*** udesale has quit IRC12:23
*** udesale has joined #openstack-nova12:23
stephenfinmdbooth: Regarding the preceding patch, do you know if we support out-of-tree volume drivers? https://review.openstack.org/#/c/529556/12:23
* stephenfin suspects not, but I guess this would break the API if we did12:23
*** dave-mccowan has quit IRC12:24
stephenfinAlso, the one you linked could probably be rebased onto master to get it in sooner. No dependency that I can see12:24
*** huanxie has quit IRC12:25
*** raghadq has joined #openstack-nova12:26
*** huanxie has joined #openstack-nova12:27
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls  https://review.openstack.org/46024312:27
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology  https://review.openstack.org/48555312:27
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology  https://review.openstack.org/48555412:27
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology  https://review.openstack.org/46603012:27
openstackgerritStephen Finucane proposed openstack/nova master: setup.cfg: Explicitly set [build_sphinx] builders  https://review.openstack.org/50848312:29
*** raghadq has quit IRC12:30
*** tbachman has quit IRC12:30
kashyaplyarwood: On this - https://review.openstack.org/#/c/464008/1/nova/virt/libvirt/config.py.  Did you forget to add the comment describing the relation b/n classes & tags?12:32
kashyapIn response to Andrew Volkov's comments12:33
openstackgerritStephen Finucane proposed openstack/nova master: Convert floating IP tests to ddt  https://review.openstack.org/47136012:35
lyarwoodkashyap: I did, I'm not sure if it's really required tbh12:36
kashyaplyarwood: Yeah, it's fine for me12:40
*** smatzek has joined #openstack-nova12:45
*** janki has quit IRC12:50
*** kumarmn has joined #openstack-nova12:51
*** tbachman has joined #openstack-nova12:55
*** kumarmn has quit IRC12:56
*** jaypipes has joined #openstack-nova12:56
*** psachin has quit IRC12:58
*** dave-mccowan has joined #openstack-nova12:58
*** crushil has joined #openstack-nova12:59
openstackgerritMerged openstack/nova master: Pass bdms to versioned notifications during finish_revert_resize  https://review.openstack.org/52744613:05
*** jpena|lunch is now known as jpena13:06
*** psachin has joined #openstack-nova13:06
*** dave-mcc_ has joined #openstack-nova13:07
*** dave-mccowan has quit IRC13:08
*** lucas-hungry is now known as lucasagomes13:08
*** yangyapeng has quit IRC13:10
openstackgerritStephen Finucane proposed openstack/nova master: tests: Use correct response type in tests  https://review.openstack.org/52718213:15
*** yangyapeng has joined #openstack-nova13:15
openstackgerritStephen Finucane proposed openstack/nova master: Handle images with no data  https://review.openstack.org/52632913:15
openstackgerritStephen Finucane proposed openstack/nova master: Remove dead parameter from '_create_domain_and_network'  https://review.openstack.org/51235213:16
*** Rambo has quit IRC13:18
*** yangyapeng has quit IRC13:19
jaypipesstephenfin: lol, sorry about the force=True thing...13:19
*** salv-orl_ has joined #openstack-nova13:20
jaypipesstephenfin: just goes to show the importance of kwargs ;)13:20
stephenfinjaypipes: I'll be honest, you had me doubting myself :D13:20
*** eharney has quit IRC13:20
jaypipesstephenfin: well, sorry about that. :) but it was not obvious...13:20
*** phuongnh has quit IRC13:21
stephenfinNo no, it's fine. Turned out the parameter wasn't necessary anyway13:21
stephenfinSo something good came out of it13:21
*** salv-orlando has quit IRC13:22
stephenfinThink you might have called this one wrong too ;) https://review.openstack.org/#/c/527472/7/nova/pci/stats.py@26513:23
jaypipesstephenfin: still looking at that one...13:23
jaypipesstephenfin: you're probably right. I'm just getting old. but I still want to look into that further.13:23
*** yamamoto has quit IRC13:24
stephenfinjaypipes: Sounds good to me. I can rework in any case if it makes it a little easier to parse13:24
mdboothWaiting for my unit tests to run is the pythonistas waiting for a compile13:25
jaypipesindeed13:25
*** crushil has quit IRC13:25
mdboothUnfortunately I don't have a foosball table13:26
* mdbooth has to be productive elsewhere instead13:26
* stephenfin is eventually going to fulfil his promise to start adding mypy typing info to oslo.libraries. Then we can just run mypy to catch most things13:27
*** yamamoto has joined #openstack-nova13:28
* cdent blinks13:28
*** yikun_ has joined #openstack-nova13:28
*** huanxie has quit IRC13:28
openstackgerritMatthew Booth proposed openstack/nova master: Fix unit test failures when direct IO not supported  https://review.openstack.org/52956813:28
openstackgerritMatthew Booth proposed openstack/nova master: Removed unused argument from LibvirtDriver._connect_volume  https://review.openstack.org/52955613:28
openstackgerritMatthew Booth proposed openstack/nova master: Remove unused argument from LibvirtDriver._disconnect_volume  https://review.openstack.org/52958513:28
mdboothI blame lyarwood ^^^13:29
*** artom has quit IRC13:30
*** artom has joined #openstack-nova13:30
*** huanxie has joined #openstack-nova13:31
*** alexchadin has quit IRC13:31
jaypipesstephenfin: to be honest, the whole stats.py module makes me want to throw up a bit in my mouth.13:32
kashyapmdbooth: Removing cruft is good.  Thank you13:32
openstackgerritLee Yarwood proposed openstack/nova master: conf: Do not inherit image signature props with snapshots  https://review.openstack.org/52704613:32
mdboothkashyap: There's also also a lesson above in how a misunderstanding in one bit of code can be copied around and mutate.13:32
stephenfinjaypipes: Agreed. The docstrings are a start. Next steps is a big de-dupe13:33
stephenfinBut I wasn't mixing that in here. Would like it to get in before the new year :)13:33
kashyapmdbooth: Yeah, /me nods13:33
*** catintheroof has joined #openstack-nova13:34
stephenfinjaypipes: Is the move to resource providers going to get swept up in efried's generic device thing'y or is it separate? If latter, dibs13:34
*** catintheroof has quit IRC13:36
*** alexchadin has joined #openstack-nova13:36
*** catintheroof has joined #openstack-nova13:36
jaypipesstephenfin: the generic device manager ideas revolve around using the ProviderTree for managing PCI (and other) devices just like any other resource provider (like a compute node for VCPU/MEMORY_MB or a pGPU group for VGPU resources). the idea is to get rid of the whole PciDevicePoolList (stats) object entirely.13:37
*** cdent has quit IRC13:37
jaypipesstephenfin: and the generic device manager would *only* gather inventory and do the binding with the guest. it wouldn't do anything with scheduling/placement logic13:37
stephenfinRight-o, that's what I suspected. So we need to do the reporting aspect and then build upon that to use what's been reported?13:38
*** salv-orl_ has quit IRC13:39
*** salv-orlando has joined #openstack-nova13:39
jaypipesstephenfin: not entirely sure what you mean by "build upon that to use what's been reported"13:42
*** cdent has joined #openstack-nova13:43
*** salv-orlando has quit IRC13:44
jaypipesstephenfin: the placement service will be the thing that allocates a device to a guest instance and the generic device manager will just be responsible for plumbing it through to the guest. for example, the GDM will get the local address for a provider's UUID and stuff like that. what the GDM *won't* do is track available/allocated/claimed crap for devices, since that information will already be in the placement service.13:44
jaypipesstephenfin: in addition, the GDM will manage the mapping of device address or local name to the resource provider's UUID.13:45
jaypipesstephenfin: and the GDM will coordinate work with the hypervisor (if necessary) to inventory the collection of devices available on a compute node for guests to use (i.e. the whitelist stuff)13:46
stephenfinjaypipes: Oh, I thought GDM was only responsible for the "reporting device (in a generic way) to placement" piece. Looks like I need to re-read that spec13:46
*** mriedem has joined #openstack-nova13:46
openstackgerritLee Yarwood proposed openstack/nova master: conf: Do not inherit image signature props with snapshots  https://review.openstack.org/52704613:46
jaypipesstephenfin: yes, that's the majority of its responsibility. that's what that "maintaining a mapping of local device name/address to resource provider UUID" is all about. :)13:47
stephenfinSo "build upon that to use what's been reported" referred to wiring up what gdm gave us with the various drivers13:47
stephenfinWhich I thought would be a separate thing. Guess that doesn't make sense though13:47
stephenfinto separate it, that is13:47
*** yamamoto has quit IRC13:48
jaypipesstephenfin: some of that I believe will depend on the hypervisor driver.13:49
jaypipesstephenfin: libvirt does a lot of that work. but drivers like powervm don't necessarily.13:49
*** kholkina has joined #openstack-nova13:51
mdboothEurgh. My local disk serials series is hitting merge conflict hell :(13:51
mdboothPrepare for another rebase of the whole stack13:51
*** abhishekk has joined #openstack-nova13:51
dtantsurhi mriedem! could you please take a look at https://bugs.launchpad.net/nova/+bug/1739440 and tell me if it's okay as a bug, or should be a blueprint, or a spec, or a bad idea at all13:51
openstackLaunchpad bug 1739440 in OpenStack Compute (nova) "Ironic virt driver is hardcoding Ironic API microversion" [Undecided,New]13:51
mriedemnot a spec,13:52
mriedemnot a bad idea13:52
jaypipesdtantsur: ^ answer.13:53
mriedemi don't know that it's a bug, or a blueprint13:53
dtantsurokay, thanks! I'll leave it as a bug for now then, we can reconsider if the actual change starts looking more like a feature13:53
mriedemi guess it's no different than having different computes with different versions of libvirt that can support some capability13:53
mriedemdtantsur: yeah that's a good route13:53
stephenfinjaypipes: Yup, that all makes sense. Hopefully swapping the driver stuff that uses PciDevicePoolList in with whatever gdm offers us will be painless enough13:54
dtantsurTheJulia: re version negotiation bug ^^^^13:54
*** edmondsw has joined #openstack-nova13:58
*** psachin has quit IRC13:58
*** ratailor has quit IRC13:59
*** lyan has joined #openstack-nova13:59
*** huanxie has quit IRC13:59
*** gouthamr has joined #openstack-nova14:00
*** pchavva has joined #openstack-nova14:00
*** huanxie has joined #openstack-nova14:02
*** Tom-Tom has joined #openstack-nova14:02
*** edmondsw has quit IRC14:02
mnaserso I've been thinking about this lately - with cinder, volume types can have a specific iops quota to them.  in an environment like ceph where performance scales (almost) linearly will capacity, being able to tie the # of iops to the volume size is used by some other cloud providers which is very interesting for this use case14:04
mnaseri think the implementation details are pretty easy because we just need a "iops per gb" and then instead of using the number provided by cinder14:05
mriedemedleafe: i'm going to update the alternate hosts patch - i extended a reschedule functoinal test to use caching scheduler which exposes a bug in the scheduler manager14:05
mnaserjust take the volume size * iops per gb and thats it14:05
edleafemriedem: you already wrote it? I was just starting on it14:06
*** jaypipes has quit IRC14:06
mriedemedleafe: yup it's just a simple functional test extend14:10
edleafemriedem: ok, I'll hold off14:10
*** awaugama has joined #openstack-nova14:13
openstackgerritMatt Riedemann proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135814:13
mriedemedleafe: ^14:13
mriedemso with that we get the test coverage and don't need a big new unit test, plus a unit test wouldn't have caught the bug in the scheduler manager14:14
*** jobewan has joined #openstack-nova14:14
mriedem+2 now14:15
mriedemand now jay is gone14:15
*** yamamoto has joined #openstack-nova14:18
*** eharney has joined #openstack-nova14:19
mnaserdoes the attach_volume() driver function at any point know the size of the disk?14:20
*** kumarmn has joined #openstack-nova14:23
*** Tom-Tom has quit IRC14:25
*** liverpooler has joined #openstack-nova14:26
*** smatzek has quit IRC14:26
mdboothmnaser: If this patch lands it would https://review.openstack.org/#/c/528363/614:29
mnasermdbooth: i see, well thankfully, it was all just noise and this was all i needed - https://review.openstack.org/#/c/447127/ :D14:30
*** huanxie has quit IRC14:30
mdboothmnaser: Are you writing a new volume driver, btw?14:30
mnasermdbooth: oh nope, just was going to try and make the existing rbd one do qos per-gb so the qos scales with the volume size14:31
mdboothmnaser: i.e. bigger disks are permitted more bandwidth, linearly?14:32
mnasermdbooth: correct, bandwidth and iops, because we're using ceph so those two are related14:32
mdboothMakes sense on the face of it.14:32
*** huanxie has joined #openstack-nova14:32
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Verify parameters in servers.inc  https://review.openstack.org/52820114:34
*** andreas_s has quit IRC14:34
openstackgerritrahul bardia proposed openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique  https://review.openstack.org/52792214:37
*** liverpooler has quit IRC14:38
*** crushil has joined #openstack-nova14:38
*** liverpooler has joined #openstack-nova14:39
*** andreas_s has joined #openstack-nova14:39
*** alexchadin has quit IRC14:40
*** salv-orlando has joined #openstack-nova14:40
*** salv-orlando has quit IRC14:44
*** yangyapeng has joined #openstack-nova14:44
mdboothIs there any chance we could chuck this one into the gate: https://review.openstack.org/#/c/529568/ ?14:44
*** mlavalle has joined #openstack-nova14:45
mdboothIt's a bug affecting execution of unit tests on a filesystem which doesn't support direct io14:45
mdboothWhich apparently my workstation's filesystem doesn't14:45
* mdbooth has to filter out a certain number of expected test failures.14:45
mriedemgmann: alex_xu: don't we have something in the docs that talks about not doing API extensions anymore?14:46
mriedemlooking for that here https://docs.openstack.org/nova/latest/contributor/index.html#the-nova-api14:46
mdboothIncoming... (sorry: merge conflict)14:46
openstackgerritMatthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping  https://review.openstack.org/24260214:46
openstackgerritMatthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid  https://review.openstack.org/24260314:46
openstackgerritMatthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid  https://review.openstack.org/52559914:46
openstackgerritMatthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr  https://review.openstack.org/52416714:46
openstackgerritMatthew Booth proposed openstack/nova master: Expose BDM uuid to drivers  https://review.openstack.org/52903714:46
openstackgerritMatthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix  https://review.openstack.org/52634614:46
openstackgerritMatthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice  https://review.openstack.org/52634714:46
openstackgerritMatthew Booth proposed openstack/nova master: Rename block_device_info_get_root  https://review.openstack.org/52902814:46
openstackgerritMatthew Booth proposed openstack/nova master: Add local_root to block_device_info  https://review.openstack.org/52902914:46
openstackgerritMatthew Booth proposed openstack/nova master: Expose driver_block_device fields as attributes  https://review.openstack.org/52836214:46
openstackgerritMatthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume  https://review.openstack.org/52836314:46
openstackgerritMatthew Booth proposed openstack/nova master: Use real block_device_info data in libvirt tests  https://review.openstack.org/52791614:46
openstackgerritMatthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info  https://review.openstack.org/52932814:46
openstackgerritMatthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info  https://review.openstack.org/52932914:46
openstackgerritMatthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver  https://review.openstack.org/52938014:46
mriedemgmann: alex_xu: found it https://docs.openstack.org/nova/latest/reference/stable-api.html14:46
mriedemre: https://review.openstack.org/#/c/527922/14:47
*** yangyapeng has quit IRC14:47
*** crushil has quit IRC14:48
mriedemmaybe i'm being harsh, but if we don't support api extensions in the server, i don't know why we need to support them in the client14:48
*** andreas_s has quit IRC14:49
mriedemsince that's the cost of having a not interoperable cloud14:49
*** abhishekk has quit IRC14:49
*** burt has joined #openstack-nova14:50
alex_xumriedem: yea, that's true14:51
*** gouthamr has quit IRC14:51
*** yangyapeng has joined #openstack-nova14:51
kholkinahi all! is it possible to update user_data on a running instance?14:52
kholkinaI've found a blueprint but it looks like abandoned https://blueprints.launchpad.net/nova/+spec/update-userdata14:52
mriedemkholkina: no. there is a new microversion in queens that allows you to rebuild an instance and change the userdata at that time,14:52
*** gouthamr has joined #openstack-nova14:52
mriedembut rebuild != running14:53
kholkinamaybe it's planned to work on that blueprint?14:54
*** andreas_s has joined #openstack-nova14:54
*** ljjjustin has joined #openstack-nova14:54
mriedemi hadn't heard of it before today, so i don't know of any plans to work on that14:54
mriedemlyarwood: you want to tickle the ocata and newton versions of this too? https://review.openstack.org/#/q/I214a44f0eee7d90be5cd89f32f6e0017b19a3fd614:54
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc  https://review.openstack.org/52952014:55
*** gouthamr has quit IRC14:55
*** andreas_s has quit IRC14:56
*** andreas_s has joined #openstack-nova14:56
lyarwoodmriedem: planning to once it lands in stable/pike14:56
*** karthiks has quit IRC14:57
kholkinaso now the only way to do this is via updating database?14:57
mriedemkholkina: on a running instance? yes that's what you'd have today14:59
mriedemkholkina: if you're interested in supporting this via the API, you'd have to take over ownership of https://blueprints.launchpad.net/nova/+spec/update-userdata14:59
mriedemand propose a spec for the rocky release14:59
mriedemto the nova-specs repo14:59
mriedemsince it's an API change14:59
kholkinaok, thanks15:00
*** vladikr has joined #openstack-nova15:00
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Introduce SMM (Systems Management Mode) config class  https://review.openstack.org/52961115:01
*** smatzek has joined #openstack-nova15:01
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Introduce SMM (System Management Mode) config class  https://review.openstack.org/52961115:02
*** justin___ has joined #openstack-nova15:04
*** danpawlik has quit IRC15:05
*** justin___ has quit IRC15:08
*** armax has joined #openstack-nova15:09
*** dave-mcc_ has quit IRC15:09
*** dave-mccowan has joined #openstack-nova15:10
*** gouthamr has joined #openstack-nova15:11
*** dave-mcc_ has joined #openstack-nova15:12
*** ljjjustin has quit IRC15:13
*** liverpooler has quit IRC15:13
*** bnemec has joined #openstack-nova15:13
openstackgerritMerged openstack/nova-specs master: Amend flavor description spec for GET /flavors API  https://review.openstack.org/51603315:14
*** dave-mccowan has quit IRC15:15
*** marst has joined #openstack-nova15:22
*** elias_ has quit IRC15:22
*** toabctl has quit IRC15:23
openstackgerritMatt Riedemann proposed openstack/nova master: Document testing guide for new API contributions  https://review.openstack.org/52961815:24
mriedemgmann: oomichi: alex_xu: ^ finally write something up on this15:24
mriedem*wrote15:24
*** toabctl has joined #openstack-nova15:25
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: Provide VGPU inventory for a single GPU type  https://review.openstack.org/51476315:25
openstackgerritSylvain Bauza proposed openstack/nova master: WIP: libvirt: create vGPU for instance  https://review.openstack.org/52883215:25
*** ejat has quit IRC15:26
*** ejat has joined #openstack-nova15:26
*** ejat has quit IRC15:26
*** ejat has joined #openstack-nova15:26
*** itlinux has quit IRC15:27
*** Apoorva has joined #openstack-nova15:34
openstackgerritStephen Finucane proposed openstack/nova master: conf: Remove deprecated 'multi_instance_display_name_template' opt  https://review.openstack.org/49961215:34
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Simplify instance name generation  https://review.openstack.org/51657315:34
*** liusheng has quit IRC15:34
*** liusheng has joined #openstack-nova15:35
*** pchavva has quit IRC15:36
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Add missing unit tests for FilterScheduler._get_all_host_states  https://review.openstack.org/52962515:39
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Fix an error in _get_host_states when deleting a compute node  https://review.openstack.org/52962615:39
mriedemmnaser: ^ for that KeyError in pike15:39
*** sridharg has quit IRC15:40
*** alee_afk is now known as alee15:48
openstackgerritStephen Finucane proposed openstack/nova master: test: Store the OutputStreamCapture fixture  https://review.openstack.org/51514615:49
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Rename 'policy_check' -> 'policy'  https://review.openstack.org/51514815:49
openstackgerritStephen Finucane proposed openstack/nova master: nova-status: Migrate to cliff  https://review.openstack.org/51514715:49
openstackgerritStephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff  https://review.openstack.org/51514915:49
*** hemna_ has joined #openstack-nova15:54
*** AlexeyAbashkin has quit IRC15:55
openstackgerritMatthew Booth proposed openstack/nova master: Remove unused argument from LibvirtDriver._disconnect_volume  https://review.openstack.org/52958515:58
mdboothlyarwood: ^^^ Fixed15:59
*** liverpooler has joined #openstack-nova16:00
mdboothlyarwood: Lets wait for Zuul, though.16:00
*** andreas_s has quit IRC16:01
*** huanxie has quit IRC16:02
kholkinamriedem: is there a better method to manage ssh-keys on instances?16:02
mriedembetter method compared to what?16:03
mriedemkeypairs?16:03
*** huanxie has joined #openstack-nova16:04
kholkinaI planned to manage keys via user_data16:04
mriedemi've heard of some deployments that don't use keypairs? but penick isn't around16:04
mriedemrybridges: ^?16:04
mriedemthey use config management to inject information into the guest once the guest registers with their service16:05
*** amodi has quit IRC16:09
*** lpetrut has quit IRC16:09
*** udesale has quit IRC16:10
*** itlinux has joined #openstack-nova16:13
*** kholkina has quit IRC16:14
*** lpetrut has joined #openstack-nova16:17
*** suresh12 has joined #openstack-nova16:21
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Fix an error in _get_host_states when deleting a compute node  https://review.openstack.org/52963516:22
*** lpetrut has quit IRC16:23
mnasermriedem: mdbooth awesome thank you so much for that KeyError fix16:24
*** rmart04 has quit IRC16:26
*** archit has joined #openstack-nova16:27
*** suresh12 has quit IRC16:27
mriedemthanks for reporting it16:28
*** damien_r has quit IRC16:29
*** suresh12 has joined #openstack-nova16:33
*** huanxie has quit IRC16:34
*** huanxie has joined #openstack-nova16:36
*** yikun_ has quit IRC16:38
*** salv-orlando has joined #openstack-nova16:40
*** chyka has joined #openstack-nova16:41
*** chyka has quit IRC16:44
*** chyka has joined #openstack-nova16:44
*** sshwarts has quit IRC16:45
*** archit has quit IRC16:52
*** damien_r has joined #openstack-nova16:54
*** pcaruana has joined #openstack-nova17:04
*** chyka has quit IRC17:04
*** chyka_ has joined #openstack-nova17:04
*** archit has joined #openstack-nova17:05
*** huanxie has quit IRC17:05
*** huanxie has joined #openstack-nova17:08
openstackgerritMerged openstack/nova master: Fix duplicate allocation candidates  https://review.openstack.org/52859117:08
mnaseri'm trying to understand something - nova seems to allow booting an instance type that has no ephemeral storage17:08
mnaserwhat would the size of that image be (or how is that determined?)17:08
mnaseri noticed when using heat, it doesnt let you do this (except the nova api actually lets you)17:09
*** yamamoto has quit IRC17:10
mriedemmdbooth: ^17:10
*** chyka_ has quit IRC17:10
*** jaypipes has joined #openstack-nova17:10
*** mdnadeem has quit IRC17:10
mriedemmnaser: what is the flavor.disk value?17:11
mriedem"The size of the root disk that will be created in GiB. If 0 the root disk will be set to exactly the size of the image used to deploy the instance. However, in this case filter scheduler cannot select the compute host based on the virtual image size. Therefore, 0 should only be used for volume booted instances or for testing purposes."17:11
mnasermriedem: where was that?  disk=017:11
mriedemhttps://developer.openstack.org/api-ref/compute/#show-flavor-details17:11
*** imacdonn has quit IRC17:12
*** imacdonn has joined #openstack-nova17:12
*** chyka_ has joined #openstack-nova17:12
kashyapmdbooth: Also if you respin, in the commit s/was is/is/;  s/gust/guest/17:13
*** kmalloc has joined #openstack-nova17:13
kashyap(In that unused argument thing.)17:13
*** smatzek_ has joined #openstack-nova17:18
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Introduce SMM (System Management Mode) config class  https://review.openstack.org/52961117:19
*** Apoorva has quit IRC17:20
*** Apoorva has joined #openstack-nova17:21
*** smatzek_ has quit IRC17:22
openstackgerritMatt Riedemann proposed openstack/nova master: Add support for getting volume details with a specified microversion  https://review.openstack.org/52965617:24
mriedemildikov: part 1 ^17:24
*** ragiman has quit IRC17:24
openstackgerritMatt Riedemann proposed openstack/nova master: Add support for getting volume details with a specified microversion  https://review.openstack.org/52965617:25
*** Apoorva has quit IRC17:25
*** yamamoto has joined #openstack-nova17:25
*** yamamoto has quit IRC17:30
*** armax has quit IRC17:31
*** yamahata has joined #openstack-nova17:31
*** ralonsoh has quit IRC17:33
*** armax has joined #openstack-nova17:34
*** edmondsw has joined #openstack-nova17:39
*** marst has quit IRC17:40
*** marst has joined #openstack-nova17:40
*** gyee has joined #openstack-nova17:44
*** edmondsw has quit IRC17:44
*** damien_r has quit IRC17:48
*** damien_r has joined #openstack-nova17:48
*** dtantsur is now known as dtantsur|afk17:49
*** fragatina has quit IRC17:54
*** jpena is now known as jpena|off17:55
*** daidv has quit IRC17:56
*** hieulq has quit IRC17:56
*** hoangcx has quit IRC17:56
*** daidv has joined #openstack-nova17:56
*** hieulq has joined #openstack-nova17:56
*** hoangcx has joined #openstack-nova17:56
openstackgerritMatt Riedemann proposed openstack/nova master: Add support for getting volume details with a specified microversion  https://review.openstack.org/52965617:57
*** lucasagomes is now known as lucas-afk17:59
*** liverpooler has quit IRC17:59
*** nore__ has joined #openstack-nova18:04
kashyapA no-brainer, already has +2: https://review.openstack.org/#/c/527691/ ("conf: libvirt: Cleanup CPU modelling related options")18:07
*** nore_rabel has quit IRC18:07
openstackgerritMerged openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689918:08
*** damien_r has quit IRC18:10
*** chyka_ has quit IRC18:11
*** chyka has joined #openstack-nova18:11
*** Tom-Tom has joined #openstack-nova18:11
*** derekh has quit IRC18:12
*** harlowja has joined #openstack-nova18:16
mriedemkashyap: no-brainer huh18:21
mriedemquestion inline18:22
*** catintheroof has quit IRC18:22
*** huanxie has quit IRC18:23
*** huanxie has joined #openstack-nova18:23
openstackgerritMatt Riedemann proposed openstack/nova master: conf: libvirt: Cleanup CPU modelling related options  https://review.openstack.org/52769118:24
mriedemjaypipes: can i interest you in some pure uncut alternate hosts https://review.openstack.org/#/c/511358 ?18:27
jaypipesmriedem: sure thing.18:28
jaypipesmriedem: currently have electrician's at my house running new conduit and installing a new dishwasher that went out earlier today. fun! on the day we're doing xmas dinner in the evening. awesome timing, as always.18:28
mriedemmake a centerpiece of leftover conduit to lighten the mood18:29
mriedemor something else that's crafty from "at home with amy sedaris"18:30
*** chyka_ has joined #openstack-nova18:32
*** yamahata has quit IRC18:32
*** chyka has quit IRC18:33
*** huanxie has quit IRC18:38
*** huanxie has joined #openstack-nova18:39
jaypipesmriedem: :)18:39
mriedembtw, i recommend that you recommend that show to julie18:40
jaypipesmriedem: I'm pretty sure she would already know about it. she loves anything sedaris. amy or david.18:41
openstackgerritMatt Riedemann proposed openstack/nova master: Add retry_on_deadlock decorator to action_event_start  https://review.openstack.org/52967218:42
mriedemthen we are kindred spirits18:42
*** oanson has quit IRC18:44
*** cdent has quit IRC18:44
*** catintheroof has joined #openstack-nova18:47
*** smatzek_ has joined #openstack-nova18:51
*** derekh has joined #openstack-nova18:52
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: don't log snapshot success unless it actually happens  https://review.openstack.org/52967418:52
*** fragatina has joined #openstack-nova18:52
jaypipesmriedem: questions for you in that alternate hosts patch19:02
*** gouthamr has quit IRC19:02
*** penick has joined #openstack-nova19:03
*** Tom-Tom has quit IRC19:05
*** sambetts is now known as sambetts|afk19:05
*** Apoorva has joined #openstack-nova19:05
*** yamamoto has joined #openstack-nova19:06
*** yamahata has joined #openstack-nova19:07
*** penick_ has joined #openstack-nova19:09
*** smatzek_ has quit IRC19:09
oomichimriedem: nice guide by https://review.openstack.org/#/c/529618/ just some comments19:09
edleafejaypipes: answering...19:09
mriedemedleafe: on it already19:10
mriedembut we can compare notes19:10
oomichimriedem: btw is it difficult to get bp approval of https://blueprints.launchpad.net/nova/+spec/vmware-boot-uefi ? the corresponding code seems ready19:10
*** penick has quit IRC19:11
edleafejaypipes: answered. I didn't have a good answer to https://review.openstack.org/#/c/511358/47/nova/compute/manager.py19:11
edleafemaybe mriedem will19:12
mriedemjaypipes: replied19:12
mriedemoomichi: no one ever brought that up for queens,19:13
mriedemin the 2-3 months they could have brought it up19:13
oomichimriedem: ok, let's skip it19:13
mriedemgiven how trivial this is we could possibly make an exception19:14
mriedemi just don't generally like to make exceptions because then you do it for one blueprint and everyone else expects it19:14
*** yamamoto has quit IRC19:14
melwittthere's a pretty simple libvirt driver refactor needing a +W that's en route to native luks encryption if anyone can review https://review.openstack.org/#/c/46024319:16
jaypipesmriedem: k, both bottom patches +Wd19:17
mriedemoomichi: replied19:17
mriedemjaypipes: danka19:17
jaypipesmriedem: bitte19:17
edleafejaypipes: thx. Guess we'll wait til Jan for the final patch19:18
edleafe(unless you're up for it now!)19:19
mriedemi haven't gone through the final patch yet19:19
jaypipesedleafe: the one that needs a rebase?19:19
mriedemso yeah figured january at this rate19:19
mriedemplus that19:19
mriedem:)19:19
jaypipesmriedem: I'm reading your tc IRC log...19:19
mriedemedleafe: would be cool if you could find an existing functional test for resize + reschedule that we could do the same caching scheduler extension test like i added this morning19:20
mriedemjaypipes: you like that?19:20
mriedembetter get comfortable19:20
edleafemriedem: I'll look19:20
jaypipesmriedem: it's "interesting" so far.19:20
melwittI read it yesterday19:20
mriedemwhich tantrum are we talking about?19:20
mriedemseveral days ago, or yesterday?19:20
jaypipesmriedem: I mean, I already knew everyone hated nova (cores), but ..19:21
jaypipesmriedem: yesterday19:21
*** derekh has quit IRC19:21
*** dave-mcc_ has quit IRC19:21
mriedemturns out they specifically hated john for one specific thing brought up years ago19:21
melwittyeah, it was ... illuminating19:21
mriedembut that now includes me and everyone else19:21
edleafeyou're all the same19:22
mriedemespecially Vek19:24
mriedemhe's offendor #119:24
mriedem*offender19:24
openstackgerritEd Leafe proposed openstack/nova master: Don't try to delete build request during a reschedule  https://review.openstack.org/52883519:25
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135819:25
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643619:25
edleafemriedem: jaypipes: ^^ the rebase got 'em all19:25
mriedemoomichi: i think "If you need help debugging" is the correct english19:27
mriedemotherwise "If you need help with debugging"?19:27
*** edmondsw has joined #openstack-nova19:28
mriedemgranted, i think most people who have english has a second language are probably more knowledgeable of the actual grammar rules :)19:28
*** suresh12 has quit IRC19:28
oomichimriedem: "If you need help with debugging" is more easy for me19:28
jrollanother option is "if you need help to debug"19:28
mriedemoomichi: 私はそれを変更する19:28
jrollI like with19:29
oomichimriedem: ありがとう19:29
mriedemoomichi: as for the "API extension" part,19:29
mriedemi know like 'resource'19:29
mriedembut not sure what to use beside extension. handler? controller? just API?19:29
mriedem"route"?19:30
*** edmondsw has quit IRC19:33
melwittmriedem: would it not be a REST API "resource"? that's what ppl told me to call it when I added /usages to placement API for example19:35
oomichimriedem: I think API is better in them. handler, controller and route are just implementation side19:35
mriedemmelwitt: but there are several API extensions under nova/api/openstack/compute/ which work on the same resource19:36
mriedemlike servers19:36
mriedemconsider the user_data, block_device_mapping_v2, etc extensions19:36
melwittah, right19:36
mriedemwe have a similar modeling in our api samples directories19:36
mriedemwhich is why i'm trying to show it's a 1:119:36
melwittyeah. I was thinking of the os-blah ones that are separate19:36
oomichiAPI becomes non-extensible, API means a single API, and we don't need to have separated tests for each extension19:37
melwittthere aren't any separate directories under doc/api_samples/servers/ fwiw. just microversion directories19:37
melwittit's nice to have them separated for organizing and finding them though, IMHO19:38
mriedemoomichi: then what do we call this? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/user_data.py19:39
mriedemit's an extension of the servers api19:39
mriedemeventually, one day, these should all be collapsed and removed, which sdague started and gmann was working on19:40
oomichimriedem: hehe, that is a good sample19:40
mriedembut i'm not sure what to call those except 'extensions'19:40
*** huanxie has quit IRC19:40
oomichimriedem: IIUC alex_xu wants to merge these code into main servers.py with json-schema19:40
oomichito remove these extension code19:41
mriedemyeah i know19:41
mriedembut until then what do we call these19:41
oomichiat this time, it is difficult to find another word except extension19:41
mriedemha19:42
mriedemexactly!19:42
*** huanxie has joined #openstack-nova19:42
mriedemi could put a note about how "extension" doesn't actually mean we support extending the API, and link to the docs about that, but it seems like that would get messy19:43
melwittthe user_data example doesn't have its own directory though, from what I see19:43
mriedemyou know what melanie19:43
mriedemok but https://github.com/openstack/nova/tree/master/doc/api_samples/server-metadata19:44
mriedemdoes19:44
mriedemand whatever the hell this thing is https://github.com/openstack/nova/tree/master/doc/api_samples/os-quota-sets-noop19:44
oomichimriedem: please don;t put such note, I am fine with current one which is better19:44
mriedem:)19:44
mriedemoomichi: ok so i'll change the 'debug' part19:44
mriedemand leave the rest19:44
oomichihehe19:44
openstackgerritMatt Riedemann proposed openstack/nova master: Document testing guide for new API contributions  https://review.openstack.org/52961819:45
mriedemhttps://docs.openstack.org/nova/latest/contributor/api.html#modularity19:46
mriedem"The Nova REST API is separated into different controllers in the directory ‘nova/api/openstack/compute/’"19:46
openstackgerritMerged openstack/nova stable/pike: [placement] Fix an error message in API validation  https://review.openstack.org/52646519:46
mriedemsince the doc already says "controllers" maybe we should just be consistent?19:46
melwittI was originally going to put the noop quotas directory under os-quota-sets/ but thought maybe the top level one was less confusing. I can try to move it under os-quota-sets/19:50
melwittanother way I tried was putting the noop samples at the same level in os-quota-sets/ without their own directory and have the noop tests look for a noop prefix, but thought maybe that was worse ¯\_(ツ)_/¯19:52
*** chyka has joined #openstack-nova19:52
*** chyka_ has quit IRC19:56
*** gmann has quit IRC19:57
*** Apoorva has quit IRC19:58
mriedemi was just giving you shit about that20:00
*** suresh12 has joined #openstack-nova20:01
*** suresh12 has quit IRC20:03
*** suresh12_ has joined #openstack-nova20:03
melwittk :)20:03
*** Tom-Tom has joined #openstack-nova20:05
*** suresh12_ has quit IRC20:09
*** suresh12 has joined #openstack-nova20:10
*** huanxie has quit IRC20:11
*** huanxie has joined #openstack-nova20:12
*** pcaruana has quit IRC20:12
*** chyka has quit IRC20:21
*** chyka has joined #openstack-nova20:22
*** pcaruana has joined #openstack-nova20:29
*** AlexeyAbashkin has joined #openstack-nova20:29
*** AlexeyAbashkin has quit IRC20:33
*** derekh has joined #openstack-nova20:35
*** catintheroof has quit IRC20:37
*** catintheroof has joined #openstack-nova20:38
openstackgerritMerged openstack/nova master: Pass RequestSpec to ConductorTaskAPI.build_instances  https://review.openstack.org/51549520:39
*** owalsh_ has joined #openstack-nova20:39
cfriesenmriedem: do you know why we're calling self._update() at https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L590 ?  Having that means that every time we call_update_available_resource() we'll save invalid data, then overwrite it a bit later with valid data.20:41
*** pcaruana has quit IRC20:42
*** owalsh has quit IRC20:42
mriedemcfriesen: oh boy20:42
mriedemyou don't know?!20:42
*** catintheroof has quit IRC20:42
mriedemcfriesen: https://review.openstack.org/#/c/520024/20:42
*** huanxie has quit IRC20:42
mriedemfeel free to update that change20:43
mriedemmaciejjozefczyk: ^20:43
mriedemnova meeting in 17 minutes20:43
mriedemlast one for the year20:43
cfriesenmriedem: sweet, thanks20:43
*** huanxie has joined #openstack-nova20:44
openstackgerritMerged openstack/nova master: Fix an error in _get_host_states when deleting a compute node  https://review.openstack.org/52935220:49
openstackgerritMerged openstack/nova master: VMware: expose max vCPUs and max memory per ESX host  https://review.openstack.org/51663520:49
openstackgerritMerged openstack/nova master: VMware: implement get_inventory() driver method  https://review.openstack.org/50617520:49
openstackgerritMerged openstack/nova master: Pass mountpoint to volume attachment_create with connector  https://review.openstack.org/52747920:50
openstackgerritMatt Riedemann proposed openstack/nova master: Use volume shared_targets to lock during attach/detach  https://review.openstack.org/52969520:50
mriedemildikov: jgriffith: ^20:50
ildikovmriedem: thanks!20:50
mriedemedleafe: you might hit a merge conflict now that https://review.openstack.org/#/c/515495/ merged20:51
*** takashin has joined #openstack-nova20:53
jgriffithmriedem: ack, I'll check it out here shortly.  Thanks!20:56
edleafemriedem: oh joy20:56
mriedemedleafe: maybe, but not sure20:56
mriedemeh, yeah it's in merge conflict20:56
mriedemi can fix it up20:57
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Don't try to delete build request during a reschedule  https://review.openstack.org/52969720:59
*** derekh has quit IRC21:06
*** huanxie has quit IRC21:13
*** huanxie has joined #openstack-nova21:14
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Don't try to delete build request during a reschedule  https://review.openstack.org/52970321:14
*** sahid has quit IRC21:16
*** smatzek has quit IRC21:17
*** smatzek has joined #openstack-nova21:20
*** gouthamr has joined #openstack-nova21:22
*** flwang has quit IRC21:23
*** Apoorva has joined #openstack-nova21:24
*** smatzek has quit IRC21:25
*** awaugama has quit IRC21:27
*** eharney has quit IRC21:29
*** threestrands_ has joined #openstack-nova21:35
*** threestrands_ has quit IRC21:35
*** threestrands_ has joined #openstack-nova21:35
*** flwang has joined #openstack-nova21:36
mriedemedleafe: got that rebase done locally, just running tests21:37
edleafek21:39
*** suresh12 has quit IRC21:40
*** priteau_ has joined #openstack-nova21:47
*** priteau has quit IRC21:50
*** smatzek has joined #openstack-nova21:51
*** suresh12 has joined #openstack-nova21:54
*** andreaf has quit IRC21:55
openstackgerritMerged openstack/nova master: conf: libvirt: Cleanup CPU modelling related options  https://review.openstack.org/52769121:56
*** andreaf has joined #openstack-nova21:56
*** zer0c00l has joined #openstack-nova21:59
*** smatzek has quit IRC22:01
*** smatzek has joined #openstack-nova22:02
mriedemhere it comes22:04
openstackgerritMatt Riedemann proposed openstack/nova master: Don't try to delete build request during a reschedule  https://review.openstack.org/52883522:05
openstackgerritMatt Riedemann proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135822:05
openstackgerritMatt Riedemann proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643622:05
mriedemedleafe: you might want to look this over quick https://review.openstack.org/#/c/511358/48..49/22:09
mriedemmake sure i didn't eff anything up22:09
edleafemriedem: looks sane22:14
mriedemalright, bombs away22:15
*** huanxie has quit IRC22:15
*** huanxie has joined #openstack-nova22:18
*** rcernin has joined #openstack-nova22:24
*** smatzek has quit IRC22:26
*** smatzek has joined #openstack-nova22:26
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060822:27
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948322:27
openstackgerritEric Berglund proposed openstack/nova master: PowerVM Driver: SEA  https://review.openstack.org/52321622:30
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI  https://review.openstack.org/52609422:30
*** smatzek has quit IRC22:31
*** priteau_ has quit IRC22:32
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI  https://review.openstack.org/52609422:33
*** catintheroof has joined #openstack-nova22:37
*** catintheroof has quit IRC22:37
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.59 - List/Show all server migration types  https://review.openstack.org/43083922:38
*** lyan has quit IRC22:46
*** salv-orlando has quit IRC22:49
*** salv-orlando has joined #openstack-nova22:51
*** salv-orlando has quit IRC22:54
*** tssurya has quit IRC22:55
*** burt has quit IRC22:58
*** Dave__ is now known as Dave23:01
*** edmondsw has joined #openstack-nova23:04
*** stvnoyes has quit IRC23:06
*** crushil has joined #openstack-nova23:08
*** edmondsw has quit IRC23:08
*** sup has joined #openstack-nova23:09
*** sup is now known as Guest9893823:10
*** kumarmn has quit IRC23:13
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild  https://review.openstack.org/52971823:13
Guest98938Hello guys ..... Has anybody here seen an issue with nova-scheduler where max_instances_per_host host aggregate tag is not honored ?23:15
Guest98938I am seeing number of instances in those hosts go beyond max_instances_per_host limit set. This is on newton.23:16
*** crushil has quit IRC23:16
mriedemGuest98938: no, but it's also likely not tested very well and it's also likely racey23:17
mriedemor if you bypass the scheduler by forcing a host during live migration or evacuate23:18
Guest98938I did further analysis on this and found stats in compute_node object to be empty, hence scheduler thinks num_instances to be 0 and schedules to this host23:18
mriedemit doesn't look like stats is where the filter pulls that value from23:21
mriedemit checks the HostState.instances value23:21
mriedem*HostState.num_instances,23:21
mriedemso maybe that comes from stats?23:22
mriedemah you're right23:22
Guest98938mriedem: Do you know of any condition which maybe causing this stats to be empty for a compute_node ? I see this happening every some time23:22
mriedemnot without digging23:22
mriedemi also see that HostState._locked_consume_from_request overrides the num_instances value23:23
mriedemthere is also HostState.instances, which is a dict tracking the instances on that host23:23
Guest98938I basically have setup a heat autoscaling with cooldown of 60 secs, to continuously try to spawn instances23:23
mriedemso apparently there are 3 different ways to screw this up23:23
Guest98938oh23:24
mriedemidk this stats stuff all looks like garbage to me23:24
Guest98938yea I thought of a fix for this problem by relying on running_vms attribute rather than on stats(which is missing sometimes)23:25
mriedemi would try this23:26
mriedemhttps://github.com/openstack/nova/blob/master/nova/scheduler/filters/num_instances_filter.py#L3723:26
mriedemnum_instances = host_state.num_instances or len(host_state.instances)23:26
*** bnemec is now known as bnemec-xmas23:26
mriedemor even just compare host_state.num_instances to len(host_state.instances) and if those are different, there is a problem somewhere and should probably log a warning23:27
mriedemcould help with your debug23:27
*** kumarmn has joined #openstack-nova23:27
*** chyka has quit IRC23:29
*** chyka has joined #openstack-nova23:30
Guest98938yea that can help23:30
Guest98938or how about making scheduler to pick num_instances from compute_node.running_vms instead of self.stats.get('num_instances', 0) ?23:31
mriedemwell, you could have stopped vms on a node right23:31
mriedem?23:31
mriedemalthough cn.running_vms = self.stats.num_instances23:32
mriedemi guess those are the same23:32
mriedemi would think that if stats.num_instances is wrong, that compute_node.running_vms will also be wrong23:33
mriedembecause the latter is set based on the former23:34
*** AlexeyAbashkin has joined #openstack-nova23:34
Guest98938so does running_vms include stopped vms ?23:34
mriedemyeah i think so23:34
Guest98938yea, I think the problem here is self.stats are missing and compute_node.running_vms exists from my observation23:34
Guest98938which is why I though of relying on running_vms23:35
*** tetsuro_ has joined #openstack-nova23:37
mriedemit's possible there could be a problem where stats.num_instances doesn't get updated properly for a move operation23:37
mriedemi.e. a migration to another host23:37
*** fragatina has quit IRC23:38
*** AlexeyAbashkin has quit IRC23:39
mriedemthis change might have regressed something in newton23:39
mriedemhttps://review.openstack.org/#/c/307124/21/nova/compute/resource_tracker.py@24523:39
mriedemsee how that comment is wrong23:39
mriedemit might have just meant _update_usage_from_migration23:40
mriedemand was a typo23:40
mriedemi think that's a typo,23:40
mriedembut i also think that a move operation isn't properly updating stats.num_instances23:40
mriedemthe update_available_resource periodic task which runs by default every minute should update stats.num_instances for the instances that currently live on that host,23:41
mriedembut you could hit a window between the periodic task running and a move happening23:41
mriedemwhich throws off your filter23:41
*** yikun_ has joined #openstack-nova23:42
Guest98938yea, but in my case there is no migration happening23:43
*** huanxie has quit IRC23:48
*** huanxie has joined #openstack-nova23:49
mriedemok, idk then23:50
mriedemi'd still check this in the filter23:50
mriedemnum_instances = host_state.num_instances or len(host_state.instances)23:50
mriedemif those 2 values on the right are different, i'd log a warning23:50
tonybmriedem: followign up from the meetign I just missed ... I'll do the newton EOL in Jan23:50
mriedemtonyb: ok23:51
*** salv-orlando has joined #openstack-nova23:55
Guest98938yea, I will try the filter approach too, thanks23:57

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