| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-10 | |||
| 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 | |
| 19:50:07 | artom | I thought we had functional live migration tests? | |
| 19:50:49 | artom | Ah, nova/tests/functional/test_servers.py | |
| 19:50:53 | artom | Ignore me | |
| 19:51:26 | dansmith | imacdonn: I don't understand sean's concern or desire | |
| 19:53:57 | imacdonn | dansmith: thanks for commenting. One thing is still a bit fuzzy .. what does "work was done" mean? The way "ran" is currently implemented, it only counts how many rows were migrated *in the last batch*, so of you use the default of 50 at a time, ran will always end up as 0 | |
| 19:55:19 | imacdonn | ... because that's the only way it can break out of the loop | |
| 19:55:26 | dansmith | imacdonn: ran is the sum of all the "done" values from any migration it4eration right? | |
| 19:55:43 | dansmith | ran becoming nonzero is how you break out | |
| 19:55:45 | imacdonn | dansmith: no, only the last iteration | |
| 19:56:23 | imacdonn | dansmith: it gets reset to 0 at https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L718 | |
| 19:56:38 | dansmith | imacdonn: oh I see, but that's a bug I guess | |
| 19:56:47 | dansmith | from when this went from a fixed number to having an --until-done | |
| 19:56:49 | dansmith | or whatever | |
| 19:57:02 | dansmith | or the opposite, but you know what I mean | |
| 19:57:10 | sean-k-mooney | dansmith: we break out 1 of 2 way. ran becomes 0 or we pass --max-count in which case we do not loop as unlimited is false | |
| 19:57:11 | dansmith | so yeah you have to fix that for this to work | |
| 19:57:31 | imacdonn | dansmith: that's what I need to get nailed down ... I tried to fix that, in PS2, but made grenade blow up, becaused grenade needs the command to exit with status 0 | |
| 19:58:01 | dansmith | imacdonn: well, if this is wrong it's possible grenade is wrong | |
| 19:58:19 | jaypipes | mriedem: yesterday, which scheduler filter did you say already looked at instance metadata by querying the BuildRequest? | |
| 19:58:41 | mriedem | jaypipes: the one you're writing | |
| 19:58:42 | dansmith | oh wait, | |
| 19:58:45 | dansmith | maybe I'm remembering this now | |
| 19:58:54 | imacdonn | dansmith: it's possible, yes ... although it seems like if you run the command without --max-count, an outcome of exit status 0 (generally interpreted as success), is what would be expected | |
| 19:59:05 | dansmith | in the unlimited case, you really need the exit code to be zero | |
| 19:59:05 | dansmith | right | |
| 19:59:30 | dansmith | oof, this should be commented in here for sure | |
| 19:59:58 | dansmith | imacdonn: so maybe just keep the full count separate from ran and use that for the gate on exit 2 | |