| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-09 | |||
| 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. | |
| 20:45:22 | mriedem | man we do not do a good job of documenting the valid values for filtering instances by status | |
| 20:45:31 | mriedem | claudiub: ok | |
| 20:55:05 | openstackgerrit | melanie witt proposed openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967 | |
| 20:55:06 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | |
| 20:55:06 | openstackgerrit | melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968 | |
| 20:55:07 | openstackgerrit | melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691 | |
| 20:57:25 | efried | mriedem sdague Sorry, I'm back now. Little lunchtime injury to deal with. Anything I need to know right now or will me email/dashboard tell all? | |
| 20:58:17 | melwitt | mriedem, dansmith: found a couple of cells bugs while working on CellDatabases. bottom two in the stack ^ | |
| 20:59:36 | openstackgerrit | Merged openstack/nova master: Always put 'uuid' into sort_keys for stable instance lists https://review.openstack.org/510140 | |
| 21:00:29 | mriedem | efried: the latter | |
| 21:00:41 | efried | mriedem Cool, thanks. | |
| 21:01:29 | efried | mriedem And to answer your question, the distance between those two patches was significant and we didn't want to collapse them together, though we could have. But you're absolutely correct that the second one just undoes what the first one adds wrt those auth options. | |
| 21:07:37 | mriedem | melwitt: don't tell me you can't get bandwidth usage with multiple cells?! | |
| 21:07:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix server status values in GET /servers docs https://review.openstack.org/510696 | |