Tuesday, 2018-10-30

*** gyee has quit IRC00:04
*** artom has joined #openstack-nova00:08
openstackgerritMatt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid  https://review.openstack.org/61401200:15
openstackgerritMatt Riedemann proposed openstack/nova master: Add Destination.cross_cell_move field  https://review.openstack.org/61403500:15
openstackgerritMatt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method  https://review.openstack.org/61403600:15
openstackgerritMatt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells  https://review.openstack.org/61403700:15
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Cross-cell resize  https://review.openstack.org/60393000:15
*** mriedem_away has quit IRC00:15
*** wangy has joined #openstack-nova01:03
*** bhagyashris_ has joined #openstack-nova01:03
*** Dinesh_Bhor has joined #openstack-nova01:12
*** spatel has joined #openstack-nova01:14
*** hongbin has joined #openstack-nova01:23
openstackgerritChason Chan proposed openstack/nova stable/rocky: Remove deprecated nova-consoleauth reference from doc  https://review.openstack.org/61405501:29
*** TuanDA has joined #openstack-nova01:29
*** tiendc has joined #openstack-nova01:35
*** binhnd10894 has quit IRC01:45
openstackgerritTakashi NATSUME proposed openstack/nova master: Use links to placement docs in nova docs  https://review.openstack.org/61405601:57
*** Dinesh_Bhor has quit IRC02:01
*** Dinesh_Bhor has joined #openstack-nova02:02
*** lei-zh has joined #openstack-nova02:06
*** wangy has quit IRC02:14
*** mhen has quit IRC02:14
*** wangy has joined #openstack-nova02:16
*** mhen has joined #openstack-nova02:19
*** lei-zh has quit IRC02:31
*** lei-zh has joined #openstack-nova02:32
*** hshiina has joined #openstack-nova02:43
*** mrsoul has quit IRC02:57
*** takashin has left #openstack-nova03:03
*** psachin has joined #openstack-nova03:11
*** wangy has quit IRC03:21
*** wangy has joined #openstack-nova03:21
*** cfriesen has quit IRC03:23
*** Dinesh_Bhor has quit IRC03:38
*** wangy has quit IRC03:39
*** Dinesh_Bhor has joined #openstack-nova03:41
*** udesale has joined #openstack-nova03:49
*** spatel has quit IRC04:04
*** ShilpaSD has joined #openstack-nova04:05
*** tetsuro has joined #openstack-nova04:10
*** lei-zh has quit IRC04:10
*** janki has joined #openstack-nova04:14
*** dave-mccowan has quit IRC04:19
*** Dinesh_Bhor has quit IRC04:21
*** hongbin has quit IRC04:23
*** TuanDA has quit IRC04:23
*** TuanDA has joined #openstack-nova04:23
*** ccamacho has quit IRC04:26
*** sapd1 has quit IRC04:38
*** Dinesh_Bhor has joined #openstack-nova04:49
*** lpetrut has joined #openstack-nova04:52
*** sapd1 has joined #openstack-nova04:59
*** Dinesh_Bhor has quit IRC05:00
*** wangy has joined #openstack-nova05:03
*** tetsuro has quit IRC05:03
*** tetsuro has joined #openstack-nova05:03
*** tetsuro_ has joined #openstack-nova05:04
*** tetsuro has quit IRC05:08
*** lei-zh has joined #openstack-nova05:11
openstackgerritMerged openstack/nova master: Drop legacy live migrate allocation compat code  https://review.openstack.org/61197505:12
*** Dinesh_Bhor has joined #openstack-nova05:15
*** lei-zh has quit IRC05:18
*** lpetrut has quit IRC05:28
*** tetsuro_ has quit IRC05:29
*** jaosorior has quit IRC05:32
*** jaosorior has joined #openstack-nova05:32
*** tetsuro has joined #openstack-nova05:34
*** pcaruana has joined #openstack-nova05:36
*** tetsuro has quit IRC05:38
*** brinzhang has joined #openstack-nova05:41
*** lei-zh has joined #openstack-nova05:42
*** pcaruana has quit IRC05:47
*** ratailor has joined #openstack-nova05:48
*** wangy has quit IRC05:59
*** tetsuro has joined #openstack-nova06:05
*** wangy has joined #openstack-nova06:06
*** tetsuro has quit IRC06:09
*** tetsuro has joined #openstack-nova06:10
*** wangy has quit IRC06:11
*** wxy-xiyuan has quit IRC06:16
*** mnaser has quit IRC06:16
*** wxy-xiyuan has joined #openstack-nova06:16
*** mnaser has joined #openstack-nova06:17
*** hamzy_ has joined #openstack-nova06:17
*** hamzy has quit IRC06:18
*** vdrok has quit IRC06:22
*** vdrok_ has joined #openstack-nova06:23
*** lyarwood has joined #openstack-nova06:25
*** hoonetorg has quit IRC06:29
*** slaweq has quit IRC06:40
*** ShilpaSD has quit IRC06:41
*** hoonetorg has joined #openstack-nova06:42
*** wangy has joined #openstack-nova06:42
*** tonyb has quit IRC06:50
*** Luzi has joined #openstack-nova06:52
*** ShilpaSD has joined #openstack-nova06:54
*** TuanDA has quit IRC07:00
*** TuanDA has joined #openstack-nova07:01
*** Shilpa has joined #openstack-nova07:01
*** tomtom001 has quit IRC07:02
*** ShilpaSD has quit IRC07:03
*** wangy has quit IRC07:04
*** tiendc has quit IRC07:05
*** wangy has joined #openstack-nova07:06
*** tiendc has joined #openstack-nova07:06
*** jangutter has joined #openstack-nova07:09
*** abhishekk has joined #openstack-nova07:10
*** wangy has quit IRC07:18
*** wangy has joined #openstack-nova07:20
*** dpawlik has joined #openstack-nova07:26
*** dpawlik has quit IRC07:27
*** dpawlik has joined #openstack-nova07:27
openstackgerritMerged openstack/nova master: Fix ironic client ironic_url deprecation warning  https://review.openstack.org/61187207:31
*** hshiina has quit IRC07:44
*** sahid has joined #openstack-nova07:46
*** slaweq has joined #openstack-nova07:50
*** tetsuro has quit IRC07:50
*** pcaruana has joined #openstack-nova07:57
*** pcaruana is now known as pcaruana|elisa|07:59
*** alexchadin has joined #openstack-nova08:01
*** ccamacho has joined #openstack-nova08:04
*** sapd1 has quit IRC08:10
*** ralonsoh has joined #openstack-nova08:11
*** sapd1 has joined #openstack-nova08:12
*** egonzalez has quit IRC08:15
*** Dinesh_Bhor has quit IRC08:16
bauzasgood morning Nova08:16
*** egonzalez has joined #openstack-nova08:17
*** helenafm has joined #openstack-nova08:18
*** wangy has quit IRC08:20
*** Dinesh_Bhor has joined #openstack-nova08:30
kashyapmnaser: Hey, I saw the ping here.  But I guess your live migration issue is nailed down now, thanks to mdbooth_390 & co08:45
*** lpetrut has joined #openstack-nova08:48
*** priteau has joined #openstack-nova08:52
*** slaweq has quit IRC08:53
mnaserkashyap: indeed, your comment on that bz helped point me towards the why :)08:55
mnaserSo thanks! :)08:55
kashyapmnaser: Which bugzilla?08:55
kashyapSorry, just catching up after 10 days of travel and conferences08:55
mnaserkashyap: https://bugzilla.redhat.com/show_bug.cgi?id=144934608:56
openstackbugzilla.redhat.com bug 1449346 in libvirt "Addition of host_mtu=XXXX changes PCI ioport size & addresses for virtio-net device" [Unspecified,Closed: errata] - Assigned to laine08:56
mnaserWhich referenced in https://review.openstack.org/#/c/614008/08:56
kashyapAh, that one.08:56
*** jpena|off is now known as jpena08:57
openstackgerritMerged openstack/nova master: Add tests for bug #1800511  https://review.openstack.org/61400408:58
openstackbug 1800511 in OpenStack Compute (nova) rocky "VMs with vif_type bridge/tap started before Rocky upgrade cannot be live migrated" [High,In progress] https://launchpad.net/bugs/1800511 - Assigned to Mohammed Naser (mnaser)08:58
*** ccamacho has quit IRC09:01
*** tetsuro has joined #openstack-nova09:07
*** derekh has joined #openstack-nova09:25
*** lei-zh has quit IRC09:33
*** tetsuro has quit IRC09:39
*** cdent has joined #openstack-nova09:42
*** tetsuro has joined #openstack-nova09:48
*** tetsuro has quit IRC09:51
*** bhagyashris_ has quit IRC09:51
*** wznoinsk has quit IRC09:51
*** Dinesh_Bhor has quit IRC09:55
*** slaweq has joined #openstack-nova10:08
openstackgerritMaciej Jozefczyk proposed openstack/nova master: WIP: Add online_data_migration script  https://review.openstack.org/61416710:10
openstackgerritMaciej Jozefczyk proposed openstack/nova master: WIP: Add fill_virtual_interface_list online_data_migration script  https://review.openstack.org/61416710:11
*** priteau has quit IRC10:11
*** ccamacho has joined #openstack-nova10:11
*** Dinesh_Bhor has joined #openstack-nova10:15
*** TuanDA has quit IRC10:21
*** ttsiouts has joined #openstack-nova10:22
*** pvc has joined #openstack-nova10:23
*** abhishekk has quit IRC10:26
*** tbachman has quit IRC10:32
openstackgerritwangqiang-bj proposed openstack/nova-specs master: fix wrong spelling of "unnecessary"  https://review.openstack.org/61417310:35
pvchi10:35
pvcanyone successfully setup cyborg on openstack and attach an accelerator on an instance?10:35
*** dpawlik has quit IRC10:37
*** tbachman has joined #openstack-nova10:37
*** dpawlik has joined #openstack-nova10:37
openstackgerritwangqiang-bj proposed openstack/nova-specs master: fix wrong spelling of "unnecessary"  https://review.openstack.org/61417310:37
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal  https://review.openstack.org/59160710:39
openstackgerritMaciej Jozefczyk proposed openstack/nova master: WIP: Add fill_virtual_interface_list online_data_migration script  https://review.openstack.org/61416710:39
*** icey has quit IRC10:40
*** cdent has quit IRC10:42
*** tbachman has quit IRC10:45
*** pvc has quit IRC10:45
*** helenafm has quit IRC10:45
*** icey has joined #openstack-nova10:52
openstackgerritJohn Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec  https://review.openstack.org/60220110:53
*** gaudenz has joined #openstack-nova10:53
openstackgerritwanghui proposed openstack/nova-specs master: Fix a typo of 'keystoneauth1'  https://review.openstack.org/61418711:02
*** erlon has joined #openstack-nova11:04
*** priteau has joined #openstack-nova11:05
openstackgerritGorka Eguileor proposed openstack/nova master: Fix unnecessary locking when connecting volumes  https://review.openstack.org/61419011:10
*** dave-mccowan has joined #openstack-nova11:10
*** ratailor has quit IRC11:11
openstackgerritwangqiang-bj proposed openstack/nova master: fix wrong spelling of "explicit"  https://review.openstack.org/61419111:12
*** cdent has joined #openstack-nova11:13
*** Dinesh_Bhor has quit IRC11:19
*** udesale has quit IRC11:24
*** janki has quit IRC11:25
*** mvkr has quit IRC11:30
*** alexchadin has quit IRC11:36
*** dtantsur|afk is now known as dtantsur11:50
*** jpena is now known as jpena|lunch11:57
*** mvkr has joined #openstack-nova12:02
*** tbachman has joined #openstack-nova12:04
*** brinzhang has quit IRC12:07
*** tbachman has quit IRC12:07
*** tbachman has joined #openstack-nova12:13
*** ttsiouts has quit IRC12:16
*** tssurya has joined #openstack-nova12:20
*** READ10 has joined #openstack-nova12:25
*** wznoinsk has joined #openstack-nova12:28
*** wznoinsk has quit IRC12:29
*** janki has joined #openstack-nova12:31
*** wznoinsk has joined #openstack-nova12:31
*** ttsiouts has joined #openstack-nova12:32
*** wznoinsk has quit IRC12:41
*** wznoinsk has joined #openstack-nova12:45
*** wznoinsk has quit IRC12:47
*** wznoinsk has joined #openstack-nova12:48
*** wznoinsk has quit IRC12:51
*** wznoinsk has joined #openstack-nova12:57
*** wznoinsk has quit IRC12:57
*** wznoinsk has joined #openstack-nova12:59
*** awaugama has joined #openstack-nova12:59
*** lpetrut has quit IRC13:05
*** helenafm has joined #openstack-nova13:06
*** cdent has quit IRC13:07
*** udesale has joined #openstack-nova13:08
*** psachin has quit IRC13:09
*** lpetrut has joined #openstack-nova13:11
*** jpena|lunch is now known as jpena13:12
*** mriedem has joined #openstack-nova13:13
*** psachin has joined #openstack-nova13:15
openstackgerritMerged openstack/nova master: Remove restart_scheduler_service() method  https://review.openstack.org/61334813:16
*** tiendc has quit IRC13:26
*** cdent has joined #openstack-nova13:27
openstackgerritGorka Eguileor proposed openstack/nova master: Fix unnecessary locking when connecting volumes  https://review.openstack.org/61419013:33
efriedbauzas: Did you see http://lists.openstack.org/pipermail/openstack-dev/2018-October/136123.html ?13:34
*** dklyle has quit IRC13:37
*** dklyle has joined #openstack-nova13:37
*** ttsiouts has quit IRC13:40
openstackgerritBalazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs  https://review.openstack.org/60412513:42
*** ttsiouts has joined #openstack-nova13:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object  https://review.openstack.org/56884013:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec  https://review.openstack.org/56726713:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes  https://review.openstack.org/57084713:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler  https://review.openstack.org/56726813:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945913:42
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331713:42
*** wangy has joined #openstack-nova13:57
*** wangy has quit IRC13:57
*** dklyle has quit IRC14:02
*** david-lyle has joined #openstack-nova14:02
jaypipeslbragstad: never got around to quota stuff yesterday, sorry. day ended up completely consumed by chasing down Chef bugs.14:06
jaypipeslbragstad: will try again this afternoon.14:06
mriedemgibi: you rebased while i was reviewing https://review.openstack.org/#/c/568840/2414:09
mriedembut you can remove those todos now14:09
*** wangy has joined #openstack-nova14:10
efriedmelwitt: Would you mind giving https://review.openstack.org/#/c/610174/ another pass today please? (<== edmondsw)14:10
efriedjaypipes: Since https://review.openstack.org/#/c/468560/ was +A before and the rebase was trivial (uuidsentinel moved to oslo), any chance of a fast approve?14:11
jaypipesefried: done14:12
efriedjaypipes: Thanks!14:12
*** itlinux has quit IRC14:20
*** cdent has quit IRC14:21
lbragstadjaypipes sounds good14:21
lbragstadi'll be around14:21
*** wangy has quit IRC14:25
*** eharney has joined #openstack-nova14:28
*** fghaas has joined #openstack-nova14:29
fghaasHi kashyap, assuming you're back and recovered from Edinburgh, could I bother you to take another peek at https://review.openstack.org/#/c/609788/ ?14:32
kashyapfghaas: Hi, I'm also recovering from "other things", will look. :-)14:32
gibimriedem: thanks for the review.14:32
fghaasPerfect. No rush, just wanted to make sure it didn't fall off the table. :)14:32
*** pcaruana|elisa| has quit IRC14:33
kashyapfghaas: Ah, you've answered the questions.  And have added live migration sections14:33
*** janki has quit IRC14:35
johnthetubaguyjaypipes: are you planning work on placement quotas, just saw your tidy up patches14:35
johnthetubaguyso I should have read the scrollback14:35
johnthetubaguyI was planning on looking at this stuff ASAP: https://review.openstack.org/#/c/60220114:36
*** psachin has quit IRC14:38
*** wangy has joined #openstack-nova14:38
*** munimeha1 has joined #openstack-nova14:38
johnthetubaguyjaypipes: I see your thread now, quota classes I think come from some out of tree rackspace idea that never made it downstream.14:39
*** mlavalle has joined #openstack-nova14:39
johnthetubaguyjaypipes: the idea was a project was assigned a class, if not assigned you fallback to the default. i.e. you could upgrade someone to the I_HAVE_BIG_CREDIT_LIMIT quota class, etc. A bit like flavors.14:40
gaudenzHi14:40
mriedemgibi: a few nits and questions in https://review.openstack.org/#/c/567267/14:40
*** jmlowe has quit IRC14:41
sean-k-mooneyjohnthetubaguy: is that related to the idea of qoutas for flavors and generic placement resouce class quotas14:41
johnthetubaguysadly no, that's different14:41
*** wangy_ has joined #openstack-nova14:42
johnthetubaguythis is more a groups of limits, rather than limits on flavors14:42
sean-k-mooneyoh ok i like that usecase but never herard of quota classes until now14:42
johnthetubaguyits half lurking in the code, and was never finished14:42
johnthetubaguybest ignored14:42
openstackgerritJack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests  https://review.openstack.org/60798914:42
sean-k-mooneyok ill continue to do so14:43
jaypipesjohnthetubaguy: or you could just have quota templates internally and assign a new project the values of that quota template... quota classes never needed to be in the API.14:43
*** wangy has quit IRC14:43
johnthetubaguyjaypipes: yeah, totally, that is the current plan for unified limits. its crazy pants stuff14:43
jaypipesin a similar way that "registered quotas" in the new keystone limits API doesn't need to be exposed to users at all.14:43
jaypipesjohnthetubaguy: ++14:43
*** pcaruana|elisa| has joined #openstack-nova14:45
johnthetubaguyjaypipes: did you get chance to look at the unified limits spec I am looking at working on with alex_xu?14:46
mriedembauzas: do you remember this? https://review.openstack.org/#/c/582417/1/nova/conductor/tasks/migrate.py@a28714:48
mriedembecause i'm having a hell of a time figuring out why i can't pass a RequestSpec object to the compute prep_resize RPC API but we can pass request spec to other things like select_destinations and build_and_run_instance14:48
jaypipesjohnthetubaguy: in my queue.14:49
johnthetubaguyjaypipes: cool, just making sure to avoid duplicate work14:49
*** dtantsur is now known as dtantsur|brb14:50
*** wangy_ has quit IRC14:50
*** Luzi has quit IRC14:52
kashyapfghaas: One latest remark in the change; regardless, I think it can be merged now.  And can fine-tune in further patches.14:55
bauzasmriedem: looking14:59
bauzascurrently on an internal priority bug hence my absence today15:00
*** wangy has joined #openstack-nova15:01
bauzasmriedem: oh I remember the tz issue, but I don't remember how I fixed it15:01
bauzasmriedem: that's only a test issue, right?15:01
*** cfriesen has joined #openstack-nova15:02
*** jmlowe has joined #openstack-nova15:02
*** dave-mccowan has quit IRC15:03
mriedemit's not fixed15:03
mriedemthere was an oslo.messaging bug for it against the fake driver, but was invalidated15:03
mriedemi'm just not sure why the prep_resize cast is a problem, but passing request spec to other rpc methods isn't a problem15:03
melwittefried: sure15:06
efriedthanks!15:06
bauzasmriedem: I think I fixed this using the call_for_cast for my tests15:07
bauzasIIRC15:07
*** cdent has joined #openstack-nova15:08
mriedemfghaas: a couple of questions in https://review.openstack.org/#/c/609788/215:09
wangyIs there anyone can help me?I used rally to boot server with concurrency 50,15:10
wangyI got error {u'message': u'No valid host was found. There are not enough hosts available.', u'code': 500,} My platform version is stable/pike, and nova-placement-api.log showed “DB error: (pymysql.err.InternalError) (1213, u'Deadlock found when trying to get lock;”15:10
*** itlinux has joined #openstack-nova15:11
mriedemwangy: you might need https://github.com/openstack/nova/commit/c66ae65775bb9d885fac059847063fee70617bc5#diff-a040eef16b56cc7862fb2a334c757dea15:12
mriedemsince you didn't mention where the deadlock is occurring, it's hard to say15:12
mriedemthere was also https://github.com/openstack/nova/commit/e001b0538a9d413f864d39584c2aea0b9e200e4b#diff-a040eef16b56cc7862fb2a334c757dea but that was fixed in pike15:12
wangyThe db deadlock was caused by "INSERT INTO allocations "15:13
fghaasmriedem, kashyap: all good points, thanks — will push update momentarily15:14
wangymriedem: thanks,I had added retry decorator, it did not help15:15
mriedemwangy: seems that would be covered with https://github.com/openstack/nova/commit/e001b0538a9d413f864d39584c2aea0b9e200e4b#diff-a040eef16b56cc7862fb2a334c757dea15:15
mriedemdo you see "Performing DB retry for function" in the placement api logs?15:16
mriedemif so, that means it's retrying on deadlock15:16
mriedemif you see "DB exceeded retry limit" that means it hit the retry limit15:17
mriedemand probably means you need more resources / more placement workers in your env15:17
wangymriedem: yes, I see the logs, thanks for your opinion, I will try it.15:18
openstackgerritTheodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0  https://review.openstack.org/55421815:19
Shilpamriedem: HI.15:21
Shilpamriedem:Can you please help me to understand in case of nova evacuate, why we are restoing allocations at source?15:21
Shilpais it for rollback?15:22
*** wangy has quit IRC15:23
Shilpaor in case of evacuate, in case of success, how allocaions at source are cleared off? Is any process set for it?15:23
mriedemi'm not sure what you mean by "restoring allocations at source"15:23
mriedemif the evacuated source nova-compute service starts up again and instances have been successfully migrated from it, the allocations are cleaned up for the source node during that startup15:24
mriedemhere https://github.com/openstack/nova/blob/d1e38f06bb5282dd9bf81d979926a0ebbfd93caa/nova/compute/manager.py#L71215:24
mriedemif the evacuation fails, the source node allocations should still exist in placement15:25
Shilpamriedem: yes in case of failure, observe source node allocations are still there in placement15:26
mriedemif the evacuation fails, the destination node allocations, which are created by the scheduler, should be cleaned up here https://github.com/openstack/nova/blob/d1e38f06bb5282dd9bf81d979926a0ebbfd93caa/nova/compute/manager.py#L3044 or here https://github.com/openstack/nova/blob/d1e38f06bb5282dd9bf81d979926a0ebbfd93caa/nova/compute/manager.py#L305815:26
Shilpamriedem: will check for evacuated source nova-compute service starts up again, and what happens there15:26
gibimriedem: thanks for the review on https://review.openstack.org/#/c/567267/ I have to get back to these patches next week. Now I'm in a heavy preparation for our bandwidth demo for the summit15:26
*** wangy has joined #openstack-nova15:27
openstackgerritsean mooney proposed openstack/nova master: harden placement init under wsgi  https://review.openstack.org/61003415:27
mriedemgibi: ok, the changes don't seem very difficult for either change, would be ok with me just addressing my comments if you don't get to them first? so we can keep moving that series?15:28
mriedem*would you be ok15:28
mriedemtssurya: i'm not sure how caught up your are on the changes dansmith made to the handling a down cell series, but i've got a slew of comments in the majority of the bottom changes - are you going to address those this week? if not, i might start working on that as well since i think dansmith is busy15:29
*** gyee has joined #openstack-nova15:29
openstackgerritsean mooney proposed openstack/nova master: harden placement init under wsgi  https://review.openstack.org/61003415:30
Shilpamriedem: checked that if evacuated source nova-compute service starts up again and instances have been successfully migrated from it, the allocations are cleaned up for the source node during that startup15:30
Shilpamriedem: thanks for this explaination15:31
mriedemShilpa: yw15:32
bauzasmriedem: oops sorry I meant cast_as_calls15:33
bauzasthis ^ fixed my tz issue for the functional tests15:33
sean-k-mooneyShilpa: note that technicall the instanaces are not migrated. they are recreaed during evacuate15:33
*** wangy has quit IRC15:33
mriedembauzas: you mean the CastAsCallFixture right?15:33
bauzasyup15:33
openstackgerritMatt Riedemann proposed openstack/nova master: Drop request spec migration code  https://review.openstack.org/61426515:33
mriedemeither way, i don't want to use that fixture if it can be helped15:33
sean-k-mooneyShilpa: but if the source comes back up it will fix the allcoations15:33
mriedemi'm also not sure why that would make a difference15:33
mriedemi suppose because it stubs out the prepare method in the rpc client so we don't hit the validation in the fake rpc driver15:34
bauzasmriedem: I didn't really had an issue when testing live, so I didn't went further down15:34
dansmithmriedem: just saw your comment on the cells fixture thing15:34
bauzasmriedem: yeah what you say rings a bell to me15:34
bauzasbut that's one15:34
bauzasold*15:34
Shilpasean-k-mooney: thanks, understood15:34
mriedemi know we have functional tests that don't use the CastAsCallFixture which still pass a request spec to select_destinations and build_and_run_instance15:34
mriedemso i'm still confused15:34
dansmithmriedem: if you hit a problem with that, there should be a runtimeerror raised15:35
*** pcaruana|elisa| has quit IRC15:35
bauzasmriedem: FWIW, I'm disecting some internal bug for a customer that sounds pretty similar to https://bugs.launchpad.net/nova/+bug/160025115:35
openstackLaunchpad bug 1600251 in OpenStack Compute (nova) "live migration does not honor server group policy" [High,Fix released]15:35
dansmithmriedem: like, it re-checks with the lock held, so if you don't see that, it should be the same state as it was before that change15:36
bauzasmriedem: mmm, good point about the other functional tests15:36
dansmithmriedem: now, entirely possible that this shook up a race that we already had and were winning though15:36
sean-k-mooneybauzas: spefically is that related to migratin multiple instnace in the server group conurrently?15:36
bauzassean-k-mooney: yup15:36
sean-k-mooneybauzas: ya that is totally undefiened15:36
mriedembauzas: well that race has never been fixed15:36
mriedemsee my comments in the bug about what starlingx did to try and mitigate it15:37
bauzasyup I saw them15:37
mriedemdansmith: yeah maybe, i haven't dug too deep15:37
sean-k-mooneydansmith: well exclutding nfv instnaces we sould have only won the race reliably if there was no race e.g. no concurrent ligrations. even in the singel sechduler case i dont belive we record the instance is on a node untill it boots so there would always be the race between the schduler selecting the node for the second insnance and the first instance booting right15:40
dansmithsean-k-mooney: um, what?15:42
dansmithsean-k-mooney: we talking about the same bug/15:42
tssuryamriedem: I am catching up on them and addressing them15:42
tssuryaI will be working on them this week15:42
tssuryamy major concern being https://review.openstack.org/#/c/567785/29/nova/compute/api.py@265815:43
sean-k-mooneysorry :) current migration of instance in a server group can race on instance boot even with one scuderler becasue of when we update the instance members https://bugs.launchpad.net/nova/+bug/160025115:43
openstackLaunchpad bug 1600251 in OpenStack Compute (nova) "live migration does not honor server group policy" [High,Fix released]15:43
sean-k-mooneyi was talking about the one bauzas linked so maybe not15:43
tssurya(just back from vacation so excuse me if I am missing something obvious)15:43
gibimriedem: if you have time to work with those patches then please go ahead. and thank you15:43
*** dpawlik has quit IRC15:43
*** dpawlik has joined #openstack-nova15:44
melwittefried: looks like johnthetubaguy already got it :)15:45
*** ccamacho has quit IRC15:48
*** dpawlik has quit IRC15:49
mriedemtssurya: i'll reply in the review15:49
tssuryathanks15:49
*** pcaruana|elisa| has joined #openstack-nova15:50
melwittkashyap: hey, your review on this tcp-based console log patch would be appreciated https://review.openstack.org/57573515:51
kashyapmelwitt: Morning, will look15:52
melwittthanks15:52
openstackgerritFlorian Haas proposed openstack/nova master: Explain cpu_model_extra_flags and nested guest support  https://review.openstack.org/60978815:56
cfriesenfollowing a discussion with mriedem late last Friday, the HPET spec has been updated with something much simpler.  (Turns out no trait is needed after all.)  https://review.openstack.org/#/c/60798915:57
*** READ10 has quit IRC15:57
openstackgerritFlorian Haas proposed openstack/nova stable/rocky: Explain cpu_model_extra_flags and nested guest support  https://review.openstack.org/60978915:58
openstackgerritStephen Finucane proposed openstack/nova master: Fail to live migration if instance has a NUMA topology  https://review.openstack.org/61108815:59
*** pcaruana|elisa| has quit IRC16:00
openstackgerritFlorian Haas proposed openstack/nova stable/queens: Explain cpu_model_extra_flags and nested guest support  https://review.openstack.org/60979016:00
openstackgerritFlorian Haas proposed openstack/nova master: Explain cpu_model_extra_flags and nested guest support  https://review.openstack.org/60978816:04
johnthetubaguymelwitt: there is a first time for everything :)16:05
melwittheh :P16:05
openstackgerritFlorian Haas proposed openstack/nova stable/rocky: Explain cpu_model_extra_flags and nested guest support  https://review.openstack.org/60978916:06
*** READ10 has joined #openstack-nova16:07
openstackgerritFlorian Haas proposed openstack/nova stable/queens: Explain cpu_model_extra_flags and nested guest support  https://review.openstack.org/60979016:07
*** READ10 has quit IRC16:07
efriedmelwitt, johnthetubaguy: Thanks!16:09
openstackgerritsean mooney proposed openstack/nova master: harden placement init under wsgi  https://review.openstack.org/61003416:10
*** lpetrut has quit IRC16:16
*** dave-mccowan has joined #openstack-nova16:23
*** dtantsur|brb is now known as dtantsur16:26
*** sahid has quit IRC16:33
melwittjohnthetubaguy: hey, thanks for the review! this is the corresponding reno update if you wouldn't mind https://review.openstack.org/61317416:34
johnthetubaguymelwitt: ah, missed that, looking16:34
melwittthat one has to go direct to stable, in order to update a already release reno16:35
melwittthanks16:35
*** rha has quit IRC16:36
melwittoh, dangit I need to rebase16:36
melwitt[the master change]16:37
melwittI used to have a master branch reno update on top, then learned it has to go direct to stable, so I abandoned the one on master, but that's preventing the approved bottom change from going to the gate16:38
*** rha has joined #openstack-nova16:39
gaudenzCloud someone have a look at https://review.openstack.org/#/c/523006/ ? This fixes a bug that has been around for a few releases and I made an effort to rebase it just a few days ago. Now all tests pass again. The bug is about resource usage calculations flapping between what nova calcualtes and what libvirt reports.16:39
gaudenzIt would be really nice if this could finally be approved and merged. If you have any comments I'm willing to do further work on the patch until it's acceptable.16:40
*** udesale has quit IRC16:43
*** tbachman has quit IRC16:45
melwitthm, looks like I already orphaned the master branch reno, so not sure what's going on16:45
melwitta-ha, it's in the gate queue, just hasn't printed "starting gate jobs" on the review yet16:47
*** tbachman has joined #openstack-nova16:50
johnthetubaguymelwitt: I just assumed I read the related changes the wrong way around again, phew16:51
melwittjohnthetubaguy: nope, you're good! thanks for reviewing those16:51
johnthetubaguyno worries, happened to just review the new console docs sfinucan wrote up, which was handy (but also need fixing due that console-auth changes)16:52
melwittyeah, I saw you're in console mode today16:52
*** tbachman_ has joined #openstack-nova16:53
*** tbachman has quit IRC16:54
*** tbachman_ is now known as tbachman16:54
openstackgerritMatthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice  https://review.openstack.org/52634716:54
openstackgerritMatthew Booth proposed openstack/nova master: Add local_root to block_device_info  https://review.openstack.org/52902916:54
*** mvkr has quit IRC16:55
openstackgerritsean mooney proposed openstack/nova master: harden placement init under wsgi  https://review.openstack.org/61003416:56
sean-k-mooneysorry for the noise ^ should now be ready for review16:56
openstackgerritmelanie witt proposed openstack/nova stable/rocky: Add nova-status upgrade check for consoles  https://review.openstack.org/61430216:56
*** itlinux has quit IRC16:57
melwittmriedem: I'm not sure if I did that right. if you find other problems in it, let me know and I'll fix at the same time ^16:58
kashyapmelwitt: Just added a bunch of comments in that review after doing some tests with TCP consoles.17:00
kashyapIn short, we need to ensure the log file for the TCP char device is created at domain (Nova instance) define time.17:00
melwittkashyap: ok, I will take a look. thanks for looking into it17:01
kashyapmelwitt: FWIW, I also double-checked it w/ a libvirt upstream dev who knows this area17:01
kashyapWill see if I can punch any other holes in this tomm / later this week17:01
* kashyap needs to head out now17:01
mriedemmelwitt: looks ok17:04
sean-k-mooneystephenfin: jaypipes  can ye take a look at https://review.openstack.org/#/q/topic:bug/1734320+(status:open) when you have time. i would like to try and include these in the next release of os-vif on thusday17:05
*** helenafm has quit IRC17:05
sean-k-mooneyother core that feel comfortable reivewint os-vif are also welcome17:05
openstackgerritMatthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice  https://review.openstack.org/52634717:06
openstackgerritMatthew Booth proposed openstack/nova master: Add local_root to block_device_info  https://review.openstack.org/52902917:06
openstackgerritMatthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume  https://review.openstack.org/52836317:06
openstackgerritMatthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info  https://review.openstack.org/52932817:06
openstackgerritMatthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info  https://review.openstack.org/52932917:06
openstackgerritMatthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config  https://review.openstack.org/53078617:06
openstackgerritMatthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume  https://review.openstack.org/53078717:06
openstackgerritMatthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver  https://review.openstack.org/52938017:06
melwittmriedem: thanks. I wasn't sure if there were any things I missed about backporting a thing that says "Added in Stein"17:06
jaypipessean-k-mooney: will try.17:06
jaypipessean-k-mooney: meeting hell day.17:06
sean-k-mooneyjaypipes: understood. if it does not make it we can do another release when its ready17:07
*** itlinux has joined #openstack-nova17:07
mriedemmelwitt: you're handling it the same as i did for this https://github.com/openstack/nova/commit/daac9a69500c318b5e9d94be0031a1c9506d0340#diff-019a63548bbf8607e233ccbc6a85b0fc17:07
jaypipessean-k-mooney: why is the next release of os-vif on thursday?17:08
melwittmriedem: a-ha, sweet. I managed to not notice it in the upgrade checks list in the file that there was another that said "also useful going back". thanks17:09
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Pass request_spec from compute to cell conductor on reschedule  https://review.openstack.org/58241717:09
openstackgerritMatt Riedemann proposed openstack/nova master: Remove legacy request spec compat code from API  https://review.openstack.org/61430917:09
sean-k-mooneyjaypipes: thursday is m1 i think so we have to do a release17:10
mriedemthat was last week17:10
mriedemderp17:10
sean-k-mooneyjaypipes: atleast that is what i got form https://review.openstack.org/#/c/612844/17:10
sean-k-mooneymriedem: m1?17:10
mriedemhttps://wiki.openstack.org/wiki/Nova/Stein_Release_Schedule17:11
mriedemOct 25: s-1 milestone17:11
sean-k-mooneyoh your right then why the november first deadline in https://review.openstack.org/#/c/612844/17:11
mriedembauzas made it up i guess17:11
mriedemor is nov 1 related to some cve deadline?17:12
sean-k-mooneymriedem: not that im aware of the cve fix was not meged yet it was the patch sets that i was asking for review on17:13
sean-k-mooneyin anycase i cant review my own patches and i would like to get as much of a mitigation into the next release as we can. if it does not make 1.12.0 however we can do a 1.12.1 easilly enough before the end of the year17:15
sean-k-mooneyim just going to grab some food but if people do review and want any change ill priotise them when i get back/when the comments are left17:16
sean-k-mooneybrb17:16
bauzasmriedem: I made what ? :)17:19
*** tssurya has quit IRC17:19
bauzasholidays, internal stuff, french situation ? :p17:19
*** lpetrut has joined #openstack-nova17:23
*** ccamacho has joined #openstack-nova17:23
stephenfinjohnthetubaguy: Would you be averse to me addressing your comments in a follow-up? That patch is already quite huge and I'm going to lose my hard-earned +2s on the follow-ups https://review.openstack.org/#/c/60614817:24
stephenfinAlso, does anyone have Visio? We're going to need someone to modify this diagram for us https://docs.openstack.org/nova/queens/_images/SCH_5009_V00_NUAC-VNC_OpenStack.png17:27
*** ttsiouts has quit IRC17:28
*** ttsiouts has joined #openstack-nova17:29
*** derekh has quit IRC17:30
melwittVisio, what a blast from the past17:32
*** ttsiouts has quit IRC17:33
openstackgerritJack Ding proposed openstack/nova master: Add I/O Semaphore to limit concurrent disk ops  https://review.openstack.org/60918017:33
stephenfinIn fairness, it's an damn fine tool (as with most of Office). I've yet to find anything as good17:36
openstackgerritmelanie witt proposed openstack/nova stable/rocky: Add nova-status upgrade check for consoles  https://review.openstack.org/61430217:36
melwittyeah, not criticizing it. just haven't heard or read the word Visio in several years17:37
melwitt"it sounds familiar..."17:37
*** panda is now known as panda|off17:37
sean-k-mooneystephenfin: png files are not normally editably in visio17:53
sean-k-mooneydo we have teh soruce file17:53
stephenfinWe do17:53
stephenfinIn the same folder17:53
sean-k-mooneyoh ok cool.17:53
sean-k-mooneyi dont see any visio files here https://docs.openstack.org/nova/queens/_images/17:54
stephenfinI don't imagine they're published17:54
sean-k-mooneyoh you mean its in the git repo that makes sense17:55
stephenfinhttps://github.com/openstack/nova/tree/master/doc/source/admin/figures17:55
sean-k-mooneyah vsd files i rememebr those17:56
*** dtantsur is now known as dtantsur|afk17:57
mriedembauzas: we were trying to figure out where you came up with the nov 1 deadline in https://review.openstack.org/#/c/612844/18:02
*** dpawlik has joined #openstack-nova18:03
bauzasah ok18:05
* bauzas calls it a day18:06
sean-k-mooneyreading the mailing list post http://lists.openstack.org/pipermail/openstack-dev/2018-October/135689.html in the commit message we should have don it last week instead18:06
openstackgerritStephen Finucane proposed openstack/nova master: Address nits on I08991796aaced2abc824f608108c0c786181eb65  https://review.openstack.org/61432218:06
openstackgerritStephen Finucane proposed openstack/nova master: Address nits on I1f1fa1d0f79bec5a4101e03bc2d43ba581dd35a0  https://review.openstack.org/61432318:06
stephenfinjohnthetubaguy: Done those as follow ups https://review.openstack.org/614322 https://review.openstack.org/61432318:07
sean-k-mooneyanyway os-vif does use cycle-with-intermediary so we can have more releases then milestones. the porposed change is just saying we should have at least 1 per milestone18:08
mriedemyes i just commented again, i don't see a reason to wait18:08
stephenfinmriedem, melwitt: If either of you fancy reviewing some console'y docs, I'd sure like to get these in https://review.openstack.org/#/q/topic:console-doc+(status:open+OR+status:merged)18:08
*** jpena is now known as jpena|off18:10
sean-k-mooneymriedem: cool sounds fine to me. i would still prefer to merge sooner rather then later but  ill just propose a 1.12.1 when they are ready rather then rushing for review feedback18:11
*** dpawlik has quit IRC18:20
*** tbachman has quit IRC18:28
*** itlinux has quit IRC18:29
*** lpetrut has quit IRC18:29
*** dpawlik has joined #openstack-nova18:36
*** dpawlik has quit IRC18:40
*** tbachman has joined #openstack-nova18:46
*** tbachman_ has joined #openstack-nova18:49
*** tbachman has quit IRC18:51
*** tbachman_ is now known as tbachman18:51
*** jmlowe has quit IRC19:07
*** tbachman has quit IRC19:22
*** jmlowe has joined #openstack-nova19:48
*** cdent has quit IRC19:56
openstackgerritMerged openstack/nova master: Fix test bug when host doesn't have /etc/machine-id  https://review.openstack.org/61271720:11
openstackgerritMerged openstack/nova master: Cleanups for the scheduler code  https://review.openstack.org/52934320:11
openstackgerritMerged openstack/nova master: PowerVM: update_provider_tree() (compatible)  https://review.openstack.org/46856020:11
openstackgerritMerged openstack/nova master: tox: Stop build *all* docs in 'docs'  https://review.openstack.org/61364520:11
*** pcaruana|elisa| has joined #openstack-nova20:12
*** rnoriega has quit IRC20:13
*** rnoriega has joined #openstack-nova20:14
*** ralonsoh has quit IRC20:27
*** spatel has joined #openstack-nova20:29
*** pcaruana|elisa| has quit IRC20:32
*** imacdonn has quit IRC20:33
*** priteau has quit IRC20:33
*** fghaas has quit IRC20:34
*** imacdonn has joined #openstack-nova20:34
*** mvkr has joined #openstack-nova20:37
openstackgerritJay Pipes proposed openstack/nova master: prevent common kwargs from glance client failure  https://review.openstack.org/61435120:51
jaypipeslyarwood: you were interested in a previous commit that addressed ^^20:52
*** tssurya has joined #openstack-nova20:58
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Cross-cell resize  https://review.openstack.org/60393021:01
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellWeigher  https://review.openstack.org/61435321:01
*** erlon has quit IRC21:02
*** spatel has quit IRC21:12
openstackgerritMerged openstack/nova master: quota: remove unused code  https://review.openstack.org/61312721:15
*** fghaas has joined #openstack-nova21:18
*** tbachman has joined #openstack-nova21:20
*** eharney has quit IRC21:21
openstackgerritMerged openstack/nova master: quota: remove unused Quota driver methods  https://review.openstack.org/61312821:24
openstackgerritMatt Riedemann proposed openstack/nova master: Update instance.availability_zone during live migration  https://review.openstack.org/56770121:27
*** tbachman has quit IRC21:27
*** mriedem has quit IRC21:30
*** awaugama has quit IRC21:32
*** fghaas has quit IRC21:42
*** moshele has joined #openstack-nova21:44
*** moshele has quit IRC21:51
*** munimeha1 has quit IRC21:55
*** ttsiouts has joined #openstack-nova21:55
*** ttsiouts has quit IRC22:04
*** ttsiouts has joined #openstack-nova22:05
*** david-lyle has quit IRC22:08
*** ttsiouts has quit IRC22:10
*** dklyle has joined #openstack-nova22:28
*** tbachman has joined #openstack-nova22:30
*** erlon has joined #openstack-nova22:35
openstackgerritMerged openstack/nova master: Simplify logic in get_enforcer  https://review.openstack.org/53100822:43
openstackgerritMerged openstack/nova master: Fix resource tracker updates during instance evacuation  https://review.openstack.org/60042522:44
openstackgerritMerged openstack/nova master: Add nova-status upgrade check for consoles  https://review.openstack.org/61121422:44
openstackgerritMerged openstack/nova stable/rocky: Add rolling upgrade info to enable_consoleauth workaround reno  https://review.openstack.org/61317422:44
openstackgerritMerged openstack/nova master: libvirt: Avoid setting MTU during live migration if unset  https://review.openstack.org/61400822:48
cfriesendo I need to install something extra to get support for placement in the "openstack" CLI?22:48
melwittyes, osc-placement package22:51
*** tssurya has quit IRC22:51
melwittit's the osc plugin for placement22:51
cfriesenwhat determines the amount of disk available in placement?  "openstack hypervisor show" is giving a "local_gb" of 39, but when I try to boot an instance placement is saying there's only 19GB available.22:51
cfriesenthanks for the plugin info22:52
melwittthis will show you the opposite, usage https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-usage-show22:55
melwittand to see capacity (total possible) https://docs.openstack.org/osc-placement/latest/cli/index.html#resource-provider-inventory-show22:56
melwittbasic approach I use is compare the cli docs to the api-ref to find what I need https://developer.openstack.org/api-ref/placement/?expanded=list-resource-provider-usages-detail,show-resource-provider-inventory-detail#list-resource-provider-allocations22:57
melwittnote that the plugin doesn't yet support microversion > 1.17 I think22:57
*** tonyb has joined #openstack-nova23:03
*** mlavalle has quit IRC23:32
*** erlon has quit IRC23:35
openstackgerritJack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots  https://review.openstack.org/60384423:38
*** erlon has joined #openstack-nova23:48

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