| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-01 | |||
| 20:37:29 | mriedem | is it? | |
| 20:39:01 | sdague | git grep inject_data | |
| 20:39:13 | mriedem | the xen ci failure was a stack failure | |
| 20:39:14 | sdague | I see libvirt and xen | |
| 20:39:22 | sdague | but xen is a different function | |
| 20:40:22 | sdague | yeh, xen half end runs around this with their own localfs call | |
| 20:40:29 | sdague | https://github.com/openstack/nova/blob/37cd9a961b065a07352b49ee72394cb210d8838b/nova/virt/xenapi/vm_utils.py#L2479 | |
| 20:41:34 | sdague | this would take it out of the libvirt path | |
| 20:41:45 | mriedem | ok, i assumed you'd be removing VFSLocalFS altogether | |
| 20:41:46 | sdague | the xen path has a fallback here when the xenagent isn't a thing | |
| 20:42:45 | sdague | well this patch was a DNM exploring a security bug | |
| 20:43:18 | sdague | a xen person would have to get on board about what the xen path looks like here, the libvirt one is pretty straight forward | |
| 20:47:25 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: remove *_topic config opts https://review.openstack.org/433924 | |
| 21:03:10 | cfriesen | bauzas: it looks like we set request_spec.ignore_hosts in compute.api.API.resize(), so we can't overwrite it later when we call reset_forced_destinations() from nova.conductor.tasks.migrate.MigrationTask.execute() | |
| 21:20:05 | dansmith | dtp: still around? | |
| 21:20:09 | dtp | yessir | |
| 21:20:30 | dansmith | dtp: so I think we're failing to get a list of tests that fail because we're just timing out the job on my patch | |
| 21:20:34 | dansmith | too many fails means too many delays | |
| 21:20:43 | dansmith | I ran the whole deal locally and got a good report | |
| 21:20:57 | dansmith | looking at it, the hypervisors list api is probably a self-contained thing you could work on if you want | |
| 21:21:04 | dansmith | tempest.api.compute.admin.test_hypervisor.HypervisorAdminTestJSON | |
| 21:21:22 | dansmith | if you run that module locally you'll get five tests that fail and then you can go from there | |
| 21:21:38 | dansmith | can probably start with just poking at hypervisors with the client and then run the test once you think it's working | |
| 21:21:41 | dansmith | interested? | |
| 21:22:43 | dtp | sure | |
| 21:22:50 | openstackgerrit | Kaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855 | |
| 21:23:21 | dansmith | cool | |
| 21:23:54 | dtp | i've not done any tempest stuff before. that's external to nova right? | |
| 21:24:23 | dansmith | yeah, you only need to run it though no mods required | |
| 21:24:34 | dansmith | and only after you poke with nova hypervisor-list and think you have it fixed | |
| 21:25:05 | dtp | oh - nova hypervisor-list is having the issue? | |
| 21:26:00 | dansmith | right, and thus tempest is breaking because it's checking that interface | |
| 21:26:23 | dansmith | it's actually not the client having the problem, nor tempest.. it's the server-side api code | |
| 21:26:32 | dtp | i see | |
| 21:26:52 | dansmith | so, nova/api/openstack/compute/hypervisors.py or something | |
| 21:27:01 | dansmith | that api now needs to iterate over cells to gather the list | |
| 21:27:17 | dtp | ah, ok | |
| 21:28:36 | dtp | and which patch of yours? | |
| 21:29:00 | dansmith | this set: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:multi-cell-testing | |
| 21:29:10 | dtp | thanks | |
| 21:29:14 | dansmith | plus this devstack patch which creates things in multiple cells: | |
| 21:29:22 | dansmith | https://review.openstack.org/#/c/436094/ | |
| 21:30:15 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: resolved final todos in libvirt conf https://review.openstack.org/429042 | |
| 21:33:06 | mriedem | sdague: left a comment about the reno in your localfs removal change | |
| 21:33:08 | mriedem | with thoughts anyway | |
| 21:33:23 | mriedem | i'm cool with leaving VFSLocalFS in for xenapi to sort out, BobBall should probably take a look | |
| 21:38:23 | dansmith | dtp: this is the tempest module: https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_hypervisor.py | |
| 21:38:26 | dtp | mriedem is there a deadline for pike blueprint submissions (no spec) | |
| 21:38:27 | dansmith | with multiple cells, it fails to list all the hypervisors | |
| 21:38:41 | dtp | ok, thanks dansmith | |
| 21:38:51 | mriedem | dtp: you probably want it done by the first milestone | |
| 21:39:00 | dtp | ok ty | |
| 21:39:02 | dansmith | dtp: fail looks like this: http://pastebin.com/wyh1jEYT | |
| 21:39:48 | dtp | cool, thanks dansmith | |
| 21:39:58 | mriedem | dtp: https://wiki.openstack.org/wiki/Nova/Pike_Release_Schedule#Dates_Overview | |
| 21:40:17 | mriedem | dtp: i was going to bring up spec freeze in the nova meeting tomorrow | |
| 21:40:21 | mriedem | since we didn't talk about that at the ptg | |
| 21:40:46 | dtp | i'll be there (it's the late one, right? ;) | |
| 21:42:00 | dansmith | yes | |
| 21:49:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle conflicts for os-assisted-volume-snapshots https://review.openstack.org/429476 | |
| 22:02:51 | dtp | what is searchlight? | |
| 22:05:42 | dansmith | dtp: http://lmgtfy.com/?q=openstack+searchlight | |
| 22:05:49 | dtp | heh | |
| 22:05:53 | dtp | thanks | |
| 22:06:30 | jroll | wow | |
| 22:06:43 | mriedem | has anyone noticed that pbr 2.0 has broken nova-specs? i don't think we use upper-constraints on nova-specs | |
| 22:07:58 | dansmith | dtp: with cells we'll have this problem of not being able to efficiently query all the cells, merge the list of instances and return them sorted, without doing slow things in python | |
| 22:08:37 | dansmith | dtp: since searchlight is aiming to be a superfast cache of other services with all those fancy gadgets, the plan is to just delegate to them when we have N>1 cells and lots of instances | |
| 22:10:36 | dtp | thanks | |
| 22:26:04 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Update requirements to fix pbr 2.0.0 fail https://review.openstack.org/439878 | |
| 22:30:02 | xinli | mnestratov: Hi | |
| 22:31:15 | openstackgerrit | Ed Leafe proposed openstack/nova master: placement: ironic inventory reporting https://review.openstack.org/437602 | |
| 22:31:19 | edleafe | jaypipes: ^^ fixed the pep8 issue | |
| 22:33:09 | openstackgerrit | Sujitha proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_create_backup.py https://review.openstack.org/435610 | |
| 22:33:23 | dims | adreznec : thorst : ping about pypowervm :) | |
| 22:33:47 | dims | adreznec : thorst : last thing we need for unblocking a bunch of people https://review.openstack.org/#/c/439790/ | |
| 22:34:00 | openstackgerrit | Sujitha proposed openstack/nova master: Remove mox from nova.tests.unit.api.openstack.compute.test_create_backup https://review.openstack.org/435610 | |
| 22:34:24 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Treat doc build warnings as errors https://review.openstack.org/439883 | |
| 22:34:41 | mriedem | dansmith: do it to it https://review.openstack.org/#/c/439878/ | |
| 22:35:45 | mriedem | dansmith: you should just fast approve | |
| 22:35:47 | dansmith | heh | |
| 22:35:47 | dansmith | mriedem: did you want the full monty? | |
| 22:35:48 | mriedem | specs are blocked | |
| 22:35:49 | mriedem | yeah | |
| 22:35:56 | mriedem | you can keep your pants on | |
| 22:35:59 | dansmith | hehe | |
| 22:36:00 | mriedem | or not, whatever | |
| 22:36:34 | dansmith | how about I do what I want and don't tell you? | |
| 22:37:13 | mriedem | that also works | |
| 22:39:44 | dims | LOL | |
| 22:42:25 | openstackgerrit | Sujitha proposed openstack/nova master: Remove mox from nova.tests.unit.api.openstack.compute.test_create_backup https://review.openstack.org/435610 | |
| 22:47:36 | jaypipes | edleafe: cheers, had a meeting... | |
| 22:49:30 | openstackgerrit | Pushkar Umaranikar proposed openstack/nova-specs master: Stop using Neutron admin token https://review.openstack.org/439890 | |
| 22:49:59 | adreznec | dims: tonyb: Updated pypowervm is up on pypi https://pypi.python.org/pypi/pypowervm/1.0.0.4.1 | |
| 22:50:08 | dims | thanks adreznec | |
| 22:50:09 | adreznec | Sorry for the delay | |
| 22:50:16 | tonyb | adreznec: #woot! Thanks | |
| 22:50:30 | tonyb | dims: I'll update "the" review | |
| 22:50:41 | dims | go for it tonyb | |
| 22:51:30 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Proposal to count resources to check quota in API for cells https://review.openstack.org/431749 | |