Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-04
15:41:13 efried cfriesen: dansmith will drop a URL in here, ping all of us, and get spanked by Sigyn
15:41:15 cfriesen I'll cover for jack
15:58:30 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova show when a cell is down https://review.openstack.org/591658
16:02:29 efried gibi: You still around?
16:02:44 gibi efried: yeah for about 10 minutes
16:02:49 gibi efried: but go ahead
16:03:01 efried okay, that may not be enough. I wanted to walk through the allocation_request_version nonsense a bit.
16:03:41 gibi efried: even if we figure out that we want to drop allocation_request_version I would like to do that separately as that a surgery again
16:04:06 efried it's just, I see it as being pretty much unnecessary/unrelated to this patch.
16:04:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ignore forcing of live migration for nested instance https://review.openstack.org/605785
16:04:13 openstackgerrit Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050
16:04:13 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ignore forcing of evacuation for nested instance https://review.openstack.org/606111
16:04:14 openstackgerrit Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125
16:04:30 efried I guess that alone isn't worth holding it up since jay is +2 at this point.
16:04:40 efried And I guess I do see that it becomes necessary again for nested.
16:04:48 efried it's just a weird transition.
16:06:18 gibi efried: totally agree that allocation_request_version is strange but I'm not 100% sure we simply drop it and fix the version in the report client globally
16:06:57 efried gibi: IIUC, it's necessary whenever there's a possibility of the conductor and the scheduler being at different versions.
16:07:40 efried ...versions of nova, leading to them being aware of different versions of placement. Placement ought to always be at least up to date with the later of the two.
16:08:07 efried I assume that's still a possible scenario, for like rolling upgrades or whatever.
16:08:39 gibi efried: I'm not sure we support that scheduler and conductor on different version. What complicates it is that we send alternates around in RPC messages for reschedule
16:08:48 efried I'm guessing there's a rule/guideline, like you have to upgrade the computes before the scheduler, or vice versa? dansmith do you know that answer?
16:09:04 gibi efried: controller first is the ruel
16:09:05 gibi rule
16:09:35 gibi but I think controller services needs to be updated at once but I'm not sure
16:09:48 gibi I have to run now. talk to you tomorrow
16:10:07 efried thx gibi
16:10:28 gibi efried: thank you
16:13:21 openstackgerrit Merged openstack/nova master: Skip test_parallel_evacuate_with_server_group until fixed https://review.openstack.org/607620
16:13:55 openstackgerrit Jack Ding proposed openstack/nova master: Handle missing marker during online data migration https://review.openstack.org/605164
16:17:36 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova service-list when a cell is down https://review.openstack.org/584829
16:23:11 dansmith efried: control services first and together, computes last and in any order you want
16:23:49 efried dansmith: Okay, so the reason for allocation_request_version is in case computes are currently downlevel from controller and/or each other. Thanks.
16:24:00 openstackgerrit Jack Ding proposed openstack/nova master: Add HPET timer support for x86 guests https://review.openstack.org/605902
16:24:09 dansmith efried: yeah
16:24:37 bauzas efried: https://docs.openstack.org/nova/latest/user/upgrade.html
16:24:39 bauzas HTH
16:25:20 bauzas anyway, calling it a day
16:25:40 bauzas do we have some hangout at 8pm UTC so ? if so, efried, could you please ping me on IRC ?
16:25:54 efried bauzas: Yes, dansmith or I will ping you.
16:26:04 bauzas cool cool
16:26:07 bauzas see you later then.
16:26:11 efried l8r
16:26:24 efried or "bis später" :)
16:26:54 bauzas heh, I won't finally have opportunity to speak German this year, but okay
16:41:45 openstackgerrit Gaudenz Steinlin proposed openstack/nova master: Ignore misleading resource updates from virt driver https://review.openstack.org/523006
16:48:16 openstackgerrit melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068
16:52:00 openstackgerrit melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068
17:04:15 sean-k-mooney dansmith: did ye have that hangout on hpet extraspec vs trait vs both ?
17:04:23 dansmith sean-k-mooney: in two hours
17:05:00 sean-k-mooney oh ok cool in that case i have time to read the specs properly
17:10:38 efried dansmith: three hours?
17:10:53 dansmith efried: 2000 UTC is 1200 PDT no?
17:11:15 dansmith oh sorry, I was looking at dublin time
17:11:17 dansmith yeah three hours
17:12:01 sean-k-mooney ya dublin is currently on irish summer time
17:12:14 sean-k-mooney hopefully daylight saving will no longer be a thing soon
17:14:26 openstackgerrit Merged openstack/nova stable/ocata: nova-manage - fix online_data_migrations counts https://review.openstack.org/605842
17:29:39 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion bump for handling-down-cell https://review.openstack.org/591657
17:41:53 openstackgerrit Merged openstack/nova master: Add attach kwarg to base/nova-net allocate_for_instance methods https://review.openstack.org/605464
17:43:30 openstackgerrit melanie witt proposed openstack/nova master: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/608040
18:06:55 openstackgerrit melanie witt proposed openstack/nova master: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/608040
18:11:10 openstackgerrit Merged openstack/nova master: consumer gen: support claim_resources https://review.openstack.org/583667
18:25:25 openstackgerrit Vlad Gusev proposed openstack/nova master: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/555160
18:38:06 jaypipes melwitt, dansmith, other stable cores... https://review.openstack.org/#/c/607626/ would be great to review. thanks in advance.
18:40:10 s10 Please approve back ports for https://review.openstack.org/#/q/I27160140cbccf3e6914aed2dcff54d547e446c1f and https://review.openstack.org/#/q/topic:bug/1785827+(status:open+OR+status:merged)
18:46:07 melwitt jaypipes: +W
18:52:02 melwitt mriedem: this backport just needs an upgrade of your +1 to a +2/+W https://review.openstack.org/592337
18:59:18 jaypipes melwitt: danke
18:59:29 jaypipes efried: left a question/thought for you on https://review.openstack.org/#/c/605902/
19:01:32 efried jaypipes: For the sake of keeping things together, perhaps we should have that discussion on the spec.
19:02:23 jaypipes efried: or the hangout later...
19:02:24 efried jaypipes: I agree it makes sense to allow the trait to be in the image. Not sure that means we shouldn't also support it in the flavor, or that we should allow it to be present in the flavor and still hit a non-capable host.
19:02:38 jaypipes efried: ack, totes
19:03:03 jaypipes efried: and it's not like we have the ability to say "this trait can only go in an image metadata k/v and not a flavor extra spec k/v..."
19:03:12 efried jaypipes: I assume we would support something like hw:hpet=True in image meta.
19:03:38 efried so really any of the three options in the spec could be applied to both flavor and image
19:03:44 jaypipes efried: I was just pointing out that for *some* of these things, the trait makes more sense as a constraint that is derived from an image, versus a constraint coming from a flavor...
19:03:55 efried oh, yeah, totally agree with you there.
19:04:17 efried If the op wants the flavor to be flexible in terms of which arch it lands on, don't put arch-specific stuff in it.
19:04:24 efried that seems like a fair rule of thumb
19:04:57 jaypipes ya
19:05:28 jaypipes efried: I guess what I'm saying is I don't think a decision on the traits thing should hold up jding's patch.
19:06:05 efried jaypipes: no, but the decision about whether to fail if I see the request but can't fulfil it.
19:06:13 efried ^ should hold up the patch
19:06:34 jaypipes efried: but again, if the guest/image arch isn't x86, the flavor's HPET extra spec makes no difference...
19:06:53 efried so that's where I think we disagree.
19:07:00 efried If you've asked for a thing in the flavor, you should get the thing, or fail.
19:07:10 efried unless you want to call it hw:hpet_if_you_can_please
19:08:55 efried jaypipes: I added a note here: https://review.openstack.org/#/c/607989/1/specs/stein/approved/support-hpet-on-guest.rst@23
19:09:04 dansmith are you guys cheating?
19:09:21 efried ah dammit, I thought you said dansmith wasn't here
19:09:31 dansmith I'm always here.
19:19:32 jaypipes efried: sorry, standup meetings...
19:19:46 efried Well, you're a standup kind of guy.
19:23:48 jaypipes lol
19:25:33 efried btw, the irony is not lost on me that standup meetings, which are supposed to be standup so they're short, are consuming your whole day.
19:25:39 mriedem fwiw, translating flavor extra specs that the driver needs to placement traits in a pre-request filter doesn't seem too hair brained, we've talked about the same kind of thing for translating volume multiattach boot requests to query allocation candidates via compute node capabilities
19:26:26 dansmith mriedem: yeah I said that in the meeting
19:26:32 dansmith it's basically what request filters are for

Earlier   Later