| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-09 | |||
| 16:03:44 | johnthetubaguy | sean-k-mooney: I guess you want to send the updated port bindings to os-vif on the destination node, before handing control back to the source node? | |
| 16:05:46 | sean-k-mooney | johnthetubaguy: am well i want to create the binding for the destionation before i create destination xml so that if the interfaces defintion change i can update that section of xml. e.g. linux bridge to ovs or iptables to contrack firewall | |
| 16:05:53 | jaypipes | dansmith: I don't get it... I cherry-pick this patch locally: https://review.openstack.org/#/c/509030/ onto master (trying to re-order the de-orm series) and the cherry-pick applies code that is from the patch underneath it (https://review.openstack.org/#/c/509029/) :( | |
| 16:05:53 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512 | |
| 16:06:30 | johnthetubaguy | sean-k-mooney: agreed | |
| 16:06:47 | dansmith | jaypipes: if you want to pastebin a list in the right order, I can try to do it | |
| 16:07:06 | jaypipes | dansmith: I thought cherry-pick was supposed to only apply the diff of the specific patch and nothing else? | |
| 16:07:08 | sean-k-mooney | johnthetubaguy: in the current flow i generate teh xml and then bind which mean today when i live migrate form a lb host to ovs it work but livirt still plugs the tap into linux brige and neutron wait for the port to show up on ovs | |
| 16:07:30 | dansmith | jaypipes: not really, but it's only supposed to do the one you ask for, which is what you're saying yeah | |
| 16:08:51 | sean-k-mooney | johnthetubaguy: works was ment to be in qoutes because it live migrate but you networking is screwed untill you hard reboot the vm | |
| 16:09:04 | jaypipes | dansmith: well, this is what I'm *trying* to do... http://paste.openstack.org/show/623122/ | |
| 16:09:44 | dansmith | jaypipes: k, lemme try | |
| 16:09:58 | jaypipes | dansmith: and what I tried was git checkout master && git pull && checkout -b de-orm-rp && git cherry-pick 0c1a5bc | |
| 16:10:16 | jaypipes | dansmith: but that applied changes from ca983c1 for some reason | |
| 16:13:32 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025 | |
| 16:13:32 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026 | |
| 16:13:33 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029 | |
| 16:13:33 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030 | |
| 16:13:34 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031 | |
| 16:13:34 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032 | |
| 16:13:35 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033 | |
| 16:13:35 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035 | |
| 16:13:36 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: Remove RP.get_traits() method https://review.openstack.org/509027 | |
| 16:13:36 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: move RP._set_traits() to module scope https://review.openstack.org/509028 | |
| 16:13:37 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036 | |
| 16:13:37 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: break functions out of _set_traits() https://review.openstack.org/509908 | |
| 16:13:39 | dansmith | jaypipes: that look right? ^ | |
| 16:14:03 | dansmith | had to re-order one thing from your list, but otherwise that ordering was fine | |
| 16:15:04 | jaypipes | dansmith: yep, that'll work. thanks man, really appreciated. | |
| 16:15:09 | dansmith | np | |
| 16:15:19 | jaypipes | dansmith: I see you just slid the CRUD one before the Streamline one. k. | |
| 16:15:30 | dansmith | had to else huge esplosion | |
| 16:15:46 | jaypipes | yes, that's the espolosion I was dealing with.. | |
| 16:46:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1721652 in the functional test env https://review.openstack.org/510176 | |
| 16:46:32 | openstack | bug 1721652 in OpenStack Compute (nova) pike "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [High,Confirmed] https://launchpad.net/bugs/1721652 | |
| 16:47:39 | gibi | mriedem: I managed to push a functional regression for bug 1721652 https://review.openstack.org/510176 | |
| 16:47:40 | openstack | bug 1721652 in OpenStack Compute (nova) pike "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [High,Confirmed] https://launchpad.net/bugs/1721652 | |
| 16:49:22 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625 | |
| 16:49:22 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | |
| 16:50:16 | mriedem | gibi: ok, cool | |
| 16:54:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | |
| 16:54:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037 | |
| 16:54:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | |
| 17:22:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove unnecessary BDM destroy during instance delete https://review.openstack.org/510635 | |
| 17:27:06 | openstackgerrit | Merged openstack/nova master: Remove useless periodic task that expires quota reservations https://review.openstack.org/506822 | |
| 17:30:03 | openstackgerrit | Merged openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | |
| 17:30:40 | openstackgerrit | Merged openstack/nova master: Fix typo in test_prep_resize_errors_migration https://review.openstack.org/510591 | |
| 17:38:48 | jaypipes | can we please boot the EMC ScaleIO CI system? :( | |
| 17:39:09 | melwitt | mriedem: ack | |
| 17:42:17 | openstackgerrit | Merged openstack/nova master: Make setenv consistent for unit, func, and api-samples https://review.openstack.org/507976 | |
| 17:50:01 | mriedem | ericyoung: can the emc scaleio ci just be run on the one patch that adds the imagebackend? | |
| 17:52:25 | Tengu | hello! | |
| 17:52:36 | Tengu | anyone can tell me a bit more about this log message? 2017-10-09 19:49:34.201 2665 DEBUG nova.compute.utils [req-08080314-ca07-499b-b559-a30632b6a773 f3cef0caace94cf0af3d7c21b3706583 553ad7544f5a4479b4eb346dc7a76a82 - default default] [instance: 9f503af3-ab45-43e6-be45-b0dbcff60483] Cannot attach VIF 4529caaf-3b69-4314-899c-a2090fe6114b to the node 4aa07b6d-ccf0-4f2b-8938-e20b07ff0156 due to error: | |
| 17:52:42 | Tengu | Unable to attach VIF 4529caaf-3b69-4314-899c-a2090fe6114b, not enough free physical ports. (HTTP 400) notify_about_instance_usage /usr/lib/python2.7/site-packages/nova/compute/utils.py:328 | |
| 17:53:36 | Tengu | I'm trying to deploy an overcloud (tripleO), and I have some issues with the Ceph nodes, the deploy script being unable to find the two nodes. I'm wondering if this log entry can explain why, and what it means. | |
| 18:11:46 | mriedem1 | efried: happy monday | |
| 18:17:57 | mriedem | sdague: you were +2 on this before, assume you still like it https://review.openstack.org/#/c/505317/6 | |
| 18:20:00 | sdague | mriedem / efried - why did that shuffle get added? | |
| 18:20:02 | mriedem | efried: looks like https://review.openstack.org/#/c/490057/ needs to be rebased and there are quite a few test failures | |
| 18:20:46 | mriedem | sdague: it's not added | |
| 18:20:47 | sdague | oh, never mind, it was there before the patch is just highlighting weirdly | |
| 18:20:51 | mriedem | yeah | |
| 18:25:04 | openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | |
| 18:25:04 | openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | |
| 18:25:05 | openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | |
| 18:25:05 | openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | |
| 18:31:40 | jaypipes | dansmith: I'll definitely fix up that unit test miss thing. just in the middle of rebasing the n-r-p series now for the ID vs. uuid stuff. | |
| 18:32:41 | dansmith | jaypipes: ack | |
| 18:46:47 | melwitt | sdague, mriedem: my devstack hack patch for the ceph job https://review.openstack.org/#/c/509663/ | |
| 18:47:45 | sdague | melwitt: we typically don't want any of the plugin conditional code to leak into there | |
| 18:48:01 | sdague | melwitt: is there an appropriate plugin phase for the ceph plugin to do it? | |
| 18:48:05 | mriedem | melwitt: so this is progress right? http://logs.openstack.org/63/509663/4/experimental/gate-tempest-dsvm-py35-full-devstack-plugin-ceph-ubuntu-xenial-nv/2b03b3a/logs/devstacklog.txt.gz#_2017-10-06_05_26_37_113 | |
| 18:48:11 | mriedem | sdague: apparently not | |
| 18:48:17 | mriedem | keystone is special as it's loaded early in stack.sh | |
| 18:48:35 | sdague | where is the ceph plugin source? | |
| 18:48:37 | melwitt | sdague: there used to be post-config. but now keystone is started before post-config and keystone start is the thing that's failing in the ceph job. so I need a new phase between pip install and keystone start | |
| 18:48:48 | mriedem | sdague: https://github.com/openstack/devstack-plugin-ceph | |
| 18:50:05 | melwitt | mriedem: that's the py35 job which won't work until the ceph plugin is installing python3-rados and python3-rbd. I have a patch up for that too | |
| 18:50:16 | melwitt | mriedem: but you want to take a look at the py27 job, it passes with that hack | |
| 18:50:16 | mriedem | oh right | |
| 18:50:31 | melwitt | I got confused by the same thing last week | |
| 18:50:45 | mriedem | i know i reported a bug for ceph libradoes in glance-store in lp somewhere | |
| 18:50:50 | mriedem | for py3 | |
| 18:51:16 | mriedem | ah here it is passing http://logs.openstack.org/63/509663/4/experimental/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/af8e25f/ | |
| 18:51:18 | melwitt | ah yup, it's that. glance-store weirdly sort of silently ignores import errors even in the rbd file | |
| 18:51:24 | sdague | melwitt: why can't it run at stack_install ? | |
| 18:51:26 | mriedem | melwitt: yup | |
| 18:51:51 | melwitt | sdague: is stack_install after pip install and before keystone start? if it is, then it can run there | |
| 18:52:33 | sdague | https://github.com/openstack-dev/devstack/blob/a29329b55983d69c3966e1699f985be8cdf6e59d/stack.sh#L914 | |
| 18:52:49 | mriedem | ah yup | |
| 18:52:53 | sdague | it's definitely before keystone start | |
| 18:53:11 | mriedem | so need something like this https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/plugin.sh#L6 | |
| 18:53:17 | mriedem | elif [[ "$1" == "stack" && "$2" == "install" ]]; then | |
| 18:53:23 | mriedem | good call | |
| 18:53:32 | melwitt | omg, thanks sdague | |
| 18:53:47 | melwitt | sweet | |
| 18:53:56 | sdague | melwitt: no prob | |
| 18:54:07 | melwitt | I need better devstack-fu | |
| 18:54:32 | sdague | no worries, it's a lot of code | |
| 18:55:09 | sdague | if no appropriate phase existed, I would have created one for it, because the whole plugin abstraction completely breaks if you have plugin names in the main code :) | |