Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-07
19:36:58 Nisha_Agarwal mriedem, it would be good if you could suggest the subject for the mail....i dont know what to write actually
19:37:10 Nisha_Agarwal in the subject
19:37:14 mriedem hmm
19:37:15 Nisha_Agarwal body i can fill up
19:37:43 mriedem [nova][ironic] Concerns over rigid resource class-only ironic scheduling
19:37:43 mriedem ?
19:39:59 Nisha_Agarwal mriedem, looks good :)
19:42:08 mriedem stvnoyes: well looky here http://logs.openstack.org/05/501805/2/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/069e82e/console.html#_2017-09-07_17_45_00_706868
19:45:46 Nisha_Agarwal mriedem, do i need to subscribe to openstack-operators@lists.openstack.org for sending an email?
19:45:51 mriedem yeah
19:45:55 Nisha_Agarwal ok
19:46:38 mriedem ildikov: why does https://review.openstack.org/#/c/330285/ need to depend on https://review.openstack.org/#/c/498828/ ?
19:47:50 ildikov mriedem: uWSGI changes in Cinder blew up
19:48:13 mriedem why would that matter here?
19:49:00 Nisha_Agarwal mriedem, i sent an email just now. if you would want to add to any thing to the body of the mail please add it.
19:49:44 ildikov mriedem: because the url wasn't working with the microversion checks in the client
19:50:02 Nisha_Agarwal mriedem, meanwhile i would open a blueprint at nova side or i should do it after responses on the email and probably after the PTG?
19:50:13 Nisha_Agarwal just asking
19:50:55 ildikov mriedem: smcginnis might be able to describe it in details
19:52:48 smcginnis Wha?
19:53:29 smcginnis Ah
19:53:53 smcginnis mriedem: There's an issue right now with the way the public endpoint is being set and how that works for microversions.
19:54:01 smcginnis So that needs to get fixed first.
19:56:24 stvnoyes mriedem: nice to see it pass. I also figured out why I put the change in the iscsi test. the other live migrate block test was skipped dur a bug. I just removed that skip decorator - https://review.openstack.org/#/c/501828/ . If you're OK with that, I'll add the new attach checks to that test too.
19:56:48 stvnoyes dur = 'due to'
19:57:28 mriedem Nisha_Agarwal: replied with some more details from our irc discussion
19:57:31 mriedem thanks for starting that thread
19:58:10 mriedem stvnoyes: probably less worthwhile to add things to a skipped test in tempest
19:58:24 mriedem fwiw this does test that volume-backed live migration test https://review.openstack.org/#/c/481290/
19:58:34 mriedem because it depends on https://review.openstack.org/#/c/481346/
19:58:58 Nisha_Agarwal mriedem, thanks for the reply there
19:59:33 stvnoyes ah you already did it. ok I'll abandon my review since it does the same thing
19:59:50 Nisha_Agarwal mriedem, with respect to "eventually lead to scheduling failures later because of the scheduler thinking a node is available for more than one instance when it's really not." doesnt happen in the customer env where he uses non-exact filters
20:00:29 mriedem mtreinish: i'm +2 on your ostestr change, maybe you want to review this now https://review.openstack.org/#/c/498828/
20:00:48 Nisha_Agarwal mriedem, the ironic node doesnt get scheduled again if use non-exact filters
20:02:34 Nisha_Agarwal atleast with this customer we have never heard of this issue
20:02:38 mtreinish mriedem: +2
20:03:04 mtreinish although I wonder how much the tls-proxy is used now. As things move to use uwsgi the standalone proxy isn't used anymore
20:12:14 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Add spec to use cinder's new attachment API https://review.openstack.org/497552
20:16:22 ildikov mriedem: thanks ^^
20:16:57 mriedem ildikov: johnthetubaguy found the same thing i did in https://review.openstack.org/#/c/493323/
20:18:45 ildikov mriedem: I bumped the version for all the calls to 3.44 in the attach patch
20:19:50 mriedem that's not clear here, it wasn't called out in the commit message, and it would probably just be cleaner if we did it in this change
20:20:12 ildikov mriedem: I got lost on the newer version note in the review though
20:20:30 mriedem mine?
20:20:31 ildikov I was debating where to do it and thought to try to keep the patches clean
20:20:47 mriedem i noticed the skip_version_check=True thing,
20:20:55 mriedem which is wrong here, as i noted
20:21:20 ildikov because I haven't bumped all versions?
20:21:41 mriedem i'd just bump the version for attachment_create to be 3.44 as john pointed out, update whatever unit tests and make a comment about it in the commit message that we need the minimum microversion for creating the attachment to equal the microversoin needed for complete
20:21:57 mnaser abaranov https://review.openstack.org/#/c/501820/1 looks lik ethis failed somewhere along the line
20:22:05 mnaser it doesnt seem unrelated but does anyone know the bug # offhand
20:22:12 ildikov mriedem: that's what I planned to do
20:23:21 mriedem mnaser: unrelated, just recheck it
20:23:26 ildikov mriedem: as I said, I had two choices on where to bump and I chose what seemed cleaner to me, but agree with this variation too so will change when I get there later today
20:23:31 mnaser done, ty
20:23:53 mriedem mnaser: btw, wanted to ask, how do you pick up changes from stable branches? do you wait for a release or just pull from the branch?
20:24:03 mnaser mriedem rdo trunk stable builds
20:24:14 mriedem ok so don't wait for release
20:24:20 mnaser in cases like this, no
20:24:32 mriedem do you patch rpms until the changes are merged upstream then?
20:24:51 mnaser in very urgent scenarios yes, with puppet after the rpm is installed (not pretty)
20:25:00 mnaser but for the most we try to get it into stable branch asap and then deploy those packages
20:25:11 mriedem yeah
20:25:52 mriedem dansmith: these stable backports are up when you've got the time https://review.openstack.org/#/q/topic:bug/1715462+branch:stable/pike
20:26:05 mriedem mtreinish: could you also go through these ^ ?
20:26:11 mriedem nasty regressions for pike
20:26:21 mriedem and we're short on stable cores atm
20:26:29 mriedem that aren't in europe anyway
20:29:46 openstackgerrit Merged openstack/nova-specs master: Add spec to use cinder's new attachment API https://review.openstack.org/497552
20:41:13 mtreinish mriedem: sure, I'll take a look
21:00:50 openstackgerrit Merged openstack/nova master: Make eventlet hub use a monotonic clock https://review.openstack.org/434327
21:14:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop support for the Cinder v2 API https://review.openstack.org/501874
21:15:10 smcginnis Woot! ^
22:05:01 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247
22:05:02 openstackgerrit Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137
22:09:09 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
22:09:10 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
22:09:11 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
22:19:04 openstackgerrit Merged openstack/nova master: Transform keypair.delete notification https://review.openstack.org/463946
22:26:04 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247
22:27:59 cdent efried: I don’t think you are being sufficiently critical of yourself in your comments
22:28:32 efried cdent I know what I mean, and I'm VICIOUS.
22:28:42 cdent :)
22:38:28 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
22:39:35 openstackgerrit Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057
22:51:09 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
22:51:10 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
22:51:10 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
#openstack-nova - 2017-09-08
00:46:09 openstackgerrit Merged openstack/nova stable/pike: Allow setting up multiple cells in the base TestCase https://review.openstack.org/501818
00:48:57 openstackgerrit Merged openstack/nova stable/pike: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501819
00:50:05 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031
00:52:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
00:52:52 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types https://review.openstack.org/430839
01:53:21 openstackgerrit liusheng proposed openstack/nova master: Correct the out-of-date flavor doc reference https://review.openstack.org/501921
02:17:19 openstackgerrit Merged openstack/nova master: Fix broken URLs https://review.openstack.org/501402
02:35:39 openstackgerrit Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
02:52:20 openstackgerrit Philip Choi proposed openstack/nova master: Add warning message for cache_utils connection failure https://review.openstack.org/501928
02:56:56 openstackgerrit Philip Choi proposed openstack/nova master: Add warning message for cache_utils connection failure https://review.openstack.org/501928
03:04:17 openstackgerrit kangyufei proposed openstack/nova master: Typo error about help resource_classes.inc https://review.openstack.org/501930

Earlier   Later