| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-09 | |||
| 16:13:37 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: break functions out of _set_traits() https://review.openstack.org/509908 | |
| 16:13:37 | openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036 | |
| 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 | 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:46:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1721652 in the functional test env https://review.openstack.org/510176 | |
| 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] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | |
| 16:49:22 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625 | |
| 16:50:16 | mriedem | gibi: ok, cool | |
| 16:54:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037 | |
| 16:54:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | |
| 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: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | |
| 18:25:04 | openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | |
| 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:25:05 | openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | |
| 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 | mriedem | oh right | |
| 18:50:16 | melwitt | mriedem: but you want to take a look at the py27 job, it passes with that hack | |
| 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 :) | |
| 18:55:35 | sdague | we've added ones in the past when we realized there was a missing place folks needed to do something | |
| 18:56:08 | sdague | I think I've got +2 on the ceph plugin repo too, so let me know when that's posted and I can help get it in | |
| 18:57:26 | melwitt | awesomesauce | |
| 19:04:34 | openstackgerrit | sean mooney proposed openstack/nova master: WIP reorder destination host setup https://review.openstack.org/510658 | |
| 19:11:01 | melwitt | sdague: posted here https://review.openstack.org/#/c/510660 will need to await test results to be 100% sure it's gonna owrk | |
| 19:11:03 | melwitt | *work | |
| 19:11:38 | sdague | melwitt: cool, I starred it, will come back around when the tests are in | |
| 19:11:50 | melwitt | thx | |
| 19:17:22 | mriedem | so uh https://review.openstack.org/#/c/394808/ | |
| 19:17:32 | mriedem | as far as i can tell, | |
| 19:17:44 | mriedem | that was added in ocata but the libvirt live migration flag it's setting is only available starting in libvirt 2.3.0 | |
| 19:17:52 | mriedem | http://libvirt.org/git/?p=libvirt.git;a=blob;f=docs/news-2016.html.in#l437 | |
| 19:18:03 | mriedem | our minimum is 1.2.9 | |
| 19:18:56 | sean-k-mooney | we can override this from the config no? | |
| 19:19:01 | mriedem | no | |
| 19:20:02 | sdague | mriedem: probably just put a conditional around that? | |
| 19:20:04 | sean-k-mooney | oh so the live_migration_flag option in the libvirt section of the nova-cpu.conf is jsut the default set ? | |
| 19:20:29 | mriedem | oh this might just be some other libvirt backend | |
| 19:21:15 | mriedem | yeah nvm, super old http://libvirt.org/git/?p=libvirt.git;a=commit;h=632be336898c9efef258e1f2d1c3df199a98a4aa | |
| 19:26:57 | sean-k-mooney | so live_migration_flag was apparently removed in kilo ... i should probably update our internal wiki to not tell you to use those when livemigrating | |
| 20:11:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove old code in post_live_migration_at_destination https://review.openstack.org/510680 | |
| 20:12:45 | openstackgerrit | Merged openstack/nova master: Remove dead code of api.fault notification sending https://review.openstack.org/505164 | |
| 20:12:58 | mriedem | sean-k-mooney: having to re-learn live migration to review this port binding spec again of course | |
| 20:45:18 | claudiub | mriedem: hellou. just an fyi, and I'm going to send a mail to the MLs and a patch for it, we're deprecating the support for Windows / Hyper-V Server 2012; the acceptable versions will be 2012 r2 and 2016. | |