Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-05
14:55:29 mriedem might just be worth removing that from master grenade and see how it goes
14:55:59 openstackgerrit Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
14:56:32 openstackgerrit Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950
14:56:33 openstackgerrit Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
14:56:33 openstackgerrit Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419
14:56:34 openstackgerrit Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
14:56:57 dansmith jaypipes: cdent: I didn't see that failure in ^ locally, so this is a rebase and we'll see how it goes this time
14:57:04 jaypipes kk
15:00:43 jaypipes dansmith, mriedem: I'm ready on this one after a number of back and forths with johnthetubaguy. https://review.openstack.org/#/c/507052/
15:01:01 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214
15:01:06 johnthetubaguy jaypipes: thanks for your time on that, glad to move that forward
15:02:51 dansmith edleafe: is this fail related to your change? http://logs.openstack.org/15/486215/14/check/gate-nova-tox-functional-py35-ubuntu-xenial/f7d2653/testr_results.html.gz
15:04:16 mriedem jaypipes: i'll put it in the tab qeuue
15:04:19 mriedem *queue
15:04:51 mriedem dansmith: i'll go through https://review.openstack.org/498950 again
15:05:11 dansmith mriedem: thanks, I don't think you had any feedback on that did you?
15:05:24 mriedem yes...
15:05:54 dansmith oh the fail cleanup, right
15:09:29 tikitavi look to the nova-conductor logs, please (http://logs.openstack.org/91/500091/8/check/gate-functional-neutron-dsvm-ec2api-ubuntu-xenial/206b664/). We have problem with starting instances in stable/ocata and stable/pike.
15:10:35 dansmith tikitavi: you mean the traceback?
15:10:44 tikitavi yes!
15:10:44 dansmith 2017-10-05 10:48:16.014 30844 ERROR oslo_messaging.rpc.server IOError: [Errno 2] No such file or directory: '/usr/local/lib/python2.7/dist-packages/six-1.10.0.dist-info/METADATA'
15:10:47 openstackgerrit Jay Pipes proposed openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025
15:10:48 openstackgerrit Jay Pipes proposed openstack/nova master: rp: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026
15:10:48 openstackgerrit Jay Pipes proposed openstack/nova master: rp: Remove RP.get_traits() method https://review.openstack.org/509027
15:10:49 openstackgerrit Jay Pipes proposed openstack/nova master: rp: move RP._set_traits() to module scope https://review.openstack.org/509028
15:10:49 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029
15:10:50 openstackgerrit Jay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030
15:10:50 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031
15:10:51 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032
15:10:51 openstackgerrit Jay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033
15:10:52 openstackgerrit Jay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035
15:10:52 dansmith tikitavi: seems clearly like some installation problem
15:10:52 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036
15:11:40 tikitavi dansmith: it is repeatable. we can't pass this review for more than a month
15:12:03 dansmith tikitavi: I understand, but still, it's a package problem, nothing to do with any nova ode
15:12:04 dansmith *code
15:13:00 edleafe dansmith: I ran those tests locally w/o a fail, so I don't think so.
15:13:18 dansmith edleafe: yea, I rechecked and noted that the next patch didn't hit the same failure
15:16:08 mriedem sdague: did your "wait for host mapping" change in devstack get backported to stable/pike? i'm seeing http://logs.openstack.org/56/509656/1/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/6851396/logs/old/screen-n-cond.txt.gz#_2017-10-04_23_25_13_456 in stable/pike grenade runs only
15:17:09 mriedem oh hm https://review.openstack.org/#/c/490635
15:17:38 sdague right, that will need to land there. I can push it in
15:18:16 mriedem +1 fwiw
15:18:17 mriedem :)
15:21:25 dansmith johnthetubaguy: a spec from you with no discernible typos.. jaypipes must have been up all night. :P
15:21:49 jaypipes lol
15:21:58 johnthetubaguy dansmith: him and efried got most of them the previous time :)
15:22:09 dansmith johnthetubaguy: heroes, those guys.
15:22:24 mriedem ooo -1
15:22:30 mriedem i'll fix after it's merged
15:25:12 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Rename ironic-driver-traits spec https://review.openstack.org/509812
15:25:53 openstackgerrit sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580
15:26:28 openstackgerrit Balazs Gibizer proposed openstack/nova master: fix unstable shelve offload functional tests https://review.openstack.org/509759
15:26:50 gibi mriedem: fixed your comment in the shelve offload test bugfix ^^
15:27:01 shvepsy Hi all! Can you help me with my issue.
15:27:02 shvepsy How I can call quiesce instance (freeze fs) from python API?
15:27:05 mdbooth dansmith: I vaguely recall some error from the dim and distants depths of time which meant that an error in a periodic task would prevent that task from continuing to run. Does that ring any bells with you?
15:27:37 mriedem johnthetubaguy: the thing on L37 of your spec confuses me
15:27:47 mriedem aren't you just describing the flavor-access API?
15:27:50 dansmith mdbooth: no, not that I know of
15:28:00 johnthetubaguy mriedem: I guess
15:28:21 johnthetubaguy mriedem: its just a public flavor and a private flavor, mapping to different groups of hosts
15:28:36 mriedem johnthetubaguy: so flavor-access with host aggregates?
15:30:01 dansmith mriedem: I'm not sure what your point is? that you can accomplish the same thing with aggregates and a private flavor?
15:30:17 mriedem yeah
15:30:29 dansmith but you can't right? because aggregates are around services not compute nodes
15:30:31 dansmith and
15:30:35 johnthetubaguy mriedem: well, ironic and host aggregates doesn't really do it
15:30:43 dansmith that's just one use of the general traits stuff being described here,
15:30:56 dansmith which also helps with external services using the same ironic
15:31:03 dansmith like some dedicated to nova and some for direct use
15:31:08 dansmith and nova aggregates wouldn't help with that
15:31:57 johnthetubaguy ++
15:32:58 mriedem ok i figured the host aggregates thing screwed it up
15:33:17 mriedem btw, at some point, can someone that understands the issues with host aggregates and ironic put some words about that in a section in https://docs.openstack.org/nova/pike/user/aggregates.html ?
15:33:24 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwitdh resource provider https://review.openstack.org/502306
15:33:24 mriedem because i think it's purely tribal knowledge
15:38:26 jaypipes ralonsoh: I'm going to have sean-k-mooney reach over and slap you if you don't fix that "bandwitdh" in your commit message :P
15:38:45 ralonsoh jaypipes: sorry!!!
15:38:48 ralonsoh one sec...
15:38:53 jaypipes ralonsoh: lol, it's ok :)
15:40:27 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306
15:40:33 jaypipes \o/
15:43:18 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/504895
15:43:40 mriedem easy functional test race fix if someone wants to +W https://review.openstack.org/#/c/509759/
15:44:05 dansmith oh this is the one I was just asking edleafe about
15:44:15 mriedem shelve tests?
15:44:21 efried mriedem Would a rename followup to https://review.openstack.org/#/c/507052/ resolve the mismatched bp/spec name issue, or does the old name stick around somehow?
15:44:30 mriedem efried: see the patch on top
15:44:45 efried mriedem Nyak.
15:51:25 openstackgerrit Balazs Gibizer proposed openstack/nova stable/pike: fix unstable shelve offload functional tests https://review.openstack.org/509818
15:51:51 dansmith cdent: fwiw, I agree with you on this: https://review.openstack.org/#/c/495854/4/nova/scheduler/utils.py
15:51:52 dansmith I would have made it a classmethod
15:51:55 dansmith not a big deal of course, but that's what I would have done
15:52:43 cdent <british>cheers
15:54:57 edleafe dansmith: heh, I had a 'from_host_state()' method on the orginal version. Should have left it there. :)
15:55:19 dansmith just my preference.. doesn't matter
15:56:04 dansmith edleafe: out of curiosity, why is this three patches instead of two with the object definition first?
15:56:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove metadata/system_metadata filter handling from get_all https://review.openstack.org/509819
15:56:28 dansmith looking now, it seems like if you did the object patch first, you could have squashed the first and third together, which would be less back and forth

Earlier   Later