| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-29 | |||
| 14:18:34 | mriedem | let's recheck it a few times | |
| 14:20:02 | bhagyashris | mriedem: ok thank you for review. you mean to depricate the create backup api and newly introduce deleteBackup api will delete the backup using the rotation parameter | |
| 14:21:16 | mriedem | claudiub: can you update this quick? https://review.openstack.org/#/c/557377/ | |
| 14:21:46 | mriedem | bhagyashris: no, the alternative is to simply deprecate the createBackup API, and not introduce a new deleteBackups API | |
| 14:22:07 | mriedem | since createBackup can all be done using orchestration of existing nova/glance APIs | |
| 14:22:33 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Filter resource providers by forbidden traits in db https://review.openstack.org/556472 | |
| 14:22:34 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Parse forbidden traits in query strings https://review.openstack.org/556819 | |
| 14:22:34 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Filter allocation candidates by forbidden traits in db https://review.openstack.org/556660 | |
| 14:22:35 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Support forbidden traits in API https://review.openstack.org/556820 | |
| 14:24:15 | jaypipes | mriedem: amen, brother. | |
| 14:25:01 | mriedem | makes me curious when/why createBackup was added | |
| 14:25:12 | mriedem | but was likely so long ago that any blueprint for it has little to no detail | |
| 14:25:22 | jaypipes | mriedem: correct. | |
| 14:26:00 | openstackgerrit | Claudiu Belu proposed openstack/nova master: vmware: Fixes _detach_instance_volumes method https://review.openstack.org/557377 | |
| 14:27:15 | bhagyashris | mriedem: ok i am not getting this: cleanup the backups using a rotation value controlled client-side | |
| 14:27:49 | mriedem | bhagyashris: createBackup can be totally done using existing compute and image APIs today, without the actual createBackup API, | |
| 14:28:02 | mriedem | i can snapshot an instance with specific metadata to indicate it's a backup | |
| 14:28:12 | dansmith | alex_xu_: if that "what are you saying" was a joke, that was excellent :P | |
| 14:28:16 | mriedem | and then i can, as a tenant user, delete older backup snapshots in glance | |
| 14:28:57 | mriedem | bhagyashris: my point being, you as the tenant (or script, or external service, whatever) can do all of the createBackup stuff without the compute API (nova) doing it for you | |
| 14:29:17 | alex_xu_ | dansmith: I'm good at pretend to not understand what you guys saying | |
| 14:29:24 | dansmith | haha | |
| 14:29:35 | bauzas | dansmith: again, one comment on https://review.openstack.org/#/c/544729/16 but not holding me to +W | |
| 14:29:53 | bauzas | dansmith: tl;dr: I guess we're talking of aggregate UUIDs, not names | |
| 14:30:11 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Support forbidden traits in API https://review.openstack.org/556820 | |
| 14:30:37 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527 | |
| 14:30:38 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update the cells FAQs and scheduler maintenance docs. https://review.openstack.org/556459 | |
| 14:30:38 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add --enable and --disable options to nova-manage update_cell https://review.openstack.org/555416 | |
| 14:30:59 | mriedem | bauzas: good point | |
| 14:31:17 | bauzas | I don't want to throw the whole series | |
| 14:31:23 | bauzas | since the code will still work | |
| 14:31:31 | bauzas | at least until that point | |
| 14:31:36 | mriedem | if we wanted to use a ListOfUUIDsField, we'd want to do that now | |
| 14:31:45 | dansmith | bauzas: list of uuids is not enough | |
| 14:31:48 | bauzas | mriedem: not really | |
| 14:31:56 | dansmith | bauzas: which is why I didn't create one | |
| 14:31:57 | bauzas | mriedem: because of the AND OR | |
| 14:32:03 | dansmith | we need a list of list of uuids, | |
| 14:32:03 | bauzas | dansmith: yeah, I know | |
| 14:32:09 | bauzas | for me, it's just a docstring correction to make | |
| 14:32:12 | dansmith | which I think is kinda nuts | |
| 14:32:16 | mriedem | oh right, the comma | |
| 14:32:22 | bauzas | I'm fine with keeping a list of strings as of noxw | |
| 14:32:37 | dansmith | so these are "aggregate requests" and depend on the placement schema (which is strict) to handle it | |
| 14:32:39 | bauzas | but the fact that we will have strings made of UUIDs and commas | |
| 14:32:58 | bauzas | so, I just want to clarify the docstring, that's it | |
| 14:34:03 | dansmith | bauzas: yeah, fair neough | |
| 14:34:20 | dansmith | bauzas: the names make it easier to reason about the example, but I can make a point of explaining what they are | |
| 14:38:29 | bhagyashris | mriedem: But giving provision through nova API to createBackup of server backup is more user friendly | |
| 14:39:07 | mriedem | bhagyashris: well, it is because that's what is available today :) | |
| 14:39:32 | mriedem | bhagyashris: to be clear, i'm not -2 on your spec, i'm just saying that the alternative to deprecate should be mentioned in there | |
| 14:42:44 | mriedem | claudiub: can you also propose a stable/queens backport for https://review.openstack.org/#/c/557377/ ? | |
| 14:42:54 | mriedem | i've got a queens 17.0.2 release patch waiting and can hold it up for this | |
| 14:43:02 | mriedem | since it was a regression in queens 17.0.1 | |
| 14:43:43 | openstackgerrit | Claudiu Belu proposed openstack/nova stable/queens: vmware: Fixes _detach_instance_volumes method https://review.openstack.org/557727 | |
| 14:44:45 | bhagyashris | mriedem: yeah... still waiting for others opinion but deprecation will lose the user friendly and that most imp IMO. | |
| 14:47:01 | bhagyashris | johnthetubaguy: Hi waiting for opinion on same ^ (https://review.openstack.org/#/c/511825/) | |
| 14:47:57 | johnthetubaguy | bhagyashris: I like the idea of documenting the alternative, even if we decide against it | |
| 14:49:31 | bhagyashris | mriedem: ok i will add this as alternative in spec and will go with proposed solution. | |
| 14:49:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use ksa session for cinder microversion check https://review.openstack.org/557508 | |
| 14:50:05 | bhagyashris | johnthetubaguy: ok i will add this as alternative in spec and will go with proposed solution. | |
| 14:52:05 | bhagyashris | johnthetubaguy: continuing implementation as per the proposed solution I will push the patch soon | |
| 14:52:43 | openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information https://review.openstack.org/312626 | |
| 14:53:34 | efried | alex_xu_, dansmith, edleafe, jaypipes, cdent, bauzas: https://etherpad.openstack.org/p/placement-dynamic-traiting for a start | |
| 14:53:43 | stephenfin | jaypipes, mriedem: Could one of you send this through? https://review.openstack.org/#/c/555000/ | |
| 14:57:35 | mriedem | stephenfin: would prefer if jaypipes or bauzas did that since i didn't review that series | |
| 14:57:50 | stephenfin | ack | |
| 14:58:10 | jaypipes | stephenfin: would you mind updating the commit message summary as cdent requests? | |
| 14:58:18 | mriedem | if someone is looking for what appears to be a low-hanging fruit bug that just needs a rev of the patch, https://review.openstack.org/#/c/541660/ | |
| 14:58:42 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Update 'share-pci-between-numa-nodes' spec to reflect reality https://review.openstack.org/555000 | |
| 14:58:46 | stephenfin | jaypipes: done | |
| 15:00:00 | jaypipes | stephenfin: done | |
| 15:00:12 | stephenfin | ta :) | |
| 15:02:05 | dansmith | efried: I added a few notes, I gotta jump on a call now | |
| 15:02:10 | jaypipes | cdent: btw, have you started on an os-resource-classes lib? if not, I was going to hop on that... | |
| 15:02:24 | efried | dansmith: ack | |
| 15:03:03 | cdent | jaypipes: go for it. I'd been trying to lay it out as bait for $random_person, but if you're keen go for it | |
| 15:03:17 | jaypipes | alex_xu_: see my question to Intel folks on line 13 of the etherpad... | |
| 15:06:32 | alex_xu_ | jaypipes: yes, I think we should say it is 'pre-programmed' or 'static' compare to 'fully dynamic' | |
| 15:08:29 | cdent | jaypipes, efried, others: watching that etherpad it appears that the conversation from the hangout is continuing rather than being summarized? | |
| 15:09:09 | edleafe | cdent: exactly. IOW, we can't summarize our conclusions if we didn't reach consensus | |
| 15:09:24 | efried | cdent: Sorry, got pulled away as soon as I posted that. Will look in a couple minutes. | |
| 15:10:43 | jaypipes | cdent: yes :( | |
| 15:11:35 | efried | mriedem: The fix https://review.openstack.org/#/c/557508/ works. But I think the hacking check fix isn't right just yet. I will revise. | |
| 15:12:12 | efried | mriedem: (The assertion that it works is based on internal discussions - I've asked them to update the bug accordingly to make it official.) | |
| 15:12:13 | mriedem | efried: pep8 passes | |
| 15:12:21 | efried | mriedem: But pyXX won't. | |
| 15:12:31 | mriedem | oh, i didn't run those... | |
| 15:12:51 | mriedem | if you change the hacking check, it likely should be a separate patch before that cinder fix | |
| 15:12:51 | efried | I didn't either, just assuming based on how mock.patch works... | |
| 15:13:06 | mriedem | i was just being lazy and expediant | |
| 15:13:53 | efried | mriedem: Yeah, I'll fix the hacking check totally separately. I'll tweak the fix you did on this patch cause we want it quickish. | |
| 15:14:10 | alex_xu_ | jaypipes: dansmith to avoid program a pre-programmed device, we need 'OR' logic in allocation candidates, I commented at line 30 | |
| 15:15:06 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa session for cinder microversion check https://review.openstack.org/557508 | |
| 15:15:07 | efried | mriedem: ^ | |
| 15:17:18 | mriedem | oh right i screwed up the order of the args to the test metohd | |
| 15:17:19 | mriedem | oops | |
| 15:18:22 | mriedem | ok i see "@efried, patch seems to be working fine with vm deploy/delete, volume attach/delete flows" in the bug | |
| 15:18:42 | efried | mriedem: Oh, yeah, they did comment on the bug. Cool. | |
| 15:19:22 | mriedem | +2 | |