| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-19 | |||
| 12:45:17 | moshele | jaypipes: hi can you review https://review.openstack.org/#/c/460278/ ? | |
| 12:50:22 | bauzas | alex_xu: jaypipes: sdague: api-ref change related to a bug | |
| 12:50:23 | bauzas | https://review.openstack.org/#/c/472733/ | |
| 12:57:46 | openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Validate behavior of empty devname https://review.openstack.org/396130 | |
| 13:08:28 | alex_xu | bauzas: +2 | |
| 13:21:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform rescue/unrescue instance notifications https://review.openstack.org/385644 | |
| 13:22:46 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: _schedule_instances() supporting a RequestSpec object https://review.openstack.org/469037 | |
| 13:25:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix unshelve asynchronous postconditions typo https://review.openstack.org/475386 | |
| 13:26:16 | jaypipes | moshele: sure, will try my best. need to read all the related papers on that. | |
| 13:27:06 | moshele | jaypipes: thanks, all the ovs patches are merged now :) | |
| 13:47:23 | dommsos | hello | |
| 13:47:36 | dommsos | i am not able to provision vm using salt-cloud and nova as driver | |
| 13:47:51 | dommsos | i am getting this error [ERROR ] There was a query error: SSL exception connecting to https://jp-blr-cloud-01.juspay.net:5000/v2.0/tokens: ("bad handshake: Error([('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')],)",) | |
| 13:48:01 | dommsos | again and again, i looked up over the iternet and cannot find out the solution | |
| 13:48:21 | dommsos | i tried skiping ssl_verification but doesn't seem to be working | |
| 13:48:32 | dommsos | anyone please help | |
| 13:51:54 | sfinucan | dommsos: That's a usage question - you'd be much better placed asking that on #openstack | |
| 13:55:17 | edleafe | Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt | |
| 13:56:29 | artom | dommsos, and honestly that doesn't look like Nova itself | |
| 13:56:57 | artom | For one thing, that's querying Keystone, the identity service | |
| 13:57:18 | artom | For another, the request isn't even getting to keystone itself, it's failing at the SSL handshake step | |
| 13:58:26 | artom | dommsos, so you need SSL help, not OpenStack help :) | |
| 14:06:05 | huanxie | Hi jaypipes, could you help review my patch https://review.openstack.org/#/c/462017/ which is to fix vm lost network connection issue during live migration with XS? I made this new patch which to hide the detailed bridge creating things | |
| 14:14:15 | dommsos | hello | |
| 14:14:31 | dommsos | in openstack channel they told me to contact here | |
| 14:24:38 | jaypipes | huanxie: will try my best. | |
| 14:26:41 | edleafe | bauzas: what you have to look forward to: https://goo.gl/photos/BKks6sPp9czXSSMo6 | |
| 14:26:56 | bauzas | hah | |
| 14:27:13 | bauzas | for the moment, she's way younger :p | |
| 14:27:28 | bauzas | and I still need to help her for homework :/ | |
| 14:27:28 | edleafe | bauzas: that doesn't last :) | |
| 14:27:36 | edleafe | it happens much too quickly! | |
| 14:27:42 | bauzas | hah | |
| 14:27:57 | bauzas | she's actually already asking me when she can drive | |
| 14:28:14 | openstackgerrit | Merged openstack/nova master: Amend api-ref for multiple networks request https://review.openstack.org/472733 | |
| 14:28:20 | bauzas | she's almost 7, I was just shocked to tell her only 8 years last | |
| 14:28:59 | edleafe | bauzas: Mika turns 18 next month, and is off to university in August | |
| 14:29:05 | edleafe | scary | |
| 14:29:19 | bauzas | oh man, I so understand you | |
| 14:39:36 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild-scheduled notification https://review.openstack.org/473929 | |
| 14:44:57 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | |
| 14:48:35 | cfriesen | has anyone ever talked about throttling the number of simultaneous deletion operations? we're seeing problems due to DB timeouts if you delete 100+ instances at the same time. | |
| 14:49:25 | gabor_antal | bauzas would you please review my patch at https://review.openstack.org/#/c/460029/? It already has +2 from sfinucan. | |
| 14:50:26 | bauzas | gabor_antal: sure | |
| 14:52:04 | gabor_antal | bauzas: thanks! :) | |
| 15:08:57 | efried | gabor_antal bauzas Doesn't notify_about_instance_usage send legacy notification? Do we really want that? | |
| 15:09:56 | efried | Whoops, sorry, I'm looking at it backwards. Disregard. | |
| 15:12:17 | efried | Perhaps we should consider making a context manager for versioned notifications, like gibi did for legacy here: https://review.openstack.org/#/c/437222/9/nova/compute/utils.py | |
| 15:16:00 | gibi | efried: that can be useful but I don't think that is a must | |
| 15:18:54 | mriedem | alex_xu: i'm not sure what you're asking for here https://review.openstack.org/#/c/464280/16/nova/api/openstack/compute/schemas/services.py@60 | |
| 15:47:44 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore https://review.openstack.org/474869 | |
| 15:47:45 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-tenant-networks instead of stevedore https://review.openstack.org/474475 | |
| 15:47:45 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463 | |
| 15:47:46 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove multiple create from stevedore https://review.openstack.org/474909 | |
| 15:47:46 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-cloudpipe endpoint instead of stevedore https://review.openstack.org/474116 | |
| 15:47:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-remote-consoles instead of stevedore https://review.openstack.org/474930 | |
| 15:47:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-quota-classes endpoint instead of stevedore https://review.openstack.org/461641 | |
| 15:49:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore https://review.openstack.org/474869 | |
| 15:49:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463 | |
| 15:49:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove multiple create from stevedore https://review.openstack.org/474909 | |
| 15:49:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-remote-consoles instead of stevedore https://review.openstack.org/474930 | |
| 15:50:50 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: produce set of allocation candidates https://review.openstack.org/473377 | |
| 15:50:50 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448 | |
| 15:51:07 | jaypipes | edleafe, dansmith, mriedem, bauzas: ^^ | |
| 15:54:29 | bauzas | jaypipes: roger | |
| 15:54:46 | bauzas | FWIW, I'll *still* be on PTO for 2 days this week | |
| 15:54:55 | bauzas | Tues-Wed | |
| 15:55:27 | bauzas | you're fully granted for making any joke about french people being always on vacations | |
| 15:55:41 | bauzas | I won't get offended | |
| 15:56:26 | jaypipes | bauzas: lol, no worries. enjoy the time off :) | |
| 15:56:50 | bauzas | jaypipes: edleafe: so, let's merge first the above before the scheduler-claims series, right? | |
| 15:57:16 | bauzas | I just want to make sure the above is blocking the other series like nested-rps or scheduler-claims | |
| 15:57:27 | jaypipes | bauzas: that is correct. | |
| 15:57:43 | bauzas | cool, then I'll be reviewing it very soon today | |
| 15:58:25 | bauzas | edleafe: FWIW, I haven't replied to your ML post because I agree with jaypipes's comments | |
| 15:58:38 | bauzas | and mriedem's nits, btw. | |
| 15:58:57 | bauzas | I was confused by you saying that the scheduler was calling the placement API for allocating :) | |
| 15:59:32 | jaypipes | bauzas: it *will* be. but doesn't yet. | |
| 15:59:43 | bauzas | yeah I know | |
| 15:59:53 | bauzas | but I was confused by the "current situation" name :p | |
| 16:02:18 | edleafe | bauzas: yeah, I meant the current *design* we had agreed upon, as compared to the new approach | |
| 16:03:19 | edleafe | bauzas: I had a mental concept of what this would look like when we were done. | |
| 16:03:31 | edleafe | bauzas: I'm not clear on where we're heading now. | |
| 16:06:29 | mriedem | edleafe: it's basically the same except some machinery changes - where filter scheduler calls placement with 1.4 today to get a list of filtered providers based on vcpu/ram/disk, we now call the allocation candidates API | |
| 16:07:05 | mriedem | we use the allocation candidates response to get our list of compute node objects to create HostState objects for filtering/weighing, | |
| 16:07:12 | mriedem | since we don't have concepts like affinity in placement yet | |
| 16:07:30 | mriedem | and we also use the allocation candidates response to PUT the allocation "claim" at the end on selected providers | |
| 16:08:36 | mriedem | so we might allocate some vcpu/memory_mb on a compute node resource provider, and some disk_gb on a shared storage provider | |
| 16:09:00 | edleafe | mriedem: sure, that's fine. But there is more to it than that, as I understood it. Placement will be returning information about the RPs, not just name/UUID. Originally that info was to replace HostState objects, but according to jaypipes today we now need both | |
| 16:09:26 | mriedem | edleafe: i don't know where/when/who ever said that we'd not need HostState objects anymore | |
| 16:09:34 | mriedem | removing their usage in the scheduler would be a huge refactor | |
| 16:09:42 | dansmith | actually, | |
| 16:10:06 | dansmith | I said we'd have to do some monkeywork to adapt what we get into hoststate objects so we could move on without that refactor having to happen synchronously | |
| 16:10:15 | dansmith | so, yeah. | |
| 16:10:53 | mriedem | ok, but we still pass host state objects to the filters and weighers right? and we have affinity/anti-affinity filters that we still need, | |
| 16:11:06 | mriedem | and i think we can agree we don't want to change all of those at the same time | |
| 16:11:23 | dansmith | for now yeah | |
| 16:11:37 | dansmith | eventually I think that refactor will need to happen, | |
| 16:11:40 | edleafe | mriedem: I had proposed adding the sub-resource UUIDs (for PCI, shared disk, etc) to the HostState object. That wasn't possible, I was told, because HostState is going away | |
| 16:11:41 | mriedem | plus the impending destructification of everyone's out of tree filters/weighers at the same time :) | |