| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-21 | |||
| 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 | |
| 22:02:32 | efried | melwitt: Dig | |
| 22:03:27 | openstackgerrit | Merged openstack/nova master: Only pull associated *sharing* providers https://review.openstack.org/545494 | |
| 22:05:35 | openstackgerrit | Merged openstack/nova master: Fix error handling in compute API for multiattach errors https://review.openstack.org/545478 | |
| 22:05:54 | openstackgerrit | Merged openstack/nova master: Avoid exploding if guest refuses to detach a volume https://review.openstack.org/546423 | |
| 22:08:40 | mriedem | dansmith: i'm +2 on the backport for efried's fix https://review.openstack.org/#/c/546740/ - do we want to hack in the (cherry picked from x) thing? | |
| 22:09:12 | mriedem | oh nvm it's higher up | |
| 22:09:36 | efried | yuh | |
| 22:09:54 | efried | git review complained about it not being at the bottom for some reason. Possibly because of the conflicts section? Dunno. | |
| 22:10:30 | mriedem | melwitt: in case you haven't seen https://review.openstack.org/#/c/546315/ and the one after it | |
| 22:10:41 | mriedem | mnaser found some problems after working on those local delete backports | |
| 22:14:43 | mriedem | i kind of sort of wonder if we should get those into rc3 as well | |
| 22:15:07 | mriedem | https://review.openstack.org/#/q/topic:bug/1404867+status:open+branch:stable/queens plus the 2 on master | |
| 22:15:14 | mriedem | maybe that's too much | |
| 22:17:35 | mnaser | mriedem: part of me says - this bug existed for a long time so it's not exactly affecting this release - the other part says it might manifest itself much more after an upgrade where all computes end up being a minimum of pike | |
| 22:18:04 | mnaser | if the computes were behind and are being updated *just enough* to upgrade api, then it'll happen | |
| 22:21:03 | mriedem | right that's my worry, any bfv with an existing volume where you have at least 1 pike compute, and scheduling fails, and you orphan the volume when deleting the instance | |
| 22:22:03 | mriedem | less of an issue in pike where you might have 1 ocata compute so we don't reserve the volume in the api, | |
| 22:22:13 | mriedem | more of an issue in pike and queens where the computes would be new enough to reserve the volume in the api | |
| 22:22:30 | mriedem | *more of an issue in queens | |
| 22:24:42 | melwitt | hm, sounds like we should include it then. the patches are small | |
| 22:26:56 | openstackgerrit | Eric Berglund proposed openstack/nova master: Add check for redundant import aliases https://review.openstack.org/543995 | |
| 22:29:53 | mriedem | except your big one | |
| 22:30:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform live_migration_post_dest notification https://review.openstack.org/469784 | |