Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-06
14:48:07 dansmith artom: no it doesn't, I'm agreeing
14:48:31 dansmith artom: I think we just kinda munged the "does it support" with "is it configured to allow sending" ... 5.3 when we first discussed it
14:48:40 artom dansmith, yep
14:48:54 dansmith if we were adding new calls on both sides, they'd both need to be configured for 5.3, but in this case, we can cheat
14:50:56 artom dansmith, ack, looks like we're all on the same page, I'll continue down that route and with the rest of mriedem's feedback after the bug call
14:51:07 dansmith cool
14:52:26 mriedem stephenfin: alex_xu: yonglihe: on that topology cli change, this is wrong, correct? https://review.opendev.org/#/c/670790/7/doc/source/cli/nova.rst@3663
14:53:04 mriedem compute:server:topology:host:index is admin-only by default and used to expose host fields
14:54:12 mriedem https://review.opendev.org/#/c/621476/62/nova/api/openstack/compute/server_topology.py@58
14:54:14 mriedem yeah looks like it
14:56:17 donnyd sean-k-mooney: looks like we are still getting a node failure
14:58:06 stephenfin looking
14:59:55 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add min service level check for migrate with bandwidth https://review.opendev.org/680394
15:01:13 stephenfin mriedem: Yup, good catch
15:01:14 mriedem efried: i'm +2 on the other ironic fix https://review.opendev.org/#/c/680542/ but we can wait until the test patch passes https://review.opendev.org/#/c/680643/ - given the state of the gate that might be sometime this weekend though, so maybe you want to +2 and one of us can approve once we have results?
15:01:58 sean-k-mooney donnyd: it might be because its requesting two vms getting one and timeing out getting the second?
15:02:47 mriedem actually that ironic patch already got test nodes, it must be nice to not be nova
15:02:49 efried mriedem: I was thinking along the same lines. But iiuc dtantsur followed up later and said the grenade fix wasn't actually related to this bug. We (still) don't have CI coverage that will hit it. I just proposed https://review.opendev.org/680718 in tripleo which will hopefully hit the job that originally uncovered the problem.
15:02:58 openstackgerrit Balazs Gibizer proposed openstack/nova master: migrate: Add bw min service level check of source compute https://review.opendev.org/680395
15:03:06 efried s/grenade fix/grenade failure/
15:03:38 mriedem alright then
15:03:40 mriedem moving on
15:04:05 efried mriedem: but given how simple and obviously-right the fix is, and that dtantsur says it is legit, I think we might want to merge it anyway.
15:04:05 sean-k-mooney donnyd: ill check it at the weekend agian when the ci does not have as high a load
15:04:13 dtantsur efried: I'd make your tripleo patch depending on both fixes. I'm not sure which one of them is most breaking.
15:04:22 mriedem efried: i'm good with that
15:04:34 mriedem i definitely trust dtantsur here and these are simple fixes
15:04:39 efried dtantsur: the symptoms would be different
15:06:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: resize: Add bw min service level check of source compute https://review.opendev.org/680396
15:06:00 dtantsur okie
15:08:32 efried dtantsur, mriedem: +A
15:08:53 dtantsur now let's see if you'll regret trusting me :D
15:09:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: prepare func test env for moving servers with bandwidth https://review.opendev.org/655109
15:09:18 efried It's not clear that the tripleo patch will work as is; I don't understand the job chain that would get us there, or whether nova needs to be added to required_projects *somewhere* (but where??). So it may take a few iterations to figure that out.
15:10:43 mriedem dtantsur: please don't break my achy breaky heart
15:11:13 dtantsur okay
15:11:21 mriedem i'm assuming you get early 90s billy ray cyrus references of course
15:11:58 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for migrate server with ports having resource request https://review.opendev.org/655113
15:14:58 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make _rever_allocation nested allocation aware https://review.opendev.org/676138
15:18:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140
15:18:57 gibi mriedem, bauzas: the next 5 patches in the bw series is up-to-date. first patch: https://review.opendev.org/#/c/680394/ last up-to-date patch: https://review.opendev.org/#/c/655113
15:19:09 mriedem gibi: yup, on my list for today
15:20:53 gibi mriedem: thanks. I still have to work on comments in th second half of the patch series and also need to do a follow up for the already merged https://review.opendev.org/#/c/656422
15:21:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972
15:24:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980
15:25:11 mriedem stephenfin: is it not possible to deprecate an entire group of config options?
15:25:15 mriedem you have to do them all individually?
15:25:40 stephenfin you can deprecate opts but not groups, afaik
15:25:41 stephenfin so no
15:25:44 mriedem blast
15:25:49 stephenfin soz
15:26:12 mriedem is that hipster millenial for sorry?
15:26:31 mriedem i had to look up what a "stan" was last night
15:26:38 stephenfin let's say...yes
15:26:45 cdent you'll be cool, eventually, mriedem
15:26:54 cdent each day, get better and better
15:27:15 mriedem https://media1.tenor.com/images/fc73ef705412eeaeb838763055918587/tenor.gif?itemid=5653229
15:27:15 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
15:28:09 mriedem oooo https://www.youtube.com/watch?v=BGrfhsxxmdE
15:28:48 jangutter The only "stan" I know is https://mc-stan.org
15:30:13 openstackgerrit Balazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize https://review.opendev.org/678827
15:37:55 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019
15:41:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Deprecate the XenAPIDriver https://review.opendev.org/680732
15:41:38 mriedem efried: dansmith: gibi: bauzas: alex_xu: melwitt: stephenfin: fyi ^
15:41:48 dansmith hoo boy
15:42:01 mriedem i'll post to the ML
15:42:12 stephenfin Ima +2 that so hard.
15:42:19 stephenfin after the ML post, of course
15:46:25 mriedem http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009180.html
15:47:37 mriedem stephenfin: since you smelled it, maybe you should deal up an etherpad for rc todo stuff like the prelude reno
15:47:47 mriedem i'm auto delegating efried delegating that to me
15:48:55 stephenfin mriedem: Sure https://etherpad.openstack.org/p/nova-train-prelude
15:51:25 efried thanks
15:51:39 efried is a prelude related to cycle highlights?
15:51:44 mriedem sort of,
15:51:46 efried cause I could use a delegate for that as well
15:51:53 mriedem the prelude generally feeds into highlights
15:52:10 mriedem the highlights have to be wordsmithed a bit for marketing speak
15:52:17 efried it's cruel that first draft of highlights is due the day of FF.
15:52:35 mriedem "the compute team leveraged the synergy of the cloud to...deprecate the xenapi driver"
15:52:45 mriedem efried: yup
15:52:50 mriedem i've complained about that before
15:53:08 mriedem smcginnis is good at suggestions on the highlights,
15:53:13 mriedem he's nearly executive level management at this point
15:53:19 mriedem he only speaks that langauge
15:53:19 stephenfin efried: I'm hoping to put together one of those internally to figure out what we need to test/document for Train/OSP16. I'll share it but it won't be 'til Wednesday week, which is probably too late?
15:53:38 mriedem stephenfin: FF is thursday
15:53:46 efried stephenfin: unless you accept that this task has been delegated to you.
15:54:02 efried then Thursday is the "deadline" for the "first draft", so Wednesday is a whole day early!
15:54:19 efried iow, it's all yours, and I don't need to vet/edit/revise it.
15:55:15 smcginnis efried: I'm not happy about the deadline either, but the OSF marketing folks said they needed that info earlier. :/
15:55:47 mriedem need to copy/paste into a chart asap
15:55:50 stephenfin efried, mriedem: okay, I'll do it Monday
15:56:17 mriedem maybe we should put an easter egg in the highlights...
15:56:21 mriedem see who's paying attention
15:56:22 efried I'm not blaming anyone, and I'm sure there's a "good" reason. It's just like, we're already all one-armed paper hangers already.
15:56:29 efried mriedem: love that idea.
15:56:45 mriedem "nova is deprecated, all hail serverless"
16:00:16 stephenfin There's one non-doc patch left in the remove-consoleauth series that I thought had already merged. If anyone has bandwidth to push it through (melwitt is +2), that'd be dandy https://review.opendev.org/#/c/652970/
16:00:29 stephenfin I'm just going to abandon the other two

Earlier   Later