| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-30 | |||
| 21:43:07 | mriedem | kukacz: check this out https://docs.openstack.org/nova/latest/user/vendordata.html | |
| 21:43:46 | mriedem | configure nova for force_config_drive=True so every instance gets a config drive, and then you can setup a vendordata v2 service for injecting whatever you need into the guest | |
| 21:46:19 | mriedem | jaypipes: for this todo, we'd want a microversion so the client can detect if it can rely on the server doing the retries or if the client has to do them https://github.com/openstack/nova/blob/965f56d7d2ca1f668f70d24d4dcc20e418bb5b9c/nova/objects/resource_provider.py#L1887 | |
| 21:46:44 | mriedem | jaypipes: but i'm wondering if we want to make that a hard-coded retry value in placement (or configurable), or allow the user to pass in the number of retries (probably not) | |
| 21:47:31 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 21:47:35 | mriedem | or allow the user to request a certain number of retries but have a configurable max in placement, defaulted to 3? | |
| 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 | dansmith | mriedem: I think we want the wrapper in reportclient though | |
| 21:51:00 | mriedem | kukacz: newton i think, and enhanced in ocata | |
| 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 | 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:14 | jaypipes | mriedem: but right now, dinner... | |
| 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: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202 | |
| 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: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: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202 | |
| 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: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: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223 | |
| 23:57:23 | openstackgerrit | Eric Fried proposed openstack/nova master: ResourceRequest.to_querystring() https://review.openstack.org/515811 | |
| 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? | |