| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-05 | |||
| 10:38:17 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969 | |
| 10:40:40 | bauzas | gibi: remember me, do we document in the API docs something like https://review.opendev.org/#/c/630723/4/nova/api/openstack/compute/migrate_server.py ? | |
| 10:40:48 | bauzas | not the API guide I mean | |
| 10:41:23 | bauzas | I can read https://docs.openstack.org/neutron/latest/admin/config-qos-min-bw.html tho | |
| 10:41:45 | bauzas | and https://developer.openstack.org/api-guide/compute/port_with_resource_request.html document what's missing | |
| 10:42:02 | bauzas | but I wonder if within the API docs, we said that too | |
| 10:42:15 | bauzas | I mean api-ref | |
| 10:42:19 | bauzas | lemme check actually | |
| 10:44:13 | bauzas | nope, we don't (eg. https://developer.openstack.org/api-ref/compute/?expanded=resize-server-resize-action-detail#resize-server-resize-action ) | |
| 10:44:31 | bauzas | anyway, not a big deal, it's just for internal docs | |
| 10:44:49 | bauzas | gibi: FWIW, ping me if you have any changes about ^ btw. | |
| 11:04:47 | mdbooth | >>> x.config_drive = True | |
| 11:04:47 | mdbooth | u'False' | |
| 11:04:47 | mdbooth | >>> x.config_drive | |
| 11:04:47 | mdbooth | >>> x.config_drive = False | |
| 11:04:56 | mdbooth | Don't you just love python? | |
| 11:05:06 | mdbooth | And that field has the wrong type, of course | |
| 11:06:06 | mdbooth | True | |
| 11:06:06 | mdbooth | >>> bool(x.config_drive) | |
| 11:06:06 | mdbooth | >>> x.config_drive = False | |
| 11:06:33 | openstackgerrit | Merged openstack/nova master: api-ref: Fix a broken link https://review.opendev.org/669248 | |
| 11:47:51 | stephenfin | kashyap: Replied to https://review.opendev.org/#/c/669129/ | |
| 11:48:12 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN https://review.opendev.org/669176 | |
| 11:51:04 | gibi | bauzas: we have the limitation separately, not in the api-ref. I think it was intentional as it is expected to be a temporary limitation | |
| 11:51:34 | gibi | bauzas: the actual implementation of server move operation still needs my time. it is started up here https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/support-move-ops-with-qos-ports | |
| 11:52:00 | gibi | bauzas: but I was busy with https://review.opendev.org/#/c/637955/ | |
| 12:29:27 | kashyap | stephenfin: Will respond (saddled with a couple off meetings); thanks for the thoughtful reply. | |
| 12:41:34 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__() https://review.opendev.org/668564 | |
| 12:47:29 | bauzas | gibi: ack ok | |
| 13:05:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857 | |
| 14:12:42 | mriedem | lyarwood: have you actually hit a recreate for this? https://review.opendev.org/#/q/I15a7c13edf78884ec223fd531a78a341106b41b8 | |
| 14:15:04 | lyarwood | mriedem: did you mean reason? | |
| 14:15:32 | mriedem | no, | |
| 14:15:40 | mriedem | when that was reported there was no recreate in the bug for it, | |
| 14:15:45 | mriedem | and i was unable to recreate it in a functional test | |
| 14:15:54 | mriedem | so i'm hesitant to backport that without knowing how to recreate it | |
| 14:15:58 | lyarwood | mriedem: ah reproducer | |
| 14:16:19 | mriedem | yeah | |
| 14:16:21 | lyarwood | mriedem: no, that's fair it just looked like a trivial thing to change | |
| 14:16:36 | mriedem | ok i'd rather not backport it unless we have someone else reporting the problem | |
| 14:16:59 | lyarwood | mriedem: ack that's fine, I'll take a look next week at reproducing, if I can't I'll close | |
| 14:18:56 | mriedem | btw it occurs to me now we maybe shouldn't have merged this https://review.opendev.org/#/c/663887/ | |
| 14:19:11 | mriedem | b/c i can't run fast8/pep8 on my bionic vm (py27 and py36) on rocky changes now | |
| 14:19:13 | mriedem | stephenfin: ^ | |
| 14:21:40 | stephenfin | Because bionic doesn't support py3.5? | |
| 14:22:02 | stephenfin | It's a tough one. Without it, I can't run the same on my Fedora 30 machine for the reasons in the commit message | |
| 14:23:15 | openstack | bug 1825537 in OpenStack Compute (nova) rocky "finish_resize failures incorrectly revert allocations" [Medium,Confirmed] https://launchpad.net/bugs/1825537 | |
| 14:23:15 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Add functional recreate test for regression bug 1825537 https://review.opendev.org/669361 | |
| 14:23:18 | mriedem | rocky doesn't support py37 at all though right? | |
| 14:23:28 | mriedem | rocky tox.ini at least has py36 targets | |
| 14:23:47 | sean-k-mooney | stephenfin: it use py36 | |
| 14:23:51 | mriedem | iow, if you're doing rocky stuff and only have 3.7, you're going to need another vm | |
| 14:24:01 | mriedem | bionic has 2.7 and 3.6 | |
| 14:24:12 | stephenfin | It doesn't, no, but the use of python3 means "give me any version of Python 3.x" | |
| 14:24:27 | sean-k-mooney | stephenfin: we dont support py3.5 anymore | |
| 14:24:34 | sean-k-mooney | its not part of the train PTI | |
| 14:24:40 | stephenfin | We did in rocky though | |
| 14:24:48 | sean-k-mooney | rocky proably yes | |
| 14:24:48 | stephenfin | That's a stable-only patch | |
| 14:25:40 | stephenfin | mriedem: Could we use basepython = python3.6 instead, assuming that's available on both Bionic and Xenial (or whatever 16.04 is?) | |
| 14:25:44 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Revert "tox: Use basepython of python3.5" https://review.opendev.org/669362 | |
| 14:25:58 | mriedem | stephenfin: in rocky i don't think so, | |
| 14:26:12 | mriedem | because rocky supported (in the gate and the tox default env list) 27 and 35 | |
| 14:26:17 | mriedem | but has a 36 target | |
| 14:26:39 | mriedem | there is no official 37 test targeting in rocky though | |
| 14:27:04 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Boot a VM with an unaddressed port https://review.opendev.org/641670 | |
| 14:27:06 | mriedem | so like i said, i think if you're doing rocky changes and only have 37, you'll just need another dev vm for working on those older branches (which is what i have to do for ocata changes at this point) | |
| 14:27:12 | mriedem | i have a xenial vm just for doing ocata stuff now | |
| 14:27:44 | mriedem | xenial less for the python thing but more b/c of native packages like postgresql | |
| 15:04:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove nova-consoleauth https://review.opendev.org/652969 | |
| 15:07:27 | stephenfin | lyarwood: The stable/stein variant of this has merged now. Could you tackle this one? https://review.opendev.org/#/c/667355/ | |
| 15:20:58 | dansmith | sean-k-mooney: mriedem: Artom made several changes to the revert resize patch yesterday but didn't respond to comments and also rebased it so there's a lot of change.. did I miss any discussion about that? | |
| 15:21:28 | dansmith | looks like he removed the try..except around the signature change which was my deal, but otherwise I either need to eye-diff it or pull it down | |
| 15:21:46 | sean-k-mooney | dansmith: he change it to the db lookup | |
| 15:21:53 | sean-k-mooney | right | |
| 15:22:02 | sean-k-mooney | i havent looked at it this mornig | |
| 15:22:09 | dansmith | alright | |
| 15:23:58 | mriedem | i was going to have to eye-diff anyway | |
| 15:34:07 | sean-k-mooney | artom is on pto while he is flying home to canada today but if there are small things like mising test exctra ill be around for an hour or two to fix them | |
| 15:34:38 | sean-k-mooney | i think artom is back on monday but if not ill continue looking at this until he is back. | |
| 15:55:03 | dansmith | sean-k-mooney: I don't see the "will wait for bind time events" string in the logs | |
| 15:55:21 | dansmith | on the hybrid plug test in experimental | |
| 15:55:51 | dansmith | maybe that test is single-host which means we're not running any of this new stuff anyway? | |
| 15:56:12 | sean-k-mooney | it is | |
| 15:56:22 | sean-k-mooney | it is testign the resize to same host case | |
| 15:56:42 | dansmith | ah yeah I see the same host debug message instead | |
| 15:56:44 | sean-k-mooney | https://review.opendev.org/#/c/664442/14 is testing resize to different host | |
| 15:57:28 | dansmith | ah, right forgot that was on top and was thinking we'd get testing from mriedem's patch, I see | |
| 15:58:24 | sean-k-mooney | ya this is the message form the dnm change http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/screen-n-cpu.txt.gz#_Jul_04_19_49_57_371761 | |
| 15:59:02 | dansmith | yeah, but lower we receive and discard that event: http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/screen-n-cpu.txt.gz#_Jul_04_19_49_58_003384 | |
| 15:59:22 | sean-k-mooney | ya i think this might be the souce node i need to check the other node | |
| 15:59:22 | dansmith | see the "no event matching" above that | |
| 15:59:29 | sean-k-mooney | and confim this is the right vm | |
| 15:59:41 | dansmith | we shouldn't emit the "will wait..." message on the wrong node right? | |
| 16:00:13 | sean-k-mooney | we should not e might get thos events on both nodes | |
| 16:00:23 | dansmith | receive, but not waiting | |
| 16:00:41 | dansmith | actually, that's the unplug event | |
| 16:00:48 | dansmith | we get the plug event shortly after that: | |
| 16:01:02 | dansmith | http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/screen-n-cpu.txt.gz#_Jul_04_19_50_00_983164 | |
| 16:02:55 | sean-k-mooney | i dont kno if we ever actully wait for unplugged events do we | |