| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-02 | |||
| 19:30:58 | cfriesen | debug might be good | |
| 19:31:21 | sean-k-mooney | there is nothing that an operator can do to scilence it if they have non nic VF currently and its not helpful in that case | |
| 19:31:33 | cfriesen | agreed | |
| 19:32:20 | sean-k-mooney | anyway the fact that your nova-compute agent didnt die means the fix is at least minimally working. | |
| 19:35:05 | sean-k-mooney | i might look at optimising this code a bit too. currently we call _get_pcidev_info from _get_pci_passthrough_devices before applying the pci whitelist so we look at way more device then we need too | |
| 19:50:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix comment in test_attach_with_multiattach_fails_not_available https://review.openstack.org/649440 | |
| 19:52:35 | openstackgerrit | Merged openstack/nova master: Fix a deprecation warning https://review.openstack.org/649234 | |
| 19:54:45 | efried | Seems to hit several different jobs, but always grenade | |
| 19:54:45 | efried | mriedem: How about this one: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22ModuleNotFoundError%3A%20No%20module%20named%20'memcache'%5C%22 | |
| 19:56:45 | efried | mriedem: https://review.openstack.org/#/c/649096/ ? | |
| 19:57:54 | mriedem | efried: known issue | |
| 19:58:02 | mriedem | yeah related to that bug | |
| 19:58:09 | mriedem | i need to update the e-r query, or you can | |
| 19:58:25 | mriedem | to include build_name:"neutron-grenade-multinode" OR build_name:"neutron-grenade-dvr-multinode" | |
| 19:58:34 | efried | mriedem: there's also grenade-py3 | |
| 19:58:46 | efried | let me try | |
| 19:58:55 | mriedem | the query is already restricted to just grenade-py3 | |
| 19:59:16 | mriedem | so need message:"..." AND tags:"screen" AND (build_name:"grenade-py3" OR build_name:"..."...) | |
| 19:59:19 | efried | k, hadn't pulled it up yet. | |
| 19:59:29 | openstack | Launchpad bug 1822873 in devstack "stack fails if NOVA_NUM_CELLS > 1 and n-novnc enabled" [Undecided,New] | |
| 19:59:29 | mriedem | melwitt: dansmith: duh duh duh https://bugs.launchpad.net/devstack/+bug/1822873 | |
| 19:59:32 | efried | the bug says the module not found is etcd | |
| 19:59:49 | mriedem | efried: it's a whole bunch of packages, | |
| 19:59:57 | mriedem | i spent half of yesterday digging into that | |
| 20:00:01 | dansmith | mriedem: duh duh duhntcare | |
| 20:00:02 | efried | okay, I see the message is just ModuleNotFound, cool. | |
| 20:00:05 | mriedem | but it's also just busted networks getting to pypi | |
| 20:00:36 | mriedem | efried: just need to update the existing query for http://status.openstack.org/elastic-recheck/#1820892 | |
| 20:00:42 | efried | mriedem: on it | |
| 20:02:08 | mnaser | mriedem: if you launch it in sjc1, you'd be launching it against stein :) | |
| 20:06:25 | efried | mriedem: Okay, so would there have been a better way for me to determine that this was 1820892 ? | |
| 20:17:04 | melwitt | mriedem: dammit novnc | |
| 20:17:37 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Docs: emulator threads: clarify expected behavior https://review.openstack.org/649416 | |
| 20:29:03 | eandersson | Does anyone recall the reason why the functionality to search based on metadata was removed from the api? | |
| 20:29:06 | eandersson | > search_options["metadata"] = '{"my_key" : "bla" }' | |
| 20:29:12 | eandersson | https://review.openstack.org/#/c/408571/ | |
| 20:29:18 | eandersson | Also, is the alternative to use tags? | |
| 20:43:31 | efried | eandersson: I have no idea, but did you look at the spec? http://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/add-whitelist-for-server-list-filter-sort-parameters.html | |
| 20:44:32 | efried | That appears to answer the first question. alex_xu could probably answer the second. | |
| 20:45:12 | mriedem | mnaser: i only see ca-ymq-2 | |
| 20:45:48 | mnaser | huh? sjc1 is a separate region | |
| 20:46:05 | mnaser | OS_REGION_NAME=sjc1 | |
| 20:46:51 | eandersson | I can't find any good documentation for tags, but maybe I just suck at googling :D | |
| 20:48:36 | eandersson | nova cli has things like > nova server-tag-add | |
| 20:48:57 | eandersson | but openstackcli has no mention of tags | |
| 20:49:00 | eandersson | > https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/server.html#server-create | |
| 20:49:10 | mriedem | mnaser: oh i thouht it was a zone | |
| 20:49:27 | mriedem | eandersson: openstack server set i think | |
| 20:49:35 | mriedem | https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/server.html#server-set | |
| 20:49:44 | mriedem | oh nvm | |
| 20:49:49 | mriedem | no tags support there either yet | |
| 20:49:55 | mriedem | consulting https://etherpad.openstack.org/p/compute-api-microversion-gap-in-osc | |
| 20:50:22 | mriedem | https://review.openstack.org/#/c/569386/ | |
| 20:50:51 | eandersson | That explains it | |
| 20:51:51 | mriedem | eandersson: if you're going to be in denver https://www.openstack.org/summit/denver-2019/summit-schedule/events/23665/closing-compute-api-feature-gaps-in-the-openstack-cli | |
| 20:52:02 | eandersson | I will | |
| 20:52:19 | mriedem | i've started the etherpad for that session https://etherpad.openstack.org/p/DEN-osc-compute-api-gaps | |
| 21:09:50 | NewBruce | hej mriedem - we look to have stumbled on a bug in Neutrons Port Binding API, in a particular area of the code which you will probably be more intimately familliar with than I | |
| 21:10:03 | NewBruce | https://bugs.launchpad.net/nova/+bug/1822884 (cc sean-k-mooney) | |
| 21:10:04 | openstack | Launchpad bug 1822884 in OpenStack Compute (nova) "live migration fails due to port binding duplicate key entry in post_live_migrate" [Undecided,New] | |
| 21:12:04 | openstackgerrit | melanie witt proposed openstack/nova stable/stein: Add doc on VGPU allocs and inventories for nrp https://review.openstack.org/649454 | |
| 21:12:51 | NewBruce | if you have a minute, id love your opinion - im trying to decide how much more debugging is worth before we move over to cold migrations | |
| 21:16:40 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Docs: emulator threads: clarify expected behavior https://review.openstack.org/649416 | |
| 21:17:51 | mriedem | NewBruce: i don't know what the differences are between RDO and OSA which would cause issues here | |
| 21:18:18 | mriedem | as you said, "This is unexpected, as even in the RDO to RDO case, both nodes are Rocky and so the new process should be in use." | |
| 21:18:31 | mriedem | are you sure you're not hitting a case where one node is really queens? | |
| 21:18:31 | NewBruce | hrmm, no - that seems to be our sticking point - i was chatting with sean-k-mooney and also mnaser on this | |
| 21:19:17 | NewBruce | yeah, absolutely sure - as I am using a single test source node which is absolutely Rocky as well as the target | |
| 21:21:17 | mriedem | is 'Port Bindings Extended' showing up in the neutron api extension list in both the rdo and osa case? | |
| 21:22:07 | mriedem | if rdo is rocky, i don't know why rdo->rdo would use the old flow | |
| 21:22:16 | mriedem | unless the 'Port Bindings Extended' neutron extension is not showing up | |
| 21:23:48 | NewBruce | i can confirm that on the target it is enabled | |
| 21:23:52 | mriedem | the live migration task should only do the new flow with the 2nd deactivated port binding if (1) the neutron 'Port Bindings Extended' API extension is available, and (2) both the source and dest compute service versions are >= 37 | |
| 21:23:58 | mriedem | *35 | |
| 21:24:27 | eandersson | mriedem, solution was to just list all VMs in the project and then just filter on metadata :p | |
| 21:24:42 | eandersson | until tags is supported everywhere (openstackclient, terraform etc) | |
| 21:25:30 | NewBruce | yeah, so we looked at that as well; | |
| 21:25:31 | NewBruce | source = 10 (RDO Rocky) Target = 29 (OSA Rocky) | |
| 21:25:31 | NewBruce | | cc-compute29-kna1 | nova-compute | 35 | | |
| 21:25:31 | NewBruce | | cc-compute10-kna1 | nova-compute | 35 | | |
| 21:26:05 | NewBruce | however we do have other compute nodes in the same environment which have not been upgraded to Rocky yet; (service version 30) | |
| 21:27:38 | mriedem | NewBruce: do you have [upgrade_levels]/compute pinned to queens or something? | |
| 21:28:09 | openstackgerrit | Jared Winborne proposed openstack/nova master: Leave the brackets on Ceph Monitor IPv6 addresses for libguestfs https://review.openstack.org/649405 | |
| 21:29:21 | NewBruce | compute = auto | |
| 21:29:21 | NewBruce | [upgrade_levels] | |
| 21:29:43 | mriedem | so that will pin to the lowest nova-compute service version in the deployment while you're upgrading, | |
| 21:29:56 | mriedem | so if you have queens computes, it will be using the queens rpc versions and backlevel the migrate_data object | |
| 21:30:11 | mriedem | which likely is dropping the vifs information for the 2nd deactivated port binding | |
| 21:30:27 | mriedem | that's my guess anyway | |
| 21:30:27 | NewBruce | Aha…. | |
| 21:30:36 | NewBruce | ok, so can we override that? | |
| 21:31:43 | NewBruce | again, mnaser / sean-k-mooney suggested that might be the case and to upgrade the entire compute - which we were in the process of, however anothe error in a node caused a bit of strife due to broken libevent so’s in OVS so we halted | |
| 21:32:06 | mriedem | well you can pin the compute rpc api version to a specific release (queens) or even rpc api version (5.0) but that could break things where the controller is sending versions of objects to older queens computes that won't understand those chnages | |
| 21:32:11 | NewBruce | also, we have other sites where we have performed the same procedure, without issue | |
| 21:32:37 | mriedem | ultimately it looks like https://github.com/openstack/nova/blob/stable/rocky/nova/conductor/tasks/live_migrate.py#L41 is faulty in that it doesn't account for pinned RPC versions | |
| 21:32:39 | NewBruce | i have a test machine, so i can override to at least test it | |
| 21:32:40 | mriedem | dansmith: yeah ^? | |
| 21:33:03 | mriedem | NewBruce: the kill switch would be to disable the 'Port Bindings Extended' neutron API extension until you're fully upgraded to rocky | |
| 21:33:08 | dansmith | mriedem: if conductor is new enough then the new objects are fine | |
| 21:33:23 | NewBruce | which would force into the old flow | |