Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-03
04:55:34 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
04:55:35 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
05:55:26 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
06:08:02 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
07:16:24 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP new migration threads control https://review.openstack.org/563505
07:43:45 sahid stephenfin: if you can have a look to the serie https://review.openstack.org/#/c/561912/
08:00:00 bauzas morning stackers
08:02:21 gibi good morning nova
08:08:52 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450
08:08:53 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514
08:08:54 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842
08:08:55 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045
08:08:56 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
08:08:57 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554
08:08:58 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487
08:23:49 gibi bauzas: could you look at a small spec template update that already has +2 from mriedem? https://review.openstack.org/#/c/562265/
08:24:11 bauzas sure
08:24:50 gibi bauzas: thanks
08:25:41 bauzas done
08:25:47 bauzas with a comment but +Wd
08:27:37 gibi bauzas: I can quickfix your nit then you can fastapprove it
08:28:23 bauzas gibi: nah, no worries
08:28:29 bauzas I understand what's an API entity
08:28:34 bauzas no need for a followup
08:28:41 bauzas and I'm not a API SIG expert ;)
08:28:54 bauzas but I use a vocabulary
08:29:05 bauzas REST verbs : GET, PUT, etc.
08:29:12 gibi bauzas: OK then let's merge it as is :)
08:29:15 bauzas REST resource : /servers, etc.
08:29:46 gibi bauzas: I think your vocabulary is the correct one
08:29:55 bauzas actions are not RESTful
08:30:12 bauzas but let's call it a subcollection
08:30:28 kashyap Those who can +2 backports, can you put this through please: https://review.openstack.org/#/c/565668/
08:30:33 kashyap (It already has a +2.)
08:31:11 bauzas kashyap: just did
08:31:25 kashyap bauzas: Fantastic. I've got two more, mind looking there too?
08:31:39 bauzas kashyap: see matt's comment
08:32:03 bauzas kashyap: https://review.openstack.org/#/c/565670/1
08:32:30 openstackgerrit Merged openstack/nova-specs master: Suggest when to propose notification changes https://review.openstack.org/562265
08:32:44 kashyap bauzas: Actually, that's what I did
08:32:48 kashyap But let me double-check
08:33:07 gibi bauzas: subcollection sound weird. as PUT and GET is called verb, the 'action' term for our unRESTful actions makes more sense to me
08:36:42 kashyap bauzas++
08:38:24 bauzas gibi: fair enoguh
08:38:34 bauzas gibi: let's call it an API action
08:38:47 gibi bauzas: action on an API resource :)
08:38:50 bauzas gibi: but not a REST action, which is an oxymore
08:39:18 bauzas anyway, we're furiously ratholing
08:39:51 gibi a good morning rathole to get us up to speed
09:23:31 openstackgerrit Merged openstack/nova-specs master: Document Rocky review priorities https://review.openstack.org/557796
09:29:50 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703
09:38:46 openstackgerrit jichenjc proposed openstack/nova master: [doc] enhance admin/configuration/api.rst https://review.openstack.org/566008
09:46:39 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add tests for alloc_cands with member_of https://review.openstack.org/561399
09:46:40 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix member_of with sharing providers https://review.openstack.org/561400
09:46:41 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Expand member_of functional test cases https://review.openstack.org/566011
09:48:05 openstackgerrit jichenjc proposed openstack/nova master: [doc]remove nova-cert leftover in doc https://review.openstack.org/566013
09:49:09 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Don't use single backticks in man pages https://review.openstack.org/540887
09:49:10 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Start using openstackdoctheme's extlink extension https://review.openstack.org/540888
09:49:11 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Fix file permissions https://review.openstack.org/566014
09:51:44 openstackgerrit Merged openstack/nova stable/queens: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565668
10:31:22 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
10:31:23 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
10:31:24 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
10:54:38 kashyap bauzas: and stephenfin Can you put this through: https://review.openstack.org/#/c/565707/
10:54:46 kashyap Quite mechanical change
10:54:57 kashyap Or gibi even :-)
10:55:31 stephenfin kashyap: done
10:55:45 kashyap Splendid
10:58:39 openstackgerrit Merged openstack/nova master: Cleanup patch for the cell-disable series https://review.openstack.org/559299
10:59:26 openstackgerrit Kashyap Chamarthy proposed openstack/nova stable/pike: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565670
10:59:57 gibi kashyap: done :)
11:05:49 kashyap Thanks!
11:07:42 kashyap gibi: I can't really figure why this is the only one that is failing: https://review.openstack.org/#/c/564012/
11:07:52 kashyap (Where you did the 'recheck')
11:07:59 moshele stephenfin: hi can you review this pike backport https://review.openstack.org/#/c/563501/?
11:08:20 gibi kashyap: based on the fail tempest test case names it feels unrelated to me
11:08:37 gibi kashyap: if the recheck fails with similar result then we have to dig
11:08:38 kashyap Indeed. Anyway, let's see if the new 'recheck' helps
11:09:10 kashyap Anyway, I have other irons in the fire...So onto them
11:21:15 openstackgerrit Merged openstack/nova master: Include only required fields in ironic node cache https://review.openstack.org/539508
11:46:33 jaypipes stephenfin: answered your query on the query on https://review.openstack.org/#/c/561315 :)
12:28:09 openstackgerrit Jim Rollenhagen proposed openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources https://review.openstack.org/565841
12:28:34 jroll dtantsur: fyi, rebased ^ on vdrok's stuff with a depends-on for the scheduler changes instead
12:31:36 dtantsur thanks!
12:32:17 stephenfin jaypipes: Makes sense. Mind addressing these two comments and I can ack https://review.openstack.org/#/c/561315/15/nova/api/openstack/placement/util.py@533 https://review.openstack.org/#/c/561315/15/nova/api/openstack/placement/handlers/resource_provider.py@200
12:33:05 stephenfin bauzas, gibi: Easy doc win here, if you fancy taking a look https://review.openstack.org/#/c/540888/
12:33:16 gibi stephenfin: looking
12:47:01 gibi stephenfin: you missed one link :) https://review.openstack.org/#/c/540888/3/api-guide/source/server_concepts.rst@895
12:47:11 gibi stephenfin: fix it quickly and I'm +2
13:02:47 openstackgerrit Jay Pipes proposed openstack/nova master: support multiple member_of qparams https://review.openstack.org/561315
13:02:58 jaypipes stephenfin: righty-o. all done ^
13:11:38 openstackgerrit Merged openstack/nova master: trivial: Fix file permissions https://review.openstack.org/566014
13:35:08 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Start using openstackdoctheme's extlink extension https://review.openstack.org/540888
13:35:24 stephenfin gibi: There were actually a few more. Think I've addressed everything that uses a versioned URL now
13:36:28 gibi stephenfin: thanks, looking
13:37:05 sahid stephenfin: can you look at this serie https://review.openstack.org/#/c/561912/9, jaypipes already +2ed most of the patches
13:37:14 stephenfin sahid: Can do
13:37:54 gibi stephenfin: is the new link at https://review.openstack.org/#/c/540888/3..4/doc/source/admin/manage-the-cloud.rst@25 more general than the original was?
13:38:42 stephenfin gibi: Possibly. I couldn't find an equivalent for the former
13:38:54 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: trivial: Fix file permissions https://review.openstack.org/566064

Earlier   Later