Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-30
21:47:58 kukacz mriedem: thanks! looking into that
21:48:17 dansmith mriedem: is that a generation mismatch thing?
21:48:21 mriedem dansmith: yup
21:48:28 cfriesen mriedem: does a rebuild in the boot-from-vol case rebuild any of the volume connection stuff in a way that wouldn't happen on a stop/start? if so then maybe we'd only want to fail if we try to rebuild to a different image
21:48:44 dansmith mriedem: so we're not retrying those in the reportclient I guess?
21:48:49 mriedem dansmith: not in put_allocations
21:48:56 mriedem which is what the migrate task uses
21:49:03 dansmith ack okay
21:49:07 mriedem the claim_resources method in the reportclient is the thing the scheduler calls and that does the double up and retry logic
21:49:15 dansmith okay
21:49:22 dansmith mriedem: you gots a bug yet?
21:49:28 mriedem so we could hacky sack a bit and have migrate call claim_resources but pass a "don't double things"
21:49:35 mriedem dansmith: yeah https://bugs.launchpad.net/nova/+bug/1728722
21:49:36 openstack Launchpad bug 1728722 in OpenStack Compute (nova) "Resize test fails in conductor during migration/instance allocation swap: "Unable to replace resource claim on source host"" [High,Triaged]
21:50:02 dansmith mriedem: nah, we can just make a wrapper for put that does it I think and refactor a bit
21:50:12 mriedem that's what i started doing,
21:50:17 dansmith mriedem: I'll work on that tomorrow
21:50:39 mriedem but then you end up copying most of the put_allocations method from placement, and have to get the response back for the status_code
21:50:45 kukacz mriedem: which release was this vendordata feature introduced in? Pike?
21:50:47 mriedem and reportclient.put doesn't have the safe_connect on it
21:50:51 mriedem but i think we said we could remove that...
21:51:00 mriedem kukacz: newton i think, and enhanced in ocata
21:51:00 dansmith mriedem: I think we want the wrapper in reportclient though
21:51:09 mriedem dansmith: ah yeah that would work
21:51:14 dansmith mriedem: I would think we could make claim_resources use the wrapped version too
21:51:16 dansmith to avoid that duplication
21:51:31 dansmith anything calling put_allocations shouldn't have to worry about generation retries
21:51:59 openstackgerrit Merged openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764
21:52:02 kukacz mriedem: great, seems this could help our usecase. thanks a lot!
21:52:22 mriedem kukacz: yw
21:54:16 melwitt mriedem: I'm not seeing where https://review.openstack.org/#/c/508163 ran the test_volume_backed_live_migration test in the results
21:55:03 mriedem melwitt: http://logs.openstack.org/63/508163/4/check/legacy-tempest-dsvm-multinode-live-migration/633c914/job-output.txt.gz#_2017-10-30_20_50_05_842520
21:55:42 melwitt mriedem: oh, thanks. I was looking at the http://logs.openstack.org/63/508163/4/check/legacy-tempest-dsvm-multinode-live-migration/633c914/logs/testr_results.html.gz and I guess it doesn't drill down that much
21:55:47 mriedem nope
21:55:52 jaypipes mriedem: I think a hard-coded server-side retry is what I would go with.
21:55:53 mriedem these are post test hook driven
21:56:01 melwitt oh :\
21:56:14 jaypipes mriedem: but right now, dinner...
21:56:14 mriedem melwitt: looks like i didn't uncomment the part to run it with ceph as the ephemeral backend, but not sure that would make any difference
21:56:22 mriedem https://review.openstack.org/#/c/508163/4/nova/tests/live_migration/hooks/run_tests.sh@67
21:56:48 melwitt oh, I see
21:57:18 melwitt I *think* that doesn't make a difference but we could do it just to be sure
22:00:14 mriedem melwitt: yeah it might given the comment for VIR_MIGRATE_NON_SHARED_INC says "Migration with non-shared storage with incremental disk copy"
22:00:23 mriedem so i guess i'll rev that test patch to see what happens
22:00:41 melwitt yeah, I think that'd be best. I'll keep an eye out for the result
22:01:11 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163
22:02:46 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163
22:02:46 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202
22:03:09 mriedem i don't know why it says it can't merge
22:05:35 melwitt I never figured out why it says that
22:11:23 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
22:15:19 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
22:15:35 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
22:44:11 cdent efried, jaypipes you see https://twitter.com/amitry/status/925112698102255616 ?
22:44:38 efried cdent I don't twit.
22:44:48 efried cdent How do I "ping" this guy?
22:45:01 cdent one moment please
22:45:14 efried (cdent I mean, without joining twitter)
22:45:45 cdent you can use me as a sock puppet if you like
22:46:10 cdent I can give him your email address
22:46:15 cdent if that works for yhou
22:46:19 efried cdent For sure.
22:52:12 mikal mriedem: you still around?
22:54:57 cdent efried: he’s already responded, says he’ll start an email thread
22:55:04 efried noyce, thanks.
22:55:10 efried That twitter thing sure is responsive
22:56:37 cdent do not be tempted
23:15:04 melwitt mriedem: I tried rebasing your tempest change and rechecking and it didn't work. sorry about that :(
23:15:19 melwitt it needs the -W reapplied https://review.openstack.org/#/c/481346/
23:51:34 openstackgerrit melanie witt proposed openstack/nova master: DNM: Run test_volume_backed_live_migration and iscsi test https://review.openstack.org/508163
23:51:34 openstackgerrit melanie witt proposed openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202
23:52:37 melwitt mriedem: there was a real merge conflict from the other ceph test improvement that got merged earlier today. oops
23:57:22 openstackgerrit Eric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151
23:57:23 openstackgerrit Eric Fried proposed openstack/nova master: ResourceRequest.to_querystring() https://review.openstack.org/515811
23:57:23 openstackgerrit Eric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223
23:57:33 efried gibi ^
#openstack-nova - 2017-10-31
00:26:34 openstackgerrit Merged openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514
00:48:30 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
00:51:56 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
00:54:33 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
01:01:53 melwitt mriedem: looks like everything's all good for ceph ephemeral on that patch too
01:26:36 mriedem mikal: not really, but maybe
01:31:33 masber good morning, I wanted to ask about how nova manages numa nodes. My problem is that nova says my instance does not fit in the numa node http://paste.openstack.org/raw/625030/
01:35:13 masber apologise, forgot to mention my image properties --> --property hw_cpu_policy=dedicated --property hw_cpu_thread_policy=isolate
01:36:48 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Fix joins in instance_get_all_by_host https://review.openstack.org/511682
01:36:49 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Fix test_instance_get_all_by_host https://review.openstack.org/516486
01:46:14 masber any idea? I really have no clue of what is wrong... my compute servers have 2 sockets with 14 cpus each which means 28cpu with hyperthreading enabled and memory is 512GB (256GB per numa node), i am asking or 20 cpus and 80GB of RAM so shouldn't be a problem to get those resources? plus I have no other vms taking space so all resources are free
03:32:08 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
04:26:24 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge flavor extensions controller code https://review.openstack.org/516104
04:28:15 gmann alex_xu: ^^ updated
06:18:07 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
06:34:30 openstackgerrit Yikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination https://review.openstack.org/516168
07:56:33 moshele stephenfin: hi can you review https://review.openstack.org/#/c/505613/ ?
08:04:47 alex_xu efried: what is the relationship between different request group, or between non-numbered group and numbered group?
08:10:35 openstackgerrit jichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement https://review.openstack.org/512215
08:13:11 openstackgerrit jichenjc proposed openstack/nova master: [placement]Enhance doc for placement allocation list https://review.openstack.org/511715
08:17:33 openstackgerrit jichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url https://review.openstack.org/511397
08:17:33 openstackgerrit jichenjc proposed openstack/nova master: remove glance usage inside compute https://review.openstack.org/511392
09:13:19 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
09:16:00 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406

Earlier   Later