| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-01 | |||
| 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? | |
| 14:28:34 | mriedem | https://review.openstack.org/#/c/524425/ | |
| 14:28:53 | mriedem | turns out that now that people are starting to roll up to using placement, they actually care about configurable policy for the service :) | |
| 14:30:33 | cdent | mriedem: I saw it briefly this morning before I rolled out but didn’t have a chance to look at it for real yet. I had thought that we were planning to put it off until we actually had some reason to have policy other than “you must be this high to ride any of this ride” to align with the “minimal config” thing? | |
| 14:31:16 | leakypipes | mriedem, superdan, giblet: melwitt did some good sleuthing on the bug report associated with https://review.openstack.org/#/c/518022. I concur with her findings and think the patch is good to go. | |
| 14:31:17 | cdent | superdan: cdenture could be workable, but it’s not quite right enough for me to be all in | |
| 14:31:49 | mriedem | cdent: given it's all admin-only apis i think that's also why we put off configurable policy, | |
| 14:31:55 | mriedem | the issue that someone brought up last night was the admin role, | |
| 14:31:59 | mriedem | they don't have an "admin" role i guess | |
| 14:32:09 | mriedem | so they are hacking around it with a config option | |
| 14:32:11 | mriedem | mgagne: ^ | |
| 14:32:22 | mriedem | this https://gist.github.com/mgagne/b43c1e085c1f1d50bebc054a7d387688 | |
| 14:32:51 | mriedem | anyway, not sure it's high priority, just something to have on the radar | |
| 14:32:55 | cdent | ah | |
| 14:32:56 | cdent | hmmm | |
| 14:32:57 | mriedem | since i totally forgot about this | |
| 14:32:59 | superdan | cdent: hah, okay | |