| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-03 | |||
| 15:50:59 | mriedem | i think it's coming from websockify | |
| 15:52:29 | stephenfin | I think this is it https://github.com/openstack/nova/blob/master/nova/console/websocketproxy.py#L156-L157 | |
| 15:52:42 | mriedem | right that's raising the exception | |
| 15:52:49 | mriedem | and this is catching it https://github.com/novnc/websockify/blob/v0.8.0/websockify/websocket.py#L938 | |
| 15:52:49 | stephenfin | Ohh, gotcha | |
| 15:53:29 | stephenfin | Guess we could could configure | |
| 15:53:41 | stephenfin | *websocket proxy's logger in DevStack | |
| 15:53:51 | stephenfin | *websockify's logger | |
| 15:54:03 | ericyoung | with queens-3 coming up... can someone review https://review.openstack.org/#/c/407440/ | |
| 15:59:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309 | |
| 16:07:34 | openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Support aggregate affinity filters/weighers https://review.openstack.org/529135 | |
| 16:10:15 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: Deprecates support for Windows / Hyper-V Server 2012 https://review.openstack.org/530953 | |
| 16:19:15 | melwitt | mriedem: hey, thanks for all the reviews on consoles | |
| 16:21:23 | mriedem | stephenfin: couple questions inline https://review.openstack.org/#/c/514309/ | |
| 16:26:29 | stephenfin | mriedem: All done | |
| 16:26:48 | stephenfin | tl;dr: It's a copy-paste patch. The real work happens once it's in-tree | |
| 16:26:56 | stephenfin | mordred won't know what hit him | |
| 16:28:01 | mriedem | ok just waiting for the new job to hit the experimental queue | |
| 16:33:46 | mordred | stephenfin: uhoh | |
| 16:36:45 | stephenfin | mordred: Just more work for you that you won't have time for. Nothing to see here :) | |
| 16:37:13 | mordred | stephenfin: \o/ | |
| 16:37:17 | mordred | stephenfin: my favorite!!! | |
| 16:37:25 | dims | hey mriedem, did you see the OpenStack++ related papers? (ops / edge computing) they have details on how they extended nova for better live migration scenarios | |
| 16:37:44 | dims | mriedem : [8][9] at https://etherpad.openstack.org/p/2017_edge_computing_working_sessions_existing_gaps | |
| 16:38:08 | dims | LOL stephenfin. right on the money :) | |
| 16:38:39 | melwitt | mriedem: do you know if we have a cells meeting today? | |
| 16:38:57 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: zuul: Enable functional tests in gate https://review.openstack.org/530961 | |
| 16:39:09 | stephenfin | ralonsoh_: ^ | |
| 16:39:15 | mriedem | dims: nope haven't seen it | |
| 16:39:22 | mriedem | heads down until feature freeze | |
| 16:39:26 | mriedem | melwitt: i assume so | |
| 16:39:45 | stephenfin | ralonsoh_: Let's see how them functional tests fare. I'm assuming you don't need a full-fledged DevStack environment to test them? | |
| 16:40:43 | ralonsoh_ | stephenfin: not in os-vif, just simple command checks | |
| 16:41:27 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Use UEFI as the default boot for AArch64 https://review.openstack.org/489951 | |
| 16:41:28 | stephenfin | ralonsoh_: Sweet. That should test it so, I hope. If it does, I might squash it into the main patch | |
| 16:42:19 | ralonsoh_ | stephenfin: which patch? | |
| 16:42:38 | stephenfin | ralonsoh_: Oh, sorry https://review.openstack.org/#/c/530961/ | |
| 16:43:08 | ralonsoh_ | stephenfin: I mean, you said you need to squash this patch with othe one | |
| 16:43:22 | stephenfin | The preceding one https://review.openstack.org/#/c/484386/30 | |
| 16:43:33 | stephenfin | i.e. the one that adds said functional tests | |
| 16:43:58 | ralonsoh_ | stephenfin: I'll rebase mine to your patch | |
| 16:44:11 | stephenfin | No, you need functional tests first | |
| 16:44:33 | stephenfin | So they need to go in in that order or together | |
| 16:44:47 | jaypipes | dims: ugh. I hate when academics invent some new term like "cloudlet" and act like it's an industry-standard term. | |
| 16:44:48 | ralonsoh_ | stephenfin: perfect | |
| 16:44:53 | stephenfin | ralonsoh_: Should wait and see if I've done it properly first too :) | |
| 16:45:05 | ralonsoh_ | stephenfin: I'll wait for the result | |
| 16:45:27 | dims | jaypipes : y once you get past some interesting stuff in there | |
| 16:45:37 | dims | jaypipes : they have to justify grants :) | |
| 16:45:48 | jaypipes | dims: there's some flat-out incorrect stuff in there, too. :) | |
| 16:46:29 | dims | yep. i was focused on how/what they changed in Nova | |
| 16:47:04 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | |
| 16:47:18 | hrw | I hope I did that right | |
| 16:48:18 | jaypipes | dims: this picture perfectly represents the coherence of most edge-computing literature. | |
| 16:48:21 | jaypipes | dims: https://i.pinimg.com/originals/d7/cc/31/d7cc31df947e74fc98942bcdedb1de31.png | |
| 16:48:53 | dims | LOLOLOL | |
| 16:49:50 | ildikov | LOL | |
| 16:50:00 | ildikov | jaypipes: I thought cloudlet is an older one | |
| 16:50:52 | jaypipes | cloudlet. it's like saying a banana is a yellow orange. | |
| 16:52:10 | ildikov | you mean a lemon? | |
| 16:53:11 | ildikov | to which we Hungarians would say that it's a little yellow, a little sour, but it's ours... :) | |
| 16:53:39 | mriedem | https://www.youtube.com/watch?v=FT7YZQEHHVE | |
| 16:55:31 | jaypipes | ildikov: :) | |
| 16:59:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't log a warning for InstanceNotFound in detach_interface https://review.openstack.org/530969 | |
| 17:09:33 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Use UEFI as the default boot for AArch64 https://review.openstack.org/489951 | |
| 17:09:39 | stephenfin | claudiub: Mind taking a look at this? https://review.openstack.org/#/c/487405/ | |
| 17:10:40 | hrw | stephenfin: thanks for review. updated commit message and fixed that ':' I skipped | |
| 17:11:42 | stephenfin | hrw: Sweet. Still need a bug report though :) | |
| 17:11:49 | hrw | stephenfin: can be done ;d | |
| 17:11:52 | stephenfin | i.e. why this is necessary | |
| 17:11:57 | stephenfin | Cheers (y) | |
| 17:12:23 | hrw | https://bugs.launchpad.net/nova/+bug/1740824 already exists... | |
| 17:12:26 | hrw | will add to commit | |
| 17:12:29 | openstack | Launchpad bug 1740824 in OpenStack Compute (nova) "Make UEFI as the default properties for AArch64" [Undecided,New] | |
| 17:13:38 | stephenfin | hrw: Cool. If you were feeling generous, a little more info in that bug would be appreciated. Why is it essential, etc. | |
| 17:13:47 | hrw | sure | |
| 17:15:13 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Use UEFI as the default boot for AArch64 https://review.openstack.org/489951 | |
| 17:19:03 | hrw | stephenfin: added comment to the bug | |
| 17:19:22 | hrw | stephenfin: please read and tell me is it enough or should I expand it | |
| 17:19:27 | claudiub | stephenfin: sure, will do. and btw, there are plenty of OVS users on Windows. :) We've added OVS vif plugging support in nova some time ago. I think it was Mitaka. | |
| 17:19:47 | claudiub | or newton | |
| 17:19:51 | hrw | Mitaka... my favourite openstack release. | |
| 17:19:57 | hrw | due to uefi support ;) | |
| 17:20:00 | stephenfin | claudiub: Ah, so I wonder how come we're not using os-vif code for that? | |
| 17:20:11 | claudiub | stephenfin: we are | |
| 17:20:40 | claudiub | stephenfin: https://github.com/openstack/nova/blob/master/nova/virt/hyperv/vif.py#L88 | |
| 17:20:44 | stephenfin | claudiub: I'm confused. How does that work with the code that's there. Surely that's the whole point of that patch? | |
| 17:21:34 | stephenfin | Ahhhh... http://git.openstack.org/cgit/openstack/compute-hyperv/tree/compute_hyperv/nova/vif.py | |
| 17:26:27 | stephenfin | hrw: So to summarize, if you don't set that property, the instance doesn't boot? | |
| 17:26:50 | hrw | stephenfin: unless it is setup to boot kernel+initrd which no one does nowadays | |
| 17:27:44 | stephenfin | hrw: Cool. +2d | |
| 17:27:47 | melwitt | uh, what's happened to tempest job logs? can't filter by log level anymore | |
| 17:28:27 | hrw | stephenfin: cool! thanks | |
| 17:29:39 | hrw | now have to find someone else to take a look ;D | |
| 17:29:41 | claudiub | stephenfin: hm, I think the main point of the patch is to make os-vif a bit more OS agnostic. If you look in the vif_plug_ovs.ovs module, linux_net.device_exists is being used, even though that is also executed on Windows | |
| 17:30:04 | claudiub | and it also adds a Windows specific implementation for device_exists, which is nice. | |
| 17:31:39 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: Deprecates support for Windows / Hyper-V Server 2012 https://review.openstack.org/530953 | |
| 17:32:16 | hrw | stephenfin: can you look at https://review.openstack.org/#/c/530965/ one too? It is probably wrong way of doing it and I am open for suggestions | |
| 17:39:09 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530980 | |
| 17:45:16 | ralonsoh_ | stephenfin: I don't know why it's failing | |