Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-16
05:46:25 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: update the document related to vdi streaming https://review.openstack.org/568444
05:55:31 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
06:13:39 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041
06:23:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (2) https://review.openstack.org/568757
06:56:02 openstackgerrit Merged openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925
07:39:07 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802
08:01:02 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833
08:01:29 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: update the document related to vdi streaming https://review.openstack.org/568444
08:03:46 openstackgerrit Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472
08:39:39 openstackgerrit Vladyslav Drok proposed openstack/nova-specs master: Allow having placement inventories with reserved value equal to total https://review.openstack.org/568613
08:45:39 openstackgerrit Balazs Gibizer proposed openstack/nova stable/queens: placement: Fix HTTP error generation https://review.openstack.org/568784
08:51:19 openstackgerrit Surya Seetharaman proposed openstack/nova stable/queens: POC: Filter/ignore NotSupportedWarning ['use_tpool'] in nova-manage https://review.openstack.org/568785
09:56:39 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: WIP: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534
10:00:56 ispp Hello, Im debugging a warning Im seeing in Neutron , it seems the warning comes from this line https://gitlab.cern.ch/cloud-infrastructure/nova/blob/cern-queens/nova/network/neutronv2/api.py#L2119
10:00:56 ispp Requesting floatingips to Neutron, in neutron I see " No controller found for: floatingips - returning response code 404: PecanNotFound" do you know if is there any bug open or is just an isolated issue? Anyway I don't know if there is a fix directly in neutron to avoid to touch nova
10:01:17 ispp Sorry wrong link
10:02:46 ispp This is the link https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L2107
12:14:01 openstackgerrit Eric Fried proposed openstack/nova master: Granular requests to get_allocation_candidates https://review.openstack.org/515811
12:16:51 mriedem mdbooth: any chance https://blueprints.launchpad.net/nova/+spec/local-disk-serial-numbers will be updated today? if not, i'll drop it from the runway
12:19:00 mriedem i've moved the zvm driver series out of the runway and moved https://blueprints.launchpad.net/nova/+spec/granular-placement-policy into a slot
12:21:05 mriedem alex_xu: can you proxy for lei zhang and say that https://review.openstack.org/#/c/560317/ is ready to go into a runway slot?
12:21:16 mdbooth mriedem: Definitely not :( Meetings.
12:21:40 mriedem mdbooth: ok, dropped it from the runway slot; time to have a meeting about how you're in too many meetings.
12:22:18 mriedem alex_xu: a bit worried that https://review.openstack.org/#/c/560317/ isn't getting attention from the author since the last -1 was from may 7, over a week and a half ago
12:23:58 mriedem naichuans: is https://blueprints.launchpad.net/nova/+spec/vgpu-rocky ready to go into a runway slot for the xenapi changes?
12:24:06 mriedem starting at https://review.openstack.org/#/c/520313/
12:25:54 alex_xu mriedem: yea, I will reach him soon
12:26:02 alex_xu very soon by wechat :)
12:26:24 alex_xu mriedem: I think his patch has a dependence
12:26:27 mriedem alex_xu: ok i guess let me know if i should put that into a runway slot today, otherwise i'll put the xenapi vgpu n-r-p series in there
12:27:00 mriedem alex_xu: efried commented that the code should be using update_provider_tree https://review.openstack.org/#/c/560317/2/nova/virt/driver.py@1684
12:27:06 mriedem but i thought that was ready
12:28:35 efried mriedem: Sorry, refreshing my memory.
12:28:54 alex_xu mriedem:, yes, one of patch for the update_provider_tree, but trying to figure out which one is the dependence
12:28:57 efried mriedem: IIRC, the spec doesn't say anything about a new driver method, so it's still reasonably okay without a delta. Let me confirm that.
12:29:35 mriedem https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/report-cpu-features-as-traits.html#work-items
12:29:41 mriedem "Implement an interface which returns a list of traits for the CPU features in each virt driver."
12:29:45 mriedem that should be update_provider_tree
12:30:01 mriedem which is what the 2nd work item says :)
12:30:49 efried mriedem: Yeah, and L114.
12:30:58 efried mriedem: So yeah, my -1 on the patch stands firm
12:32:21 alex_xu efried: it should be calle dby the update_provider_tree, is there anyone work on the update_provider_tree for libvirt driver?
12:33:09 efried alex_xu: I started this series, which bhagyashris is working on finishing up: https://review.openstack.org/#/c/560444/
12:33:36 efried alex_xu: bhagyashris is having trouble with the functional testing. I've got it on my backlog to help out there, but haven't gotten to it yet.
12:34:06 mriedem ok so i'll skip the cpu features bp this time for a runway slot since it's not ready
12:34:21 alex_xu efried: got it, thanks
12:34:32 alex_xu mriedem: yea, thanks, let us skip this time
12:35:30 mriedem naichuans: i've put https://blueprints.launchpad.net/nova/+spec/vgpu-rocky into a runway slot for the xenapi driver changes
12:35:34 mriedem https://etherpad.openstack.org/p/nova-runways-rocky
12:35:46 efried alex_xu: If you wanted to help with that, it would be welcomed. Here's the test case bhagyashris has so far: http://paste.openstack.org/raw/720752/ (test_servers.py)
12:35:46 mriedem dansmith: we have a few new runway entries so should probably update the channel topic
12:37:46 bhagyashris efried: Hi, i will push the patch soon
12:38:04 efried bhagyashris: Okay. Sorry I haven't gotten to that yet. Have you made progress?
12:46:28 alex_xu bhagyashris: let me know if you still have question
12:50:06 bhagyashris efried: no issue yes
12:50:18 efried great
12:50:18 bhagyashris alex_xu: thank you :)
12:54:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425
12:54:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578
12:54:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for resource providers inventories https://review.openstack.org/568666
12:54:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for usages https://review.openstack.org/568706
13:01:29 openstackgerrit Eric Fried proposed openstack/nova master: Use placement.inventory.inuse in report client https://review.openstack.org/568639
13:05:29 openstackgerrit Eric Fried proposed openstack/nova master: Use GET.get instead of GET.getall in alloc-cands https://review.openstack.org/568713
13:06:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
13:06:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
13:06:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make placement.lib.RequestGroup a versioned object https://review.openstack.org/568840
13:18:49 openstackgerrit Vladyslav Drok proposed openstack/nova-specs master: Allow having placement inventories with reserved value equal to total https://review.openstack.org/568613
13:23:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Deduplicate config/policy reference docs from main index https://review.openstack.org/564846
13:23:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425
13:23:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578
13:23:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for resource providers inventories https://review.openstack.org/568666
13:23:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Add granular policy rules for usages https://review.openstack.org/568706
13:24:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup ugly stub in TestLocalDeleteAllocations https://review.openstack.org/561997
13:27:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make placement.lib.RequestGroup a versioned object https://review.openstack.org/568840
13:27:31 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
13:27:32 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
13:44:42 dansmith jaypipes: on this: https://review.openstack.org/#/c/568613/5
13:45:00 jaypipes dansmith: yes?
13:45:01 dansmith jaypipes: isn't not allowing reserved=total just a bug? I feel like maybe we don't need the whole microversion dance
13:45:28 dansmith like someone did < instead of <= in some code somewhere?
13:45:30 jaypipes dansmith: I feel the same way.
13:45:50 dansmith jaypipes: does anyone not feel that way?
13:45:53 mriedem lyarwood: you +2ed the ocata backport but not the pike one :) https://review.openstack.org/#/c/568369/
13:46:15 dansmith my feeling is that there is no client that would depend on total=reserved returning a 400
13:46:17 mriedem dansmith: i brought it up when it was being discussed,
13:46:23 mriedem because no one knew why the capacity check was that way
13:47:21 mriedem and its a behavior change in the api, so you kind of need to know when you can make that kind of request and expect it to be ok
13:47:22 lyarwood mriedem: appears I closed the tab before gerrit actually updated the review, noice!
13:48:01 dansmith mriedem: but microversions aren't for providing bug-for-bug compatibility right? that's why we don't do one to fix a 500?
13:48:22 mriedem this isn't a 500
13:48:31 dansmith I know
13:48:51 dansmith but really, if we do it and get a 400, we can fall back to the same behavior as today
13:48:56 dansmith I dunno, this just seems overkill to me
13:49:13 mriedem i don't think i ever said i was going to block this when we talked about it in -placement weeks ago,
13:49:21 mriedem i -2ed the change this week because there is a spec up,
13:49:29 mriedem if people don't care about the api behavior change, then whatever
13:50:22 dansmith I would think that if this is a bug, with no microversion, that we'd backport the fix as .. a fix
13:50:54 mriedem meh - this doesn't even fix any real problem right now does it?
13:51:09 mriedem the actual bug was already fixed some other way is my understanding
13:51:09 dansmith just MHO, I'm not strongly against a microversion I guess, I just .. I dunno.

Earlier   Later