| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-01 | |||
| 02:11:06 | gmann | mriedem: only API left is cell (v1), which anyway going to be deprecated and removed so i am thinking to skip the schema for this | |
| 02:11:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007 | |
| 02:14:05 | gmann | mriedem: else this BP is completed. | |
| 02:19:13 | gmann | mriedem: we did for deprecated APIs but cell one is not just deprecation case but going to be removed that why we can skip this ? or its easy schema for cell i can implement if you want | |
| 02:28:07 | openstackgerrit | Jack Lu proposed openstack/nova master: Update supported transports for iscsi connector https://review.openstack.org/524443 | |
| 02:53:19 | openstackgerrit | Merged openstack/nova master: Remove old-style quotas code https://review.openstack.org/511689 | |
| 03:01:32 | alex_xu | jaypipes: sir, good evening, do you want to unlock this patch https://review.openstack.org/#/c/517119/? | |
| 03:01:47 | alex_xu | jaypipes: or anything I can help to unlock that patch | |
| 03:58:48 | efried | alex_xu I believe jaypipes said he was working on it locally. | |
| 04:00:09 | efried | alex_xu Never mind - he rebased it, but I think he thought he was done at that point :) | |
| 04:02:41 | openstackgerrit | jichenjc proposed openstack/nova master: Refactor placement version check https://review.openstack.org/512497 | |
| 04:07:56 | openstackgerrit | jichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement https://review.openstack.org/512215 | |
| 04:33:53 | openstackgerrit | Merged openstack/nova stable/pike: doc: fix flavor notes https://review.openstack.org/509438 | |
| 04:43:13 | openstackgerrit | Nguyen Van Trung proposed openstack/nova-specs master: Add doc8 to pep8 check for nova-specs project https://review.openstack.org/519561 | |
| 04:44:00 | alex_xu | efried: cool, then just let us waiting for the update | |
| 04:50:38 | openstackgerrit | Merged openstack/nova master: Do not use “-y” for package install https://review.openstack.org/518238 | |
| 05:02:42 | openstackgerrit | jichenjc proposed openstack/nova master: Add description for resource class creation https://review.openstack.org/508083 | |
| 05:21:01 | openstackgerrit | jichenjc proposed openstack/nova master: remove glance usage inside compute https://review.openstack.org/511392 | |
| 05:21:02 | openstackgerrit | jichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url https://review.openstack.org/511397 | |
| 06:25:36 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904 | |
| 06:31:29 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | |
| 06:45:05 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances https://review.openstack.org/523653 | |
| 07:11:07 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676 | |
| 07:44:55 | nsingh | efried: To create keystoneauth session we are using some parameters like insecure, verify, cacert, cert, and key. And to creating sessionClient we are passing keystoneauth session for nova/glance/cinder. but these conf options from session are not used anywhere while creating session client. so is it only necessary to create session? | |
| 08:13:51 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506 | |
| 08:17:50 | openstackgerrit | Jack Lu proposed openstack/nova master: Update supported transports for iscsi connector https://review.openstack.org/524443 | |
| 08:39:40 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | |
| 08:39:41 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | |
| 08:49:46 | openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | |
| 09:00:17 | openstackgerrit | Erik Berg proposed openstack/nova stable/pike: Fix binary name https://review.openstack.org/524531 | |
| 09:00:19 | openstackgerrit | Merged openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871 | |
| 09:16:51 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273 | |
| 10:11:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | |
| 10:15:01 | stephenfin | Man, Gerrit is slooowww this morning | |
| 10:19:44 | bauzas | stephenfin: probably because of the close week-end ? :p | |
| 10:20:00 | bauzas | or because it's snowing here | |
| 10:20:16 | bauzas | FWIW, resorts are opening this week-end \o/ | |
| 10:20:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | |
| 11:10:47 | openstackgerrit | Merged openstack/nova stable/pike: Fix race in delete allocation in ServerMovingTests https://review.openstack.org/508872 | |
| 12:08:54 | efried | nsingh I have noticed that too - In Nova we go through all this trouble to create a session from conf parameters, and then we create the client for the service which creates its own session. | |
| 12:09:18 | efried | nsingh Eventually the vision (per mordred) is to get rid of the clients altogether and just use the ksa adapter for the API communication. | |
| 12:32:51 | nsingh | efried: thank you for the info. :) | |
| 12:38:12 | ildikov | johnthetubaguy: hi, daily pretty please: https://review.openstack.org/#/c/330285/ | |
| 12:43:52 | openstackgerrit | Merged openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655 | |
| 13:22:22 | fried_rice | Friday! | |
| 13:24:16 | openstackgerrit | Chen Hanxiao proposed openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command https://review.openstack.org/524588 | |
| 13:26:08 | fried_rice | stephenfin You around? | |
| 13:27:34 | fried_rice | or bauzas or gibi or alex_xu ? Easy review, which will merge another patch for a backportable bug: https://review.openstack.org/#/c/524245/ | |
| 13:40:46 | stephenfin | fried_rice: I am. Looking now | |
| 13:40:53 | fried_rice | stephenfin Thanks! | |
| 13:42:14 | finucannot | fried_rice: Are you finding Gerrit slow today too? | |
| 13:42:28 | fried_rice | finucannot Yeah, totally. I asked in -infra, but nobody bit yet. | |
| 13:42:43 | finucannot | So not my network. Good to know. | |
| 13:42:52 | fried_rice | That was an hour ago - more people are awake now. I'll ask again... | |
| 13:43:05 | jaypipes | fried_rice: mornin | |
| 13:43:10 | fried_rice | jaypipes Howdy! | |
| 13:43:19 | cdent | I’ve been out of a bed for a long time, but I make no claims to being awake. | |
| 13:43:28 | leakypipes | fried_rice: what was the thing with alex from earlier? | |
| 13:43:46 | fried_rice | leakypipes He was asking about https://review.openstack.org/#/c/517119/ | |
| 13:44:04 | leakypipes | fried_rice: ah, k | |
| 13:44:06 | fried_rice | I had thought you were making local changes to it, but looking back it seemed you had only rebased it. | |
| 13:44:13 | leakypipes | fried_rice: yeah, I'll work on that this mroning. | |
| 13:44:17 | fried_rice | Cool beans. | |
| 13:45:30 | fried_rice | finucannot FYI apparently we're "waiting for an infra-root to speak some healing words" (per AJaeger) | |
| 13:45:48 | fried_rice | Oh, you're in -infra too. I'll quit relaying. | |
| 13:48:46 | AJaeger | fried_rice: I'm not here in general, continue relaying, please | |
| 13:48:52 | mriedem | gmann_afk: no we don't need to add query param schema for the cells APIs | |
| 13:48:56 | mriedem | those are likely gone in rocky | |
| 13:49:13 | fried_rice | AJaeger But finucannot is in -infra, so he can follow the conversation there. | |
| 13:49:20 | AJaeger | ;) | |
| 14:04:28 | openstackgerrit | Eric Fried proposed openstack/nova stable/pike: Proper error handling by _ensure_resource_provider https://review.openstack.org/524618 | |
| 14:04:55 | fried_rice | mriedem ^ pike cherry-pick, with conflicts. Think I'll wait for zuul before doing the ocata one. | |
| 14:06:15 | mriedem | ok | |
| 14:07:19 | finucannot | lyarwood: I assume you could review this, seeing as you only cherry-picked it (vs. writing it)? https://review.openstack.org/#/c/524157/ | |
| 14:09:05 | mriedem | finucannot: not really how backport reviews work | |
| 14:09:43 | finucannot | mriedem: ok, I was curious about that :) Can you review because you wrote it? Can he review because he backported? | |
| 14:09:44 | mriedem | i must say, however, that whoever did write the original must be very handsome indeed | |
| 14:10:39 | mriedem | done | |
| 14:14:53 | Guest78_ | I'm having issues unshelving an instance that has PCI passthrough aliases associated with it......the wrong hypervisor host is selected. Is there a setting that I'm missing to include PciPassthroughFilter when scheduling for unshelve? | |
| 14:16:56 | mriedem | unshelve should go through the same scheduling as the initial create | |
| 14:17:07 | mriedem | so if you have the filters setup for the initial create properly, unshelve should work the same | |
| 14:17:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027 | |
| 14:18:05 | leakypipes | Guest78_: note that this channel is for development questions, so generally that would be a question for the openstack@ mailing list. However, since it's Friday... if you just want a quick and dirty fix, you can probably muck with the data in the instance_extra table for that shelved instance and change the flavor's PCI requirements manually. | |
| 14:18:06 | Guest78_ | 80',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed: PciDeviceRequestFailed: PCI device request [InstancePCIRequest(alias_name='GeForce1080',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed | |
| 14:18:06 | Guest78_ | That's what I thought but that doesn't seem to be the case: 2017-12-01 13:55:52.843 2450 WARNING nova.scheduler.host_manager [req-b7876409-1149-4f8a-837d-46f0023fed10 6136e80cc67e4759843a5f95f7a9a98b 7618d5a893f04349b9feb2ca541a6fce - default default] Selected host: overcloud-novacompute-0.localdomain failed to consume from instance. Error: PCI device request [InstancePCIRequest(alias_name='GeForce10\ | |
| 14:19:11 | ildikov | mriedem: hi | |
| 14:19:33 | ildikov | mriedem: rebased the new attach patch and ran your test on it too, it looks good so far | |
| 14:19:57 | leakypipes | giblet: ++ | |
| 14:20:04 | leakypipes | giblet: on both the beer and the learning. | |
| 14:20:25 | coffee_cat | giblet: I think it's late enough in the afternoon to get a beer now :) | |
| 14:20:43 | cdent | there’s no beer in the hosue | |
| 14:24:50 | giblet | ahh I'm still in the tea phase for this afternoon | |
| 14:25:49 | openstack | Launchpad bug 1735719 in OpenStack Compute (nova) "Cell_v2 discover_hosts Not Working, after deleted a host from a cell " [Medium,Triaged] | |
| 14:25:49 | mriedem | superdan: just came across this in triage https://bugs.launchpad.net/nova/+bug/1735719 - seems the delete_host command should mark the compute node as mapped=0 so it can be discovered again | |
| 14:25:57 | mriedem | since we don't have a way to directly delete compute node records | |
| 14:26:14 | mriedem | coffee_cat: ack | |
| 14:26:17 | superdan | hrm, will look | |
| 14:27:27 | denture | meh | |
| 14:28:23 | superdan | cdent: cdenture maybe? | |
| 14:28:25 | mriedem | cdent: did you see that patch i started for some policy rules/file stuff for placement? how much do you remember of all that when laski was doing it a few releases ago? | |