Wednesday, 2017-09-06

*** harlowja has joined #openstack-nova00:03
*** yangyapeng has quit IRC00:06
*** annegentle has joined #openstack-nova00:07
*** yangyapeng has joined #openstack-nova00:07
*** harlowja has quit IRC00:09
*** yangyapeng has quit IRC00:11
*** moshele has joined #openstack-nova00:13
*** edleafe has quit IRC00:14
*** harlowja has joined #openstack-nova00:14
*** edleafe has joined #openstack-nova00:16
*** yangyapeng has joined #openstack-nova00:18
*** edmondsw has joined #openstack-nova00:19
*** thorst has joined #openstack-nova00:20
*** yangyapeng has quit IRC00:22
*** thorst has quit IRC00:23
*** Swami has quit IRC00:24
*** jaypipes has quit IRC00:24
*** edmondsw has quit IRC00:24
*** yassine has joined #openstack-nova00:32
*** thorst has joined #openstack-nova00:32
mriedemgmann: these don't require microversion changes, correct?00:33
openstackgerritwanghongtaozz proposed openstack/nova stable/pike: spelling error availiable  change to available  https://review.openstack.org/50104300:34
*** esberglu has joined #openstack-nova00:34
gmannmriedem, yes. only thing i want to confirm from alex_xu about restricting the int convertible string as limit like '1' it used to be valid and converted by utils previously and now it will be 40000:34
gmanni think we discussed it in original spec but i do not remember the consensus .00:35
mriedemgmann: if a microversion bump is required then i think we need a spec,00:36
mriedemotherwise i'm ok with specless00:36
gmannmriedem, yea, if so it need spec. we will discuss it in today meeting for all cases and ll update you00:37
mriedemthanks00:37
openstackgerritwanghongtaozz proposed openstack/nova stable/pike: spelling mistake availiable change to available  https://review.openstack.org/50104500:38
*** esberglu has quit IRC00:38
*** rajathagasthya has quit IRC00:40
openstackgerritwanghongtaozz proposed openstack/nova stable/pike: spelling mistake prefered  change to preferred  https://review.openstack.org/50104600:41
*** baoli has joined #openstack-nova00:41
*** hshiina has joined #openstack-nova00:42
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: WIP spelling error availiable  change to available  https://review.openstack.org/50104300:43
*** gouthamr has quit IRC00:45
*** baoli_ has joined #openstack-nova00:45
*** baoli has quit IRC00:46
openstackgerritwanghongtaozz proposed openstack/nova stable/pike: spelling mistake intergration change to integration  https://review.openstack.org/50104700:47
alex_xumriedem: gmann it needn't microversion, I think just just add query params validation for the exist API and keep it same behavour for the API00:47
gmannalex_xu, but we are doing non negative integer for limit - https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/keypairs.py#L10900:49
*** dave-mccowan has quit IRC00:49
*** gouthamr has joined #openstack-nova00:49
gmannhttps://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/validation/parameter_types.py#L43800:49
gmannalex_xu, this is only case change the behavior for 'int' 200 -> 40000:50
alex_xugmann: that isn't backward-compatible?00:50
alex_xugmann: I remember there are some validation in the python code for the limits, that should check the non-negative value also00:50
gmannalex_xu, humm00:51
gmannalex_xu, i found the flow like this00:51
gmann1. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/openstack/common.py#L18900:51
gmann2. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/api/openstack/common.py#L20200:51
*** jichen has joined #openstack-nova00:51
gmann3. https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/utils.py#L88300:52
gmann3rd one convert the '1' to 100:52
*** zhurong has joined #openstack-nova00:53
*** TuanLA has joined #openstack-nova00:54
alex_xugmann: there is parameter 'min_value', that is limited to non-negative?00:54
* alex_xu rush to next meeting...00:55
*** mriedem has quit IRC00:55
*** harlowja has quit IRC00:57
*** dixiaoli has joined #openstack-nova00:57
*** harlowja has joined #openstack-nova01:01
openstackgerritMerged openstack/nova master: tests: Remove useless test  https://review.openstack.org/48303101:02
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certificates to REST API  https://review.openstack.org/48620401:02
*** phuongnh has joined #openstack-nova01:05
*** Apoorva_ has joined #openstack-nova01:05
*** nicolasbock has quit IRC01:06
openstackgerritMerged openstack/nova master: Cleanup allocations on invalid dest node during live migration  https://review.openstack.org/49886101:06
*** thorst has quit IRC01:06
*** rajathagasthya has joined #openstack-nova01:07
alex_xugmann: I may not clear the question01:08
alex_xuthe value of '1' will get 400?01:08
openstackgerritMerged openstack/nova master: iso8601.is8601.Utc No Longer Exists  https://review.openstack.org/49828701:09
*** Apoorva has quit IRC01:09
*** Apoorva_ has quit IRC01:10
openstackgerritChris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007301:14
openstackgerritChris Dent proposed openstack/nova master: WIP: [placement] Allow _set_allocations to delete allocations  https://review.openstack.org/50105101:15
*** yamamoto has joined #openstack-nova01:15
*** yangyapeng has joined #openstack-nova01:15
cdentdansmith: some of the jiggery pokery in that ^ is starting to get a bit wack01:15
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.54 - Add trusted_certificates param  https://review.openstack.org/50039601:15
*** nicolasbock has joined #openstack-nova01:21
*** hongbin has joined #openstack-nova01:22
*** cdent has quit IRC01:23
*** thorst has joined #openstack-nova01:24
*** scottda_ has quit IRC01:24
*** thorst has quit IRC01:25
openstackgerritjichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt  https://review.openstack.org/50105701:38
*** thorst has joined #openstack-nova01:39
*** thorst has quit IRC01:39
*** markvoelker has joined #openstack-nova01:43
*** baoli_ has quit IRC01:44
*** baoli has joined #openstack-nova01:45
*** vikrant has joined #openstack-nova01:48
*** itlinux has joined #openstack-nova01:54
*** edmondsw has joined #openstack-nova01:58
*** edmondsw has quit IRC02:02
openstackgerritjichenjc proposed openstack/nova master: conf: Remove quota_networks options  https://review.openstack.org/50106802:05
*** thorst has joined #openstack-nova02:07
*** thorst has quit IRC02:07
*** thorst has joined #openstack-nova02:07
*** vikrant has quit IRC02:08
*** thorst has quit IRC02:12
*** yamahata has quit IRC02:14
*** xinliang has quit IRC02:17
*** markvoelker has quit IRC02:18
openstackgerritChenghui Yu proposed openstack/nova master: Remove usage of kwarg retry_on_request in API  https://review.openstack.org/50107302:21
*** esberglu has joined #openstack-nova02:22
*** coreywright has quit IRC02:26
*** esberglu has quit IRC02:26
*** gbarros has joined #openstack-nova02:27
*** baoli has quit IRC02:28
*** baoli has joined #openstack-nova02:29
*** xinliang has joined #openstack-nova02:30
*** thorst has joined #openstack-nova02:34
*** thorst has quit IRC02:34
*** baoli has quit IRC02:40
*** coreywright has joined #openstack-nova02:44
*** liusheng has quit IRC02:51
*** jichen has quit IRC02:52
*** liusheng has joined #openstack-nova02:52
*** zhurong has quit IRC02:58
*** baoli has joined #openstack-nova02:59
*** penick has joined #openstack-nova03:06
*** abhi89 has joined #openstack-nova03:10
*** gvrangan has joined #openstack-nova03:10
*** penick has quit IRC03:10
*** yangyape_ has joined #openstack-nova03:11
*** baoli has quit IRC03:11
*** moshele has quit IRC03:11
*** yangyap__ has joined #openstack-nova03:14
*** yangyape_ has quit IRC03:14
*** yangyapeng has quit IRC03:14
*** jichen has joined #openstack-nova03:15
*** markvoelker has joined #openstack-nova03:15
*** yangyap__ has quit IRC03:19
*** yangyapeng has joined #openstack-nova03:19
*** nicolasbock has quit IRC03:20
*** yamamoto_ has joined #openstack-nova03:25
*** links has joined #openstack-nova03:28
*** yamamoto has quit IRC03:28
*** yangyape_ has joined #openstack-nova03:30
*** yangyapeng has quit IRC03:30
*** hshiina has quit IRC03:34
*** annegentle has quit IRC03:37
*** takashin has joined #openstack-nova03:44
*** gouthamr has quit IRC03:46
*** edmondsw has joined #openstack-nova03:46
*** udesale has joined #openstack-nova03:46
*** markvoelker has quit IRC03:48
*** rajathagasthya has quit IRC03:48
*** edmondsw has quit IRC03:51
*** adisky__ has quit IRC03:55
*** gvrangan has quit IRC03:56
*** kiennt has joined #openstack-nova03:58
*** hongbin has quit IRC03:58
*** hshiina has joined #openstack-nova03:59
*** gyee has quit IRC04:00
*** gbarros has quit IRC04:02
*** zhurong has joined #openstack-nova04:08
*** tbh_ has joined #openstack-nova04:09
*** esberglu has joined #openstack-nova04:10
*** esberglu has quit IRC04:14
*** gcb has quit IRC04:19
*** trungnv has quit IRC04:19
*** gcb has joined #openstack-nova04:21
*** thorst has joined #openstack-nova04:26
*** psachin has joined #openstack-nova04:31
*** thorst has quit IRC04:31
*** trungnv has joined #openstack-nova04:37
*** trungnv has quit IRC04:37
*** trinaths has joined #openstack-nova04:42
openstackgerritPooja Jadhav proposed openstack/nova master: Fix ValueError if invalid max_rows passed to db purge  https://review.openstack.org/50077104:43
*** armax has quit IRC04:44
*** markvoelker has joined #openstack-nova04:45
*** gibi has quit IRC04:48
*** gvrangan has joined #openstack-nova04:48
*** zxy has quit IRC04:49
*** zxy has joined #openstack-nova04:49
*** moshele has joined #openstack-nova05:00
itlinuxthere are 4 compute nodes.. and those images are cirros..05:03
*** diga has joined #openstack-nova05:04
*** gvrangan has quit IRC05:04
*** claudiub|2 has joined #openstack-nova05:06
*** yamamoto_ has quit IRC05:07
*** yamamoto has joined #openstack-nova05:09
itlinuxhello team..05:11
itlinuxlooks like the scheduler gets stuck when I try to spin up few machines at one time..05:11
itlinuxlike 4 or 505:11
itlinuxI have 4 compute nodes and the images are cirros05:11
itlinuxI checked the hypervisors are up05:11
*** zxy has quit IRC05:14
*** zxy has joined #openstack-nova05:14
*** markvoelker has quit IRC05:18
*** shan has joined #openstack-nova05:19
*** kairo_ has joined #openstack-nova05:20
*** kairo has quit IRC05:22
*** zxy has quit IRC05:22
*** zxy has joined #openstack-nova05:22
*** moshele has quit IRC05:23
*** Tom_ has joined #openstack-nova05:24
*** pcaruana has joined #openstack-nova05:27
*** gvrangan has joined #openstack-nova05:33
*** yamahata has joined #openstack-nova05:39
*** ratailor has joined #openstack-nova05:41
*** liverpooler has joined #openstack-nova05:46
openstackgerritjichenjc proposed openstack/nova master: Update doc to indicate nova-network deprecated  https://review.openstack.org/50065405:47
*** yamahata has quit IRC05:48
openstackgerritjichenjc proposed openstack/nova master: conf: Remove quota_networks options  https://review.openstack.org/50106805:49
gmannalex_xu, i mean, if we pass any int as string in limit like '1' not 1, it will be 400 in current case of keypair. because of schema05:49
itlinuxhello nova team.. anyone has a tweak since when I spin up some vms the compute that gets like two vms takes forever..05:49
itlinuxlike 100sec for a cirros image..05:49
itlinuxsounds a little too much!05:50
*** wlfightup has quit IRC05:53
*** yamahata has joined #openstack-nova05:53
*** udesale__ has joined #openstack-nova05:53
gmannalex_xu, but when i ran it without schema it is also same(400). cannot this convert '1' -? 1 - https://github.com/openstack/nova/blob/dbfde14978d4e4374c1fc1085c7f061b2a22d2c6/nova/utils.py#L88305:55
*** udesale has quit IRC05:56
*** wlfightup has joined #openstack-nova05:56
*** yangyapeng has joined #openstack-nova05:57
*** yangyape_ has quit IRC05:57
*** esberglu has joined #openstack-nova05:58
*** esberglu has quit IRC06:03
*** yamahata has quit IRC06:03
openstackgerritjichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt  https://review.openstack.org/50105706:04
*** dixiaoli has quit IRC06:05
*** dixiaoli has joined #openstack-nova06:05
openstackgerritjichenjc proposed openstack/nova master: Enhance doc for nova services  https://review.openstack.org/49953606:09
openstackgerritjichenjc proposed openstack/nova master: propagate OSError to MigrationPreCheckError  https://review.openstack.org/46977906:09
alex_xugmann: that schema should work with '1', not sure I missed something06:11
gmannalex_xu, oh, 1 sec06:13
*** yamahata has joined #openstack-nova06:17
*** moshele has joined #openstack-nova06:18
openstackgerritMerged openstack/nova master: Add uuid to migration object and migrate-on-load  https://review.openstack.org/49693406:19
*** thorst has joined #openstack-nova06:27
*** udesale__ has quit IRC06:28
*** links has quit IRC06:29
*** shan has quit IRC06:29
*** ratailor has quit IRC06:29
*** psachin has quit IRC06:29
*** yingjun has joined #openstack-nova06:30
*** thorst has quit IRC06:32
*** yamamoto has quit IRC06:34
*** yangyapeng has quit IRC06:35
*** yamamoto has joined #openstack-nova06:35
*** yangyapeng has joined #openstack-nova06:35
*** sridharg has joined #openstack-nova06:36
*** adisky__ has joined #openstack-nova06:38
openstackgerritMerged openstack/nova master: Add uuid online migration for migrations  https://review.openstack.org/49858906:40
*** ssmith has quit IRC06:43
*** sahid has joined #openstack-nova06:51
*** udesale has joined #openstack-nova06:55
*** links has joined #openstack-nova06:55
*** ratailor has joined #openstack-nova06:55
*** rcernin has joined #openstack-nova06:57
openstackgerritGao Zexu proposed openstack/os-traits master: Update the documentation urls  https://review.openstack.org/50112206:57
*** udesale__ has joined #openstack-nova06:57
*** udesale__ has quit IRC06:58
*** udesale__ has joined #openstack-nova06:59
*** psachin has joined #openstack-nova06:59
*** gibi has joined #openstack-nova07:00
*** udesale has quit IRC07:00
*** andreas_s has joined #openstack-nova07:01
openstackgerritjichenjc proposed openstack/nova master: WIP:check query param for used_limits function  https://review.openstack.org/49909107:02
*** yingjun has quit IRC07:06
*** udesale__ has quit IRC07:09
openstackgerritjichenjc proposed openstack/nova master: WIP:Remove quota_networks and enable_network_quota  https://review.openstack.org/50106807:09
*** udesale__ has joined #openstack-nova07:10
jichengmann: hi, in https://review.openstack.org/#/c/501057/2/nova/tests/unit/api/openstack/compute/test_config_drive.py,  you mentioned :i like to have some uuid as project id here instead of removing.07:11
jichenI think in v2.18, we have:07:11
jichen2.18 Establishes a set of routes that makes project_id an optional construct in v2.1.07:12
jichenwhy we still need add project id in the test ?07:12
*** udesale__ has quit IRC07:15
*** udesale has joined #openstack-nova07:15
*** markvoelker has joined #openstack-nova07:15
gmannjichen, yea, but we do have tests to test both07:21
*** jpena|off is now known as jpena07:22
gmannjichen, i do not have string opinion on this but i we can discuss whether it is ok to remove the project_id from tests07:22
*** edmondsw has joined #openstack-nova07:22
gmannjichen, its unit test only right? does it matter anywhere.  i mean url is really dummy things in unit tests right07:23
*** kairo_ is now known as kairo07:23
*** takashin has quit IRC07:23
*** kairo has quit IRC07:23
*** kairo has joined #openstack-nova07:23
*** edmondsw has quit IRC07:26
openstackgerritLajos Katona proposed openstack/nova master: Add functional migrate force_complete test  https://review.openstack.org/49620207:28
*** tesseract has joined #openstack-nova07:30
*** ralonsoh has joined #openstack-nova07:32
jichengmann: yeah, we should have test for both..I will see whether it's possible to at least include some project id into test07:34
jichengmann: and yes, it's in unit test , but PS#1 shows a lot of error if I use /v2.1/fake/xxx, so I tried to move to /v2.1/xxx and it works locally07:35
jichenso I think at least in some cases it matters :)07:35
*** mdrabe has quit IRC07:36
gmannjichen, can we use some dummy UUID in this change and later we decide whether to remove project-id in unit tests or not as whole07:37
openstackgerritsahid proposed openstack/os-vif master: ovs-hybrid: should permanently keep MAC entries  https://review.openstack.org/50113207:37
jichenok, that's doable, I think you are saying is /v2.1/0000-0000-00000-00000/xxx  (a uuid format) instead /v2.1/fake/xxx ,right?07:38
gmannsomething like '11111111-1111- '07:38
gmannjichen, yea07:38
jichengmann: ok, let me try it, thanks for comments07:39
gmannjichen, thanks07:39
*** hieulq has quit IRC07:40
*** TuanLA has quit IRC07:40
*** TuanLA has joined #openstack-nova07:41
*** hieulq has joined #openstack-nova07:41
*** claudiub|2 has quit IRC07:43
openstackgerritsahid proposed openstack/nova master: libvirt: slowly live-migration to ensure network is ready  https://review.openstack.org/49745707:43
openstackgerritsahid proposed openstack/nova master: libvirt: add method to configure migration speed  https://review.openstack.org/49745607:43
openstackgerritsahid proposed openstack/nova master: libvirt: bandwidth param should be set in guest migrate  https://review.openstack.org/49745507:44
*** esberglu has joined #openstack-nova07:46
*** markvoelker has quit IRC07:49
openstackgerritMerged openstack/nova master: Refactor LiveMigrationTask._find_destination  https://review.openstack.org/49887407:50
*** esberglu has quit IRC07:50
openstackgerritMerged openstack/nova master: Add missing unit tests for FilterScheduler._get_all_host_states  https://review.openstack.org/49680307:54
*** hoonetorg has quit IRC07:58
*** udesale has quit IRC07:58
*** udesale has joined #openstack-nova07:59
*** hferenc has joined #openstack-nova08:00
*** edmondsw has joined #openstack-nova08:03
openstackgerritLajos Katona proposed openstack/nova master: Add functional for live migrate delete  https://review.openstack.org/49958308:08
*** alexpilotti has quit IRC08:14
*** ociuhandu_ has quit IRC08:14
*** hoonetorg has joined #openstack-nova08:14
*** phuongnh has quit IRC08:15
*** phuongnh has joined #openstack-nova08:16
*** yangyapeng has quit IRC08:19
*** yangyapeng has joined #openstack-nova08:20
*** gcb has quit IRC08:23
*** phuongnh has quit IRC08:23
*** phuongnh has joined #openstack-nova08:24
*** liverpooler has quit IRC08:25
*** jaosorior has quit IRC08:27
*** lucas-afk is now known as lucasagomes08:28
*** thorst has joined #openstack-nova08:28
*** thorst has quit IRC08:33
openstackgerritTetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm  https://review.openstack.org/46516008:37
*** yamahata has quit IRC08:38
*** suresh12 has quit IRC08:41
*** markvoelker has joined #openstack-nova08:46
*** alexchadin has joined #openstack-nova08:47
*** derekh has joined #openstack-nova08:51
*** alexpilotti has joined #openstack-nova08:54
*** moshele has quit IRC08:54
*** tbh_ has quit IRC08:58
*** moshele has joined #openstack-nova09:03
*** hferenc has quit IRC09:05
*** ociuhandu has joined #openstack-nova09:08
*** sambetts|afk is now known as sambetts09:10
*** lpetrut has joined #openstack-nova09:13
*** rabel has joined #openstack-nova09:14
*** yangyape_ has joined #openstack-nova09:15
*** hshiina has quit IRC09:15
*** ociuhandu has quit IRC09:18
*** openstackgerrit has quit IRC09:18
*** yangyapeng has quit IRC09:18
*** yangyape_ has quit IRC09:19
*** yangyapeng has joined #openstack-nova09:19
*** markvoelker has quit IRC09:19
*** ociuhandu has joined #openstack-nova09:20
*** adisky__ has quit IRC09:32
*** yamamoto has quit IRC09:40
*** jichen has quit IRC09:41
stephenfinbauzas: Is my point about not using 'prelude' in reno notes correct here? https://review.openstack.org/#/c/459166/09:41
*** gvrangan has quit IRC09:42
*** alexchadin has quit IRC09:42
bauzasstephenfin: looking09:42
bauzasstephenfin: that's correct09:43
bauzasstephenfin: I did a bit of writing in the doc about that09:43
bauzassec, finding it09:43
bauzas(that new docs rearrangement is making me finding my kids :) )09:43
bauzasstephenfin: https://docs.openstack.org/nova/latest/contributor/releasenotes.html09:44
stephenfinbauzas: Perfect. I'll leave a note for mikal saying as much. Ta :)09:47
bauzasstephenfin: just did09:47
bauzashonestly, that privsep effort is important, so I'll see how I can help with reviews09:48
stephenfinYup, I started on that yesterday. It's getting pretty big and has been there for a while now09:49
*** dixiaoli_ has joined #openstack-nova09:51
*** dixiaoli has quit IRC09:52
*** ociuhandu_ has joined #openstack-nova09:52
bauzasgood opportunity then for understanding how privsep works :)09:53
bauzasI barely understand it, but those changes will help09:53
*** sdague has joined #openstack-nova09:53
*** ociuhandu has quit IRC09:54
*** yangyapeng has quit IRC09:56
*** yangyapeng has joined #openstack-nova09:56
*** claudiub|2 has joined #openstack-nova09:57
*** dixiaoli_ has quit IRC09:57
*** nicolasbock has joined #openstack-nova10:02
*** dixiaoli has joined #openstack-nova10:02
*** diga has quit IRC10:06
*** nicolasbock has quit IRC10:07
*** openstackgerrit has joined #openstack-nova10:08
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin  https://review.openstack.org/49308210:08
*** yamamoto has joined #openstack-nova10:11
*** abhi89 has quit IRC10:11
*** edmondsw has quit IRC10:13
*** diga has joined #openstack-nova10:13
*** jaosorior has joined #openstack-nova10:14
*** yamamoto has quit IRC10:16
*** markvoelker has joined #openstack-nova10:17
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add VersionedObjectPrintable mixin  https://review.openstack.org/49308210:17
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add ``HostPortProfileInfo`` class  https://review.openstack.org/44159010:17
*** ociuhandu_ has quit IRC10:18
openstackgerritGhanshyam Mann proposed openstack/nova master: [WIP] Add tests for 'limit' filter as string for GET keypair API  https://review.openstack.org/50120210:23
*** yangyapeng has quit IRC10:25
*** yangyapeng has joined #openstack-nova10:25
*** thorst has joined #openstack-nova10:29
*** yangyapeng has quit IRC10:29
openstackgerritsahid proposed openstack/nova master: objects: add new field MASK for emulator threads policy  https://review.openstack.org/50120510:31
openstackgerritsahid proposed openstack/nova master: objects: add cpu_emulator_threads_mask field  https://review.openstack.org/50120610:31
openstackgerritsahid proposed openstack/nova master: hardware: add method to get emulator threads mask  https://review.openstack.org/50120710:31
openstackgerritsahid proposed openstack/nova master: hardware: handle/check emulator threads mask constraints  https://review.openstack.org/50120810:31
openstackgerritsahid proposed openstack/nova master: hardware: add method to compute emulator threads placement  https://review.openstack.org/50120910:31
openstackgerritsahid proposed openstack/nova master: libvirt: configure emulator threads of a guest based on mask  https://review.openstack.org/50121010:31
*** yamamoto has joined #openstack-nova10:32
*** thorst has quit IRC10:34
*** nicolasbock has joined #openstack-nova10:36
*** dixiaoli has quit IRC10:36
openstackgerritGhanshyam Mann proposed openstack/nova master: [DNM] Fir testing only  https://review.openstack.org/50121110:37
openstackgerritGhanshyam Mann proposed openstack/nova master: [DNM] For testing only  https://review.openstack.org/50121110:38
*** gszasz has joined #openstack-nova10:40
*** kiennt has quit IRC10:41
*** suresh12 has joined #openstack-nova10:42
*** tbachman has quit IRC10:42
*** Tom_ has quit IRC10:43
*** tbachman has joined #openstack-nova10:43
*** Tom has joined #openstack-nova10:43
*** suresh12 has quit IRC10:47
*** tbachman has quit IRC10:47
*** Tom has quit IRC10:48
*** gaurangt has quit IRC10:48
*** yamamoto has quit IRC10:48
*** markvoelker has quit IRC10:49
*** udesale has quit IRC10:50
openstackgerritwanghongtaozz proposed openstack/nova master: spelling mistake  https://review.openstack.org/50121510:50
*** gaurangt has joined #openstack-nova10:52
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API  https://review.openstack.org/47661210:52
*** liverpooler has joined #openstack-nova10:52
*** zhurong has quit IRC10:54
openstackgerritsahid proposed openstack/nova master: libvirt: configure trust mode for vfs  https://review.openstack.org/45851410:55
openstackgerritsahid proposed openstack/nova master: network: update pci request spec to handle trusted tags  https://review.openstack.org/45882010:55
openstackgerritsahid proposed openstack/nova master: network: add command to configure trusted mode for VFs  https://review.openstack.org/45851310:55
*** TuanLA has quit IRC10:55
*** alexchadin has joined #openstack-nova10:58
*** alexchadin has quit IRC10:58
*** alexchadin has joined #openstack-nova10:58
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API  https://review.openstack.org/48222610:58
*** nicolasbock has quit IRC11:07
*** nicolasbock has joined #openstack-nova11:09
*** phuongnh has quit IRC11:12
*** hrw has quit IRC11:17
*** psachin has quit IRC11:19
*** ratailor has quit IRC11:20
*** links has quit IRC11:20
*** lucasagomes is now known as lucas-hungry11:21
*** rabel has quit IRC11:23
*** rabel has joined #openstack-nova11:24
*** abhi89 has joined #openstack-nova11:27
*** trinaths has left #openstack-nova11:28
openstackgerritsahid proposed openstack/nova master: compute: reset instance events  https://review.openstack.org/42002611:29
*** yangyapeng has joined #openstack-nova11:29
*** hrw has joined #openstack-nova11:29
openstackgerritsahid proposed openstack/nova master: compute: reset instance events  https://review.openstack.org/42002611:30
*** ratailor has joined #openstack-nova11:31
*** links has joined #openstack-nova11:32
*** psachin has joined #openstack-nova11:32
*** Tom has joined #openstack-nova11:37
*** dave-mccowan has joined #openstack-nova11:39
*** nicolasbock has quit IRC11:40
*** Tom has quit IRC11:42
*** yamamoto has joined #openstack-nova11:45
*** Tom_ has joined #openstack-nova11:45
*** markvoelker has joined #openstack-nova11:46
*** smatzek has joined #openstack-nova11:47
*** smatzek has quit IRC11:49
*** smatzek has joined #openstack-nova11:49
*** Tom_ has quit IRC11:49
*** mamandle has joined #openstack-nova11:49
*** lpetrut has quit IRC11:51
*** tbachman has joined #openstack-nova11:52
*** nicolasbock has joined #openstack-nova11:52
*** kylek3h has quit IRC11:52
*** kylek3h has joined #openstack-nova11:53
*** alexchadin has quit IRC11:57
openstackgerritSergey Nikitin proposed openstack/nova-specs master: PCI NUMA Policies  https://review.openstack.org/36114011:57
*** tbachman has quit IRC11:57
*** kylek3h has quit IRC11:57
*** jpena is now known as jpena|lunch12:00
*** gvrangan has joined #openstack-nova12:01
*** tbachman has joined #openstack-nova12:04
*** hrw has quit IRC12:04
*** yangyapeng has quit IRC12:05
*** yangyapeng has joined #openstack-nova12:05
*** gcb has joined #openstack-nova12:05
*** udesale has joined #openstack-nova12:07
mamandlemriedem: after commit 1b2f3c7bb25667527e7e1d2afd84d5d5533e7751, the evacuated instance cleanup is failing in delete_allocation_for_evacuated_instance complaining that the compute node key is not present in resource_tracker compute_nodes . I looked a bit into it and it12:07
*** weshay_PTO is now known as weshay12:08
*** kylek3h has joined #openstack-nova12:09
mamandleit looks like we need to ensure that resources have initialized (via _update_available_resource) before this is called?12:09
*** yangyapeng has quit IRC12:09
*** edmondsw has joined #openstack-nova12:10
*** kylek3h has quit IRC12:10
*** ociuhandu has joined #openstack-nova12:10
*** edmondsw has quit IRC12:11
*** edmondsw has joined #openstack-nova12:11
*** hrw has joined #openstack-nova12:11
*** thorst has joined #openstack-nova12:11
*** thorst has quit IRC12:12
*** thorst has joined #openstack-nova12:13
*** yamamoto has quit IRC12:13
*** diga has quit IRC12:14
*** hrw has quit IRC12:14
*** mamandle has quit IRC12:18
*** markvoelker has quit IRC12:20
*** ratailor has quit IRC12:20
*** hrw has joined #openstack-nova12:25
*** markvoelker has joined #openstack-nova12:28
*** lpetrut has joined #openstack-nova12:28
*** yamamoto has joined #openstack-nova12:28
*** jaypipes has joined #openstack-nova12:30
*** lucas-hungry is now known as lucasagomes12:34
*** lyan has joined #openstack-nova12:37
openstackgerritMerged openstack/nova master: Hyper-V: Perform proper cleanup after cold migration  https://review.openstack.org/48695512:39
*** alexchadin has joined #openstack-nova12:40
*** kylek3h has joined #openstack-nova12:40
openstackgerritMerged openstack/nova master: Add release note for requiring shred 8.22 or above.  https://review.openstack.org/50102212:42
openstackgerritMerged openstack/nova master: Update doc to indicate nova-network deprecated  https://review.openstack.org/50065412:43
openstackgerritMerged openstack/nova master: Replace http with https for doc links in nova  https://review.openstack.org/50069312:43
*** trinath has joined #openstack-nova12:45
stephenfinsdague: Now that Queens is open, could you take another look at this patch? https://review.openstack.org/#/c/466799/12:46
trinathHi, While compiling qemu 2.6 from source, "./configure --target-list=aarch64-softmmu --enable-kvm --enable-vhost-net --enable-vnc --enable-curses " , I get the error " ERROR: "cc" cannot build an executable (is your linker broken?)".12:46
trinathplease help me resolve this issue.12:46
*** scottda_ has joined #openstack-nova12:49
*** gmann_ has joined #openstack-nova12:55
sdaguestephenfin: +212:55
stephenfinthank you :)12:55
stephenfintrinath: Totally wrong channel, buddy. You want #qemu or Google12:56
*** udesale has quit IRC12:56
*** pchavva has joined #openstack-nova13:00
alex_xunova api meeting is running at #openstack-meeting-413:01
*** mikal has quit IRC13:05
*** xyang1 has joined #openstack-nova13:05
*** slaweq has quit IRC13:05
*** vladikr has joined #openstack-nova13:07
*** adisky__ has joined #openstack-nova13:07
*** david-lyle has quit IRC13:09
*** david-lyle has joined #openstack-nova13:09
*** takedakn has joined #openstack-nova13:10
openstackgerritBalazs Gibizer proposed openstack/nova master: reno: note that custom resources are not supported  https://review.openstack.org/50125213:10
*** lucasxu has joined #openstack-nova13:11
*** baoli has joined #openstack-nova13:11
*** mikal has joined #openstack-nova13:12
*** jpena|lunch is now known as jpena13:12
openstackgerritBalazs Gibizer proposed openstack/nova stable/pike: reno: note that custom resources are not supported  https://review.openstack.org/50052113:13
*** mriedem has joined #openstack-nova13:13
*** gouthamr has joined #openstack-nova13:14
*** trinath has quit IRC13:14
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Hyper-V: Perform proper cleanup after cold migration  https://review.openstack.org/50125413:15
*** smatzek has quit IRC13:16
*** liverpooler has quit IRC13:16
*** yangyapeng has joined #openstack-nova13:16
*** liverpooler has joined #openstack-nova13:16
*** gmann__ has joined #openstack-nova13:17
*** gmann_ has quit IRC13:19
*** yangyapeng has quit IRC13:20
*** tidwellr has joined #openstack-nova13:23
*** esberglu has joined #openstack-nova13:24
*** sean-k-mooney has quit IRC13:25
openstackgerritMerged openstack/nova master: Enhance doc for nova services  https://review.openstack.org/49953613:25
*** evrardjp has joined #openstack-nova13:26
*** gvrangan has quit IRC13:26
evrardjphello guys, sorry for my glitch in your meeting.13:26
alex_xujaypipes: mriedem bauzas, a team in the intel is working on upstream CIT https://github.com/opencit/opencit-openstack-extensions/blob/v3.2/nova/scheduler/asset_tag_filter.py, I think it is a upgrade version of trustedfilter. I know we have some problem with the trustedfilter, performance, depend on external service. I'm thinking is it ok using traits API instead of the filter implementation, is a13:27
alex_xuright direction?13:27
bauzasalex_xu: I saw your proposal for the PTG13:27
bauzasalex_xu: MHO is that I think we don't need to see a TrustedFilter13:27
evrardjpI was in the wrong chan.13:27
bauzasI mean, in the tree13:27
alex_xubauzas: yea, the team reaches to me recently, try to figure out the things13:27
alex_xubauzas: why13:28
bauzasalex_xu: but in case operators or deployers want to use a custom scheduler filter for that, I'm totally fine13:28
openstackgerritMerged openstack/nova master: Fix ValueError if invalid max_rows passed to db purge  https://review.openstack.org/50077113:28
bauzasalex_xu: for the exact reason you mentioned13:28
bauzasreasons* even13:28
openstackgerritMerged openstack/nova master: Mark LXC as missing for swap volume support  https://review.openstack.org/48221613:29
alex_xubauzas: ok, is it ok for external agent report trusted info as traits to the placement service directly? it sounds like out of nova totatly also13:29
gmann__mriedem: query schema BP is good without microversion. i updated info in BP  - https://blueprints.launchpad.net/nova/+spec/json-schema-validation-for-query-param13:29
*** catintheroof has joined #openstack-nova13:29
bauzasalex_xu: also, the TrustedFilter is deprecated and will be removed by Queens13:29
openstackgerritMerged openstack/nova master: explain payload inheritance in notification devref  https://review.openstack.org/45366713:30
bauzasalex_xu: we can surely discuss that in the PTG13:30
*** thegreenhundred has joined #openstack-nova13:30
alex_xubauzas: yea, i know13:30
alex_xubauzas: yea, that will be great, try to figure out a way we liked13:30
bauzasalex_xu: if things need to be proposed to Nova for helping operators to have that feature, that's fine by me13:30
*** annegentle has joined #openstack-nova13:31
alex_xubauzas: ok, probably I need to talk with that team what is the key thing needs to integrate with nova13:31
bauzasalex_xu: I just want to make sure we don't create an in-tree module that would need a 3rd party system that is not tested in the gate :)13:31
jaypipesalex_xu: who is using this openCIT technology?13:31
bauzasand again, removing TrustedFilter from upstream doesn't mean operators or deployers could not use it13:32
alex_xubauzas: ok :)13:32
alex_xujaypipes: I don't know :)13:32
bauzaswell, I know that :)13:32
bauzasIntel customers, I'd say13:32
alex_xubauzas: better answer than me :)13:32
bauzashonestly, the OAT feature is nice13:33
bauzasthat's how it's implemented in Nova that is bad to me13:33
bauzass/bad/wrong13:33
jaypipesbauzas: OAT?13:34
alex_xubauzas: yea, so I'm thinking we have placement API, we can have something report to the placement API direclty out of Nova13:34
bauzasjaypipes: Open Attestation Service or something like that13:34
*** tbachman has quit IRC13:35
bauzasjaypipes: that's basically some agent running on computes that verify whether the compute kernel is not modified13:35
bauzass/modified/tainted I'd say13:35
jaypipesalex_xu: sure, you can do that. but then aren't we opening up the "trust" attack vector to anyone who can write a trait for a resource provider? ;)13:35
*** gmann__ has quit IRC13:35
bauzasjaypipes: that necessarly supposes that the Placement service can't be tainted13:36
jaypipesbauzas: zactly ;)13:36
bauzasthat's an operator problem honestly :)13:36
*** burt has joined #openstack-nova13:36
bauzasif they want to go that direction, fine by me13:36
alex_xujaypipes: so you mean the only right way is access the OAT server in each scheduling...13:37
bauzasjaypipes: OpenAttestation13:37
bauzasoop13:37
bauzasjaypipes: https://wiki.openstack.org/wiki/OpenAttestation13:37
*** gbarros has joined #openstack-nova13:38
bauzasoh interesting13:39
bauzasOAT has been superseded by OpenCIT13:39
mriedemgmann: ok approved13:39
mriedemthanks for following up13:39
mriedembauzas: jaypipes: i'm +2 on dan's migration allocations spec if one of you wants to take a gander https://review.openstack.org/#/c/498510/13:40
bauzasAFAICS, instead of querying a manifest, the compute agent sends a report to the attestation server13:40
bauzasmriedem: for sure13:40
jaypipesalex_xu: no, I'm saying that the "trust" part of all of this stuff would be dependent on how much you trust the caller to the placement API that is setting traits. Obviously, the Placement API doesn't verify callers or attest to their content.13:40
bauzasmy review bag is pretty empty those days13:40
*** ssmith has joined #openstack-nova13:40
bauzasjaypipes: well, the placement API accepts credentials, so the real problem is whether they accept Keystone tokens as safe enough or not13:41
*** jaosorior has quit IRC13:41
openstackgerritStephen Finucane proposed openstack/nova master: docs: Rename cellsv2_layout -> cellsv2-layout  https://review.openstack.org/49882113:41
openstackgerritStephen Finucane proposed openstack/nova master: WIP! doc: Add contents page  https://review.openstack.org/49882013:41
openstackgerritStephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages  https://review.openstack.org/49881913:41
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add configuration index page  https://review.openstack.org/49881813:41
jaypipesbauzas: right...13:41
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add user index page  https://review.openstack.org/49881713:41
bauzasjaypipes: but again, it's not my problem :)13:41
stephenfinralonsoh: You about. Question about binding profiles13:41
jaypipesdamn you stephenfin :) was just about to +W the bottom of that.13:41
mriedembauzas: if your review bag is empty, this bug fix and the changes below it are needed for pike https://review.openstack.org/#/c/499878/13:42
mriedemonce that's done and backports are merged i'll cut a release13:42
*** hrw has quit IRC13:43
alex_xujaypipes: yea, i see that, so we don't have that problem for the existed trusted filter13:43
stephenfinjaypipes: You still can - it was a rebase to move the dodgy contents patch to the end13:43
bauzasmriedem: I'm already on https://review.openstack.org/#/q/topic:bp/request-spec-use-by-compute13:43
stephenfin:)13:43
bauzasjaypipes: mriedem: dansmith: I'm gonna ask a question for https://review.openstack.org/#/c/498510/5/specs/queens/approved/migration-allocations.rst13:43
mriedembauzas: cool - gerrit somehow auto-changed my topic branch which was annoying there13:43
*** alexchadin has quit IRC13:43
*** wlfightup has quit IRC13:43
*** wlfightup has joined #openstack-nova13:43
*** zxy has quit IRC13:43
*** yangyapeng has joined #openstack-nova13:43
*** zxy has joined #openstack-nova13:44
bauzasjaypipes: mriedem: dansmith: if we consider that allocations need to be set/corrected by the conductor for migrations (and we already do that), could we consider having a single boot request to have the simple allocation claim to be also done in the conductor ?13:44
bauzasI know it has been a long conversation previously13:44
*** hrw has joined #openstack-nova13:44
bauzasand I don't want to open wounds13:45
bauzasbut I just want to consider how that could help us having a single driver using Placement13:45
*** cleong has joined #openstack-nova13:45
alex_xujaypipes: is it another layer problem, since you can attach traits to the resource provider, basically you already get the admin user in the system...13:45
*** alexchadin has joined #openstack-nova13:45
jaypipesalex_xu: I'm not saying it's a problem, per se. I'm just saying that the OpenCTI team should be aware of the fact that communication with the placement API itself is not attested.13:46
openstackgerritStephen Finucane proposed openstack/nova master: Make eventlet hub use a monotonic clock  https://review.openstack.org/43432713:46
alex_xujaypipes: yea, i see, so let me tell the opencit team...13:47
bauzasjaypipes: alex_xu: tbh, the current implementation of the TrustedFilter is already considering that the communication to the OpenCIT system is not having a man in the middle13:47
bauzasor that what is passed by the OpenCIT server is correct13:47
*** yangyapeng has quit IRC13:48
bauzasthat's just adding a proxy, but the design is the same13:48
jaypipesbauzas: k. but this is opening up another communication channel. that's all I was saying.13:48
bauzasjaypipes: sure, I'm just trying to explain that given the current implementation, those folks don't see that as a problem already :p13:48
jaypipesbauzas: cool.13:48
*** hrw has quit IRC13:49
alex_xubauzas: yea13:49
*** crushil has quit IRC13:50
mriedembauzas: the only reason conductor is setting/correcting allocations during a move operation is because of bugs13:50
*** liverpooler has quit IRC13:50
bauzasI know :(13:50
mriedembauzas: well, or failures in certain cases, like live migration pre-check failing after we've already claimed on the dest host in the scheduler13:50
mriedembut the force cases are bugs,13:50
*** hrw has joined #openstack-nova13:51
bauzasbut since the scheduler doesn't know whether it's a move or a boot, we need to do something like that13:51
mriedemand i plan on having the force scenarios still call the scheduler but with a skip_filters flag13:51
mriedemthe scheduler, since pike, determines if it's a move by checking for existing allocations on another node13:51
mriedemand doubles those up13:51
bauzashonestly, dansmith proposed to have the scheduler code to be run by the conductor service, and I just think all of this tends to that13:51
mriedemthe issue with force is that we bypass the scheduler altogether13:51
bauzasyup13:52
mriedemso are you asking if we should stop doing the claim in the scheduler and move that to conductor now in queens?13:52
*** liverpooler has joined #openstack-nova13:52
bauzasmriedem: yep, I'm wondering if we should reconsider the opportunity to have claims done by the conductor for the reasons you mentioned13:53
bauzas#1 if you force, then you need to reconcile claims13:53
mriedemi think we can still do force and have the scheduler handle the claim13:53
mriedemi'm going to write a bp for that13:53
mriedemsince it's going to be an rpc change and require some consideration13:53
bauzas#2 for move operations, there are a list of corner cases depending on the success of the move that require the conductor to reconcile allocations as well13:53
mriedemsure, but for #2 we can also fail on the compute and need to cleanup allocations,13:54
bauzasgood point13:54
mriedemwhich we're seeing with prep_resize failing, evacuate moveclaim failing, and unshelve claim failing13:54
mriedemi do'nt really see this any different from cleaning up ports and volume attachments13:54
mriedemwhich happen both in conductor sometimes and in the compute13:54
bauzasabout the force thingy, what if placement returns "Sorry"' to the scheduler if it claims ?13:54
*** hongbin has joined #openstack-nova13:55
mriedemthen we fail13:55
bauzasI'm not sure operators would accept that13:55
mriedemthey are going to have to13:55
bauzasLOL13:55
mriedemif you force to a compute today, the claim on the compute could still fail13:55
mriedemexcept live migration doesn't do a claim13:56
mriedembut evacuate does13:56
bauzasthat's more complicated :13:56
mriedemmaybe those claims never fail because we pass an empty limits dict13:56
mriedemso the claim just considers unlimited cpu/ram/disk13:56
bauzassince we don't call the scheduler, the scheduler isn't passing limits to the conductor which eventually gives them to the compute13:56
bauzasso we don't really verify the resource usage13:56
mriedemyeah i just said the same thing13:57
mriedemhonestly i didn't realize that's how things worked until last week when writing some functional tests for this13:57
bauzasyou typed too fast, damn you13:57
jaypipesmriedem, bauzas, alex_xu: https://twitter.com/jaypipes/status/90542957101261619213:57
*** moshele has quit IRC13:57
mriedemjaypipes: i can't tell if they look freaked out like that because of the storm or if that's just normal13:57
bauzasjaypipes: orly? :( you're going to be impacted ? :(13:57
jaypipesbauzas: yeah. we're probably going to evacuate tonight or tomorrow morning.13:58
bauzasjaypipes: le woof is hugging them13:58
jaypipesmriedem: just normal.13:58
jaypipesheh13:58
mriedembauzas: point is, we need the resource allocation representation in placement to be accurate,13:58
mriedemand since the RT isn't adjusting claims anymore,13:58
mriedemafter the fact i mean,13:58
mriedemwe can't just not create the allocations,13:58
*** crushil has joined #openstack-nova13:58
mriedemwith the force stuff before with claims, the RT would still report the usage on the compute node to the scheduler and if it was full then that compute node is out of the running for placement decisions13:59
mriedemif we don't report the allocations to placement, then things are incorrect for the scheduler13:59
mriedemregardless of force13:59
mriedemplus, as i said in the ML, i think force is a bad idea anyway14:00
mriedemi'm semi ok with continuing to ignore the filters (even though the live migration force scenario still checks ram and compute filters, but in conductor code)14:00
mriedembut i'm not ok with bypassing placement14:00
mriedembauzas: so maybe what you want to see in https://review.openstack.org/#/c/499399/ is a release note that says, evacuate with force=True may fail due to placement14:01
mriedemand at some point an update to the api-ref for the force flag on evacuate and live migrate14:02
mriedemi.e. the force flag bypasses the scheduler filters but can still fail due to placement14:02
*** Tom_ has joined #openstack-nova14:03
*** Tom_ has quit IRC14:03
mriedemwe merged the same for live migrate for the pike GA but didn't have a release note for that https://review.openstack.org/#/c/496727/14:03
mriedemi could add a release note for the 16.0.1 release14:04
bauzasmriedem: so, I agree with you about having "something" that would reconcile allocations if you force14:04
mriedemsince the RT isn't going to do that once all computes are upgraded to pike,14:05
bauzasthe fact is, some operators for example want to live-migrate a whole host to another one with the price of degradation14:05
dansmithwe can audit without heal14:05
mriedemit's better to try and allocate in the controller and fail fast than do it in the compute after we've already live migrated or evacuated something14:05
*** mamandle has joined #openstack-nova14:05
bauzasbecause that's an emergency situation14:05
bauzasand they don't want their customers to know that the host is bad14:05
mriedemif they are live migrating, the customer doesn't know14:06
mriedemif it moved or not14:06
dansmithwell, the customer will know of course14:06
bauzassure, but they want to migrate sooner than later and somehow want to migrate the host to some other they know14:06
dansmithminimum disruption, but that's all14:06
bauzasagreed14:07
mriedembtw, unrelated, but can we get this merged https://review.openstack.org/#/c/500968/ to unblock the requirements upper-constraints change from merging?14:07
mriedem^ is needed to get nova unit tests to pass with the latest os-xenapi lib14:07
dansmithI'm not sure I understand what the concern is, I just started reading14:07
dansmithall we have to do is make sure force does a caim first, and if that fails, fail, right?14:07
mriedemdansmith: which is exactly what this does https://review.openstack.org/#/c/496727/14:08
mriedemand https://review.openstack.org/#/c/499399/14:08
dansmithright14:08
mriedemwe're really talking about two things and diverged from the first, which is bauzas wants to consider moving the claim code out of the scheduler and into conductor14:09
mriedemand then we went down the force flag rathole14:09
alex_xujamespage: bauzas there also a proposal in the intel about integrate an agent which managing cpu l3 cache into the nova. I'm also thinking report the l3 cache inventory by the agent to the placement directly is option, but there missed a way for the agent to know new instance boot up in the node except polling the libvirt14:09
*** jaosorior_ has joined #openstack-nova14:09
mriedemalex_xu: listen for notifications14:09
alex_xujaypipes: ^14:09
alex_xujamespage: sorry for pointed to wrong person14:09
alex_xumriedem: but the notification isn't for single host, we must filter the info14:10
mriedemalex_xu: if you want a hypervisor agnostic solution, listen for notifications rather than polling libvirt directly14:10
jaypipesalex_xu: you mean RDT/CAT, right?14:10
alex_xujaypipes: yea14:10
*** armax has joined #openstack-nova14:10
mriedemalex_xu: the instance.create.end payload would have the host/node information in it so yeah you'd have to filter but that doesn't seem hard14:11
bauzasI don't get why it requires libvirt to be polled14:11
bauzasand I second mriedem about subscribing to the notifications bus14:12
kashyapbauzas: What requires libvirt to be polled?  /me read it out of context; will read the scrollback14:12
*** Tom_ has joined #openstack-nova14:12
alex_xumriedem: yea, that won't be so hard, that probably each node have a agent to listen the notification14:12
mriedembauzas: xenserver ci isn't going to care about unit test changes https://review.openstack.org/#/c/500968/14:12
bauzasholy ship, you're right14:12
alex_xubauzas: kashyap an external agent try to know there is new instance boot up14:13
mriedemalex_xu: plus then you can use the fancy new versioned notifications and provide feedback to gibi14:13
alex_xumriedem: yeah14:13
gibiyeah :)14:14
kashyapalex_xu: Ah, it's the L3 CAT interface.  I see some in-progress design work of exposing it in libvirt upstream14:14
alex_xukashyap: yea, looks like libvirt interface is slow14:14
kashyapalex_xu: Yeah, there's a new thread that started a day ago14:14
kashyapDid you see that?  About the new RFC / design14:14
*** ekuris has quit IRC14:15
alex_xukashyap: not yet, another team member follow that, probably I will tell him14:15
*** yamahata has joined #openstack-nova14:15
*** priteau has joined #openstack-nova14:16
mdboothmelwitt: You about? Spotted something potentially funky looking at your change https://review.openstack.org/#/c/498983/14:16
mdboothNot directly related to your patch14:16
*** yamamoto has quit IRC14:17
alex_xukashyap: thanks for the info14:17
mdboothI couldn't see where we wrap swap_volume in any kind of lock14:17
*** takedakn has quit IRC14:17
kashyapalex_xu: Here it is: "Yet another RFC for [Intel] CAT" - https://www.redhat.com/archives/libvir-list/2017-September/msg00072.html14:17
mdboothe.g. by setting task_state14:17
mdboothI expect I just missed it, though.14:17
*** ociuhandu has quit IRC14:17
*** yangyapeng has joined #openstack-nova14:18
mdboothDo you happen to know if there is any locking around that method?14:18
*** tuanla_fujitsu has joined #openstack-nova14:18
*** tuanla__ has joined #openstack-nova14:18
mdboothIf there isn't, the save/restore xml after a long-running rebase mechanism would be potentially pretty bad.14:19
*** smatzek has joined #openstack-nova14:19
kashyapmdbooth: I read your comment this morning, and did find your task exclusion observation interesting14:19
gibimriedem: could you confirm that we want to delete the allocation of the instance during _destroy_evacuated_instances if the instance failed an evacuation?14:19
gibimriedem: context is in https://review.openstack.org/#/c/499237/14:19
kashyapHmm, although the patch is a strict improvement as-is, now I wonder about your comment - where is the bug lurking...14:19
dansmithmriedem: I'm going to be in an airport when the cells meeting is going on14:19
dansmithmriedem: I can try to run it from there, but no promises14:19
alex_xukashyap: thanks!14:20
mriedemdansmith: let's just skip14:20
*** priteau has quit IRC14:20
*** priteau_ has joined #openstack-nova14:20
dansmithmriedem: cool14:20
gibimriedem: I'm asking because then a subsequent rebuild needs to allocate resources again14:20
*** annegentle has quit IRC14:21
*** priteau_ has quit IRC14:21
*** yangyapeng has quit IRC14:22
*** priteau has joined #openstack-nova14:22
mriedemgibi: hmm, and a rebuild to the same host won't do a claim, and the RT won't auto-heal the allocations on the node14:22
mriedemand rebuild (not evacuate) bypasses the scheduler14:23
mriedemthat's a good point14:23
*** eharney has joined #openstack-nova14:23
gibimriedem: yes, that is my fear14:23
gibimriedem: in the other hand we could put the instance to error state after the failed evac and keep the allocation14:24
gibimriedem: this way the rebuild doesn't need to allocate14:24
mriedemi'm in the middle of something and will have to digest the replies in that review later14:24
mriedemgibi: we might also just have to leave this until the ptg14:24
gibimriedem: sure14:24
mriedemcould you add it to the ptg etherpad in case it has to wait until next week?14:24
*** hogepodge has joined #openstack-nova14:24
gibimriedem: adding...14:25
hogepodgemriedem: initial schedule is up, let me know if times work for you, leave notes if things need to be shuffled. It's all preliminary right now and subject to change. https://etherpad.openstack.org/p/InteropDenver2017PTG14:26
*** Dave has quit IRC14:27
mriedemhogepodge: ok thanks14:27
*** awaugama has joined #openstack-nova14:30
*** links has quit IRC14:31
gibimriedem: I added it to the etherpad14:32
*** hrw has quit IRC14:34
*** gbarros has quit IRC14:35
*** hrw has joined #openstack-nova14:35
mriedemgibi: thanks14:36
hogepodgedtantsur|afk: ^^14:40
openstackgerritHongbin Lu proposed openstack/nova master: Handle exception on adding secgroup  https://review.openstack.org/46517314:41
openstackgerritMerged openstack/nova master: rbd: Remove unnecessary 'encode' calls  https://review.openstack.org/41235614:41
stephenfinHurrah ^14:42
*** Dave has joined #openstack-nova14:43
*** hrw has quit IRC14:43
openstackgerritMerged openstack/nova master: Remove qpid description in doc  https://review.openstack.org/49908714:44
*** annegentle has joined #openstack-nova14:44
*** gbarros has joined #openstack-nova14:45
*** tbachman has joined #openstack-nova14:46
*** hrw has joined #openstack-nova14:46
*** mdrabe has joined #openstack-nova14:48
*** alexchadin has quit IRC14:51
*** slaweq has joined #openstack-nova14:53
dansmithmriedem: dtantsur|afk: see this ironic migration thing? https://review.openstack.org/#/c/501025/214:54
*** kairo has quit IRC14:54
*** kairo has joined #openstack-nova14:54
mriedemnow i do14:56
esbergluI'm getting the following unauthorized command error from nova-rootwrap intermittently14:58
esbergluhttp://paste.openstack.org/show/620547/14:58
mriedemsdague: looks like https://review.openstack.org/#/c/457636/ is happy - the devstack patch to install the osc-placement plugin14:58
mriedemhttp://logs.openstack.org/36/457636/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ad2b234/logs/pip2-freeze.txt.gz14:59
esbergluIn /etc/nova/rootwrap.d/compute.filters I have the following line14:59
esberglutee: CommandFilter, tee, root14:59
esbergluAnyone have any idea why sometimes that filter is failing to match?14:59
*** marst has joined #openstack-nova15:01
*** efried_zzz is now known as efried15:01
*** kairo has quit IRC15:02
*** kairo has joined #openstack-nova15:02
*** andreas_s_ has joined #openstack-nova15:02
sdaguemriedem: yep15:03
*** rabel has quit IRC15:03
*** tbachman has quit IRC15:03
*** felipemonteiro has joined #openstack-nova15:03
*** felipemonteiro_ has joined #openstack-nova15:05
*** andreas_s has quit IRC15:05
stephenfinsdague: Is the RXTX factor flavor key still applicable with a neutron backend?15:07
* stephenfin thinks it was you discussing it a while back15:07
sdaguestephenfin: I thought it was nova net (and maybe xen) only. But I don't know.15:08
mriedemhmm, did we ever say in the pike release notes that conductor now needs it's nova.conf to have the [placement] section filled in?15:08
*** felipemonteiro has quit IRC15:08
mriedemstephenfin: no, i plan on deprecating that out of the api15:08
mriedemjust need to write the spec15:08
mriedemand yes, it's really only nova net + xen15:09
stephenfinmriedem: Excellent. I'm going to put that in the docs and mark bug 1688054 as invalid15:09
openstackbug 1688054 in openstack-manuals "Flavors in Administrator Guide - confusing description for rxtx factor" [Medium,Confirmed] https://launchpad.net/bugs/168805415:09
mriedemstephenfin: that doesn't make the bug invalid15:09
mriedemif the docs are wrong for how it's used today15:09
sdagueesberglu: where in the code is that called?15:09
*** kairo has quit IRC15:09
stephenfinYeah, that's a good point. OK, I'll clarify the intent a little better so15:10
stephenfin...and join the flavor/flavor2 files15:10
stephenfin#RefactoringFTW15:10
*** david-lyle has quit IRC15:10
*** itlinux has quit IRC15:11
efriedesberglu IT or OOT?  SDE or traditional?  Is it a snapshot operation that's being tested?15:13
openstackgerritBalazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase  https://review.openstack.org/49953915:13
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete  https://review.openstack.org/49515915:13
*** david-lyle has joined #openstack-nova15:15
efriedOkay, I think I answered my own questions esberglu15:16
*** yamamoto has joined #openstack-nova15:17
efriedsdague: This is being done as part of the snapshot operation in our OOT driver: https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/mgmt.py#L97-L10015:17
*** tbachman has joined #openstack-nova15:18
*** chyka has joined #openstack-nova15:19
esbergluSorry got pulled away for a few15:19
esbergluefried: Yeah this is OOT snaphot15:19
efriedesberglu And presumably this is causing the test to fail.15:20
esbergluefried: Yep15:20
efriedBut the _tee_as_root doesn't raise, right?  We just fail to find the disk in the next part of the code?15:20
*** chyka has quit IRC15:21
*** chyka has joined #openstack-nova15:21
esbergluefried: The nova-rootwrap command raises when trying to execute that tee command15:24
efriedesberglu Oh.  Boo.15:24
*** gmann has quit IRC15:24
esbergluefried: https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/mgmt.py#L5815:24
*** gmann has joined #openstack-nova15:24
esbergluIt calls that execute which goes into the rootwrap filters and should match the tee line15:24
esbergluBut sometimes it isn't matching that tee filter15:25
efriedWhich rootwrap filter?15:25
*** yamamoto has quit IRC15:25
esberglutee: CommandFilter, tee, root15:25
esbergluin /etc/nova/rootwrap.d/compute.filters15:26
*** suresh12 has joined #openstack-nova15:26
esbergluThat line should allow tee to be run as root with any parameters15:26
efriedesberglu Sorry, I mean are we using RootwrapDaemonHelper or RootwrapProcessHelper?15:27
*** priteau_ has joined #openstack-nova15:29
*** priteau has quit IRC15:29
*** gszasz has quit IRC15:29
openstackgerritMatt Riedemann proposed openstack/nova master: Add release note for force live migration allocations  https://review.openstack.org/50131415:30
mriedembauzas: here you go ^15:30
bauzasmriedem: roger. just in a meeting now15:30
*** gszasz has joined #openstack-nova15:31
*** priteau has joined #openstack-nova15:31
openstackgerritMatt Riedemann proposed openstack/nova master: Add release note for force live migration allocations  https://review.openstack.org/50131415:31
efriedesberglu Can you try setting `use_rootwrap_daemon` to True in the compute conf and see if the problem magically disappears?15:31
efriedesberglu That's in the [DEFAULT] section15:31
esbergluefried: Yeah that isn't set currently. I can give it a try15:32
efriedesberglu Though if non-daemon is busted, that seems like it oughtta be a bug to nova.15:32
*** tbachman has quit IRC15:33
openstackgerritMerged openstack/nova master: Trim the fat from InstanceInfo  https://review.openstack.org/47114615:34
sdagueefried: yeh, though the real fix would be to get rid of it entirely for privsep15:34
*** priteau_ has quit IRC15:34
sdaguebe aware, that mikal's patch series is doing that15:34
efriedsdague "it" which?15:34
efriedThe Process version?15:34
sdaguehttps://review.openstack.org/#/c/48943815:35
sdaguethat's 5 patches in15:35
*** andreas_s_ has quit IRC15:35
sdagueand it will probably take some time to land, but rootwrap for tee is removed at that point15:35
sdagueefried: the tee call at all15:36
*** priteau has quit IRC15:36
efriedsdague Mm.  And the resolution is to decorate my function with this privsep gizmo and then do a regular ol `with open(): write()` ?15:37
sdagueyep15:37
sdaguebut his first patch has to land before that will work15:37
*** felipemonteiro__ has joined #openstack-nova15:38
efriedesberglu ^^ FYI.  Need to add that to the watchlist so we can react when it lands.15:38
esbergluefried: Will do15:39
*** tbachman has joined #openstack-nova15:40
efriedesberglu It would seem we could transition as soon as https://review.openstack.org/#/c/459166/ is in.15:40
efriedBut in the meantime, <shrug> let's see if using the rootwrap daemon helps.15:40
*** priteau has joined #openstack-nova15:40
*** felipemonteiro_ has quit IRC15:41
*** priteau has quit IRC15:41
*** priteau has joined #openstack-nova15:42
*** felipemonteiro_ has joined #openstack-nova15:43
*** burt has quit IRC15:43
*** tbachman_ has joined #openstack-nova15:43
stephenfinjaypipes: https://review.openstack.org/#/c/498817/ and friends have their gate +1 now, if you were idle enough :)15:44
*** tbachman has quit IRC15:44
*** tbachman_ is now known as tbachman15:44
*** burt has joined #openstack-nova15:46
*** tidwellr has quit IRC15:46
*** priteau has quit IRC15:46
*** felipemonteiro__ has quit IRC15:46
*** priteau has joined #openstack-nova15:46
*** priteau_ has joined #openstack-nova15:47
*** priteau_ has quit IRC15:48
*** priteau__ has joined #openstack-nova15:50
*** moshele has joined #openstack-nova15:50
*** priteau has quit IRC15:51
*** tylerderosagrund has joined #openstack-nova15:52
*** gyee has joined #openstack-nova15:55
dansmithmriedem: yeah I had marked that as -W ahead of me writing the release note, but then I wiped it when I pushed a little fix15:56
mriedemi don't know how the tests are even passing15:57
*** moshele has quit IRC15:57
dansmithmriedem: because you missed that there is a patch before this?15:57
mriedemyes15:57
*** claudiub has joined #openstack-nova15:58
*** tuanla_fujitsu has quit IRC15:59
*** tuanla__ has quit IRC15:59
esbergluefried: ^ 459166 just got W+116:00
*** lucasxu has quit IRC16:00
*** lpetrut_ has joined #openstack-nova16:00
esbergluefried: Oh wait nvm looking at the wrong thing16:00
efriedesberglu Ya know, we could patch that guy in and make our own patch that uses it...16:01
*** claudiub|2 has quit IRC16:01
*** lpetrut has quit IRC16:01
*** lpetrut__ has joined #openstack-nova16:03
*** suresh12 has quit IRC16:04
*** david-lyle has quit IRC16:04
*** dklyle has joined #openstack-nova16:04
openstackgerritMerged openstack/nova master: Remove unneeded attributes from context  https://review.openstack.org/49861516:05
*** lpetrut_ has quit IRC16:05
openstackgerritMerged openstack/nova master: Pass config object to oslo_reports  https://review.openstack.org/48557516:06
*** cdent has joined #openstack-nova16:06
openstackgerritMerged openstack/nova master: Enhance support matrix document  https://review.openstack.org/48202016:07
*** lucasagomes is now known as lucas-afk16:07
openstackgerritMerged openstack/nova master: trivial: Remove some single use function from utils  https://review.openstack.org/49151316:08
*** gbarros has quit IRC16:08
*** gbarros has joined #openstack-nova16:10
*** yamahata has quit IRC16:11
*** itlinux has joined #openstack-nova16:14
*** dklyle has quit IRC16:15
*** dtantsur|afk is now known as dtantsur16:16
*** dave-mccowan has quit IRC16:17
*** david-lyle has joined #openstack-nova16:18
openstackgerritStephen Finucane proposed openstack/nova master: doc: Split flavors docs into admin and user guides  https://review.openstack.org/50134216:18
dtantsurdansmith: thanks, I'll check it tomorrow - day full of errands today :(16:20
openstackgerritDan Smith proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations  https://review.openstack.org/50102516:20
openstackgerritDan Smith proposed openstack/nova master: Split out the core of the ironic flavor migration  https://review.openstack.org/50102416:20
openstackgerritDan Smith proposed openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type()  https://review.openstack.org/50134316:20
dansmithdtantsur: ack, np16:20
*** Tom_ has quit IRC16:21
*** krtaylor has quit IRC16:21
dtantsurcdent: any specific schedule plans for API SIG PTG?16:21
cdentdansmith: could you give me some early feedback on whether you think this is the right approach: https://review.openstack.org/#/c/501051/ (it’s related to post /allocations)16:21
dtantsurcdent: I'm figuring out conflicts with https://etherpad.openstack.org/p/InteropDenver2017PTG16:21
*** shan has joined #openstack-nova16:21
dimsjaypipes : when is the hurricane expected?16:22
dansmithcdent: yeah I know I need to do that, sorry16:22
dansmithdims: weekend I think16:22
dansmithdims: meanwhile portland is on fire and florida refuses to share the water: the16:22
openstackgerritStephen Finucane proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0  https://review.openstack.org/48666016:23
cdentdtantsur: edleafe and I talked about that yesterday and we’re going to try to cook a schedule, but as there aren’t specific topics (nobody signed up for guided reviews) that are really on the agenda, it is mostly going to be workshopping guideline improvements and whatever comes up16:23
dansmithhttp://www.oregonlive.com/wildfires/index.ssf/2017/09/eagle_creek_fire_rages_in_colu.html16:23
cdentno worries dansmith16:23
dimsdansmith : y i heard :)16:23
dtantsurcdent: surprisingly to me, the guided review idea has not got too much traction with other ironic team members..16:23
dimsdansmith : how far is it away from you?16:23
dansmithdims: not far enough, but not currently a risk for us16:24
dansmithdims: but it's crazy smoky outside and ash is falling on everything16:24
*** tidwellr has joined #openstack-nova16:25
dansmithwalking outside is like getting a face full of camp fire16:25
dimsdansmith : dear lord16:25
dansmithmeanwhile, florida is all like "oh water, what are we going to do with all this water? did we mention we have a lot of water on the way?"16:25
*** gvrangan has joined #openstack-nova16:25
cdentthat’s just rude16:25
dansmithikr16:25
mnaserahem im trying to troubleshoot (what i think is a nova bug).. any idea why instances can be seen in the list, but viewing them returns a 404?16:26
mnaserex: nova list shows them, nova show <uuid> says no server matching :X16:26
dansmithmnaser: instance mapping missing?16:26
dimsdansmith : jaypipes : hopefully you will both make it to ptg16:26
dansmithdims: thanks :)16:26
*** gszasz has quit IRC16:27
dansmithmnaser: sounds like cell mappings are in place, so it can iterate and list, but no instance mapping for when it needs to go from instance uuid -> cell16:27
dansmithmnaser: look in api database, instance_mappings table, see if there is one for the instance you're trying16:27
*** rajathagasthya has joined #openstack-nova16:27
mnaseri wonder why the mapping didnt go in then (customer says they recently failed builds)16:27
dansmithmnaser: idk, should get an instance mapping once you're scheduled (or map you to cell0 if you fail) so we'd need to know what happened exactly to figure it out16:28
mnaser /var/log/nova/nova-conductor.log:2017-09-06 10:42:20.512 829 WARNING nova.scheduler.utils [req-dd7ba87f-8539-4642-b238-3d1e4e4a686c 8873fec3d88b45c09af91ac942917650 eb60ffb2745a40e9b76223e5481e7c5e - default default] [instance: 90ecb4cc-8ea9-4419-9a3d-f67c9436df82] Setting instance to ERROR state.: TooManyInstances: Quota exceeded for ram: Requested 32768, but already used 716800 of 716800 ram16:28
mnaseri wonder if it fails there meaning it doesn't end up getting scheduled?16:28
dansmithmnaser: yeah so that should map you to cell016:28
mnasertime to check if it did16:29
dansmithmnaser: well, it seems like it's not mapped, but after confirming that, see if it's in the cell0 database16:29
*** Apoorva has joined #openstack-nova16:29
mnaserMariaDB [nova_cell0]> select * from instances where id='90ecb4cc-8ea9-4419-9a3d-f67c9436df82'; => empty set16:29
dansmithmnaser: uuid16:30
mnaseroh duh16:30
mnaserempty16:30
*** felipemonteiro__ has joined #openstack-nova16:30
dansmithcheck the other?16:30
mnaserselect * from instance_mappings where instance_uuid='90ecb4cc-8ea9-4419-9a3d-f67c9436df82'; => cell_id is null16:30
*** itlinux_ has joined #openstack-nova16:31
mnaseri can see it in my default clel db16:31
mnaser(one that was created upgrading to ocata)16:31
mnaserand the only one other than cell016:32
dansmithokay that's weird16:32
dansmithmnaser: can you file a bug with those details?16:32
*** itlinux has quit IRC16:32
*** licanwei has quit IRC16:32
mnaserdansmith will do and ill do a bit more research too16:32
mnasermaybe it happens when you hit quotas16:32
dansmithmnaser: also, you should be able to do map_instances on your non-cell0 cell and fix that mapping I think16:32
*** felipemonteiro__ has quit IRC16:33
dansmithalthough you might have to nuke the instance mapping row first, which is also a bug for maintainabiity16:33
dansmithmnaser: thanks16:33
*** felipemonteiro__ has joined #openstack-nova16:33
*** annegentle has quit IRC16:33
*** felipemonteiro_ has quit IRC16:34
openstackgerritMerged openstack/nova master: Glance download: only fsync files  https://review.openstack.org/45109416:36
openstackgerritMerged openstack/nova master: spelling mistake  https://review.openstack.org/50121516:36
mnaserdansmith found the bug im pretty sure16:36
mnaserhttps://github.com/openstack/nova/blob/a6c429bd7916211aa04d96caa54443ad92cb99ff/nova/conductor/manager.py#L992-L100616:37
mnaserno bury in there16:37
mnaserunlike the other failure cases16:37
mnaseri guess i can work around it now by setting quota/recheck_quota=false for now.. but i think that's the failure case16:37
*** dave-mccowan has joined #openstack-nova16:38
*** slaweq has quit IRC16:39
*** gszasz has joined #openstack-nova16:39
*** tidwellr has quit IRC16:43
*** jpena is now known as jpena|off16:43
*** penick has joined #openstack-nova16:44
*** lucasxu has joined #openstack-nova16:47
dansmithmnaser: that's not the bug16:47
dansmithmnaser: at that point we've created it in the one cell, so we can no longer bury it in cell016:47
dansmithmnaser: so the bug is that we need to finish updating the mapping in that case, but apparently we aren't16:47
dansmithmnaser: this is pike or ocata?16:47
mnaserpike16:48
*** mamandle is now known as manasm16:48
*** gszasz has quit IRC16:48
mnaserthis use case would launch like 30 instances at once16:48
dansmithokay16:48
mnasers/user use case/16:48
mnaserand i guess it goes over quota in that big burst16:48
dansmithmnaser: L1042 is the problem16:49
dansmithmeaning, we never run L1042, but we should16:49
*** shan7 has joined #openstack-nova16:49
dansmithin that quota casw16:49
mnaserso technically we should be saving before checking the quota again16:49
manasmmriedem: did you get a chance to look at my comment regarding commit 1b2f3c7bb25667527e7e1d2afd84d5d5533e7751 earlier today?16:49
openstackgerritMerged openstack/nova master: Put base policy rules at first  https://review.openstack.org/50073616:50
* mnaser has to run16:50
mnaserif folks are busy i dont mind picking up the work cause its a bit critical for us16:50
mnaserjust need a little pointing to what has to be done16:50
*** felipemonteiro_ has joined #openstack-nova16:50
*** gbarros has quit IRC16:51
dansmithmnaser: sure, I'll help guide when you're back to work on it. thanks for excellent triage16:51
manasmthe evacuated instance cleanup is failing in delete_allocation_for_evacuated_instance complaining that the compute node key is not present in resource_tracker compute_nodes. It looks like we need to ensure that resources have initialized (via _update_available_resource) before this is called?16:52
*** shan has quit IRC16:53
mriedemmnaser: did you report a bug yet?16:53
*** annegentle has joined #openstack-nova16:53
mriedemmanasm: i didn't that no, but i think gibi has a related bug open16:53
*** shan has joined #openstack-nova16:54
mriedemabout a PUT /allocations with an empty body/16:54
mriedem?16:54
*** felipemonteiro__ has quit IRC16:54
*** shan7 has quit IRC16:54
*** sambetts is now known as sambetts|afk16:54
*** psachin has quit IRC16:55
*** dave-mccowan has quit IRC16:56
manasmmriedem: gibi: can you please help me with defect number so that I can confirm if its the same?16:56
*** dave-mccowan has joined #openstack-nova16:57
*** lpetrut_ has joined #openstack-nova16:59
mriedemmanasm: https://bugs.launchpad.net/nova/+bug/1714924 ?16:59
openstackLaunchpad bug 1714924 in OpenStack Compute (nova) "placement api fails when nova tries to delete resource allocation after failed evacuation" [Medium,Confirmed]16:59
* mriedem eats lunch16:59
*** gbarros has joined #openstack-nova17:00
*** annegentle has quit IRC17:00
*** lpetrut__ has quit IRC17:01
*** derekh has quit IRC17:01
*** sahid has quit IRC17:01
*** rcernin has quit IRC17:02
*** moshele has joined #openstack-nova17:03
openstackgerritMichael Still proposed openstack/nova master: Move nbd commands to privsep.  https://review.openstack.org/50035117:03
openstackgerritMichael Still proposed openstack/nova master: Move lvm handling to privsep.  https://review.openstack.org/49551617:03
openstackgerritMichael Still proposed openstack/nova master: Move xend existence probes to privsep.  https://review.openstack.org/49553817:03
openstackgerritMichael Still proposed openstack/nova master: Move shred to privsep.  https://review.openstack.org/49553717:03
openstackgerritMichael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls  https://review.openstack.org/49442317:03
openstackgerritMichael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more.  https://review.openstack.org/49554217:03
openstackgerritMichael Still proposed openstack/nova master: Move loopback setup and removal to privsep.  https://review.openstack.org/49566417:03
openstackgerritMichael Still proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197217:04
openstackgerritMichael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree()  https://review.openstack.org/49232617:04
openstackgerritMichael Still proposed openstack/nova master: Move the idmapshift binary into privsep.  https://review.openstack.org/49554117:04
openstackgerritMichael Still proposed openstack/nova master: Move ploop commands to privsep.  https://review.openstack.org/49232517:04
openstackgerritMichael Still proposed openstack/nova master: Read from console ptys using privsep.  https://review.openstack.org/48948617:04
openstackgerritMichael Still proposed openstack/nova master: Move kpartx calls to privsep.  https://review.openstack.org/50035417:04
openstackgerritMichael Still proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222917:04
openstackgerritMichael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916617:04
*** annegentle has joined #openstack-nova17:04
openstackgerritMichael Still proposed openstack/nova master: Move execs of touch to privsep.  https://review.openstack.org/48919017:04
openstackgerritMichael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep.  https://review.openstack.org/49073717:04
*** yarokifor has quit IRC17:04
openstackgerritMichael Still proposed openstack/nova master: Move blkid calls to privsep.  https://review.openstack.org/50039817:04
openstackgerritMichael Still proposed openstack/nova master: Move execs of tee to privsep.  https://review.openstack.org/48943817:04
sdagueoh, a wild mikal17:05
*** krtaylor has joined #openstack-nova17:05
mikalsdague: I _think_ I've addressed your concerns on https://review.openstack.org/#/c/459166/33, but your comments are welcome as always17:05
*** jaosorior_ has quit IRC17:05
*** suresh12 has joined #openstack-nova17:06
*** yangyapeng has joined #openstack-nova17:06
*** harlowja has quit IRC17:07
mikalsdague: I know I've broken things later in the series with your requested changes, but I am going through and fixing those up now (it will take a while because of test runs)17:07
*** harlowja has joined #openstack-nova17:07
dansmithsdague: what percentage of the remaining gate resources would you say mikal's set above consumes? :)17:08
*** felipemonteiro_ has quit IRC17:08
sdaguedansmith: meh, I think we've been pegged all day already anyway17:09
sdaguemikal: the patch above that has an easy fix17:09
*** lpetrut_ has quit IRC17:09
*** felipemonteiro_ has joined #openstack-nova17:09
sdaguewhich is just delete the uml section17:09
dansmithheh17:09
*** yangyapeng has quit IRC17:10
sdaguestephenfin: you deleted uml support at some point, right?17:10
*** felipemonteiro__ has joined #openstack-nova17:11
sdaguedansmith: well it looks like zuul went super grumpy anyway and just merge conflicted the whole stack on itself17:12
sdagueso no tests being run17:12
dansmithkarma is a bitch17:12
*** shan has quit IRC17:13
kmallocsdague: I want to talk with you re some limit stuff being stored in keystone before the PTG session if you'll have a moment or two17:13
*** yamamoto has joined #openstack-nova17:13
kmallocI want to make sure I'm not way off in the weeds. we can meet somewhere around the room(s), but just want to make sure i'm not thinking about this side-angled.17:14
*** felipemonteiro_ has quit IRC17:14
sdaguekmalloc: sure, though I think the bigger question is finding someone else to drive the limits/quota work, because I definitely don't have the bw on it any more.17:14
sdaguebut, yet, I'm happy to chat up current thinking17:14
*** moshele has quit IRC17:14
kmallocright. i expect someone will have to help drive it17:15
*** kristian__ has joined #openstack-nova17:15
kmallocbut i know you and I spent a chunk of time going over the details and requirements :)17:15
kmalloci also don't have the bw to drive it17:15
kmalloc=/17:15
sdagueyeh, I would be more inclined to spend time figuring out if there is someone that wants to own it, and then supporting them. I don't think it makes a ton of sense to work on details until that is figured out17:16
*** JoseMello has joined #openstack-nova17:16
manasmmriedem: thanks for bug link, will check and revert.17:17
*** yamamoto has quit IRC17:17
*** dtantsur is now known as dtantsur|afk17:19
openstackgerritErik Berg proposed openstack/nova master: Fix binary name.  https://review.openstack.org/50135917:20
*** Tom_ has joined #openstack-nova17:21
*** manasm has quit IRC17:21
*** moshele has joined #openstack-nova17:21
mikaldansmith: hey, perhaps these could have been reviewed five months ago when they were written then?17:22
dansmithmikal: no I'm pretty sure nobody can look at them until they're more than three months old. isn't that a rule or something?17:23
*** cfriesen has quit IRC17:23
dansmithmikal: I'm poking fun at the sad state of the gate, not your set of course :)17:23
*** lpetrut has joined #openstack-nova17:25
*** tidwellr has joined #openstack-nova17:25
*** Tom_ has quit IRC17:25
*** suresh12 has quit IRC17:26
mikaldansmith: possibly... my plan is to keep growing the series and hold the gate to ransom until you give into my demands.17:26
*** suresh12 has joined #openstack-nova17:26
dansmithmikal: seems like a solid strategy to me17:26
mikaldansmith: I know, but I am hoping to emotionally pressure you into rubber stamping my thing17:26
dansmithhah17:26
mikalAlso, UML WTF?17:26
mikalAlso, said thing caused cburgess a heart attack, so clearly its ready for prime time17:27
*** jamesdenton has joined #openstack-nova17:27
dansmithI run a for-profit public cloud on UML17:28
dansmithstill waiting for customers though17:28
*** baoli has quit IRC17:28
sdaguemikal: I had thought that stephenfin had actually removed that, but maybe not17:28
mikalsdague: or this is a dangling bit?17:28
sdagueI went through git log -S on it, it looks like it was put in place in the before times, for CI testing17:28
mikalsdague: I found some dead code in libvirt.vif as well on this little journey17:29
sdagueI think that uml references are fair to just delete17:29
sdagueI guess we should probably get final sign off from fearless leader on that, maybe I'll push up a blanket patch for it17:29
dansmithI'll bet you a dollar that dear leader thinks UML is a diagram technique17:30
sdagueheh17:30
sdagueyeh, I did have to explain it to folks17:30
mikalWe should deprecate it17:30
mikalBecause that's always fun17:30
*** yamahata has joined #openstack-nova17:31
*** baoli has joined #openstack-nova17:31
*** moshele has quit IRC17:32
*** kristia__ has joined #openstack-nova17:32
*** adisky__ has quit IRC17:32
jaypipesdims: currently the hurricane is projected to hit Miami on Sunday midday17:33
*** shan has joined #openstack-nova17:33
jaypipesdims: but that means Sarasota will be no-fly and my flight leaves Sunday morning.17:34
*** kristian__ has quit IRC17:36
dimsjaypipes : too close ... :( any evac orders?17:37
*** slaweq has joined #openstack-nova17:39
*** cfriesen has joined #openstack-nova17:40
jaypipesdims: not in sarasota, only the keys17:40
dansmiththe keys.. aka "the small sand bars formerly known as the keys"17:42
jaypipesdansmith: right.17:42
*** slaweq has quit IRC17:42
*** slaweq has joined #openstack-nova17:43
jaypipesdansmith: you know... "embers in a fire pit formerly known as Portland"17:43
dansmithjaypipes: yep17:44
dansmithwe're past 20,000 acres now17:44
dansmiththey've found the 15 year old that started it with a firework17:45
dansmithunfortunately they have not yet released his name17:45
cburgessmikal What caused me a heart attack?17:47
mriedemthose fires made it all hazy here in SE MN last weekend17:48
mriedemreally ruined my fun17:48
mikalcburgess: privsep being the wave of the future17:48
cburgessmikal I'm over it. I think you see/agree with most of my concerns and I'll bring them up in the patch train over the next few days as I review it.17:48
mikalcburgess: yeah I know, but I think the restart thing will be hard. We can discuss next week as performance art in front of everyone else.17:49
cburgessUgh... ok17:49
mikalcburgess: also, I ate your carrots. You're welcome.17:49
cburgessBecause thats probably my biggest concern.17:49
cburgessmikal lol ok. Anne might be grumpy at you.17:49
mikalcburgess: I didn't eat the _all_, just some17:50
cburgesscool17:50
mikalcburgess: let's just deny it was me and it will be fine17:50
cburgessmikal lol17:50
*** gbarros has quit IRC17:50
jaypipesdansmith: have they begun building the hangman's platform for said child? or has that burned down as well?17:51
dansmithjaypipes: I've been welding it with fire-proof steel in my garage waiting for the announcement17:51
jaypipesdansmith: in other news, +W'd the migration allocation spec17:51
jaypipesdansmith: well played.17:51
cburgessjaypipes So thats real? I saw something on facebook but you know you can't trust facebook.17:51
dansmithjaypipes: if this isn't an advertisement for not having children, I don't know what is17:51
jaypipesdansmith: :) ++17:52
dansmith"Just look at the widespread devastation that can result from a single child. JUST SAY NO"17:52
mriedemyeah, because drunk 40 year old yokuls shooting off guns never caused any harm17:53
mriedemYEEEEEEEEEEHHHHHAW!17:53
cburgessmriedem We probably shouldn't have them either.17:53
dansmithheh17:53
mikalHeh17:53
*** nicolasbock has quit IRC17:53
jaypipesmriedem: who you calling drunk?17:53
mriedemyokuls17:53
mriedemduh17:53
mikalGoogle had multiple fibre outages in Oregon because of yokels using the fibre for target practise17:54
dansmithmriedem: problem is, that yokul would go to jail for this, but the kid will get a slap on the wrist and community service17:54
jaypipesmriedem: I'll let you know I done drunk my dranks responsibally17:54
* mriedem set a fire in a field once when he was 14 on accident...17:54
cburgessmriedem In dansmith's world I think we have to take you out back and shoot you now.17:55
mriedemtold the authorities it was some other bad kids that they just missed when we got there17:55
jaypipes"accident". yeah, right, mriedem.17:55
dansmithmriedem: well, this kid wasn't even _that_ smart17:55
mriedem"geez i'm real sorry mister"17:55
dansmithcburgess: retroactive "was a kid once" punishment? harsh17:55
cburgessdansmith Thats how I role.17:55
* dansmith was never a kid17:56
dansmithjust ask my parents17:56
cburgesslol17:56
mriedemthe old benjamin button disease17:56
openstackgerritMerged openstack/nova-specs master: Add migration-allocations spec  https://review.openstack.org/49851017:56
dansmithran my middle school bedroom like an office with an inbox and credenza, made my mom call me on the internal phone system if she needed something17:56
jaypipesnice17:57
jaypipesI should do that with Julie and the pugs.17:57
jaypipes"excuse me, Fiona. you need to make an appointment."17:57
dansmithand then put nothing but an answering machine on the other end?17:57
jaypipesdansmith: indeed.17:57
cburgessNow I have this image of a small dansmith sitting a little desk -2ing requests from his mother.17:57
dansmithcburgess: not far off17:57
jaypipeslmao17:57
dimscburgess : LOL17:59
mikal"Press 1 to ask about my grades, press 2 to complain about my girlfriend, press 3 to..."18:00
*** Tom_ has joined #openstack-nova18:00
*** ralonsoh has quit IRC18:01
cburgessmikal lol18:01
*** itlinux_ has quit IRC18:01
*** harlowja has quit IRC18:02
mikal"Press 4 to ask what that loud fan noise coming from the garage is"18:03
openstackgerritMerged openstack/nova master: Make xen unit tests work with os-xenapi>=0.3.0  https://review.openstack.org/50096818:03
mikalAnswer, by the way, its a loud fan18:03
dansmithheh18:04
dansmithI did have a server catch fire in my room one time18:04
mikalHaven't we all...18:04
dansmithstill have the (remnants of the) motherboard18:04
mikalAnyways, its time for me to brave LA traffic and go to the airport.18:04
mikalI shall say sweet sweet lies about you all at the opendev thing18:05
mikal"Oh yes, they're quite competent. Have you asked them how they feel about remote compute nodes?"18:05
*** Tom_ has quit IRC18:05
mikal"They love that stuff, want more of it. Please send many patches."18:05
cburgesslol18:05
mikal"Ask for Dan, he'll help"18:06
melwittyou're going to opendev? so am I18:06
* dansmith scowls at mikal and cburgess 18:07
cburgessdansmith Wait what did I do?18:08
dansmithcburgess: you encouraged him by laughing at his jokes18:08
cburgessdansmith Oh that...18:08
*** egonzalez has joined #openstack-nova18:10
openstackgerritMerged openstack/nova master: doc: Remove deprecated call to sphinx.util.compat  https://review.openstack.org/49882418:14
*** ssmith has quit IRC18:17
*** Swami has joined #openstack-nova18:21
*** thorst has quit IRC18:22
*** thorst has joined #openstack-nova18:23
*** ijw has joined #openstack-nova18:24
*** ijw has quit IRC18:26
*** ijw has joined #openstack-nova18:26
mnaserdansmith / mriedem: https://bugs.launchpad.net/nova/+bug/171546218:26
openstackLaunchpad bug 1715462 in OpenStack Compute (nova) "Instances failing quota recheck end up with no assigned cell" [Undecided,New]18:26
*** thorst has quit IRC18:27
sdaguecburgess: what's your biggest concern on privsep?18:28
sdagueI apparently didn't see that in the patch (or glossed past it)18:28
mnaseri guess a test to make sure that a cell is assigned if the quota recheck fails would be #1 and then the fix to show that its working would be the correct path?18:28
dansmithmnaser: that'd be ideal yeah18:28
mnaserok, ill try to work on a test first18:28
cburgesssdague I haven't reviewed the patch this was a review mikal and I did verbally in my dinning room this morning.18:28
*** krtaylor has quit IRC18:28
*** thorst has joined #openstack-nova18:29
sdaguecburgess: were mimosas involved?18:29
cburgesssdague I have a few concerns but the big one is around how the privsep daemons get started. I would prefer that if a nova component detects that there is no daemon it just started/restarted it. Having to restart the entire compute/conductor/whatever process is faily heavy weight just to get the privsep daemon running again.18:29
cburgesssdague Oh I wish.. that would have made it better18:30
sdaguecburgess: yeah, the watchdogging seems reasonable18:30
mnaserdansmith mind if i bother you a bit as i figure out the best way to go about this with small questions?  https://github.com/openstack/nova/blob/master/nova/tests/unit/conductor/test_conductor.py#L1722-L1760 -- would it be beneficial to just add an assert there instead of writing a whole new test?18:30
*** jaosorior has joined #openstack-nova18:30
openstackgerritMerged openstack/nova master: trivial: Remove "vif" script  https://review.openstack.org/49144318:30
*** hrw has quit IRC18:30
*** tesseract has quit IRC18:31
dansmithmnaser: since that test is specifically aimed at checking that late quota check I think that's probably fine18:31
*** krtaylor has joined #openstack-nova18:31
mnaseralright cool, i'll give it a shot, lets hope it fails :>18:31
openstackgerritMerged openstack/nova master: trivial: Remove files from 'tools'  https://review.openstack.org/49144418:31
cburgesssdague Cool so we agree and mikal did in theory when I went to the gym this morning but now he claims there might be issues and promises performance art in Denver.18:31
*** thorst has quit IRC18:33
mnaserwoo, i have a failing test case18:36
mnasercan i get some advice on how would be the ideal way of where to move the cell_mapping saving part?  should all of it be moved earlier or should i refactor it into a private fucntion such as _update_instance_mapping(instance, cell) and then call that in the exception handling part of the quota recheck?18:39
openstackgerritErik Berg proposed openstack/nova master: Fix binary name.  https://review.openstack.org/50135918:39
mnaserthat way we don't risk introducing other weird problems that might come up from shuffling the order of things18:39
dansmithmnaser: this is a sticky place we have to be super careful18:42
dansmithgive me a few to read18:42
mnaserdansmith: no problem!  that's why i thought by refactoring it into a function and calling it in exception handling, we touch the *least* amount of codepath possible (but i'm sure folks know the codebase far more than me :-)18:43
*** thorst has joined #openstack-nova18:46
dansmithmnaser: so I think the right thing to do is really to map the instance right after we create it above18:46
dansmithmnaser: so that if we end up with any instance created (which will show up in a list) it'll have a corresponding map show that show will work18:46
dansmiths/show that/so that/18:46
mnaserdansmith so right after the with(..) block spanning 988-990?18:47
dansmithmnaser: year18:48
*** bauzas has quit IRC18:48
dansmithheh, that was either yeah or yar18:48
mnaserdansmith in my research of the code, i found the _populate_instance_mapping function which seemed pretty robust at setting the instance mapping18:48
*** bauzas has joined #openstack-nova18:48
mnaserwould it beneficial to use that instead?18:48
*** ijw has quit IRC18:49
mnaserwe have the host in there so we can pass it (but i'll take what you think is best overall)18:49
dansmithmnaser: yeah if that works should be okay18:50
*** tbachman has quit IRC18:50
mnaserdansmith cool, i'll get on this and see if it affects any other tests as well18:51
dansmithcool18:51
*** jaosorior has quit IRC18:53
mnaseryay, that specific test is passing now, i'll just rerun all the conductor tests because that code shuffle might have affected other tests18:55
*** esberglu has quit IRC18:55
*** gvrangan has quit IRC18:55
*** esberglu has joined #openstack-nova18:55
dansmithyeah, entirely possible because that extra thing isn't mocked out now18:55
*** shan has quit IRC18:56
*** ijw has joined #openstack-nova18:57
*** tidwellr has quit IRC18:59
mnaseronly one test failing after that.. a bit less painless than i expected :>18:59
dansmithrun functional tests too?18:59
*** esberglu has quit IRC19:00
*** ijw has quit IRC19:02
*** ijw has joined #openstack-nova19:03
*** catinthe_ has joined #openstack-nova19:04
*** avolkov has quit IRC19:05
mnasernope, didnt do that yet, just the uni test of conductor19:05
dansmithmight scare up another failure or two in there depending19:05
*** catintheroof has quit IRC19:06
*** catintheroof has joined #openstack-nova19:06
dansmithmnaser: I have to head to the airport in a few but will be back online from there19:06
*** esberglu has joined #openstack-nova19:06
mnaserdansmith np, thank you for your help so far19:06
dansmithnp19:06
openstackgerritMerged openstack/nova master: doc: Add user index page  https://review.openstack.org/49881719:07
*** tidwellr has joined #openstack-nova19:08
openstackgerritMerged openstack/nova master: doc: Add configuration index page  https://review.openstack.org/49881819:08
*** ijw has quit IRC19:08
*** catinthe_ has quit IRC19:10
*** moshele has joined #openstack-nova19:10
sdaguehmmmm.... https://bugs.launchpad.net/nova/+bug/1715463 doesn't seem good19:15
openstackLaunchpad bug 1715463 in OpenStack Compute (nova) "binary name gets confused under upgrades of osapi_compute and metadata" [High,Incomplete] - Assigned to Ebbex (eb4x)19:15
*** rajathagasthya has quit IRC19:15
*** tbachman has joined #openstack-nova19:16
*** moshele has quit IRC19:16
cdentsdague: the fix didn’t get linked to the bug yet (because the bug came after the code): https://review.openstack.org/#/c/501359/19:21
cdentit’s effectively a logic problem in the code19:22
cdentname change in the wrong place19:22
sdagueyeh, so this the deployment change after the upgrade, so they come up wsgi for the first time?19:22
*** abhi89 has quit IRC19:23
openstackgerritMatt Riedemann proposed openstack/nova master: Remove dest node allocation if evacuate MoveClaim fails  https://review.openstack.org/49987819:23
openstackgerritMatt Riedemann proposed openstack/nova master: Pass migration from API to conductor for evacuate  https://review.openstack.org/50017619:23
openstackgerritMatt Riedemann proposed openstack/nova master: Add a test to make sure failed evacuate cleans up dest allocation  https://review.openstack.org/49987719:23
openstackgerritMatt Riedemann proposed openstack/nova master: Add recreate test for evacuate claim failure  https://review.openstack.org/49987419:23
openstackgerritMatt Riedemann proposed openstack/nova master: Create allocations against forced dest host during evacuate  https://review.openstack.org/49939919:23
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor out claim_resources_on_destination into a utility  https://review.openstack.org/49971819:23
openstackgerritMatt Riedemann proposed openstack/nova master: Modernize set_vm_state_and_notify  https://review.openstack.org/49979919:23
cdentsdague: dunno19:26
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028519:28
*** itlinux has joined #openstack-nova19:29
ildikovmriedem: hi19:29
mriedemo/19:30
ildikovmriedem: just wanted to ask how to move forward with the Cinder-Nova open reviews?19:30
mriedemlooks like that dependent cinderclient change is released and in g-r19:31
*** crushil has quit IRC19:31
mriedemso i can start looking at https://review.openstack.org/#/c/493323/ later19:31
ildikovmriedem: yep, the cinderclient is all set and the test runs looked clean so far19:32
ildikovmriedem: I also uploaded the specs and left the multi-attach in WIP for now, but happy to get feedback on both19:33
*** ijw has joined #openstack-nova19:33
*** itlinux has quit IRC19:33
openstackgerritAndreas Jaeger proposed openstack/nova master: Fix broken link  https://review.openstack.org/50139119:34
*** pcaruana has quit IRC19:34
*** rmcall has joined #openstack-nova19:39
*** ijw has quit IRC19:39
*** ijw has joined #openstack-nova19:40
*** lbragstad has quit IRC19:43
*** pchavva has quit IRC19:44
*** ijw has quit IRC19:45
*** lbragstad has joined #openstack-nova19:45
mnaserconductor tests all passing with that change, functional are all passing so far so hopefully i can push this up soon :>19:46
*** kuzko has quit IRC19:48
*** kuzko has joined #openstack-nova19:50
*** awaugama has quit IRC19:51
*** annegentle has quit IRC19:51
*** suresh12 has quit IRC19:53
openstackgerritMatt Riedemann proposed openstack/nova master: Fix test_rpc_consumer_isolation for oslo.messaging 5.31.0  https://review.openstack.org/50140019:55
*** suresh12 has joined #openstack-nova19:55
*** ijw has joined #openstack-nova19:58
*** Apoorva has quit IRC20:00
*** tylerderosagrund has quit IRC20:00
openstackgerritDan Smith proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations  https://review.openstack.org/50102520:00
openstackgerritDan Smith proposed openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type()  https://review.openstack.org/50134320:00
openstackgerritAndreas Jaeger proposed openstack/nova master: Fix broken URLs  https://review.openstack.org/50140220:02
*** eharney has quit IRC20:04
openstackgerritAndreas Jaeger proposed openstack/nova stable/pike: Fix broken link  https://review.openstack.org/50140320:05
*** kristian__ has joined #openstack-nova20:05
*** kristian__ has quit IRC20:07
*** sridharg has quit IRC20:08
*** kristia__ has quit IRC20:08
*** felipemonteiro_ has joined #openstack-nova20:09
*** crushil has joined #openstack-nova20:09
openstackgerritMerged openstack/nova master: HyperV: Perform proper cleanup after failed instance spawns  https://review.openstack.org/49969020:10
*** dave-mccowan has quit IRC20:11
*** liverpooler has quit IRC20:11
*** felipemonteiro__ has quit IRC20:12
openstackgerritMohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails  https://review.openstack.org/50140820:12
mnaserdansmith ^20:12
dansmithwooo20:12
dansmithmelwitt: ^20:13
melwittsweet20:13
*** Apoorva has joined #openstack-nova20:14
melwittthanks for running with that mnaser20:15
mnasermelwitt np! :)20:15
*** vvargaszte has joined #openstack-nova20:16
*** vvargaszte has quit IRC20:16
*** suresh12 has quit IRC20:20
*** rajathagasthya has joined #openstack-nova20:21
*** tinwood has quit IRC20:22
*** tinwood has joined #openstack-nova20:23
sdaguelooking at old patches, is this still a thing - https://review.openstack.org/#/c/375400/ ?20:24
openstackgerritNicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes  https://review.openstack.org/48499720:24
*** annegentle has joined #openstack-nova20:25
*** suresh12 has joined #openstack-nova20:26
*** vladikr has quit IRC20:26
mriedemsdague: yeah that doesn't seem worth it right now, plus yeah we don't want to touch older release notes if we can help it20:27
openstackgerritNicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes  https://review.openstack.org/48499720:27
*** smatzek has quit IRC20:29
*** awaugama has joined #openstack-nova20:31
*** jmlowe_ has quit IRC20:33
*** egonzalez has quit IRC20:33
mriedemmnaser: melwitt: dansmith: one concern in that change related to mapping the instance before we create the bdms/tags in the cell20:33
mriedemthis is why this gets all really gorpy20:34
*** jmlowe has joined #openstack-nova20:34
dansmithmriedem: yeah I was thinking about that, but if we just have the buildrequest we have less info visible right20:34
* dansmith actually reads the comment20:34
melwittyeah, I was similarly concerned but not sure if it's a problem yet20:34
*** Apoorva_ has joined #openstack-nova20:36
dansmithwon't we keep using the buildrequest if present?20:36
dansmithlike, that's the lock we use to say "okay now you can look at the instance"20:36
dansmithlike, when we remove the buildrequest, I mean20:36
*** vvargaszte has joined #openstack-nova20:37
mriedemno we look for the instance mapping first20:38
mriedemhttps://github.com/openstack/nova/blob/b79492f70257754f960eaf38ad6a3f56f647cb3d/nova/compute/api.py#L224020:38
dansmithah, and use the cell_mapping I guess20:39
*** dave-mccowan has joined #openstack-nova20:39
*** vladikr has joined #openstack-nova20:39
mriedemso in the before times, you couldn't apply tags to a server until it was active,20:40
mriedembut in pike you can create a server with tags now20:40
*** Apoorva has quit IRC20:40
dansmithmriedem: do we need to wait until later to do that? couldn't we do those two steps after the create before this save? and then check the quota?20:40
mriedemidk20:41
dansmithalternately we could just do this if we fail the quota check, it just seemed better to me to do it in fewer places20:41
mriedemthis is all wonky why it's all done in two steps20:41
dansmithyeah20:41
mnaseradded a docstring while we figure out the rest :>20:41
openstackgerritMohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails  https://review.openstack.org/50140820:41
mriedemalthough,20:42
mriedemit was split into two parts due to melwitt's quota check change20:42
*** lucasxu has quit IRC20:42
melwittI didn't change the 2-partness for the quota check20:42
mriedemhttps://review.openstack.org/#/c/416521/20:42
mriedemmelwitt: i mean the 2 loops over the tuple20:43
mriedemthat was a single for loop before20:43
melwittoh, I see (looking at the patch now)20:43
dansmithhm why is that?20:43
mriedemhave to create them all to count20:44
mriedemand then continue creating stuff20:44
melwittat the time I was thinking, just check all of the quota first before creating a bunch of other resources if it's just gonna possibly fail in the middle20:44
dansmithwhy not do that at the end I mean20:44
dansmithyeah, but, then we're kinda checking the quota without everything created, which is a little odd20:44
melwittwe could. I was probably just thinking it would be less wasteful, why create BDMs and all of that if it's gonna fail the recheck anyway20:44
dansmithI mean, I guess it's just checking instances20:45
melwittyeah20:45
mriedemif we did just quota check at the end, then we also have to cleanup the bdms and tags20:45
mriedemin addition to the stuff that _cleanup_build_artifacts is already removing20:45
dansmithdo we?20:45
dansmithwe delete the instance.. are those things cleaned up by compute normally or something?20:46
mriedemit wouldn't be mapped to a host20:46
mriedemso no compute involved20:46
dansmithno,20:46
dansmiththis would be like a local delete20:46
dansmithbecause we obviously need not call to compute in this case, so I'm asking in a normal local delete where compute is down,20:46
dansmithdo we have to clean up bdms and tags separately?20:47
mriedemno, don't think so, those get removed via the db api20:47
mriedemwhen you delete the instance20:47
*** slaweq has quit IRC20:47
dansmithinstance delete will delete BDMs20:47
mriedemhttps://github.com/openstack/nova/blob/b79492f70257754f960eaf38ad6a3f56f647cb3d/nova/db/sqlalchemy/api.py#L188620:48
dansmithand tags20:48
mriedemhttps://github.com/openstack/nova/blob/b79492f70257754f960eaf38ad6a3f56f647cb3d/nova/db/sqlalchemy/api.py#L189420:48
dansmithyeah20:48
dansmithso when you delete the instance after the quota check those will get cleaned up if we have created them right?20:48
mriedemwell we don't delete the instance here,20:49
mriedemwe just set it to ERROR state20:49
mriedemand then the user does the (local) delete20:49
dansmithright20:50
mriedemwhich should cleanup the bdms/tags20:50
openstackgerritMerged openstack/nova master: Fix broken link  https://review.openstack.org/50139120:50
mriedemwell, the bdms and tags wouldn't even exist in the cell db20:50
mriedemif the quota check fails20:50
mriedemoh but if we moved it to the end..20:50
mriedemyeah i think that would work20:51
*** prometheanfire has left #openstack-nova20:51
dansmithwe can also do what mnaser wanted to do in the first place, which is just do this quick map fixup if we fail this quota check,20:52
melwittcells meeting in 8 minutes?20:52
dansmithbut then we will have an instance without those things in error state, and we'll be doing it from yet another place20:52
*** lpetrut has quit IRC20:52
dansmithI kinda want to keep it less spaghetti-like and do it in one place, personally20:52
*** annegentle has quit IRC20:52
dansmithmelwitt: I'm in an airport so we said this morning we'd cancel20:52
melwittoh, cool20:53
dansmithif you have things to discuss we can, because I'm clearly connected, but, I didn't see the point20:53
melwittno, just forgot to check the log20:53
dansmithack20:53
mriedemanother alternative is just don't delete the build request in _cleanup_build_artifacts, or is that crazy?20:54
dansmithno, I think we can't do that20:54
mriedemthe problem is that we haven't mapped the instance to a cell, and _cleanup_build_artifacts removes the build request, so the API literally can't find it anywhere20:54
mriedemyeah?20:54
dansmithor you mean in the failure case only?20:54
mriedemyes20:54
*** chyka has quit IRC20:54
*** chyka_ has joined #openstack-nova20:54
mriedembut then you'd have a build request in the api and an instance in the cell, unmapped20:55
dansmithmriedem: we wouldn't know where to find the instance record to mark it as deleted when they deleted the buildreq, so we'd leave that undeleted but unfindable instance forever20:55
mriedemand that instance in the cell db wouldn't get removed,20:55
mriedemunless _cleanup_build_artifacts removes it20:55
dansmithright20:55
*** vvargaszte has quit IRC20:55
mriedemso, change _cleanup_build_artifacts to delete the instance from the cell and leave the build request for API requests?20:55
mriedemunless we just overhaul this code to remove the 2 loops and do the quota check at the end20:56
melwittdoing that would remove the instance fault message which explains the quota check failed20:56
mriedemmelwitt: can't the build request have a fault on it?20:56
dansmithand we have the instance in the cell db, that's better than a buildreq, so we should use it20:56
melwittI didn't know it can20:56
dansmithmriedem: I don't think we ever update that instance do we?20:56
dansmithbecause it's fake right?20:56
mriedembuild request doesn't have a fault field, so yeah, can't do that20:57
melwittit has an instance field which would have a fault field I guess20:57
dansmithno, I think the instance is fake and constructed on the fly20:57
melwittoh, okay20:57
mriedemthe BuildRequest.instance is persisted20:58
mriedemit's just serialized json20:58
mriedembut, that starts to get weird...20:59
dansmithyeah, I guess so20:59
mriedemmanaging state on an instance nested in a build requst20:59
dansmithI'm not sure why that's better though20:59
dansmithyeah20:59
dansmiththis makes a bunch of other regular code just be used for anything else you do to this dead instance20:59
mriedemit's only better if we want to keep the 2 loops b/c we're afraid of changing them20:59
*** chyka_ has quit IRC21:00
dansmithI ain't skeered21:00
mriedemdude,21:00
mriedemi shit my pants everytime we touch nova these days21:00
dansmithhah21:00
mnaser(which is totally why i thought putting a tiny little call/fixup in the exception handling = less scary21:01
mnaser:p21:01
melwittwell, doesn't moving the quota check to the end and having only one loop address the issue fine since we know the BDMs and tags will get cleaned up at delete time?21:01
mriedemmelwitt: i think so21:01
melwittso the fake instance thing isn't the only way to get rid of the two loop thing21:01
dansmithmnaser: yeah, but we already have lots of "clean this up in this snowflake way" calls21:01
*** krtaylor has quit IRC21:01
*** kuzko has quit IRC21:01
mriedema test that creates an instance with a bdm and tag and asserts those are removed from the db would make me feel safer21:01
dansmithmelwitt: the fake instance is the way to keep the two loops he said21:01
dansmithmelwitt: fearing change21:02
*** eharney has joined #openstack-nova21:02
mriedemi fear the landmines21:02
mriedemi have literally 2 toes left21:02
*** dave-mcc_ has joined #openstack-nova21:02
*** mriedem has quit IRC21:02
melwittoh. well, my thinking is the two loops are new and it's probably better to put it back to one now that we know we can21:02
dansmiththe easier way to do that is just do what mnaser wanted to do in the first place21:02
dansmithmelwitt: I agree21:02
dansmithmelwitt: and also, the fake instance in a build request is a special case and we should have that oooonly in the case where we haven't created the instance yet or can't,21:03
*** mriedem has joined #openstack-nova21:03
mnaserfor me it's the fact that it makes it more comfortable to backport this sort of thing (which i hope we can do to stable/pike)21:03
dansmithso putting more things in that basket when we already have an instance created in a real cell seems wrong to me21:03
dansmithmnaser: right, so,21:03
dansmithI was going to suggest:21:03
dansmithhow about we let mnaser do the simpler but uglier thing of only mapping if we hit that quota check,21:03
melwittdansmith: yeah, agreed21:03
mriedemand then collapse the loops in queens only21:03
mriedemi like that21:03
dansmithbackport that mofo' and then do this cleanup separately with a functional test to get rid of the two loop21:03
dansmithyeah21:04
mnaseri think that would be best: so that stable can remain stable and you have plenty of time to review a much bigger change like that :>21:04
mnaserand you're not worried about stable breaking21:04
*** kuzko has joined #openstack-nova21:04
mriedemmnaser: btw, how is it that vexxhost is already on pike code?21:04
dansmithI think that's reasonable21:04
*** dave-mccowan has quit IRC21:05
dansmithmriedem: you mean "thank you for already being on pike" right?21:05
mnasermriedem puppet+rdo packages, trusting the openstack CI and puppet-openstack-integration we use21:05
mnaseraha21:05
mriedemdansmith: yes21:05
dansmiththought so21:05
dansmith:)21:05
mriedemi'm just impressed and happy i mean21:05
mnaserhonestly it was an easy upgrade21:05
*** chyka has joined #openstack-nova21:05
melwittyeah, it's awesome ppl are running pike21:05
mnasernewton->ocata was a bit of uh a mess21:05
mnaser400,000 records being mapped in the new default cell?21:05
mriedemwell, placement and cells v221:05
mnasertakes a little while :p21:06
*** chyka has quit IRC21:06
mnaseri backported the placement puppet manifests so we had placement running in newton so it was *one less thing* to worry about21:06
mriedemsmart21:06
dansmithhttps://twitter.com/get_offmylawn/status/90553780590394163321:07
mnaser:>21:07
melwittf yeah21:07
mnaseralso well21:08
mnaserrunning vanilla openstack means we don't much around a lot :)21:08
*** chyka has joined #openstack-nova21:09
*** cleong has quit IRC21:09
jaypipesmnaser: ++ :)21:09
mnaserso do we come to quorum to leave things as is except call set cell mapping in the exception handling code?21:09
melwittrunning vanilla openstack is the way to go. I have been part of running custom openstack and regretted doing it that way21:09
dansmithmnaser: yeah21:09
mnaseror i'll add a note as well21:10
mriedemTODO i'd say21:11
mnaserdamn21:11
mriedemyou'll also have to keep track of which cells each instance goes into21:11
mnaserum21:11
mnaseri dont know what cell21:11
mnaseryeah21:11
mnaseri'll have to figure out a clean way to do that21:11
mriedemjust save that into a dict in the first loop21:11
mnaseralright21:11
*** slaweq has joined #openstack-nova21:12
*** itlinux has joined #openstack-nova21:12
mriedemeverything already goes into that instances list variable, and is sometimes None for things that failed, but for things that did get mapped, i think you could map the instance uuid to the cell in a dict and then use those 2 variables to do the mapping in the failure block21:12
mriedems/mapped/created/21:12
*** chyka_ has joined #openstack-nova21:13
mriedemhmm, i think i just realized another bug here21:13
dansmithum21:14
*** chyka has quit IRC21:14
dansmithhah yesh21:14
mriedemwe're always mapping to the last cell21:14
dansmithyeah21:14
mriedem:)21:14
*** Apoorva_ has quit IRC21:14
dansmithoof21:14
* mriedem reports a new bug21:14
*** Apoorva has joined #openstack-nova21:14
*** gouthamr has quit IRC21:15
dansmithmnaser: so you might want to stack your change on top of the fix for that bug21:15
dansmithwhich I can cook up21:15
mnaserokay cool, ill work on the stuff to set mappings21:15
*** krtaylor has joined #openstack-nova21:16
dansmithmriedem: I'm guessing that this would have uncovered that if I had ever finished it: https://review.openstack.org/#/c/452006/21:17
dansmithah, no,21:18
dansmiththat's the migrate one21:18
mriedemhttps://bugs.launchpad.net/nova/+bug/171549321:18
openstackLaunchpad bug 1715493 in OpenStack Compute (nova) "Instances always get mapped into the last processed cell in conductor" [High,Triaged]21:18
*** chyka_ has quit IRC21:19
dansmithI guess this bug came from splitting that loop amirite?21:19
mriedemyup21:19
*** tidwellr has quit IRC21:20
mriedemso when we combine we can remove both workarounds21:20
dansmithyeah21:20
openstackgerritChris Dent proposed openstack/nova master: WIP: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007321:20
openstackgerritChris Dent proposed openstack/nova master: [placement] Allow _set_allocations to delete allocations  https://review.openstack.org/50105121:20
melwittugh :(21:20
mriedemplus, if mnaser's patch creates a instance -> cell mapping dict to keep track, the 2nd loop could use that too21:20
dansmithyeah21:20
openstackgerritDan Smith proposed openstack/nova master: Track which cell each instance is created in and use it consistently  https://review.openstack.org/50145221:20
dansmithgonna work on tests, but pushed this up in case I have to run ^21:21
* mnaser wil rebase on that21:21
*** yangyapeng has joined #openstack-nova21:22
*** chyka has joined #openstack-nova21:23
dansmithhmm, we kinda have a test for this, I'm not sure why it's not failing21:25
*** gbarros has joined #openstack-nova21:25
*** yangyapeng has quit IRC21:27
mnaserdansmith is it okay that when i try to do git review with your patch below mine, it mentions that it will submit two commits?21:29
* mnaser is not an expert at stacking gerrit changes21:29
*** crushil has quit IRC21:29
dansmithmnaser: yep21:29
mnaserokay cool21:29
openstackgerritMohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails  https://review.openstack.org/50140821:29
mnaseroh it didnt send both, nice.  that's leveraging your cell_instance_cache21:30
mnaseroh i should update the commit msg21:30
melwittmnaser: I usually double check to make sure the commit hash of the dependent change is the same as what shows on the review being rebased upon21:30
*** crushil has joined #openstack-nova21:30
melwittthat's how you can tell whether it will push more than just your change21:30
mnasermelwitt oh, if it matches it wont submit it? i'm just terrified of the embarassement that happens sometimes when an irc channel gets spammed :p21:30
mriedemmnaser: git review -R also avoids the rebase of the base changes21:31
melwittmnaser: yeah. if the hash hasn't changed it won't submit it. so you can know before you do it21:31
mriedemmelwitt: btw i think i figured out the git review / rebase author change thing,21:31
mriedemit happens when rebasing on a series that involves a merge conflict,21:31
mriedemso you rebase, merge conflict, fix, git add / commit, git rebase continue21:31
mriedemand that has changed the author21:31
openstackgerritMohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails  https://review.openstack.org/50140821:32
melwittweird. I feel like I've rebased other ppl's changes with merge conflict and it didn't change the author before21:32
mnaserthere, that should have most of it covered21:32
melwittbut, that's great data to go on. I'll pay attention next time21:32
dansmithokay, the test I thought we had was build not schedule_and_build21:33
*** Apoorva_ has joined #openstack-nova21:33
dansmithI have to board soon, but I can work on tests for this on the plane unless someone else wants to?21:33
mnaseri'd help but it sits outside my relm of knowledge :) have a safe flight though dansmith21:34
dansmithmnaser: np, thanks21:35
mnasertime to go figure out why horizon pike is logging users out now :> thanks for the help everyone, i'll keep an eye out on reviews for the patch21:35
mriedemmnaser: one issue in your patch21:36
mnasersure whats up21:36
mnaseroh /me looks21:36
mriedemcomment inline21:36
melwittdansmith: I can write the tests for your change if you want to not do it on the plane21:36
*** Apoorva has quit IRC21:37
mnasermriedem took me a while to figure out what you meant till i realized that super hidden for loop in the top lol21:38
mriedem:) everything else looks good21:39
mriedemsdague: low hanging fruit review https://review.openstack.org/#/c/499001/21:42
mriedemskip more cells v1 random failure tests21:42
openstackgerritMohammed Naser proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails  https://review.openstack.org/50140821:42
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter()  https://review.openstack.org/48813721:44
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005721:44
*** harlowja has joined #openstack-nova21:45
openstackgerritMerged openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916621:45
openstackgerritEric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter()  https://review.openstack.org/48813721:48
*** priteau__ has quit IRC21:50
openstackgerritChris Dent proposed openstack/nova-specs master: Add a spec for POST /allocations in placement  https://review.openstack.org/49925921:52
*** edmondsw has quit IRC21:52
*** cdent has quit IRC21:54
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Add release note for force live migration allocations  https://review.openstack.org/50147721:55
*** catintheroof has quit IRC21:56
*** dave-mcc_ has quit IRC21:57
*** lyan has quit IRC21:58
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005721:58
*** priteau has joined #openstack-nova21:58
*** chyka has quit IRC22:02
*** priteau has quit IRC22:03
*** ijw has quit IRC22:03
*** awaugama has quit IRC22:04
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests  https://review.openstack.org/49224722:05
*** ijw has joined #openstack-nova22:05
*** xyang1 has quit IRC22:09
*** erlon has quit IRC22:11
*** ijw has quit IRC22:11
*** suresh12 has quit IRC22:13
mriedemmelwitt: i'm going to step away to make dinner but if you get that test written ping me and i'll take a look tonight22:15
melwittmriedem: okay. I wasn't sure if dansmith got my message before boarding. so I wasn't sure if he's already working on it on the plane22:16
*** kylek3h has quit IRC22:18
*** baoli has quit IRC22:18
*** edmondsw has joined #openstack-nova22:23
*** suresh12 has joined #openstack-nova22:24
*** jaypipes has quit IRC22:24
*** edmondsw has quit IRC22:28
*** suresh12 has quit IRC22:29
*** ijw has joined #openstack-nova22:35
*** JoseMello has quit IRC22:35
*** krtaylor has quit IRC22:37
*** slaweq has quit IRC22:37
*** Apoorva_ has quit IRC22:40
*** Apoorva has joined #openstack-nova22:41
*** suresh12 has joined #openstack-nova22:42
mnaserdo live migration jobs often fail22:46
mnaseror did i break something :>22:46
*** vladikr has quit IRC22:47
mnaserDetails: (LiveAutoBlockMigrationV225Test:test_live_block_migration) Server 44884988-d3f2-4e32-a367-4962acda6346 failed to reach ACTIVE status and task state "None" within the required time (196 s). Current status: BUILD. Current task state: scheduling.22:47
*** penick has quit IRC22:50
*** gouthamr has joined #openstack-nova22:52
*** thegreenhundred has quit IRC22:55
*** gbarros has quit IRC22:55
*** scottda_ has quit IRC23:00
*** vladikr has joined #openstack-nova23:00
*** felipemonteiro_ has quit IRC23:01
*** Apoorva_ has joined #openstack-nova23:01
*** cdent has joined #openstack-nova23:04
*** Apoorva has quit IRC23:05
*** ijw has quit IRC23:07
*** cdent has quit IRC23:08
*** felipemonteiro has joined #openstack-nova23:09
*** marst has quit IRC23:11
*** itlinux has quit IRC23:16
openstackgerritMerged openstack/nova master: Amend uuid4 hacking rule  https://review.openstack.org/48831523:21
openstackgerritMerged openstack/nova stable/pike: Fix broken link  https://review.openstack.org/50140323:21
*** kylek3h has joined #openstack-nova23:23
*** suresh12 has quit IRC23:26
*** kylek3h has quit IRC23:29
*** Dave has quit IRC23:33
*** suresh12_ has joined #openstack-nova23:34
*** yangyapeng has joined #openstack-nova23:38
*** slaweq has joined #openstack-nova23:38
*** baoli has joined #openstack-nova23:38
*** baoli has quit IRC23:39
mriedemmnaser: ah ./nova/conductor/manager.py:1017:20: F821 undefined name 'instance_mapping_cache'23:40
*** baoli has joined #openstack-nova23:40
mnaseroopsie23:40
mriedemthat's a bug in dan's change23:40
mnaseroh thats not my path23:40
mnaseryeah23:40
mriedemi'll fix quick23:41
*** hongbin has quit IRC23:42
*** hongbin has joined #openstack-nova23:43
*** yangyapeng has quit IRC23:43
*** slaweq has quit IRC23:43
*** hongbin has quit IRC23:44
openstackgerritMatt Riedemann proposed openstack/nova master: Track which cell each instance is created in and use it consistently  https://review.openstack.org/50145223:44
openstackgerritMatt Riedemann proposed openstack/nova master: Ensure instance mapping is updated in case of quota recheck fails  https://review.openstack.org/50140823:44
*** suresh12_ has quit IRC23:47
*** suresh12 has joined #openstack-nova23:48
*** Dave has joined #openstack-nova23:49
*** baoli has quit IRC23:51
openstackgerritMerged openstack/nova master: Add release note for force live migration allocations  https://review.openstack.org/50131423:58
*** priteau has joined #openstack-nova23:59

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