Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-21
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 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: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: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: Store block device mappings in cell0 https://review.openstack.org/546219
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: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: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: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 mnaser melwitt: fyi https://bugs.launchpad.net/nova/+bug/1686116
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: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)"
00:48:46 melwitt http://logs.openstack.org/00/546200/2/gate/neutron-grenade/dbcbf60/logs/grenade.sh.txt.gz#_2018-02-21_23_43_04_990
00:50:01 mnaser melwitt: indeed i tracked it down
00:50:08 mnaser melwitt: https://review.openstack.org/#/q/Ic1fa3a98b6bcd151c489b078028687892655a19b
00:50:38 melwitt oh, nice
00:52:33 mnaser melwitt: got a first +2, maybe if you can track down someone to +A .. hopefully
00:54:06 melwitt okay, lemme see
01:03:35 openstackgerrit Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813
01:06:20 melwitt dansmith: looks like you're a grenade core, if you could review these to unblock our stable/queens and stable/pike backports when you get a chance, maybe tomorrow morning if you can real quick https://review.openstack.org/#/c/546828/ https://review.openstack.org/#/c/546831
01:35:51 openstackgerrit Eric Fried proposed openstack/nova master: WIP: rp: GET /resource_providers?required= https://review.openstack.org/546837
01:37:03 efried cdent, jaypipes: FYI ^ -- I think it's mostly done. Please do vet my sqla for potential optimizations. But be gentle - it took me a couple hours to get it working at all.
01:37:48 efried melwitt: Potential to have a Rocky bp closed before the PTG :)
02:10:39 melwitt dansmith: nvm on my earlier message, mtreinish was able to fast-approve
02:32:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop API compat handling for old compute error cases https://review.openstack.org/546256
02:39:43 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle spawning error on unshelving https://review.openstack.org/378009
02:55:33 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls https://review.openstack.org/544158
03:28:11 openstackgerrit Merged openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008
03:50:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
03:52:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
03:53:17 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
05:43:05 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734
05:52:58 openstackgerrit Merged openstack/python-novaclient master: Clean imports in code https://review.openstack.org/546543
06:56:09 openstackgerrit Merged openstack/nova stable/queens: Drop extra loop which modifies Cinder volume status https://review.openstack.org/546200
07:01:36 openstackgerrit blkart proposed openstack/nova-specs master: don't show description content in code-block https://review.openstack.org/539478
07:12:30 openstackgerrit Merged openstack/nova stable/queens: Store block device mappings in cell0 https://review.openstack.org/546201

Earlier   Later