| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-21 | |||
| 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 | |
| 22:30:20 | melwitt | me? no | |
| 22:31:32 | mriedem | ok i'll hit these queens ones and let dansmith make a call | |
| 22:33:21 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | |
| 22:34:01 | melwitt | was just kidding on that. I had thought the others had already gone to stable/queens | |
| 22:35:56 | mriedem | done https://review.openstack.org/#/q/topic:bug/1404867+status:open+branch:stable/queens | |
| 22:36:43 | mriedem | https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/queens+label:Code-Review=2 | |
| 22:37:15 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | |
| 22:37:22 | melwitt | thanks. dansmith ^ if you think those are cool for rc3 | |
| 22:38:51 | openstackgerrit | Merged openstack/nova master: Update plugs Contrail methods to work with privsep https://review.openstack.org/533212 | |
| 22:39:27 | mriedem | will likely need a +2/+W from dan on the backport for ^ as well | |
| 22:40:10 | dansmith | sorry I had a thing | |
| 22:41:09 | dansmith | mriedem: so just the stable/queens list from above? | |
| 22:43:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| 22:44:26 | dansmith | mriedem: need someone else for that backport of mine I guess | |
| 22:44:31 | mnaser | mriedem: idk if we want to land those other last 2 on top for rc3 | |
| 22:46:40 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | |
| 22:48:09 | mriedem | mnaser: we do | |
| 22:48:30 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 22:48:51 | mriedem | dansmith: need this one also https://review.openstack.org/#/c/546778/ | |
| 22:48:58 | mriedem | dansmith: you'll likely need to just approve that one, | |
| 22:49:04 | mriedem | consider my stable core backport as a +2 | |
| 22:49:32 | mriedem | mnaser: mel approved https://review.openstack.org/#/c/546315/ and the one after it so can you backport those to queens also so we can get them into rc3? | |
| 22:51:13 | dansmith | mriedem: ack | |
| 22:51:51 | wolverineav | hey, mriedem stephenfin, just a quick heads-up on the IVS vif change https://review.openstack.org/#/c/534371/ i've addressed the comments. let me know if any further changes are required :) | |
| 22:55:02 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839 | |
| 22:55:19 | mnaser | mriedem: I’ll do that right now | |
| 22:56:18 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform servergroup.addmember notification https://review.openstack.org/541101 | |
| 23:07:53 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/queens: Add functional test for deleting BFV server with old attach flow https://review.openstack.org/546803 | |
| 23:07:54 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/queens: Ensure attachment_id always exists for block device mapping https://review.openstack.org/546804 | |
| 23:08:54 | mnaser | dansmith: mriedem ^ | |
| 23:09:04 | mnaser | ill do the pike one but that's not a big priority i guess | |
| 23:09:46 | dansmith | moar? | |
| 23:10:06 | dansmith | I'm going off mriedem's judgment for these attachment things | |
| 23:10:51 | mnaser | dansmith: yeah it was a corner case we discovered upon backporting to pike that we fixed | |
| 23:25:46 | mnaser | mriedem: i think i will have to squash https://review.openstack.org/#/c/546398 + https://review.openstack.org/#/c/545132 in the stable/pike backport. if i put 546398 first, the integration test for the fix fails (because early-delete patch is not there). if i put 545132 before, the patch doesn't pass because 546398 is not there to make it do the right thing(tm) | |
| 23:25:49 | mnaser | is that ok? | |
| 23:25:55 | mriedem | mnaser: dansmith: +2 on those backports also | |
| 23:26:10 | mnaser | unless you have other suggestions | |
| 23:26:47 | mriedem | mnaser: ok, sounds like we have to do that, i can't think of fancy ways around that | |
| 23:27:12 | mnaser | ok ill propose a squashed commit in that case | |
| 23:27:54 | mriedem | dansmith: all yours https://review.openstack.org/#/q/topic:bug/1750666+status:open+branch:stable/queens | |
| 23:33:06 | openstack | Launchpad bug 1729584 in OpenStack Compute (nova) "boot from volume + configdrive broken (regression)" [High,In progress] - Assigned to Jay Pipes (jaypipes) | |
| 23:33:06 | melwitt | one thing I was thinking about that's weird is a lot of those fixes are for bfv, but we've a regression since ocata with bfv + configdrive https://bugs.launchpad.net/nova/+bug/1729584 | |
| 23:33:22 | melwitt | mnaser: have yall run into that at all ^ | |
| 23:34:47 | melwitt | ? | |
| 23:35:00 | mriedem | melwitt: squeeky wheel gets the grease | |
| 23:35:42 | melwitt | yup ... I admit I keep forgetting about that one too with all of the whirlwind of other stuff | |
| 23:35:49 | mnaser | melwitt: checking | |
| 23:36:10 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/pike: Drop extra loop which modifies Cinder volume status https://review.openstack.org/546218 | |
| 23:36:11 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/pike: Add functional tests to ensure BDM removal on delete https://review.openstack.org/546220 | |
| 23:36:11 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/pike: Store block device mappings in cell0 https://review.openstack.org/546219 | |
| 23:36:12 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/pike: Ensure attachment_id always exists for block device mapping https://review.openstack.org/546275 | |
| 23:36:12 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/pike: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/546221 | |
| 23:36:13 | openstackgerrit | Mohammed Naser proposed openstack/nova stable/pike: Add functional test for deleting BFV server with old attach flow https://review.openstack.org/546812 | |
| 23:36:59 | mnaser | btw not sure if this is how to properly squash things but yeah https://review.openstack.org/#/c/546275 | |
| 23:37:07 | mnaser | and now reading | |
| 23:37:24 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813 | |
| 23:38:34 | mnaser | melwitt: two things, we don't have configdrive by default + that issue seems to be happening when using virtio-scsi | |
| 23:38:52 | mnaser | i remember another issue of someone who had a similar issue a while back in the openstack canada group | |
| 23:39:01 | melwitt | okay, right ... so it's more niche | |
| 23:39:25 | melwitt | so not as hot as these other bugs | |
| 23:39:55 | mnaser | yeah, you have to be using virtio-scsi (which i believe you have to set an image flag for that) and config drive | |
| 23:40:14 | melwitt | anyway, I gotta take care of that bug and hopefully backport it to ocata where it broke before ocata goes away | |
| 23:40:31 | melwitt | yeah | |
| 23:41:26 | mnaser | i posted a message in the openstack canada slack group to see who was the one who hit it | |
| 23:43:30 | openstack | Launchpad bug 1686116 in OpenStack Compute (nova) ocata "domain xml not well defined when using virtio-scsi disk bus" [Medium,Fix committed] - Assigned to sahid (sahid-ferdjaoui) | |
| 23:43:30 | mnaser | melwitt: fyi https://bugs.launchpad.net/nova/+bug/1686116 | |
| 23:44:03 | mnaser | "This issue was fixed in the openstack/nova 15.0.7 release." | |
| 23:44:13 | efried | jaypipes: yt? | |
| 23:44:21 | mnaser | which is the exact release that jaypipes mentioned affected him | |
| 23:44:26 | mnaser | so that might be an interesting review to look at | |
| 23:57:17 | mriedem | jgriffith: smcginnis: stvnoyes: ildikov: finally got it https://youtu.be/hZg6wqxdEHk | |
| #openstack-nova - 2018-02-22 | |||
| 00:02:59 | melwitt | mnaser: that's the series that caused the regression | |
| 00:09:42 | efried | dansmith: yt? | |
| 00:31:04 | openstackgerrit | Merged openstack/nova master: Add functional test for deleting BFV server with old attach flow https://review.openstack.org/546315 | |
| 00:31:25 | openstackgerrit | Merged openstack/nova master: Ensure attachment_id always exists for block device mapping https://review.openstack.org/546398 | |
| 00:37:20 | mnaser | well | |
| 00:37:26 | mnaser | neutron-grenade is blowing up for the queens backports | |
| 00:37:29 | efried | Somebody, anybody with sqlalchemy skills | |
| 00:37:44 | efried | or SQL skills. edleafe yt? | |
| 00:46:16 | melwitt | so the grenade is blowing up | |
| 00:48:34 | melwitt | something is messed up: "rsync: change_dir "/opt/stack/old/tempest/.testrepository" failed: No such file or directory (2)" | |