Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-09
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: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026
16:13:32 openstackgerrit Dan Smith proposed openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025
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:33 openstackgerrit Dan Smith proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029
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:34 openstackgerrit Dan Smith proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031
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:35 openstackgerrit Dan Smith proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033
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:36 openstackgerrit Dan Smith proposed openstack/nova master: rp: Remove RP.get_traits() method https://review.openstack.org/509027
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

Earlier   Later