Earlier  
Posted Nick Remark
#openstack-nova - 2018-09-17
13:37:00 bauzas do we started runways ?
13:37:51 gibi mriedem: true, then I remove the patches from the queue and bring up the bp on the Thursday meeting for approval
13:37:59 bauzas mriedem: FWIW https://review.openstack.org/#/c/527728/22/nova/tests/functional/test_servers.py should cover the boot case
13:42:52 kashyap Dear folks, I can't tell _why_ the 'neutron-grenade' job is failing here: https://review.openstack.org/#/c/602592/
13:43:18 kashyap I'm looking at the "ara-report", but I can't find a log file.
13:45:32 bauzas kashyap: http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/job-output.txt.gz
13:45:54 kashyap bauzas: Ah, let me look
13:46:10 bauzas mmm, actually
13:46:24 mriedem http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/logs/devstack-gate-setup-workspace-old.txt
13:47:30 mriedem i've seen that in a couple of other jobs but don't know what's causing it
13:47:59 kashyap http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/job-output.txt.gz#_2018-09-17_10_12_38_647361
13:48:03 bauzas http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/job-output.txt.gz#_2018-09-17_11_27_33_877513
13:48:10 kashyap 2018-09-17 10:12:38.647361 | primary | smoke installed: ----------------------------------------,Error when trying to get requirement for VCS system Command "git config --get-regexp remote\..*\.url" failed with error code 1 in /opt/stack/new/tempest, falling back to uneditable format,Could not determine repository location of /opt/stack
13:48:26 mriedem that's common
13:48:32 mriedem the failure is in http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/logs/devstack-gate-setup-workspace-old.txt
13:48:53 mriedem http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22error%3A%20The%20following%20untracked%20working%20tree%20files%20would%20be%20overwritten%20by%20checkout%3A%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d
13:49:35 kashyap Ah, it's the untracked files
13:50:06 bauzas http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/logs/devstack-gate-setup-workspace-old.txt
13:50:13 bauzas dammit, I'm burned
13:50:32 kashyap bauzas: Thought you'll be refreshed after the PTG?
13:50:41 kashyap Ah, _that_ kind of a burn
13:50:42 bauzas I am
13:50:53 bauzas nah, burned because I'm too slow
13:53:47 mriedem someone want to backport this to stable/rocky? https://review.openstack.org/#/c/546920/
13:54:03 mriedem someone not on the stable core team...
13:59:32 bauzas stephenfin: ^ ?
14:00:10 gibi mriedem: elod is still in Denver but he is back on Wednesday so if nobody take this backport then I can encourage them to take it
14:00:30 stephenfin bauzas, mriedem: Sure
14:00:53 bauzas do we have a scheduler meeting ?
14:01:28 mriedem yes
14:02:33 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: Fix soft deleting vm fails after "nova resize" vm https://review.openstack.org/603140
14:02:48 stephenfin mriedem, bauzas: as requested ^
14:03:22 bauzas ta
14:09:35 mriedem dansmith: if you're ok with what i've proposed for updates in https://review.openstack.org/#/c/579520/ (the volume type spec) then i'll just make those updates myself
14:09:51 dansmith looking
14:11:40 dansmith mriedem: the latest two small things? sure, but I should re-review the whole thing
14:11:55 dansmith if you do the update I'll do that once the caffeine starts hitting my bloodstream
14:12:23 mriedem ok, i might not get to that until after the placement meeting
14:26:57 openstackgerrit Vlad Gusev proposed openstack/nova stable/pike: [placement] Retry allocation writes server side https://review.openstack.org/590745
14:35:12 stephenfin bauzas: Does this need a reno, actually? https://review.openstack.org/#/c/603140/
14:42:50 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Add support specify volume type when boot instance https://review.openstack.org/579520
14:43:03 mriedem dansmith: fleshed out the proposed change section and also the specific test scenarios i'd want to see at a minimum ^
14:43:19 dansmith ack
14:48:31 mriedem bauzas: as far as i can tell, the only change i need to make in the libvirt vgpu reshaper patch is the naming convention we discussed at the ptg correct? https://review.openstack.org/#/c/599208/
14:48:50 mriedem <HYPERVISOR_HOSTNAME>_VGPU_<TYPE_NAME>
14:50:19 dansmith mriedem: yauntmeta fix up my grammar nits, or just stack something on top?
14:51:19 mriedem i'll fix them quick
14:52:54 dansmith aight
14:58:07 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Add support specify volume type when boot instance https://review.openstack.org/579520
14:58:10 mriedem dansmith: done
15:00:15 dansmith mriedem: also done
15:01:30 dansmith mriedem: heading to the twitters
15:02:21 dansmith https://twitter.com/get_offmylawn/status/1041703857460412419
15:02:39 mriedem heh
15:02:41 mriedem oh chet
15:10:22 bauzas mriedem: sorry missed your ping
15:10:31 bauzas mriedem: yup, I guess, lemme check notes on this
15:11:07 bauzas mriedem: there was an upgrade concern, but I feel this should only be written for the multiple types support
15:11:15 bauzas I feel => I'm sure
15:11:56 mriedem yes multiple gpu types can't be supported until the reshape is done
15:12:10 bauzas mriedem: I reproposed the spec FWIW
15:12:28 bauzas mriedem: https://review.openstack.org/#/c/602474/
15:13:06 bauzas jaypipes: I know you had concerns on https://review.openstack.org/#/c/602474/ but you eventually got a +2
15:13:31 bauzas jaypipes: my take on this is that I'm fine if someone writes something using YAML
15:13:43 bauzas jaypipes: I'll then just use it for multiple VGPU types
15:13:56 bauzas (I mean, speaking of the inventory)
15:14:38 bauzas if by Stein, nobody proposes it, then I could consider proposing myself for Train
15:14:54 mriedem resource provider name looks like it's just a string in the api schema,
15:14:55 bauzas for the YAML inventory recording
15:15:02 mriedem wonder if we should replace whitespaces with underscores just to be safe
15:15:18 mriedem or remove whitespaces
15:15:23 bauzas mriedem: I think we agreed on underscores, nope ?
15:15:36 mriedem the etherpad says <HYPERVISOR_HOSTNAME>_VGPU_<TYPE_NAME>
15:15:43 mriedem the conf docs show:
15:15:44 mriedem <HYPERVISOR_HOSTNAME>_VGPU_<TYPE_NAME>
15:15:46 mriedem oops
15:15:50 mriedem enabled_vgpu_types = GRID K100,Intel GVT-g,MxGPU.2,nvidia-11
15:16:02 bauzas hah, that
15:16:08 mriedem so you could have foo.bar.something_VGPU_GRID K100
15:16:34 bauzas for libvirt, there is (AFAIK) only "vendorshit-number"
15:16:52 bauzas the above example was for Xen
15:17:09 bauzas but yeah, we could try to replace
15:18:30 bauzas https://libvirt.org/drvnodedev.html#MDEVCap
15:18:37 bauzas "The element has one attribute id which holds an official vendor-supplied identifier for the type. Since 3.4.0"
15:18:56 bauzas I don't know what "official vendor-supplied ID" means :)
15:19:28 bauzas "official" and "vendor-supplied" seem opposite to me
15:23:14 mriedem i'm just going to pass the configured type through
15:23:31 bauzas mriedem: uh, I'm being told that this string is just passed thru by the vendor driver to the kernel, period
15:23:47 bauzas mriedem: so there is litterally no formatting
15:23:48 mriedem and i'm passing it through to placement
15:23:57 bauzas and passing to operators
15:24:05 bauzas that's awesome
15:24:11 mriedem well the operator is the one that configures nova with the type
15:24:12 mriedem so yeah
15:26:08 jaypipes bauzas: done.
15:26:55 jaypipes bauzas: by "done", I mean I've re-reviewed the Stein vGPU types spec.
15:28:24 bauzas jaypipes: gotcha, and thanks
15:28:34 bauzas jaypipes: yeah, I think your comment is all good with me
15:30:08 bauzas jaypipes: tbc, I think I can discover certain things like framebuffer size and what's described in https://docs.nvidia.com/grid/6.0/grid-vgpu-user-guide/index.html#vgpu-types-tesla-m60 and others
15:30:26 bauzas jaypipes: but GPU capabilities will have to be described by operators directly

Earlier   Later