| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-21 | |||
| 18:09:17 | melwitt | dansmith: noted. thanks | |
| 18:10:35 | dansmith | melwitt: ah, this isn't specifically tree related stuff | |
| 18:11:14 | melwitt | the other rc3 candidate is https://review.openstack.org/#/c/545478 which is a simple one to handle specific multiattach exceptions in compute/api | |
| 18:12:33 | dansmith | melwitt: that one is just to avoid raising InvalidBDMVolume for those cases? | |
| 18:12:41 | dansmith | are those caught in the api layer and handled differently? | |
| 18:12:42 | melwitt | yes | |
| 18:13:48 | melwitt | yeah, catch them and reraise them instead of letting it fall through to Exception where it raises a generic unhelpful InvalidBDMVolume | |
| 18:14:36 | dansmith | jebus that's a huge list of exceptions caught around create_server | |
| 18:14:37 | dansmith | oof | |
| 18:17:12 | melwitt | dansmith: which? that there will be 4 now? | |
| 18:17:52 | dansmith | melwitt: no, where it's caught in the api: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L576-L625 | |
| 18:18:12 | melwitt | oh, yes. that's an epic list | |
| 18:21:40 | melwitt | mriedem: both rc3 changes are traveling gateward. I'll propose the release patch when I know which hash to use | |
| 18:30:04 | cdent | johnthetubaguy: is this permanently dead, or may come back to life? https://review.openstack.org/#/c/438640/ (spot instances). Or is blazar taking its place, or maybe preemptible as cern's doing it? http://openstack-in-production.blogspot.co.uk/2018/02/maximizing-resource-utilization-with.html | |
| 18:31:50 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: Fix error handling in compute API for multiattach errors https://review.openstack.org/546729 | |
| 18:34:16 | melwitt | efried_omalley: do you want to propose the backport for your bug fix to stable/queens? | |
| 18:34:32 | efried_omalley | melwitt: I'd be happy to. Did we decide it's backport-worthy? | |
| 18:34:47 | melwitt | I think it has to be if it's for queens rc3 | |
| 18:36:12 | melwitt | backing up, yes, based on what you and mriedem said about the severity earlier I think we should | |
| 18:36:26 | efried_omalley | melwitt: Roger wilco, stand by. | |
| 18:36:40 | melwitt | thanks | |
| 18:39:10 | efried_omalley | melwitt: Quit standing by. Gonna be a manual one. | |
| 18:39:41 | melwitt | ah, bummer. good luck | |
| 18:44:25 | efried_omalley | ahjeez, it's because of all our diligent work getting contexts passed around everywhere. | |
| 18:44:56 | efried_omalley | some of which is in Q, some not. | |
| 18:55:58 | efried_omalley | ...which would have to be redone if we plan to merge https://review.openstack.org/#/q/project:openstack/nova+topic:bug/1734625+branch:stable/queens -- mriedem, melwitt thoughts? | |
| 18:56:43 | openstackgerrit | Eric Fried proposed openstack/nova stable/queens: Only pull associated *sharing* providers https://review.openstack.org/546740 | |
| 18:56:48 | efried_omalley | melwitt: ^ | |
| 18:56:54 | melwitt | efried_omalley: can we just backport more things to stable/queens to provide the needed base? or is it more than a few patches? | |
| 18:57:21 | efried_omalley | melwitt: See commit message. | |
| 18:57:31 | efried_omalley | The additonal backport would be smallish. | |
| 18:57:38 | efried_omalley | But I've already done the un-rebase :) | |
| 18:57:49 | efried_omalley | melwitt: So it's your call, o great and powerful. | |
| 18:59:58 | melwitt | okay. I think we'd usually just pull in the prereq patches too, assuming they meet stable branch requirements. I'd like mriedem's recommendation when he's around | |
| 19:00:24 | melwitt | we've got some time before the changes on master merge anyway | |
| 19:01:14 | cdent | efried_omalley: is the blocker still the context stuff? having the rest of that is a useful backport | |
| 19:01:25 | efried_omalley | cdent: Not a blocker, but yes. | |
| 19:13:54 | cdent | edleafe, efried_omalley, jaypipes: I'm going off-grid as much as possible until sunday evening shortly. Anything I should attend before I do? | |
| 19:14:29 | efried_omalley | cdent: I think you've +1ed all the stuff I care about :) | |
| 19:14:50 | efried_omalley | So yeah, quick, go offline before you change your mind about any of it. | |
| 19:14:52 | cdent | oh, that's just my efried+1 bot, not me | |
| 19:22:32 | efried_omalley | I need to let ubuntu upgrade a bunch of software. So I may not be back. Wish me luck. | |
| 19:25:34 | mriedem | ok so what's going on? we need https://review.openstack.org/#/q/project:openstack/nova+topic:bug/1734625+branch:stable/queens to make the backport of https://review.openstack.org/545494 clean? | |
| 19:27:13 | mriedem | we could take those for rc3, they are just large and i figured they weren't super high priority for an RC | |
| 19:29:45 | openstackgerrit | Mohammed Naser proposed openstack/nova master: Ensure attachment_id always exists for block device mapping https://review.openstack.org/546398 | |
| 19:29:50 | melwitt | yeah. backport was messy/manual because that refactor isn't in stable/queens | |
| 19:29:57 | jaypipes | cdent: nope, see you in Dublin | |
| 19:30:06 | cdent | ✔ | |
| 19:35:05 | ingy | ‼ | |
| 19:40:28 | openstackgerrit | Hongbin Lu proposed openstack/nova master: Skip placement on rebuild in same host https://review.openstack.org/546357 | |
| 19:40:59 | efried | woot | |
| 19:45:31 | openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle IpAddressAlreadyAllocated exception https://review.openstack.org/535532 | |
| 19:53:47 | melwitt | efried, mriedem: upon looking at the backport now I think it's good as-is | |
| 19:54:09 | edleafe | cdent: we got it. | |
| 20:10:47 | Nasir | Hi i needed assistance trying to figure out why I cannot retrieve the serial console of my ironic/baremetal instance from nova, even though its configured properly in ironic | |
| 20:12:32 | Nasir | This is what i see when i try to do "nova get-serial-console |
|
| 20:13:17 | Nasir | https://thepasteb.in/p/NxhVxv9661JiN | |
| 20:18:28 | Nasir | Can anyone assist please ? Thanks | |
| 20:29:50 | melwitt | Nasir: this is a nova development channel, please see topic | |
| 20:30:20 | mnaser | melwitt: just fyi https://review.openstack.org/#/c/546689/ -- this okay with you? :) | |
| 20:32:41 | melwitt | mnaser: looks cool to me | |
| 20:32:48 | mriedem | melwitt: this is also going to be in RC3 https://review.openstack.org/#/c/533212/ | |
| 20:32:54 | mriedem | since it's a regression due to privsep | |
| 20:33:12 | mnaser | melwitt: perfect ty, +A'd | |
| 20:33:27 | melwitt | mriedem: okay, thanks | |
| 20:37:49 | openstackgerrit | Eric Berglund proposed openstack/nova master: Add check for redundant import aliases https://review.openstack.org/543995 | |
| 20:45:41 | dansmith | belmoreira: since you're around I assume you folks have stuff to discuss this week? | |
| 20:47:10 | belmoreira | dansmith tssurya upload a new spec and we would like to have your thoughts about it | |
| 20:47:15 | melwitt | I think tssurya added something to the agenda but I haven't looked at it yet | |
| 20:47:23 | dansmith | orly | |
| 20:47:59 | tssurya | dansmith, melwitt : https://review.openstack.org/#/c/546684/ | |
| 20:48:25 | tssurya | melwitt : I just updated the bugs/ reviews we have on the agenda :D | |
| 20:48:40 | melwitt | tssurya: a-ha, thank you | |
| 20:49:18 | dansmith | tssurya: belmoreira: makes sense I think | |
| 20:50:56 | belmoreira | dansmith this was something missing in cellsV1 and we and at least rackspace have hacks for it | |
| 20:52:17 | melwitt | agreed that a feature for disabling a cell makes sense | |
| 20:52:58 | belmoreira | dansmith melwitt great, we can then continue the discussion in the spec | |
| 20:53:55 | dansmith | tssurya: it's against the rules to approve a spec on revision 1, so I have noted a few nits for you to address :) | |
| 20:54:32 | tssurya | dansmith: hehe, noted, thanks! | |
| 20:54:35 | dansmith | heh | |
| 20:55:04 | dansmith | belmoreira: in case it's not obvious, we're glad you brought tssurya to us | |
| 20:55:20 | belmoreira | :) | |
| 20:55:43 | melwitt | ++ | |
| 20:56:06 | tssurya | dansmith , melwitt : wow, thanks :D truly honoured | |
| 20:56:28 | belmoreira | dansmith thanks for all the work in the placement filtering. Do you think it will be easy to backport to Pike? | |
| 20:56:42 | melwitt | tssurya: and thank you for all the great work | |
| 20:57:09 | dansmith | belmoreira: it's not backportable upstream for sure.. I think pike lacks the placement aggregate stuff that will be needed to make it actually work | |
| 20:57:30 | tssurya | melwitt : couldn't have done it without the cells team ;) | |
| 20:58:23 | belmoreira | dansmith I was not asking for the upstream backport. But I would like/need to backport it for us | |
| 20:58:36 | dansmith | belmoreira: the actual filter mechanism should apply back that far I think, so if you'd be willing to run a newer placement with the newer support it has then it's probably not too bad yeah | |
| 20:59:57 | belmoreira | Currently I started collecting data about the scheduling time (I'm running local placement). I would like to see the impact when starting consolidating placement in only one instance | |
| 21:00:19 | belmoreira | I will let you the numbers | |
| 21:01:11 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Update plugs Contrail methods to work with privsep https://review.openstack.org/546778 | |
| 21:01:25 | dansmith | belmoreira: cool | |
| 21:01:32 | melwitt | fyi I've added the cells review links to the priorities etherpad https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | |
| 21:02:17 | tssurya | melwitt : thank you | |
| 21:04:40 | mriedem | efried: do you see any issues with including these in RC3? https://review.openstack.org/#/q/project:openstack/nova+topic:bug/1734625+branch:stable/queens | |
| 21:32:04 | efried | mriedem: No. Go for it. | |
| 21:32:30 | efried | mriedem: But they'll have to be rebased. | |
| 21:32:51 | efried | The alternative is to rebase https://review.openstack.org/#/c/546740/ on top of them instead. | |