Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-10
16:36:58 dansmith mriedem: it used to though yeah?
16:37:04 mriedem in the long long ago i guess,
16:37:07 mriedem oh well,
16:37:12 mriedem the facade would i guess yeah...
16:37:30 dansmith point being, 0.0 is nonsense
16:37:30 mriedem the ComputeNode._from_db_object would read from config if 0.0 in the db
16:37:38 jaypipes dansmith: ++
16:37:55 dansmith so we could leave that, and only set it if they override per compute.. same logic as placement
16:38:05 dansmith and anything existing with 0.0 gets the same treatment as usual I guess
16:38:26 mriedem leave the facade in ComputeNode._from_db_object?
16:38:49 dansmith i imagine it won't affect compute's use if it is ignoring what is set on the object, so .. sure?
16:38:58 sean-k-mooney stephenfin: for https://github.com/openstack/nova/blob/0163b9bfb54aaa89b0574c86e7fd36321eebccfe/nova/scheduler/filters/numa_topology_filter.py#L91-L102 can you think of a use case wehre we actully need the allcoation ratiios here
16:39:33 sean-k-mooney stephenfin: we are not allowed to over subsibe against our selves wehn fitting to a host so setting them to 1.0 i think would be valid
16:42:05 sean-k-mooney stephenfin: i would have to go through the code to check but i think we could make the numa topolgy filter work without them.
16:42:50 sean-k-mooney dansmith: although it sound like we dont need to remove them if we go wtih the facade right
16:43:14 dansmith sean-k-mooney: it won't be right if you do
16:43:32 dansmith sean-k-mooney: because if people set the ratio in placement per-compute, which is what we're trying to enable with all of this work,
16:43:38 dansmith the filter will consider a value other than what is in placement
16:44:04 dansmith I'm guessing we don't get back ratios in /a_c, but if we did, we could update our host states before we call the filters
16:44:12 sean-k-mooney dansmith: yes but this is not using the placement value anyway
16:44:18 jaypipes mriedem: why would we keep the facade stuff in ComputeNode._from_db_object()?
16:44:38 dansmith sean-k-mooney: right but right now they have to be the same
16:44:39 dansmith sean-k-mooney: in the future they will not be
16:44:52 dansmith jaypipes: see the discussion just now on the filters that use it, and existing computes in the db with 0.0 set
16:45:21 mriedem jaypipes: we either need to leave that or online data migrate the 0.0 entries from existing records in the db on read
16:45:21 sean-k-mooney dansmith: placement woudl have already filtered out any host that did not pass its allocation right. so we should not have to check twice and sicne we can oversubsibe againat ourselve an allocation ration of 1 i think would still be correct
16:45:46 sean-k-mooney dansmith: anywway its not important right now i guess
16:45:49 mriedem jaypipes: because the compute won't deal with those fields on the object, and if not set, the scheduler reads them from config via the facade
16:45:55 dansmith sean-k-mooney: I didn't look to see what that code in the filter was doing, so maybe?
16:46:00 dansmith sean-k-mooney: if so, we just remove it right?
16:46:43 sean-k-mooney dansmith: ya i think soo but i would have to double check the hardware.numa_fit_instance_to_host fucntion first
16:47:28 dansmith we get back the resource summaries from the providers in /a_c, so including the allocation_ratios in there might be useful for things like this and for the weigher case
16:48:30 sean-k-mooney dansmith: ya. the filters dont currently have access to the allocation candiates today is that correct
16:48:39 sean-k-mooney unless there in the spec_objec?
16:48:42 dansmith sean-k-mooney: that isn't what I'm saying
16:49:01 dansmith I'm saying the scheduler, when it gets back candidates, gets a summary of all covered providers, with inventory information
16:49:11 dansmith if that included the ratios, it could update host_states before calling the filter loop
16:49:46 jaypipes sean-k-mooney: pls see my comment on https://review.openstack.org/#/c/609414/
16:49:59 sean-k-mooney oh ok that would work too ya i was assuming you were suggsting passing in the candiates but your way we do that update once and dont have to update any code in the filters
16:50:52 sean-k-mooney jaypipes: there isnt a bug number because i just got pingged on irc this morning but i can open one
16:51:11 dansmith mriedem: melwitt tssurya: cells meeting today?
16:51:17 mriedem nack
16:51:24 mriedem we might want to just cancel that meeting
16:51:39 dansmith I'd also be fine with that
16:51:50 dansmith I think I suggested that last year even
16:53:39 jaypipes sean-k-mooney: yes pls. if this is truly a "currently broken for hardware offloaded ovs" scenario, it definitely should be a bug.
16:54:50 tssurya dansmith: no problems in cancelling
16:55:03 sean-k-mooney its broke for all ovs backends that use ip command because api _get_impl on linux retrun a module instead of the insatnce of the pyroute2 class
16:56:12 mriedem dansmith: you want to propose the change to cancel the meeting or want me to?
16:56:13 sean-k-mooney jaypipes: os that is ovs + iptables or ovs + hardware offloads. i have added some extra test to catch this case
16:56:22 dansmith melwitt: what do you think about canceling the cells meeting altogether and making it ad-hoc as needed?
16:56:35 dansmith mriedem: I figure we don't need to make a federal case out of it
16:56:42 dansmith if everyone agrees, we just take it off the schedule
16:56:45 mriedem dansmith: it's literally in a schedule though
16:56:56 dansmith I know
16:56:56 mriedem http://git.openstack.org/cgit/openstack-infra/irc-meetings/tree/meetings/nova-cells-v2-meeting.yaml
16:57:02 dansmith oh,
16:57:04 dansmith that schedule
16:57:16 mriedem yeah, free up the time slot in that channel if we're not going to use it
16:57:18 dansmith I thought it was just on the old wiki list
16:57:28 mriedem no this is very official and federale
16:57:34 dansmith I'll propose.. I need to do something useful today
17:01:44 melwitt dansmith: yup sounds ok to me
17:01:57 dansmith https://review.openstack.org/#/c/609496/
17:09:35 mriedem hmm live migration failure in the gate, not something i've seen before i don't think, looks like it was aborted but i'm not sure why
17:09:35 mriedem http://logs.openstack.org/31/606031/4/check/nova-live-migration/9d106bb/logs/subnode-2/libvirt/libvirtd.txt.gz#_2018-10-10_15_27_01_313
17:09:40 mriedem 2018-10-10 15:27:01.313+0000: 18210: error : qemuMigrationFinish:5533 : migration successfully aborted
17:14:43 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: API _get_impl() for Linux should return PyRoute2() object https://review.openstack.org/609354
17:26:06 orange_julius I've been looking into ways to use ARM images inside of an Openstack installation and was wondering if anybody had any experience with this. From what I've seen we basically have two options: Purchase an ARM server and set up as a compute node. Configure a server to use qemu instead of KVM and virtualize. Is it possible to tell a compute node to us
17:26:07 orange_julius e both kvm and qemu depending on the image? Is there a better way to accomplish this?
17:26:36 openstackgerrit sean mooney proposed openstack/os-vif master: clean up ip_command interface https://review.openstack.org/609414
17:26:37 openstackgerrit sean mooney proposed openstack/os-vif master: add support for generic tap device plug https://review.openstack.org/602384
17:27:12 sean-k-mooney jaypipes: done ^ let me know if there is anything else you would like me to change
17:32:46 jaypipes sean-k-mooney: +Wd
17:34:15 sean-k-mooney jaypipes: thanks i should have created the bug when it was reported to me on irc instead of starting on the patch but i need to base my other patch on top of it anyway
17:34:26 jaypipes sean-k-mooney: no worries man
17:36:01 sean-k-mooney ok so time for dinner ill be back online later
17:42:39 cfriesen orange_julius: currently I think you'd need to make it a nova-compute config option. theoretically you could make it depend on the image properties, but I think that'd be a feature to be added.
17:43:53 orange_julius cfriesen: Just so I understand, you are saying that we'd have to change the nova config on a hypervisor to use qemu instead of KVM. Basically dedicating that entire machine to QEMU workloads instead of KVM
17:44:55 mriedem orange_julius: mnaser can probably help here
17:45:01 mriedem i think he's running arm nodes somewhere
17:52:53 cfriesen orange_julius: the libvirt driver in nova looks at caps.host.cpu.arch which comes from libvirt. so you'd need to make that return an arm architecture I think.
18:30:41 larsks Hey folks. There are comments in the nova history that suggest the quota_usage_refresh manage command was replaced by API functionality. What is that API?
18:36:05 larsks stephenfin: ...if you're around, they were your comments :) ^^^^
18:39:40 melwitt larsks: as of Pike, we don't track quota usages separately from resource counts, so there's no notion of refreshing quota. we only use resource counting
18:40:09 melwitt so quota can't get out-of-sync
18:40:39 melwitt are you asking about a version earlier than Pike?
18:46:24 larsks melwitt: pike, actually, so that's good to know. I'm an indirect conduit for the issue right now, so I will need to wait until I get my hands on the environment in question I guess before I can better assess what's going on. Thanks!
18:47:14 melwitt larsks: ok, feel free to ping me if you have more questions
18:51:31 mriedem dansmith: https://github.com/kk7ds/openstack-gerrit-dashboard/pull/29
18:51:39 dansmith mriedem: yep will look in a sec
18:51:46 dansmith mriedem: was going to ask you this morning if you had fixed that yet :D
18:54:04 dansmith omg it works!
19:03:42 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Handle volume API failure in post_live_migration https://review.openstack.org/609517
19:08:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix NoneType error in _notify_volume_usage_detach https://review.openstack.org/609518
19:13:36 mriedem dansmith: i needed something to do besides review code and/or specs
19:25:33 openstackgerrit Merged openstack/nova-specs master: Update blueprint name so spec matches launchpad https://review.openstack.org/607347
19:27:59 imacdonn dansmith: I implemented what I understood from our discussion yesterday at https://review.openstack.org/608091 . It needs a little polish, but sean-k-mooney wants to use new the exit status if there are any exceptions, whether or not any migrations may still be pending. I don't think we can do that, because it can't be automated
19:41:09 mriedem hey gang, two easy +Ws https://review.openstack.org/#/c/608802/ https://review.openstack.org/#/c/609467/
19:48:19 melwitt dansmith, mriedem: I linked my nova-consoleauth patch on L52 here that is ready for subteam review https://etherpad.openstack.org/p/stein-nova-subteam-tracking

Earlier   Later