| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-26 | |||
| 16:07:33 | wolverineav | stephenfin: ah. i'm fairly certain we don't. i'll confirm in a bit. | |
| 16:07:44 | wolverineav | thanks sahid stephenfin! | |
| 16:07:52 | stephenfin | wolverineav: Anytime! | |
| 16:18:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: General policy sample file for placement https://review.openstack.org/524425 | |
| 17:36:52 | openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Showing some strange behavior https://review.openstack.org/564541 | |
| 18:43:27 | melwitt | mriedem: do we know if anyone is working on a spec amendment for https://review.openstack.org/#/q/topic:bp/nova-validate-certificates ? | |
| 18:44:28 | cdent | nice blog post mriedem | |
| 18:45:32 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: resource requests for nested providers https://review.openstack.org/554529 | |
| 18:48:27 | dansmith | melwitt: yeah I think she said someone was | |
| 18:48:29 | mriedem | melwitt: i don't, i asked about it last week | |
| 18:48:31 | dansmith | peter? | |
| 18:48:34 | mriedem | oh right | |
| 18:48:38 | mriedem | cdent: oh did they post it? | |
| 18:48:56 | cdent | mriedem: showed up in my planet feed | |
| 18:48:59 | openstackgerrit | Jay Pipes proposed openstack/nova master: support multiple member_of qparams https://review.openstack.org/561315 | |
| 18:49:07 | melwitt | dansmith: okay, thanks. I thought I had read something like but second guessed it | |
| 18:57:41 | melwitt | esberglu: I'm going to move your blueprint out of the runway for now since the series is on hold while the CI is worked on. I'll put it at the front of the queue if we expect the CI to be adjusted soon | |
| 19:02:06 | esberglu | melwitt: ack. Working on the CI pipeline now. I'm hoping to get the scenario testing up early next week, but depends on what kind of roadblocks I hit | |
| 19:04:17 | melwitt | esberglu: k, sounds good. keep us posted | |
| 19:09:15 | melwitt | naichuans: hi, your blueprint https://blueprints.launchpad.net/nova/+spec/xenapi-image-handler-option-improvement has been added to a review runway. I know you have been active on the patch series already but please let us know if now is not a good time to quickly iterate on the series | |
| 19:09:56 | melwitt | jianghuaw_: ^ | |
| 19:09:58 | openstackgerrit | Eric Fried proposed openstack/nova master: Base test module/class for functional placement db https://review.openstack.org/564590 | |
| 19:11:12 | mriedem | channel topic needs to be updated | |
| 19:12:34 | melwitt | I was thinking I can probably remove "Add host/hostId to instance action events API" too since it's all approved but maybe should wait until merge | |
| 19:13:25 | mriedem | that still needs the novaclient change | |
| 19:13:45 | melwitt | okay, so keep it until novaclient done | |
| 19:14:12 | melwitt | thanks | |
| 19:14:23 | mriedem | i think so, i assume yikun_ can crank out the novaclient change in the next day | |
| 19:14:30 | mriedem | if takashin doesn't beat him to it | |
| 19:14:47 | melwitt | yeah, that's cool. I wasn't sure if the novaclient thing was going to be off-runway but makes sense to be in-runway | |
| 19:15:04 | mriedem | like off broadway? | |
| 19:15:11 | melwitt | yes exactly like that | |
| 19:15:17 | mriedem | fwiw i always waited to complete a bp when the novaclient changes were done | |
| 19:15:30 | mriedem | helped remind me to pester people to do them | |
| 19:15:49 | melwitt | makes sense | |
| 19:21:27 | openstackgerrit | Eric Fried proposed openstack/nova master: Base test module/class for functional placement db https://review.openstack.org/564590 | |
| 19:21:28 | openstackgerrit | Eric Fried proposed openstack/nova master: Use test_base symbols directly https://review.openstack.org/564592 | |
| 20:15:06 | Sundar | Hi edleafe | |
| 20:15:18 | mordred | efried: I just pushed up an update to https://review.openstack.org/#/c/564299/ that should make it work - and also the previous patch has been updated with a co-authored-by for you :) | |
| 20:16:08 | efried | mordred: I already +1'd the latter. Thanks for that. | |
| 20:16:18 | mordred | \o/ | |
| 20:16:58 | mordred | efried: thanks, btw, for your excellent tests of the version normalization code - theymade adding testing here much easier | |
| 20:17:12 | efried | oh, good. | |
| 20:17:13 | openstackgerrit | Dan Smith proposed openstack/nova master: Make service all-cells min version helper use scatter-gather https://review.openstack.org/539590 | |
| 20:17:14 | openstackgerrit | Dan Smith proposed openstack/nova master: Make host_manager use scatter-gather and ignore down cells https://review.openstack.org/539617 | |
| 20:18:34 | jaypipes | stephenfin: fyi, finally getting around to reviewing the numa-aware-vswitches spec again. | |
| 20:30:27 | efried | mordred: Reviewed | |
| 20:33:05 | mordred | efried: thank you sir | |
| 20:36:46 | edleafe | Sundar: hi - was away from my desk for a bit | |
| 20:38:56 | Sundar | edlefe, NP. Thanks for reaching out | |
| 20:39:01 | Sundar | *edleafe | |
| 20:39:43 | Sundar | I am trying to understand why nested Resource Provider support is not usable for Cyborg in Rocky. I understand it is incomplete. But could you give me some details or pointers? | |
| 20:41:06 | edleafe | Sundar: here's the current work for getting NRPs in allocation candidates: https://review.openstack.org/#/q/topic:bp/nested-resource-providers-allocation-candidates+(status:open+OR+status:merged) | |
| 20:42:01 | openstackgerrit | Merged openstack/nova master: Handle deprecation of inspect.getargspec https://review.openstack.org/530777 | |
| 20:42:09 | openstackgerrit | Merged openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145 | |
| 20:43:10 | edleafe | Sundar: so until that series is complete, we can't really begin to use it | |
| 20:47:19 | Sundar | edleafe: It seems to me that https://review.openstack.org/#/c/554529/ is the key one. Others relate to nested+shared RPs etc., which may not be of immediate relevance to Cyborg. Does that sound right? | |
| 20:47:28 | melwitt | takashin: hi, have you seen this ML thread about your blueprint bp/list-show-all-server-migration-types http://lists.openstack.org/pipermail/openstack-dev/2018-April/129736.html ? your response would be appreciated | |
| 20:48:32 | edleafe | Sundar: it's difficult to say which will impact Cyborg. | |
| 20:49:05 | edleafe | Sundar: is 1400 UTC a good time for you? We have the scheduler subteam meeting then | |
| 20:49:13 | edleafe | Sundar: on Mondays | |
| 20:50:15 | melwitt | nova meeting in 10 minutes | |
| 20:50:30 | Sundar | edleafe: Sure, I can make it at 7 am PDT Monday (I am in SF Bay Area). | |
| 20:50:46 | takashin | melwitt: Yes. | |
| 20:51:19 | Sundar | edleafe: Also, this series does not seem to address nested RPs + granular resource requests. Where is that tracked? | |
| 20:53:10 | edleafe | Sundar: hmmm... efried, are you around? | |
| 20:53:24 | melwitt | takashin: okay, cool. if you could send a response to the mailing list, it would be helpful | |
| 20:53:31 | edleafe | Sundar: https://wiki.openstack.org/wiki/Meetings/NovaScheduler | |
| 20:53:48 | takashin | melwitt: okay. | |
| 20:53:54 | melwitt | thanks | |
| 20:59:25 | Sundar | edleafe: Thanks! | |
| 21:02:37 | efried | edleafe: sup | |
| 21:03:43 | edleafe | Can you shed some light on Sundar's question above about nested RPs and granular requests? | |
| 21:05:34 | efried | Sundar: https://review.openstack.org/#/c/554529/ is the first step in getting nested to work in GET /allocation_candidates. Without it, nested resource providers aren't usable. | |
| 21:06:11 | efried | Sundar: Granular will happen here: https://review.openstack.org/#/q/topic:bp/granular-resource-requests+status:open | |
| 21:06:59 | efried | Sundar: There are some who think granular should wait until nested is done. I don't necessarily agree, but we'll see how it shakes out. | |
| 21:07:25 | Sundar | efried: Got it. Thanks! | |
| 21:12:55 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Deprecate rule:context_is_admin https://review.openstack.org/564349 | |
| 21:16:03 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: General policy sample file for placement https://review.openstack.org/524425 | |
| 21:17:07 | openstackgerrit | Lance Bragstad proposed openstack/nova master: Deprecate rule:context_is_admin https://review.openstack.org/564349 | |
| 21:17:43 | lbragstad | mriedem: two tests are still failing and i'm not quite sure why, it seems unrelated but that's probably not the case ^ | |
| 21:17:59 | lbragstad | otherwise - i attempted to simplify it a bit more | |
| 21:18:24 | Sundar | edleafe, efried: Apparently there is a convention/agreement that only the entity that created an RP can publish traits on it (apart from administrators). If so, Cyborg cannot publish traits on the compute node RP, as Nova created it. Can you throw some light on this? | |
| 21:18:37 | lbragstad | i think nova was just using rule:context_is_admin to set is_admin on the context object (which makes sense given the name) | |
| 21:18:49 | lbragstad | so - in order to deprecate it, that has to live somewhere | |
| 21:18:56 | openstackgerrit | Adam Harwell proposed openstack/nova stable/pike: Add support for listing hosts in cellv2 https://review.openstack.org/563797 | |
| 21:18:57 | mriedem | lbragstad: ok, i've made progress on the granular placement policy patch, except it breaks all placement-related tests now... | |
| 21:19:07 | lbragstad | so i put it in the context object itself | |
| 21:19:26 | lbragstad | and i have it checking for roles set by keystonemiddleware, which should hopefully make things more clear | |
| 21:20:20 | efried | Sundar: That convention/agreement was reversed on Friday in Dublin. | |
| 21:20:48 | efried | Sundar: You can put traits on the compute node RP if you want, as long as they're not traits that compute thinks it "owns". | |
| 21:21:10 | efried | Sundar: but... why would you want to do that? I would have expected you to want to put traits on the child providers representing your devices. | |
| 21:22:11 | edleafe | efried: I think that the virt driver would be the "creator", since it reports its presence | |
| 21:22:22 | edleafe | Cyborg would be an outside agent in this case | |
| 21:23:03 | Sundar | efried: Yes, I'd prefer using nRPs. But edleafe has advised us not to rely upon that in Rocky. So, we are creating a backup plan to apply the traits on the compute node RP instead. That in turn has consequences for supporting multiple device types on the same node. It is getting a bit complicated. | |
| 21:23:14 | efried | edleafe: Yes, but we decided to allow merging of traits supplied by outside parties. | |
| 21:23:57 | efried | Sundar: Yeah, the only way you could reliably support multiple device types on the same provider would be via custom resource classes. And you would then have to unwind that model when nrp is in place. | |
| 21:26:24 | edleafe | efried: that was why I advised Cyborg to wait until NRP was working. | |
| 21:26:58 | edleafe | efried: do you have an educated guess as to how much of NRP will make it into Rocky? | |
| 21:27:44 | efried | edleafe: It's really binary: will nrp-in-alloc-cands make it or not. | |