Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-17
09:08:52 kashyap Given the complexity of it, and the "head-wrapping" required
09:08:59 johnthetubaguy +1
09:09:28 bauzas gibi: now I need to rebase on top of your last update, but that should be fine
09:09:40 gibi bauzas: cool
09:10:12 bauzas gibi: well, sorry I wasn't clear
09:10:14 johnthetubaguy kashyap: feels like it should work a bit like volume encryption, once its working correctly, but want to get my hands on it first, which is hard given my customer is an Intel only shop right now, this might change things I guess!
09:10:25 johnthetubaguy kashyap: will let you know if I hear any more
09:10:26 bauzas I was working on rebasing tetsuro's branch on top of master
09:11:01 kashyap johnthetubaguy: Sure. (And yeah, the use cases are definitely critical.)
09:11:24 bauzas gibi: most of the conflicts were due to the 1.30 microversion for reshapes
09:12:45 gibi bauzas: no problem. If nested a_c support ends up top of top of consumer_gen support then I'm OK.
09:13:07 bauzas gibi: yup, I just rebased the whole series
09:13:13 gibi bauzas: coolio
09:13:27 bauzas gibi: now, rebasing on top of your latest rev should be trivial hopefully
09:14:31 bauzas gibi: https://review.openstack.org/#/c/583667/ is the top patch of your series, right?
09:14:57 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Use 'virt' as the default machine type for ARMv7 https://review.openstack.org/602592
09:15:15 kashyap bauzas: ^ If you want to put it out of its misery :-)
09:18:55 gibi bauzas: yes, it is
09:21:03 openstackgerrit Sylvain Bauza proposed openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672
09:23:38 bauzas gibi: ^
09:23:44 gibi bauzas: awesome
09:24:15 gibi bauzas: shall I rebase the functional tests for the change or you will do that too?
09:24:24 bauzas gibi: I can try
09:24:41 gibi bauzas: if you haven't started yet, then I can continue with the functionals
09:24:56 bauzas gibi: fair enough
09:25:05 bauzas I'll have to bail out by 11:35
09:25:15 gibi bauzas: then I will jump on the functionals
09:25:17 bauzas so I won't have time to fix this
09:25:31 bauzas wait a sec, I'll amend tetsuro's change based on your good commebnt
09:25:35 bauzas gibi: ^
09:25:38 gibi bauzas: sure
09:30:03 bauzas gibi: good news is that I checked the unittests with the rebase and they're good
09:32:04 kashyap Lately, I've begun doing this when running `git-review`, others too?
09:32:08 kashyap $ git review -R --no-custom-script
09:32:13 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Remove the allocation ratios adjusting logic https://review.openstack.org/602805
09:33:26 openstackgerrit Sylvain Bauza proposed openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672
09:33:38 bauzas gibi: updated change ^
09:33:47 bauzas gibi: you can rebase functional tests on top of this
09:33:56 gibi bauzas: thanks, I will do that
09:47:36 openstackgerrit Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276
10:05:48 bauzas naichuans_: sorry, just saw your reply on the ML
10:06:15 bauzas naichuans_: I'm back but I'm off until 1140UTC
10:06:31 bauzas naichuans_: after that, I think you should be off too
10:06:42 bauzas naichuans_: so let's discuss tomorrow if you want
10:10:28 openstackgerrit Balazs Gibizer proposed openstack/nova master: Functional test for booting with nested resources https://review.openstack.org/527728
10:10:29 openstackgerrit Balazs Gibizer proposed openstack/nova master: Functional test for moving with nested resources https://review.openstack.org/587350
10:12:35 bauzas gibi: just saw ^
10:12:50 bauzas coolio, that will allow me to add a new functional test
10:13:13 gibi bauzas: functional tests passes, but I have to think about what we can consider as enough test coverage
10:16:05 bauzas gibi: we need allocation candidates for creating a new instance
10:16:08 bauzas needed*
10:16:13 bauzas anyway, me is off now
10:39:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840
10:39:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
10:39:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847
10:39:41 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
10:39:43 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459
10:39:43 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
11:08:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Deprecate the unversioned notifications https://review.openstack.org/603079
11:16:45 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Remove the allocation ratios adjusting logic https://review.openstack.org/602805
11:29:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor NeutronFixture https://review.openstack.org/588338
11:31:16 openstackgerrit Merged openstack/nova master: Fix docs and add functional test for AggregateMultiTenancyIsolation https://review.openstack.org/601835
11:31:42 openstackgerrit John Garbutt proposed openstack/nova master: Deprecate ComputeCapabilitiesFilter https://review.openstack.org/603102
11:45:26 openstackgerrit huanhongda proposed openstack/nova stable/pike: Fix bug case by none token context https://review.openstack.org/603044
12:23:03 Roamer` hi, possibly stupid question incoming :) If a company has two different physical datacenters, and an OpenStack setup in each of them, how would it be best to make it possible to (cold-)migrate instances (with attached volumes) between them? Is there a way to migrate an instance between two regions of the same cluster? The two options I've found so far is "openstack server image create" (which
12:23:09 Roamer` creates a zero-byte image for a volume-backed instance, though maybe I've configured something wrong) and cinder backup/restore; are there other options?
12:45:51 bauzas gibi: thanks for the updates
12:46:17 gibi bauzas: you are welcome
12:47:05 gibi bauzas: I think the series works as my bandwidth functional tests worked on top. But I wouldn't be surprised if migrate or evacuate would eventually be found broken
12:47:26 bauzas gibi: for migrate, we at least need to pass allocations
12:47:34 bauzas I have an uploaded change for it
12:47:42 bauzas to the drivers, I meant
12:47:54 gibi bauzas: for VGPU I guess
12:48:10 bauzas alas yes
12:49:00 gibi bauzas: yeah, that make sense. I also have to work on the bandwidth patch to see migrate works or not as-is. So fare I only have boot and delete tests there
12:50:15 bauzas yup, I just wanted to make sure you knew it
12:52:45 gibi VGPU and bandwidth differs in a sense that the physical use of the resource for VGPU happens in the virt driver (i.e. generating the proper xml) while for bandwidth it happens in neutron and os-vif.
12:53:24 gibi so at the moment I assumes that for bandwidth we don't need the allocations in the virt driver. But I very well be wrong in the SRIOV case due to the pci handling there
12:59:24 bauzas gibi: ah good point
13:29:54 mriedem got a clean run on the evacuate integration test https://review.openstack.org/#/c/602174/
13:30:03 mriedem with no mdbooth around to celebrate
13:33:36 mriedem we have something in a runway for stein now https://review.openstack.org/#/c/599276/
13:35:56 gibi mriedem: I've just added the consumer generation support to the runway queue
13:36:08 mriedem is that a blueprint?
13:36:33 mriedem https://blueprints.launchpad.net/nova/+spec/use-nested-allocation-candidates isn't approved
13:36:49 gibi it is part of bp/use-nested-allocation-candidates
13:37:00 bauzas do we started runways ?
13:37:51 gibi mriedem: true, then I remove the patches from the queue and bring up the bp on the Thursday meeting for approval
13:37:59 bauzas mriedem: FWIW https://review.openstack.org/#/c/527728/22/nova/tests/functional/test_servers.py should cover the boot case
13:42:52 kashyap Dear folks, I can't tell _why_ the 'neutron-grenade' job is failing here: https://review.openstack.org/#/c/602592/
13:43:18 kashyap I'm looking at the "ara-report", but I can't find a log file.
13:45:32 bauzas kashyap: http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/job-output.txt.gz
13:45:54 kashyap bauzas: Ah, let me look
13:46:10 bauzas mmm, actually
13:46:24 mriedem http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/logs/devstack-gate-setup-workspace-old.txt
13:47:30 mriedem i've seen that in a couple of other jobs but don't know what's causing it
13:47:59 kashyap http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/job-output.txt.gz#_2018-09-17_10_12_38_647361
13:48:03 bauzas http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/job-output.txt.gz#_2018-09-17_11_27_33_877513
13:48:10 kashyap 2018-09-17 10:12:38.647361 | primary | smoke installed: ----------------------------------------,Error when trying to get requirement for VCS system Command "git config --get-regexp remote\..*\.url" failed with error code 1 in /opt/stack/new/tempest, falling back to uneditable format,Could not determine repository location of /opt/stack
13:48:26 mriedem that's common

Earlier   Later