Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-05
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
16:39:29 openstackgerrit Chris Friesen proposed openstack/nova master: Add validation for cpu_realtime_mask https://review.openstack.org/468203
16:44:57 openstackgerrit Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950
16:44:58 openstackgerrit Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
16:44:58 openstackgerrit Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419
16:44:59 openstackgerrit Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
16:44:59 openstackgerrit Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638
16:50:33 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: fix unstable shelve offload functional tests https://review.openstack.org/509818
16:54:46 openstackgerrit Merged openstack/nova master: Add attachment_get call to volume/cinder_api https://review.openstack.org/506805
17:01:21 mriedem melwitt: you should probably look at this one https://review.openstack.org/#/c/506822/
17:01:28 mriedem i think it's ok, but don't want to mess it up

Earlier   Later