Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-24
00:55:15 takashin
00:55:25 Spazmotic
00:56:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
00:56:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
00:57:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_provider) https://review.openstack.org/528629
00:58:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix the order of target host checks https://review.openstack.org/526225
00:58:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
01:01:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
01:14:11 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/537093
01:23:43 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994
01:23:52 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
01:31:09 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/537171
01:56:20 jackie-truong If anyone has a free moment to review this patch, it would be much appreciated: https://review.openstack.org/#/c/486204/
02:00:57 Spazmotic Thanks jianghuaw, I will spin up that bug report tomorrow night when I am at my dev machine and can pull some XenAPI logs for it :)
02:01:40 jianghuaw Spazmotic, good:-)
02:01:58 Spazmotic Also should go ahead and rebase and fix that inline while i'm at it tomorrow.
02:14:59 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994
02:15:08 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
02:24:28 alex_xu mriedem: dansmith good news is we have allocation_req_version in Selection obj, then I can convert the alloc_req dict to consistent format in the begining of claim method probably
02:25:31 alex_xu mriedem: dansmith here is https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L629
02:31:34 openstackgerrit sean mooney proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257
02:32:59 sean-k-mooney stephenfin: https://review.openstack.org/#/c/449257/ now uses objects. ill be in around lunch tomorow if you have any questions.
03:10:40 openstackgerrit Dao Cong Tien proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
04:06:19 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle TZ change in iso8601 >=0.1.12 https://review.openstack.org/535700
04:25:30 openstackgerrit sean mooney proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257
04:25:31 openstackgerrit sean mooney proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
04:25:31 openstackgerrit sean mooney proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
04:47:57 openstackgerrit Alex Xu proposed openstack/nova master: placement: using the dict format for the allocation in claim_resources https://review.openstack.org/536083
04:48:28 openstackgerrit Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085
05:19:29 openstackgerrit Merged openstack/nova master: Set server status to ERROR if rebuild failed https://review.openstack.org/536268
05:37:17 openstackgerrit Alex Xu proposed openstack/nova master: placement: using the dict format for the allocation in claim_resources https://review.openstack.org/536083
05:37:18 openstackgerrit Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085
06:18:03 ameeda Morning :)
06:18:32 ameeda can you please check this code to detach volumes "http://paste.openstack.org/show/651488/" and let me know if that need to fix or anything else
07:10:47 alex_xu gibi: hi, I update the patch and addressed a upgrade case which pointed by Matt https://review.openstack.org/#/c/536083
07:42:59 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Do not normalize allocation_ratios https://review.openstack.org/532924
07:43:15 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Do not normalize allocation_ratios https://review.openstack.org/532924
07:46:25 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: provide VGPU_DISPLAY_HEAD inventory in compute node https://review.openstack.org/523342
07:54:16 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Do not normalize allocation_ratios https://review.openstack.org/532924
08:20:27 kashyap dmsimard: Hey, I'm fully occupied with something more urgent until 04-Feb, I'm afraid. But I've added that to my TODO queue.
08:52:39 gibi alex_xu: looking...
08:52:58 alex_xu gibi: thanks
08:59:45 ameeda please please please, can anyone approve my gerrit here ? https://review.openstack.org/#/c/526900/
09:23:20 mdbooth stephenfin: You were holding off on a +2 for melwitt ? https://review.openstack.org/#/c/523958/18
09:58:20 bauzas morning folks
09:58:28 maciejjozefczyk bauzas: hiho
09:58:29 bauzas my presence is a bit off, given I'm in a conference
09:58:47 bauzas just ping me directly if you need me
09:59:02 maciejjozefczyk bauzas: good to know cause I have something for you
10:08:18 openstackgerrit nalini proposed openstack/nova master: Modify show aggregate to display 'updated_at' value https://review.openstack.org/537334
10:11:34 Spazmotic morning
10:23:31 openstackgerrit Sylvain Bauza proposed openstack/nova master: Avoid suspending guest with attached vGPUs https://review.openstack.org/535693
10:53:06 Spazmotic Too full.. will die
10:53:11 Spazmotic or burp.. one of those things
11:21:40 openstackgerrit Alex Xu proposed openstack/nova master: placement: using the dict format for the allocation in claim_resources https://review.openstack.org/536083
11:21:40 openstackgerrit Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085
11:22:21 alex_xu gibi: stephenfin, ^ sorry for I missed that two unittests, just fix them
11:22:33 stephenfin (y)
11:24:24 gibi alex_xu: no worries, I was pulled into something internally so I have to go back to your patches
11:25:02 alex_xu gibi: no problem, thanks for your time
11:25:56 openstackgerrit Alex Xu proposed openstack/nova master: Fix nits in support traits on allocation candidates API https://review.openstack.org/537351
11:44:25 rgerganov cdent, the issue I had yesterday: https://review.openstack.org/#/c/533821/7/nova/scheduler/client/report.py
11:44:46 rgerganov cdent, you were right that the problem is in update_from_provider_tree :)
11:48:50 cdent rgerganov: ah, interesting, nice sleuthing. Was the trait failure because you were using a not-allowed trait?
11:49:02 rgerganov yes :)
11:49:23 rgerganov speaking of that, how do I add a new trait?
11:52:18 cdent rgerganov: are you wanting to establish a new official trait, or set a CUSTOM trait in the local deployment?
11:52:42 cdent if the latter: https://developer.openstack.org/api-ref/placement/#update-traits
11:52:58 cdent if the former: https://github.com/openstack/os-traits
11:53:01 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Update tests to set 'NUMACell.siblings' https://review.openstack.org/537362
11:53:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Don't filter out sibling sets with one core https://review.openstack.org/537361
11:53:02 openstackgerrit Stephen Finucane proposed openstack/nova master: Ensure emulator threads are always calculated https://review.openstack.org/537363
11:53:02 openstackgerrit Stephen Finucane proposed openstack/nova master: Always pass 'NUMACell.siblings' to _pack_instance_onto_cores' https://review.openstack.org/537364
11:53:14 stephenfin sahid: Fancy taking a look at those ^ ?
11:53:16 rgerganov cdent, thanks, I believe it is the latter
12:57:21 efried_back_wed rgerganov Haven't caught up fully; what news?
12:58:43 rgerganov efried, hi, I found why the local tree was missing the nested RP, see my comment here: https://review.openstack.org/#/c/533821/7/nova/scheduler/client/report.py
12:59:32 rgerganov efried, my driver was trying to create an incorrect trait and update_from_provider_tree silently removed the nested RP
13:02:42 efried rgerganov Hum, yeah, that code will remove the entire sub-branch from the tree (but only remove its root from the association cache - yet another leak in that cache, though it's resolved a couple patches up the series).
13:03:45 efried rgerganov But that's *supposed* to be okay, because we should wind up restoring it next time we refresh_associations
13:04:47 rgerganov efried, this is not the case, we don't restore the nested RP and when we try to recreate it, we fail
13:05:34 efried rgerganov So the bug we need to track down is *why* we don't restore it in the cache.
13:05:48 efried rgerganov Oh, is all of this happening in a single invocation of update_from_provider_tree ?
13:06:00 rgerganov efried, yes
13:06:13 efried Aha
13:06:26 rgerganov efried, in one invocation we create the nested RP in placement and mess up the local tree
13:06:46 rgerganov and on the invocation we try to create it again and then fail
13:06:53 rgerganov and on the next invocation we try to create it again and then fail
13:07:06 efried Wellll
13:07:31 efried I would have expected us to refresh at some point before that second invocation
13:07:43 rgerganov that's why I was thinking that https://review.openstack.org/#/c/536902 is a good idea
13:08:09 rgerganov even if the root node exists, refresh the tree before calling the virt driver for changes
13:09:29 efried rgerganov As written, that change will *not* refresh the tree, though
13:10:17 efried If you want a simple try-out, change to force=True
13:11:10 rgerganov efried, this loop doesn't run at all: for u in self._provider_tree.get_provider_uuids(uuid):
13:11:25 rgerganov because there are no children in the local tree
13:11:39 efried rgerganov That call should at least return `uuid` itself.
13:11:48 rgerganov ah, correct
13:11:53 rgerganov my bad
13:12:00 efried And then the _refresh_associations call should re-grab all the tree-associated providers

Earlier   Later