| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-23 | |||
| 13:36:49 | dansmith | jaypipes: your concern is that it can't run in cell conductor right? | |
| 13:37:02 | jaypipes | dansmith: sorry, it's also the end of the day for me so I very well might just be tired :) | |
| 13:37:08 | jaypipes | dansmith: yeah, that is my concern | |
| 13:37:27 | dansmith | jaypipes: and I'm telling you it is running in superconductor, so I don't think there's a problem | |
| 13:37:46 | jaypipes | dansmith: k, got it. I just wanted to double-check, that's all. | |
| 13:39:00 | dansmith | also, the way devstack in the gate runs, it should fail if he's doing something that requires api db access | |
| 13:39:51 | jaypipes | oh, good to know we have the safeguard in place. nice. ++ | |
| 13:40:57 | jaypipes | mriedem: k, that series +W'd | |
| 13:41:51 | mriedem | jaypipes: thanks | |
| 13:42:24 | stephenfin | sean-k-mooney: Remind me - we can't guarantee that the portbinding extension is enabled, right? https://review.openstack.org/#/c/357726/ | |
| 13:42:29 | stephenfin | i.e. it's still optional | |
| 13:43:57 | stephenfin | wait - after reading it, it's clear we want to enforce this. nvm me :) | |
| 13:44:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Assume neutron port_binding extensions enabled https://review.openstack.org/357726 | |
| 13:47:38 | mriedem | stephenfin: the assumption is that no one is *not* enabling the portbinding extension | |
| 13:47:57 | stephenfin | mriedem: Yeah, I read the reno. That's a fair assumption | |
| 13:48:44 | mriedem | like i said in my comment, that's a thing we won't know about until queens-eol when someone says, "i just upgraded to queens and now it's saying i need the port binding extension but we don't use that b/c $some_reason" | |
| 13:49:07 | mriedem | "because GLUON!" | |
| 13:49:31 | mriedem | nvm, if you're using gluon you're not using this code anyway | |
| 13:50:43 | mriedem | dansmith: i will commence the backporting | |
| 13:50:49 | dansmith | cool | |
| 13:52:16 | auggy | ratailor stephenfin Ugh, I did a bunch of work on bug #1538011 with mtreinish and came to the conclusion that the added complexity was not worth the small gain. But others may have different opinions | |
| 13:52:18 | openstack | bug 1538011 in OpenStack Compute (nova) "Metadata keys are not case sensitive" [Medium,In progress] https://launchpad.net/bugs/1538011 - Assigned to Rajesh Tailor (ratailor) | |
| 13:55:37 | auggy | But yeah, I'm not doing much in the way of openstack these days :) I'm doing more "Meta" openstack. Data science stuff around developing better ways to describe open source community activity. | |
| 13:56:05 | openstackgerrit | Ed Leafe proposed openstack/nova master: docs: Document the scheduler workflow https://review.openstack.org/475810 | |
| 13:56:10 | edleafe | stephenfin: cdent: updated ^^ | |
| 13:56:44 | stephenfin | auggy: Aye, I recall looking at the same bug and quickly dropping it. ratailor: you've been warned | |
| 13:58:17 | auggy | ratailor stephenfin I'll update it later today with the work mtreinish and I did. I abandoned a lot of patches haha | |
| 14:05:08 | auggy | Oh it looks like the patches and blueprint are linked already | |
| 14:05:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix key_name note formatting https://review.openstack.org/496718 | |
| 14:05:15 | dansmith | mriedem: do you know if this person is on irc? https://review.openstack.org/#/c/496202/ | |
| 14:06:31 | mriedem | dansmith: don't think so | |
| 14:06:37 | mriedem | gibi would know but he's on vacation | |
| 14:06:44 | mriedem | ildikov might know | |
| 14:06:53 | dansmith | okay I wonder if I should just update that patch myself | |
| 14:07:33 | mriedem | is it an rc2 thing? | |
| 14:08:09 | ildikov | dansmith: mriedem: don't think so, but will try to double check | |
| 14:08:18 | dansmith | mriedem: it's another thing that is broken and needs fixing right? | |
| 14:08:48 | dansmith | I actually thought this was in the middle and a fix was on top, but I guess this is yet another broken bit | |
| 14:09:41 | mriedem | i'd have to dig into it, it's not clear what the issue is from the commit message | |
| 14:10:24 | dansmith | well you can see the assertions commented out of the test: https://review.openstack.org/#/c/496202/1/nova/tests/functional/test_servers.py | |
| 14:10:46 | dansmith | I'm not really sure why this is force specific though | |
| 14:10:50 | mriedem | dansmith: is it basically the same thing you already fixed, but different place? | |
| 14:11:02 | dansmith | maybe I should re-run this on top of everything else we have fixed recently and make sure | |
| 14:11:46 | mriedem | yeah bc you're doing it in _post_live_migration _post_live_migration | |
| 14:11:48 | mriedem | oops | |
| 14:11:51 | mriedem | https://review.openstack.org/#/c/496032/4/nova/compute/manager.py | |
| 14:12:47 | mriedem | i need to see where that live_migration_force_complete is called from | |
| 14:13:18 | mriedem | oh i guess the api | |
| 14:13:45 | mriedem | well, that just calls through to the driver to do post-copy right? so the other thread that's waiting for live migration to complete should then see it complete and eventually get to _post_live_migration, which is where your fix is | |
| 14:13:52 | mriedem | so i think your fix probably already handles this | |
| 14:16:33 | dansmith | yeah test fails now, so let me swap the assertions and make sure they're all good | |
| 14:18:39 | dansmith | yeah, it's all good now | |
| 14:18:55 | dansmith | so I'll fix it up to swap the assertions and fix up the epic long sleep thing | |
| 14:23:54 | openstackgerrit | Dan Smith proposed openstack/nova master: Add functional migrate force_complete test https://review.openstack.org/496202 | |
| 14:23:58 | dansmith | mriedem: ^ | |
| 14:24:06 | dansmith | so you want to wait on that since it's just a test I assume? | |
| 14:26:15 | mriedem | yeah i don't think we need to backport and hold up rc2 for a test | |
| 14:29:33 | mriedem | backports are all up too now https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike+topic:bug/1712008 | |
| 14:30:05 | mriedem | johnthetubaguy: are you around? | |
| 14:44:12 | openstackgerrit | Eric Fried proposed openstack/nova master: Monkey patch the blockdiag extension https://review.openstack.org/476159 | |
| 14:57:06 | openstackgerrit | Merged openstack/nova master: Allocate resources on forced dest host during live migration https://review.openstack.org/496031 | |
| 14:59:56 | clarkb | mriedem: did you see my comment about the cell1 cond logs? | |
| 15:05:59 | openstackgerrit | Merged openstack/nova master: Remove source node allocation after live migration completes https://review.openstack.org/496032 | |
| 15:06:32 | mriedem | clarkb: nope | |
| 15:07:17 | clarkb | mriedem: lwft it as acomment on the change but the format is funny in that gile so indexing wont work as is | |
| 15:07:27 | clarkb | *file | |
| 15:07:59 | mriedem | it's different from screen-n-cond? | |
| 15:08:43 | mriedem | oh it's just screen-n-cond-cell1? | |
| 15:08:44 | mriedem | weird | |
| 15:19:34 | openstackgerrit | Merged openstack/nova master: Restrict live migration to same cell https://review.openstack.org/496419 | |
| 15:20:07 | openstackgerrit | Merged openstack/nova master: Clarifying node_uuid usage in ironic driver. https://review.openstack.org/485803 | |
| 15:29:35 | cfriesen_ | is there a novaclient interface to do a PUT on /os-quota-class-sets ? | |
| 15:34:13 | dims | cfriesen_ : this one? http://git.openstack.org/cgit/openstack/python-novaclient/tree/novaclient/v2/quota_classes.py#n47 | |
| 15:37:28 | openstackgerrit | Merged openstack/nova master: Pull out code that builds VIF in _build_network_info_model https://review.openstack.org/466449 | |
| 15:37:29 | cfriesen_ | dims: thanks, that's the one. for some reason I'm blind. incidentally that API missing from openstackclient | |
| 15:38:08 | dims | cfriesen_ : hound is indispensable for me :) http://codesearch.openstack.org/ | |
| 15:42:13 | cfriesen_ | dims: oooh, I like it. | |
| 15:51:58 | stephenfin | mriedem, dansmith: ooh, should we deprecate the cells v1 nova-manage commands? https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1030 | |
| 15:53:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add missing unit tests for FilterScheduler._get_all_host_states https://review.openstack.org/496803 | |
| 15:54:02 | mriedem | stephenfin: yes, should have done that when i deprecated cells v1 | |
| 15:54:18 | stephenfin | mriedem: I'll do that now so. RC2 material? | |
| 15:55:10 | mriedem | i don't know that it needs to be rc2, but realize it's kind of weird for a deprecation timer if the server side code is deprecated in pike and the CLI is deprecated in queens | |
| 15:55:42 | mriedem | you could still theoretically run the cli against an older installation that still has cells v1 code | |
| 15:55:47 | mriedem | in a venv or something | |
| 15:56:55 | stephenfin | True. I'll shove it up anyway. Can "backport" if it makes sense | |
| 15:57:01 | mriedem | yeah, thanks | |
| 16:01:48 | clarkb | mriedem: ya just that one service has a weird log, once its fixed I can approve the change to index it | |
| 16:02:19 | mriedem | ok i'll investigate | |
| 16:08:49 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'cell' commands https://review.openstack.org/496815 | |
| 16:09:14 | stephenfin | mriedem, dansmith: ^ | |
| 16:09:57 | stephenfin | I'd opt for bringing it into Pike, but only because it makes cliff migration in Queens simpler. Good to get rid of it either way | |
| 16:11:32 | mriedem | stephenfin: comments inline | |
| 16:11:34 | mriedem | dansmith: ^ | |
| 16:16:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'cell' commands https://review.openstack.org/496815 | |
| 16:17:00 | stephenfin | mriedem: I think that should do it ^ | |
| 16:19:04 | sean-k-mooney | stephenfin: regarding portbinding i dont think there is a neutron that is still supported that does not use portbining | |
| 16:19:35 | stephenfin | sean-k-mooney: Yeah, seems the decision is that we expect portbinding to be there be default now | |
| 16:20:09 | sean-k-mooney | stephenfin: i can ask on the neutorn channel but if you are using ml2 it is required. it may be posible to not use it if you use a monolitic plugin | |
| 16:20:31 | sean-k-mooney | monolitic plugins are effectivly deprected | |
| 16:21:06 | sean-k-mooney | to be part of the neutron stadium i belive you have to intergrate via ml2. that was a requirement for ovn anyway | |