| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-21 | |||
| 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. | |
| 21:33:02 | efried | which will make the cherry-pick clean. | |
| 21:33:14 | efried | I think | |
| 21:41:36 | melwitt | smcginnis is already in the March spirit :) | |
| 21:47:32 | mriedem | efried: that's why i'm asking, | |
| 21:47:35 | mriedem | if you put yours on top | |
| 21:48:05 | mriedem | efried: i didn't include those request id backports since they were large and didn't think we needed them for an RC | |
| 21:48:14 | mriedem | now if it helps with conflicts both ways, we might as well | |
| 21:48:55 | efried | mriedem: The existing cherry-pick is already proposed, and it reads pretty clean. So it's more work, not less, to backport the global_id stuff. | |
| 21:49:05 | efried | mriedem: It may make subsequent backports easier. | |
| 21:49:18 | efried | mriedem: It will make this cherry-pick cleaner (possibly zero-conflict) | |
| 21:49:27 | efried | mriedem: But it'll be work to get that done. | |
| 21:49:35 | efried | mriedem: Which I'm happy to do if you wish it to be so. | |
| 21:50:40 | melwitt | which additional backports would the global_id patches help? | |
| 21:50:56 | melwitt | because I agree, the one that's already proposed reads clean to me too | |
| 21:51:12 | mriedem | if it's not clean on top of https://review.openstack.org/#/q/project:openstack/nova+topic:bug/1734625+branch:stable/queens then nvm | |
| 21:51:20 | mriedem | we can just go with what's there, that's fine | |
| 21:51:36 | mriedem | https://review.openstack.org/#/q/project:openstack/nova+topic:bug/1734625+branch:stable/queens is just more LOC than i'm comfortable with for an RC when it's not something that is required | |
| 21:51:49 | melwitt | but if not having global_id patches dominos into making other things difficult too, then we bring in global_id patches | |
| 21:57:58 | mriedem | dansmith: want to get this for RC3? https://review.openstack.org/#/c/546729/ | |
| 21:58:15 | mriedem | and https://review.openstack.org/#/c/546778/ | |
| 21:58:26 | efried | melwitt: Depends how many backport-worthy bugs we find in the report client. | |
| 21:58:40 | efried | Considering the amount of change it saw in Q.... | |
| 21:59:29 | melwitt | yeah. I'd say since we don't know yet and you already dealt with the conflicts and it looks clean now, just go with what we have and we'll burn the other bridge when we get to it | |
| 22:00:19 | openstackgerrit | Eric Berglund proposed openstack/nova master: Add check for redundant import aliases https://review.openstack.org/543995 | |